Rename codex-rs workspace and codex-acp package#443
Merged
Conversation
- Update AGENTS.md paths and directory references from codex-rs/ to nori-rs/ so agent instructions point at the real workspace. - Delete nori-rs/.github/workflows/cargo-audit.yml. GitHub Actions only discovers workflows under the top-level .github/workflows/, so this nested copy (inherited from upstream OpenAI codex) never ran. The cargo-deny workflow at the repo root already covers advisory checks. 🤖 Generated with [Nori](https://noriagentic.com) Co-Authored-By: Nori <contact@tilework.tech>
Minor leftovers the rename commit did not catch: - scripts/debug-codex.sh ran `--bin codex`, but the crate only exposes the `nori` bin; the script was broken both before and after the rename. Switch to `--bin nori` and update the helper variable name. - .gitignore still listed `/codex-rs/tui/target/`; point it at the new directory so the cached tui target is actually ignored. - nori-cli/package.json had repository.directory="codex-cli"; the npm registry "source" link pointed at a path that no longer exists. - nori-rs/acp/Cargo.toml depended on codex-protocol via a direct path instead of `workspace = true`; align with every other workspace dep. - nori-rs/acp/docs.md referenced `codex_tui/acp` in the release env-filter example; the tracing target is now `nori_tui/acp`. 🤖 Generated with [Nori](https://noriagentic.com) Co-Authored-By: Nori <contact@tilework.tech>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
🤖 Generated with Nori
codex-rs/tonori-rs/codex-acp/codex_acptonori-acp/nori_acpTest Plan
cargo test -p nori-acpcargo test -p nori-tuicargo test -p nori-clicargo testcargo build --bin nori && cargo test -p tui-pty-e2e./target/debug/nori --agent elizacp --skip-trust-directoryjust fmtjust fixShare Nori with your team: https://www.npmjs.com/package/nori-skillsets