fix(commonjs): handle requiring mixed ES modules

packages/commonjs/src/transform.js

@@ -691,5 +691,5 @@ export function transformCommonjs(
   code = magicString.toString();
   const map = sourceMap ? magicString.generateMap() : null;
 
-  return { code, map, syntheticNamedExports: '__moduleExports' };
+  return { code, map, syntheticNamedExports: isEsModule ? false : '__moduleExports' };
 }

packages/commonjs/test/fixtures/function/transform-mixed-modules/_config.js

@@ -0,0 +1,6 @@
+module.exports = {
+  description: 'transforms mixed modules',
+  pluginOptions: {
+    transformMixedEsModules: true
+  }
+};

packages/commonjs/test/fixtures/function/transform-mixed-modules/dep.js

@@ -0,0 +1,3 @@
+const other = require('./other.js');
+
+export { other };

packages/commonjs/test/fixtures/function/transform-mixed-modules/main.js

@@ -0,0 +1,6 @@
+import { other } from './dep.js';
+
+const dep = require('./dep.js');
+
+t.is(other, 'other');
+t.deepEqual(dep, { other: 'other' });

packages/commonjs/test/fixtures/function/transform-mixed-modules/other.js

@@ -0,0 +1 @@
+module.exports = 'other';

packages/commonjs/test/snapshots/function.js.md

@@ -4142,6 +4142,25 @@ Generated by [AVA](https://ava.li).
       `,
     }
 
+## transform-mixed-modules
+
+> Snapshot 1
+
+    {
+      'main.js': `'use strict';␊
+      ␊
+      var other = 'other';␊
+      ␊
+      var dep = /*#__PURE__*/Object.freeze({␊
+      	__proto__: null,␊
+      	other: other␊
+      });␊
+      ␊
+      t.is(other, 'other');␊
+      t.deepEqual(dep, { other: 'other' });␊
+      `,
+    }
+
 ## transpiled-esm-default
 
 > Snapshot 1