Skip to content

docs(patterns): operationalize auto-review runbook + verify-before-claiming + node-signature convention - #1795

Merged
POWERFULMOVES merged 1 commit into
mainfrom
ops/auto-review-runbook
Jun 11, 2026
Merged

POWERFULMOVES merged 1 commit into
mainfrom
ops/auto-review-runbook

Conversation

@POWERFULMOVES

@POWERFULMOVES POWERFULMOVES commented Jun 11, 2026

Copy link
Copy Markdown
Owner

Operationalizes this session's review-infra learnings into .claude/PATTERNS.md (the loaded Known-Roads/debug-recipes doc) so they're enforced in operations, not just memory — per the request to "ensure not just in memory but in operations."

Adds to PR Review & Merge Workflow:

  • Verify-before-claiming — never assert "X fixed" without running the real qualifying case; use the verifier subagent. (Earned: a version bump was asserted to fix the auto-review outage twice before a real PR proved it didn't.)
  • claude-code-review.yml failure-signature triage table — the 3 signatures + fixes: 4s oven-sh/setup-bun allowlist (version bump does NOT fix it), benign 40s Workflow validation failed 401 (PR edits the workflow → ignore, clears on main), bot-trigger default-deny (allowed_bots).
  • Node-signature convention — primary + 1M-context mirror can both sign 4090-CLAUDE; union-merge racing claims, disambiguate the signature so claim-collision-agent can tell lanes apart.

Pure docs; no behaviour change.

Note (and a live demonstration of the runbook itself): this PR touches only .claude/PATTERNS.md, which is not in the widened review path filter (#1787 added .claude/agents|skills/**, not docs), so claude-review will not fire here — by design, doc edits aren't code-reviewed. The pipeline was already verified live on #1785 (a clean 4m27s claude-review run on an agents PR).

🤖 Generated with Claude Code

…-claiming + node-signature convention

Bakes this session's review-infra learnings into the loaded dev-patterns doc (not just memory): (1) claude-code-review.yml failure-signature triage table — 4s allowlist (oven-sh/setup-bun, NOT a version-bump fix), benign 40s workflow-self-validation 401, bot-trigger default-deny needing allowed_bots; (2) verify-a-claimed-fix-before-asserting (use the verifier subagent); (3) union-merge + disambiguate same-node-identity claims (primary vs 1M-context mirror both signing 4090-CLAUDE).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

@POWERFULMOVES, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 59 minutes and 5 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more credits in the billing tab to continue.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: f9c1176c-f644-428b-97a4-165dafdd1922

📥 Commits

Reviewing files that changed from the base of the PR and between 528c3b7 and 3b629bd.

📒 Files selected for processing (1)
  • .claude/PATTERNS.md
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ops/auto-review-runbook

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@POWERFULMOVES
POWERFULMOVES merged commit 7bd9c21 into main Jun 11, 2026
11 of 14 checks passed
@POWERFULMOVES
POWERFULMOVES deleted the ops/auto-review-runbook branch June 11, 2026 17:27
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