PR 代码审查

请对以下 PR 进行代码质量审查：

## PR 描述
{{pr_description}}

## 代码变更
{{pr_diff}}

请从以下维度进行详细审查：
1. **代码规范**：命名是否清晰、风格是否一致、是否符合项目约定
2. **逻辑正确性**：边界条件处理、错误处理是否完善、是否有逻辑漏洞
3. **可维护性**：代码复杂度、函数拆分是否合理、是否有重复代码
4. **可读性**：注释是否充分、代码意图是否清晰
5. **测试覆盖**：关键路径是否有测试、边界情况是否覆盖

请对每个维度给出评分（1-5）和具体问题列表。

请对以下 PR 进行安全审查：

## PR 描述
{{pr_description}}

## 代码变更
{{pr_diff}}

请重点检查以下安全风险：
1. **注入风险**：SQL 注入、XSS、命令注入等
2. **认证授权**：权限检查是否充分、是否存在越权风险
3. **数据安全**：敏感数据是否加密、日志是否泄露敏感信息
4. **依赖安全**：新引入的依赖是否有已知漏洞
5. **配置安全**：是否有硬编码密钥、不安全的默认配置

请对每个问题标注严重等级（高/中/低），并给出修复建议。

请对以下 PR 进行性能审查：

## PR 描述
{{pr_description}}

## 代码变更
{{pr_diff}}

请重点评估以下方面：
1. **算法效率**：时间复杂度和空间复杂度是否合理
2. **资源使用**：内存分配、文件句柄、数据库连接是否正确管理
3. **并发安全**：是否有竞态条件、死锁风险
4. **I/O 性能**：网络请求、数据库查询是否有 N+1 问题或不必要的重复调用
5. **缓存策略**：是否有适合缓存的场景未使用缓存

请对每个性能问题给出影响评估和优化建议。

综合三维度审查结果，给出最终结论。

## 代码质量
{{quality_report}}

## 安全审查
{{security_report}}

## 性能审查
{{perf_report}}

输出格式（markdown）：
### 结论：[✅ 可合并 / ⚠️ 需修改 / ❌ 需重写]

### 综合评分
| 维度 | 评分(1-5) | 关键发现 |
|------|----------|---------|
| 代码质量 | | |
| 安全性 | | |
| 性能 | | |

### 必须修改（Blocking）
1. [严重/高] 问题 → 修复建议

### 建议改进（Non-blocking）
1. 问题 → 优化方向

### 亮点
- 值得肯定的实践

代码审查员
请对以下 PR 进行代码质量审查：

## PR 描述
{{pr_description}}

## 代码变更
{{pr_diff}}

请从以下维度进行详细审查：
1. **代码规范**：命名是否清晰、风格是否一致、是否符合项目约定
2. **逻辑正确性**：边界条件处理、错误处理是否完善、是否有逻辑漏洞
3. **可维护性**：代码复杂度、函数拆分是否合理、是否有重复代码
4. **可读性**：注释是否充分、代码意图是否清晰
5. **测试覆盖**：关键路径是否有测试、边界情况是否覆盖

请对每个维度给出评分（1-5）和具体问题列表。

安全架构师
请对以下 PR 进行安全审查：

## PR 描述
{{pr_description}}

## 代码变更
{{pr_diff}}

请重点检查以下安全风险：
1. **注入风险**：SQL 注入、XSS、命令注入等
2. **认证授权**：权限检查是否充分、是否存在越权风险
3. **数据安全**：敏感数据是否加密、日志是否泄露敏感信息
4. **依赖安全**：新引入的依赖是否有已知漏洞
5. **配置安全**：是否有硬编码密钥、不安全的默认配置

请对每个问题标注严重等级（高/中/低），并给出修复建议。

性能基准师
请对以下 PR 进行性能审查：

## PR 描述
{{pr_description}}

## 代码变更
{{pr_diff}}

请重点评估以下方面：
1. **算法效率**：时间复杂度和空间复杂度是否合理
2. **资源使用**：内存分配、文件句柄、数据库连接是否正确管理
3. **并发安全**：是否有竞态条件、死锁风险
4. **I/O 性能**：网络请求、数据库查询是否有 N+1 问题或不必要的重复调用
5. **缓存策略**：是否有适合缓存的场景未使用缓存

请对每个性能问题给出影响评估和优化建议。

代码审查员
综合三维度审查结果，给出最终结论。

## 代码质量
{{quality_report}}

## 安全审查
{{security_report}}

## 性能审查
{{perf_report}}

输出格式（markdown）：
### 结论：[✅ 可合并 / ⚠️ 需修改 / ❌ 需重写]

### 综合评分
| 维度 | 评分(1-5) | 关键发现 |
|------|----------|---------|
| 代码质量 | | |
| 安全性 | | |
| 性能 | | |

### 必须修改（Blocking）
1. [严重/高] 问题 → 修复建议

### 建议改进（Non-blocking）
1. 问题 → 优化方向

### 亮点
- 值得肯定的实践