Jack's Second Brain
首页 / 资料

AI 时代的代码审查:Claude Code 团队实践解读(PPT)

📥 主动分享/收藏:Jack 于 2026-10-21 要求收藏(经 TARS chat)。

二手总结:本 PPT 由 TARS(另一个助手)制作,是对 Anthropic claude.dev 博客文章的中文解读,并非 Anthropic 原件。下文要点均标注为"据 PPT" 本地导入,不等同于 Anthropic 已核实的原话;与原文的核对结果见文末。

要点(据 PPT)

  1. 背景:Claude Code 约 90% 的代码由 Claude 编写;Anthropic monorepo 每日落地改动从 5 月约 2,500 增至 30,000+ 本地导入。
  2. 基础设施:git 扛不住(主分支历史近 100 GB、每天约 2,500 万次 CI checkout、曾需数百个 git 镜像),5 个月内迁移到自研版本控制系统 Scion,CPU/内存占用约为原 git 镜像方案的 1/200;同期自研代码审查界面 Antique 本地导入。
  3. 成果:6–9 月 Claude Code 合并 PR 数翻倍以上,需回滚的 PR 占比下降 40% 以上;"没有取消审查,而是把大部分阅读交给机器,并让撤回变得快速简单" 本地导入。
  4. 重新定义审查:两项职能——发现并修复问题、在团队内共享上下文;关键问题不是"有人读过代码吗",而是"这个改动可能出什么错、由哪道检查来抓最合适";审查从终点关卡变成像拼写检查一样持续进行,多数改动在人看之前已被 Claude 审查并修复至少一次 本地导入。
  5. 目标与指标:多数 PR 至多一人参与(可以是作者本人),且回滚与事故不增加;指标为每 PR 人工批准数(中位数/P95)、人工评论数、PR 等待人的时长;回滚率与事故率是护栏 本地导入。
  6. 分层检查 + 回滚兜底:开 PR 前自证(测试、构建、截图、新上下文子代理复审)→ 每个 PR 先过 AI 审查 → 有未处理发现不能合并的安全审查 → lint/类型/全平台测试/构建 → 按风险分级的人工审批 → 渐进上线(合并队列、功能开关、内部先用、分批发布)→ 快速回滚 本地导入。
  7. 按风险分级审批:低风险(文档、文案、仅内部功能、开关后的新增改动)无需人工批准;普通产品改动及缓存、自动更新器、系统提示等敏感区域需 1 人;最高风险代码需 2 人。风险等级由改动决定而非作者(分类器判定);仅受信任作者可走轻量路径;机器人批准不能替代人;批准只对应特定 commit;指挥 agent 的人算作必需审查者之一 本地导入。
  8. AI 审查机制:多个代理并行找 bug,另一组逐条投票,足够多认定才发布;纯风格/诊断/清理类发现直接丢弃(经 A/B 测试);标注 Important / Nit / Pre-existing,每条必须修复或附理由关闭,常由 agent 夜间自动修复 本地导入。
  9. 审查必须收敛:一个 CLI PR 一夜经历 50+ 轮审查与修复;反复标记的问题 59% 来自首轮就已存在的旧代码。措施:3 轮上限、后续只审新 diff、标记 bug 类别让作者一次修全、不重复已发布的发现 本地导入。
  10. 人留在哪里:方向与取舍、高风险代码(计费、自动更新器、CI 工作流)、会被复制的架构模式、读证据、为结果负责 本地导入。
  11. "谁合并,谁负责"(You land it, you own it)与共享上下文:要了解哪块代码就让 Claude 写讲解;在公开 Slack 频道工作;改动小、一切可撤回、按影响面定标准、不追责只看系统 本地导入。
  12. 落地清单:PR 前(CLAUDE.md 列可运行检查、hooks、本地 /code-review 与 /security-review、REVIEW.md);每个 PR(必需 CI、PR 模板含测试/风险/如何回滚、AI 审查先试一个仓库、安全审查、轮次上限);审批(成文风险分级、CODEOWNERS + 分支保护、新提交使批准失效);合并后(合并队列、功能开关、一键回滚、事故转为评测集)本地导入。

关键结论/数据(据 PPT)

数据 值 来源
Claude Code 由 Claude 编写的代码占比 约 90% 本地导入
monorepo 每日落地改动 5 月约 2,500 → 30,000+ 本地导入
合并 PR 数(6–9 月) 翻倍以上 本地导入
需回滚 PR 占比 下降 40% 以上 本地导入
有实质性审查评论的内部 PR 占比 16% → 54% 本地导入
单个 PR 一夜审查/修复轮次 50+ 本地导入
反复标记问题中属于旧代码的比例 59% 本地导入
Scion 相对 git 镜像方案的 CPU/内存占用 约小 200 倍 本地导入

结论:把"人读代码"换成"按风险选择合适的检查",用多层自动审查 + 快速回滚换取更少的人工时间,同时以回滚率/事故率作护栏 本地导入。

与原文核对(2026-10-21 WebFetch)