Replacing esbuild with rolldown

[RobinTail]

Includes:

• Replacing tsx with unrun #3051
• Override vite to rolldown-vite #3053

Summary by CodeRabbit

• Chores

  • Switched build/test/CI tooling from tsx to unrun across the project and updated dev dependencies accordingly.
  • Added development utilities (compression, express-fileupload, typescript, zod).
  • Updated workspace configuration, removed certain build exclusions, and added dependency overrides.

• Tests

  • Adjusted a test expectation to accommodate a minor formatting change in an error message.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)

• pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

Replaced tsx with unrun across CI workflows, test specs, package scripts, and workspace config; removed tsx from devDependencies and added unrun; added devDependencies in example/package.json; minor test expectation formatting change.

Changes

Cohort / File(s)	Summary
CI / Workflow /.github/workflows/headers.yml	Replaced pnpm tsx tools/headers.ts with pnpm unrun tools/headers.ts.
Test specs compat-test/quick-start.spec.ts, esm-test/quick-start.spec.ts, example/index.spec.ts	Swapped spawned runtime from tsx to unrun for example/test scripts.
Example package example/package.json	Replaced tsx with unrun in start, build:docs, build:client; added devDependencies compression, express-fileupload, typescript, zod.
Root package package.json	Replaced tsx with unrun in prebuild/pretest scripts; removed tsx from devDependencies; added unrun@^0.2.17.
Workspace config pnpm-workspace.yaml	Removed esbuild from onlyBuiltDependencies; removed tsx from minimumReleaseAgeExclude; added overrides for lightningcss and vite.
Test expectation express-zod-api/tests/common-helpers.spec.ts	Adjusted expected function formatting from multi-line to single-line () => {}.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

• Review attention:
  • Confirm unrun invocation and flags match prior tsx behavior in scripts/tools (headers, make-tests, contributors/license).
  • Validate added devDependencies in example/package.json and potential type/tooling implications.
  • Verify pnpm workspace overrides (vite, lightningcss) and removal of entries (esbuild, tsx) are intended and do not break builds.
  • Ensure the test expectation formatting change is stable across environments.

Possibly related PRs

• Switching to pnpm #2428 — similar change to .github/workflows/headers.yml (switching runner invocation).
• Well known headers in typescript file #3014 — changes to tools/headers.ts, directly related to workflow invocation changes.
• Upgrading tsdown #3033 — updates around unrun usage and devDependencies, closely related to replacing tsx.

Suggested labels

external bug

Poem

🐇 I swapped my boots from tsx to unrun today,
Hopped through scripts and tests along the way.
New deps in the burrow, a tiny formatting fix,
A tidy little hop — quick change, clever mix. ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title claims to replace esbuild with rolldown, but the changes show replacing tsx with unrun and updating pnpm workspace overrides—no rolldown migration is evident.	Revise the title to accurately reflect the main changes, such as 'Replace tsx with unrun and update pnpm workspace overrides' or clarify the rolldown changes if present.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coveralls-official]

coverage: 100.0%. remained the same
when pulling 8ae01bc on exp-no-esbuild
into eb875ef on master.

[coveralls-official]

coverage: 100.0%. remained the same
when pulling 7485523 on exp-no-esbuild
into f61a1b3 on master.

[RobinTail]

unrun after certain update is failing to import ramda and it's causing failure

Filed an issue to unrun:
Gugustinette/unrun#11

[RobinTail]

The experiment is a success.
I'm going to wait vite to stabilize on migrating to rolldown

[RobinTail]

Can now use vite v8 beta for this
https://vite.dev/blog/announcing-vite8-beta

[RobinTail]

I noticed that unrun being spawned by vitest and then killed, does actually leave the process running.
Gugustinette/unrun#13
it's fixed