| name | pr-governance-review |
| description | Use when reviewing an incoming pull request for north-star alignment, trust-boundary regressions, malicious or low-signal degradation, stale-vs-current CI interpretation, and true merge readiness beyond a green gate. Also owns PR train planning, merge queue cohorts, maintainer patch-then-merge, contributor harvest decisions, and backlog review waves. |
PR Governance Review Skill
Purpose and Trigger
- Primary scope:
pr-governance-review-intake
- Trigger on PR review, merge readiness, maintainer patching, backlog train planning, PR governance reports, or contributor-facing PR decisions.
- Avoid overlap with
repo-operations, quality-contracts, backend-runtime-governance, frontend-architecture, and security-audit.
Coverage and Ownership
- Role:
owner
- Owner family:
pr-governance-review
Owned repo surfaces:
.codex/skills/pr-governance-review
Non-owned surfaces:
repo-context
repo-operations
quality-contracts
backend-runtime-governance
frontend-architecture
security-audit
Do Use
- Current-head PR review beyond green CI.
- Async PR-train planning, queue cohorts, collision groups, patch trains, and decision waves.
- Maintainer patch/harvest decisions, contributor acknowledgement, and lane-specific GitHub comment posture.
- Review for north-star drift, trust-boundary regression, stale CI, duplicate architecture, or unreachable helpers.
Do Not Use
- Broad repo orientation before a PR or owner surface is known.
- CI root-cause repair after a check failure is already classified.
- Domain implementation work after a PR has been handed to an owner skill.
Read First
docs/reference/operations/ci.md
docs/reference/quality/pr-impact-checklist.md
.codex/skills/repo-operations/SKILL.md
.codex/skills/pr-governance-review/references/review-axes.md
.codex/skills/pr-governance-review/references/operator-batch-output-contract.md
.codex/skills/pr-governance-review/references/blocker-gates.md
.codex/skills/pr-governance-review/references/comment-and-report-contract.md
.codex/skills/pr-governance-review/references/pr-train-review-sop.md
Workflow
Default PR-Train Mode: Use the async PR-train method as the default for
multi-PR work, now formalized as the PR governance subagent train method.
Lane is decided by AUTHOR, not branch name. Community/contributor/agent PRs:
intake against main is retargeted to integration/pr-train before review,
approval, maintainer patching, harvest, queue, or merge. Governed maintainers
(in main.review_bypass_users / main.merge_queue_bypass_users) shipping their
OWN code branch from origin/main and open a PR DIRECTLY into main — no train
detour, no cherry-pick, no promote-branch whitelist (the PR Base Policy gate
passes them by actor identity; CI gate + merge queue + post-merge smoke still
apply). main therefore receives both train-promotion PRs and maintainer
direct-to-main PRs; the train carries everything non-maintainer.
- Lock the current PR head SHA, current
CI Status Gate, mergeability, draft state, and review state before judging.
- Run the delegation router at intake; use real read-only subagent evidence lanes for non-trivial, high-risk, or multi-PR work.
- Run the PR checklist or hybrid live report and treat
contract_set, duplicate_group, public_comment_policy, lane, and live_report_action as decision records.
- Use the async PR governance subagent train method as the default for more than one PR: first lock the operator-approved surface scope, identify only trains inside that scope or its hard dependencies, run scoped non-touching trains in parallel through subagent evidence lanes, sequence touching PRs oldest-first inside each train, queue independent green PRs together, run disjoint patch trains, and run decision waves asynchronously. Unrelated green-clean PRs stay in
out_of_scope_candidates until a separate operator checkpoint approves a broader sweep.
- Exclude PRs with failing/missing/stale required checks or failing auxiliary checks from executable trains unless the task is CI repair.
- Check
references/anti-rationalization.md at intake; then apply blocker gates before merge: north-star drift, duplicate architecture, trust-boundary regression, caller/proxy/backend mismatch, unreachable helpers, stacked diff, proof gaps, and local dirty-file overlap.
- Prefer direct contributor PR merge, then
maintainer_patch_then_merge, then maintainer harvest. Harvest is allowed only when the source PR should not be the merge vehicle and the plan names accepted value, canonical attach point, write set, dropped/deferred pieces, proof, source PR close-or-hold state, and co-author attribution when code/tests are materially reused.
- Use
comment-and-report-contract.md for every GitHub write; edit existing maintainer records first and post one post-merge closeout after smoke.
- Keep branch switching, commits, GitHub writes, approvals, merges, deploys, report refreshes, and final decisions in the parent session.
- Preserve the current operating-history lessons in
evolution-history.md; do not append incident history to this compact kernel.
Handoff Rules
- CI, queue, deploy, or branch-protection authority routes to
repo-operations.
- Test placement or verification policy routes to
quality-contracts.
- Backend runtime boundaries route to
backend-runtime-governance.
- Frontend caller/route contracts route to
frontend-architecture.
- IAM, consent, vault, PKM, or sensitive data boundaries route to
security-audit.
Required Checks
python3 -m py_compile .codex/skills/pr-governance-review/scripts/pr_review_checklist.py .codex/skills/pr-governance-review/scripts/test_pr_review_checklist.py .codex/skills/pr-governance-review/scripts/build_runtime_schematics.py .codex/skills/pr-governance-review/scripts/contributor_impact_report.py
python3 .codex/skills/pr-governance-review/scripts/test_pr_review_checklist.py
python3 .codex/skills/agent-orchestration-governance/scripts/agent_router_smoke.py
python3 .codex/skills/pr-governance-review/scripts/build_runtime_schematics.py --text
python3 .codex/skills/pr-governance-review/scripts/pr_review_checklist.py --repo hushh-labs/hushh-research --live-report --scan-mode hybrid --selection-order oldest --limit 100 --candidate-limit 40 --text --output tmp/pr-governance-live-report.md
python3 .codex/skills/pr-governance-review/scripts/test_contributor_impact_report.py
./bin/hushh codex audit --text
./bin/hushh docs verify