跳转到主要内容

实践指南

给 AI 做 Code Review:审查清单与对话话术模板

说明如何让 AI 承担第一轮 Code Review,同时保留人工终审,并提供可复用的审查维度与提问模板。

RuleHub 编辑组3 分钟阅读
Code ReviewAI 编程质量模板

AI Review 能做什么、不能做什么

AI 适合做第一轮:找明显 bug、风格不一致、缺失测试、可疑权限逻辑。它不适合做最终责任人:业务正确性、产品取舍、合规签字仍须人类承担。

把 AI 当「不知疲倦的初级审查员」,而不是「自动合并机器人」。

推荐审查维度

按 diff 过一遍:

1. 正确性:边界条件、空值、并发与重试

2. 安全:鉴权、注入、敏感信息(可对照 API 安全清单

3. 可读性:命名、函数长度、是否过度抽象

4. 测试:是否覆盖关键路径与失败路径

5. 性能:明显 N+1、无界循环、过大同步 IO

6. 一致性:是否符合仓库 Rules / Skills 约定

对话话术模板

把下列模板存成 Skill 或固定 Prompt:

「你是本仓库的审查员。只根据提供的 diff 发言。输出结构:必须修改 / 建议修改 / 可选优化。每条指出文件与理由。不要重写无关文件。若信息不足,先列出需要我补充的问题。」

然后粘贴 diff 或指出分支。比「帮我看看代码行不行」有效得多。

工作流建议

个人开发

提交前本地让 AI Review 一轮,自行消化「必须修改」;再开 PR。

团队

  • CI 可跑静态检查;AI Review 作为可选机器人评论
  • 人类 Reviewer 聚焦业务与架构,不重复抠格式
  • 争议项以团队 Rules 为准,而不是模型口吻

防幻觉技巧

  • 要求「引用具体行或符号」,禁止空泛「建议优化性能」
  • 对「必须修改」项要求给出反例或复现思路
  • 不把 AI 的「看起来安全」当成渗透测试结论

与 Skills 的结合

将审查维度写成 code-review Skill,并在反模式中写明:禁止为了让 CI 绿而删除测试;禁止引入未讨论的新依赖。格式见 SKILL.md 指南

在 RuleHub 搜索 可发现社区 review 类 Skills,合入前改成你们的语言与严重级别定义。

度量

观察两周:

  • PR 首轮人类评论中「低级问题」是否减少
  • 回滚或热修次数是否变化
  • 审查耗时是否下降(而不是上升——若上升说明话术或范围失控)

小结

AI Code Review 的价值是压缩低级错误与统一视角,不是取代工程师判断。用结构化输出、人工终审、Skill 固化标准,才能形成可持续的质量闸门。

下一步:选一个中等大小的 PR,用本文话术跑一遍,把反复出现的问题写进团队 Rules。

本文由 RuleHub 编辑组 撰写并发布于 RuleHub 洞察专栏。转载请注明出处并链接至原文。

有建议或纠错?请访问 联系我们