allow to disable esm shim?

[himself65]

bunchee/src/build-config.ts

Line 278 in c4fd618

bundleConfig.format === 'esm' && esmShim(),

[himself65]

There are some reasons, for example, there is a TS file that is

const fetch = globalThis.fetch === 'undefined' ? require('node-fetch') : fetch

I won't have node:module because this package might be published to the browser. I just want to do some environment check here

[huozhi]

If you disable the esm shim then the bundled code will be incorrect? How the require is supposed to be handled.

I feel like you should just do polyfill on server side chunks or exports if that can solve your case.

[himself65]

For example Im writing the polyfill code in a function it will both emit to cjs and esm. And in this case require function shouldn’t cause a side effect

[huozhi]

It would cause other error if you don't shim require call, probably should consider dynamic import

[huozhi]

Reopen since new Node.js starts supporting the require(ESM)