Skip to content

doc/guides/ai_policy: Make human communication mandatory#22433

Open
AnnsAnns wants to merge 1 commit into
RIOT-OS:masterfrom
AnnsAnns:ai_policy_stronger_wording
Open

doc/guides/ai_policy: Make human communication mandatory#22433
AnnsAnns wants to merge 1 commit into
RIOT-OS:masterfrom
AnnsAnns:ai_policy_stronger_wording

Conversation

@AnnsAnns

@AnnsAnns AnnsAnns commented Jul 2, 2026

Copy link
Copy Markdown
Member

Contribution description

Recently we have had a wave of new first time contributors, which would be great in theory if it wasn't for the fact that at the same time we have seen a rise of heavy LLM usage. While our current policy mostly handles that, something that I personally did not expect to be such a huge problem is the over-reliance on LLM output for human to human communication. Some PRs that appear fairly alright by our LLM policy standards then hit you with walls of clearly AI generated texts in the comments when trying to review the PR, making the reviewing process extremely frustrating and rage inducing, feeling more like human to llm communication. And I don't need to be a prompter for somebody else, I can also prompt claude myself 💢

Testing procedure

make doc-starlight

Issues/PRs references

Declaration of AI-Tools / LLMs usage:

AI-Tools / LLMs that were used are:

  • Human

@AnnsAnns AnnsAnns requested review from aabadie and jia200x as code owners July 2, 2026 09:28
@github-actions github-actions Bot added the Area: doc Area: Documentation label Jul 2, 2026
@AnnsAnns AnnsAnns added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs labels Jul 2, 2026
@riot-ci

riot-ci commented Jul 2, 2026

Copy link
Copy Markdown

Murdock results

✔️ PASSED

ae94968 Update doc/guides/general/ai_policy.md

Success Failures Total Runtime
1 0 1 01m:17s

Artifacts

Comment thread doc/guides/general/ai_policy.md Outdated
@crasbe crasbe added the Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation label Jul 2, 2026
Comment thread doc/guides/general/ai_policy.md Outdated
@maribu maribu added Process: API change Integration Process: PR contains or issue proposes an API change. Should be handled with care. Process: needs >1 ACK Integration Process: This PR requires more than one ACK labels Jul 3, 2026
@maribu

maribu commented Jul 3, 2026

Copy link
Copy Markdown
Member

Not really an "API" change, but I think the "needs two ACKs" rule for breaking API changes should apply here as well, so I just abused the label.

Co-authored-by: Marian Buschsieweke <maribu@users.noreply.github.com>
@AnnsAnns AnnsAnns force-pushed the ai_policy_stronger_wording branch from ae94968 to 8cf937a Compare July 3, 2026 11:13
@AnnsAnns AnnsAnns enabled auto-merge July 3, 2026 11:14
@AnnsAnns AnnsAnns disabled auto-merge July 3, 2026 11:14
@AnnsAnns

AnnsAnns commented Jul 3, 2026

Copy link
Copy Markdown
Member Author

Not really an "API" change, but I think the "needs two ACKs" rule for breaking API changes should apply here as well, so I just abused the label.

I agree, do we have that in our policy though or is this vibe based?

@maribu

maribu commented Jul 3, 2026

Copy link
Copy Markdown
Member

I did not find any written documentation about our process with regard to API changes last time I checked.

We do have some documentation with regard to changing GOVERNANCE.md. That would probably be a closer match than an API change here, anyway.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: doc Area: Documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs Process: API change Integration Process: PR contains or issue proposes an API change. Should be handled with care. Process: needs >1 ACK Integration Process: This PR requires more than one ACK Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants