RomanNum3ral
/
gstack
mirror of https://github.com/garrytan/gstack.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

feat: add codex plan review option to /plan-eng-review 

After scope challenge (Step 0), offer to have Codex independently
review the plan with a brutally honest tech reviewer persona.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Garry Tan 2026-03-18 22:01:54 -07:00
parent 00fd28ba83
commit 22b75ff402
No known key found for this signature in database
GPG Key ID: C1F69E85C74EFE1D
2 changed files with 16 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
plan-eng-review/SKILL.md
View File

@ -222,11 +222,17 @@ A) Yes — let Codex critique the plan independently
B) No — proceed with the Claude review only B) No — proceed with the Claude review only
``` ```
If the user chooses A: read the plan file and run Codex with the plan review persona: If the user chooses A: read the plan file content, then pass it into the Codex prompt so it has the actual plan to critique:
```bash ```bash
codex exec "You are a brutally honest technical reviewer. Review this plan for: logical gaps and unstated assumptions, missing error handling or edge cases, overcomplexity (is there a simpler approach?), feasibility risks (what could go wrong?), and missing dependencies or sequencing issues. Be direct. Be terse. No compliments. Just the problems." -s read-only PLAN_CONTENT=$(cat <plan-file-path>)
codex exec "You are a brutally honest technical reviewer. Review this plan for: logical gaps and unstated assumptions, missing error handling or edge cases, overcomplexity (is there a simpler approach?), feasibility risks (what could go wrong?), and missing dependencies or sequencing issues. Be direct. Be terse. No compliments. Just the problems.
THE PLAN:
$PLAN_CONTENT" -s read-only -c 'model_reasoning_effort="high"' --enable web_search_cached
``` ```
Replace `<plan-file-path>` with the actual path to the plan file detected earlier.
Present the full output under a `CODEX SAYS (plan review):` header. Note any concerns Present the full output under a `CODEX SAYS (plan review):` header. Note any concerns
that should inform the subsequent engineering review sections. that should inform the subsequent engineering review sections.

10
plan-eng-review/SKILL.md.tmpl
View File

@ -93,11 +93,17 @@ A) Yes — let Codex critique the plan independently
B) No — proceed with the Claude review only B) No — proceed with the Claude review only
``` ```
If the user chooses A: read the plan file and run Codex with the plan review persona: If the user chooses A: read the plan file content, then pass it into the Codex prompt so it has the actual plan to critique:
```bash ```bash
codex exec "You are a brutally honest technical reviewer. Review this plan for: logical gaps and unstated assumptions, missing error handling or edge cases, overcomplexity (is there a simpler approach?), feasibility risks (what could go wrong?), and missing dependencies or sequencing issues. Be direct. Be terse. No compliments. Just the problems." -s read-only PLAN_CONTENT=$(cat <plan-file-path>)
codex exec "You are a brutally honest technical reviewer. Review this plan for: logical gaps and unstated assumptions, missing error handling or edge cases, overcomplexity (is there a simpler approach?), feasibility risks (what could go wrong?), and missing dependencies or sequencing issues. Be direct. Be terse. No compliments. Just the problems.
THE PLAN:
$PLAN_CONTENT" -s read-only -c 'model_reasoning_effort="high"' --enable web_search_cached
``` ```
Replace `<plan-file-path>` with the actual path to the plan file detected earlier.
Present the full output under a `CODEX SAYS (plan review):` header. Note any concerns Present the full output under a `CODEX SAYS (plan review):` header. Note any concerns
that should inform the subsequent engineering review sections. that should inform the subsequent engineering review sections.