Code Review

触发

• /review — 审查当前项目的最近改动
• /review <文件路径> — 审查指定文件
• "帮我 review"、"代码审查" 等自然语言

审查流程

1. 确定审查范围（git diff / 指定文件 / 整个项目）
2. 按 checklist 逐项检查
3. 输出审查报告（问题分级：🔴 严重 / 🟡 建议 / 🟢 良好）

Checklist

🔴 必须修复

• 安全漏洞（硬编码密钥、SQL 注入、路径穿越）
• 明显的 bug（类型错误、空指针、越界）
• 数据丢失风险（无事务、无备份）

🟡 建议改进

• 代码重复（DRY 违反）
• 函数过长（>50 行考虑拆分）
• 命名不清晰
• 缺少错误处理
• 缺少类型注解（Python）
• 缺少 docstring

🟢 架构层面

• 模块职责是否单一
• 接口设计是否清晰
• 依赖方向是否合理
• 是否有循环依赖

输出格式

## 审查报告: {文件/项目名}

### 🔴 严重问题 (N 个)
1. **[文件:行号]** 问题描述
   建议修复方案

### 🟡 改进建议 (N 个)
1. **[文件:行号]** 建议描述

### 🟢 总体评价
一句话总结代码质量和架构状态

快速命令

# 审查最近 commit 的改动
git diff HEAD~1 --stat
git diff HEAD~1 -- <有改动的文件>

# 审查未提交的改动
git diff --stat
git diff -- <文件>

# 检查 Python 代码质量
python3 -m py_compile <文件>
python3 -m pytest --tb=short