code-review

name	code_review
description	代码生成后的自动化审查技能。对新增/修改的代码执行项目规范合规检查。

Code Review — 代码生成后审查

触发条件（满足任一即执行）：

1. 完成了多文件代码生成
2. 完成了涉及架构边界的代码修改
3. 用户明确要求"审查/review/检查"
4. plan_mode 的计划执行完毕后

跳过条件：单行修改、纯配置文件修改、用户说"不需要审查"。

---

1 审查清单

1.1 ❌ 致命级（阻断性问题，必须修复）

#	检查项	检查方法
1	模块依赖方向违规	import 的包所属模块是否在 build.gradle.kts 的依赖中
2	使用了禁止模式	搜索 project_rule.md §3 中列出的模式
3	ARouter 路由未注册 @Route 注解	新页面是否有 @Route(path = "/模块/页面") 注解
4	主线程网络请求	网络请求是否在 ViewModel.viewModelScope 中发起
5	继承体系错误	新 Activity/Fragment 是否继承正确的 Base 类
6	硬编码密钥/隐私数据	代码中是否出现明文密钥
7	LiveData 观察未使用 viewLifecycleOwner	Fragment 中 observe 是否使用 viewLifecycleOwner
8	RecyclerView 使用 notifyDataSetChanged()	是否使用 ListAdapter + DiffUtil

1.2 ⚠️ 警告级（应修复，不阻断）

#	检查项	检查方法
7	硬编码字符串/颜色/尺寸	代码中是否直接写字符串/色值/dp/sp 值
8	未使用项目封装工具类	是否绕过 XLog/SPManager/ApiService 直接调用底层 API
9	JSON 解析无 try-catch	Gson/Moshi 解析是否被异常处理包裹
10	资源文件未以 shop_ 为前缀	新资源文件命名是否符合前缀规范

1.3 💡 建议级（可选优化）

#	检查项	检查方法
11	日志工具选择	是否使用 XLog 而非 Log.d/e
12	弱引用模式	回调中引用 Activity 是否用了弱引用
13	单例线程安全	新单例是否使用 lazy(SYNCHRONIZED)
14	代码复用	同类代码块是否已抽取
15	RecyclerView 优化	列表项是否使用 DiffUtil 进行局部更新

---

2 审查输出格式

## 代码审查报告

**审查范围**：[变更文件列表]
**总体结果**：✅ 通过 / ⚠️ N 项警告 / ❌ N 项致命

### ❌ 致命问题

**问题 1**：[标题]
- **位置**：`文件路径:行号`
- **违反规则**：[对应规则条目]
- **当前代码**：`违规代码片段`
- **修复方案**：`修复后代码片段`

### ⚠️ 警告

### 💡 优化建议

### ✅ 已验证通过

---

3 审查流程

代码生成完成
    ↓
[Step 0] 前置纠错检查 — 主动扫描已修改文件的存量合规性
    │         （委派 proactive-correction agent 执行维度 2 扫描）
    │         ├─ 发现致命违规 ──→ 立即修正后再进入审查
    │         └─ 无致命违规 ──→ 进入正式审查
    ↓
[Step 1] 收集变更文件列表
    ↓
[Step 2] 按致命 → 警告 → 建议顺序逐项检查
    ↓
[Step 3] 输出审查报告
    ↓
┌── ❌ 有致命问题 ──→ 告知用户，给出修复方案
│                      修正后触发 proactive-correction 验证修复效果
└── ✅/⚠️ ──→ 输出报告，继续后续任务

---

4 与 SubAgent 协作

• 前置纠错检查 → 委派 proactive-correction agent
• 架构合规 → 委派 arch-review agent
• 资源同步 → 委派 resource-sync agent
• 委派时机：前置纠错检查为必经步骤；其余为变更涉及 2+ 模块，或涉及资源文件