News for hackers. Stuff that matters.
Re:Writing Code Is Easy. Reading It Isn't by ppeetteerr on Monday September 08, 2025 @03:10PM (#45172522)
This is not unique to the age of LLMs. PR reviews are often shallow because the reviewer is not giving the contribution the amount of attention and understanding it deserves.

With LLMs, the volume of code has only gotten larger but those same LLMs can help review the code being written. The current code review agents are surprisingly good at catching errors. Better than most reviewers.

We'll soon get to a point where it's no longer necessary to review code, either by the LLM prompter, or by a second reviewer (the volume of generate code will be too great). Instead, we'll need to create new tools and guardrails to ensure that whatever is written is done in a sustainable way.

Re:Writing Code Is Easy. Reading It Isn't by gyomu on Monday September 08, 2025 @03:22PM (#45172689)
> We'll soon get to a point where it's no longer necessary to review code, either by the LLM prompter, or by a second reviewer (the volume of generate code will be too great). Instead, we'll need to create new tools and guardrails to ensure that whatever is written is done in a sustainable way.

The real breakthrough would be finding a way to not even do things that don’t need to be done in the first place.

90% of what management thinks it wants gets discarded/completely upended a few days/weeks/months later anyway, so we should have AI agents that just say “nah, actually you won’t need that” to 90% of our requests.

Re:Writing Code Is Easy. Reading It Isn't by foxfired on Monday September 08, 2025 @03:18PM (#45172636)
One thing to take into account is that PR reviews aren't there for just catching errors in the code. They also ensure that the business logic is correct. For example, you can have code that pass all tests, and look good, but they don't align with the business logic.
Re:Writing Code Is Easy. Reading It Isn't by prybeng on Monday September 08, 2025 @04:21PM (#45173435)
I wonder if the paradigm shift is the adoption of a higher level language. Akin to what python did to blackboxing C libraries.
Re:Writing Code Is Easy. Reading It Isn't by Ianjit on Tuesday September 09, 2025 @08:45AM (#45181197)
I'm not a programmer but I always had the impression that different languages were appropriate for different tasks. My question is, "For what type of programming tasks is English the correct level of abstraction?"
Re:Writing Code Is Easy. Reading It Isn't by Bukhmanizer on Monday September 08, 2025 @07:36PM (#45175529)
> We'll soon get to a point where it's no longer necessary to review code, either by the LLM prompter, or by a second reviewer (the volume of generate code will be too great). Instead, we'll need to create new tools and guardrails to ensure that whatever is written is done in a sustainable way.

This seems silly to me. In most cases, the least amount of work you can possibly do is logically describe the process you want and the boundaries, and run that logic over the input data. In other words, coding.

The idea that we should, to avoid coding or reading code, come up with a whole new process to keep generated code on track - would almost certainly take more effort than just getting the logical incantations correct the first time.

Re:Writing Code Is Easy. Reading It Isn't by vjvjvjvjghv on Monday September 08, 2025 @07:39PM (#45175556)
“ With LLMs, the volume of code has only gotten larger ”

It’s even worse with offshore devs. They produce a ton of code you have to review every morning.

Re:Writing Code Is Easy. Reading It Isn't by fzeroracer on Monday September 08, 2025 @07:43PM (#45175589)
Can you define what an "error" is?
Re:Writing Code Is Easy. Reading It Isn't by ppeetteerr on Monday September 08, 2025 @08:37PM (#45176012)
Logic error, for instance
Re:Writing Code Is Easy. Reading It Isn't by fzeroracer on Monday September 08, 2025 @08:52PM (#45176103)
Well, it depends on the logic error doesn't it? And it depends on how the system is intended to behave. A method that does 2+2=5 is a logic error, but it could be a load-bearing method in the system that blows up when changed to be correct.

Something like blowing up the stack or going out of bounds is more obviously a bug, but detecting those will often require inferences from how a code behaves during runtime to identify. LLMs might work for detecting the most basic cases because those appear most often in their data set, but whenever I see people suggest that they're good at reviewing I think it's from people that don't deeply review code.