开发与工程 workflowcode-reviewsuperpowers

Receiving Code Review(接收代码评审反馈)

禁止「你说得对!」式的表演性附和,要求先核实反馈再决定实施还是据理反驳

FollowSkills 评估 · FSRS-1.0
谨慎使用
65/ 100
价值适配16 / 20

该技能针对“接收代码审查反馈并决定是否实施”的常见开发任务,明确要求先理解、验证和评估,再回应和实施;对避免盲目修改、处理不清晰意见、执行 YAGNI 检查具有实际价值。边界主要停留在代码审查接收阶段,未覆盖完整的变更管理或特定技术栈。

可靠性13 / 20

步骤顺序清晰,并要求不清晰时停止、逐项实施、逐项测试和检查回归;对无法验证的情况也要求说明限制并请求方向。但技能没有自带可执行测试、输入输出契约、成功判定标准或针对不同代理工具的可重复验证流程,较依赖代码库、测试环境和人类判断。

安全性17 / 25

核心行为降低了盲目实施、误删代码和错误架构变更的风险,要求核查兼容性、既有功能、上下文和安全问题,并在与人类伙伴的决策冲突时先讨论。证据中没有明确的最小权限、修改前备份、回滚步骤、敏感数据处理或外部操作确认要求;GitHub 回复示例也未定义权限和发布前确认边界。

证据7 / 15

技能提供流程、禁止与推荐措辞、决策条件、表格和示例,能追溯其方法主张。缺少独立测试、行为评估结果、引用的代码证据格式、版本化变更记录以及事实与推断的明确分离;示例是说明性文本,不能证明实际效果。

易用性7 / 10

触发条件和核心流程容易理解,覆盖清晰反馈、不清晰反馈、外部审查者、YAGNI、多项修改和推翻后纠正等场景。未明确所需输入格式、输出模板、工具兼容性、测试失败时的退出标准、成本或故障排查方式;“检查代码库”和“测试”也没有具体操作接口。

维护性5 / 10

技能有名称和描述,仓库上下文提供 MIT 许可证、版本号和一般性更新/贡献说明,具备基本维护信号。技能自身没有版本、变更历史、维护负责人、响应目标或明确的技能更新路径;发布者未通过 FollowSkills 企业注册表验证,因此不提供额外治理加分。

证据充分度: 评估于 2026年7月15日 审查版本 d884ae04edeb
使用前请注意
  • 不要把“先验证”误认为已验证:该文档未提供可执行测试或独立评估证据。
  • 实施或发布任何修改前,应明确确认范围、权限、敏感信息边界和可回滚方案;这些要求未在技能中充分规定。
  • “STOP”与“Just start working”适用场景不同,代理需要结合反馈是否清晰及人类授权判断,避免在未获授权时修改代码或回复外部审查线程。
评估证据 [1][2][3][4][5][6][7]
查看完整评分方法 →

它能做什么 & 适用场景

规范 Claude 收到代码评审意见后的反应方式:读完整反馈→用自己的话复述需求(或直接追问)→对照代码库现状核实→判断在这个代码库里是否技术上站得住脚→给出技术性回应(认同或有理据地反驳)→逐项落实并逐项测试。明确禁止诸如「你说得对!」「好建议!」这类没有验证过程的表演性附和,以及在没搞懂全部条目前就先做一部分、含糊带过剩下的。

把反馈来源分成「自己人(human partner)」和「外部评审者」两类区别处理:自己人的反馈默认可信但范围不清时仍要问清楚;外部评审的建议则要先核实是否符合这个代码库现状、是否会破坏现有功能、评审者是否掌握了完整上下文,站不住脚就据理反驳,验证不了就明说「我没法在不做 X 的情况下验证这个,要不要我去查/问/先做」。对于「实现得更专业」这类建议,会先 grep 代码库确认功能是否真的被用到,没用到就按 YAGNI 提出去掉而不是照单全收。多条反馈会先把不清楚的问题一次性问完,再按阻断性问题→简单修复→复杂修复的顺序逐项处理并逐项测试。

  1. 同事或外部评审给出一批修改建议,其中有几条含糊或存疑,需要先问清楚/核实再动手而不是照单全收
  2. 评审者建议「加上完整的指标追踪系统」,但代码库里这个接口其实没人调用,按 YAGNI 原则提出去掉而不是硬实现
  3. 评审意见和自己之前的判断冲突,需要用技术论据(现有测试、代码引用)来反驳而不是无脑接受

优缺点一览

优点
  • 把「先核实再实施」拆成具体的检查项(技术上是否成立、是否破坏现有功能、评审者是否有完整上下文),而不是一句笼统的「要批判性思考」
  • 给出了大量正反例对照(表演性附和 vs 技术性核实),便于直接判断当前回应是否踩了坑
  • 对「反馈不清楚」的处理方式很具体:全部问清楚再动手,而不是先做懂的部分、剩下的悄悄跳过
局限
  • 只在会自动读取和遵循 SKILL.md 指令的 harness 里可靠工作(Claude Code、Codex CLI 等);纯聊天式界面里等于一份说明文档,需要人工照做。
  • 这本质是一份行为规范文档,效果高度依赖使用它的 harness 是否真的把 SKILL.md 当强约束执行,而不只是参考
  • 「据理反驳」的前提是评审者的建议确实站不住脚;如果 Claude 自己对代码库的理解有偏差,反驳反而可能是错的一方坚持己见

如何安装

作为 obra/superpowers 插件的一部分安装,不能单独安装某一个 skill:

  1. Claude Code:官方市场 /plugin install superpowers@claude-plugins-official;或先注册 Superpowers 自建市场 /plugin marketplace add obra/superpowers-marketplace,再 /plugin install superpowers@superpowers-marketplace
  2. 也支持 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 给出了区分「自己人反馈」和「外部反馈」的不同处理策略,以及具体的核实步骤清单,减少了「无脑接受」和「无脑反驳」两个极端。

常见问题

为什么要禁止说「你说得对」?
SKILL.md 的立场是:这类表达是没有验证过程的表演性附和,真正该做的是核实后直接修复代码,用代码本身说明听懂了反馈,而不是用语言表态。

同仓库的其他 Skills

均来自 obra/superpowers

开发与工程

Test-Driven Development(测试驱动开发)

铁律:没有先写失败测试就不能写生产代码,写了也要删掉重来,不允许「补测试」

开发与工程

Brainstorming(需求澄清与设计)

写代码前先把需求逼问清楚,禁止在没有确认设计前直接动手实现

开发与工程

Verification Before Completion(完成前必须验证)

没有当次运行过的验证证据,就不能说「完成了」「测试通过了」——包括「应该」「看起来」这类措辞

开发与工程

Finishing a Development Branch(收尾开发分支)

开发完成后先验证测试通过,再用固定的 4 选项菜单让人决定合并/PR/保留/丢弃,并按规则清理 worktree

开发与工程

Requesting Code Review(发起代码评审)

派一个独立上下文的评审子代理来审查改动,而不是让主会话自己一边写一边给自己打分

开发与工程

Systematic Debugging(系统化调试)

任何 bug 都必须先完成根因排查再提修复方案,明确禁止「先随手改一下试试」

开发与工程

Writing Plans(撰写实现计划)

把需求拆成假设执行者对代码库一无所知也能照做的、每步 2-5 分钟的具体任务,杜绝「TBD」「补充细节」类占位符

开发与工程

Subagent-Driven Development(子代理驱动开发)

同一会话内为每个任务派发全新子代理实现,配两阶段审查,用进度台账防止压缩后重复劳动

开发与工程

Executing Plans(分会话执行实现计划)

在没有子代理能力的场景下,按人工检查点批量执行一份已写好的实现计划

开发与工程

Using Git Worktrees(隔离工作区管理)

开始新功能开发前,优先用平台原生隔离工具,没有的话再退回手动 git worktree

开发与工程

Dispatching Parallel Agents(并行子代理调度)

面对多个互不依赖的问题时,一次性并行派发多个子代理分别处理,而不是串行排队

开发与工程

Writing Skills(撰写与测试 Skill 本身)

把 TDD 的红绿重构套用到「写 Skill 文档」这件事上:先用无 skill 基线测试暴露真实失败模式,再针对性写文档,而不是凭直觉写

开发与工程

Using Superpowers(Superpowers 入口引导)

整套 Superpowers 方法论的启动引导 skill,规定「只要有 1% 可能相关就必须调用对应 skill」

相关 Skills