Conversation
Validates Claude Code plans against the codebase before implementation. Codex searches the repo to check completeness, safety, exclusion correctness, ordering, and edge cases. Picks the most recently modified plan from ~/.claude/plans/ by default, or accepts --plan <name>. Supports --background, --wait, --model, and free-form focus text.
Update command list test to include plan-review.md, add assertion test for the command markdown, and document the new command in README.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8d366a258e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Validate that the resolved plan path stays inside ~/.claude/plans/ before reading. Rejects inputs like --plan ../../etc/passwd.
|
Let's merge it guys, need to this feature |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: fa160af61c
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| } | ||
|
|
||
| async function executePlanReviewRun(request) { | ||
| ensureCodexReady(request.cwd); |
There was a problem hiding this comment.
executePlanReviewRun calls ensureCodexReady, but that function is not defined or imported anywhere in the repository, so any plan-review run that reaches this code throws a ReferenceError immediately (e.g., with a valid ~/.claude/plans/*.md). This makes the new command unusable in normal operation; switch this to an existing readiness guard (such as ensureCodexAvailable, plus any intended repo check) so execution can continue.
Useful? React with 👍 / 👎.
2 participants
Summary
/codex:plan-review— validates a Claude Code plan against the codebase before implementation~/.claude/plans/by default, or accepts--plan <name>--background,--wait,--model, and free-form focus textMotivation
Common workflow: write a detailed plan in Claude Code's plan mode, then want a second opinion from Codex before executing. Currently requires manually constructing a
codex execcommand with the plan content piped in. This automates that into a single slash command.Changes
plugins/codex/commands/plan-review.md— command definitionplugins/codex/prompts/plan-review.md— prompt template with 5 review dimensionsplugins/codex/scripts/codex-companion.mjs—plan-reviewsubcommand (+135 lines)tests/commands.test.mjs— test for new command, updated command list assertionREADME.md— documentation and typical flows