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]

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.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch exp-no-esbuild

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 90fcd3e and bb5f3c2.

⛔ Files ignored due to path filters (1)

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

📒 Files selected for processing (1)

• package.json (2 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

• package.json

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Socket Security: Pull Request Alerts

---

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 bb5f3c2 on exp-no-esbuild
into ddd0173 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