PR Review
Must read: ../pack-shared/standards.md. Read /taste and /architecture doctrines this turn before adjudicating Standards. Do not skip.
Ask style: ../pack-shared/asking.md
This skill is a user start. Do not nest it under /task.
Read doctrine.md and reference.md. The shared review contract owns review depth, evidence, the review output fence, finding records, and severity mapping. The shared execution context owns stateless authority and handoff rules.
Standards: always Read /taste and /architecture this turn. Apply
code-review:blocker-vs-follow-up, code-review:naming-alignment, the
Correctness hunt, and pr-review:* extras on initial and full-rescan (and on
newly introduced follow-up surface). Do not run a Design axis.
Process
- Pin the PR fixed point (
headSha) and load its body, commits, diff, all inline review comments, and all review threads, including resolved threads and every prior review page. RecordpreviousReviewedHeadfrom the last review this skill completed on this PR when available (from chat or the latest review commit association). - Dispatch
pr-reviewerTasks per ../pack-shared/subagents.md. Feed the what (open GitHub PR, injected plan, bars). The worker owns how to hunt. The parent reviews Completions and owns the publish question. Do not solo non-trivial PR review labor. - With no prior finding thread, run the shared contract's
initialreview. - On every follow-up, complete Pass A first (doctrine Output), then Pass B.
- Use
full-rescanonly when the user explicitly requests it or materially expands the review scope. - Show drafts, ask one publish question, apply the stale-head guard, then post only after approval.