Skip to content

docs(codex): avoid prefixing shell builtins - #3349

Open
nightcityblade wants to merge 1 commit into
rtk-ai:developfrom
nightcityblade:fix/issue-3331
Open

docs(codex): avoid prefixing shell builtins#3349
nightcityblade wants to merge 1 commit into
rtk-ai:developfrom
nightcityblade:fix/issue-3331

Conversation

@nightcityblade

Copy link
Copy Markdown

Summary

  • Clarify that Codex should prefix executable commands, not shell builtins such as cd.
  • Document how rtk rewrite places RTK within a compound cd && uv run pytest command.
  • Add a regression test for that exact rewrite path and explain how to verify RTK availability.

Fixes #3331.

Test plan

  • cargo fmt --all -- --check && cargo clippy --all-targets -- -D warnings && cargo test
    (2,512 unit tests and 64 integration tests passed; 8 unit tests ignored)
  • Manual testing: ./target/debug/rtk rewrite 'cd backend && uv run pytest tests/'
    printed cd backend && uv run rtk pytest tests/

Important: All PRs must target the develop branch (not master).
See CONTRIBUTING.md for details.

@CLAassistant

CLAassistant commented Aug 1, 2026

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Codex generates invalid rtk cd ... commands and then misdiagnoses RTK as unavailable

2 participants