name: review description: 全面的代码审查工作流 - 并行专业评审 → 综合
/review - 代码审查工作流
多视角代码审查，并行专家评审。
何时使用
- “审查这段代码”
- “审查我的PR”
- “合并前检查”
- “获取实现反馈”
- 合并重大更改前
- 质量门
工作流概述
┌──────────┐
│ critic │ ─┐
│ (code) │ │
└──────────┘ │
│
┌──────────┐ │ ┌──────────────┐
│plan-reviewer│ ─┼────▶ │ review-agent │
│ (plan) │ │ │ (synthesis) │
└──────────┘ │ └──────────────┘
│
┌──────────┐ │
│plan-reviewer│ ─┘
│ (change) │
└──────────┘
并行 顺序
视角 综合
代理序列
| # | 代理 | 焦点 | 执行 |
|---|---|---|---|
| 1 | critic | 代码质量、模式、可读性 | 并行 |
| 1 | plan-reviewer | 架构、计划遵循 | 并行 |
| 1 | plan-reviewer | 变更影响、风险评估 | 并行 |
| 2 | review-agent | 综合所有评审、最终裁决 | 之后 |
评审视角
- critic: 这是好代码吗？（风格、模式、可读性）
- plan-reviewer: 这符合设计吗？（架构、计划）
- plan-reviewer: 这个变更安全吗？（风险、影响、回归）
- review-agent: 整体评估和建议
执行
阶段1：并行评审
# 代码质量评审
Task(
subagent_type="critic",
prompt="""
评审代码质量：[SCOPE]
评估：
- 代码风格和一致性
- 使用的设计模式
- 可读性和可维护性
- 错误处理
- 测试覆盖
输出：问题列表及严重性（关键/主要/次要）
""",
run_in_background=true
)
# 架构评审
Task(
subagent_type="plan-reviewer",
prompt="""
评审架构对齐：[SCOPE]
检查：
- 遵循既定模式
- 匹配实施计划（如存在）
- 与系统设计一致
- 无架构违规
输出：对齐评估及关注点
""",
run_in_background=true
)
# 变更影响评审
Task(
subagent_type="plan-reviewer",
prompt="""
评审变更影响：[SCOPE]
评估：
- 变更的风险级别
- 影响的系统/组件
- 向后兼容性
- 潜在回归
- 安全含义
输出：风险评估及建议
""",
run_in_background=true
)
# 等待所有并行评审
[检查所有三个的TaskOutput]
阶段2：综合
Task(
subagent_type="review-agent",
prompt="""
综合评审：[SCOPE]
评审：
- critic: [代码质量发现]
- plan-reviewer: [架构发现]
- plan-reviewer: [变更影响发现]
创建最终评审：
- 整体裁决（批准/请求更改/需要讨论）
- 优先行动项
- 阻塞与非阻塞问题
- PR描述摘要
"""
)
评审模式
完整评审
用户: /review
→ 所有四个代理，全面评审
快速评审
用户: /review --quick
→ 仅critic，快速反馈
安全焦点
用户: /review --security
→ 在并行阶段添加aegis（安全代理）
PR评审
用户: /review PR #123
→ 获取PR差异，评审变更
示例
用户: /review 认证变更
Claude: 启动/review工作流...
阶段1: 运行并行评审...
┌────────────────────────────────────────────┐
│ critic: 评审代码质量... │
│ plan-reviewer: 检查架构... │
│ plan-reviewer: 评估变更影响... │
└────────────────────────────────────────────┘
critic: 发现2个问题
- [次要] auth.ts中的错误消息不一致
- [主要] login()中缺少输入验证
plan-reviewer: ✅ 匹配认证计划
plan-reviewer: 中等风险
- 影响: 登录、注册、密码重置
- 破坏性变更: 会话令牌格式
阶段2: 综合...
┌─────────────────────────────────────────────┐
│ 评审摘要 │
├─────────────────────────────────────────────┤
│ 裁决: 请求更改 │
│ │
│ 阻塞: │
│ 1. 为login()添加输入验证 │
│ │
│ 非阻塞: │
│ 2. 标准化错误消息 │
│ │
│ 备注: │
│ - 记录会话令牌格式变更 │
│ - 考虑现有会话的迁移路径 │
└─────────────────────────────────────────────┘
裁决
- 批准: 准备合并，所有问题都是次要
- 请求更改: 必须修复阻塞问题
- 需要讨论: 架构决策需要输入