> I can recognize within seconds if a PR is an easy merge or a 5 minute phone call

I could, before LLM-authored PRs. They broke all my heuristics for what to check. LLMs can write code that looks incredibly professional and well thought out, and nowadays they almost never make the usual human mistakes (like off-by-one errors). But at the same time, they can make incredibly silly mistakes of the kind people would never make (like inventing brand-new parameters for some remote API call).

It feels like everything I’ve learned over the years about reviewing code no longer applies.

One thing that's interesting is code review is still very static in my experience across gitub, gitlab, and bitbucket. Moving from a text editor to an IDE with LSP made a big difference for coding for me with click-through links and autocomplete of valid functions and variables. Raw text diff for review feels very old in comparison, especially as PRs generated by LLM agents get bigger and bigger compared to human PRs of the past.

I watched someone vibe code with a top of the line model, something they did quite regularly, and it was the most insane mess of code I have ever seen.