docs: apply owner review gate override

[cristim]

The existing review gate pins one model and rejects realistic local verification. Update only that section to reflect the owner's authorization for capable independent review and actual-path verification with honest evidence labels and recorded real-account coverage gaps.

Exact-head review, resolved actionable findings, fresh CI, clean mergeability, unchanged reviewed head, and normal protections remain required. The quota-only CodeRabbit waiver retains its exact evidence text and retrospective review requirement.

Validation: independent plan and two implementation reviews, pinned Markdown hook, fresh build, and normal commit hooks passed. Only CLAUDE.md changed; independent byte checks confirmed all text outside Review gate is unchanged. No runtime code changed. Runtime suite attempts encountered existing environment limits, recorded separately; they are not claimed green. No real-account acceptance is claimed.

This owner-requested documentation PR closes no issue, so closing-issue label mirroring does not apply.

Summary by CodeRabbit

• Process Updates
  • Clarified that an independent review from a capable model can satisfy the review gate if it identifies the exact current commit.
  • Required fresh green CI, clean mergeability, and an unchanged reviewed commit before merging. New commits or rebases restart the gate.
  • Clarified evidence expectations for local verification, including honest coverage notes, applicable regression evidence, a fresh build, and relevant tests.
  • Allowed merges when CodeRabbit is blocked by quota or throttling, provided the waiver and retrospective review are recorded and actionable findings are resolved.
  • Added branch protection and coordination requirements for merging.

[cristim]

Independent Codex review of exact current head 6112b4fe3bff35cdf28abeb227279139218afad0: no actionable findings. The reviewer independently inspected the complete committed diff and compared it with the owner's instructions. Only CLAUDE.md changed. Bytes outside Review gate are identical to the parent, the approved wording matches, and whitespace checks pass.

Local documentation verification: pinned Markdown hook, fresh make build, and normal commit hooks passed. Two implementation reviews and two staged reviews were clean. The affected path is the written review gate, verified directly against the owner contract; no runtime changes or real-account acceptance are claimed.

Runtime suite attempts are not claimed green. The rerun failed compiling or vetting imports because existing shared Go build cache files were absent. Receipt: /private/tmp/herdr-codex-takeover-20261007/guidance-mcp-tests.log. No cache or source files were deleted or changed to hide the failure. Fresh CI and clean mergeability remain mandatory before normal merge.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Repository: LeanerCloud/cloud-commitments-mcp/.coderabbit.yaml
• Review profile: CHILL
• Plan: Essentials
• Run ID: a43655d3-28a0-4954-bfcb-98cd51be55a6

📥 Commits

Reviewing files that changed from the base of the PR and between 4f84920 and 6112b4f.

📒 Files selected for processing (1)

• CLAUDE.md

Included review availability: This review used your included allowance. 0 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

---

📝 Walkthrough

Walkthrough

CLAUDE.md updates independent review and local verification requirements. It also revises the CodeRabbit quota or throttling exception and adds merge coordination requirements.

Changes

Review and merge gates

Layer / File(s)	Summary
Review and verification requirements CLAUDE.md	The review gate accepts an available capable model that names the exact current head SHA. It adds fresh green CI, clean mergeability, an unchanged reviewed head, and specific local verification evidence requirements.
Merge conditions and coordination CLAUDE.md	The quota or throttling exception permits merging when the waiver and retrospective review are recorded and available actionable findings are resolved. Merge instructions add branch protection and coordination requirements. A new commit or rebase restarts the gate.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 6112b

This documentation change preserves the stated review and merge protections; no concrete merge-blocking risk is established.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately identifies a documentation change that applies an owner-authorized review gate override. It is concise and specific to the main change.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

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

[cristim]

Gate verdict (mcp gate-1) at 6112b4f: PASS.

• Only CLAUDE.md changed (Review gate section). Wording matches the owner override (capable independent reviewer incl. Codex on exact head, mocks/fixtures/local integration acceptable with honest labeling, no exact-model pin, no mandatory real-account acceptance, CR quota waiver with literal evidence string and retrospective tracking). No weaker rule than the platform gate: head-unchanged, clean mergeability, all actionables, fresh green CI, evidence on PR, no bypass all kept. No secrets, no links added, no em-dashes.
• Independent Codex review of this head is posted above (no findings). CodeRabbit: no actionable comments.
• Local evidence (real tests, docs-only change): git archive of this SHA, GOTOOLCHAIN=go1.26.9 go test ./... exit 0 (466 passed, 3 packages). CI: all 12 checks green incl. pre-commit (markdown hooks); mergeStateStatus CLEAN.
  No real-account verification applicable.