risk-review
Installation
SKILL.md
risk-review
risk-review answers one question for the factory: may this PR merge without an engineer? It reviews the diff through the specialist lenses the diff touches, then applies a fixed rubric so the answer is the same on every run — the model's confidence never lowers the tier.
Inputs
- PR — number or URL; read
gh pr view <pr> --json number,headRefOid,baseRefName,state,isDraft,files,body,closingIssuesReferences,statusCheckRollup,reviewDecision,latestReviews,comments. Draft or closed → stop. - Green checks on head — every required check
passonheadRefOid(gh pr checks <pr> --required). No required checks reported → inspect all checks and record that fallback. A failed lookup is unknown. Anything failing or pending → stop and suggest/ci-loop <pr>. - Diff —
gh pr diff <pr>andgh pr diff <pr> --name-only. Lanes read surrounding code at head from the PR's worktree (<project-repo>/.worktrees/pr-<n>) when it exists, else withgit show <head-sha>:<path>; this read-only review never creates a worktree and never reads the main checkout as if it were head. - Ticket scope — the linked issue's
## Acceptance criteriabody section/checklist, otherwise its newest standalone## Acceptance criteriacomment (legacy criteria markers remain readable). No criteria → acceptance remains unproven. - Acceptance and QA evidence — the
Acceptance criteriaverdict in a body explicitly naming head, otherwise the newest## QA handoffcomment whoseChange:SHA is head. Read the newest agentic-qa marker for that SHA; stale acceptance and QA evidence never count. - Merge danger — the
Door:andBlast radius:lines in the PR body or QA comment, read as the author's claim.