Skip to content

test(bun): Run all Node integration suites on Bun - #24610

Open
JPeer264 wants to merge 2 commits into
jp/node-suites-bun-shared-runnerfrom
jp/node-suites-bun
Open

test(bun): Run all Node integration suites on Bun#24610
JPeer264 wants to merge 2 commits into
jp/node-suites-bun-shared-runnerfrom
jp/node-suites-bun

Conversation

@JPeer264

@JPeer264 JPeer264 commented Sep 23, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Bun now runs every Node suite except an explicit exclude list, grouped by cause, so a new Node test runs on Bun by default and a gap shows up as a failing test rather than as missing coverage. Excluded files do not run, which keeps the job fast even though most exclusions would otherwise wait for a timeout.

The exclude groups document what @sentry/node currently cannot do on Bun: no http.server span (Bun does not publish http.server.request.start), no fetch spans (Bun's fetch does not use undici), no outgoing node:http instrumentation on Bun 1.3.14 (#23881), and no auto-instrumentation under bun run (#23882). The list is based on Bun 1.3.14, the version CI pins; several suites already pass on newer Bun.

The suites that assert the SDK name use EXPECTED_SDK_NAME, so they also pass in the @sentry/bun project.

FWIW in a later PR more tests will be enabled as Bun will then run Bun.build before - see #24612

Part of #23889.

🤖 Generated with Claude Code

@linear-code

linear-code Bot commented Sep 23, 2026

Copy link
Copy Markdown

JS-3515

@github-actions

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

Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size % Change Change
@sentry/browser 29.23 kB -0.03% -6 B 🔽
@sentry/browser - with treeshaking flags 27.49 kB -0.04% -10 B 🔽
@sentry/browser - with treeshaking flags tracing without tracing 27.4 kB -0.01% -1 B 🔽
@sentry/browser (incl. Tracing) 51.16 kB +0.01% +3 B 🔺
@sentry/browser (incl. Tracing + Span Streaming) 51.17 kB -0.02% -8 B 🔽
@sentry/browser (incl. Tracing, Profiling) 54.16 kB -0.03% -12 B 🔽
@sentry/browser (incl. Tracing, Replay) 90.75 kB -0.02% -15 B 🔽
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 79.85 kB -0.02% -11 B 🔽
@sentry/browser (incl. Tracing, Replay with Canvas) 95.45 kB -0.02% -18 B 🔽
@sentry/browser (incl. Tracing, Replay, Feedback) 108.42 kB +0.01% +9 B 🔺
@sentry/browser (incl. Feedback) 46.76 kB -0.02% -8 B 🔽
@sentry/browser (incl. sendFeedback) 34.29 kB -0.03% -10 B 🔽
@sentry/browser (incl. FeedbackAsync) 39.39 kB -0.06% -22 B 🔽
@sentry/browser (incl. Metrics) 30.25 kB -0.01% -2 B 🔽
@sentry/browser (incl. Logs) 30.5 kB -0.05% -14 B 🔽
@sentry/browser (incl. Metrics & Logs) 31.16 kB -0.05% -14 B 🔽
@sentry/react 30.98 kB -0.05% -14 B 🔽
@sentry/react (incl. Tracing) 53.44 kB -0.02% -6 B 🔽
@sentry/vue 36.72 kB -0.05% -15 B 🔽
@sentry/vue (incl. Tracing) 53.69 kB -0.03% -12 B 🔽
@sentry/svelte 29.25 kB -0.03% -8 B 🔽
CDN Bundle 30.93 kB +0.01% +2 B 🔺
CDN Bundle (incl. Tracing) 51.69 kB +0.01% +5 B 🔺
CDN Bundle (incl. Logs, Metrics) 33.19 kB -0.03% -9 B 🔽
CDN Bundle (incl. Tracing, Logs, Metrics) 53.66 kB -0.02% -7 B 🔽
CDN Bundle (incl. Replay, Logs, Metrics) 73.91 kB -0.01% -3 B 🔽
CDN Bundle (incl. Tracing, Replay) 89.27 kB -0.02% -9 B 🔽
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 91.23 kB -0.02% -11 B 🔽
CDN Bundle (incl. Tracing, Replay, Feedback) 95.43 kB -0.02% -19 B 🔽
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 97.41 kB -0.02% -11 B 🔽
CDN Bundle - uncompressed 91.4 kB -0.01% -8 B 🔽
CDN Bundle (incl. Tracing) - uncompressed 153.76 kB -0.01% -8 B 🔽
CDN Bundle (incl. Logs, Metrics) - uncompressed 97.97 kB -0.01% -8 B 🔽
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 159.72 kB -0.01% -8 B 🔽
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 227.54 kB -0.01% -8 B 🔽
CDN Bundle (incl. Tracing, Replay) - uncompressed 273.49 kB -0.01% -8 B 🔽
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 279.43 kB -0.01% -8 B 🔽
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 287.2 kB -0.01% -8 B 🔽
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 293.13 kB -0.01% -8 B 🔽
@sentry/nextjs (client) 55.78 kB +0.02% +6 B 🔺
@sentry/sveltekit (client) 51.58 kB -0.03% -11 B 🔽
@sentry/core/server 39.92 kB - -
@sentry/core/browser 13.63 kB - -
@sentry/node 133.86 kB +0.01% +12 B 🔺
@sentry/node/import (ESM hook with diagnostics-channel injection) 82.4 kB - -
@sentry/node - without tracing 90.39 kB -0.01% -7 B 🔽
@sentry/node - without channel injection 112.4 kB +0.02% +17 B 🔺
@sentry/aws-serverless 98.69 kB +0.02% +18 B 🔺
@sentry/cloudflare (withSentry) - minified 206.38 kB -0.01% -12 B 🔽
@sentry/cloudflare (withSentry) 513.49 kB -0.01% -26 B 🔽

View base workflow run

@JPeer264
JPeer264 added this pull request to stack #24613 September 23, 2026 06:14
@JPeer264
JPeer264 force-pushed the jp/node-suites-bun branch 2 times, most recently from 9099fef to 34ea3db Compare September 23, 2026 08:27
@JPeer264
JPeer264 removed this pull request from stack #24613 September 23, 2026 09:51
@JPeer264
JPeer264 added this pull request to stack #24633 September 23, 2026 09:51
@JPeer264
JPeer264 force-pushed the jp/node-suites-bun branch 3 times, most recently from f7addac to 1feee30 Compare September 23, 2026 13:31
@JPeer264 @claude
test(bun): Run all Node integration suites on Bun
278df2a 
The Bun package now selects every Node suite and excludes the ones that
fail on Bun, grouped by cause: Node-only features, no `http.server` span
because Bun does not publish `http.server.request.start`, no `fetch`
instrumentation because Bun's `fetch` does not publish undici channels,
no outgoing `node:http` instrumentation on Bun 1.3.14 (JS-3507), and no
auto-instrumentation under `bun run` (JS-3508). Excluded files do not
run, so the job stays fast. The list is based on Bun 1.3.14, the version
CI pins.

The suites that assert the SDK name use `EXPECTED_SDK_NAME`, so they
also pass with `@sentry/bun`, and `bun-runtime-metrics` runs on Bun
instead of Node.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@JPeer264
JPeer264 removed this pull request from stack #24633 September 23, 2026 16:19
@JPeer264
JPeer264 changed the base branch from jp/node-suites-runtime-switch to jp/node-suites-bun-shared-runner September 23, 2026 16:19
@JPeer264
JPeer264 added this pull request to stack #24670 September 23, 2026 16:19
@JPeer264 @claude
fixup! test(bun): Run all Node integration suites on Bun
2f6d13a 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@JPeer264
JPeer264 marked this pull request as ready for review September 23, 2026 16:42
@JPeer264
JPeer264 requested a review from a team as a code owner September 23, 2026 16:42
@JPeer264
JPeer264 requested review from isaacs and mydea and removed request for a team September 23, 2026 16:42
@JPeer264 JPeer264 self-assigned this Sep 23, 2026
@JPeer264
JPeer264 removed this pull request from stack #24670 September 23, 2026 17:26
@JPeer264
JPeer264 added this pull request to stack #24671 September 23, 2026 17:31

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@isaacs isaacs Awaiting requested review from isaacs isaacs is a code owner automatically assigned from getsentry/team-javascript-sdks-server
@mydea mydea Awaiting requested review from mydea mydea is a code owner automatically assigned from getsentry/team-javascript-sdks-server
@andreiborza andreiborza Awaiting requested review from andreiborza

At least 1 approving review is required to merge this pull request.

Assignees

@JPeer264 JPeer264

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@JPeer264