FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

feat(ai): add cache writes for gen_ai by shellmayr · Pull Request #5319 · getsentry/sentry-python · GitHub

/ sentry-python Public

feat(ai): add cache writes for gen_ai - #5319

Merged
shellmayr merged 7 commits into
masterfrom
shellmayr/feat/add-cache-write-for-genai
Jan 19, 2026
ConversationCommits (7)ChecksFiles changed
Merged

feat(ai): add cache writes for gen_ai#5319
shellmayr merged 7 commits into
masterfrom
shellmayr/feat/add-cache-write-for-genai

Conversation

shellmayr commented Jan 15, 2026 •
edited
Loading

Copy link
Copy Markdown
Member
  • Implement cache token tracking for AI integrations
  • Adds support for tracking cache read and cache write tokens across AI integrations, following new semantic convention that added gen_ai.usage.input_tokens.cache_write
  • Updated integrations to track cache tokens:

Closes TET-1732
Co-Authored-By: Claude Sonnet 4.5 noreply@anthropic.com

linear Bot commented Jan 15, 2026

Copy link
Copy Markdown

TET-1732 Record cache read & cache write tokens in Python SDK

github-actions Bot commented Jan 15, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Semver Impact of This PR

🟡 Minor (new features)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

New Features ✨

Ai

  • feat(ai): add cache writes for gen_ai by shellmayr in #5319
  • feat(ai): add parse_data_uri function to parse a data URI by constantinius in #5311

Other

  • feat(asyncio): Add on-demand way to enable AsyncioIntegration by sentrivana in #5288
  • feat(openai-agents): Inject propagation headers for HostedMCPTool by alexander-alderman-webb in #5297
  • feat: Support array types for logs and metrics attributes by alexander-alderman-webb in #5314

Bug Fixes 🐛

Integrations

  • fix(integrations): pydantic-ai: properly format binary input message parts to be conformant with the gen_ai.request.messages structure by constantinius in #5251
  • fix(integrations): Anthropic: add content transformation for images and documents by constantinius in #5276
  • fix(integrations): langchain add multimodal content transformation functions for images, audio, and files by constantinius in #5278

Litellm

  • fix(litellm): fix gen_ai.request.messages to be as expected by constantinius in #5255
  • fix(litellm): Guard against module shadowing by alexander-alderman-webb in #5249

Other

  • fix(ai): redact message parts content of type blob by constantinius in #5243
  • fix(clickhouse): Guard against module shadowing by alexander-alderman-webb in #5250
  • fix(gql): Revert signature change of patched gql.Client.execute by alexander-alderman-webb in #5289
  • fix(grpc): Derive interception state from channel fields by alexander-alderman-webb in #5302
  • fix(pure-eval): Guard against module shadowing by alexander-alderman-webb in #5252
  • fix(ray): Guard against module shadowing by alexander-alderman-webb in #5254
  • fix(threading): Handle channels shadowing by sentrivana in #5299
  • fix(typer): Guard against module shadowing by alexander-alderman-webb in #5253
  • fix: Stop suppressing exception chains in AI integrations by alexander-alderman-webb in #5309
  • fix: Send client reports for span recorder overflow by sentrivana in #5310

Documentation 📚

  • docs(metrics): Remove experimental notice by alexander-alderman-webb in #5304
  • docs: Update Python versions banner in README by sentrivana in #5287

Internal Changes 🔧

Release

  • ci(release): Bump Craft version to fix issues by BYK in #5305
  • ci(release): Switch from action-prepare-release to Craft by BYK in #5290

Other

  • chore(gen_ai): add auto-enablement for google genai by shellmayr in #5295
  • ci: 🤖 Update test matrix with new releases (01/19) by github-actions in #5330
  • ci: Add periodic AI integration tests by alexander-alderman-webb in #5313
  • chore: Use pull_request_target for changelog preview by BYK in #5323
  • chore: add unlabeled trigger to changelog-preview by BYK in #5315
  • chore: Add type for metric units by sentrivana in #5312
  • ci: Update tox and handle generic classifiers by sentrivana in #5306

🤖 This preview updates automatically when you update the PR.

shellmayr force-pushed the shellmayr/feat/add-cache-write-for-genai branch from bf3310b to 37782db Compare January 16, 2026 12:10
shellmayr marked this pull request as ready for review January 16, 2026 14:02
shellmayr requested a review from a team as a code owner January 16, 2026 14:02
alexander-alderman-webb approved these changes Jan 19, 2026
View reviewed changes
Comment thread sentry_sdk/integrations/pydantic_ai/spans/utils.py Outdated
Comment on lines +47 to +50
if (
hasattr(usage, "output_tokens_reasoning")
and usage.output_tokens_reasoning is not None
):

alexander-alderman-webb Jan 19, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Is adding the reasoning tokens to pydantic-ai intentional?

shellmayr Jan 19, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Ah no, forgot to remove that - thanks for catching that!

Comment thread tests/integrations/pydantic_ai/test_pydantic_ai.py Outdated
Comment on lines +2738 to +2741
import sentry_sdk
from pydantic_ai.usage import RequestUsage
from sentry_sdk.integrations.pydantic_ai.spans.utils import _set_usage_data
from sentry_sdk.consts import SPANDATA

alexander-alderman-webb Jan 19, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

please move these to the top of the file before merging 🙏

shellmayr Jan 19, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

excuse the slop 🙏

shellmayr merged commit ab5cdde into master Jan 19, 2026
154 of 155 checks passed
shellmayr deleted the shellmayr/feat/add-cache-write-for-genai branch January 19, 2026 12:01
sentry-release-bot Bot mentioned this pull request Jan 20, 2026
Closed
6 tasks
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
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

alexander-alderman-webb alexander-alderman-webb approved these changes

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

Back | FazBrowse Home | New Git URL