Requesting Code Review(发起代码评审)
派一个独立上下文的评审子代理来审查改动,而不是让主会话自己一边写一边给自己打分
该技能针对完成任务、重大功能和合并前的代码审查,能通过隔离上下文的审查子代理及按严重性处理反馈来降低缺陷扩散风险。目标受众和触发时机清晰,但适用边界、审查失败时的替代流程以及不同项目类型的适配说明不足。
流程包含获取 BASE_SHA/HEAD_SHA、填充审查模板、按 Critical/Important/Minor 处理反馈,并要求只读审查,具有一定可重复性。证据未说明子代理不可用、Git 历史不足、提交范围错误、模板文件缺失或审查结果不完整时如何处理,也没有该技能自身的自动化行为测试。
审查模板明确要求只读,不修改工作树、索引、HEAD 或分支状态,并建议使用独立 worktree 检查其他修订,降低了直接改动风险。技能仍会把代码变更、需求和 Git 差异发送给子代理,未说明敏感信息脱敏、用户同意、访问权限边界、外部服务数据流或反馈修复前后的回滚策略。
技能要求审查 Git diff、计划对齐、错误处理、安全性、测试和生产准备度,并要求文件行号、影响和修复建议,具备基本可追溯性。静态材料没有显示该技能的执行测试、独立评估或实际审查样例;模板中的测试与合并结论是期望输出,不是已验证事实,且未明确区分观察结果与推断。
触发条件、三步操作、占位符和预期审查输出都较明确,用户可以理解何时调用以及如何处理反馈。仍缺少子代理调度的具体接口/配置、输入缺失时的处理、成本与耗时提示、跨代理兼容性要求、故障排查以及在审查无法完成时的退出标准。
技能有稳定的名称和描述,且相邻模板定义了较完整的审查格式。选定技能本身没有版本、变更记录、维护者、响应承诺或更新路径;仓库提供 MIT 许可和版本信息,但发布者 provenance 未经企业注册表验证,不能据此增加治理或维护分数。
- 发送代码、需求和 Git diff 前,应确认其中不含凭据、个人数据或机密,并明确子代理及其运行环境的访问范围。
- 不要把模板要求的“测试通过”或“可合并”当作已验证事实;当前评估未执行该技能或任何独立测试。
- 应补充审查子代理不可用、Git SHA 无效、模板缺失或反馈冲突时的停止、重试和人工接管流程。
它能做什么 & 适用场景
在完成一项任务、一个大功能,或合并主干前,派发一个只拿到必要上下文(描述、需求、起止 commit SHA)的评审子代理,而不是自己继续用同一个上下文自我评审。这样评审关注的是产出物本身而不是主会话的思考过程,也不占用主会话的上下文。文档很短,核心是「什么时候必须评审」「怎么派发」「怎么处理反馈」三段。
先取出起止 commit 的 SHA;用 code-reviewer.md 模板派发一个 general-purpose 子代理,填入改动描述、需求/计划、起止 SHA 四个占位符;根据评审子代理返回的反馈按严重程度处理——Critical 立刻修,Important 在继续前修,Minor 记录留待后续;如果认为评审子代理判断错了,允许带着技术理由反驳。列出了「必须评审」的场景(subagent-driven-development 每个任务之后、完成大功能后、合并主干前)和「建议评审」的场景(卡壳时、重构前建立基线、修完复杂 bug 后)。
- subagent-driven-development 流程里每完成一个任务,派发一次子代理评审再继续下一个任务
- 合并到主干前,用独立上下文的评审子代理做一次把关,而不是自己审自己写的代码
- 卡壳时主动请一次评审,借助「新鲜视角」换个人(子代理)看问题
优缺点一览
- 评审子代理拿到的是精心构造的上下文而不是主会话完整历史,评审更聚焦在改动本身
- 给出了具体的严重度处理顺序(Critical 立即修/Important 继续前修/Minor 记录后续),避免「有问题但先放着」变成无限拖延
- 明确允许对评审子代理的判断提出技术性反驳,不是单向的「评审说了算」
- 只在会自动读取和遵循 SKILL.md 指令的 harness 里可靠工作(Claude Code、Codex CLI 等);纯聊天式界面里等于一份说明文档,需要人工照做。
- 依赖 code-reviewer.md 这个模板文件(skill 目录内),如果只看主 SKILL.md 而没找到模板,流程会不完整
- 评审质量取决于子代理本身的能力和给到的上下文是否完整,模板填得潦草,评审也会流于表面
如何安装
作为 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 统一引导、按场景自动触发,无需单独调用。
如何使用
在完成任务节点时被自动或主动调用,核心动作是取 git SHA 区间、填 code-reviewer.md 模板、派发子代理,收到反馈后按严重度分级处理。
对比同类
跟没有独立评审、靠模型自己在同一上下文里做完再回头检查一遍相比,这个 skill 通过「独立子代理 + 精心构造的评审上下文」提供了更接近真实团队代码评审的隔离性;跟 subagent-driven-development 里内置的两阶段评审相比,这个 skill 更通用、可以在任意开发流程的节点上单独调用。