Skip to content

feat(review): Tighten review workflow with 4-tier severity and 5-cycl…#102

Merged
mhersson merged 1 commit into
mainfrom
ctxmax-515/tighten-the-review-workflow
May 18, 2026
Merged

feat(review): Tighten review workflow with 4-tier severity and 5-cycl…#102
mhersson merged 1 commit into
mainfrom
ctxmax-515/tighten-the-review-workflow

Conversation

@mhersson
Copy link
Copy Markdown
Owner

…e halt

  • Add production-ready gate framing to review-task.md
  • Split specialist severity into Critical / Important / Minor / Nits; only Nits-only outcomes approve_with_notes, anything else forces revise
  • Missing or malformed specialist report forces revise (rule kept once, in Step 3)
  • Bump orchestrator halt threshold from 3 to 5 review cycles in create-plan.md and run-autonomous.md
  • Raise server-side review-attempts cap from 5 to 7 to preserve the defense-in-depth gap above the orchestrator
  • Rename UI MAX_REVIEW_ATTEMPTS into REVIEW_ATTEMPTS_WARN_THRESHOLD (=4) and REVIEW_ATTEMPTS_HALT (=5); display "max 5" to match the halt value
  • Add TestReviewTaskSkill_StrictProductionReadyBar and TestOrchestratorHaltThreshold_FiveCycles regression guards

…e halt

- Add production-ready gate framing to review-task.md
- Split specialist severity into Critical / Important / Minor / Nits;
  only Nits-only outcomes approve_with_notes, anything else forces revise
- Missing or malformed specialist report forces revise (rule kept once,
  in Step 3)
- Bump orchestrator halt threshold from 3 to 5 review cycles in
  create-plan.md and run-autonomous.md
- Raise server-side review-attempts cap from 5 to 7 to preserve the
  defense-in-depth gap above the orchestrator
- Rename UI MAX_REVIEW_ATTEMPTS into REVIEW_ATTEMPTS_WARN_THRESHOLD (=4)
  and REVIEW_ATTEMPTS_HALT (=5); display "max 5" to match the halt value
- Add TestReviewTaskSkill_StrictProductionReadyBar and
  TestOrchestratorHaltThreshold_FiveCycles regression guards
@mhersson mhersson merged commit b5c2697 into main May 18, 2026
7 checks passed
@mhersson mhersson deleted the ctxmax-515/tighten-the-review-workflow branch May 18, 2026 11:45
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.

1 participant