# Review Pr

> Review Forge pull requests from Codex with Forge-aware heuristics and the shared Forge verdict/merge model.

- Skill: `rapiercraftstudios/review-pr` (Agent Skill)
- Install (CLI): `npx skillmds@latest add rapiercraftstudios/review-pr`
- Raw SKILL.md: https://api.skillmd.com/api/skills/rapiercraftstudios/review-pr/raw
- Safety review: pending
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: AI & ML
- Author: rapiercraftstudios (https://skillmd.com/u/rapiercraftstudios)
- Updated: 2026-09-17
- Page: https://skillmd.com/skills/rapiercraftstudios/review-pr

---


# Forge Repo Adapter: review-pr

Use `commands/review-pr.md` as the lifecycle source for review phases, GitHub outputs, and merge semantics, but override the project-specific repo assumptions when the PR belongs to `RapierCraftStudios/forgedock`.

## Forge Repo Review Override

For Forge repo PRs:
- Do not require project-specific service paths like `services/api`, `services/worker`, or `web/`.
- Do not depend on `commands/review-pr-agents.md` as an authoritative domain catalog; it remains reference-only for the shared command system.
- Do not route Forge PRs into `review-pr-staging` solely because the base/head pair matches staging patterns. Continue in a Forge-aware full review flow in this adapter.

## Forge Review Categories

Replace the shared command’s service/domain classification with these categories:
- **Router / state machine**: `commands/work-on.md`, `commands/orchestrate.md`, `commands/milestone.md`, `commands/review-pr.md`
- **Nested pipeline modules**: `commands/work-on/**`
- **Runtime / installer layer**: `install.sh`, `install-codex.sh`, `AGENTS.md`, `CLAUDE.md`, `docs/CODEX.md`
- **Verification scripts**: `scripts/**`
- **Docs / changelog alignment**: `docs/**`, `CHANGELOG.md`

## Forge Review Checks

Run the shared review lifecycle, but use Forge-relevant checks:
- `bash -n` on changed shell scripts
- stale reference scan for outdated `.claude/commands`, `.claude/worktrees`, deleted docs, or wrong repo-local paths
- nested skill mapping sanity: `Skill("work-on:...")` / `Skill("review-pr...")` references touched by the PR still correspond to installed `forge-*` skills or repo-local adapters
- installer sanity: if `install-codex.sh` changed, verify it still generates namespaced skills and does not overwrite unrelated global skills
- changelog coverage for meaningful workflow behavior changes

## Review Fan-Out

Use Codex sub-agents only when the diff justifies it. Good fan-out slices for Forge PRs are:
- workflow/state-machine reviewer
- GitHub/CLI integration reviewer
- runtime/installer reviewer
- docs/changelog reviewer

## Output Contract

Keep the shared review-pr output behavior:
- evidence-based findings only
- structured findings when reporting issues
- GitHub comment summary
- issue creation for real findings
- merge/approve behavior compatible with `work-on/review`