review-implementing

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

For each todo item:

Locate relevant code:

• Use Grep to search for functions/classes
• Use Glob to find files by pattern
• Read current implementation

Make changes:

• Use Edit tool for modifications
• Follow project conventions (CLAUDE.md)
• Preserve existing functionality unless changing behavior

Verify changes:

• Check syntax correctness
• Run relevant tests if applicable
• Ensure changes address reviewer's intent

Update status:

• Mark todo as completed immediately after finishing
• Move to next todo (only one in_progress at a time)

4. Handle Different Feedback Types

Code changes:

• Use Edit tool for existing code
• Follow type hint conventions (PEP 604/585)
• Maintain consistent style

New features:

• Create new files with Write tool if needed
• Add corresponding tests
• Update documentation

Documentation:

• Update docstrings following project style
• Modify markdown files as needed
• Keep explanations concise