Review Response
Review Comment Workflow
1. Read All Unresolved Comments
- •Use GitHub MCP tools to fetch PR comments and conversation threads
- •Identify which comments still require work:
- •Check for follow-up commits addressing the comment
- •Check for reviewer replies indicating resolution
- •Skip comments clearly already addressed
- •If uncertain whether a comment needs work, include it and ask for clarification
2. Create TODOs for Comments
Create one TODO per comment (group small related comments together):
code
TODO: Address review comment [link/ID] - What change is needed - Which file(s) to modify - Commit message reference
Grouping guidelines:
- •Group comments affecting the same file or section
- •Group comments addressing the same concern
- •Keep complex comments as separate TODOs
- •Each TODO = one focused commit
3. Address Comments Sequentially
Work through TODOs one at a time. For each:
- •Make the changes to address the comment
- •Check
.gitignorebefore staging.paw/artifacts:- •If
.paw/work/<feature-slug>/.gitignoreexists: skip.paw/files - •Otherwise: stage all changed files
- •If
- •Stage selectively:
git add <file1> <file2>(nevergit add .) - •Verify staged changes:
git diff --cached - •Commit with message referencing the comment
- •Push/Reply timing: Determined by calling activity skill—some push immediately per-comment, others batch commits and defer push to a verification step
CRITICAL: Complete each TODO's changes and commit before starting the next. Push and reply timing per activity skill instructions.
4. Verify Completion
- •Ensure all review comments have been addressed
- •Verify no open questions remain
- •Run any applicable verification (tests, linting)
Reply Format
Use this format when replying to review comments:
code
**🐾 [Activity] 🤖:** [What was changed and commit hash reference]
Activity names by PR type:
- •Planning PR:
Implementation Planner - •Phase PR:
Implementation Reviewer - •Docs PR:
Documenter - •Final PR:
Implementation Reviewer
Examples:
code
**🐾 Implementation Planner 🤖:** Updated Phase 2 to include error handling tests as suggested. See commit abc1234.
code
**🐾 Implementation Reviewer 🤖:** Added the missing null check and updated the docstring. Commit def5678.
Commit Message Format
Reference the review comment in commit messages:
code
Address review comment: [brief description] [More detail if needed] Ref: [PR comment URL or ID]
Role Separation
This utility skill handles mechanics (how to commit/push/reply). Activity skills handle domain logic (what changes to make).
| Responsibility | Handled By |
|---|---|
| What to change | Activity skill |
| How to commit | This utility skill |
| How to reply | This utility skill |
| When to push | Activity skill (based on workflow) |
Checklist Before Completion
- • All unresolved review comments addressed
- • Each comment addressed with focused commit
- • Commits pushed to remote
- • Replies posted to each addressed comment
- • Verification passed (tests, lint)