# FORGE-pr-review

> Governed checklist for reviewing GitHub pull requests in the arifOS federation Ensures PRs meet constitutional, structural, and safety standards before merge.

- Skill: `ariffazil/forge-pr-review` (Agent Skill, multi-file: 5 files)
- Install (CLI): `npx skillmds@latest add ariffazil/forge-pr-review`
- Raw SKILL.md: https://api.skillmd.com/api/skills/ariffazil/forge-pr-review/raw
- Safety review: pending
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Coding & Dev Tools
- Author: ariffazil (https://skillmd.com/u/ariffazil)
- Updated: 2026-09-10
- Page: https://skillmd.com/skills/ariffazil/forge-pr-review

---


# GitHub PR Governance Review

## Overview

Pull requests are the constitutional boundary of a federation repo. A PR is the
unit at which code, governance, and human judgment meet. This skill defines the
**per-PR checklist** an agent must run before recommending merge or approval.

Scope: structural review of the diff itself, not the high-level risk-tier and
reviewer-routing rules (those live in `pr-review-governance`). This skill is the
**checklist**, that skill is the **policy layer**.

Companion skills:
- `github-runbook` — local git/gh inspection (read-only first)
- `github-ci-diagnose` — if CI is red on the PR branch
- `github-issue-triage` — if the PR resolves an open issue
- `pr-review-governance` — risk tier + reviewer routing (delegate first if risk ≥ high)

## When to Use

- Any PR touching >1 file
- Any PR modifying contracts, schemas, or registries
- Any PR from an external contributor
- Any PR marked as "high risk" by the author

## When NOT to Use

- **Do NOT use on cross-repo PRs without first invoking `pr-review-governance`** to determine risk tier.
- **Do NOT use as a substitute for CI** — CI runs first, this skill reads CI artifacts second.
- **Do NOT use to approve your own PR** — F1 AMANAH + F13 SOVEREIGN prohibit self-seal.
- **Do NOT use to dismiss a security finding** — escalate to `secret-safety-scan` + 888_JUDGE.
- **Do NOT use to merge to main directly** — merge is F13 SOVEREIGN unless branch protection already enforces the policy.

## arifOS-ACT Embedding

Before using this skill on any mutating, irreversible, or high-blast-radius task:
1. **ART** — Attune (what is the real task?), Recognize (what class of power?), Test (fit · authority · evidence · blast · reversible).
2. **Kernel** — Route to arifOS for F1–F13 judgment if action class is Maker/Messenger/Mutator/Destroyer/Sovereign.
3. **ACT** — Apply narrow, Constrain scope, Trace witness, STOP before corruption.
4. **Receipt** — Leave evidence of what changed, why, and under whose authority.