review-code

/volley:review-code

Claude reads what Codex built and judges it.

Steps for Claude

1. Verify .volley/ is initialized and the lock allows Claude to act.

   [ -d .volley ] || { echo "ERROR: .volley/ not found. Run /volley:setup first." >&2; exit 1; }
   [ -f .volley/STATE ] || { echo "ERROR: .volley/STATE not found. Run /volley:setup first." >&2; exit 1; }
   . "${CLAUDE_PLUGIN_ROOT}/scripts/lib.sh"
   volley_state_assert_active .volley/STATE claude || exit 1
   

2. Resolve the diff to review.

   • If user passed a range argument (e.g. HEAD~3..HEAD), use that.
   • Otherwise, default to uncommitted changes:

     git diff HEAD
     

   • If diff is empty, tell user "No changes to review. Did Codex finish?" and stop.

3. Locate the plan and HANDOFF. Same logic as /volley:review-plan for finding the latest PLAN.md. Read the plan and .volley/HANDOFF.md content.

4. Read the diff into context. Use Bash to capture the diff and Read tools as needed for full context on changed files.

5. Review with these prompts in mind:

   • Does each change implement a specific PLAN task or acceptance criterion? (Cite which.)
   • Are there changes that go beyond the plan's scope?
   • Are there gaps - acceptance criteria not addressed?
   • Are the tests covering the new code?
   • Any obvious correctness, safety, or style issues for this codebase?

6. Write .volley/CODE-REVIEW.md with this structure:

   # Claude Code Review
   
   **Diff range:** <range used>
   **Reviewed:** <ISO timestamp>
   **Plan:** <plan path>
   
   ## Verdict
   SHIP | FIX | DISCUSS
   
   ## Coverage of acceptance criteria
   - Criterion 1: ✓ addressed in <file:line>
   - Criterion 2: ✗ not addressed
   - ...
   
   ## Coverage of plan tasks
   - Task N: ✓
   - Task M: partial (missing X)
   
   ## Issues found
   - (file:line — description)
   
   ## Out-of-plan changes
   - (file:line — what was changed beyond the plan, with judgement)
   
   ## Suggested next moves
   - (concrete actions)