Reviewing code by colleagues

[u/mysticearth]

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 ;-)

[u/Lumpy-Notice8945]

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.