agentspec: Skill for Claude Code

Review a Pull Request (GitHub)

One responsibility: turn a pull request into a defensible verdict backed by reproduced evidence, packaged as comment text a human posts. The verdict comes from canonical state — the integration branch and the design records the change claims to implement — never from the pull request's own framing of itself, and never from what the reviewing session happens to remember. Nothing is believed that was not either read in canonical state or reproduced by a command whose exit code ends up in the comment.

When to use / Skip if

Use when:

• A pull request in this repository needs review before merge.
• A previously reviewed pull request has a new round of fixes whose claims need re-verification.
• Someone states that a review finding is addressed, and the claim needs checking against the code.

Skip if:

• The change is uncommitted work in the local tree — that is the /review command; there is no pull request to ground against.
• The task is authoring or publishing an issue or ADR — github-cr-issue, github-cr-adr, github-post-issue.
• The ask is to merge, release, or back-merge — that is the release procedure (docs/reference/releasing.md), not a review.

Reviewer policy is out of scope: who reviews is decided by the repository, not by this skill. It enforces one staffing rule and no others — the reviewer is not the author.

Step 0 — detect the repo and pin the target

REPO=$(gh repo view --json nameWithOwner -q .nameWithOwner)
[ -n "$REPO" ] || REPO=$(git remote get-url origin | sed -E 's#(git@|https://)github.com[:/]##; s#\.git$##')
N="${1:?pull request number}"
BASE=$(gh pr view "$N" --repo "$REPO" --json baseRefName -q .baseRefName)

Pass --repo "$REPO" on every gh command, so the review reads the repository the session is in rather than a hardcoded one.

Phase 1 — Ground the review before judging anything

Nothing is assessed until canonical state has been read. In order:

#	Read	How
1	The tip of the branch the change lands on — develop for ordinary work, main for release and hotfix pull requests	git fetch origin && git log --oneline -1 "origin/$BASE"
2	The pull request's own metadata, including its author	gh pr view "$N" --repo "$REPO" --json title,body,author,baseRefName,headRefName,isDraft,mergeable,mergeStateStatus,files
2b	The discussion and every prior round — for the context verifier only	gh pr view "$N" --repo "$REPO" --json comments,reviews
3	The record it implements — the Implements / Closes line of the body	gh issue view <record> --repo "$REPO"
4	Every ADR the change or the record cites, with its status	gh issue view <adr> --repo "$REPO"
5	The acceptance criteria, re-derived from steps 3–4	read them out of the record; never copy the pull request's summary of them