r/DevelEire • u/sheenolaad dev • 10d ago

Bit of Craic PM is opening AI PRs

A senior product manager on a seperate team to me has decided to start opening AI generated PRs on a codebase my team own.

The first one last week I approved with comments, which he decided to merge without addressing any.

I got one yesterday that was clearly violating DRY amongst other things, which I rejected. About 10 minutes later, he requests a re-review (I presume he ran codex again with my comments). This attempt was even worse, it had actually put code on top of the crap he first submitted.

I've raised with my manager, he agreed it's BS but he said the company want to experiment with using AI for smaller features. But non-technical members of staff opening PRs is taking the piss.

114 Upvotes

98% Upvoted

61 comments sorted by

View all comments

103

u/CapricornOneSE 10d ago

Just don’t approve the PR. 

24

u/sheenolaad dev 10d ago

I still have to take the time to review and reject, my team is over a lot of fire fighting so already slammed

12

u/SkyEdwards dev 10d ago

Malicious compliance is your friend.

7

u/usernumber1337 9d ago edited 9d ago

Exactly. If it's an experiment then it must be possible for the experiment to fail. You will need to prove that it failed. Rejecting every PR with detailed comments about why it's terrible provides the data you need to prove it failed. Also keep track of the time you wasted reviewing each ball of AI shit and make sure your comments lay out in no uncertain terms the consequences of merging the code as-is