frontpage.
newsnewestaskshowjobs

Open Source @Github

fp.

Open in hackernews

Ask HN: How do you code review?

4•idose•15h ago
In the past year, from my personal experience in 2 different companies, the amount of PRs has increased dramatically, and each PR is getting much bigger.

How do you approach code review when dealing with large and high volume AI-generated code?

Comments

davydm•14h ago
Review is the bottleneck in those scenarios - and it's a _good thing_.

AI-gen code is often subtly (or obviously) incorrect. Human eyeballs are the cure. There is no way to go faster without compromising the value of the review.

What I've been super annoyed about recently is the number of times I've seen a completely stupid question on a PR - eg "why are you doing this?" when there's a comment right above that line about the why. Or "where else is this used?" - we all have IDEs and reviewers obviously have access to the code - stop being a lazy potato and check out the code locally.

I try to avoid adding these lazy people to PRs as they create more work than they help. But sometimes they end up on there anyway, because they're "important".

othmanosx•4h ago
Have you tried reviewing code on https://pyor.review instead of checking out the code locally?
saidnooneever•13h ago
if this is a problem there is a problem in alignment in management or teams.

one team uses AI to create larger volume of work, but another does not use AI for whatever reason.

its really asking for problems.

either both should use it or neither.

if both is not perceived to be ok for quality reasons, then use neither.

dylanratcliffe•2h ago
We did some measurement at the beginning of the year and found that our PRs were getting bigger and review time was coming down even without us doing anything, which is not a good sign.

We ended up moving peer review to the implementation plan rather than the PR, then having a loop that validates the code against the plan when the PR is raised. That way the agent gets a CI failure if it deviates from the plan, which it then fixes or acknowledges. Anything with no differences gets merged without human review, differences get approved by the original person who peer reviewed the plan.

I puled some stats the other day for a presentation I'm working on about what we did:

Matched plan on first pass: 25% (169/677) Had differences: 75% (508/677)

Differences per PR: Median 2 Mean 2.60 P90 6 Max 20

1,757 findings:

- Missing (skipped planned work): 44.6% of findings, 51.7% of PRs - Changed (done differently): 42.7% of findings, 51.3% of PRs - Beyond (extra, still in scope): 9.1% of findings, 19.8% of PRs - Scope (unplanned feature): 3.6% of findings, 6.6% of PRs

Ask HN: Why do tools like customer.io abandon self-serve after their Series A?

6•alessandroetc•2h ago•1 comments

I keep getting flagged for AI, but I suck at writing. How can I solve this?

4•quesobob•1h ago•6 comments

Ask HN: How about delayed MIT/GPL?

2•afiori•4h ago•1 comments

Ask HN: Where else do you spend time on the internet other than HN?

6•Gecko4072•7h ago•10 comments

Tell HN: Even HN is getting heavy traffic by AI crawlers

8•ksec•4h ago•3 comments

Ask HN: How do you use artifacts, and how do they help in your projects?

2•AmirMakirr•5h ago•0 comments

Ask HN: Phished Microsoft account, recovery useless

3•tgmatt•7h ago•0 comments

Requests for Startups: Energy

2•NarcisMirandes•7h ago•3 comments

Ask HN: Are you still using AI code autocomplete?

4•jawuilp•8h ago•11 comments

Tell HN: Google Search to begin using opaque redirect URLs on search results

5•exploraz•10h ago•3 comments

One MCP directory for all docs.domain.tld (No auth needed)

3•respectattentio•10h ago•0 comments

Ask HN: What are you using to access x.com at the moment?

5•throwaw12•14h ago•7 comments

Looking for people to follow designing systems to ship code with AI agents

7•mutahn•15h ago•1 comments

Ask HN: Why are Claude models so verbose?

4•JacobWolf•17h ago•8 comments

Ask HN: Will RAM / NVMe / Platter Drives ever come back down in price?

4•krautburglar•9h ago•4 comments

Ask HN: Good large format (>20 inches) touchscreen E-Paper display options?

30•foota•1d ago•8 comments

Open source in-browser log sanitizer, feedback required

2•andriimb•13h ago•2 comments

Ask HN: Is what Open Commonplace legal?

2•VishnuShayeq01•13h ago•0 comments

Ask HN: OpenBSD or FreeBSD?

3•simonebrunozzi•8h ago•3 comments

Ask HN: Why do you use Ubuntu?

3•flanked-evergl•14h ago•4 comments

Ask HN: Opus 5 is unusable for writing, even internal use. Alternatives/fix?

7•m3h•12h ago•1 comments

Ask HN: Those making $500/month on side projects in 2026 – Show and tell

72•kaladan•2d ago•87 comments

Ask HN: How do you code review?

4•idose•15h ago•4 comments

Xiaomi's Xring Series suggests more OEM's switching to in-house SoCs

2•StizzurpXDD•17h ago•0 comments

What platform has the best AI CMO?

2•probablygrillin•19h ago•0 comments

Ask HN: What business would you build if you couldn't use the internet?

6•spking•1d ago•9 comments

You've reached the end!