代码评审 Agent 不是替代工程师，而是把重复检查前置

代码评审Code Review是软件工程中质量保障的核心环节。但在很多团队中它正在变成瓶颈。合并请求排着长队资深工程师被反复拉去审查同样的编码规范问题——空指针风险、硬编码密钥、缺失的错误处理……真正需要架构层面讨论的设计问题反而被淹没在这里少了个空格这个变量命名不符合规范的琐碎反馈中。极狐GitLab Duo 的代码审查流程Code Review Flow正在改变这个局面。01 问题根源评审者不是格式检查器先看一个普遍现象团队引入 AI 编程助手后开发者写出代码的速度确实变快了合并请求的数量随之增长。但评审侧的产能并没有同步提升瓶颈从写代码转移到了审代码。这背后有一个结构性矛盾代码评审的本质是工程判断但大量评审时间花在了重复性检查上。具体来说评审者每天要做的事情包括编码规范检查命名是否符合约定、缩进是否统一、注释是否完整。安全基础检查有没有硬编码密钥、有没有 SQL 注入风险、用户输入是否做了校验。可维护性检查函数是否过长、是否有明显的重复代码、错误处理是否到位。跨文件依赖检查这个改动会不会影响其他模块的接口、有没有破坏既有契约。这些检查重要吗当然重要。但它们应该是自动化的第一道筛选而不是占用资深工程师大量时间的手工劳动。当评审者把精力花在格式和规范上留给架构设计、业务逻辑和边界条件的判断力就会稀释。02 Code Review Flow平台级 AI 评审不是 IDE 插件市面上有不少 AI 编程工具能在 IDE 里做行内建议但它们的上下文有限——通常只能看到当前打开的文件或当前函数。对于代码评审这个场景这远远不够。极狐GitLab Duo 的代码审查流程运行在合并请求上下文中而不是开发者的本地编辑器里。这意味着完整的变更上下文它能分析整个合并请求的代码变更理解这次改动做了什么而不仅仅是改了哪一行。代码仓结构理解它对整个仓库的结构和跨文件依赖有增强的上下文理解能判断一个改动是否会影响到其他模块。可操作的反馈审查评论不是泛泛而谈的建议优化而是包含具体的代码位置、问题描述和修改建议。使用方式也很直接在合并请求中把GitLabDuo分配为审查者或者使用快速操作/assign_reviewer GitLabDuo。审查启动后你可以实时监控会话进度直到审查完成。评审完成后你还可以与极狐GitLab Duo 交互——回复它的审查评论请求澄清或替代方案在任何讨论线程中 GitLabDuo 提问后续问题。这不是一个跑完就结束的静态报告而是一个可以对话的评审过程。03 自定义审查指令把团队规范变成 AI 规则不同团队有不同的编码规范和评审重点。一个金融科技团队可能最关心 SQL 注入和密钥泄露一个 IoT 团队可能更关注嵌入式代码的内存安全。如果 AI 评审只能做通用检查价值就打了折扣。极狐GitLab Duo 支持通过mr-review-instructions.yaml文件定义项目专属的审查指令。这个文件放在仓库的.gitlab/duo/目录下使用 YAML 格式通过 glob 模式匹配不同文件。一个实际的配置示例instructions: - name: TypeScript Source Files fileFilters: - **/*.ts - !**/*.test.ts instructions: | 1. 确保正确的 TypeScript 类型避免 any 2. 遵循命名约定 3. 为复杂函数编写文档 - name: All Files Except Tests fileFilters: - !**/*.test.* - !**/*.spec.* instructions: | 1. 确保正确的错误处理 2. 为复杂逻辑添加有意义的注释 3. 避免硬编码凭据几个关键特性附加而非替换自定义指令附加到极狐GitLab Duo 的标准审查标准之后不会覆盖内置的安全检查。模式匹配支持 glob 模式可以按语言、目录、文件类型精确匹配也支持!前缀排除特定文件。多指令叠加单个文件可以同时应用多个指令组比如同时检查 TypeScript 规范和通用安全规则。可追溯由自定义指令触发的审查评论会标注根据 [instruction_name] 中的自定义指令标准评论不使用此格式方便区分。更实用的是你还可以用极狐GitLab Duo Chat 分析代码库让它帮你生成初始的审查指令草稿再根据实际需要调整。04 自动审查让每个合并请求都有初审手动分配 GitLabDuo 适合需要时按需使用但很多团队希望每个合并请求都自动获得一次 AI 初审确保基础质量问题不会溜过去。极狐GitLab Duo 支持项目级、群组级和应用级的自动审查设置项目级在项目设置 合并请求中启用极狐GitLab Duo 自动审查。创建合并请求后非草稿状态极狐GitLab Duo 会自动进行审查。群组级群组所有者可为整个群组启用设置从群组级联到下属项目。应用级管理员可为实例上所有项目启用。设置从应用级联到群组再到项目更具体的设置会覆盖更宽泛的设置。这意味着企业可以先在群组级设定基线标准再让个别项目按需调整。05 人在回路AI 做初审人做终审极狐GitLab Duo 的代码审查流程不是要取代人类评审者。它的定位是初审助手——把重复性、规则性的检查前置让工程师把精力留给真正需要人类判断的事情。一个典型的协作模式是这样的开发者提交合并请求极狐GitLab Duo 自动进行初审。AI 识别出编码规范、安全隐患和可维护性问题在合并请求中给出具体反馈。开发者根据反馈修复问题或者与极狐GitLab Duo 对话讨论替代方案。人类评审者介入时基础问题已经被过滤掉了他们可以专注于架构设计、业务逻辑和边界条件。评审者可以回复极狐GitLab Duo 的评论进一步追问或确认。这个流程的关键在于分工明确AI 负责广度和速度人负责深度和判断。当代码生成越来越快如果没有一个同样快速的评审机制来承接合并请求队列只会越排越长。写在最后代码评审的真正价值从来不在于找出每个分号和空格的问题而在于确保每一次合并都在架构上合理、在业务上正确、在安全上可靠。极狐GitLab Duo 的代码审查流程通过平台级的上下文理解、可自定义的审查指令和自动化的初审机制把重复性检查交给了 AI。这不是让工程师变得多余而是让他们从格式检查器的角色中解放出来回到代码评审最该做的事情上——用工程判断守护代码质量。当代码生成越来越快评审也必须跟上。把重复交给 AI把判断留给人。