/engineering:review
/review — Address PR Review Comments
Address PR review comments interactively. Fetch comments, group by file, triage each one (fix or decline with reasoning), and post inline replies. Use when user says "address the PR comments", "go through the review", or wants to clear a review queue on an open PR. Distinct from `/code-review`, which reviews code before it's merged.
<PR number> Interactive workflow for fetching, triaging, and responding to PR review comments. Each comment is fixed or declined with an inline reply.
Full skill instructions
This is the same file the agent loads when the skill triggers. Source: plugins/engineering/skills/review/SKILL.md.
Instructions
When the user runs /review <pr-number>, follow this workflow:
Phase 1: Fetch and Present Comments
Fetch review comments:
gh api repos/pixeltowers/quik/pulls/<pr-number>/commentsGroup comments by file and present them in a numbered list. For each comment show:
- File path and line number
- Reviewer name
- Comment body (summarized if long)
If there are no comments, report that and stop.
Phase 2: Triage Each Comment
For each comment, ask the user whether to:
- Fix: Implement the change
- Decline: Provide reasoning
Process comments one at a time (or in small batches if they relate to the same file/concern).
Phase 3: Apply Fixes
For each comment marked as fix:
- Make the code change
- Stage and commit with a clear message
- Push the branch
- Reply to the comment thread with the commit SHA:
gh api repos/pixeltowers/quik/pulls/<pr-number>/comments/<comment-id>/replies -X POST -f body='Fixed in <commit-sha>'
Phase 4: Reply to Declined Comments
For each comment marked as decline:
- Ask the user for their reasoning (or draft a response for approval)
- Reply to the comment thread:
gh api repos/pixeltowers/quik/pulls/<pr-number>/comments/<comment-id>/replies -X POST -f body='<reasoning>'
Phase 5: Summary
After all comments are addressed, present a summary:
- Number of comments fixed (with commit SHAs)
- Number of comments declined
- Any remaining unaddressed comments
Rules
- Never reply to a comment without explicit user approval
- Each fix should be its own commit following Conventional Commits:
type(scope): description(e.g.fix(ingestor): handle empty payload,refactor(dispatcher): simplify routing) - Use
gh api .../comments/{id}/repliesfor inline thread replies, notgh pr comment - For general PR comments (not tied to code lines), use
gh pr comment <pr-number> --body "message" - Always push after committing fixes so the reply references a pushed commit