用 LLM 自动化代码评审,又不惹烦团队
来源:dev.to — 2026-08-09
📋 概述
一个 LLM 评审者该在抓那些人类会略过、枯燥的东西时赢得位置——缺失的空值检查、被吞掉的错误、什么都没断言的测试——而在其他时候保持安静。它一旦在每个 PR 上发十二条评论、一半在复述 diff 已说过的内容,就成了负债。差别几乎全在于你怎么界定范围和设置门槛,而不是选哪家厂商。作者在几个仓库上跑过 LLM 评审(托管工具和自建 GitHub Action),能坚持过第二周的团队都做了同一件事:让机器人保持咨询性质、收窄它被允许评论的范围、并把每个误报当作要修复的配置 bug 而非要忍受的噪音。LLM 值得介入的只是 linter 之上的"模糊层":看起来没问题但吞掉或误分类失败的错误处理、新代码里的 off-by-one 和边界逻辑、跑绿却没断言所宣称行为的测试、以及未参数化的查询或粘贴进配置的密钥等安全隐患。
🔑 核心要点
- LLM 评审的定位:抓人类会略过、linter 抓不到的模糊层,其余时间保持安静
- linter/类型检查已覆盖确定性层,LLM 只该补上静态规则编码不了的推理
- 高价值捕获:被吞掉的错误、off-by-one/边界逻辑、跑绿却不真正断言的测试、安全异味
- 最高杠杆设置是评论预算:限制为三到四条内联评论
- 坚持过第二周的团队都做到三点:咨询性质、收窄评论范围、误报当配置 bug 修
- 避免让 LLM 回答需要diff 之外仓库上下文的问题,那是噪音主要来源
💡 金句
一个 LLM 评审者抓得住人类略过的枯燥东西、其余时间保持安静,才算赢得位置;一旦它在每个 PR 上发十二条评论,一半还在复述 diff 已说过的话,它就变成了负债。
👍 0
👎 0
← 返回 dev.to 首页