requesting-code-review

请求代码审查 (Requesting Code Review)

派发一个 code reviewer subagent，在问题级联之前捕获它们。reviewer 获得精确构建的用于评估的上下文——绝不是你会话的历史。这使 reviewer 聚焦于工作产物，而非你的思考过程，并为你继续工作保留自己的上下文。

核心原则： 尽早 review，频繁 review。

何时请求审查 (When to Request Review)

强制：

• subagent-driven development 中每个任务之后
• 完成主要功能之后
• 合并到 main 之前

可选但有价值：

• 卡住时（新鲜视角）
• 重构之前（基线检查）
• 修复复杂 bug 之后

如何请求 (How to Request)

1. 获取 git SHA：

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

2. 派发 code reviewer subagent：

派发一个 general-purpose subagent，填写 code-reviewer.md 中的模板

占位符：

• {DESCRIPTION} - 你构建内容的简要摘要
• {PLAN_OR_REQUIREMENTS} - 它应该做什么
• {BASE_SHA} - 起始 commit
• {HEAD_SHA} - 结束 commit

3. 根据反馈行动：

• 立即修复 Critical 问题
• 在继续之前修复 Important 问题
• Minor 问题记下留待以后
• 如果 reviewer 错了，提出反驳（附理由）

示例 (Example)

[刚完成任务 2：添加验证函数]

你：继续之前，让我请求代码审查。

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

[派发 code reviewer subagent]
  DESCRIPTION: 添加了 verifyIndex() 和 repairIndex()，含 4 种问题类型
  PLAN_OR_REQUIREMENTS: docs/superpowers/plans/deployment-plan.md 中的任务 2
  BASE_SHA: a7981ec
  HEAD_SHA: 3df7661

[Subagent 返回]：
  优点：架构清晰，测试真实
  问题：
    Important: 缺少进度指示器
    Minor: 报告间隔的魔法数字（100）
  评估：可以继续

你：[修复进度指示器]
[继续任务 3]

与工作流集成 (Integration with Workflows)

Subagent-Driven Development：

• 每个任务之后 review
• 在问题累积之前捕获它们
• 进入下一个任务之前修复

Executing Plans：

• 每个任务或自然检查点之后 review
• 获取反馈，应用，继续

临时开发（Ad-Hoc Development）：

• 合并前 review
• 卡住时 review

危险信号 (Red Flags)

绝不：

• 因为"很简单"而跳过 review
• 忽略 Critical 问题
• 带着未修复的 Important 问题继续
• 与有效的技术反馈争论

如果 reviewer 错了：

• 以技术理由反驳
• 展示证明它有效的代码/测试
• 请求澄清

见模板：code-reviewer.md