Review Feedback Implementation

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

• 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

• 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

- 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

• 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

• Notify user before implementing
• Discuss impact and alternatives

• Fix tests before marking todo complete
• Ensure all related tests pass

• 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 CLAUDE.md conventions for all code changes
• Use conventional commits if creating commits afterward