Skip to main content Process and implement code review feedback systematically. Use when user provides reviewer comments, PR feedback, code review notes, or asks to implement suggestions from reviews.
npx skills add mhattingpete/claude-skills-marketplace --skill review-implementing ai-agents anthropic automation claude-code claude-skills developer-tools
Review Feedback Implementation
Systematically process and implement changes based on code review feedback.
When to Use
Provides reviewer comments or feedback
Pastes PR review notes
Mentions implementing review suggestions
Says "address these comments" or "implement feedback"
Shares list of changes requested by reviewers
Systematic Workflow
1. Parse Reviewer Notes
Identify individual feedback items:
Split numbered lists (1., 2., etc.)
Handle bullet points or unnumbered feedback
Extract distinct change requests
Clarify ambiguous items before starting
2. Create Todo List
Use TodoWrite tool to create actionable tasks:
Each feedback item becomes one or more todos
Break down complex feedback into smaller tasks
Make tasks specific and measurable
Mark first task as in_progress before starting
Example:
- Add type hints to extract function
- Fix duplicate tag detection logic
- Update docstring in chain.py
- Add unit test for edge case
3. Implement Changes Systematically
Use Grep to search for functions/classes
Use Glob to find files by pattern
Read current implementation
Use Edit tool for modifications
Follow project conventions (CLAUDE.md)
Preserve existing functionality unless changing behavior
Check syntax correctness
Run relevant tests if applicable
Ensure changes address reviewer's intent
Mark todo as completed immediately after finishing
Move to next todo (only one in_progress at a time)
4. Handle Different Feedback Types
Use Edit tool for existing code
Follow type hint conventions (PEP 604/585)
Maintain consistent style
Create new files with Write tool if needed
Add corresponding tests
Update documentation
Update docstrings following project style
Modify markdown files as needed
Keep explanations concise
Write tests as functions, not classes
Use descriptive names
Follow pytest conventions
Preserve functionality
Improve code structure
Run tests to verify no regressions
5. Validation After implementing changes:
Run affected tests
Check for linting errors: uv run ruff check
Verify changes don't break existing functionality
6. Communication
Update todo list in real-time
Ask for clarification on ambiguous feedback
Report blockers or challenges
Summarize changes at completion
Edge Cases
Ask user for guidance
Explain conflict clearly
Breaking changes required:
Notify user before implementing
Discuss impact and alternatives
Tests fail after changes:
Fix tests before marking todo complete
Ensure all related tests pass
Referenced code doesn't exist:
Ask user for clarification
Verify understanding before proceeding
Important Guidelines
Always use TodoWrite for tracking progressMark todos completed immediately after each itemOnly one todo in_progress at any timeDon't batch completions - update status in real-timeAsk questions for unclear feedbackRun tests if changes affect tested codeFollow CLAUDE.md conventions for all code changesUse conventional commits if creating commits afterward Create or update AgentSkills. Use when designing, structuring, or packaging skills with scripts, references, and assets.
Create or update AgentSkills. Use when designing, structuring, or packaging skills with scripts, references, and assets.
Set up and use 1Password CLI (op). Use when installing the CLI, enabling desktop app integration, signing in (single or multi-account), or reading/injecting/running secrets via op.
CLI to manage emails via IMAP/SMTP. Use `himalaya` to list, read, write, reply, forward, search, and organize emails from the terminal. Supports multiple accounts and message composition with MML (MIME Meta Language).
Create or update AgentSkills. Use when designing, structuring, or packaging skills with scripts, references, and assets.
Create or update AgentSkills. Use when designing, structuring, or packaging skills with scripts, references, and assets.
Set up and use 1Password CLI (op). Use when installing the CLI, enabling desktop app integration, signing in (single or multi-account), or reading/injecting/running secrets via op.
CLI to manage emails via IMAP/SMTP. Use `himalaya` to list, read, write, reply, forward, search, and organize emails from the terminal. Supports multiple accounts and message composition with MML (MIME Meta Language).
