Implemented kill-switch for mixed commonjs in mixed es-cjs (Closes ro…

…llup#348, rollup#342)

`transformMixedEsModules = false`

packages/commonjs/README.md

@@ -106,6 +106,13 @@ Default: `true`
 
 If false, skips source map generation for CommonJS modules.
 
+### `transformMixedEsModules`
+
+Type: `Boolean`<br>
+Default: `false`
+
+In case of modules that contain commonjs code you can enable/disable the transformer. Set it to `true` if you know that `require` calls should be transformed, or `false` if you know that the code contains env detection and the `require` should survive the transform.
+
 ### `ignore`
 
 Type: `Array[...String | (String) => Boolean]`<br>

packages/commonjs/src/index.js

@@ -66,10 +66,14 @@ export default function commonjs(options = {}) {
 
     const isDynamicRequireModule = dynamicRequireModuleSet.has(normalizePathSlashes(id));
 
-    if (isEsModule && !isDynamicRequireModule) {
+    if (isEsModule && (!isDynamicRequireModule || !options.transformMixedEsModules)) {
       (hasDefaultExport ? esModulesWithDefaultExport : esModulesWithoutDefaultExport).add(id);
+      if (!options.transformMixedEsModules) {
+        setIsCjsPromise(id, false);
+        return null;
+      }
     }
-    // it is not an ES module but it does not have CJS-specific elements.
+    // it is not an ES module AND it does not have CJS-specific elements.
     else if (!hasCjsKeywords(code, ignoreGlobal)) {
       esModulesWithoutDefaultExport.add(id);
       setIsCjsPromise(id, false);

packages/commonjs/test/fixtures/form/unambiguous-with-default-export/_config.js

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

packages/commonjs/test/fixtures/form/unambiguous-with-import/_config.js

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

packages/commonjs/test/fixtures/form/unambiguous-with-named-export/_config.js

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

packages/commonjs/test/fixtures/function/dynamic-require-code-splitting/_config.js

@@ -7,6 +7,7 @@ module.exports = {
     ]
   },
   pluginOptions: {
-    dynamicRequireTargets: ['fixtures/function/dynamic-require-code-splitting/target?.js']
+    dynamicRequireTargets: ['fixtures/function/dynamic-require-code-splitting/target?.js'],
+    transformMixedEsModules: true
   }
 };

packages/commonjs/test/fixtures/function/dynamic-require-es-entry/_config.js

@@ -1,6 +1,7 @@
 module.exports = {
   description: 'works when the entry point is an es module',
   pluginOptions: {
-    dynamicRequireTargets: ['fixtures/function/dynamic-require-es-entry/submodule.js']
+    dynamicRequireTargets: ['fixtures/function/dynamic-require-es-entry/submodule.js'],
+    transformMixedEsModules: true
   }
 };

packages/commonjs/test/fixtures/function/es6-export-with-global-sniffing/_config.js

@@ -4,5 +4,7 @@ module.exports = {
   options: {
     plugins: [nodeResolve()]
   },
-  pluginOptions: {}
+  pluginOptions: {
+    transformMixedEsModules: true
+  }
 };

packages/commonjs/test/types.ts

@@ -13,6 +13,7 @@ const config: import('rollup').RollupOptions = {
       extensions: ['.js', '.coffee'],
       ignoreGlobal: false,
       sourceMap: false,
+      transformMixedEsModules: false,
       ignore: ['conditional-runtime-dependency'],
       dynamicRequireTargets: ['node_modules/logform/*.js']
     })

packages/commonjs/types/index.d.ts

@@ -30,6 +30,11 @@ interface RollupCommonJSOptions {
    * @default true
    */
   sourceMap?: boolean;
+  /**
+   * In case of modules that contain commonjs code you can enable/disable the transformer. Set it to `true` if you know that `require` calls should be transformed, or `false` if you know that the code contains env detection and the `require` should survive the transform.
+   * @default false
+   */
+  transformMixedEsModules?: boolean;
   /**
    * explicitly specify unresolvable named exports
    * ([see below for more details](https://github.com/rollup/plugins/tree/master/packages/commonjs#named-exports))