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

test(bun): Run the auto-instrumentation suites with bundled scenarios by JPeer264 · Pull Request #24612 · getsentry/sentry-javascript · GitHub

  1. getsentry
  2. sentry-javascript

Repository navigation

test(bun): Run the auto-instrumentation suites with bundled scenarios - #24612

Merged
JPeer264 merged 1 commit into
developfrom
jp/node-suites-bun-build
Sep 29, 2026
ConversationCommits (1)ChecksFiles changed
Merged

test(bun): Run the auto-instrumentation suites with bundled scenarios#24612
JPeer264 merged 1 commit into
developfrom
jp/node-suites-bun-build

Conversation

JPeer264 commented Sep 23, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Under bun run the SDK cannot inject diagnostics channels into libraries (#23882), so a Bun app only gets framework, database and AI spans when it is built with @sentry/bun/plugin. That path had no integration coverage. The new node-suites-bun-build project bundles each auto-instrumentation scenario with the plugin right before it starts and runs it with @sentry/bun. Express, Fastify, Hapi, Koa, Apollo, most AI suites and the Docker database suites now run on Bun.

pg-native and the CommonJS postgresjs tests are skipped on Bun: the libpq addon needs a Node symbol that Bun does not have, and through the bun export condition require('postgres') returns the ES module namespace.

Closes #23889.

🤖 Generated with Claude Code

linear-code Bot commented Sep 23, 2026

Copy link
Copy Markdown

JS-3515

github-actions Bot commented Sep 23, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size % Change Change
@sentry/browser 29.24 kB - -
@sentry/browser - with treeshaking flags 27.5 kB - -
@sentry/browser - with treeshaking flags tracing without tracing 27.4 kB - -
@sentry/browser (incl. Tracing) 51.15 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 51.17 kB - -
@sentry/browser (incl. Tracing, Profiling) 54.18 kB - -
@sentry/browser (incl. Tracing, Replay) 90.76 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 79.86 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 95.46 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 108.41 kB - -
@sentry/browser (incl. Feedback) 46.76 kB - -
@sentry/browser (incl. sendFeedback) 34.3 kB - -
@sentry/browser (incl. FeedbackAsync) 39.41 kB - -
@sentry/browser (incl. Metrics) 30.25 kB - -
@sentry/browser (incl. Logs) 30.51 kB - -
@sentry/browser (incl. Metrics & Logs) 31.18 kB - -
@sentry/react 31.08 kB - -
@sentry/react (incl. Tracing) 53.54 kB - -
@sentry/vue 36.74 kB - -
@sentry/vue (incl. Tracing) 53.7 kB - -
@sentry/svelte 29.26 kB - -
CDN Bundle 31.02 kB - -
CDN Bundle (incl. Tracing) 51.77 kB - -
CDN Bundle (incl. Logs, Metrics) 33.29 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 53.75 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 74 kB - -
CDN Bundle (incl. Tracing, Replay) 89.36 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 91.33 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 95.53 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 97.5 kB - -
CDN Bundle - uncompressed 91.66 kB - -
CDN Bundle (incl. Tracing) - uncompressed 154.03 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 98.23 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 159.99 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 227.8 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 273.76 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 279.7 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 287.46 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 293.39 kB - -
@sentry/nextjs (client) 55.78 kB - -
@sentry/sveltekit (client) 51.59 kB - -
@sentry/core/server 39.99 kB - -
@sentry/core/browser 13.63 kB - -
@sentry/node 141.89 kB +0.02% +15 B 🔺
@sentry/node/import (ESM hook with diagnostics-channel injection) 82.88 kB - -
@sentry/node - without tracing 90.9 kB +0.02% +13 B 🔺
@sentry/node - without channel injection 120.26 kB +0.01% +9 B 🔺
@sentry/aws-serverless 99.14 kB +0.01% +4 B 🔺
@sentry/cloudflare (withSentry) - minified 206.62 kB - -
@sentry/cloudflare (withSentry) 514.02 kB - -

View base workflow run

JPeer264 added this pull request to stack #24613 September 23, 2026 06:14
JPeer264 force-pushed the jp/node-suites-bun-build branch 2 times, most recently from 99acd71 to 5eb22a2 Compare September 23, 2026 08:11
JPeer264 force-pushed the jp/node-suites-bun-build branch 2 times, most recently from 48d16a8 to 51be7dc Compare September 23, 2026 09:49
JPeer264 removed this pull request from stack #24613 September 23, 2026 09:51
JPeer264 changed the base branch from jp/node-suites-deno to jp/node-suites-bun-sdk-fixes September 23, 2026 09:51
JPeer264 added this pull request to stack #24633 September 23, 2026 09:51
JPeer264 force-pushed the jp/node-suites-bun-build branch 2 times, most recently from 9fc4460 to 131b223 Compare September 23, 2026 10:02
JPeer264 force-pushed the jp/node-suites-bun-sdk-fixes branch from 33f7607 to 77ca363 Compare September 23, 2026 10:04
JPeer264 force-pushed the jp/node-suites-bun-build branch from 131b223 to 6c3bef1 Compare September 23, 2026 10:04
JPeer264 force-pushed the jp/node-suites-bun-sdk-fixes branch from 77ca363 to 72e8bec Compare September 23, 2026 12:21
JPeer264 force-pushed the jp/node-suites-bun-build branch 2 times, most recently from 686046a to b0517c0 Compare September 23, 2026 12:24
JPeer264 force-pushed the jp/node-suites-bun-sdk-fixes branch from 373d5ab to 49333f3 Compare September 23, 2026 13:23
JPeer264 force-pushed the jp/node-suites-bun-build branch 2 times, most recently from f58de0f to b8c9dc6 Compare September 23, 2026 13:31
JPeer264 force-pushed the jp/node-suites-bun-sdk-fixes branch from 49333f3 to 7452887 Compare September 23, 2026 13:31
JPeer264 force-pushed the jp/node-suites-bun-build branch from b8c9dc6 to f25eee7 Compare September 23, 2026 16:19
JPeer264 force-pushed the jp/node-suites-bun-sdk-fixes branch from 7452887 to 27c4421 Compare September 23, 2026 16:19
JPeer264 removed this pull request from stack #24633 September 23, 2026 16:19
JPeer264 added this pull request to stack #24670 September 23, 2026 16:19
JPeer264 mentioned this pull request Sep 23, 2026
Merged
JPeer264 force-pushed the jp/node-suites-bun-build branch from f25eee7 to 4739836 Compare September 23, 2026 16:40
JPeer264 force-pushed the jp/node-suites-bun-sdk-fixes branch from 27c4421 to ff5d855 Compare September 23, 2026 16:40
JPeer264 force-pushed the jp/node-suites-bun-build branch from 1f2cec2 to 203cf3a Compare September 28, 2026 09:19
JPeer264 force-pushed the jp/node-suites-bun-sdk-fixes branch from 78e269f to 984e443 Compare September 28, 2026 09:38
JPeer264 force-pushed the jp/node-suites-bun-build branch from 203cf3a to 3621721 Compare September 28, 2026 09:38
Base automatically changed from jp/node-suites-bun-sdk-fixes to develop September 28, 2026 10:00
JPeer264 force-pushed the jp/node-suites-bun-build branch from 3621721 to 7cf9531 Compare September 28, 2026 10:00
JPeer264 marked this pull request as ready for review September 28, 2026 10:24
JPeer264 requested a review from a team as a code owner September 28, 2026 10:25
JPeer264 requested review from andreiborza and isaacs and removed request for a team September 28, 2026 10:25
JPeer264 force-pushed the jp/node-suites-bun-build branch from 7cf9531 to ea954f6 Compare September 28, 2026 13:43
test(bun): Run the auto-instrumentation suites with bundled scenarios
ff45d6d 
Under `bun run` the SDK cannot inject the diagnostics channels into
libraries (JS-3508), so Bun apps must be built with `@sentry/bun/plugin`
to get framework, database and AI spans. The new `node-suites-bun-build`
project tests that path: the runner builds each scenario with the plugin
right before it starts (`RUNTIME_BUILD_SCRIPT`), and runs the bundle
with `@sentry/bun`.

The Docker suites run as well, so the Bun CI job gets the disk space
step of the Node and Deno jobs. `pg-native` and the CommonJS
`postgresjs` tests are skipped on Bun: the `libpq` addon needs a Node
symbol that Bun does not have, and through the `bun` export condition
`require('postgres')` returns the ES module namespace.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
JPeer264 force-pushed the jp/node-suites-bun-build branch from ea954f6 to ff45d6d Compare September 29, 2026 06:25
JPeer264 mentioned this pull request Sep 29, 2026
Open
8 of 10 tasks
This was referenced Sep 29, 2026
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
sentry-release-bot Bot mentioned this pull request Oct 1, 2026
Closed
47 tasks
github-actions Bot mentioned this pull request Oct 1, 2026
Closed
github-actions Bot mentioned this pull request Oct 9, 2026
Draft
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

andreiborza andreiborza approved these changes
isaacs Awaiting requested review from isaacs isaacs is a code owner automatically assigned from getsentry/team-javascript-sdks-server

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

bun: test coverage is thin

2 participants

Back | FazBrowse Home | New Git URL