r/AskProgramming • u/mysticearth • 1d ago
Reviewing code by colleagues
Let me start by admitting: I'm having a hard time...
I have been reviewing a lot of code by my frontend dev colleagues the last months. While this can be a tedious job it helps both them and me while making sure we follow conventions, keep our codebase as clean as possible, etc.
Now, a lof of those colleagues create more and more using AI. Heck, some even let AI do the code reviews for them. While this boosts productivity in terms of finishing stories and delivering features, it makes me worry as well. What about code quality? How do we stay "in control" when we let AI create and review everything?
And besides the worrying, it straight out sucks to review thousands of changes and AI comments (which often don't make any sense and make everything even more complicated) manually.
While AI gives me a real productivity boost when it comes to solving problems, creating prototypes or hunting down bugs, it's safe to say I have a love/hate relationship with AI and everything that comes with it in my profession.
How do you feel about this in your daily work? I'd love to read some (other) perspectives and hope to use those to shift my complicated relationship with AI to something healthier ;-)
2
u/neon_cloud_mender 1d ago
Productivity without ownership is just technical debt accumulation. I refuse to review code where the author cannot explain the implementation details themselves
2
u/MarsupialLeast145 16h ago
The problem is really AI written posts on Reddit.
Who do you even work for? How many in your org?
1
u/FormalAd7367 1d ago
i’m the guy who reviews both front end and back end (by different developers). we mainly do fintech…it’s not fun at all’s.
1
1
u/memelordtf 18h ago
The issue is not AI-written code but the fact that nobody wants to take responsibility for it anymore. If you can’t explain why a change is there, defend the tradeoffs, or fix it when it breaks, it probably shouldn’t be merged.
1
u/Moby1029 14h ago
Read up on rules and skills and ensure AI coding agents adhere to the standards you set up in the .rules directory. This is what we've been doing and now we architect and do the final code reviews, but most of our development is all AI generated and reviewed.
1
u/mxldevs 1d ago
You are seeing real productivity boosts? That is literally all that matters.
If you're not including maintenance and code quality in your measure of productivity, you are just shooting yourself in the foot.
That's called over promising and under delivering.
People will say "tell me which business owner cares about the quality of the code" and of course the answer is none of them: they literally pay YOU to care about that stuff, because at the end of the day, if the software doesn't work, it's 100% your fault.
0
u/Lumpy-Notice8945 1d ago
The responsible person is the developer.
I dont care if they typed in something into chatGPT and copy pasted that or if they typed out every word in a notepad to not use autocomplete from the IDE.
So sure let some AI look over the code to test for bugs, but thats not a review! A review is a human validating the changes.
I have no issue with automated tests, build pipelines and things like sonaqube that check source code for errors. I dont realy care if there is an LLM behind that tool or not, all it can ever do is catch the first obvuous issues to streamline the actual review by a human. A developer review should not be about if the code does what its supposed to do or if its crashing on startup.
2
u/DepthMagician 1d ago
AI creates a tradeoff between speed and attention. You can’t stay properly in control if AI is moving faster than your limitations. There’s no way around that. The proper thing to do is figure out how much attention the code needs, and choose your use of AI accordingly.
In the past there was no need to answer that question because when you do everything by hand you have to choice but to pay attention to it. Now that there’s an alternative, the question becomes more complicated. There are genuinely scenarios where you can afford to pay little attention, but speed is not always a feature, and too many people seem to be addicted to the dopamine rush of speed and lack the wisdom to understand that just because tech debt doesn’t punish you immediately, it doesn’t mean you don’t incur its costs.
I don’t let AI generate anything that will take me more than 30 seconds to review. That lets me enjoy a productivity boost and still stay in control of what I do. To my developers I say that they can do whatever they want but they must maintain a deep familiarity with everything they create as if they did it by hand. If that means they can’t vibe code their next PR so be it.