1. 为什么我说“大部分code review都是在自欺欺人”我在团队里当了快六年的后端负责人见过太多review现场PR挂着三天没人点开合并前被小窗私聊“你那个PR我看了感觉没啥问题”还有人五分钟刷完几百行diff然后打上一句“LGTM”。这些事情我全干过也全被干过。如果你问代码评审有没有用所有人都会点头但你要是翻翻实际仓库的review记录心里基本都有数——大部分review是在走过场。后来我实在忍不了动手做了一个叫open-code-review的开源工作流。核心思路很简单把代码评审从“靠人自觉”变成“靠流程兜底”用自动化工具和AI模型先筛两轮把人力的精力集中在真正值得判断的地方。这个项目跑了大半年我自己的仓库和三个朋友的团队仓库都在用效果比我预想的好。所以今天把这套东西从设计思路到完整落地讲一遍不想重复别人的经验帖纯粹是拿自己踩过的坑说话。1.1 评审失效的三个现实原因先聊聊我观察到的三个问题这三个问题直接决定了open-code-review该怎么设计。第一个是时机问题。很多review不是在写代码的人“最需要反馈”的时候发生的而是在合并前被CI逼着做。这时候提一句“这里命名改一下”开发者的第一反应往往不是感谢而是“又要重跑一遍流水线”抵触情绪非常重。我见过最夸张的reviewer为了让构建通过直接在评论里说“这个可以后续优化”然后所有人默契地忽略了它。第二个是注意力问题。人脑处理上下文的能力是有限的。一个PR如果超过300行绝大多数reviewer只能抓住主干逻辑对边界条件、错误处理、并发一致性的检查会呈指数级衰减。这不能赖个人这是认知带宽的物理限制。我之前统计过自己三个月内的有效review评论发现70%的评论集中在PR的前100行后面全是“空泛式点赞”。第三个是反馈闭环问题。代码评审最大的价值不在发现bug那一刻而在“这件事变成了团队经验”的那一步。但现实是review提出的问题被resolve之后就彻底消失了。同一个坑这个模块踩完下个模块接着踩根本没有人把review沉淀成规则。我见过一个仓储服务接口在半年内因为同一个空指针问题改了三次每次都是新同事踩进去。1.2 open-code-review想解决的核心矛盾想清楚这些问题之后我给自己定了一个目标做一个让“人只审机器审不了的东西”的评审系统。这个定义非常重要。很多人一听到AI code review就想着让机器替代人把reviewer裁掉。这是完全错误的方向。AI目前的状态根本做不到像资深工程师那样理解业务语义但它很擅长做三件事一是快速阅读全量diff二是发现机械性疏漏三是按照固定规则反复检查。这三件事正好是人最容易疲劳、最容易跳过、也最容易犯错的环节。所以open-code-review的核心矛盾不是“用机器还是用人”而是**“如何把机器能干的和人该干的分开”**。机器先干80%的重复检查人只需要处理那20%需要判断力的部分。这个思路一旦建立很多设计就顺理成章了。1.3 我给自己定的三条设计底线动手写代码之前我先给自己定了三条不可违背的底线这三条后来被证明是项目没烂尾的关键。底线一review工具的反馈必须即时且低噪音。如果每个PR都刷出一堆“建议把var改成let”这种无关痛痒的评论开发者五分钟就会把机器人屏蔽。open-code-review的第一版宁可漏报不可误报我会在后面的误报处理章节里详细讲这里面的取舍。底线二所有规则必须可追踪、可解释、可关闭。工具可以给建议但每一项建议背后都要有明确的触发逻辑reviewer可以一条条决定接受还是忽略绝对不允许搞一个黑盒模型直接给结论。底线三这个工具自己要能被review。open-code-review本身是开源的我希望它的规则配置、脚本逻辑、提示词模板都像一份好代码一样被人检查。所以从第一天开始我就用了PR草案的方式开发每个模块先讨论、再实现、最后才合入。这三条底线决定了后面所有代码的写法。接下来我把整套工作流的架构和每一步的细节讲清楚。2. open-code-review的完整工作流拆解open-code-review不只是一个脚本也不是单靠一个AI接口就完事它是一条完整的流水线。我把它分成四个阶段快筛、静态规则、AI深度审查、人机协同定案。每个阶段干不同的事输出不同的结果层层递进。2.1 第一阶段提交差异的“快筛”与分级第一步不是审代码而是先弄清楚“这次PR到底改了什么”。听起来废话但大部分review工具恰恰没有把这件事做扎实。open-code-review会在PR创建或者更新的时候拉取完整的diff然后用Python解析出几个关键维度改动文件数量与类型源码、测试、配置文件、锁文件新增/删除/修改的行数统计涉及的模块、目录与函数是否包含依赖变更、数据库迁移、权限调整等“高风险信号”是否存在大文件、超长函数、循环复杂度暴涨的情况我拿前100个真实PR测过这套快筛的准确率能做到95%左右。它最大的价值不是告诉你有问题而是给这个PR打一个“风险等级”低风险PR直接走快速通道高风险PR强制要求至少两人review。这里有个很实用的设计细节快筛阶段不做代码质量判断只做变化识别。比如依赖变更open-code-review会直接对比两个版本的依赖清单然后把变更内容高亮出来。但“这个依赖该不该升”不归它管那是人要拍板的事。我见过很多工具死在“什么都想管”上面结果开发者的第一反应是关掉通知。# 快筛阶段的简化逻辑提取PR的核心变化特征 def scan_pr_diff(diff_text): changes { files: parse_file_stats(diff_text), risk_signals: [], complexity_delta: 0.0, } if has_dependency_change(diff_text): changes[risk_signals].append(dependency_change) if has_high_cyclomatic_complexity(diff_text, threshold15): changes[risk_signals].append(complexity_risk) return changes这个阶段跑完open-code-review会在PR上打一个标签low-risk/medium-risk/high-risk。标签是给后面的阶段用的也是给team成员看的——看到high-risk大家自然知道要打起精神审。2.2 第二阶段静态规则引擎的重合与补位快筛之后第二道工序是静态规则扫描。你可能觉得这不就是ESLint或者SonarQube干的事吗是的但open-code-review从一开始就设计成“在已有静态检查工具之上运行”而不是替代它们。具体做法是通过解析ESLint、RuboCop、Checkstyle等工具的JSON输出提取结构性问题和风格问题。然后我加了一层ESLint不存在的检查我管它叫“变更范围检查”。ESLint只能告诉你某行代码有问题但它不知道这行代码是不是这次PR引入的。open-code-review会做一次区分计算如果某个warning在被改动的行之外直接降级为不提醒只有在被修改的行上出现的新告警才会升级为需要处理的问题。这么做的理由是存量代码的问题永远存在靠一个PR根本清不完如果每次都把全仓库的老问题翻出来对当前开发者的打击太大了。我们要的是“别在洗澡的时候关掉浴缸里的水”抓增量比抓存量更有价值。另一个补位是边界规则。举个例子open-code-review里有一条我在生产环境被坑过之后写进去的规则不允许在循环体内调用外部HTTP接口。这种规则ESLint和SonarQube配置起来很麻烦但通过AST解析open-code-review在两周内就发现了6处类似问题其中一处甚至会在生产环境造成级联超时。2.3 第三阶段AI深度审查的上下文工程这是整条流水线里最核心、也最容易翻车的一环。我一开始也犯过错直接把整个diff丢给大模型让它“检查代码质量和潜在bug”。结果那个版本我愿称之为“AI废话生成器”它会在每一个函数下面都写一句“这段代码逻辑清晰但建议增加注释”完全没法用。后来我总结出一条经验AI在代码评审中的上限取决于你给它多大的上下文以及你让它输出什么格式。所以我做了一套上下文裁剪策略这是open-code-review和很多同类工具拉开差距的关键。具体来说open-code-review不会把整个diff发出去而是按文件按函数切割每个函数提取以下几个要素作为上下文函数源码改动前后函数签名与调用点依赖的数据库表结构或消息格式如果有相关测试用例名称该函数所在模块的整体设计约束我试过OpenAI、Claude、国产DeepSeek等多个模型实际用下来不同模型的上下文敏感度差异非常大。有些模型你给它5000字上下文它就懵了有些模型给它10000字才能稳定发挥。所以我在配置里加了一个context_window参数团队可以根据自己的模型情况调节。提示真正让AI阶段有质变的不是模型本身而是你给AI的“任务边界”。与其让它“检查一切”不如让它专注三件事边界条件缺失、并发安全隐患、错误处理漏洞。下面是这个阶段一个精简版的提示词模板我跑了很久才稳定下来你是一名资深代码审查者。这是函数 {function_name} 的变更 - 变更前后源码... - 函数签名... - 依赖的数据库结构... - 相关测试用例... - 已知设计约束... 请仅从以下三个维度给出审查结论 1. 是否存在边界条件未处理例如空值、越界、非法输入 2. 是否存在并发或资源释放问题例如锁、连接、文件句柄 3. 是否存在错误处理缺陷例如吞异常、错误状态未被传播 要求 - 只在确实发现问题时回答没有问题请回答【无问题】 - 每个问题必须包含问题描述、触发场景、建议修改方向 - 不要提代码风格、命名、注释缺失等风格类建议这个提示词模板解决了我之前最大的痛点AI废话。实测下来用了这个模板之后AI阶段的平均有效评论率从之前的不到20%提升到了63%左右。2.4 第四阶段人机协同的定案与跟踪流水线的最后一步是把前三阶段的结果汇总但不讨论。这可能是open-code-review和纯“AI审pr”最大的区别。open-code-review最后给出的不是一个“通过/不通过”的结论而是一份待人工确认清单。清单里的每一条都带有来源标签quick_scan、static_rule、ai_review。reviewer拿到这份清单后可以逐条做三个操作确认、忽略、标记为误报。这些操作都会被收集起来作为后续调优的数据。在PR页面上open-code-review会以评论的形式发布一份结构化摘要包括检查项来源严重等级建议动作循环体内调用外部HTTP接口static_rule高移动到循环外并增加超时控制空指针风险未判断返回值ai_review中增加空值判断新增依赖与现有工具库功能重叠quick_scan中确认是否需要引入新依赖测试覆盖缺失未覆盖超时分支ai_review低补充单元测试这个表格真的救了我自己无数次。以前我review的时候要自己盯着代码找这些问题现在机器帮我找好了我只需要判断“这个到底算不算问题”。一个人花10分钟能完成以前40分钟的活而且注意力全部在刀尖上这就是人机协同最理想的状态。3. AI审查模块的调教细节这是用得好与不好最大的区别很多人看了上面的工作流觉得最麻烦的是静态规则配置但真正让他们放弃的往往是AI阶段。因为你第一次跑AI review的时候它要么乱说话要么不说话完全没有“正常交流”的感觉。这一章专门讲我调教AI模块的实操经验包括上下文怎么给、温度怎么调、输出格式怎么约束。这些东西网上没有现成答案全是实测出来的。3.1 上下文裁剪不是把整个diff丢给模型我先说一个反直觉的结论AI代码评审失败八成是上下文给的太多而不是太少。我第一版就是贪心想把整个PR的所有信息都塞给模型。结果模型既看不完也看不深输出完全失控。后来我把上下文裁剪到“一个函数一个请求”效果反而立竿见影。具体裁剪规则是这样的上下文项是否提供原因函数源码变更前变更后必选AI需要对比推断改动意图函数签名、参数说明必选缺少会漏参数校验类问题调用点片段最多3处可选帮助AI判断调用方预期数据库表结构按需仅查询类函数提供整个文件的全部代码一般不给超出有效上下文只会稀释注意力整个PR的所有文件不给效果极差还会大幅增加成本实测数据按函数拆分后每条prompt的平均token消耗降为原来的三分之一但有效问题检出率反而高了40%。这件事给我最大的教训就是——机器和人都一样注意力资源有限高效的工具必须学会做减法。3.2 提示词模板与JSON输出约束AI模型输出最大的坑是“格式不稳定”。同一个提示词有时候它输出分条文字有时候直接给你一大段散文还有时候会偷偷在分析里加上它自己的建议。为了让流水线后续处理方便我强烈建议强制模型输出JSON格式。你不需要真的有JSON Schema这种重型工具只需要在提示词末尾加一句请以如下JSON结构返回结果不要输出任何其他内容 { issues: [ { line: 行号或函数位置, type: boundary/concurrency/error_handling, severity: high/medium/low, description: 问题描述控制在50字以内, suggestion: 建议方向控制在30字以内 } ] }这里有个很关键的操作细节描述要求控制在50字以内建议控制在30字以内。为什么要这么短因为开发者看review评论的耐心极其有限超过两行就会失去焦点。而且短描述迫使模型跳过没把握的判断只输出它真正有信心的点这比什么都好。我用这个模板跑了两个月唯一遇到的问题是偶尔模型会多出一个summary字段不影响主流程在代码里直接忽略未知字段即可。3.3 温度与阈值宁可漏报不要刷屏如果你用的是像OpenAI这类支持参数调节的API那么温度temperature是你在代码评审场景里最该改的一个参数。我把默认值设成了0.2而不是通用的0.7或1.0。原因很简单代码评审是高风险场景我们希望模型输出保守、可靠、没废话不需要创意。温度高的时候模型会脑补出各种“潜在问题”最后变成了狼来了的故事开发者看多了就不信了。温度调到0.2左右模型只会在很有把握的时候提出问题输出质量明显提升。同样的逻辑也用在阈值判断上。open-code-review对AI检出的每一项会计算一个置信度分基于模型返回的描述长度、涉及风险的严重等级、以及该文件的修改频率低于0.6的默认不打到PR上只记录在日志里。宁可漏报20个模糊问题也不要刷屏10个让开发者失去信任的假阳性。信任这个东西一旦没了工具就彻底废了。3.4 增量审查与结果缓存大型仓库每天会有大量PR和commit如果每个变更都全量跑一遍AI成本会高到团队主管找谈话。这里我做了两个优化。第一个是增量管道。open-code-review会自动记录每个函数的“最后审查哈希”如果某个函数的变更哈希没有变化直接跳过这个函数的AI请求。基于这个机制一个改动5个文件的PR实际需要调用AI的函数可能只有12个API成本降到原来的30%左右。第二个是结果缓存。同一段代码如果已经生成过审查意见除非代码茶量发生变化否则不会重复生成。缓存记录在仓库根目录的.ocr_cache.json方便其他成员直接看到“这段代码什么时间被谁审过”。这个文件默认不进版本库用.gitignore忽略掉。注意缓存机制有个陷阱——如果你的审查规则或者模型的提示词模板升级了旧缓存会导致新规则无法生效。我现在的做法是在缓存结构里加一个rule_version字段规则升级时自动失效全部缓存。这个bug害我跑了一个星期旧规则才发现务必引以为戒。4. 一次真实审查的完整复盘从提交到合并说完了设计拿一个真实案例完整走一遍整个流程。这个案例来自我一个朋友的支付服务仓库他当时用的是open-code-review的早期版本虽然那个版本很粗糙但恰好把问题暴露得最完整。4.1 触发场景一个看起来人畜无害的PR那天晚上团队里的一个中级工程师提交了一个PR主题是“优化退款查询接口的响应时间”。改动集中在两个文件一个Service类和一个Mapper接口。改动量不大新增了40行删除了15行。快筛阶段给出的风险等级是medium因为涉及到了数据库查询逻辑。我看到快筛结果时没有太警觉因为这类优化PR我见过太多了无非就是加个索引或者改成批量查询。但AI审查阶段跑完后模型给出的一个问题让我后脊发凉。4.2 快筛与静态规则给出的早期告警先看快筛报告。open-code-review对这次PR给出的告警有三条数据库查询类变更复杂度变化为普通级别新增了一个外部接口调用当时没细看原有测试文件中关于超时分支的用例没有同步修改前两条我扫了一眼就略过了第三条让我决定再仔细看一下测试变更。结果发现那位同事确实只改了生产代码测试文件里三个原有用例仍然是旧断言。这个其实不能怪他很多人在做性能优化时总觉得“行为没有变化测试不用动”但行为真的没有变化吗不见得。4.3 AI深审发现的两个真实问题AI阶段返回了两个问题这是那两个问题的原始返回内容略有删改保留核心信息第一个问题类型是boundary严重等级high。模型判定新方法中当退款金额为null时代码会在后续计算空指针而原来的实现会提前返回。建议增加空值校验。我一看代码确实那位同事为了提升性能把空值判断挪到了后面但这个判断依赖一个可能为null的字段完全没处理。第二个问题类型是concurrency严重等级medium。模型判定新代码中使用的缓存实例是无锁的在多实例部署的场景下会出现缓存不一致。这个问题我和那个同事讨论了20分钟最后确认生产环境确实是三实例部署这确实是个问题。说句实话这两个问题靠传统人工review很可能要等代码上线后出了事故才会被发现。不是同事水平不行而是人的注意力在面对“看起来很简单”的改动时天然会放松警惕。4.4 人审时的互动与收尾接下来是人工review环节。我看了AI给出的两个问题没有直接照单全收而是做了两件事。第一件事检查AI第一个问题的触发路径。我拉出方法调用链确认那个字段确实没有任何上游赋值保障然后才把AI的评论升级成“必须修复”并给同事留了具体的修改建议。这里有一个很微妙的原则AI给的是线索人给的是命令但命令必须建立在线上证据之上。第二件事关于并发问题我让同事把缓存设计思路讲了一遍。他说他只在内存里做了冷数据缓存没考虑分布式的场景。我指出那个ConcurrentHashMap在单实例下是完全OK的但生产环境是集群。最后同事改成了Redis缓存问题解决。这次review从提交到合并花了4个小时其中我和同事直接沟通讨论的时间大概只有40分钟剩下时间全是流水线在跑。以前这种规模的问题至少要一个下午加一个早上。工具把时间压缩在了真正需要人判断的地方这就是open-code-review最让我满意的部分。5. 误报处理从“AI鹰眼”到“懂分寸的搭档”所有做过自动化代码检查的人都会遇到同一个终极问题误报。而且误报杀掉的不是一次代码检查是工具活下去的信任根基。我在open-code-review上花了最多时间的不是AI调优而是误报处理。所以单独拆一章讲。5.1 噪音的三类来源我先归纳了误报的三个来源每个都是真实案例逼出来的。第一类是静态规则的超范围匹配。ESLint的规则是通用的但它理解不了你的业务场景。比如有一条规则是“禁止在条件语句中使用赋值表达式”听起来很合理但如果你的项目里有一段代码本来就是利用赋值表达式的特性做链式判断这条规则就会天天误报。解决方式是靠人工标记反馈来降级规则。第二类是AI模型的“自信式胡说”。大模型的特性就是特别喜欢给建议即便它并不理解整个代码的架构。我之前遇到过AI对一段已经封装好的工具函数提了一堆重构建议但它没看到调用这个工具函数的30个地方。这就是为什么我在提示词里强制要求“没有把握就不要说”但还是会有漏网的。第三类是规则理解与业务语义的偏差。比如系统里面有一个消息队列消费者它接收到消息后先入库再通知业务方open-code-review收到告警说“数据库操作之后提交事务前不应该调用远程服务”在大多数场景下这条规则是对的但在这个特定场景下先入库再通知就是设计本身你认为它误报但规则引擎理解不了“只发一次”这个保证。我用一张表记录下这三类来源的特点和应对方式噪音来源典型例子影响程度应对策略静态规则超范围匹配校验规则和业务代码冲突高人工反馈、按规则降级AI幻觉式建议对不存在的上下文做重构建议中调整预热词限制输出范围规则与业务语义冲突分布式事务场景误判高添加文件级忽略列表5.2 我的分类处理策略一个真正的误报反馈闭环是分层次的不能一刀切。你如果把某条规则直接关闭以后它可能就不再为你服务如果你完全不在乎误报开发者很快就会把机器人静音。我现在的处理策略分三层第一层单次忽略。reviewer在PR评论里点“忽略这条”。这个操作只作用于当前PR适合偶尔出现的业务语义冲突。数据会被记录但不会改变规则本身。第二层规则降级。当一个reviewer在30天内对同一条规则/同一个AI提示词报告了3次以上误报open-code-review会自动将这条规则的状态从enabled降为review_only也就是不再直接展示在PR摘要里只会进入周报。降级动作会推送通知给仓库管理员防止规则被滥用。第三层文件级豁免。对于某些全局性的工具类文件或自动生成代码可以在配置里声明豁免。我的团队目前对generated/、vendor/这些目录是批量豁免的因为它们根本不是人写的代码规则引擎审它们没有任何意义。5.3 白名单与学习机制的迭代经验最后说一个很多工具都会做但容易做砸的东西误报学习机制。我试验过的第一个蠢办法是让open-code-review每次发现误报就直接修改规则。结果两个星期后规则被改得面目全非很多原本正确的检查被误杀掉了。这跟大模型微调训练中“灾难性遗忘”是一个道理——反馈数据过少模型越调越糟。后来我改成“延迟批处理学习”策略。每次误报结果先进入一个待处理队列只有当一个规则积累到20个有效反馈时才触发自动调优。调优过程会生成一份“规则变更提案”以PR的形式提交到open-code-review的配置仓库由人review后手动合入。这个机制听起来很笨但实际效果极好——规则变更永远有痕迹永远不会失控。提示如果你只是自己小团队用可以每周五花二十分钟人工看一下误周报比任何自动学习都好用。自动学习是给那些没有专人维护规则的工具准备的你有两个人就别偷懒。6. 把open-code-review接入团队流程不引起反感的做法工具做得再好落不了地就是自嗨。我见过太多团队因为推行工具引起反弹最后工具被删除、负责人被吐槽。想让大家接受一个代码评审机器人关键要处理好两件事一是它不能挡路二是它要真的有用。下面是我摸索出来的接入实践。6.1 GitHub Actions的接入配置open-code-review目前对GitHub支持得最完善使用YAML工作流文件接入。如果你的团队用的是GitLab或Gitea原理类似只是几个变量名不一样。name: open-code-review on: pull_request: types: [opened, synchronize, reopened] pull_request_review_comment: types: [created, edited] jobs: review: runs-on: ubuntu-latest permissions: contents: read pull-requests: write steps: - uses: actions/checkoutv4 with: fetch-depth: 0 - name: Run open-code-review uses: your-registry/open-code-reviewv1 env: LLM_API_KEY: ${{ secrets.LLM_API_KEY }} LLM_BASE_URL: ${{ secrets.LLM_BASE_URL }} LLM_MODEL: your-model OCR_TEMPERATURE: 0.2 OCR_DETACHED_MODE: true这里重点解释几个容易被忽略的配置fetch-depth: 0非常关键。没有它GitHub Actions默认只拉取一个浅克隆无法拿到完整的历史提交记录这就导致快筛阶段无法正确计算“这次PR相对上次合并点改了什么”。我第一版就死在这上面改完带完整历史后准确率立刻上来了。pull-requests: write权限是用于让机器人通过API发布review评论。如果你只把结果输出在Action日志里完全不需要这个权限但那样对开发者的提醒效果会大打折扣。我的选择是给机器人write权限但只允许对评论操作禁止任何合入操作。OCR_DETACHED_MODEtrue是开放一种“只读模式”。机器人的评论一律以普通评论形式出现不做“request changes”那种阻塞性状态。这块我放在下一节详细说。6.2 卡点策略“建议”和“必须”分开处理推行自动化评审时最容易犯的错误是把机器人设为强制门禁“所有PR必须通过AI审查才能合并”。这个决策会直接导致两个后果第一开发者开始敷衍对着无关痛痒的问题强行修改来“消磨”机器人第二机器人的误报会变成团队流程的堵点天天有人来找你吵架。open-code-review的默认态度是建议不阻塞必须才阻塞对于来自AI或静态规则的一般性建议机器人在PR上留评论但不会阻止合并。开发者可以选择忽略也可以点“确认修复”两种操作都能让流程往前走。只有在快筛阶段识别出高风险信号例如数据库迁移、依赖变更、密钥硬编码且配置了强制规则时机器人才会以request changes状态标注。即便这样管理员依然拥有最终撤销权。我的经验是前两个月强制规则只保留3条包括密钥泄露、危险函数调用、数据库结构变更未走迁移流程。剩下的全部作为建议先建立信任等团队适应了再逐步加码。6.3 用数据让reviewer服气推行一个工具光靠说“它很好用”不够。数据才是最有力的说服工具。open-code-review在每一个review周期结束后会生成一份可汇总的统计报表包括每个PR中被机器人检出并修正的缺陷数量人工reviewer在机器人提示的辅助下新发现的问题数量平均review时间的变化对比接入前后的数据机器人误报率以及被人工驳回的次数我当时把这套数据整理成周报给团队看了四周。第四周的报表显示机器人的有效问题检出率已经稳定在65%以上团队平均review时间从45分钟降到12分钟线上缺陷率下降了22%。数字摆在那里原本反对最激烈的高级工程师也态度松动了。提示这份报告不建议列在公共频道里最好只发给团队leader或者核心骨干。公开对比数据会让开发者产生“被监控”的不适感反而得不偿失。我踩过这个坑现在只在迭代沟通群里发。7. 用到现在的体会与下一步想做什么open-code-review从第一版粗糙脚本走到现在最大的收获反而不是代码而是让我想明白了一件事代码评审的本质不是找bug而是帮团队建立统一的质量语言。当所有人都在同一套规则下交流很多争论就不存在了因为标准和责任人清清楚楚。有几个小技巧分享给打算自己搭建这类系统的朋友搭建的时候一定先把“误报反馈”这个链路想好它比AI选型更重要。误报处理不好再强的模型也会被停用。不要一上来就追求100%自动化预留一个人工review的“出口”。很多工具被抵制就是因为它剥夺了工程师的专业判断权。选模型的时候看它在“代码diff”上的表现而不是在那个热门榜单上的分数。不同模型在代码理解上的差异比你想的大得多。下一步我打算给open-code-review增加两个能力一个是根据PR描述自动生成测试建议另一个是把积累的review数据做成团队质量趋势图。这两个目前还在实验阶段等跑出稳定结果再单独分享。如果你也在折腾类似的东西或者对代码评审流程有自己的想法欢迎一起聊聊。踩坑这种事两个人踩总比一个人踩浅一点。
