receiving-code-review

收到代码审查反馈后、实施建议之前使用,尤其当反馈不明确或技术上有疑问时——需要技术严谨性和验证,而非敷衍附和或盲目执行

By jnmetacode · 1,077 installs

npx skills add jnmetacode/superpowers-zh --skill receiving-code-review

Source repository · Upstream listing

接收代码审查 概述 代码审查需要的是技术评估,不是情绪表演。 核心原则: 先验证再实施。先提问再假设。技术正确性优先于社交舒适度。 响应模式 禁止的回应 绝不要说: "你说得太对了!"(明确违反 CLAUDE.md 规定) "好观点!"/"反馈很棒!"(敷衍表演) "让我立刻实施"(在验证之前) 应该这样做: 复述技术需求 提出澄清性问题 如果审查意见有误,用技术理由反驳 直接动手做(行动胜于言辞) 处理不明确的反馈 示例: 按来源区别处理 来自搭档的反馈 可信赖 —— 理解后直接实施 仍然要问 如果范围不明确 不要敷衍附和 直接行动 或给出技术性确认 来自外部审查者的反馈 搭档的原则: "对外部反馈要持怀疑态度,但要仔细核实" YAGNI 检查——针对"专业化"功能建议 搭档的原则: "你和审查者都对我负责。如果我们不需要这个功能,就不要加。" 实施顺序 何时反驳 在以下情况反驳: 建议会破坏现有功能 审查者缺少完整上下文 违反 YAGNI(功能没人用) 对当前技术栈来说技术上不正确 存在遗留/兼容性原因 与搭档的架构决策冲突 如何反驳: 用技术理由,不要带防御情绪 提出具体问题 引用可正常工作的测试/代码 如果涉及架构问题,让搭档参与 如果觉得不方便当众反驳,暗号是: "Strange things are afoot at the Circle K" 确认正确的反馈 当反馈确实正确时: 为什么不用感谢: 行动说明一切。直接修复。代码本身就能表明你收到了反馈。 如果你发现自己要写"感谢": 删掉它。直接说明修复内容。 优雅地纠正自己的反驳 如果你反驳了但事后发现自己错了: 如实陈述纠正,然后继续。 常见错误 错误 修正 敷衍附和 复述需求或直接行动 盲目实施 先对照代码库验证 批量实施不测试 一次一项,逐个测试 假设审查者一定对 检查是否会破坏现有功能 回避反驳 技术正确性 社交舒适度 部分理解就开始实施 先澄清所有项 无法验证却继续推进 说明限制,请求指导 真实案例 敷衍附和(反面例子): 技术验证(正面例子): YAGNI(正面例子): 不明确的项(正面例子): GitHub 评论回复 在 GitHub 上回复行内审查评论时,在评论线程中回复( gh api repos/{owner}/{repo}/pulls/{pr}/comments/{id}/replies ),不要发顶层 PR 评论。