code-review

name	code-review
description	代码评审指南。当用户提交一段代码让你评审 / 重构 / 找 bug 时启用，按结构化清单检查。
license	MIT

代码评审指南

按下面的清单逐项检查，发现什么说什么，不要遗漏也不要编造。

评审清单（按优先级）

1. 正确性（最高优先级）

• 边界情况：空列表 / None / 负数 / 极大值 / Unicode
• 错误处理：失败是否被静默吞掉？异常层级对不对？
• 并发安全：共享状态、race condition、死锁
• 资源释放：文件、连接、锁是否被关闭

2. 安全性

• 输入校验：SQL / command injection / path traversal
• 秘密泄露：硬编码 key / 日志打印 token
• 权限边界：是否越权读写

3. 可读性

• 命名：动词+名词动词、变量语义清晰
• 函数长度：> 40 行考虑拆分
• 嵌套深度：> 3 层考虑早返回
• 注释：解释"为什么"而不是"是什么"

4. 性能

• N+1 查询
• 不必要的循环 / allocation
• 算法复杂度（仅当量级明显不对时才提）

5. 测试

• 覆盖率不光看数字，看关键分支有没有测
• 是否有测试能复现 bug
• Mock 是否过度导致测试失去意义

输出模板

## 总体评价
<1-2 句话总体结论>

## 必须修（Must fix）
- [ ] <点 1：文件:行号 — 描述 — 建议>

## 建议改（Nice to have）
- [ ] ...

## 可以借鉴的地方
<诚恳点出优点>

禁止：不要泛泛说"考虑重构"——要具体指出哪行、为什么、怎么改。