Pr Review Interactive

PR Review Interactive Skill

Run a PR review and triage every finding with the user before any comment lands on GitHub. Comments are accumulated in a single PENDING review and submitted as one batch.

Never use gh pr comment or gh pr review --comment for inline comments. Both silently drop the body on rocm-systems and similar orgs due to a known Projects-classic GraphQL bug. Always use gh api POST /repos/$O/$R/pulls/$N/comments with pull_request_review_id to attach to the PENDING review.

Comment format is short. 1-3 sentences, root cause first, fix snippet only when non-obvious. ASCII hyphen only. No praise, no recap, no Claude attribution.

When to Use

• User asks "interactive pr review", "walk me through the PR review", /pr-review-interactive, or invokes this skill by name.
• User wants tight control over what lands on a PR vs. just generating a report.

Do NOT use for:

• Quick local review with no GitHub posting: use pr-review.
• Posting a fixed set of pre-written comments: use gh api directly.

Inputs

• PR URL or <owner>/<repo>#<number> or repo + PR number.
• Optional: pre-existing report path (skip step 2 if provided and fresh).

Workflow

Step 1. Resolve PR

Parse owner, repo, PR number. Verify with gh api /repos/$O/$R/pulls/$N. Fetch head SHA. Save SHA to $CLAUDE_JOB_DIR/pr-review-interactive/head-sha.txt.

Step 2. Generate report

Invoke pr-review skill against the PR. If user already has a report file, ask whether to reuse or regenerate. Save report path to state.

Step 3. Parse findings

Read the report and extract a list of findings. Each finding has:

Store as JSON at $CLAUDE_JOB_DIR/pr-review-interactive/findings.json.

Step 4. Cache diff line set

gh api /repos/$O/$R/pulls/$N/files and parse patches. Build a set of (path, line, side) tuples that are valid review-comment targets. Inline review comments must target lines in the diff; others get the subject_type=file fallback (see step 6e).

Step 5. PENDING review

Detect existing PENDING review by current user on this PR: gh api /repos/$O/$R/pulls/$N/reviews --jq '.[] | select(.state=="PENDING" and .user.login==env.USER_LOGIN) | .id'.

• Found: reuse, save id.
• Not found: create empty PENDING review:

  gh api -X POST /repos/$O/$R/pulls/$N/reviews \
    -F commit_id=$SHA
  

  Save returned id to $CLAUDE_JOB_DIR/pr-review-interactive/review-id.txt.

Step 6. Iterate findings

Order: Must-Fix -> Should-Fix -> Nits. For each finding:

a.0. Fetch context (before presenting):

For each finding, fetch three source-code views and cache them in $CLAUDE_JOB_DIR/pr-review-interactive/snippets/<finding-id>.json:

1. Problem snippet - Read(path, offset=max(1, line-8), limit=17) to get ±8 lines around the finding line. If the file is part of the PR diff and the line numbers match the HEAD SHA, use the working-copy file. If the finding references the old side, fetch via git show $BASE_SHA:<path> instead. Mark the problem line(s) with a > prefix when rendering.

2. Proposed change - extract from the finding body any fenced code block following text like "Fixed code:", "Suggested fix:", "Replace with:", or the Fix column in a table. If none present, render a diff hunk derived from the analysis (best-effort) or write (no concrete patch in finding - reviewer to draft).

3. Related code (up to 3 sites) - parse the finding body for additional path:line mentions, fully-qualified symbol names, or "see also" references. For each, fetch ±5 lines via the same Read mechanism. If the finding body has none AND the title contains an identifier (function/class/macro), run one grep -n -R <symbol> <repo-root> capped at 3 hits, fetch ±3 lines for each. Skip entirely when there is nothing meaningful to show - do not pad with random call sites.

Cap total snippet bytes at 4 KB per finding; truncate the related block first, then the problem block (keep at least ±3 lines around the problem line), never the proposed change.

a. Present:

[MF-1] Must-Fix
location: <path>:<line>
title:    <title>

analysis:
<2-4 lines distilled from the report body, plain prose>

problem code (<path>:<line-N>-<line+N>):
\`\`\`<lang>
  <line-8>:  context line
  <line-7>:  context line
  ...
> <line>:    THE PROBLEM LINE
  <line+1>:  context line
  ...
\`\`\`

proposed change:
\`\`\`<lang>
<fix snippet, or "(no concrete patch - reviewer to draft)">
\`\`\`

related code:
- <path1>:<line1>-<line1+M>
  \`\`\`<lang>
  <snippet>
  \`\`\`
- <path2>:<line2>-<line2+M>
  \`\`\`<lang>
  <snippet>
  \`\`\`
(omit this block entirely when no related sites)

proposed comment:
<the short body, code fence only when needed>

Rendering rules:

• Show line numbers as a left gutter (%4d: ) so the user can map to the file without counting.
• Use > as the problem-line marker (1 char + space, so gutter stays aligned).
• Pick the language tag from the file extension (cpp, py, cmake, rs, go, sh, md, ...); default to no tag when unknown.

b. Ask user via AskUserQuestion:

c. Accept path:

gh api -X POST /repos/$O/$R/pulls/$N/comments \
  -f commit_id="$SHA" \
  -f path="$PATH" \
  -F line=$LINE \
  -f side=RIGHT \
  -F pull_request_review_id=$REVIEW_ID \
  -f body="$BODY"

On 422 "line is not part of the diff": retry as file-level comment (subject_type=file, no line).

d. Edit path: prompt user for new body; validate non-empty; then take accept path with new body.

e. Skip path: append to skipped.json with {id, reason}.

f. State persistence: after every action append to $CLAUDE_JOB_DIR/pr-review-interactive/accepted.json so a session crash does not lose work.

Step 7. Wrap-up summary

When loop done (or user quit):

Accepted: <N>
Skipped:  <M>
Pending:  <K> (not yet reviewed)

Ask user via AskUserQuestion:

Step 8. Submit (or hold)

If user picked an event:

gh api -X POST /repos/$O/$R/pulls/$N/reviews/$REVIEW_ID/events \
  -f event="$EVENT" \
  -f body="$OPTIONAL_OVERALL"

Print URL of submitted review.

If Discard: gh api -X DELETE /repos/$O/$R/pulls/$N/reviews/$REVIEW_ID.

If Keep pending: leave id in state file for resume next session.

Comment Format Rules

Good (MF-1 example):

`ROCPROFSYS_GPU_PERF_COUNTERS` is never registered via
`ROCPROFSYS_CONFIG_SETTING`, so `find(...)` returns `end()` and the
env var is a no-op. SDK PMC source gated on this so feature only
reachable via JSON injection.

\`\`\`cpp
ROCPROFSYS_CONFIG_SETTING(std::string, "ROCPROFSYS_GPU_PERF_COUNTERS",
    "...", "", "backend", "rocprofiler-sdk", "pmc");
\`\`\`

Bad: 12-line paragraph recapping the diff and listing every downstream caller.

State Files

All under $CLAUDE_JOB_DIR/pr-review-interactive/:

Resume on next session: read all state files, skip past last accepted/skipped index, continue.

Common Mistakes

Integration with Other Skills

Dependencies

• pr-review - generates the report this skill walks through
• git-gh-client - core gh CLI helpers