Skip to content

module: CJS exports detection for all CJS modules

Rodrigo Muino Tomonari requested to merge github/fork/guybedford/cjs-export-detection-all into master

This is another PR for CommonJS named exports identical to https://github.com/nodejs/node/pull/33416 but without the restriction to modules with the __esModule flag.

This PR is made to separate the consensus process for this PR from the PR at https://github.com/nodejs/node/pull/33416, which strictly speaking could land given that it now has consensus.

If all those involved could review this PR with their opinions on relaxing the __esModule restriction that would be a help to determine which if any approach might ship. This way we can clearly choose between the three choices.

//cc @nodejs/modules-active-members

Merge request reports