| name | code-review |
| description | Conduct thorough, constructive code reviews for quality, security, and best practices. Use when reviewing pull requests, checking code quality, identifying bugs, or auditing security. |
Review Framework
当进行代码审查时，请遵循以下框架：
1. 正确性检查 (Correctness)
- 边界情况和极端输入是否被处理？
- 错误处理是否完善？
- 是否有潜在的空指针/数组越界？
- 断言是否合理？
2. 安全检查 (Security)
- SQL 注入：检查是否使用参数化查询
- XSS：检查用户输入是否被正确转义
- 敏感信息：检查是否硬编码了密钥、密码、API tokens
- 文件操作：检查路径遍历漏洞
- 输入验证：检查是否有必要的验证
3. 性能分析 (Performance)
- 时间复杂度是否合理？
- 是否有 N+1 查询问题？
- 是否有不必要的重复计算？
- I/O 操作是否可优化？
4. 可维护性 (Maintainability)
- 代码是否符合 SOLID 原则？
- 命名是否清晰直观？
- 是否有重复代码可以提取？
- 注释是否充分？
5. 测试覆盖 (Testing)
- 核心逻辑是否有测试？
- 边界情况是否被覆盖？
- 测试是否真正验证功能而非假装通过？
Output Format
请按以下格式输出审查结果：
## 审查摘要
[总体评价]
## 🔴 严重问题
[必须修复的问题]
## 🟡 建议改进
[建议但非强制的改进]
## 🟢 优点
[代码中做得好的地方]
## 最终判定
- [ ] 可以合并
- [ ] 需要修改后重审
