quick-review
# 快速代码审查
审查当前代码变更,提供简洁、可操作的反馈。
## 步骤
1. 执行 `git diff --staged` 获取暂存区的变更。如果为空,回退到 `git diff` 获取未暂存的变更。如果两者都为空,告知用户没有可审查的变更。
2. 按以下分类分析 diff:
**Bug 与逻辑错误**
- 边界错误、空值访问、竞态条件
- 布尔逻辑错误或遗漏的边界情况
- 资源泄漏(未关闭的句柄、缺失的清理逻辑)
**安全问题**
- 注入漏洞(SQL、XSS、命令注入)
- 硬编码的密钥或凭证
- 不安全的反序列化或输入处理
**性能问题**
- 热路径中不必要的内存分配
- N+1 查询或缺失的索引
- 异步上下文中的阻塞调用
**代码质量**
- 死代码或不可达分支
- 应该复用的重复逻辑
- 具有误导性的命名或不清晰的意图
3. 按文件分组展示发现的问题,格式如下:
### `path/to/file.ext`
- **[Bug]** 第 42 行:问题描述及修复建议
- **[安全]** 第 78 行:问题描述及修复建议
4. 最后用一句话总结:发现的问题总数,以及这些变更是否可以安全提交。
## 准则
- 只标记真实问题,不要挑剔代码风格、格式或提出不必要的建议。
- 要具体——引用 diff 中的确切行号和变量名。
- 如果变更没有问题,简要说明即可,不要凭空制造问题。
- 保持审查简洁,重质量不重数量。
标签
skill
ai