| name | forgedock-review-pr |
| description | Run the authoritative context-aware ForgeDock review for an exact PR, including routing, verification, risk-derived fresh reviewers, finding issues, verdict, and optional guarded merge. |
ForgeDock Review PR
Required loading
- Read
../../specs/pi-adapter.md completely.
- Read
../../specs/original/commands/review-pr.md completely in bounded chunks.
- Parse the arguments appended to this skill invocation.
- During reviewer selection, read
../../specs/original/commands/review-pr-agents/protocols.md and only the selected
persona files in that directory.
Execution contract
Use direct Bash with gh and git commands for all review operations; verify gh
authentication and repository access first. The original specification is authoritative
for phase order, hard rules, automated checks, reviewer selection, findings, verdicts,
and merge policy. Follow it. The Pi adapter only translates runtime mechanics (skill
references, subagent dispatch, $FORGE_HOME paths).
Freeze the exact PR head/base from GitHub before review. Every PR is independently
reviewable: a standalone review requires no work-on pipeline state and runs the same
checks and reviewer panel as a pipeline review.
Automatically switch to the staging strategy when the selector or actual route targets
the protected/default branch as specified; load
../../specs/original/commands/review-pr-staging.md directly rather than emitting a
nested slash command.
Run configured automated and integration checks. Derive the reviewer roster from the
actual risk surface, and calibrate reviewer effort to that risk: documentation-only,
template, or single-file metadata lanes run their panel at medium thinking effort;
lanes touching executable code paths, security/auth/data/concurrency surfaces, or
cross-file integration run it at high. Set the effort per reviewer task via the model
thinking suffix — never by lowering the blocking standard. Prepare each reviewer bundle
deterministically yourself: fetch the full diff once, slice it per reviewer with its
persona and identity (repository, PR, head SHA; remaining tuple fields optional), and
pass it inline — reviewers receive a complete bundle and never search for one. Launch
the fresh-context reviewer panel with Pi subagents and join every selected reviewer.
Reviewers start from the frozen diff but retain repository read/search access for
evidence tracing.
Create or deduplicate a GitHub issue for every finding before summary publication
(via the packaged forgedock-issue skill). Post an official PR review tied to the
frozen SHA. Merge only when --auto-merge was explicit and the original blocking
policy passes. Review never closes the linked issue or cleans the work-on tree.