Add AGENTS.md with project structure and commands#4233
Merged
Conversation
aabmass
approved these changes
Feb 20, 2026
aabmass
left a comment
aabmass
left a comment
Member
There was a problem hiding this comment.
Looks like a good starting point, thanks!
MikeGoldsmith
requested changes
Feb 23, 2026
10 tasks
xrmx
reviewed
Feb 24, 2026
tammy-baylis-swi
moved this from Ready for review
to Reviewed PRs that need fixes
in Python PR digest
- 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
approved these changes
Apr 10, 2026
MikeGoldsmith
approved these changes
Apr 12, 2026
MikeGoldsmith
left a comment
MikeGoldsmith
left a comment
Member
There was a problem hiding this comment.
Looks good - thanks @Kludex 👍🏻
MikeGoldsmith
changed the title
Member
|
Hmm, on second thought - maybe we should remove the recommendation for the Kat Cosgrove LinkedIn post - https://www.linkedin.com/posts/katcosgrove_pull-request-process-activity-7448418179258933248-iA2n |
Member
|
After some more discussion, OpenTelemetry still advises using the |
Merged
9 tasks
shuningc
pushed a commit
to shuningc/opentelemetry-python-contrib
that referenced
this pull request
Apr 16, 2026
* Add CLAUDE.md with project structure and commands
* Add AGENTS.md symlink to CLAUDE.md
* Move guidance to AGENTS.md and address review feedback
- 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.
---------
Co-authored-by: Riccardo Magliocchetti <[email protected]>
aabmass
pushed a commit
that referenced
this pull request
Apr 16, 2026
* linting fix * Updating changelog * Updating changelog PR number * Removing embedding events emission * Refatoring and adding input token metric for Embedding invocation * Merging and removing unused import * build(deps): bump aiohttp from 3.13.3 to 3.13.4 (#4386) --- updated-dependencies: - dependency-name: aiohttp dependency-version: 3.13.4 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Revert "Build list of required jobs in generate-workflow (#4326)" (#4413) This reverts commit 22879d6. Now that we have just one job to check we don't need to build the list anymore. * Drop Python 3.9 support (#4412) * Drop Python 3.9 support Signed-off-by: emdneto <[email protected]> * generate-workflows Signed-off-by: emdneto <[email protected]> * fixes Signed-off-by: emdneto <[email protected]> * remove extra reference to pypy310 Signed-off-by: emdneto <[email protected]> * changelog Signed-off-by: emdneto <[email protected]> * fix flask tests Signed-off-by: emdneto <[email protected]> * fix google-genai tests Signed-off-by: emdneto <[email protected]> * fix google-genai tests Signed-off-by: emdneto <[email protected]> * fix Signed-off-by: emdneto <[email protected]> * remove unused _ensure_gzip_single_response Signed-off-by: emdneto <[email protected]> --------- Signed-off-by: emdneto <[email protected]> Co-authored-by: Riccardo Magliocchetti <[email protected]> * Add AGENTS.md with project structure and commands (#4233) * Add CLAUDE.md with project structure and commands * Add AGENTS.md symlink to CLAUDE.md * Move guidance to AGENTS.md and address review feedback - 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. --------- Co-authored-by: Riccardo Magliocchetti <[email protected]> * scripts: drop update_sha (#4430) It's buggy and unused. * Fix pylint false positives for ThreadPoolExecutor (#4244) * Bump pylint to 4.0.5 to fix Python 3.14 concurrent.futures false positives * Fix too-many-positional-arguments pylint failures - Add max-positional-arguments=10, Add pylint: disable=too-many-positional-arguments to functions that legitimately exceed the limit * Bump max-positional-arguments to 12 and remove unnecessary disable comments * Address review comments: fix CHANGELOG, remove stale pylintrc comment, add openai-agents disable * Fix formatting: restore blank line in CHANGELOG and remove extra blank line in .pylintrc * Update processor.py * Update test_botocore_bedrock.py --------- Co-authored-by: Riccardo Magliocchetti <[email protected]> * feat(util-genai): refactor and make API smaller and more user-friendly (#4391) * Refactor public API on GenAI utils * more lint * review feedback * update tests to use named params * address some of the comments * up * fix failing checks and clean up imports * lint * lint * fix lint * replace @deprecated with docstring info to avoid warnings for users * up * common code for context manager * Adding metrics call for Embedding type after refactoring * Updating metrics tests with embedding * Adding fix for markdown-link-check --------- Signed-off-by: dependabot[bot] <[email protected]> Signed-off-by: emdneto <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Riccardo Magliocchetti <[email protected]> Co-authored-by: Emídio Neto <[email protected]> Co-authored-by: Marcelo Trylesinski <[email protected]> Co-authored-by: Sri Kaaviya <[email protected]> Co-authored-by: Liudmila Molkova <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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.