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.