Skip to content

fix(exports): correctly generate export paths for nested index files#559

Merged
sxzz merged 5 commits intorolldown:mainfrom
userquin:fix-nested-subpackage-exports
Oct 26, 2025
Merged

fix(exports): correctly generate export paths for nested index files#559
sxzz merged 5 commits intorolldown:mainfrom
userquin:fix-nested-subpackage-exports

Conversation

@userquin
Copy link
Copy Markdown
Contributor

@userquin userquin commented Oct 24, 2025

Description

This fix normalizes entrypoint paths to correctly handle subpackage exports, especially for nested index files and paths generated on Windows.

Linked Issues

resolves #558

Additional context

After updating my fork, on my local there are 3/4 tests failing: env and shims related to define/inject changes at Rolldown

@netlify
Copy link
Copy Markdown

netlify bot commented Oct 24, 2025

Deploy Preview for tsdown ready!

Name Link
🔨 Latest commit 236db36
🔍 Latest deploy log https://app.netlify.com/projects/tsdown/deploys/68fcb510518eb40008e511c2
😎 Deploy Preview https://deploy-preview-559--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 24, 2025

Open in StackBlitz

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

commit: 236db36

chunk.type === 'chunk' &&
chunk.isEntry &&
!RE_DTS.test(chunk.fileName),
!RE_DTS.test(slash(chunk.fileName)),
Copy link
Copy Markdown
Member

@sxzz sxzz Oct 24, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line can be reverted, since suffix name is the same in both Windows and Linux.

@sxzz sxzz merged commit ff946bc into rolldown:main Oct 26, 2025
14 checks passed
@userquin userquin deleted the fix-nested-subpackage-exports branch October 26, 2025 10:26
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.

export true doesn't normalize subpackages exports paths on Windows

2 participants