docs: add the final-HEAD review gate to CLAUDE.md - #31
Conversation
Moves the CUDly review gate here from the global dotclaude pr-lifecycle skill, refreshed to the current policy: Opus 5.5 adversarial review, CodeRabbit optional under exact-revision verification, green CI, macOS local plus Linux CI, merge only at the reviewed SHA. Closes #30
|
Warning Review limit reached
This review includes 1 billable file and costs up to $0.25.
Reviews can continue after your included limit without a manual trigger. An admin must approve usage-based billing. Or wait 29 minutes for your next included review. View limit detailsLimit details: You’ve used the included review currently available. Your 60 included PR review attempts over the past 7 days set your current allowance at 1 review per hour. Review configuration: ⚙️ Run configurationConfiguration used: Repository: LeanerCloud/cloud-commitments-mcp/.coderabbit.yaml Review profile: CHILL Plan: Essentials Run ID: 📒 Files selected for processing (1)
Comment |
|
Independent review (Opus 5.5) found two gaps (no-bypass rule; an unconditional CR-loop delegation paragraph); both are fixed at this head. Orchestrator verified the final text: the Review gate section is byte-identical across the 4 repos, has no em-dashes, and the only Fable mention says 'never Fable'. Docs-only; pre-commit passed. |
Adds a short "## Review gate" section to CLAUDE.md, moved from dotclaude's global pr-lifecycle skill (LeanerCloud/dotclaude#111) and refreshed to the current policy: independent adversarial review on Opus 5.5 (claude-opus-5-5, never Fable); CodeRabbit optional when exact-revision local verification plus a thorough independent review cover the final SHA; green CI; local macOS verification plus Linux CI, no Windows scope; merge only at the reviewed SHA.
Closes #30
The gate also requires merging normally (no
--admin, no--no-verify), all actionable findings from any reviewer resolved, the verdict, SHA and evidence recorded on the PR, fixture or mock evidence labeled, and it never authorizes real purchases, deploys or--yes; the CodeRabbit delegation and label-mirror rules no longer assume CR (carried over from dotclaude#114).