refactor(commonjs): add early exit for namespace augmentation

rollup · Sep 18, 2020 · 6a91d84 · 6a91d84
1 parent cf662bd
commit 6a91d84
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 0 deletions.
diff --git a/packages/commonjs/src/helpers.js b/packages/commonjs/src/helpers.js
@@ -54,6 +54,7 @@ export function getDefaultExportFromNamespaceIfNotNamed (n) {
 }
 
 export function getAugmentedNamespace(n) {
+	if (n.__esModule) return n;
 	var a = Object.defineProperty({}, '__esModule', {value: true});
 	Object.keys(n).forEach(function (k) {
 		var d = Object.getOwnPropertyDescriptor(n, k);

diff --git a/packages/commonjs/test/snapshots/function.js.md b/packages/commonjs/test/snapshots/function.js.md
@@ -2824,6 +2824,7 @@ Generated by [AVA](https://avajs.dev).
       var externalEsmDefault = require('external-esm-default');␊
       ␊
       function getAugmentedNamespace(n) {␊
+      	if (n.__esModule) return n;␊
       	var a = Object.defineProperty({}, '__esModule', {value: true});␊
       	Object.keys(n).forEach(function (k) {␊
       		var d = Object.getOwnPropertyDescriptor(n, k);␊
@@ -2869,6 +2870,7 @@ Generated by [AVA](https://avajs.dev).
       var externalEsmDefault = require('external-esm-default');␊
       ␊
       function getAugmentedNamespace(n) {␊
+      	if (n.__esModule) return n;␊
       	var a = Object.defineProperty({}, '__esModule', {value: true});␊
       	Object.keys(n).forEach(function (k) {␊
       		var d = Object.getOwnPropertyDescriptor(n, k);␊
@@ -2910,6 +2912,7 @@ Generated by [AVA](https://avajs.dev).
       var externalEsmDefault = require('external-esm-default');␊
       ␊
       function getAugmentedNamespace(n) {␊
+      	if (n.__esModule) return n;␊
       	var a = Object.defineProperty({}, '__esModule', {value: true});␊
       	Object.keys(n).forEach(function (k) {␊
       		var d = Object.getOwnPropertyDescriptor(n, k);␊
@@ -3049,6 +3052,7 @@ Generated by [AVA](https://avajs.dev).
       });␊
       ␊
       function getAugmentedNamespace(n) {␊
+      	if (n.__esModule) return n;␊
       	var a = Object.defineProperty({}, '__esModule', {value: true});␊
       	Object.keys(n).forEach(function (k) {␊
       		var d = Object.getOwnPropertyDescriptor(n, k);␊
@@ -3246,6 +3250,7 @@ Generated by [AVA](https://avajs.dev).
       });␊
       ␊
       function getAugmentedNamespace(n) {␊
+      	if (n.__esModule) return n;␊
       	var a = Object.defineProperty({}, '__esModule', {value: true});␊
       	Object.keys(n).forEach(function (k) {␊
       		var d = Object.getOwnPropertyDescriptor(n, k);␊
@@ -3351,6 +3356,7 @@ Generated by [AVA](https://avajs.dev).
       var trueMixed = 'default';␊
       ␊
       function getAugmentedNamespace(n) {␊
+      	if (n.__esModule) return n;␊
       	var a = Object.defineProperty({}, '__esModule', {value: true});␊
       	Object.keys(n).forEach(function (k) {␊
       		var d = Object.getOwnPropertyDescriptor(n, k);␊
@@ -3438,6 +3444,7 @@ Generated by [AVA](https://avajs.dev).
       }␊
       ␊
       function getAugmentedNamespace(n) {␊
+      	if (n.__esModule) return n;␊
       	var a = Object.defineProperty({}, '__esModule', {value: true});␊
       	Object.keys(n).forEach(function (k) {␊
       		var d = Object.getOwnPropertyDescriptor(n, k);␊
@@ -3643,6 +3650,7 @@ Generated by [AVA](https://avajs.dev).
       });␊
       ␊
       function getAugmentedNamespace(n) {␊
+      	if (n.__esModule) return n;␊
       	var a = Object.defineProperty({}, '__esModule', {value: true});␊
       	Object.keys(n).forEach(function (k) {␊
       		var d = Object.getOwnPropertyDescriptor(n, k);␊
@@ -3708,6 +3716,7 @@ Generated by [AVA](https://avajs.dev).
       }␊
       ␊
       function getAugmentedNamespace(n) {␊
+      	if (n.__esModule) return n;␊
       	var a = Object.defineProperty({}, '__esModule', {value: true});␊
       	Object.keys(n).forEach(function (k) {␊
       		var d = Object.getOwnPropertyDescriptor(n, k);␊
@@ -4262,6 +4271,7 @@ Generated by [AVA](https://avajs.dev).
       });␊
       ␊
       function getAugmentedNamespace(n) {␊
+      	if (n.__esModule) return n;␊
       	var a = Object.defineProperty({}, '__esModule', {value: true});␊
       	Object.keys(n).forEach(function (k) {␊
       		var d = Object.getOwnPropertyDescriptor(n, k);␊
@@ -4302,6 +4312,7 @@ Generated by [AVA](https://avajs.dev).
       });␊
       ␊
       function getAugmentedNamespace(n) {␊
+      	if (n.__esModule) return n;␊
       	var a = Object.defineProperty({}, '__esModule', {value: true});␊
       	Object.keys(n).forEach(function (k) {␊
       		var d = Object.getOwnPropertyDescriptor(n, k);␊
@@ -4479,6 +4490,7 @@ Generated by [AVA](https://avajs.dev).
       });␊
       ␊
       function getAugmentedNamespace(n) {␊
+      	if (n.__esModule) return n;␊
       	var a = Object.defineProperty({}, '__esModule', {value: true});␊
       	Object.keys(n).forEach(function (k) {␊
       		var d = Object.getOwnPropertyDescriptor(n, k);␊

diff --git a/packages/commonjs/test/snapshots/function.js.snap b/packages/commonjs/test/snapshots/function.js.snap
diff --git a/packages/commonjs/test/test.js b/packages/commonjs/test/test.js
@@ -575,6 +575,7 @@ var esm = /*#__PURE__*/Object.freeze({
 });
 
 function getAugmentedNamespace(n) {
+	if (n.__esModule) return n;
 	var a = Object.defineProperty({}, '__esModule', {value: true});
 	Object.keys(n).forEach(function (k) {
 		var d = Object.getOwnPropertyDescriptor(n, k);