Skip to content

fix: allow to configure advanced treeshake options#573

Merged
sxzz merged 1 commit intorolldown:mainfrom
BlankParticle:feat/advance-treeshake
Oct 29, 2025
Merged

fix: allow to configure advanced treeshake options#573
sxzz merged 1 commit intorolldown:mainfrom
BlankParticle:feat/advance-treeshake

Conversation

@BlankParticle
Copy link
Copy Markdown
Contributor

Description

tsdown's treeshake option now takes an config object along with enable/disable boolean, just like how its configured in rolldown
https://rolldown.rs/options/treeshake

Linked Issues

Additional context

I wanted to configure treeshake.moduleSideEffects option based on the rolldown docs, but tsdown gave me type errors, still I added the option ignoring the type error and it worked as expected, so this PR adds the relevant type to the config

@netlify
Copy link
Copy Markdown

netlify bot commented Oct 29, 2025

Deploy Preview for tsdown ready!

Name Link
🔨 Latest commit 164feaa
🔍 Latest deploy log https://app.netlify.com/projects/tsdown/deploys/6901c40e80dd840008388d53
😎 Deploy Preview https://deploy-preview-573--tsdown.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Oct 29, 2025

Open in StackBlitz

npm i https://pkg.pr.new/tsdown@573
npm i https://pkg.pr.new/create-tsdown@573

commit: 164feaa

@sxzz sxzz merged commit 30f14ff into rolldown:main Oct 29, 2025
14 checks passed
@BlankParticle BlankParticle deleted the feat/advance-treeshake branch October 29, 2025 15:04
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.

2 participants