Skip to content

docs: update contributor guidelines for AI policy and PR adoption - #1307

Open
bengbengbalabalabeng wants to merge 7 commits into
apache:mainfrom
bengbengbalabalabeng:chore-update-contribution-guidelines
Open

bengbengbalabalabeng wants to merge 7 commits into
apache:mainfrom
bengbengbalabalabeng:chore-update-contribution-guidelines

Conversation

@bengbengbalabalabeng

@bengbengbalabalabeng bengbengbalabalabeng commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

Purpose of the pull request

Related: #1308

As title.

What's changed?

  • Add AI Policy.
  • Add PR Adoption Guidelines.
  • Sync to the Contributor Guide section on the fesod.apache.org.
  • Update behavioral rules in AGENTS.md

Checklist

  • I have read the Contributor Guide.
  • I have written the necessary doc or comment.
  • I have added the necessary unit tests and all cases have passed. (N/A, documentation change only)

@bengbengbalabalabeng
bengbengbalabalabeng marked this pull request as draft November 2, 2026 14:22
@nkuprins

nkuprins commented Nov 2, 2026

Copy link
Copy Markdown
Member

People who use AI blindly do not probably read CONTRIBUTING.md. But their agents do read AGENTS.md. Do you think it would make sense to add something like this to AGENTS.md?

## AI-assisted contributions

Follow [AI_POLICY.md](AI_POLICY.md).

@bengbengbalabalabeng

Copy link
Copy Markdown
Contributor Author

People who use AI blindly do not probably read CONTRIBUTING.md. But their agents do read AGENTS.md. Do you think it would make sense to add something like this to AGENTS.md?

## AI-assisted contributions

Follow [AI_POLICY.md](AI_POLICY.md).

Done.

@bengbengbalabalabeng
bengbengbalabalabeng marked this pull request as ready for review November 5, 2026 09:00
@nkuprins

nkuprins commented Nov 6, 2026 •

Copy link
Copy Markdown
Member

People who use AI blindly do not probably read CONTRIBUTING.md. But their agents do read AGENTS.md. Do you think it would make sense to add something like this to AGENTS.md?

## AI-assisted contributions

Follow [AI_POLICY.md](AI_POLICY.md).

Done.

I tried it with an AI agent from a fresh repo. The agent loaded AGENTS.md but never opened AI_POLICY.md. It went straight to the code. When I asked, it said the "AI-assisted contributions" section is only a link at the end of the file, and nothing tells the agent to open it before it starts.

So I suggest:

  1. Move the section to the top of AGENTS.md.
  2. Explicitly tell it to read it.
  3. Add some hard rules already in it.
  4. Tell the agent to say that it read the policy.

For example:

## AI Policy (read first)

Before any task, read [AI_POLICY.md](./AI_POLICY.md) and tell the user you read it.
Hard rules for agents:
- Do not open PRs, issues, or comments on your own. A human submits them.
- Do not write replies to reviewers. Only fix the language of text the human wrote.
- The human must read and understand the whole diff.
- List unknowns and assumptions so the human can put them in the PR description.

This example worked for me when I tried again (from a fresh start).
What do you think?

@bengbengbalabalabeng bengbengbalabalabeng added the PR: developing This feature will be added in future releases label Nov 8, 2026
@bengbengbalabalabeng

Copy link
Copy Markdown
Contributor Author

@nkuprins, thanks for catching that!

Once some progress is made in the discussion in #1308, I will continue working on the current PR.

…gents

- Refine boundaries and verification standards for AI-assisted contributions
- Shorten PR adoption timeline to a 14+7 model (14-day inactivity + 7-day grace period)
- Update strict behavioral constraints for coding agents in AGENTS.md

This branch has not been deployed

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

Labels

PR: developing This feature will be added in future releases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants