跳到正文
HNHacker News·

How the Claude Code team does code review

AI 摘要

The Claude Code team leverages a structured code review process despite Claude writing 90 percent of its own code. During the research preview, substantive review comments on internal PRs increased from 16% to 54%. Large PRs (over 1,000 lines) often receive findings, while smaller ones are typically clean. Repositories are configured for Quick, Standard, or Deep review, with Deep review reserved for critical areas where a missed bug is costly. Although automated, detecting bad merges still relies on authors, on-call personnel, and monitoring dashboards.

时间与来源

时间显示为 UTC

显示时区:UTC

本地时区尚不可用,暂时显示 UTC。

发布当时偏移:UTC+02026年10月16日 19:04 UTC

收录当时偏移:UTC+02026年10月16日 20:00 UTC

发布
2026年10月16日 19:04
收录
2026年10月16日 20:00
来源类型
未分类
档位
社区
信源状态
正常

档位是按信源手工设定的编辑判断,不是逐条打分。

爆款判定
判定依据
热度约为该来源近期上榜条目中位水平的 3.6 倍
指标对比
282 vs 中位 78(20 条基线样本)
检出时间
10/17 01:00

We've said before that Claude writes about 90 percent of Claude Code's own code. So how do we review code at the volume we produce it? Does anyone read code anymore? And if something breaks, who catches it?

This post covers the code review flow on the Claude Code team and how it has evolved. From June to September, merged pull requests to Claude Code's codebase more than doubled while the share that had to be reverted fell by more than 40 percent.

We didn't remove review. But we did move most of the reading to machines, optimized for human time and attention, and invested heavily in making it fast and easy to undo mistakes.

RETHINKING THE PURPOSE OF CODE REVIEW

When agents write code, the number of changes goes up, fast. In Anthropic's monorepo, changes landing per day went from about 2,500 in May to more than 30,000 in September this year.

Claude Code lives in its own repository, separate from the monorepo, but it's seeing a lot of pressure too. When you are shipping this fast, manual, line-by-line code review is incredibly difficult if not impossible.

Code review does two jobs:

- Find and fix issues before they ship

- Build shared context and codebase understanding across the team

This article mainly focuses on finding and fixing issues, but we will come back to the second job near the end.

We think "did a human read the code?" is an imprecise proxy for "what could go wrong with this change, and which check—automated or human—is the right one to catch it?"

Asking that question also changed when review happens. Before, review was a gate at the end of the road: write the code, open a PR, wait for your colleague.

Now, review runs continuously, more like spellcheck while you write. Most changes have been reviewed and fixed by Claude at least once before a human is asked to look. This catches most routine mistakes, and saves team attention for higher-level questions and scrutiny.

THE SCARCE RESOURCE: HUMAN TIME AND ATTENTION

We want to preserve human time and attention for issues only humans can solve and make sure automated checks catch the rest. At our current rate of change, there is still an incredible amount of work for humans to do, and our team encounters new and novel challenges almost daily.

Human time covers both sides of review's cost: the time people spend reviewing, and the time they spend fixing what bad code breaks. Cutting the first doesn't help if the time just shows up in the second.

So the goal is that most pull requests in the Claude Code repos (the CLI, the desktop and web apps, and the mobile apps) merge with at most one human in the loop, who can be the author, with no increase in reverts or incidents. To measure human time, we track human approvals per PR (median and 95th percentile), human comments, and how long a PR waits on a person. Revert and incident rates are the guardrail: if they rise, we haven't saved human time, we've only moved it.

THE LIFE OF A CHANGE

A typical change, from an engineer's laptop (or a Claude session in Slack), goes through multiple layers of code review.

The life of a change

A typical Claude Code change, from the author's machine to users

FIG A Most of the path is automated; a person's attention goes where the risk is

LAYER BY LAYER

Layers, not one gate

来源·Hacker News·claude.dev