[whateverboat]

This was the original rule in linux kernel as well. No more than 200 loc per patch. We should also introduce this to git commits and pull request descriptions:

1. 400 chars/10 lines per commit

1b. Not more than 3 commits in the initial pull request

2. 20 lines of explanation for pull request

3. not more than 3 pull request open at any one time

[TomasBM]

Seems like this policy would apply pretty well regardless of who/what generated the code.

[mexicocitinluez]

YES! "No AI" policies that are purely based on technical grounds make no sense to me. Bad PR's are bad PR's regardless of their source.

Are we really in a situation where good code that solves a problem won't be merged because the person the person checked the "I used AI" box on the PR?

Ban PR's that are too big, don't have a clear purpose, touch too many areas, etc.

[overgard]

It's really a question of how much time you're willing to spend sorting through spam. "No AI" might be a blunt hammer, but the people submitting slop aren't reading guidelines anyway, and it's easier just to reject it early. Frankly, I'm sure if people wanted to sneak in an AI generated code by carefully reviewing it and making sure it's targeted and well tested... I'm sure they could, but those people aren't the problem.

[mexicocitinluez]

> Frankly, I'm sure if people wanted to sneak in an AI generated code by carefully reviewing it and making sure it's targeted and well tested

But this is exactly the point I'm making. If the code is carefully reviewed, targeted, and tested, then why make people have to lie in order to submit PRs?

Why not just say "Irresponsible use or agentic-based PR's will be auto-rejected"?

And that's not even mentioning that tools like Github Copilot can just act like fancy autocomplete. There are dozens (if not hundreds) of different ways to use these tools.

I guess I'm just really not sure how you can unequivocally forbid these insanely powerful tools when they are almost certainly going to be a large part of developer's workflows going forward?

[ytpete]

> Why not just say "Irresponsible use or agentic-based PR's will be auto-rejected"? ... tools like Github Copilot can just act like fancy autocomplete.

For what it's worth, this is basically what the Godot policy does say. It's not a blanket ban:

> No autonomous AI agent use or vibe coding

> No use of AI to generate substantial pieces of code ... AI assistance should be limited to menial things (like code completion, regex, or find and replace).