| name | check-pr-review |
| description | Review all comments and reviews on a GitHub PR — inline code annotations, review verdicts (APPROVED / CHANGES_REQUESTED), and conversation threads. Classifies each as stale/valid/deferred/skip, fixes valid ones in the working tree, creates GitHub issues for deferred ones. Pass a PR number or omit to auto-detect from the current branch. |
| argument-hint | <PR number> (optional) |
| allowed-tools | Bash, Read, Edit, Write, Grep, Glob |
PR context
!PR_ARG="$ARGUMENTS"; REPO=$(gh repo view --json nameWithOwner --jq .nameWithOwner 2>/dev/null); PR=${PR_ARG:-$(gh pr view --json number --jq .number 2>/dev/null)}; gh pr view "$PR" --json number,title,state,headRefName,additions,deletions --jq '"PR #\(.number): \(.title)\nState : \(.state)\nBranch: \(.headRefName)\nDiff : +\(.additions) / -\(.deletions) lines"' 2>/dev/null || echo "ERROR: could not resolve PR — pass a PR number explicitly"
Inline code comments
!PR_ARG="$ARGUMENTS"; REPO=$(gh repo view --json nameWithOwner --jq .nameWithOwner 2>/dev/null); PR=${PR_ARG:-$(gh pr view --json number --jq .number 2>/dev/null)}; gh api "repos/$REPO/pulls/$PR/comments" --paginate --jq 'sort_by(.path, (.line // .original_line)) | .[] | (if .in_reply_to_id then " ↳ [reply \(.id)] [\(.user.login)]" else "[\(.id)] [\(.user.login)] \(.path):\(.line // .original_line // "?")" end) + "\n" + (if .in_reply_to_id then " " else " " end) + (.body | gsub("\n"; "\n ")) + "\n"' 2>/dev/null || echo "(none)"
Review verdicts
!PR_ARG="$ARGUMENTS"; REPO=$(gh repo view --json nameWithOwner --jq .nameWithOwner 2>/dev/null); PR=${PR_ARG:-$(gh pr view --json number --jq .number 2>/dev/null)}; gh api "repos/$REPO/pulls/$PR/reviews" --paginate --jq '.[] | "[\(.user.login) — \(.state)]" + (if .body != "" then "\n " + (.body | gsub("\n"; "\n ")) else " (no summary)" end) + "\n"' 2>/dev/null || echo "(none)"
Conversation comments
!PR_ARG="$ARGUMENTS"; REPO=$(gh repo view --json nameWithOwner --jq .nameWithOwner 2>/dev/null); PR=${PR_ARG:-$(gh pr view --json number --jq .number 2>/dev/null)}; gh api "repos/$REPO/issues/$PR/comments" --paginate --jq 'if length == 0 then "(none)" else (.[] | "[\(.user.login)]\n " + (.body | gsub("\n"; "\n ")) + "\n") end' 2>/dev/null || echo "(none)"
Classification rules
For each top-level inline comment (entries without ↳):
- Stale — the concern no longer applies to the current code (already fixed, file removed, logic changed). Do nothing.
- Valid — a real bug, security issue, or correctness problem not yet addressed. Fix it in the working tree now.
- Deferred — a reasonable suggestion (architecture, refactor, edge case handling) but out of scope for this PR. Create a GitHub issue.
- Skip — factually wrong, opinion-only with no actionable substance, or noise. Dismiss it.
For reply threads (↳): read the full thread for context before classifying the parent comment. A reply that resolves the concern makes the parent stale.
For CHANGES_REQUESTED reviews: treat each distinct concern in the review body as a comment to classify independently.
For APPROVED reviews and plain COMMENTED reviews with no code concerns: classify as Skip (acknowledge, no action).
Steps
- Read the injected data above
- For each top-level inline comment:
a. Read the referenced file at the referenced line (
path:line) to see the current code state
b. Read any reply thread beneath it for context
c. Classify with a one-line reason
- For each CHANGES_REQUESTED review body: extract distinct concerns and classify each
- Fix all valid issues directly in the source files
- After all fixes:
cargo clippy -- -D warnings && cargo test
- For each deferred issue:
gh issue create --title "..." --body "..." — write a clear title and body explaining the concern and why it's worth tracking
- Report the summary table:
| ID | Author | Location | Classification | Action |
|---|
#id | login | path:line | Stale / Valid / Deferred / Skip | Fixed in path / Issue #N / Dismissed |
