Skip to content

ref(core): Use conversation ID constant from conventions - #24966

Merged
nicohrubec merged 1 commit into
developfrom
feat/conversation-id-conventions
Oct 2, 2026
Merged

nicohrubec merged 1 commit into
developfrom
feat/conversation-id-conventions

Conversation

@nicohrubec

Copy link
Copy Markdown
Member

Uses GEN_AI_CONVERSATION_ID from @sentry/conventions/attributes in the conversation-ID integration and its tests. Deprecates the existing core constant while preserving its public export and the identical gen_ai.conversation.id key.

Part of #24961 / JS-3868.

Co-Authored-By: GPT-6 <codex@openai.com>
@nicohrubec
nicohrubec marked this pull request as ready for review October 2, 2026 07:36
@nicohrubec
nicohrubec requested review from JPeer264 and Lms24 October 2, 2026 07:37
@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size % Change Change
@sentry/browser 29.52 kB +0.02% +5 B 🔺
@sentry/browser - with treeshaking flags 27.68 kB - -
@sentry/browser - with treeshaking flags tracing without tracing 27.57 kB -0.02% -4 B 🔽
@sentry/browser (incl. Tracing) 51.46 kB +0.01% +1 B 🔺
@sentry/browser (incl. Tracing + Span Streaming) 51.46 kB -0.01% -2 B 🔽
@sentry/browser (incl. Tracing, Profiling) 54.47 kB -0.01% -2 B 🔽
@sentry/browser (incl. Tracing, Replay) 91.04 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 80.03 kB +0.01% +2 B 🔺
@sentry/browser (incl. Tracing, Replay with Canvas) 95.74 kB +0.01% +3 B 🔺
@sentry/browser (incl. Tracing, Replay, Feedback) 108.71 kB -0.01% -2 B 🔽
@sentry/browser (incl. Feedback) 47.04 kB +0.01% +1 B 🔺
@sentry/browser (incl. sendFeedback) 34.57 kB +0.01% +3 B 🔺
@sentry/browser (incl. FeedbackAsync) 39.68 kB -0.01% -3 B 🔽
@sentry/browser (incl. Metrics) 30.54 kB +0.03% +7 B 🔺
@sentry/browser (incl. Logs) 30.82 kB +0.03% +7 B 🔺
@sentry/browser (incl. Metrics & Logs) 31.48 kB +0.02% +5 B 🔺
@sentry/react 31.36 kB - -
@sentry/react (incl. Tracing) 53.82 kB - -
@sentry/vue 37.51 kB - -
@sentry/vue (incl. Tracing) 54.35 kB -0.01% -3 B 🔽
@sentry/svelte 29.55 kB +0.03% +8 B 🔺
@sentry/remix (Remix 3 client bundle) 55.74 kB - -
CDN Bundle 31.22 kB - -
CDN Bundle (incl. Tracing) 51.98 kB - -
CDN Bundle (incl. Logs, Metrics) 33.46 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 53.94 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 74.2 kB - -
CDN Bundle (incl. Tracing, Replay) 89.56 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 91.54 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 95.73 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 97.74 kB - -
CDN Bundle - uncompressed 92.14 kB - -
CDN Bundle (incl. Tracing) - uncompressed 154.53 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 98.71 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 160.48 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 228.28 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 274.26 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 280.2 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 287.96 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 293.89 kB - -
@sentry/nextjs (client) 56.33 kB - -
@sentry/sveltekit (client) 51.88 kB -0.01% -3 B 🔽
@sentry/core/server 40.59 kB - -
@sentry/core/browser 13.63 kB - -
@sentry/node 144.66 kB +0.01% +2 B 🔺
@sentry/node/import (ESM hook with diagnostics-channel injection) 83.17 kB - -
@sentry/node - without tracing 93.22 kB +0.02% +10 B 🔺
@sentry/node - without channel injection 122.96 kB +0.01% +7 B 🔺
@sentry/aws-serverless 101.49 kB +0.01% +3 B 🔺
@sentry/cloudflare (withSentry) - minified 208.62 kB - -
@sentry/cloudflare (withSentry) 518.29 kB -0.02% -60 B 🔽

View base workflow run

@nicohrubec
nicohrubec enabled auto-merge (squash) October 2, 2026 07:47
@nicohrubec
nicohrubec merged commit bc5889c into develop Oct 2, 2026
353 of 354 checks passed
@nicohrubec
nicohrubec deleted the feat/conversation-id-conventions branch October 2, 2026 07:49
Sign up for free to 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.

3 participants