Yes and it’s a giant bottleneck.
What’s depressing is that the AI agents are actually way better than humans at catching weird edge cases that are easy to overlook. So the AI code reviews are actually super valuable.
But it’s still an LLM and still doesn’t understand anything and can’t be accountable, so humans are still in the picture and that slows everything down.
All that, I think, is fine. The true problem I’m seeing is that reviewers are starting to get lazy and sign off on things because the AI gave it a pass.
I think it’s fine to lean on the AI to catch weird race conditions. It’s less fine to blindly accept that 2 lines of code requires a 12 line comment explaining a bunch of bullshit about what the code used to do.
Standards are dropping. It feels like a race: will developers become incompletely lazy before the AI is actually good enough to do their old jobs?
It depends on the company. If the company is run by morons, they’ll have the AI do the code reviews.
It’s all the fucking job is now
I long for the days when I reviewed something, noticed an issue, commented, the recieving engineer learned something and we all got better as a result
Now I basically get the privilege of wasting 10 mins and telling someone they fucked up in a way unfathomable to them, because they haven’t got a clue what they just put in front of me
Oh and it’s my manager’s manager doing it
This may explain why the same code review feedback needed to be written in every single PR made by our Indian offshore team. Either they didn’t give a shit, were incapable of learning, or used AI (which implies not giving a shit). I don’t understand how people can have so little pride in their work. Please do not drag other people down because you don’t want to do your job.
Merge requests have exponentially increased too because people are using AI, and obviously a lot of them don’t even care to review the code they generate. So what do the reviewers do just to be able to keep up? Fight fire with fire. Oh so you used AI to create these 50 MRs? Guess what, we have an AI bot that has a security-aligned skillset and it found all these issues in your code. Go fix them.
i needed this laugh 😂😂😂
We don’t really use AI lol. It’s great for the simple task - but I use it for giving me tldr on documentation than anything now. It’s basically a better Google
We do ai code reviews
They are if you want functional code.
I have never seen a project without serious reviews. But you should definitely trust the AI and stop doing those stupid code reviews. This way, I will have more jobs and more money to fix all the mess that was pushed in the repositories.
Even more so. AI may be able to assist with complex code but it does not have the ability to determine the nuance of how the code it creates will impact the overall code base.
Basically programmers are becoming designers and code reviewers.
There are now so many code changes that the code review is the bottleneck more than the coding.
We depend more now on tests to validate software does what it should. We get AI to write the gazillion tests.
And we get AI to summarize and explain blocks of code.
Are you new to engineering? Coding was never the bottleneck. Having someone who knows what to do, and planning usually is.
I spend 20-30% of my time at most actively coding. 30-40% helping people with dumb things that take my time when they shouldn’t, and the remainder in meetings, scoping my work, or debugging.
That is just the thing, developer and tester should never be the same person. Let alone same AI model. IMO testing is never taken seriously enough, just seen as unnecessary step and merged together with dev testing. From my years of experience I know that everything testers find is just explained away rather than properly adressed, and then with all of the obvious stuff in the way you never see the real issues.
Interested in how much actual experience you have with AI geneated testsuites.
My code was never tested this well.
I have experience with AI generated test suites, and while its good for generating coverage, it isn’t so good for actually ensuring correctness, which is the actual point.
I’ve watched the robot happily introduce bugs to pass broken tests, and also break tests to match code, and everything in between.
I don’t want lots of tests, I want good tests.
You have to prompt for that, I do that regularly along with refactors. ‘Examine all tests to ensure they are testing functionality and not just passing a test.’ It finds them and will work on it. I think the problem continues to be engineering discipline. People are lazy with AI on multiple levels, not just copy pasta slop.
Yes, but the situation is getting strange.
Our model has always been that the reviewer is responsible for protecting the repository. This led to one IC getting fired for “letting in” a catastrophic bug his teammate generated with Claude.
That seems … backward.
I’m sorry that you work at a company that thinks code reviews and tests guarantee bug-free code.
The primary function of code reviews is to increase visibility, and therefore maintainability. It keeps other members of the team in the loop about how codebase is changing and how it might affect their current or future work.
The primary function of tests is to catch regressions, not new bugs, and especially not new bugs related to use cases that weren’t even possible in previous versions
If you mean right now, yes code reviews are still a thing.[*] More so than ever, really. If you mean some hypothetical future age, yes code reviews will still be a thing.
[*] Among people who know what they’re doing.
This is kind of why I asked. What I’m seeing (as a non-programmer) is management pushing AI and LLM use so hard that I could imagine someone high enough up mandating away code reviews.
I’m glad you (and most responders) seem confident the the levy will hold…
They don’t know enough about the process to do that. They just want us to “use ai when doing code”. Reviews, merge requests, etc. aren’t even in their vocabulary.
Oh my - I like the sound of that structure. Around here managers don’t have to “know enough” to make decisions; they just have to think they do. And boy do they.
We’re still in the process of un-lift-and-shifting their big-brain “lift and shift” cloud plan back on-prem.
We’re all AI babysitters now. For a baby that never learns anything.
Arguably you could improve the instructions it follows by fixing it when AI does something wrong. I think I use AI more for design specs and acceptance criteria than I do for writing actual code. Though, to be fair, my current role as tech lead offers very little opportunity for writing code other than one-off scripts.
It’s not going to learn long term until AI starts meaning something smarter than an LLM. I’m not trusting my phone’s autocomplete to be able to replicate my knowledge and experience.
And if I do have to perform a teaching duty, I’d rather teach a human that will actually get better, have fresh insights and eventually pass it on to another human.
That may fix the current task, but the next one is going to need all those corrections repeated back to it. That’s what I mean by never learning. Each task is its own little world that’s completely disconnected from everything else. Copilot has “memories” that seem like they are trying to fix it, but I’ve never had any luck with it actually working.
For ease of differentiation we should have another name for AI governed repositories. As they are clearly superior to the mere human crap you so-called experts use to lock in that sweet job security, I propose we call them suppositories - an obvious and illustrative contraction of “superior repositories”.
I think currently companies tend to have the same process they always had.
In my case, that tends to be that you create a PR, someone assumes you knew what you were doing and approves it with no comments.
I actually expect the fact that we’re promoting AI internally might result in this tightening up.
Probably unlike others, I work in an R&D team of only about 12 people total (all disciplines) so probably not the usual experience.
Yes, code reviews are also useful for sharing knowledge among team members. Also, no all projects can use AI due to fears of copyright strikes
