PR 提交前自动代码评审
🎯 解决什么问题
review 同事的 PR 时，风格问题、安全隐患、性能坑要一条条看，很耗神。智能体在 CI 里自动跑一遍：检查空指针、SQL 注入、N+1 查询等常见问题，并按严重程度分级输出评论。人工 reviewer 只需关注业务逻辑对不对。
😣 团队 PR 越积越多，reviewer 要逐行看风格、空指针、SQL 注入、N+1 查询，低级问题反复出现，评审轮次动不动 3-4 轮，合并一个 PR 拖一周。
💡 PR 评审轮次平均减少一半，低级 bug 不再漏网
🛠 操作步骤（共4步）
1
CI 接入评审在 GitHub Actions 里加一个 job：PR 创建或更新时自动触发，把 diff 发给评审智能体。
2
自动扫描问题Claude 按检查清单扫描：空指针/越界、SQL 注入、N+1 查询、硬编码密钥、异常吞没等，SonarQube 并行跑静态规则。
3
分级输出评论结果按 blocker / warning / suggestion 三级直接评论到 PR 的对应代码行，blocker 必须修，suggestion 可讨论。
4
人工聚焦业务人类 reviewer 跳过已被 AI 覆盖的低级问题，只看业务逻辑是否正确、接口设计是否合理。
📋 提示词模板（复制即用）
你是资深代码评审员。请评审以下 PR diff，按严重程度分级输出：[Blocker] 必须修复（空指针、越界、SQL 注入、密钥硬编码、并发安全问题）；[Warning] 建议修复（N+1 查询、异常被吞、魔法数字）；[Suggestion] 可改进（命名、重复代码）。每条指出文件和行号、问题说明、修改建议。PR diff：[粘贴 diff]
🧰 用到工具
GitHub Actions · Muse · SonarQube
⚠️ 注意事项
- 先在团队内对齐 blocker 的定义，避免 AI 报一堆 warning 导致大家直接忽略所有评论。
- 给 AI 提供项目的编码规范和常见坑（如本项目的 N+1 重灾区），评审针对性会强很多。
- AI 评审替代不了业务逻辑评审：它看不出“这个折扣算错了”，这部分永远留给人。