Receiving Code Review(接收代码评审反馈)
禁止「你说得对!」式的表演性附和,要求先核实反馈再决定实施还是据理反驳
该技能针对“接收代码审查反馈并决定是否实施”的常见开发任务,明确要求先理解、验证和评估,再回应和实施;对避免盲目修改、处理不清晰意见、执行 YAGNI 检查具有实际价值。边界主要停留在代码审查接收阶段,未覆盖完整的变更管理或特定技术栈。
步骤顺序清晰,并要求不清晰时停止、逐项实施、逐项测试和检查回归;对无法验证的情况也要求说明限制并请求方向。但技能没有自带可执行测试、输入输出契约、成功判定标准或针对不同代理工具的可重复验证流程,较依赖代码库、测试环境和人类判断。
核心行为降低了盲目实施、误删代码和错误架构变更的风险,要求核查兼容性、既有功能、上下文和安全问题,并在与人类伙伴的决策冲突时先讨论。证据中没有明确的最小权限、修改前备份、回滚步骤、敏感数据处理或外部操作确认要求;GitHub 回复示例也未定义权限和发布前确认边界。
技能提供流程、禁止与推荐措辞、决策条件、表格和示例,能追溯其方法主张。缺少独立测试、行为评估结果、引用的代码证据格式、版本化变更记录以及事实与推断的明确分离;示例是说明性文本,不能证明实际效果。
触发条件和核心流程容易理解,覆盖清晰反馈、不清晰反馈、外部审查者、YAGNI、多项修改和推翻后纠正等场景。未明确所需输入格式、输出模板、工具兼容性、测试失败时的退出标准、成本或故障排查方式;“检查代码库”和“测试”也没有具体操作接口。
技能有名称和描述,仓库上下文提供 MIT 许可证、版本号和一般性更新/贡献说明,具备基本维护信号。技能自身没有版本、变更历史、维护负责人、响应目标或明确的技能更新路径;发布者未通过 FollowSkills 企业注册表验证,因此不提供额外治理加分。
- 不要把“先验证”误认为已验证:该文档未提供可执行测试或独立评估证据。
- 实施或发布任何修改前,应明确确认范围、权限、敏感信息边界和可回滚方案;这些要求未在技能中充分规定。
- “STOP”与“Just start working”适用场景不同,代理需要结合反馈是否清晰及人类授权判断,避免在未获授权时修改代码或回复外部审查线程。
它能做什么 & 适用场景
规范 Claude 收到代码评审意见后的反应方式:读完整反馈→用自己的话复述需求(或直接追问)→对照代码库现状核实→判断在这个代码库里是否技术上站得住脚→给出技术性回应(认同或有理据地反驳)→逐项落实并逐项测试。明确禁止诸如「你说得对!」「好建议!」这类没有验证过程的表演性附和,以及在没搞懂全部条目前就先做一部分、含糊带过剩下的。
把反馈来源分成「自己人(human partner)」和「外部评审者」两类区别处理:自己人的反馈默认可信但范围不清时仍要问清楚;外部评审的建议则要先核实是否符合这个代码库现状、是否会破坏现有功能、评审者是否掌握了完整上下文,站不住脚就据理反驳,验证不了就明说「我没法在不做 X 的情况下验证这个,要不要我去查/问/先做」。对于「实现得更专业」这类建议,会先 grep 代码库确认功能是否真的被用到,没用到就按 YAGNI 提出去掉而不是照单全收。多条反馈会先把不清楚的问题一次性问完,再按阻断性问题→简单修复→复杂修复的顺序逐项处理并逐项测试。
- 同事或外部评审给出一批修改建议,其中有几条含糊或存疑,需要先问清楚/核实再动手而不是照单全收
- 评审者建议「加上完整的指标追踪系统」,但代码库里这个接口其实没人调用,按 YAGNI 原则提出去掉而不是硬实现
- 评审意见和自己之前的判断冲突,需要用技术论据(现有测试、代码引用)来反驳而不是无脑接受
优缺点一览
- 把「先核实再实施」拆成具体的检查项(技术上是否成立、是否破坏现有功能、评审者是否有完整上下文),而不是一句笼统的「要批判性思考」
- 给出了大量正反例对照(表演性附和 vs 技术性核实),便于直接判断当前回应是否踩了坑
- 对「反馈不清楚」的处理方式很具体:全部问清楚再动手,而不是先做懂的部分、剩下的悄悄跳过
- 只在会自动读取和遵循 SKILL.md 指令的 harness 里可靠工作(Claude Code、Codex CLI 等);纯聊天式界面里等于一份说明文档,需要人工照做。
- 这本质是一份行为规范文档,效果高度依赖使用它的 harness 是否真的把 SKILL.md 当强约束执行,而不只是参考
- 「据理反驳」的前提是评审者的建议确实站不住脚;如果 Claude 自己对代码库的理解有偏差,反驳反而可能是错的一方坚持己见
如何安装
作为 obra/superpowers 插件的一部分安装,不能单独安装某一个 skill:
- Claude Code:官方市场
/plugin install superpowers@claude-plugins-official;或先注册 Superpowers 自建市场/plugin marketplace add obra/superpowers-marketplace,再/plugin install superpowers@superpowers-marketplace - 也支持 Antigravity、Codex App/CLI、Cursor、Factory Droid、GitHub Copilot CLI、Kimi Code、OpenCode、Pi 等 harness,具体命令见仓库 README
装好后这个 skill 会和其余 13 个 Superpowers skill 一起启用,由 using-superpowers 统一引导、按场景自动触发,无需单独调用。
如何使用
在 subagent-driven-development 或普通开发流程里收到评审反馈时自动触发。典型效果:面对「删掉这段遗留代码」的建议,Claude 会先检查兼容性目标版本再决定是保留还是删除,而不是直接照做。
对比同类
跟只在系统提示里写「认真对待代码评审」相比,这个 skill 给出了区分「自己人反馈」和「外部反馈」的不同处理策略,以及具体的核实步骤清单,减少了「无脑接受」和「无脑反驳」两个极端。