Skip to content

feat(anthropic) Update span attributes to use gen_ai.* namespace instead of ai.*#4674

Merged
antonpirker merged 23 commits into
masterfrom
constantinius/feat/anthropic/ai-to-gen-ai-span-attributes
Aug 8, 2025
Merged

feat(anthropic) Update span attributes to use gen_ai.* namespace instead of ai.*#4674
antonpirker merged 23 commits into
masterfrom
constantinius/feat/anthropic/ai-to-gen-ai-span-attributes

Conversation

@constantinius
Copy link
Copy Markdown
Contributor

@constantinius constantinius commented Aug 5, 2025 •
edited by antonpirker
Loading

Update AnthropicIntegration to support Otel and Sentry AI Agents module compatible span attributes of gen_ai.* family.

Closes https://linear.app/getsentry/issue/TET-996/improve-integration-for-anthropic-sdk

@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 5, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 91.37931% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.10%. Comparing base (3ef02a1) to head (6236f9d).
⚠️ Report is 1 commits behind head on master.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
sentry_sdk/integrations/anthropic.py 90.00% 3 Missing and 2 partials ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##           master    #4674      +/-   ##
==========================================
+ Coverage   85.09%   85.10%   +0.01%     
==========================================
  Files         156      156              
  Lines       15842    15856      +14     
  Branches     2679     2684       +5     
==========================================
+ Hits        13480    13494      +14     
- Misses       1582     1583       +1     
+ Partials      780      779       -1
Files with missing lines Coverage Δ
sentry_sdk/ai/utils.py 72.72% <100.00%> (ø)
sentry_sdk/integrations/starlite.py 81.94% <100.00%> (ø)
sentry_sdk/integrations/anthropic.py 86.59% <90.00%> (-1.29%) ⬇️

... and 2 files with indirect coverage changes

@constantinius constantinius marked this pull request as ready for review August 6, 2025 18:33
@constantinius constantinius requested a review from a team as a code owner August 6, 2025 18:33
@linear
Copy link
Copy Markdown

linear Bot commented Aug 6, 2025

TET-996 Improve integration for Anthropic SDK

cursor[bot]

This comment was marked as outdated.

Sign in to view

cursor[bot]

This comment was marked as outdated.

Sign in to view

cursor[bot]

This comment was marked as outdated.

Sign in to view

cursor[bot]

This comment was marked as outdated.

Sign in to view

@constantinius constantinius mentioned this pull request Aug 7, 2025
Merged
3 tasks
cursor[bot]

This comment was marked as outdated.

Sign in to view

@antonpirker antonpirker changed the title feat(anthropic) update span attributes to use gen_ai.* namespace instead of ai.* feat(anthropic) Update span attributes to use gen_ai.* namespace instead of ai.* Aug 8, 2025
antonpirker
antonpirker approved these changes Aug 8, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@antonpirker antonpirker left a comment

Choose a reason for hiding this comment

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

Now it is ready to merge. Thanks @constantinius for the work!

@antonpirker antonpirker merged commit 378fe81 into master Aug 8, 2025
138 checks passed
@antonpirker antonpirker deleted the constantinius/feat/anthropic/ai-to-gen-ai-span-attributes branch August 8, 2025 11:03
sentrivana pushed a commit that referenced this pull request Aug 12, 2025
@constantinius @antonpirker @sentrivana
feat(anthropic) Update span attributes to use gen_ai.* namespace in…
66f47ec 
…stead of `ai.*` (#4674)

Update `AnthropicIntegration` to support Otel and Sentry AI Agents
module compatible span attributes of `gen_ai.*` family.

Closes
https://linear.app/getsentry/issue/TET-996/improve-integration-for-anthropic-sdk

---------

Co-authored-by: Anton Pirker <anton.pirker@sentry.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cursor cursor[bot] cursor[bot] left review comments

+1 more reviewer

@antonpirker antonpirker antonpirker approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@constantinius @antonpirker