Back to Details

feedback-application

Compare original and translation side by side

🇺🇸

Original

English
🇨🇳

Translation

Chinese

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
  • 收到评审人员的意见或反馈
  • 粘贴了PR评审记录
  • 提及要落实评审建议
  • 提到“处理这些意见”或“落实反馈”
  • 分享了评审人员要求的变更列表

Systematic Workflow

系统化工作流

1. Parse Reviewer Notes

1. 解析评审记录

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
识别单个反馈项：
  • 拆分编号列表（1.、2.等）
  • 处理项目符号或无编号反馈
  • 提取明确的变更请求
  • 开始前先澄清模糊的内容

2. Create Todo List

2. 创建待办事项列表

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
使用TodoWrite工具创建可执行任务：
  • 每个反馈项对应一个或多个待办事项
  • 将复杂反馈拆分为更小的任务
  • 确保任务具体且可衡量
  • 开始前将首个任务标记为
    in_progress
示例：
- 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

3. 系统落实变更

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)
针对每个待办事项：
定位相关代码：
  • 使用Grep搜索函数/类
  • 使用Glob按模式查找文件
  • 阅读当前实现
进行变更：
  • 使用Edit工具进行修改
  • 遵循项目约定（CLAUDE.md）
  • 除非需要更改行为，否则保留现有功能
验证变更：
  • 检查语法正确性
  • 如有适用，运行相关测试
  • 确保变更符合评审人员的意图
更新状态：
  • 完成后立即将待办事项标记为
    completed
  • 处理下一个待办事项（同一时间仅一个
    in_progress
    任务）

4. Handle Different Feedback Types

4. 处理不同类型的反馈

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
代码变更：
  • 使用Edit工具修改现有代码
  • 遵循类型提示约定（PEP 604/585）
  • 保持风格一致
新功能：
  • 如有需要，使用Write工具创建新文件
  • 添加相应的测试
  • 更新文档
文档：
  • 遵循项目风格更新文档字符串
  • 按需修改markdown文件
  • 保持解释简洁
测试：
  • 以函数形式编写测试，而非类
  • 使用描述性名称
  • 遵循pytest约定
重构：
  • 保留现有功能
  • 改进代码结构
  • 运行测试以确保无回归问题

5. Validation

5. 验证

After implementing changes:
  • Run affected tests
  • Check for linting errors: 
    uv run ruff check
  • Verify changes don't break existing functionality
落实变更后：
  • 运行受影响的测试
  • 检查代码规范错误：
    uv run ruff check
  • 确保变更不会破坏现有功能

6. Communication

6. 沟通

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 CLAUDE.md conventions for all code changes
  • Use conventional commits if creating commits afterward
  • 始终使用TodoWrite跟踪进度
  • 完成后立即标记待办事项为已完成
  • 同一时间仅一个待办事项处于in_progress状态
  • 不要批量标记完成 - 实时更新状态
  • 对不明确的反馈提出疑问
  • 如果变更影响了已测试代码，运行测试
  • 所有代码变更遵循CLAUDE.md约定
  • 如果后续要创建提交，使用规范提交格式