AgentSkillsCN

paw-review-response

PAW 活动技能共享 PR 评审意见回复机制。提供 TODO 工作流程、提交模式以及回复格式,帮助您高效应对评审者的反馈。

SKILL.md
--- frontmatter
name: paw-review-response
description: Shared PR review comment response mechanics for PAW activity skills. Provides TODO workflow, commit patterns, and reply format for addressing reviewer feedback.

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:

  1. Make the changes to address the comment
  2. Check .gitignore before staging .paw/ artifacts:
    • If .paw/work/<feature-slug>/.gitignore exists: skip .paw/ files
    • Otherwise: stage all changed files
  3. Stage selectively: git add <file1> <file2> (never git add .)
  4. Verify staged changes: git diff --cached
  5. Commit with message referencing the comment
  6. 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).

ResponsibilityHandled By
What to changeActivity skill
How to commitThis utility skill
How to replyThis utility skill
When to pushActivity 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)