AI code review (Claude, maybe Codex)

[Valent@MeshPoint]

Any LLM is as good as person using it, yes they hallucinate, but if you 
know how to use  LLMs properly and have domain knowledge in which you 
are using it they it can't produce "bad code". Also if you give it your 
perferences and coding style it can follow it quite nicely.

Just my 2c.


------ Original Message ------
>From "Hauke Mehrtens" <hauke at hauke-m.de>
To "Eric" <evil.function at proton.me>
Cc "OpenWrt Development List" <openwrt-devel at lists.openwrt.org>
Date 12.4.2026. 1:22:43
Subject Re: AI code review (Claude, maybe Codex)

>On 4/9/26 20:04, Eric wrote:
>>After reading various AI generated reviews, I'd say my only functional objection is that they are so f'ing verbose.
>>
>>If I (speaking as a human reviewer) see code like this in a PR:
>>
>>     cp $file $dest
>>
>>I just say "Please quote the variables to avoid issues with blanks, wildcard expansions, etc (do you know about 'shellcheck'?)", assuming a high level of comprehension on the submitter.  When an AI sees that, it appears to assume the coder has the expertise of a 3-year-old, so produces a PhD dissertation as to why you should add quotes.  Reviewing the review becomes 10x more burdensome than it should.
>>
>>
>
>Hi,
>
>I played around with the Claude code github action again and added an additional system prompt to do less verbose messages. They should also be cheaper. ;-)
>
>It now also should do inline comments. See here my test review:
>https://github.com/hauke/openwrt/pull/10
>
>The inline comments are also nice, you can answer on each of them and then mark then as resolved.
>
>Hauke
>
>_______________________________________________
>openwrt-devel mailing list
>openwrt-devel at lists.openwrt.org
>https://lists.openwrt.org/mailman/listinfo/openwrt-devel