Skip to content

docs: add ai contributing policy#3437

Open
chaokunyang wants to merge 2 commits intoapache:mainfrom
chaokunyang:ai_contributing_policy
Open

docs: add ai contributing policy#3437
chaokunyang wants to merge 2 commits intoapache:mainfrom
chaokunyang:ai_contributing_policy

Conversation

@chaokunyang
Copy link
Collaborator

@chaokunyang chaokunyang commented Feb 28, 2026

Why?

Apache Fory is aperformance-critical foundational serialization framework.
As AI-assisted contributions increase, we need a project policy that keeps review quality high, protects legal/provenance safety, and preserves long-term maintainability.

This policy is intended to:

  • keep human accountability as the core rule
  • reduce low-signal or unverified AI-generated changes
  • require verifiable testing/performance evidence for sensitive paths
  • align contribution practice with ASF legal and governance expectations

What does this PR do?

  • Adds a new top-level policy document: AI_CONTRIBUTION_POLICY.md.
  • Defines requirements for AI-assisted contributions, including:
    • contributor responsibility and line-by-line self review
    • privacy-safe disclosure expectations
    • verification requirements (tests/spec/perf evidence where applicable)
    • licensing/provenance requirements
    • quality gate and maintainer enforcement process
  • Updates .github/pull_request_template.md to add an AI contribution checklist for PR authors.

Related issues

N/A

Does this PR introduce any user-facing change?

  • Does this PR introduce any public API change?
  • Does this PR introduce any binary protocol compatibility change?

Benchmark

N/A (documentation and PR template change only)

@chaokunyang chaokunyang force-pushed the ai_contributing_policy branch from 94e6b82 to 646b992 Compare February 28, 2026 13:58
@chaokunyang chaokunyang force-pushed the ai_contributing_policy branch from 646b992 to 8b8c41a Compare February 28, 2026 14:02
Copy link
Contributor

@pandalee99 pandalee99 left a comment

Choose a reason for hiding this comment

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

LGTM

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants