| name | address-pr-comments |
| description | Triage and fix PR review comments. Reads comments, filters false positives (outdated, misread, wrong context, style preference), validates relevance against current diff, and fixes valid ones with confirmation. Use when: "fix pr comments", "address review feedback", "handle pr feedback". |
User request: $ARGUMENTS
If no PR specified: Detect PR from current branch. If detection fails, report and stop.
Default scope: All unresolved comments on the PR.
Goal
Classify all unresolved PR comments against false positive criteria, report triage findings, then fix actionable items with user confirmation per fix.
False Positive Detection
| Type | Detection Signal |
|---|
| Outdated | Suggested change already exists in code; commit history shows it was addressed |
| Misread code | Comment's premise contradicts what code actually does |
| Wrong context | Comment references behavior/variables not present in the target code |
| Style preference | No functional or readability improvement—evaluate substance, not reviewer's tone. Naming changes that aid comprehension are actionable, not style preference |
| Resolved in thread | Reply discussion shows issue was already addressed or withdrawn |
Classification:
- Strong false-positive signal →
false-positive
- Real issue with clear fix →
actionable
- Unclear validity →
uncertain (report, don't fix)
Output
Triage report: Classification, reasoning, and summary counts for each comment. Format flexibly based on volume.
Per-fix flow: For each actionable item, show original comment and proposed change, then ask user to confirm with these options:
- Fix + resolve thread (recommended) — apply fix, resolve the PR comment thread
- Fix only — apply fix, leave thread unresolved
- Skip — don't apply this fix
Completion summary: After all items processed, summarize actions taken (fixes applied, false positives skipped, uncertain items flagged).
Constraints
| Constraint | Enforcement |
|---|
| Verify before classifying | Read actual code and reply threads; never classify from comment text alone |
| Conservative on fixes | Uncertain → report only |
| Explain rejections | Every false-positive needs reasoning |
| Per-fix confirmation | User confirms each fix individually |
| Resolve only when chosen | Only resolve threads or reply to comments when user selects "Fix + resolve" |
