Ask HN: How are you doing code reviews in the era of GenAI?

1•lenerdenator•7mo ago

I'm a more senior member on my team, and like many software engineering companies, we're using various GenAI technologies like LLMs and agentic AI to help with coding tasks.

The good news is, it does increase the output of less-experienced developers. The bad news is, it also increases the output of less-experienced developers.

We use a fairly standard code review process: make a PR, have it pass CI/CD tests, have other developers review, and merge into main if approved. The problem is that we are getting a larger volume of PRs to review and it's beginning to cut into the productivity of more senior members of the team.

How do you handle this increase in PR review activity?

Comments

codingdave•7mo ago

I'd question your definition of productivity. If reviewing PRs is part of your job, and you are doing it, but some measurement of your productivity is down, then your measurements are wrong, not the code reviews.

I've seen teams try too hard to keep the seniors "productive", and they just end up with the juniors/mids bottlenecked because their code never gets reviewed. So if you measure productivity by overall flow of code through your process (think Kanban, not Scrum), you'll be able to resolve this conundrum.

lenerdenator•7mo ago

I guess the problem I see with that is that the bigger "lifts" that we count on to carry out business strategy are typically carried out by the senior engineers, and reviewing PRs, while technically productive, delays those bigger lifts.

codingdave•7mo ago

Why wouldn't the senior break off smaller tasks within the "lift" and delegate those parts to juniors? Then reviewing those PRs is a productive part of the lift. Or, if you really cannot break down a task into smaller parts, have one senior take over more PR work while another does the lift.

lenerdenator•7mo ago

Right now we're doing a lot of architectural stuff in the big lifts surrounding design patterns and stuff that perhaps junior devs might not understand as well.

We're doing more of the latter suggestion, though that is coming with more "reviewer's burnout". We are trying to deal with it through smaller PRs, but then of course, you fall back to the problem of having more of them.

dearilos•7mo ago

Gather all my acquired tribal knowledge and put it into a code reviewer [1]

This way you're not asking juniors to fix every little thing and they can learn from standards that already exist.

[1] https://wispbit.com

The Path to Mojo 1.0

Show HN: I'm 75, building an OSS Virtual Protest Protocol for digital activism

Show HN: I built Divvy to split restaurant bills from a photo

Hot Reloading in Rust? Subsecond and Dioxus to the Rescue

Skim – vibe review your PRs

Show HN: Open-source AI assistant for interview reasoning

Tech Edge: A Living Playbook for America's Technology Long Game

Golden Cross vs. Death Cross: Crypto Trading Guide

Hoot: Scheme on WebAssembly

What the longevity experts don't tell you

Monzo wrongly denied refunds to fraud and scam victims

They were drawn to Korea with dreams of K-pop stardom – but then let down

Show HN: AI-Powered Merchant Intelligence

Bash parallel tasks and error handling

Let's compile Quake like it's 1997

Reverse Engineering Medium.com's Editor: How Copy, Paste, and Images Work

Go 1.22, SQLite, and Next.js: The "Boring" Back End

Laibach the Whistleblowers [video]

Slop News - The Front Page right now but it's only Slop

Economists vs. Technologists on AI

Life at the Edge

RISC-V Vector Primer

Show HN: Invoxo – Invoicing with automatic EU VAT for cross-border services

A Tale of Two Standards, POSIX and Win32 (2005)

Ask HN: Is the Downfall of SaaS Started?

Flirt: The Native Backend

OpenAI's Latest Platform Targets Enterprise Customers

Goldman Sachs taps Anthropic's Claude to automate accounting, compliance roles

Ai.com bought by Crypto.com founder for $70M in biggest-ever website name deal

Big Tech's AI Push Is Costing More Than the Moon Landing