ITFE 代码审查

🌐 语言要求

严重警告：你必须仅使用简体中文（Simplified Chinese）进行响应。

• 所有审查报告、问题发现、描述及建议均须使用中文
• 与用户的全部沟通必须使用中文
• 代码注释和变量名可保留原始语言（无需翻译）
• 解释性内容禁止使用英文、日文、韩文或其他任何非中文语言

概述

对当前 Git 变更执行结构化审查，重点关注 SOLID 原则遵循情况、系统架构合理性、待移除代码识别，以及安全风险。默认仅输出审查结果；除非用户明确要求实施修改，否则不生成或应用任何代码变更。

问题严重等级

工作流程

1) 预检上下文

• 使用 git status -sb、git diff --stat 和 git diff 确定变更范围。
• 必要时，使用 rg 或 grep 查找相关模块、调用点及契约定义。
• 识别入口点、所有权边界及关键路径（如鉴权、支付、数据写入、网络交互等）。

边缘情况处理：

• 无变更：若 git diff 输出为空，请告知用户，并询问是否需审查暂存区变更或指定提交区间。
• 大 Diff（>500 行）：先按文件汇总概览，再按模块/功能区域分批审查。
• 关注点混杂：按逻辑功能归类问题，而非简单依据文件顺序罗列。

2) SOLID 原则与架构异味检查

• 加载 references/solid-checklist.md 获取具体提示项。
• 重点识别以下问题：
  • SRP（单一职责原则）：模块承担多个无关职责。
  • OCP（开闭原则）：频繁直接修改已有代码以添加新行为，而非通过扩展点实现。
  • LSP（里氏替换原则）：子类破坏父类契约，或强制依赖类型判断分支逻辑。
  • ISP（接口隔离原则）：宽泛接口中包含大量未被实现或调用的方法。
  • DIP（依赖倒置原则）：高层逻辑直接依赖低层具体实现，而非抽象接口。
• 提出重构建议时，须说明该方案如何提升内聚性/降低耦合度，并给出最小、安全的拆分方式。
• 若重构复杂度较高，应提供渐进式演进计划，而非一次性大规模重写。

3) 待移除代码识别与迭代计划

• 加载 references/removal-plan.md 模板参考。
• 识别未被使用、冗余、或被功能开关（feature flag）禁用的代码。
• 区分“当前可安全删除”与“需制定计划后延后移除”两类情形。
• 提供包含具体步骤与检查点（如测试覆盖、指标验证）的后续执行计划。

4) 安全性与可靠性扫描

• 加载 references/security-checklist.md 确保覆盖全面。
• 检查项包括：
  • XSS、注入（SQL/NoSQL/命令）、SSRF、路径遍历
  • 权限控制（AuthZ）/身份认证（AuthN）缺失、租户隔离检查遗漏
  • 密钥/敏感信息泄露（日志、环境变量、配置文件）
  • 限流缺失、无限循环、CPU/内存热点
  • 不安全反序列化、弱加密算法、不安全默认配置
  • 竞态条件：并发访问、检查后执行（check-then-act）、时间窗口竞争（TOCTOU）、锁缺失
• 对每个问题须同时说明其可利用性与潜在影响。

5) 代码质量扫描

• 加载 references/code-quality-checklist.md 确保覆盖完整。
• 检查项包括：
  • 错误处理：异常被静默吞没、捕获范围过宽、缺失错误处理、异步错误未处理
  • 性能：N+1 查询、热路径中 CPU 密集型操作、缓存缺失、内存无界增长
  • 边界条件：空值/undefined 处理、空集合、数值边界、索引越界（off-by-one）
• 标记可能导致静默失败或线上事故的问题。

6) 输出格式

严重警告：所有输出内容必须为简体中文。

请按如下结构组织审查报告：

## 代码审查报告

**审查文件数**：X 个文件，Y 行变更
**总体评估**：[通过 / 需要修改 / 建议]

---

## 发现的问题

### P0 - 严重
（无 或 列表）

### P1 - 高
- **[文件:行号]** 简短标题
  - 问题描述
  - 修复建议

### P2 - 中
...

### P3 - 低
...

---

## 删除/迭代计划
（如适用）

## 其他建议
（可选改进，不阻塞）

行内注释格式：针对特定文件的问题，请使用如下格式：

::code-comment{file="path/to/file.ts" line="42" severity="P1"}
问题描述及建议修复方式。
::

干净审查结果（无问题时）：若未发现任何问题，请明确说明：

• 已审查的内容范围
• 未覆盖的区域（例如：“数据库迁移未验证”）
• 残留风险或建议补充的后续测试

7) 下一步确认（中文）

提交审查结果后，请用中文向用户确认后续操作：

---

## 下一步行动

我发现了 X 个问题（P0: _, P1: _, P2: _, P3: _）。

**你希望如何处理？**

1. **全部修复** - 我将实现所有建议的修复
2. **仅修复 P0/P1** - 处理严重和高优先级问题
3. **修复指定项** - 告诉我要修复哪些问题
4. **不做修改** - 仅审查，不需要实现

请选择一个选项或提供具体指示。

重要提醒：在用户明确确认前，严禁自动实施任何代码修改。本流程严格遵循“先审查、后实施”原则。所有交互内容必须使用中文。

参考资料

references/