Skip to content

fix(bun): Add conversationIdIntegration to the default integrations - #25003

Merged
JPeer264 merged 1 commit into
jp/flue-map-messages-conventionsfrom
jp/bun-conversation-id-integration
Oct 6, 2026
Merged

JPeer264 merged 1 commit into
jp/flue-map-messages-conventionsfrom
jp/bun-conversation-id-integration

Conversation

@JPeer264

@JPeer264 JPeer264 commented Oct 2, 2026 •

Copy link
Copy Markdown
Member

@sentry/bun builds its own list of default integrations, and that list has no conversationIdIntegration. The other runtime SDKs install it (#24299 added it to Deno), and @sentry/elysia takes Bun's list. So on Bun and Elysia, a conversation id set with Sentry.setConversationId(), or by an AI integration on the scope, never reached the gen_ai spans.

This was the only reason the bundled Bun runs of the OpenAI and Vercel AI (v4) suites failed, so they run again.

Ref #23887

🤖 Generated with Claude Code

@JPeer264
JPeer264 added this pull request to stack #25004 October 2, 2026 13:57
@JPeer264
JPeer264 force-pushed the jp/bun-conversation-id-integration branch from 26a12b2 to 75d4397 Compare October 2, 2026 16:59
@github-actions

github-actions Bot commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size % Change Change
@sentry/browser 29.6 kB - -
@sentry/browser - with treeshaking flags 27.75 kB - -
@sentry/browser - with treeshaking flags tracing without tracing 27.65 kB - -
@sentry/browser (incl. Tracing) 51.52 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 51.52 kB - -
@sentry/browser (incl. Tracing, Profiling) 54.5 kB - -
@sentry/browser (incl. Tracing, Replay) 91.14 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 80.12 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 95.83 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 108.83 kB - -
@sentry/browser (incl. Feedback) 47.12 kB - -
@sentry/browser (incl. sendFeedback) 34.65 kB - -
@sentry/browser (incl. FeedbackAsync) 39.76 kB - -
@sentry/browser (incl. Metrics) 30.61 kB - -
@sentry/browser (incl. Logs) 30.89 kB - -
@sentry/browser (incl. Metrics & Logs) 31.55 kB - -
@sentry/react 31.43 kB - -
@sentry/react (incl. Tracing) 53.84 kB - -
@sentry/vue 37.55 kB - -
@sentry/vue (incl. Tracing) 54.4 kB - -
@sentry/svelte 29.63 kB - -
@sentry/remix (Remix 3 client bundle) 55.85 kB - -
CDN Bundle 31.33 kB - -
CDN Bundle (incl. Tracing) 52.07 kB - -
CDN Bundle (incl. Logs, Metrics) 33.56 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 54.03 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 74.3 kB - -
CDN Bundle (incl. Tracing, Replay) 89.67 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 91.62 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 95.83 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 97.8 kB - -
CDN Bundle - uncompressed 92.46 kB - -
CDN Bundle (incl. Tracing) - uncompressed 154.77 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 99.04 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 160.72 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 228.66 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 274.58 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 280.52 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 288.28 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 294.21 kB - -
@sentry/nextjs (client) 56.19 kB - -
@sentry/sveltekit (client) 51.9 kB - -
@sentry/core/server 40.52 kB - -
@sentry/core/browser 13.51 kB - -
@sentry/node 145.02 kB +0.01% +4 B 🔺
@sentry/node/import (ESM hook with diagnostics-channel injection) 83.22 kB - -
@sentry/node - without tracing 93.46 kB +0.01% +3 B 🔺
@sentry/node - without channel injection 123.17 kB +0.01% +8 B 🔺
@sentry/aws-serverless 101.69 kB +0.01% +4 B 🔺
@sentry/cloudflare (withSentry) - minified 209.04 kB - -
@sentry/cloudflare (withSentry) 517.75 kB - -

View base workflow run

@JPeer264
JPeer264 force-pushed the jp/bun-conversation-id-integration branch from 75d4397 to 8994101 Compare October 3, 2026 16:23
@JPeer264
JPeer264 removed this pull request from stack #25004 October 5, 2026 07:21
@JPeer264
JPeer264 added this pull request to stack #25029 October 5, 2026 07:22
@JPeer264 JPeer264 self-assigned this Oct 5, 2026
@JPeer264
JPeer264 marked this pull request as ready for review October 5, 2026 08:49
@JPeer264
JPeer264 requested a review from a team as a code owner October 5, 2026 08:49
@JPeer264
JPeer264 requested review from andreiborza, isaacs and mydea and removed request for a team and mydea October 5, 2026 08:49
@JPeer264
JPeer264 force-pushed the jp/bun-conversation-id-integration branch from 8994101 to 84bccb6 Compare October 5, 2026 09:44

@andreiborza andreiborza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice

@JPeer264
JPeer264 force-pushed the jp/bun-conversation-id-integration branch from 84bccb6 to 79eb85a Compare October 5, 2026 11:22
`@sentry/bun` builds its own list of default integrations, and it did
not include `conversationIdIntegration`, which every other server SDK
installs. So on Bun, a conversation id set on the scope never reached
the gen_ai spans.

This was the only reason the bundled Bun runs of the OpenAI and Vercel
AI (v4) suites failed, so both run again.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@JPeer264
JPeer264 force-pushed the jp/bun-conversation-id-integration branch from 79eb85a to c285d1e Compare October 6, 2026 07:08
@JPeer264
JPeer264 merged commit cfa5c75 into develop Oct 6, 2026
81 of 110 checks passed
@JPeer264
JPeer264 deleted the jp/bun-conversation-id-integration branch October 6, 2026 07:55
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