持续代码审查助手
在任务、重大功能和合并前调度审查,尽早发现代码问题。
证据显示审查上下文被明确限定,模板要求只读检查且不得修改工作树、索引、HEAD或分支状态,降低了权限和数据泄露风险。扣分原因是未要求用户确认,未说明敏感代码如何处理、外部子代理的数据边界、反馈变更的回滚流程,也未提供选定来源的具体归属证据;未验证发布者身份本身不额外扣分。
指令流程基本一致:获取Git SHA、派发通用子代理、按严重性处理反馈;模板还规定了只读行为和结构化输出。扣分原因是依赖可用的通用子代理、Git环境和未随本技能提供的模板路径,缺少异常输入、命令失败、子代理不可用及失败反馈的处理;静态评估不超过10分。
技能明确面向完成任务、重大功能、合并前以及卡住或重构前的代码审查,并区分强制和可选触发场景。扣分原因是未定义不适用范围、输入完整性要求、不同宿主环境的兼容边界或中文使用支持;语义触发虽清楚,但依赖具备子代理能力的编码代理。
文档结构清晰,包含触发条件、步骤、占位符、工作流集成、红旗规则和完整模板;仓库上下文提供MIT许可证及版本信息。扣分原因是技能本身没有安装/依赖说明、版本变更记录、维护负责人或更新路径,且引用的code-reviewer.md未在该技能文件内展开其可达性保证。
该技能能将代码审查任务转化为基于需求、Git范围、质量、架构、测试和生产就绪性的结构化审查,并要求按严重性给出修复建议,核心价值明确。扣分原因是示例输出仍是示意性文本,未有该技能关键路径的实际执行证据,且最终修复与验证仍需用户或代理继续完成;静态评估上限为7分。
技能提供可审计的Git基线/头部提交、diff命令、明确检查项和结构化输出格式,具备有限的静态可追溯性。扣分原因是没有随技能提供测试套件、真实CI覆盖或第三方执行结果,结论无法独立复现验证;静态评估上限为5分。
- 该技能会把代码差异和需求上下文发送给子代理;使用前应确认代码、凭据或专有信息不会超出允许的数据边界。
- 技能依赖Git、子代理调度能力及code-reviewer.md模板,未定义这些依赖不可用时的降级和可诊断错误路径。
- “必须审查”属于流程约束,可能增加成本;应由用户确认适用于当前仓库和合并流程。
这个 Skill 能做什么,适合哪些场景?
这是 Superpowers 软件开发方法论中的代码审查技能,专注于请求代码审查,而不是直接实现功能。它要求在子代理驱动开发的每个任务后、重大功能完成后以及合并到主分支前进行审查。技能会为审查子代理提供精确的工作上下文,使其基于变更范围和需求评估工作成果。审查反馈按 Critical、Important 和 Minor 分级,并指导后续修复或推进决策。
运行 git rev-parse 获取基准提交和当前提交的 SHA;根据 code-reviewer.md 模板调度 general-purpose 代码审查子代理;向子代理传入已完成工作的描述、计划或需求、BASE_SHA 和 HEAD_SHA;接收审查结果并按严重程度处理问题:立即修复 Critical 问题,在继续前修复 Important 问题,记录 Minor 问题,或针对错误反馈提出技术性异议。
- 采用子代理驱动开发的团队在每个工程任务完成后验证变更。
- 开发者完成重大功能后,希望在继续工作前获得独立审查。
- 准备合并主分支的贡献者,需要检查实现是否符合需求。
- 遇到复杂问题或准备重构的开发者,需要获得新的审查视角。
这个 Skill 有哪些优点和局限?
- 明确规定了任务后、重大功能后和合并前的审查时机。
- 只向审查者提供精心准备的上下文,减少对原始思考过程的依赖。
- 按严重程度处理反馈,并明确 Critical 问题会阻塞推进。
- 采用 MIT 许可证的 Superpowers 集合提供该技能。
- 需要可用的 general-purpose 子代理和 code-reviewer.md 模板。
- 提供的材料没有说明该技能的独立安装方式。
- 技能本身不实现修复,也没有提供审查结果、测试套件或自动修复证据。
- 依赖 Git 提供提交 SHA;对非 Git 工作流的支持未被说明。
如何安装这个 Skill?
该技能位于 obra/superpowers 仓库的 skills/requesting-code-review/SKILL.md 中,不能从提供的材料确认独立安装步骤。README 说明 Superpowers 集合需按使用的 coding agent 分别安装;例如 Codex CLI 可运行 /plugins,搜索 superpowers,并选择 Install Plugin。使用多个 harness 时需分别安装。
如何使用这个 Skill?
在完成任务、重大功能或准备合并时触发,例如:“我刚完成用户认证功能,请在合并前请求代码审查。”技能会获取 BASE_SHA=$(git rev-parse HEAD~1)(或 origin/main)和 HEAD_SHA=$(git rev-parse HEAD),然后按 code-reviewer.md 调度审查子代理。审查返回后,修复 Critical 和 Important 问题,再决定是否继续。