Skip to content

fix: Extend AI communication #63211

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
artonge wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix: Extend AI communication #63211

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/CONTRIBUTING.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ Nextcloud allows contributions made with the help of AI tools. You are the autho

* **Accountability:** You must be able to explain, defend, and modify every line you submit. If a reviewer asks why something works a certain way, "the AI wrote it" is not an answer.

* **Communication:** PR descriptions, review comments, and issue reports must be written in your own words. This applies throughout the review process - passing reviewer feedback to an AI and posting whatever comes out is not acceptable.
* **Communication:** PR descriptions, review comments, issue reports, code comments and commit messages must be written in your own words. This applies throughout the review process - passing reviewer feedback to an AI and posting whatever comes out is not acceptable.

@susnux susnux Aug 13, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why no code comments?

We recently adjusted that in the AI_POLICY to make them useful. And from experience if you properly guide the model on what to add as comments they are quite good.

@AndyScherzinger AndyScherzinger Aug 13, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same thought on the comment - but also on the commit messages. Yes they get veeery lengthy when you ask AI to do them (while you can of course tell it not to do that). When I i.e. worked on Talk Android aspects I asked explicitly if it should be a short (like 1-liner) commit message or if I should push long(er) AI generated commit messages - the feedback has been: the long8er) AI ones (to have this "for later"/future).

So my interpretation from that would be, to still allow them, while the 1-line semantic commit message should be what is relevant for the reviewer(s) while the lenghty version serves as a documentation if one ever would need to dig into the commit history to gain more context by having more git based documentation


* **Quality:** AI output must be quality assured by the human, i.e. reviewed, cleaned up, and tested before submission. New features must be tested on a live instance by you, not by an agent. Code that has never been executed, or that shifts debugging work onto maintainers, will not be accepted.

Expand Down
Loading