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 (.clinerules/ or AGENTS.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

Tests:

• Write tests as functions, not classes
• Use descriptive names
• Follow pytest conventions

Refactoring:

• 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

Keep user informed:

• Update todo list in real-time
• Ask for clarification on ambiguous feedback
• Report blockers or challenges
• Summarize changes at completion

Edge Cases

Conflicting feedback:

• 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 progress
• Mark todos completed immediately after each item
• Only one todo in_progress at any time
• Don't batch completions - update status in real-time
• Ask questions for unclear feedback
• Run tests if changes affect tested code
• Follow .clinerules/ or AGENTS.md conventions for all code changes
• Use conventional commits if creating commits afterward