Remove module.exports in esm bundle

[egoist]

Issue Type

• Bug Report
• Feature Request
• Other

Description

cac/src/index.ts

Lines 13 to 14 in 5060bd9

	// @remove-for-deno
	module.exports = cac

This should be removed in esm bundle for Node.js too.

Normally it won't cause issues, unless you're trying to bundle your CLI into a single js file.

Possible Solutions

Using a rollup plugin to remove it, could be added here: https://github.com/cacjs/cac/blob/master/rollup.config.js

Info

• CAC version: 6.7.3
• Reproduction link:

[HomyeeKing]

I also met this kind of bug in vite,
here I export a variety called SHORTCUTS from 'cli.ts', and rollup bundle it into cjs format
once some other file refenect SHORTCUTS, the bundled cli.js file just have dependency requirments and exports.SHORTCUTS=cli.SHORTCUTS, no other codes in bundle file which will crash the CI flow

how to fix this