fix(commonjs): convert module.exports with __esModule property(#939) (#942)

* fix(commonjs): convert module.exports with `__esModule` property(#939)

* test(commonjs): add test sample for cjs with esm-property

* feat(commonjs): defaultIsModuleExports support module.exports

* fix(commonjs): do not check right node in module.exports reassgin when defaultIsModuleExports = false(#935)

Author: githoniel

packages/commonjs/src/ast-utils.js

@@ -107,3 +107,17 @@ function getDefinePropertyCallName(node, targetName) {
 export function isShorthandProperty(parent) {
   return parent && parent.type === 'Property' && parent.shorthand;
 }
+
+export function hasDefineEsmProperty(node) {
+  return node.properties.some((property) => {
+    if (
+      property.type === 'Property' &&
+      property.key.type === 'Identifier' &&
+      property.key.name === '__esModule' &&
+      isTruthy(property.value)
+    ) {
+      return true;
+    }
+    return false;
+  });
+}

packages/commonjs/src/transform-commonjs.js

@@ -9,6 +9,7 @@ import MagicString from 'magic-string';
 import {
   getKeypath,
   isDefineCompiledEsm,
+  hasDefineEsmProperty,
   isFalsy,
   isReference,
   isShorthandProperty,
@@ -147,6 +148,18 @@ export default function transformCommonjs(
               } else if (!firstTopLevelModuleExportsAssignment) {
                 firstTopLevelModuleExportsAssignment = node;
               }
+
+              if (defaultIsModuleExports === false) {
+                shouldWrap = true;
+              } else if (defaultIsModuleExports === 'auto') {
+                if (node.right.type === 'ObjectExpression') {
+                  if (hasDefineEsmProperty(node.right)) {
+                    shouldWrap = true;
+                  }
+                } else if (defaultIsModuleExports === false) {
+                  shouldWrap = true;
+                }
+              }
             } else if (exportName === KEY_COMPILED_ESM) {
               if (programDepth > 3) {
                 shouldWrap = true;

packages/commonjs/test/fixtures/form/defaultIsModuleExports-auto-reassign-exports-__esModule/_config.js

@@ -0,0 +1,5 @@
+module.exports = {
+  options: {
+    defaultIsModuleExports: 'auto'
+  }
+};

packages/commonjs/test/fixtures/form/defaultIsModuleExports-auto-reassign-exports-__esModule/input.js

@@ -0,0 +1 @@
+module.exports = { __esModule: true, default: { foo: 'bar' }}

packages/commonjs/test/fixtures/form/defaultIsModuleExports-auto-reassign-exports-__esModule/output.js

@@ -0,0 +1,9 @@
+import * as commonjsHelpers from "_commonjsHelpers.js";
+import { __module as inputModule, exports as input } from "\u0000fixtures/form/defaultIsModuleExports-auto-reassign-exports-__esModule/input.js?commonjs-module"
+
+(function (module) {
+module.exports = { __esModule: true, default: { foo: 'bar' }}
+}(inputModule));
+
+export default /*@__PURE__*/commonjsHelpers.getDefaultExportFromCjs(input);
+export { input as __moduleExports };

packages/commonjs/test/fixtures/form/defaultIsModuleExports-auto-reassign-exports-no-__esModule/_config.js

@@ -0,0 +1,5 @@
+module.exports = {
+  options: {
+    defaultIsModuleExports: 'auto'
+  }
+};

packages/commonjs/test/fixtures/form/defaultIsModuleExports-auto-reassign-exports-no-__esModule/input.js

@@ -0,0 +1 @@
+module.exports = { default: { foo: 'bar' }}

packages/commonjs/test/fixtures/form/defaultIsModuleExports-auto-reassign-exports-no-__esModule/output.js

@@ -0,0 +1,6 @@
+import * as commonjsHelpers from "_commonjsHelpers.js";
+
+var input = { default: { foo: 'bar' }}
+
+export default input;
+export { input as __moduleExports };

packages/commonjs/test/fixtures/form/defaultIsModuleExports-false-reassign-exports-__esModule/_config.js

@@ -0,0 +1,5 @@
+module.exports = {
+  options: {
+    defaultIsModuleExports: false
+  }
+};

packages/commonjs/test/fixtures/form/defaultIsModuleExports-false-reassign-exports-__esModule/input.js

@@ -0,0 +1 @@
+module.exports = { __esModule: true, default: { foo: 'bar' }};