Repository navigation
docs: explain protected-branch operating gates - #18
Elodie Mirza (elodiejmirza) merged 1 commit into
Conversation
Signed-off-by: Elodie Mirza <elodiejmirza@gmail.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
🔵 Needs a closer look
Inaccessible governance issues and unavailable live ruleset state prevent independent verification of ruleset/App identities and emergency-bypass boundaries, requiring an authorized maintainer’s review against approved policy.
No open issues 🎉
What changed in this PR
This documentation-only PR explains the protected-branch operating requirements for contributors without changing runtime behavior or enforcement.
Changes:
- Records dated ruleset IDs and required check/App identities, with fresh live verification required.
- Documents signed/DCO merge gates and the narrow PR-only emergency bypass boundary.
- Clarifies that protection and scanner changes require central governance and current evidence.
| File | Description |
|---|---|
| AGENTS.md | Adds protected-branch operating guidance, merge verification requirements, and governance boundaries. |
Review effort: Balanced
|
Review summary reconciled: root used existing owner-authorised API access to independently inspect active rulesets26542532/26542536, effective default-branch union and exact context/App producers. The reviewer's lack of live/cross-repo tools is preserved as a limitation, not counted as verification. The approved policy is merged centrally in#182; selected actual activation readback is merged in central#183 at d600a7e0f61d968adeb52ffa423f2d928b37f35f. Hard controls have no bypass; only PR review has OrganizationAdmin/pull_request, with0 mandatory approvals and resolved threads. Both native runs/3 analyses/aggregate at exact24a678c are successful and the signature/DCO is valid. No code/source finding or unresolved thread remains; no bot comment is represented as human approval/operational acceptance. Normal merge only, no admin override; wb2osz#112/wb2osz#179 stay open for their remaining criteria. |
BLUF
Security Review Evidence and validation
Lessons Learned