Receiving code review
Skill yinqd3/workbuddy-skills/superpowers/receiving-code-review
A curated collection of WorkBuddy agent skills — academic pipeline, frontend slides, PPTX generator, tool-call repair, and more.
npx -y skills add yinqd3/workbuddy-skills --skill receiving-code-reviewAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
One thing to look at
- 4 stars4 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.
What its author says it does
Copied from the file, not written here
接收代码审查反馈后的正确处理流程。分类问题、确认理解、逐项修复、请求重新审查。触发词:审查反馈、review comments、修改意见、改一下。
SKILL.md
1.8 KB, as published. Nobody here has run it
接收代码审查
概述
收到审查反馈后,正确分类、确认、逐项处理。不要让审查意见堆成山再一次性改。
处理流程
接收审查 → 分类问题 → 确认理解 → 逐项修复 → 请求重新审查
步骤 1:分类问题
将每个问题按严重程度分类:
| 严重程度 | 示例 | 处理 |
|---|---|---|
| 严重 | 逻辑错误、安全漏洞、数据丢失风险 | 立即修复 |
| 重要 | 性能问题、可维护性问题、测试缺失 | 优先修复 |
| 次要 | 命名建议、风格偏好 | 评估后决定 |
| 讨论 | 设计分歧、替代方案 | 先讨论再决定 |
步骤 2:确认理解
对每个你不确定的问题,先确认理解再动手:
- "确认一下,你是说 [你的理解],对吗?"
- 不要假设自己理解了就改
步骤 3:逐项修复
- 一次修复一个类别的问题
- 每项修复后验证测试
- 用清晰提交信息
git commit -m "fix: address review - [简要描述修改]"
步骤 4:请求重新审查
修复完成后:
- 汇总修改清单
- 说明每个问题如何解决或为什么不改
- 派发
requesting-code-review重新审查变更部分
回复模板
审查反馈处理完毕:
严重问题 (N 个):
- [问题1]: [如何修复的]
- [问题2]: [如何修复的]
重要问题 (N 个):
- [问题1]: [如何修复的]
次要问题 (N 个):
- [问题1]: [采纳/不采纳及原因]
讨论问题 (N 个):
- [问题1]: [我的看法,等你确认]
请重新审查 [commit range] 的变更。