Skip to content

ref(core)!: Align gen_ai cache token attributes with conventions - #22811

Merged
nicohrubec merged 3 commits into
ref/remove-unused-genai-op-constantsfrom
ref/align-cache-token-attributes-with-conventions
Jul 29, 2026
Merged

ref(core)!: Align gen_ai cache token attributes with conventions#22811
nicohrubec merged 3 commits into
ref/remove-unused-genai-op-constantsfrom
ref/align-cache-token-attributes-with-conventions

Conversation

@nicohrubec

@nicohrubecnicohrubec commented Jul 29, 2026

Copy link
Copy Markdown
Member

Switches the two cache-token attributes from the local underscore keys (gen_ai.usage.cache_creation_input_tokens / cache_read_input_tokens) to the dotted @sentry/conventions exports (gen_ai.usage.cache_creation.input_tokens / cache_read.input_tokens), removing the corresponding local constants.

nicohrubecand others added 3 commits July 29, 2026 12:36
Emit the dotted `gen_ai.usage.cache_creation.input_tokens` and
`gen_ai.usage.cache_read.input_tokens` keys from `@sentry/conventions`
instead of the local underscore variants, removing the two local
constants in favor of the conventions exports.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nicohrubec
nicohrubec marked this pull request as ready for review July 29, 2026 10:51
@github-actions

Copy link
Copy Markdown
Contributor

size-limit report 📦

PathSize% ChangeChange
@sentry/browser29.83 kB--
@sentry/browser - with treeshaking flags28.04 kB--
@sentry/browser (incl. Tracing)47.11 kB--
@sentry/browser (incl. Tracing + Span Streaming)47.12 kB--
@sentry/browser (incl. Tracing, Profiling)51.82 kB--
@sentry/browser (incl. Tracing, Replay)86.42 kB--
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags75.85 kB--
@sentry/browser (incl. Tracing, Replay with Canvas)91.14 kB--
@sentry/browser (incl. Tracing, Replay, Feedback)103.78 kB--
@sentry/browser (incl. Feedback)47.14 kB--
@sentry/browser (incl. sendFeedback)34.67 kB--
@sentry/browser (incl. FeedbackAsync)39.77 kB--
@sentry/browser (incl. Metrics)30.9 kB--
@sentry/browser (incl. Logs)31.13 kB--
@sentry/browser (incl. Metrics & Logs)31.81 kB--
@sentry/react31.62 kB--
@sentry/react (incl. Tracing)49.34 kB--
@sentry/vue34.75 kB--
@sentry/vue (incl. Tracing)49.07 kB--
@sentry/svelte29.86 kB--
CDN Bundle31.88 kB--
CDN Bundle (incl. Tracing)47.45 kB--
CDN Bundle (incl. Logs, Metrics)33.43 kB--
CDN Bundle (incl. Tracing, Logs, Metrics)48.83 kB--
CDN Bundle (incl. Replay, Logs, Metrics)72.79 kB--
CDN Bundle (incl. Tracing, Replay)85.1 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)86.39 kB--
CDN Bundle (incl. Tracing, Replay, Feedback)90.86 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)92.17 kB--
CDN Bundle - uncompressed95.08 kB--
CDN Bundle (incl. Tracing) - uncompressed142.29 kB--
CDN Bundle (incl. Logs, Metrics) - uncompressed99.79 kB--
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed146.27 kB--
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed224.55 kB--
CDN Bundle (incl. Tracing, Replay) - uncompressed261.55 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed265.51 kB--
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed275.25 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed279.21 kB--
@sentry/nextjs (client)51.94 kB--
@sentry/sveltekit (client)47.53 kB--
@sentry/core/server79.61 kB+0.03%+20 B 🔺
@sentry/core/browser51.6 kB--
@sentry/node121.7 kB+0.01%+4 B 🔺
@sentry/node/import (ESM hook with diagnostics-channel injection)166 B--
@sentry/node - without tracing85.28 kB+0.01%+1 B 🔺
@sentry/aws-serverless93.5 kB--
@sentry/cloudflare (withSentry) - minified197.59 kB--
@sentry/cloudflare (withSentry)485.75 kB--

View base workflow run

@nicohrubec
nicohrubec merged commit 943505a into developJul 29, 2026
523 of 529 checks passed
@nicohrubec
nicohrubec deleted the ref/align-cache-token-attributes-with-conventions branch July 29, 2026 13:54
JPeer264 added a commit to getsentry/sentry-conventions that referenced this pull request Aug 27, 2026
Adds `gen_ai.usage.cache_creation_input_tokens` and
`gen_ai.usage.cache_read_input_tokens`, which the JavaScript SDK stopped
emitting in v11 in favour of the dotted `gen_ai.usage.cache_*.input_tokens`
names. Both join the existing alias groups that already hold the Vercel AI
spellings.
Deprecated by getsentry/sentry-javascript#22811
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
JPeer264 added a commit to getsentry/sentry-conventions that referenced this pull request Aug 27, 2026
Adds `gen_ai.usage.cache_creation_input_tokens` and
`gen_ai.usage.cache_read_input_tokens`, which the JavaScript SDK stopped
emitting in v11 in favour of the dotted `gen_ai.usage.cache_*.input_tokens`
names. Both join the existing alias groups that already hold the Vercel AI
spellings.
Deprecated by getsentry/sentry-javascript#22811
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
JPeer264 added a commit to getsentry/sentry-conventions that referenced this pull request Aug 27, 2026
Adds `gen_ai.usage.cache_creation_input_tokens` and
`gen_ai.usage.cache_read_input_tokens`, which the JavaScript SDK stopped
emitting in v11 in favour of the dotted `gen_ai.usage.cache_*.input_tokens`
names. Both join the existing alias groups that already hold the Vercel AI
spellings.
Deprecated by getsentry/sentry-javascript#22811
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Sign up for freeto 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.

2 participants

@nicohrubec@andreiborza