Skip to content

Use tsdown for building the library#812

Open
timokoessler wants to merge 18 commits intomainfrom
new-build-process
Open

Use tsdown for building the library#812
timokoessler wants to merge 18 commits intomainfrom
new-build-process

Conversation

@timokoessler
Copy link
Copy Markdown
Member

@timokoessler timokoessler commented Nov 7, 2025

  • Would allow us to bundle dependencies, e.g. unplugin, to ensure we still have 0 npm dependencies
  • Reduce lib size & loading time

Todos

  • Some benchmarks are trying to import internal files from the build directory
  • Rolldown does not work on old Node.js versions

Summary by Aikido

Security Issues: 0 Quality Issues: 0 Resolved Issues: 0

⚡ Enhancements

  • Switched library build to tsdown and updated build scripts
  • Adjusted CI to produce original and flattened build artifacts
  • Changed build output handling and added d.ts imports // @ts-ignore

🔧 Refactors

  • Introduced getLibraryRoot and isLibBundled and replaced hardcoded paths
  • Updated ESM test script to require Node 24 and async parsing

More info

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 13, 2025

Codecov Report

❌ Patch coverage is 94.11765% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
library/helpers/getLibraryRoot.ts 75.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@timokoessler timokoessler marked this pull request as ready for review November 13, 2025 13:57
Comment thread scripts/tests-esm.mjs
@timokoessler timokoessler changed the base branch from main to build-once-in-ci December 15, 2025 08:37
Base automatically changed from build-once-in-ci to main December 15, 2025 13:42
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.

1 participant