/cr

name: cr
description: Use when the user invokes /cr, requests a code review, or before committing to verify correctness, security, and quality of new or modified code in the working tree.

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