Cursor rule

1# 代码审查标准
2 
3## 目的
4 
5代码审查确保代码合并前的质量、安全性和可维护性。此规则定义何时以及如何进行代码审查。
6 
7## 何时审查
8 
9**强制审查触发条件：**
10 
11- 编写或修改代码后
12- 提交到共享分支之前
13- 更改安全敏感代码时（认证、支付、用户数据）
14- 进行架构更改时
15- 合并 pull request 之前
16 
17**审查前要求：**
18 
19在请求审查之前，确保：
20 
21- 所有自动化检查（CI/CD）已通过
22- 合并冲突已解决
23- 分支已与目标分支同步
24 
25## 审查检查清单
26 
27在标记代码完成之前：
28 
29- [ ] 代码可读且命名良好
30- [ ] 函数聚焦（<50 行）
31- [ ] 文件内聚（<800 行）
32- [ ] 无深层嵌套（>4 层）
33- [ ] 错误显式处理
34- [ ] 无硬编码密钥或凭据
35- [ ] 无 console.log 或调试语句
36- [ ] 新功能有测试
37- [ ] 测试覆盖率满足 80% 最低要求
38 
39## 安全审查触发条件
40 
41**停止并使用 security-reviewer 代理当：**
42 
43- 认证或授权代码
44- 用户输入处理
45- 数据库查询
46- 文件系统操作
47- 外部 API 调用
48- 加密操作
49- 支付或金融代码
50 
51## 审查严重级别
52 
53| 级别 | 含义 | 行动 |
54|-------|---------|--------|
55| CRITICAL（关键） | 安全漏洞或数据丢失风险 | **阻止** - 合并前必须修复 |
56| HIGH（高） | Bug 或重大质量问题 | **警告** - 合并前应修复 |
57| MEDIUM（中） | 可维护性问题 | **信息** - 考虑修复 |
58| LOW（低） | 风格或次要建议 | **注意** - 可选 |
59 
60## 代理使用
61 
62使用这些代理进行代码审查：
63 
64| 代理 | 用途 |
65|-------|--------|
66| **code-reviewer** | 通用代码质量、模式、最佳实践 |
67| **security-reviewer** | 安全漏洞、OWASP Top 10 |
68| **typescript-reviewer** | TypeScript/JavaScript 特定问题 |
69| **python-reviewer** | Python 特定问题 |
70| **go-reviewer** | Go 特定问题 |
71| **rust-reviewer** | Rust 特定问题 |
72 
73## 审查工作流
74 
75```
761. 运行 git diff 了解更改
772. 先检查安全检查清单
783. 审查代码质量检查清单
794. 运行相关测试
805. 验证覆盖率 >= 80%
816. 使用适当的代理进行详细审查
82```
83 
84## 常见问题捕获
85 
86### 安全
87 
88- 硬编码凭据（API 密钥、密码、令牌）
89- SQL 注入（查询中的字符串拼接）
90- XSS 漏洞（未转义的用户输入）
91- 路径遍历（未净化的文件路径）
92- CSRF 保护缺失
93- 认证绕过
94 
95### 代码质量
96 
97- 大函数（>50 行）- 拆分为更小的
98- 大文件（>800 行）- 提取模块
99- 深层嵌套（>4 层）- 使用提前返回
100- 缺少错误处理 - 显式处理
101- 变更模式 - 优先使用不可变操作
102- 缺少测试 - 添加测试覆盖
103 
104### 性能
105 
106- N+1 查询 - 使用 JOIN 或批处理
107- 缺少分页 - 给查询添加 LIMIT
108- 无界查询 - 添加约束
109- 缺少缓存 - 缓存昂贵操作
110 
111## 批准标准
112 
113- **批准**：无关键或高优先级问题
114- **警告**：仅有高优先级问题（谨慎合并）
115- **阻止**：发现关键问题
116 
117## 与其他规则的集成
118 
119此规则与以下规则配合：
120 
121- [testing.md](testing.md) - 测试覆盖率要求
122- [security.md](security.md) - 安全检查清单
123- [git-workflow.md](git-workflow.md) - 提交标准
124- [agents.md](agents.md) - 代理委托
125