Skip to content

ref(build): Replace ts-node with tsx for build and CI scripts#22186

Merged
logaretm merged 1 commit into
developfrom
awad/migrate-ts-node-to-tsx
Jul 13, 2026
Merged

ref(build): Replace ts-node with tsx for build and CI scripts#22186
logaretm merged 1 commit into
developfrom
awad/migrate-ts-node-to-tsx

Conversation

@logaretm

@logaretm logaretm commented Jul 10, 2026

Copy link
Copy Markdown
Member

Swap the ts-node script runner for tsx across all build/CI/e2e scripts and the node integration-test runners (node --import tsx).

tsx uses esbuild so it runs on any TypeScript version, which unblocks upgrading the compiler. This is needed for TypeScript 7.0 upgrade and doesn't change much in our case.

Some tests required some changes because it relied on ts-node inlining require calls without hoisting which esbuild in tsx is doing (matches ESM semantics), causing instrumentations to "miss". I adjusted the tests to use our instrumentation wrappers so that they behave like a real ESM environment.

Added bonus is it has less dependencies, which our lock file now reflects.

@logaretm logaretm added this to the 11.0.0 milestone Jul 10, 2026
@logaretm logaretm force-pushed the awad/migrate-ts-node-to-tsx branch from 8f29231 to 34a136b Compare July 10, 2026 18:37
@logaretm logaretm changed the base branch from develop to awad/js-1293-drop-typescript-38-support July 10, 2026 18:38
@logaretm logaretm force-pushed the awad/js-1293-drop-typescript-38-support branch from fb2b2d8 to 410a72f Compare July 10, 2026 18:49
@logaretm logaretm force-pushed the awad/migrate-ts-node-to-tsx branch from 34a136b to 9549b5c Compare July 10, 2026 18:49
@logaretm logaretm force-pushed the awad/js-1293-drop-typescript-38-support branch from 410a72f to bc07973 Compare July 10, 2026 18:56
@logaretm logaretm force-pushed the awad/migrate-ts-node-to-tsx branch 2 times, most recently from f0bcefb to 96ab381 Compare July 10, 2026 19:00
@logaretm logaretm changed the base branch from awad/js-1293-drop-typescript-38-support to develop July 10, 2026 19:00
@logaretm logaretm requested a review from mydea July 10, 2026 19:02
@github-actions

github-actions Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size % Change Change
@sentry/browser 27.6 kB - -
@sentry/browser - with treeshaking flags 26.04 kB - -
@sentry/browser (incl. Tracing) 46.35 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 48.14 kB - -
@sentry/browser (incl. Tracing, Profiling) 51.13 kB - -
@sentry/browser (incl. Tracing, Replay) 85.62 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 75.26 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 90.33 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 102.99 kB - -
@sentry/browser (incl. Feedback) 44.78 kB - -
@sentry/browser (incl. sendFeedback) 32.4 kB - -
@sentry/browser (incl. FeedbackAsync) 37.53 kB - -
@sentry/browser (incl. Metrics) 28.68 kB - -
@sentry/browser (incl. Logs) 28.93 kB - -
@sentry/browser (incl. Metrics & Logs) 29.61 kB - -
@sentry/react 29.39 kB - -
@sentry/react (incl. Tracing) 48.62 kB - -
@sentry/vue 33.03 kB - -
@sentry/vue (incl. Tracing) 48.33 kB - -
@sentry/svelte 27.63 kB - -
CDN Bundle 30 kB - -
CDN Bundle (incl. Tracing) 48.33 kB - -
CDN Bundle (incl. Logs, Metrics) 31.58 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 49.65 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 70.82 kB - -
CDN Bundle (incl. Tracing, Replay) 85.84 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 87.16 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 91.64 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 92.92 kB - -
CDN Bundle - uncompressed 89.33 kB - -
CDN Bundle (incl. Tracing) - uncompressed 146.07 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 94.03 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 150.05 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 218.76 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 265.28 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 269.24 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 278.98 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 282.93 kB - -
@sentry/nextjs (client) 51.17 kB - -
@sentry/sveltekit (client) 46.8 kB - -
@sentry/core/server 78.42 kB - -
@sentry/core/browser 64.77 kB - -
@sentry/node-core 62.73 kB - -
@sentry/node 125.35 kB -0.01% -1 B 🔽
@sentry/node (incl. diagnostics channel injection) 139.63 kB -0.01% -1 B 🔽
@sentry/node/import (ESM hook with diagnostics-channel injection) 69.96 kB - -
@sentry/node/light 50.73 kB - -
@sentry/node - without tracing 74.04 kB - -
@sentry/aws-serverless 83.26 kB +0.01% +1 B 🔺
@sentry/cloudflare (withSentry) - minified 181.47 kB - -
@sentry/cloudflare (withSentry) 448.98 kB - -

View base workflow run

Swap the ts-node script runner for tsx across all build/CI/e2e scripts
and the node integration-test runners (node --import tsx). tsx uses
esbuild so it runs on any TypeScript version, which unblocks upgrading
the compiler. No behavior change on the current TypeScript.
@logaretm logaretm force-pushed the awad/migrate-ts-node-to-tsx branch from 96ab381 to df9681d Compare July 11, 2026 00:56
@logaretm logaretm marked this pull request as ready for review July 11, 2026 02:30
@logaretm logaretm requested review from a team as code owners July 11, 2026 02:30
@logaretm logaretm requested review from JPeer264, Lms24, chargome, isaacs and s1gr1d and removed request for a team July 11, 2026 02:30
@logaretm logaretm removed this from the 11.0.0 milestone Jul 11, 2026

@JPeer264 JPeer264 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.

🚀

@logaretm logaretm merged commit 1232969 into develop Jul 13, 2026
310 checks passed
@logaretm logaretm deleted the awad/migrate-ts-node-to-tsx branch July 13, 2026 14:09
logaretm added a commit that referenced this pull request Jul 13, 2026
…lder (#22190)

Replaces `@nuxt/module-builder` with a plain rollup + tsc setup for the
`build/module` output. Rollup bundles the module entry and emits the
runtime files one-to-one (preserveModules), tsc emits the declarations,
and a small script writes `module.cjs`, `module.json` and `types.d.ts`.

The point is to drop a build tool that consumes the TypeScript compiler
API, so the package no longer pins us to a specific TS compiler, also
streamlines nuxt SDK build step for future changes like rolldown.

We still have E2E tests ensuring the module output is consumable by Nuxt
apps, so that's not something I would worry about drifting in the
future.

Stacked on #22186.
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