名称: 请求代码审查描述: 在完成任务、实现主要功能或在合并前验证工作是否符合要求时使用 - 调度超级能力:代码审查子代理根据计划或要求在继续之前审查实现

请求代码审查

调度超级能力:代码审查子代理以在问题级联前发现问题。

核心原则: 早期审查，频繁审查。

何时请求审查

强制性:

在子代理驱动开发的每个任务后
完成主要功能后
合并到主干前

可选但有价值:

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

如何请求

1. 获取git SHAs:

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

2. 调度代码审查子代理:

使用Task工具与超级能力:代码审查子代理类型，填写模板在 code-reviewer.md

占位符:

{WHAT_WAS_IMPLEMENTED} - 你刚刚构建了什么
{PLAN_OR_REQUIREMENTS} - 它应该做什么
{BASE_SHA} - 起始提交
{HEAD_SHA} - 结束提交
{DESCRIPTION} - 简要摘要

3. 根据反馈行动:

立即修复关键问题
在继续前修复重要问题
记录次要问题以后处理
如果评审者错误，请推送反馈（附理由）

示例

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

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

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

[调度超级能力:代码审查子代理]
  WHAT_WAS_IMPLEMENTED: 对话索引的验证和修复函数
  PLAN_OR_REQUIREMENTS: 来自 docs/plans/deployment-plan.md 的任务 2
  BASE_SHA: a7981ec
  HEAD_SHA: 3df7661
  DESCRIPTION: 添加了 verifyIndex() 和 repairIndex() 函数，包含 4 种问题类型

[子代理返回]:
  优点: 清晰的架构，真实测试
  问题:
    重要: 缺少进度指示器
    次要: 魔法数字 (100) 用于报告间隔
  评估: 准备继续

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

与工作流程的集成

子代理驱动开发:

每个任务后审查
在问题复合前发现问题
在移动到下一个任务前修复

执行计划:

每批任务 (3 个任务) 后审查
获取反馈，应用，继续

临时开发:

合并前审查
卡住时审查

红色标志

从不:

跳过审查因为“很简单”
忽略关键问题
在未修复重要问题时继续
与有效技术反馈争论

如果评审者错误:

附技术理由推送反馈
展示证明工作的代码/测试
请求澄清

参见模板在: requesting-code-review/code-reviewer.md

名称: 请求代码审查 描述: 在完成任务、实现主要功能或在合并前验证工作是否符合要求时使用 - 调度超级能力:代码审查子代理根据计划或要求在继续之前审查实现