Pull Request Comment Resolver
Reviewer feedback is input, not authority. Verify every claim against the
current head, repository rules, code, and tests before deciding.
Workflow
- Build a complete two-surface snapshot: all issue comments in the PR
conversation, plus the pull request review surface containing all submitted
reviews and all review threads with every inline review comment and reply.
Exhaust pagination independently for issue comments, submitted reviews,
review threads, and nested replies; retain resolved and outdated items for
context; and record counts and retrieval cursors. An incomplete surface is a
blocker, not evidence that feedback is absent.
- Classify each item as fix, already handled, no fix, disagreement, uncertain,
or out of scope.
- Apply only validated fixes, verify them, and commit by coherent topic.
- Reply with evidence through the supported mechanism. Reply directly to a
review thread or PR conversation issue comment. GitHub has no reply mutation
for a top-level submitted-review body, so answer it in the PR conversation
with the review author and link identified.
- Resolve handled bot review threads. Leave human review threads unresolved
unless the user explicitly directs otherwise. Submitted reviews and PR
conversation issue comments have no resolvable state.
- Re-fetch feedback and report remaining risk or required user judgment.
Convergence for automated feedback
Keep bot feedback tied to the requested change, its acceptance criteria, and
the evidence needed for safe delivery. Fix verified correctness or regression
problems within scope. When current code or proportionate test evidence
disproves a finding, explain that evidence and close the eligible bot thread
instead of changing correct code. Defer speculative hardening, preference-only
refactors, and unrelated improvements once the delivery criteria are satisfied.
Batch related findings into a coherent correction pass. New feedback reopens
only the affected decision and verification unless it reveals a broader
problem. Do not keep expanding implementation or verification merely because a
bot can suggest another improvement.
Pause for user judgment when feedback reveals a severe security, data
integrity, compatibility, or operational risk; requires a material product or
architectural decision; or would substantially expand the agreed scope or
cost. Otherwise, make the narrowest evidence-backed decision and continue the
authorized workflow.
Within authorized feedback handling, explain a demonstrably incorrect or
already-handled claim with current-code or test evidence without asking again.
Ask when unresolved evidence prevents a safe decision, a disagreement requires
a material product or architectural decision, feedback reveals a severe risk
or substantial scope or cost expansion, or history rewriting or an external
write is not already authorized. Human threads still require explicit
direction to resolve. Read only the relevant reference:
references/workflow.md
references/decision-tree.md
references/author-classification.md
references/commit-strategy.md
references/reply-templates.md
references/github-cli.md
Related skills
vp-pr-briefing
when the PR needs orientation before feedback is resolved.
vp-git for commit
strategy and pull-request lifecycle decisions.
vp-pr-review-followup
when acting as the reviewer instead of the PR author.
1---2name: vp-pr-comment-resolver3description: Process author-side GitHub PR feedback: verify comments, make focused fixes, reply through the supported mechanism, and resolve eligible threads. Use for actionable submitted-review bodies, unresolved pull request review comments (inline code comments), or actionable issue comments in the PR conversation. Boundary: not for writing reviews or verifying checklists.4---56# Pull Request Comment Resolver78Reviewer feedback is input, not authority. Verify every claim against the9current head, repository rules, code, and tests before deciding.1011## Workflow12131. Build a complete two-surface snapshot: all issue comments in the PR14 conversation, plus the pull request review surface containing all submitted15 reviews and all review threads with every inline review comment and reply.16 Exhaust pagination independently for issue comments, submitted reviews,17 review threads, and nested replies; retain resolved and outdated items for18 context; and record counts and retrieval cursors. An incomplete surface is a19 blocker, not evidence that feedback is absent.202. Classify each item as fix, already handled, no fix, disagreement, uncertain,21 or out of scope.223. Apply only validated fixes, verify them, and commit by coherent topic.234. Reply with evidence through the supported mechanism. Reply directly to a24 review thread or PR conversation issue comment. GitHub has no reply mutation25 for a top-level submitted-review body, so answer it in the PR conversation26 with the review author and link identified.275. Resolve handled bot review threads. Leave human review threads unresolved28 unless the user explicitly directs otherwise. Submitted reviews and PR29 conversation issue comments have no resolvable state.306. Re-fetch feedback and report remaining risk or required user judgment.3132## Convergence for automated feedback3334Keep bot feedback tied to the requested change, its acceptance criteria, and35the evidence needed for safe delivery. Fix verified correctness or regression36problems within scope. When current code or proportionate test evidence37disproves a finding, explain that evidence and close the eligible bot thread38instead of changing correct code. Defer speculative hardening, preference-only39refactors, and unrelated improvements once the delivery criteria are satisfied.4041Batch related findings into a coherent correction pass. New feedback reopens42only the affected decision and verification unless it reveals a broader43problem. Do not keep expanding implementation or verification merely because a44bot can suggest another improvement.4546Pause for user judgment when feedback reveals a severe security, data47integrity, compatibility, or operational risk; requires a material product or48architectural decision; or would substantially expand the agreed scope or49cost. Otherwise, make the narrowest evidence-backed decision and continue the50authorized workflow.5152Within authorized feedback handling, explain a demonstrably incorrect or53already-handled claim with current-code or test evidence without asking again.54Ask when unresolved evidence prevents a safe decision, a disagreement requires55a material product or architectural decision, feedback reveals a severe risk56or substantial scope or cost expansion, or history rewriting or an external57write is not already authorized. Human threads still require explicit58direction to resolve. Read only the relevant reference:5960- `references/workflow.md`61- `references/decision-tree.md`62- `references/author-classification.md`63- `references/commit-strategy.md`64- `references/reply-templates.md`65- `references/github-cli.md`6667## Related skills6869- [`vp-pr-briefing`](https://github.com/VdustR/skills/tree/main/skills/vp-pr-briefing)70 when the PR needs orientation before feedback is resolved.71- [`vp-git`](https://github.com/VdustR/skills/tree/main/skills/vp-git) for commit72 strategy and pull-request lifecycle decisions.73- [`vp-pr-review-followup`](https://github.com/VdustR/skills/tree/main/skills/vp-pr-review-followup)74 when acting as the reviewer instead of the PR author.