---
description: 仔细评审代码 — 调用 Oracle 进行多角度代码审查
---

# 代码审查

调用 Oracle Agent 深度审查（只读，不执行修改）。

**第一步：准备审查材料**

整理：需求方案、修改文件清单及改动说明、代码检查结果。

**第二步：调用 Oracle**

使用 `task(subagent_type="oracle")`，prompt 包含：

1. 完整审查材料
2. 审查维度：功能正确性、边界情况（空值/异常/并发）、代码规范（`AGENTS.md`）、安全性、可维护性、简洁性、兼容性、可读性（详细注释）
3. 心态：跳出计划者思路，不预设代码正确
4. 输出格式：无问题 → `APPROVED`；有问题 → 按严重程度（🔴 严重 / 🟡 建议 / 🔵 风格）列出

**第三步：处理结果**

- **APPROVED** → AI 自行判断是否需要：`/dev-feature`（有新增功能/命令时，先用一句话点明功能特性概要再引导）/ `/dev-experience`（学到了新经验时，先用一句话点明经验概要再引导）/ `/dev-commit`（始终可用）
- **代码级 bug** → 逐一验证 Oracle 问题 → 确实存在则修改，不确定则问用户 → 修复后重新执行 `/dev-fire` 第二步完整检查（静态检查 + 整体验证）→ 用 `session_id` 继续 Oracle 审查 → 循环直到 APPROVED
- **设计级问题** → 建议用户 `/dev-new` 重新分析

## 约束

- Oracle 是只读顾问，不执行修改
- 不确定时必须问用户，不擅自决定
- 修复后必须重新验证
- Oracle 风格建议与项目规范冲突时以项目规范为准