Skip to content

review-loop v2.2.0: Independent Validation Round

Choose a tag to compare

View all tags
@Whisker17 Whisker17 released this 23 Mar 06:52
· 4 commits to main since this release
v2.2.0
7fab8e1

What's New

Independent Validation Round

After the existing Claude-Codex review loop converges (or exhausts its 5 rounds), a fresh Codex instance with zero shared review history re-examines the output using blind-spot-focused prompts. If issues are found, they're fed back into the regular loop for fix + re-validation (max 2 cycles per stage).

This addresses the false convergence problem: Claude and Codex agree there are no remaining issues, but independent review still finds entirely new categories of problems — shared blind spots from operating under the same prompt framework.

New Modes

Mode Role Purpose
independent-design-review Codex (READ-ONLY) Blind-spot review of design docs
independent-code-review Codex (READ-ONLY) Blind-spot review of code changes
validation-design-fix Codex (READ-ONLY) Review design fixes after validation
validation-fix Codex (implementer) Fix code issues found by validation

Key Details

  • Context isolation: prompt-assembly layer injects zero review history; prompt-instruction layer explicitly prohibits reading specs/reviews/ and .claude/
  • Composite round tokens: c1/c2 for validation cycles, c1f1/c2f2 for fix rounds — no artifact naming collisions
  • Failure handling: retry once on timeout/failure, skip + log on double failure, surface in final output (never falsely claim validation passed)
  • Ordering: validation runs before verify round (verify remains the terminal pass)
  • Role separation: AGENTS.md distinguishes independent reviewers, design fix reviewers, and code fixers with distinct constraints
  • 20 new tests (37 total), all passing

File Changes

  • 4 new prompt templates in review-loop/prompts/
  • Modified: common.sh (4 new modes, composite round validation, append_diff_section with temp git index), review-loop.md (validation + verify + output flows), AGENTS.md (role separation), tests/review-loop.test.sh
  • No changes to run-review-bg.sh, check-review.sh, kill-review.sh, hooks, or state file schema

See specs/design.md for the full implementation design.

Assets 2
Loading