Comment by trey-jones
8 hours ago
To me, an old guy, I would rather have LLM doing (assisting with) the code review than the actual code production. Is that stupid?
8 hours ago
To me, an old guy, I would rather have LLM doing (assisting with) the code review than the actual code production. Is that stupid?
LLMs are great at reviewing. This is not stupid at all if it's what you want; you can still derive benefit from LLMs this way. I like to have them review at the design level where I write a spec document, and the LLM reviews and advises. I don't like having the LLM actually write the document, even though they are capable of it. I do like them writing the code, but I totally get it; it's no different than me and the spec documents.
Right, I'd say this is the best value I've gotten out of it so far: I'm planning to build this thing in this way, does that seem like a good idea to you? Sometimes I get good feedback that something else would be better.
If LLMs are great at reviewing, why do they produce the quality of code they produce?
Reviewing is the easier task: it only has to point me in the right direction. It's also easy to ignore incorrect review suggestions.
Imho it's because you worked before asking the LLM for input, thus you already have information and an opinion about what the code should look like. You can recognize good suggestions and quickly discard bad ones.
It's like reading, for better learning and understanding, it is advised that you think and question the text before reading it, and then again after just skimming it.
Whereas if you ask first for the answer, you are less prepared for the topic, is harder to form a different opinion.
It's my perception.
1 reply →
This makes sense to me.
I need to make decisions about how things are implemented. Even if it can pick “a way” that’s not necessarily going to be a coherent design that I want.
In contrast for review I already made the choices and now it’s just providing feedback. More information I can choose to follow or ignore.
Take a look at CodeRabbit and Sourcery if you want to give that a go.