Skip to content

Fix ESM package entrypoints#673

Merged
Simplify merged 1 commit intoSimplify:masterfrom
jahusa02:fix/672-esm-entrypoints
Apr 10, 2026
Merged

Fix ESM package entrypoints#673
Simplify merged 1 commit intoSimplify:masterfrom
jahusa02:fix/672-esm-entrypoints

Conversation

@jahusa02
Copy link
Copy Markdown
Contributor

Summary

  • route package-root import to the existing ESM build and require to an explicit CommonJS entry via an exports map
  • point main to build/ibantools.cjs and generate that file during the node build so type: module no longer breaks CJS resolution
  • add a package-root regression test covering both named ESM imports and CommonJS require('ibantools')

Verification

  • npm test
  • npm run lint

Closes #672

@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 88.921% (-10.8%) from 99.766% — jahusa02:fix/672-esm-entrypoints into Simplify:master

@Simplify
Copy link
Copy Markdown
Owner

@jahusa02 thank you for your contribution!

@Simplify Simplify merged commit 0e1b247 into Simplify:master Apr 10, 2026
9 of 10 checks passed
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.

ibantools@4.5.3 appears to expose an incompatible entrypoint to ESM consumers

3 participants