Ben RamseyMar 3

Before AI, I felt like I could review PRs and, based on whose code I was reviewing, I could be more trusting and casual with my review, since I knew they usually wrote good code and paid attention to details. Now, with AI, no one pays attention to details, so I have to deeply scrutinize every PR, which takes twice or 3x the effort.

AI agents almost always add things that aren’t necessary, or they follow anti-patterns based on the surrounding legacy code context.

Show threadJulietteMar 3

@ramsey I have the same experience. Reviewing AI-generated/assisted PRs takes about five times as long as reviewing a human created PR as there is just so much wrong with them.

In nearly all cases, I could have written a better and cleaner fix myself in less than half the time the initial review cost me.
And then the PR still has to go through additional review rounds before it's merge-ready.

It's a time-sink on open source's most precious commodity: maintainer time.
And this is unacceptable.

Show threadJaapioMar 3

@jrf_nl @ramsey the responses of the "authors" are even worse. I closed multiple pr's last months something I didn't do for a very long time.
But the arguments against closing are so offending you that I have been thinking about stopping with reviews.

I nearly lost the joy in working on my projects. Because of the shit posts I got.

Show threadDerick Rethans
@jaapio @jrf_nl @ramsey For research purposes, do you have an example?
Mar 3 at 8:29pmWeb
Show threadJaapioMar 3

@derickr But I think this one will give you an impression: https://github.com/phpDocumentor/guides/pull/1295

Especially the talking with a bot, not a human... Long walls of text that say nothing. And maybe something I take too personal, the suggestion that "I don't care". Because all the time they spend...

Talking about walls: https://github.com/TYPO3-Documentation/render-guides/pull/1143

In the moment, responses might feel like an attack, but reading them back. Most of this is just a waist of time.

@jrf_nl @ramsey

Show threadAlerta! Alerta!Mar 3

@jaapio

"You don't care Because all the time I spend..." should qualify for an insta-block (unless they also provided a 50,000€ submission without strings attached).

Sadly those fuckers will set up the next account with another throwaway emailaddress in minutes and send in the exact same PR 35 times.... 🤬

🫂

/cc @derickr @jrf_nl @ramsey

Show threadBen RamseyMar 3
@heiglandreas @jaapio @derickr @jrf_nl I hate how offended folks who use AI get when you reject their work. They act as if it’s discriminatory.
Show threadDerick RethansMar 3
@ramsey @heiglandreas @jaapio @jrf_nl Yup. It's much better to tell them (through CONTRIBUTING.{md,rst}) isn't welcome. Then you can close it with AI;DR.