SuperAPI AI 工具集

requesting-code-review

编程与开发

查看项目

更新于 10-16 09:13

技能介绍

用途

Requesting Code Review 用于在完成开发任务、实现重大功能或合并前,验证工作是否符合需求。其核心原则是:尽早审查、频繁审查(Review early, review often)。

主要能力

该技能会派发一个代码审查子代理,在问题级联扩散之前将其捕获。审查者获得的是为评估精确构造的上下文,而不是当前会话的历史记录。这样可以让审查者专注于工作产出,而不是你的思考过程,同时也保留你自己的上下文以便继续工作。

何时请求审查

强制性的时机:

  • 子代理驱动开发中,每个任务完成后
  • 完成重大功能后
  • 合并到主分支(main)前

可选但有价值的时机:

  • 卡住时(获取新视角)
  • 重构前(基线检查)
  • 修复复杂 Bug 后

使用方式

第一步,获取 git SHA:

BASE_SHA=$(git rev-parse HEAD~1)  # 或 origin/main
HEAD_SHA=$(git rev-parse HEAD)

第二步,派发代码审查子代理:

使用 Task 工具,类型为 general-purpose,填写 code-reviewer.md 中的模板。

占位符:

  • {DESCRIPTION} - 对所构建内容的简要总结
  • {PLAN_OR_REQUIREMENTS} - 它应该做什么
  • {BASE_SHA} - 起始提交
  • {HEAD_SHA} - 结束提交

第三步,根据反馈行动:

  • 立即修复 Critical 问题
  • 继续之前先修复 Important 问题
  • 记录 Minor 问题以便后续处理
  • 如果审查者有误,带理由进行反驳

示例

[刚完成 Task 2:添加验证函数]

你:让我继续之前先请求代码审查。

BASE_SHA=$(git log --oneline | grep "Task 1" | head -1 | awk '{print $1}')
HEAD_SHA=$(git rev-parse HEAD)

[派发代码审查子代理]

  • DESCRIPTION: 添加了 verifyIndex() 和 repairIndex(),包含 4 种问题类型
  • PLAN_OR_REQUIREMENTS: 来自 docs/superpowers/plans/deployment-plan.md 的 Task 2
  • BASE_SHA: a7981ec
  • HEAD_SHA: 3df7661

[子代理返回]:

  • 优点:架构清晰,有真实的测试
  • 问题:
    • Important:缺少进度指示器
    • Minor:报告间隔使用了魔法数字(100)
  • 评估:可以继续推进

你:[修复进度指示器] [继续到 Task 3]

与工作流的集成

子代理驱动开发:

  • 在每一个任务之后审查
  • 在问题叠加之前捕获它们
  • 在进入下一个任务之前修复

执行计划:

  • 在每个任务之后或自然检查点进行审查
  • 获取反馈、应用、继续

临时开发:

  • 合并前审查
  • 卡住时审查

危险信号(Red Flags)

绝不要:

  • 因为“它很简单”就跳过审查
  • 忽略 Critical 问题
  • 在 Important 问题未修复的情况下继续
  • 与有效的技术反馈争辩

如果审查者有误:

  • 带技术理由进行反驳
  • 展示能证明其可用的代码/测试
  • 请求澄清

限制

模板位于:requesting-code-review/code-reviewer.md。关于具体安装方式、支持的平台细节、收费与价格信息,现有资料暂未提供。