Imported from ZhangShenao/harness9 (
skills/cr/SKILL.md). Install upstream withnpx skills add ZhangShenao/harness9 --skill cr. Copyright stays with the author.
cr — Code Review
Overview
对当前工作区所有新增或修改的代码执行详细的 Code Review，按严重级别输出问题，不做任何修改。
执行步骤
1. 收集变更范围
git status # 查看新增 / 修改 / 删除的文件列表
git diff # 未暂存的改动
git diff --cached # 已暂存的改动将两者合并视为本次 Review 的完整范围。
2. 逐文件审查
对每个变更文件，依次检查：
| 维度 | 检查点 |
|---|---|
| 正确性 | 逻辑错误、边界条件、空值/类型异常 |
| 安全性 | SQL 注入、XSS、命令注入、敏感信息硬编码、不安全的默认值 |
| 可维护性 | 函数/类职责单一、命名清晰、不必要的复杂度 |
| 性能 | N+1 查询、不必要的循环、大对象复制 |
| 测试覆盖 | 关键路径是否有对应测试，新代码是否破坏现有测试 |
| 依赖安全 | 新引入的第三方包是否合理，版本是否锁定 |
3. 检查敏感文件
若 git status 中出现以下类型的文件，单独标注，不得进入后续提交：
.env、*.pem、*credentials*、*secret*- 包含明文密码、API Key 的配置文件
4. 输出 Review 报告
按以下格式输出，无问题的级别可省略：
## Code Review 报告
### 🔴 Critical（必须修复，阻断提交）
- `文件路径:行号` — 问题描述
### 🟡 Warning（建议修复）
- `文件路径:行号` — 问题描述
### 🔵 Suggestion（可选优化）
- `文件路径:行号` — 建议描述
### ✅ 总结
- 变更文件数：N
- 通过提交：是 / 否（存在 Critical 问题时为否）
注意事项
- 只审查，不修改代码
- Critical 问题存在时，明确说明不建议提交，等待用户确认修复
- 若变更集为空（
git status无输出），告知用户当前无需 Review
