fix(governance): restack independent approval audit after nanoid - #90
fix(governance): restack independent approval audit after nanoid#90seonghobae wants to merge 3 commits into
Conversation
|
Warning Review limit reached
Next review available in: 39 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
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 |
Purpose
Clean protected-main successor to superseded #87 after #76 integrated. This PR starts from current protected
mainand replays only #87's unique governance/Security-Scan guidance delta; predecessor and sibling checks/reviews do not transfer.Test-first proof
Exact RED head
f0ee29936cc3f46f38c04b5f32992396e4b543faran application CI and failed exactly two intended contracts intest/main-governance-audit.test.ts:required_approving_review_countwas zero;AGENTS.mdstill claimed central Security Scan ran on every stacked PR and documented a staleCRITICAL/HIGHthreshold.The narrow GREEN implementation then changed only:
test/main-governance-audit.test.ts;scripts/lib/main-governance-audit.mjs;AGENTS.md.Current exact head:
4ee417372c7a7e084767b50692c376bf03ff4b0a.Implemented boundary
AGENTS.mdnow matches the live central Security Scan event contract: protected bases (main,master,develop) are eligible, feature-base absence is non-passing evidence that must be refreshed after dependency integration, and the fixable Trivy boundary isMEDIUM/HIGH/CRITICAL;Current exact-head verification
For unchanged exact head
4ee417372c7a7e084767b50692c376bf03ff4b0a:cirun31366570251: terminal success;reviewer-cirun31366570220: terminal success;Security Scanrun31366570232: terminal success under its own scanner/revision semantics;The successful technical gates do not create live ruleset evidence or a qualifying independent formal approval. Issue #27 remains the enforceable-governance owner and #29 remains the Reviewer/Maintainer App provisioning owner.
Supersession
#87 was closed only after proving this clean protected-main successor preserves its three unique final blobs exactly. Old #87 checks/reviews do not transfer.
No repair workflow, self-modifying Action, force-push, synthetic approval, protection weakening, version bump, release or deployment authority is introduced.