Skip to content

fix: skip module field when commonjs require#385

Merged
fi3ework merged 2 commits intomainfrom
module-field
Jul 14, 2025
Merged

fix: skip module field when commonjs require#385
fi3ework merged 2 commits intomainfrom
module-field

Conversation

@9aoy
Copy link
Copy Markdown
Collaborator

@9aoy 9aoy commented Jul 14, 2025

Summary

skip module field when commonjs require.

By default, rspack resolves the "module" field for commonjs first, but this is not always returned synchronously in esm.

image image

Related Links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@netlify
Copy link
Copy Markdown

netlify bot commented Jul 14, 2025

Deploy Preview for rstest-dev ready!

Name Link
🔨 Latest commit 8fe6395
🔍 Latest deploy log https://app.netlify.com/projects/rstest-dev/deploys/6874d3f9a5eec500083b37ba
😎 Deploy Preview https://deploy-preview-385--rstest-dev.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.

@9aoy 9aoy requested a review from fi3ework July 14, 2025 10:01
@fi3ework fi3ework merged commit ee63de9 into main Jul 14, 2025
15 checks passed
@fi3ework fi3ework deleted the module-field branch July 14, 2025 12:31
@9aoy 9aoy mentioned this pull request Jul 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants