| name | resolve-pr-comments |
| description | Address GitHub PR review comments end-to-end, fetch unresolved review threads (via `gh` GraphQL), implement fixes, reply with what changed, and resolve threads using the bundled scripts. Use when asked to “address PR comments”, “resolve review threads”, or “clear requested changes”. |
Resolve PR Comments
Use this workflow when asked to “address PR comments”, “resolve review threads”, or “clear requested changes” on a GitHub pull request.
Preconditions
- You are in the repo working tree for the PR.
gh auth statussucceeds and you have permission to comment/resolve threads.jqis installed (used by the helper scripts).
Workflow
Identify the PR for the current branch
- Prefer:
gh pr view --json number,url,headRefName,baseRefName --jq '{number,url,headRefName,baseRefName}' - If no PR exists for the current branch, stop and ask the user what PR to target.
- Prefer:
Fetch unresolved review threads
- Run:
bash skills/resolve-pr-comments/scripts/fetch-unresolved-review-threads.sh - If you need to target a different PR explicitly:
bash skills/resolve-pr-comments/scripts/fetch-unresolved-review-threads.sh 137 - This prints a JSON object with
{ pr, threads }wherethreadsincludes onlyisResolved=false.
- Run:
Triage and decide action per thread
- If the comment is valid: implement the fix (prefer smallest change that satisfies intent).
- If it’s incorrect / not applicable: reply explaining why (with concrete reasoning), then resolve.
- If it requires product/architecture decision: reply with options + ask for direction; do not resolve unless the reviewer explicitly said it’s optional.
- If it’s outdated (
isOutdated=true): still reply with what changed + where, then resolve.
Implement fixes
- Locate the referenced code by:
- using
path/linefrom the thread (when present), and/or - searching for identifiers mentioned in the comment.
- using
- Keep fixes scoped; do not refactor unrelated code.
- Add/adjust tests where the comment implies a behavioral requirement.
- Locate the referenced code by:
Reply + resolve
- Reply (keep it short; include file path and what changed):
bash skills/resolve-pr-comments/scripts/reply-and-resolve-review-thread.sh '<THREAD_ID>' '<REPLY_BODY>' - For longer replies, pass the body via stdin:
cat <<'EOF' | bash skills/resolve-pr-comments/scripts/reply-and-resolve-review-thread.sh '<THREAD_ID>' -- (write reply)
EOF
- If you need to reply without resolving yet:
bash skills/resolve-pr-comments/scripts/reply-to-review-thread.sh '<THREAD_ID>' '<REPLY_BODY>'
- Reply (keep it short; include file path and what changed):
Notes
- Prefer resolving threads only after the code is updated (or you’ve explained why no change is needed).
- If CI is required for confidence, mention it in the reply (“will resolve after CI is green”) and avoid resolving prematurely.