Skip to content

docs: add the final-HEAD review gate to CLAUDE.md - #31

Merged
cristim merged 3 commits into
mainfrom
docs/review-gate
Sep 28, 2026
Merged

cristim merged 3 commits into
mainfrom
docs/review-gate

Conversation

@cristim

@cristim cristim commented Sep 28, 2026 •

Copy link
Copy Markdown
Member

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

  • Also rewords the CodeRabbit loop heading and opening so it applies only when CodeRabbit is the chosen review path or the independent-review condition is not met.

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).

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
@cristim cristim added impact/internal Team-internal only triaged Item has been triaged priority/p3 Polish / idea / may never ship severity/low Minor harm urgency/eventually No deadline effort/xs Trivial / one-liner type/docs Documentation labels Sep 28, 2026
@coderabbitai

coderabbitai Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

  • Run on-demand review

This review includes 1 billable file and costs up to $0.25.

  • Ask an admin to make reviews automatic

Open in CodeRabbit

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.

Check out review usage here.

View limit details

Limit 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.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Repository: LeanerCloud/cloud-commitments-mcp/.coderabbit.yaml

Review profile: CHILL

Plan: Essentials

Run ID: 362ed051-3f18-4795-8d02-8e74857cd1df

📥 Commits

Reviewing files that changed from the base of the PR and between 4e40f97 and dd7fbca.

📒 Files selected for processing (1)
  • CLAUDE.md

Comment @coderabbitai help to get the list of available commands.

@cristim

cristim commented Sep 28, 2026

Copy link
Copy Markdown
Member Author

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.

@cristim
cristim merged commit 2d9a95f into main Sep 28, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

effort/xs Trivial / one-liner impact/internal Team-internal only priority/p3 Polish / idea / may never ship severity/low Minor harm triaged Item has been triaged type/docs Documentation urgency/eventually No deadline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: add the final-HEAD review gate to CLAUDE.md

1 participant