Skip to content

Conversation

@jinghaihan
Copy link
Contributor

Description

Adds documentation warning about mainFields configuration when using the neutral platform.

Changes

  • Add warning block in platform documentation (EN & ZH-CN)
  • Explain that neutral platform defaults mainFields to [], relying only on exports field
  • Provide solution for resolving issues with older packages
  • Link to Rolldown resolve options documentation

Additional context

Fixes issues where users encounter [UNRESOLVED_IMPORT] warnings when using neutral platform with packages that don't have exports field defined.

@netlify
Copy link

netlify bot commented Dec 13, 2025

Deploy Preview for tsdown ready!

Name Link
🔨 Latest commit 94515c5
🔍 Latest deploy log https://app.netlify.com/projects/tsdown/deploys/693f07e5d0899600085a4be6
😎 Deploy Preview https://deploy-preview-651--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

pkg-pr-new bot commented Dec 13, 2025

Open in StackBlitz

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

commit: 94515c5

@jinghaihan jinghaihan force-pushed the docs/main-fields branch 2 times, most recently from c50b86e to f2634f2 Compare December 13, 2025 11:52
@jinghaihan jinghaihan changed the title docs: add warning about mainFields configuration for neutral platform docs: improve platform docs with module resolution strategy Dec 14, 2025
Copy link
Member

@sxzz sxzz left a comment

Choose a reason for hiding this comment

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

LGTM for the rest part.

@jinghaihan jinghaihan requested a review from sxzz December 14, 2025 16:04
@sxzz sxzz merged commit 0bd8adc into rolldown:main Dec 14, 2025
15 of 16 checks passed
@jinghaihan jinghaihan deleted the docs/main-fields branch December 14, 2025 23:58
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