Skip to content

test(deno): Run all Node integration suites on Deno - #24611

Draft
JPeer264 wants to merge 3 commits into
jp/node-suites-deno-esm-fixfrom
jp/node-suites-deno
Draft

JPeer264 wants to merge 3 commits into
jp/node-suites-deno-esm-fixfrom
jp/node-suites-deno

Conversation

@JPeer264

@JPeer264 JPeer264 commented Sep 23, 2026

Copy link
Copy Markdown
Member

Deno now runs every Node suite except an explicit exclude list, including the database suites that need Docker, so the Deno CI job gets the disk space step and timeout of the Node integration job. Unlike the Deno-only orchestrion suites, which mostly publish fake channel messages, these load the real libraries (#23897). They depend on the ES module fix in the previous PR of the stack.

Harness changes: scenarios always start in the node-integration-tests folder (some read the working directory), a test that times out prints the child's output and state, and yarn test <filter> works in this package as in the others.

Closes #23897. Part of #24052.

🤖 Generated with Claude Code

@linear-code

linear-code Bot commented Sep 23, 2026

Copy link
Copy Markdown

JS-3587

JS-3523

@github-actions

github-actions Bot commented Sep 23, 2026

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.96 kB +0.08% +106 B 🔺
@sentry/node/import (ESM hook with diagnostics-channel injection) 82.57 kB +0.21% +172 B 🔺
@sentry/node - without tracing 90.48 kB +0.1% +89 B 🔺
@sentry/node - without channel injection 112.4 kB +0.02% +18 B 🔺
@sentry/aws-serverless 98.78 kB +0.12% +113 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-deno branch 2 times, most recently from a873afb to e62dd7f 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-deno branch 2 times, most recently from 8a5e324 to 77995a1 Compare September 23, 2026 10:04
@JPeer264
JPeer264 force-pushed the jp/node-suites-deno branch 2 times, most recently from 17fdfc4 to e4e9494 Compare September 23, 2026 16:19
@JPeer264
JPeer264 removed this pull request from stack #24633 September 23, 2026 16:19
@JPeer264
JPeer264 changed the base branch from jp/node-suites-bun to jp/node-suites-deno-esm-fix September 23, 2026 16:19
@JPeer264
JPeer264 added this pull request to stack #24670 September 23, 2026 16:19
The Deno package now selects every Node suite and excludes the ones that
do not run on Deno, grouped by cause: Node-only features, no `fetch`
instrumentation because Deno's `fetch` does not publish undici channels,
a Prisma interop limit, suites that fail for a cause not investigated
yet, and one flaky suite. The list is based on Deno 2.8.3, the version CI
pins. The database suites run too, so the Deno CI job gets the disk
space step and the 20 minute timeout of the Node integration job.

The runner now starts every scenario in the `node-integration-tests`
folder, because the Deno and Bun packages run the suites from their own
folder and scenarios such as `modules` read the working directory. When
a test times out, it prints the child output with the child state.
`yarn test <filter>` now filters the shared suites, as in the other
packages.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
JPeer264 and others added 2 commits September 23, 2026 18:51
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

deno: 22 of 24 orchestrion suites never load the real library

1 participant