Top
Best
New

Posted by pretext 2 hours ago

How the Claude Code team does code review(claude.dev)
126 points | 121 comments
status_quo69 42 minutes ago|
No wonder most people at Anthropic believe our jobs are in jeopardy, they've already made a system wherein the machines are treating them like pets.

I really don't get the tendency here from Anthropic to describe what's most likely churn? 1000 "PRs" a day on a TUI to wrap a while loop? I'm not going to be a reductionist here so I'm going to cut it off there, I'm positive that the complexity of handling multiple terminal emulators is something that needs some handling (even if doing their own rendering engine was a self made problem). Yes I've vibe coded my own harness from scratch without writing any of it, I still know how it works.

I'm also not sure that they addressed point 2 in any actual capacity? They say "the system verifies itself" which is cool and is something that automated testing has done for decades, but they eschew the mental model of how the system works as a whole and prefer instead to have the LLM focus and summarize a slice of the system when they need to know about it or ask for it. But.... That's not knowledge? That's just search a la google? And thus seems to be subject to the same problems where the quality of your search dictates the quality of your knowledge that you surface at any given time?

jonahx 2 minutes ago||
> have the LLM focus and summarize a slice of the system when they need to know about it or ask for it. But.... That's not knowledge? That's just search a la google? And thus seems to be subject to the same problems where the quality of your search dictates the quality of your knowledge that you surface at any given time?

This is the crux. Take pause, because hidden here is a sea-change in our fundamental trust model of software...

There is no longer anyone to turn to if this system breaks by, eg, having a critical bug that no one can figure out. No causal model in anyone's head that can be appealed to, to slowly and methodically debug. By design, the system is too large for that, and everything hinges on Claude itself being able to find and understand the right slice of the system in any situation. As long as that very new invariant holds, everything will be all right. If it fails catastrophically, there is no recovery. You are the owner of a recovered alien spacecraft with tech you have no hope of understanding.

The new trust model is: We trust Claude to always understand the system enough to debug anything or make any reasonable change we could want. The risk might be worth the reward. But it's worth explicitly laying out the deal we're making.

Terr_ 35 minutes ago|||
> wherein the machines are treating them like pets

I know some people will groan hearing the phrase again, but: "Reverse centaur." A centaur is a synergistic combination of two things, but now those things are in the wrong position: The human is the computer's tool.

Naturally, how "wrong" it is tends to depend on whether you ask the people who do all the work or the people who own all the computer-capital.

frereubu 5 minutes ago|||
My favourite quote at the moment: "May not man himself become a sort of parasite upon the machines? An affectionate machine-tickling aphid?" Samuel Butler, Erewhon, 1872
markbao 10 minutes ago||
> I really don't get the tendency here from Anthropic to describe what's most likely churn? 1000 "PRs" a day on a TUI to wrap a while loop?

From the article, the 30,000 changes a month are in the monorepo, and Claude Code is in a separate repo. So it’s not the TUI that has 1,000 PRs a day.

I could easily see 1,000 PRs a day on the backend. Big tech was doing more than that pre-AI.

status_quo69 3 minutes ago||
Fair enough, I did invent the 1000 a day because the article was vague, I'll edit my submission accordingly to also be vague
mk12 37 minutes ago||
I saw someone say the other day (in response to another blog post being well written) that Anthropic employees have a rule against using AI for writing/external comms? That seems clearly not the case here, normal people don't write like this:

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

> The eval grows with every incident, and every change to the reviewer is scored against all of it.

jazzyjackson 26 minutes ago||
100%, although if people are getting 90% of their interaction from a robot, one may start to pick up some vocal tics and colloquialisms from machine world. a kind of robo-pidgin.
paimapi 22 minutes ago|||
Pangram reports 80% of the text is AI-generated: https://www.pangram.com/history/72309aad-6e3f-4002-bdc5-c062...
wisdomvictory24 21 minutes ago||
[dead]
jansport123 17 minutes ago|||
this definitely seemed like something claude generated, i know because i frequently do all my research in chat and then ask it to generate a doc based on what we talked about and when i read what it generates the verbosity, the style it writes in makes me delete the whole thing. I would never subject readers to that.
mstaoru 25 minutes ago||
Before all came to this, I've been promoting reviews at $dayjob not as gatekeeping but as method to exchange information and build more team spirit. We are an internal service team working by "projects", which makes us a very disjointed team with barely any communication. (Part of it is also German work culture.)

And now this. Suffice to say, my efforts did not bear fruit. I feel more isolated than ever before.

grey-area 1 hour ago||
At 30k commits a day, is there anything that can meaningfully be called review?
stillpointlab 1 hour ago||
I'm thinking about high volume mass manufacturing as an example here. I like those "How it is made" type YouTube videos. You see a production line that has some kind of QA process where cameras + automated systems do spot checks at high speeds and these crazy actuators are punching out bad examples. These processes often also involve spot checks where some statistical sample is selected and viewed carefully.

I don't think we've gotten to that level of reliable review with code, but I don't see any reason we can't continue moving in that direction.

fipar 44 minutes ago|||
I really like those videos too, but a very big difference is, in manufacturing QA, the system outputs a very high number of copies of a very small number of possible types, so spot checks work fine from an operations research point of view, as the variance in output has a known upper bound.

For code, I think things go in the opposite direction: the system outputs artifacts that are all different among themselves (even more so now that we no longer distribute software via physical copies. Well, bits are always physical, but I mean we no longer ship floppies/CDs/DVDs).

That said, at least in my experience, code review has never been a direct part of factory-line-type QA. That QA is typically done by treating software as a black box and just verifying that it complies with a spec (often times based on a very loose definition of "spec" ...), and this is something that works equally well regardless of whether a human or a machine wrote the software. The type of QA that is done (among other things) by code reviews is checking whether the software's design is such that it will be economically feasible to maintain and extend it, for example.

The one part the code reviews takes a part of (but is in no way error-free) and that you just can't do with black-box testing (or it is much harder to do) is to look for things like backdoors. As our industry moves more towards "agents write software and tests validate that it works," I think a lot about "Reflections on Trusting Trust" and the new and wide attack surface added by people no longer reading the code shipped on their behalf.

grey-area 46 minutes ago||||
That’s a category error. Software isn’t stamping out the same part again and again; if it were review would be simple. Analogies often mislead.

Of course Anthropic would love you to think you are heading towards that future so we have to bear in mind their livelihood depends on LLMs being the future of work and 30k changes a day being normal.

But why would we want 30k changes a day in a given product, what possible justification could there be?

stillpointlab 28 minutes ago||
I understand the skepticism and I can't claim I am right. But some points come to mind.

First, I think we overestimate how different code is from other code. We even joke about CRUD apps and how cookie-cutter they are. A lot of changes in those apps are: add some new features to a data access layer (SQL, ORM, whatever), add new routes, add new UI surface, etc. There is an entire industry around "Design Patterns" and people live by the DRY axioms. Things like Ruby on Rails exists because rebuilding these systems over and over is wasteful. I feel we are just moving further down that same path.

Second, your question of "why would we want" is totally backwards. Progress doesn't follow your desires. Progress doesn't require justification. Sometimes we adapt the environment to our needs, other times we adapt to a changing environment. Which case we are currently in is still undecided and you are free to fight for the future you prefer. But demanding an industry justify itself before it is allowed to change fails in the face of evolution. If it works, we will justify it in hindsight.

jeremyjh 27 minutes ago|||
Those machines are not doing design work. They automate production of designs that had long development and testing lifecycles. Code is design. The compiler is the automated production process.
mirekrusin 20 minutes ago|||
It's conducting/steering/oversight/supervision, not reviewing.
swaptr 15 minutes ago||
I dont believe it, its FOMO marketing to me.
koolba 1 hour ago||
> Low - Docs, copy, internal-only features, additive changes behind a flag

> None required once automated review and security checks are green

How anybody can let an LLM write docs and not even read them is pure insanity.

YmFzZTY0ZGFkZHk 1 hour ago||
And so the wildly unprofitable AI company explains how it token maxes it's code and review process.

Authors should really try and view this article from the perspective of someone who has to think about cost to use the models.

enraged_camel 1 hour ago||
Anthropic has been profitable for the past two quarters.
tedd4u 29 minutes ago|||
"Anthropic told investors that it will profit on a measure called adjusted operating income.

"According to the report, adjusted operating income excludes stock-based compensation, which is not unusual for technology companies. However, the report also quotes its gross margins - above 80% - exclude revenue shared with distribution partners, as well as the cost of training its models."

From https://www.morningstar.com/news/marketwatch/2026091414/the-...

trog 13 minutes ago|||
I wonder how they account for free credits given in this spend. We got given US$5k free API credits from them a couple months ago and even accounting for our Claude Code subscription spend, I think we're still ahead overall - they haven't made any money from us! And they just gave us another few hundred bucks a month of API credit as part of our subscription.
rhdunn 48 minutes ago||||
How so? Can you link to the numbers?

Antropic's revenue is up [1], [2], [3] and if you calculate adjusted operating income (by excluding most expenses like training costs) then by that measure it is profitable. However, if you factor in all the numbers it is operating at a loss.

[1] https://finance.yahoo.com/technology/ai/articles/anthropic-v...

[2] https://fortune.com/2026/09/29/anthropic-leaked-ipo-prospect...

[3] https://www.cnbc.com/2026/10/15/anthropics-third-quarter-rev...

layoric 1 hour ago||||
Source? Only reports I’ve seen was they were profitable for one month in last quarter due to xAI discount of renting out colossus . Be interested to learn otherwise.
YmFzZTY0ZGFkZHk 57 minutes ago||||
Profitable as in after expenses.

https://youtu.be/T-oXyXwD6sE?si=tauzXVyGmQ6MhI_5

tracerbulletx 27 minutes ago||||
“I think that, every time you see the word EBITDA, you should substitute the words "bullshit earnings.” ― Charles T. Munger

And what they're reporting is adjusted even further than EBITDA

surgical_fire 1 hour ago||||
We're talking about being actually profitable, not fake profitable.
adammarples 1 hour ago|||
Yes when you subtract expenses and capex
markbao 39 minutes ago||
[dead]
RGS1811 1 hour ago||
How is the ability of a human to understand the code via hands-on familiarity not a direct dependency of their ability to do manual code review? I don't get it. Having claude write an explainer (something I do a lot!) doesn't get you to the point where you can confidently assess a complex diff. And you've funneled your priors on what the diff says and does through the same system that already reviewed and approved it. This article isn't bad but it doesn't give enough weight to the human context-building part of the problem.
jazzyjackson 19 minutes ago|
Yes it used to be said, the code is not the asset, the people who know what the code does and how to change it are the asset.

Now we’re just going to outsource our capacity to design and review because it’s faster that way, as if the thing that was wrong with the software industry was that releases weren’t being pushed out often enough.

vatsachak 22 minutes ago||
Mfw tui needs a billion prs per day.

Opus 5.5 is obviously stellar but come on what are you doing lol

jwpapi 1 hour ago||
What I’m more worried about is feature identification and how to monitor them. How can you get the product to the next level?

Honestly I’m not worried about code anymore in my codebase I’ve not had bug problems for a while and incidents get fixed automatically.

I monitor new features with an agent who checks recordings.

Most actually good features need still 20-30 iterations to design the userflow to actually be interactive, but then there is also another challenge to translate usage patterns into features and how to present functionality with the least amount of friction as possible.

How to efficiently run monitors without wasting your token spends. A best 'monitor' best practice would be nice.

Still this article was very helpful and gave me a lot of new ideas.

vavkamil 2 hours ago|
> Security gets its own automated review too. ... You either fix it or close the thread with a short reason, which is how you clear a false positive.

I hope this is only for low-risk false positives. Dismissing a critical finding as a false positive should still involve AppSec.

You could improve revert rates while simultaneously introducing more security vulnerabilities.

And the linked "claude-code-security-review" explicitly states in the README

> This action is not hardened against prompt injection attacks

That's a significant limitation for an AI security review control, no?

More comments...