Skip to content

Add AGENTS.md with project structure and commands#4233

Merged
xrmx merged 7 commits intoopen-telemetry:mainfrom
Kludex:add-claude-md
Apr 14, 2026
Merged

Add AGENTS.md with project structure and commands#4233
xrmx merged 7 commits intoopen-telemetry:mainfrom
Kludex:add-claude-md

Conversation

@Kludex
Copy link
Copy Markdown
Member

@Kludex Kludex commented Feb 20, 2026

Based on discussion from last week's meeting.

It's minimal.


You can't prevent people from spamming your repo with thousands of AI based PRs, but you can guide their AI to do the right thing.

@Kludex Kludex requested a review from a team as a code owner February 20, 2026 10:32
Copy link
Copy Markdown
Member

@aabmass aabmass left a comment

Choose a reason for hiding this comment

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

Looks like a good starting point, thanks!

@aabmass aabmass added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Feb 20, 2026
@MikeGoldsmith MikeGoldsmith mentioned this pull request Feb 23, 2026
10 tasks
@tammy-baylis-swi tammy-baylis-swi moved this to Approved PRs in Python PR digest Feb 23, 2026
@tammy-baylis-swi tammy-baylis-swi moved this from Approved PRs to Ready for review in Python PR digest Feb 23, 2026
@tammy-baylis-swi tammy-baylis-swi moved this from Ready for review to Reviewed PRs that need fixes in Python PR digest Feb 26, 2026
Kludex added 3 commits April 10, 2026 10:39
- Move main content from CLAUDE.md to AGENTS.md so all AI agents
  (not only Claude) pick up the guidance; CLAUDE.md now just references
  it via `@AGENTS.md`.
- Add general rules, PR scoping, and `Assisted-by:` commit trailer
  guidance (inspired by the Collector's AGENTS.md).
- Clarify that only instrumentation packages live under
  `src/opentelemetry/instrumentation/{name}/`; other package types use
  their own namespace.
@xrmx xrmx requested a review from MikeGoldsmith April 10, 2026 16:05
@xrmx xrmx moved this from Reviewed PRs that need fixes to Approved PRs in Python PR digest Apr 10, 2026
Copy link
Copy Markdown
Member

@MikeGoldsmith MikeGoldsmith left a comment

Choose a reason for hiding this comment

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

Looks good - thanks @Kludex 👍🏻

@MikeGoldsmith MikeGoldsmith changed the title Add CLAUDE.md with project structure and commands Add AGENTS.md with project structure and commands Apr 12, 2026
@MikeGoldsmith
Copy link
Copy Markdown
Member

Hmm, on second thought - maybe we should remove the recommendation for the assisted-by commit trailer. Kubernetes has moved to disallow it because it's been used by some projects as a marketing gimmick.

Kat Cosgrove LinkedIn post - https://www.linkedin.com/posts/katcosgrove_pull-request-process-activity-7448418179258933248-iA2n
Kubernetes AI Guidance - https://www.kubernetes.dev/docs/guide/pull-requests/#ai-guidance

@MikeGoldsmith
Copy link
Copy Markdown
Member

After some more discussion, OpenTelemetry still advises using the assisted-by commit trailer in it's GenAI policy. Probably okay to leave in for now 👍🏻.

@xrmx xrmx merged commit baffc8c into open-telemetry:main Apr 14, 2026
750 checks passed
@github-project-automation github-project-automation bot moved this from Approved PRs to Done in Python PR digest Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants