Jev Based Code Review

lm2s•about 1 hour ago

I think your engineering process is fundamentally broken if you are generating PRs with 230 files changed so regularly that you need to bolt on more AI. You’re solving the wrong problem.

stuaxo•about 1 hour ago

That really stood out.

Code is there to be read and understood by the human developers who come later.

The git history is a similar record, that's why the commits that make it to main (the squashed PRs) should cover one(ish) thing each and be self contained.

Something covering 230 files should be a mechanical change like running a linter or the AI is moving an API from one signature to another.

If an LLM generated a 230 file change they are also capable of going back and breaking it up.

One thing they are bad at is comments that are succinct since they almost only ever add words.

realusername•about 1 hour ago

I can confirm that our code quality at work is regressing and we are shipping less product features than before with AI.

ncruces•about 1 hour ago

Welcome to 2026.

My dev branch got broken when I rebased to main after a week of drift.

I had to bisect over around a million commits to the monorepo to find the culprit.

yube01•21 minutes ago

confidence score really feels like real machine learning project which we test to check the confusion matrix

namanbhulawat•about 4 hours ago

Most PRs that are generated by agents today get YOLO merged because its hard for human mind to comprehend when your agent just suddenly shows up with 230 file changes. This is an attempt to reduce the mental burden by classifying each change in a review to P0, P1, P2. Only P0 are shown by default. The priorities are configurable. The diffs are also show using a natural language. The original code is one toggle away.

fny•about 1 hour ago

You can tell it not to do this?

adityamishra241•about 4 hours ago

Nice idea. This could make large agent-generated PRs much easier to review.

manojlds•about 1 hour ago

Slop projects to solve slop

Jev Based Code Review

⚡ Community Insights

Discussion (13 Comments)Read Original on HackerNews