docs(process): record direct-main docs exception#340
Conversation
📝 WalkthroughWalkthroughThis PR establishes a governance process for controlled exceptions to direct-main pushes and documents the first such exception. A new exception log file defines the policy and records the specific 2026-05-14 docs-only checkpoint exception with commits, changed files, and validation details. The CHANGELOG documents this exception entry. ChangesException Process Governance
🎯 1 (Trivial) | ⏱️ ~3 minutes
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/procedures/DIRECT-MAIN-EXCEPTION-LOG.md`:
- Around line 73-82: Update the future-rule checklist in the
DIRECT-MAIN-EXCEPTION-LOG document to require an explicit "authorization
evidence" bullet: record the identity of the approver (who authorized) and a
link or citation to where that authorization is captured
(issue/thread/comment/PR number), and make this a mandatory item alongside the
existing items (state the exception, run docs validation, record commits/changed
files, confirm no source/runtime code touched, add process note). Ensure the new
bullet is phrased consistently with the checklist tone and placed with the other
pre-push requirements so provenance is auditable.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: 2ed95ea6-d281-4e3b-b578-6692b989ef3f
📒 Files selected for processing (2)
CHANGELOG.mddocs/procedures/DIRECT-MAIN-EXCEPTION-LOG.md
|
| Issue | Severity | Addressed by | Regression coverage | Outcome |\n| :--- | :--- | :--- | :--- | :--- |\n| Direct-main exception checklist did not require explicit authorizer identity and evidence citation. | P1 | |
Records the docs-only direct-main exception for the built-in Echo graph data model commits and pins the future rule: prefer PRs unless emergency or explicitly authorized docs-only fast path.
Summary by CodeRabbit