Skip to content

feat(pr-policy): add role separation gate#27

Merged
ArchonVII merged 1 commit into
mainfrom
agent/codex/26-role-separation-pr-policy
May 29, 2026
Merged

feat(pr-policy): add role separation gate#27
ArchonVII merged 1 commit into
mainfrom
agent/codex/26-role-separation-pr-policy

Conversation

@ArchonVII

Copy link
Copy Markdown
Owner

Summary

  • Adds a pure scripts/role-policy.mjs helper for F7 role-separation decisions.
  • Wires pr-policy.yml to warn by default and optionally hard-fail protected agent-managed PRs without independent Release-Admiral approval/marker.
  • Documents the new pr-policy inputs and owner-maintenance non-exemption in README/example caller docs.

Linked Issue

Closes #26

Scope

  • In scope:
    • pr-policy.yml role-separation warning and opt-in hard gate.
    • Unit tests for soft warning, protected-path hard block, Release-Admiral approval/marker, owner-maintenance PR non-exemption, and Dependabot exception.
    • README/example documentation for consumers.
  • Out of scope:

Changelog

N/A: this repo does not currently use changelog fragments.

Verification

Verification Notes

  • Tests pass

    command: npm test
    location: local
    result: 3 files passed, 63 tests passed
    timestamp: 2026-05-29T00:01:49Z
    
  • Whitespace check passes

    command: git diff origin/main...HEAD --check
    location: local
    result: passed with no output
    timestamp: 2026-05-29T00:01:49Z
    
  • Local actionlint

    command: actionlint -version; npx --yes actionlint -version; go version
    location: local
    result: unavailable locally; actionlint not installed, npx package had no executable, go not installed
    timestamp: 2026-05-29T00:01:59Z
    

Risks

  • Risk level: Medium. This changes reusable PR policy behavior, but the hard gate is opt-in and the default behavior is warning-only.
  • Rollback: Revert this PR, or consumers can leave enforce-role-separation unset/false.
  • Follow-ups:

Add a pure role-policy helper with tests and wire pr-policy.yml to warn by default while allowing protected agent-managed PRs to opt into hard Release-Admiral enforcement. Document the new inputs in README and the example caller.

Closes #26
@ArchonVII
ArchonVII merged commit 007ad49 into main May 29, 2026
1 check passed
@ArchonVII
ArchonVII deleted the agent/codex/26-role-separation-pr-policy branch June 5, 2026 04:10
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.

F7 follow-up: role-separation PR policy enforcement

1 participant