如果在 git 仓库内且未指定 PR

bash

获取当前分支的 PR

Step 2: 分析变更内容

获取 diff 后，按以下维度分析：

变更概览

📊 变更概览

• - 文件数: X 个文件变更
• 新增: +X 行
• 删除: -X 行
• 涉及模块: [列出模块/目录]

文件分类

将变更文件分为以下类别：

Step 3: 安全审查

对每个变更文件检查以下安全维度：

安全检查清单

markdown
🔐 安全审查

认证/授权 (Auth)

• - [ ] 是否有未授权的端点暴露？
• [ ] 权限检查是否完整？
• [ ] Token/Session 处理是否安全？

数据暴露 (Data Exposure)

• - [ ] 是否有敏感数据泄露到日志？
• [ ] API 响应是否暴露内部信息？
• [ ] 错误消息是否泄露系统细节？

注入风险 (Injection)

• - [ ] SQL 注入：是否使用参数化查询？
• [ ] XSS：用户输入是否正确转义？
• [ ] 命令注入：是否有未经验证的 shell 执行？
• [ ] 路径遍历：文件路径是否验证？

加密与密钥 (Crypto)

• - [ ] 是否使用了弱加密算法？
• [ ] 密钥是否硬编码？
• [ ] 是否使用安全的随机数生成？

依赖安全 (Dependencies)

• - [ ] 新增依赖是否有已知漏洞？
• [ ] 版本是否锁定？
• [ ] 是否引入了不必要的权限？

安全严重性分级

Step 4: 测试覆盖检查

测试分析清单

markdown
🧪 测试覆盖分析

新增代码测试

• - [ ] 新增功能是否有对应测试？
• [ ] 测试是否覆盖正常流程？
• [ ] 测试是否覆盖异常/边界用例？

测试质量

• - [ ] 断言是否有意义（非 trivial assert）？
• [ ] 测试是否独立（无共享状态）？
• [ ] Mock/Stub 使用是否合理？

回归测试

• - [ ] 修改的逻辑是否有回归测试？
• [ ] 是否破坏了现有测试？

覆盖率评估

• - 关键路径覆盖率: %
• 建议覆盖率目标: 80%+

测试建议模板

markdown

🧪 测试建议

缺失测试

1. 1. [文件路径] — [建议添加的测试]
2. ...

边界用例

1. 1. [场景描述] — [建议的测试方法]
2. ...

测试改进

1. 1. [现有测试] — [改进建议]

Step 5: 性能 & 质量评估

性能检查

markdown
⚡ 性能评估

算法复杂度

• - [ ] 是否有 O(n²) 或更高复杂度的操作在热点路径？
• [ ] 是否有不必要的嵌套循环？

数据库/IO

• - [ ] 是否有 N+1 查询问题？
• [ ] 是否有不必要的全表扫描？
• [ ] 文件操作是否有适当的缓冲？

内存使用

• - [ ] 是否有内存泄漏风险？
• [ ] 大数据集是否分页/流式处理？

缓存

• - [ ] 是否应该缓存的结果未缓存？
• [ ] 缓存失效策略是否正确？

代码质量

markdown
📝 代码质量

可读性

• - [ ] 命名是否清晰、一致？
• [ ] 函数/方法是否职责单一？
• [ ] 是否有过度复杂的条件逻辑？

可维护性

• - [ ] 是否遵循项目的代码规范？
• [ ] 是否有重复代码可以提取？
• [ ] 错误处理是否完整？

架构

• - [ ] 变更是否符合项目架构模式？
• [ ] 是否引入了不当的耦合？
• [ ] 是否有向后兼容性问题？

Step 6: 生成报告 + 飞书通知

审查报告模板

markdown

🔍 PR 审查报告

PR: # —