Skip to content

docs: explain protected-branch operating gates - #18

Merged
Elodie Mirza (elodiejmirza) merged 1 commit into
masterfrom
codex/direwolf-protection-guidance-20261104
Nov 4, 2026
Merged

Elodie Mirza (elodiejmirza) merged 1 commit into
masterfrom
codex/direwolf-protection-guidance-20261104

Conversation

@elodiejmirza

Copy link
Copy Markdown
Collaborator

BLUF

  • Tracks central governance#112/Feature Request: control beacon transmission using an external command wb2osz/direwolf#179: document the owner-approved protected-branch operating contract after the two rulesets were activated and independently read back.
  • AGENTS.md now records the dated rule IDs, eight exact context/App identities, normal signed/DCO merge gates and narrow PR-only emergency boundary. It requires fresh live readback rather than treating these dated instructions as current enforcement proof.
  • No source/runtime, dependency, workflow, rule, scanner, credential or publication change. This is useful contributor guidance and an ordinary maintenance PR; it may provide positive merge evidence under the new controls, not a negative-test or acceptance substitute.

Security Review Evidence and validation

  • Reviewed the one-file diff against active rulesets26542532/26542536 and central#182's payloads; no privilege expansion, bypass over hard gates, invented status context or fake human acceptance. Existing instructions retained.
  • Diff check and staged Gitleaks passed. No local native tests rerun for documentation-only changes; every repository-required hosted native/CodeQL gate must actually pass at this signed/DCO head before normal merge. Inspect full reviews/threads and GitHub signatures.
  • No admin bypass or unsafe master probe. Root will read integrated ancestry, checks/analyses and applicable policy evidence afterward. Governance#112/Feature Request: control beacon transmission using an external command wb2osz/direwolf#179 remain open for their own acceptance criteria and board readback.

Lessons Learned

  • LL0002 preserves the distinction between symptom/root control; LL0003 uses existing workflows/scanner owner; LL0004/0011 require live exact-head, App and output correlation; LL0005/0010 require signing/DCO and authoritative verification; LL0007 preserves unrelated work. No factory/Codespace/catalogue/identity-scope change.
  • Complexity removed or justified: reference central policy once, explain existing controls in operating guidance; add no duplicate enforcement or new workflow.
  • Project9/10 owning issues remain In Progress; PR field addition/readback remains pending the recorded authorised UI fault.

Signed-off-by: Elodie Mirza <elodiejmirza@gmail.com>
@elodiejmirza
Elodie Mirza (elodiejmirza) marked this pull request as ready for review November 4, 2026 15:43
Copilot AI balanced review requested due to automatic review settings November 4, 2026 15:43
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-04T15:44:54.351499Z 24a678c Draft marked ready
🔒 Security Review ✅ Completed 2026-11-04T15:46:04.614633Z 24a678c Draft marked ready
ℹ️ About Codex in GitHub

Your 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.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 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

Copy link
Copy Markdown
Collaborator Author

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.

@elodiejmirza
Elodie Mirza (elodiejmirza) merged commit 5f4dda3 into master Nov 4, 2026
13 checks passed
@elodiejmirza
Elodie Mirza (elodiejmirza) deleted the codex/direwolf-protection-guidance-20261104 branch November 4, 2026 16:35
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.

2 participants