Code Review

---
name: code-review
description: 当用户要求进行代码审查、提到"review"/"审查"/"重构"/"优化代码"等关键词、或代码需要质量检查时使用。检查重复代码、逻辑错误、多余逻辑、SRP违反、模块耦合、硬编码/死代码等6个维度
---

## 目标
检查 6 个维度：重复代码、逻辑冲突、多余逻辑、SRP违反、模块耦合、硬编码/死代码

## 执行步骤
检查以下 6 个维度，用🔴严重/🟡警告/🟢建议分级：

1. **重复代码** - 相同或相似的代码逻辑在多处出现
2. **逻辑错误** - 存在矛盾、互相抵消、错误的逻辑
3. **多余逻辑** - 不必要的计算、判断或冗余代码
4. **SRP违反** - 单一职责原则违反，一个函数/类做了太多事情
5. **模块耦合** - 模块间依赖关系过于复杂或紧耦合
6. **硬编码/死代码** - 魔法数字、未使用的代码、永远不会执行的分支

## 输出格式
```markdown
## 问题 1 — 重复逻辑
- 位置: `src/foo.ts` 行 120-160；`src/bar.ts` 行 20-40
- 描述: 两处实现相同的数据转换逻辑
- 严重度: 🟡警告
- 推荐修复: 提取 `utils/transform.ts`，新增单元测试
```

## 注意
- 只进行观察和分析，不修改代码直到用户确认
- 优先报告严重问题（🔴），再是警告（🟡），最后是建议（🟢）
- 每个问题都要说明"为什么这是个问题"和"如何修复"
- 如果代码整体质量很好，也要明确指出