rollup-plugin-commonjs - npm Package Compare versions

Comparing version 8.2.1 to 8.2.3

dist/rollup-plugin-commonjs.cjs.js

@@ -15,3 +15,3 @@ 'use strict';
 
-var HELPERS = "\nexport var commonjsGlobal = typeof window !== 'undefined' ? window : typeof global !== 'undefined' ? global : typeof self !== 'undefined' ? self : {};\n\nexport function commonjsRequire () {\n\tthrow new Error('Dynamic requires are not currently supported by rollup-plugin-commonjs');\n}\n\nexport function unwrapExports (x) {\n\treturn x && x.__esModule ? x['default'] : x;\n}\n\nexport function createCommonjsModule(fn, module) {\n\treturn module = { exports: {} }, fn(module, module.exports), module.exports;\n}";
+var HELPERS = "\nexport var commonjsGlobal = typeof window !== 'undefined' ? window : typeof global !== 'undefined' ? global : typeof self !== 'undefined' ? self : {};\n\nexport function commonjsRequire () {\n\tthrow new Error('Dynamic requires are not currently supported by rollup-plugin-commonjs');\n}\n\nexport function unwrapExports (x) {\n\treturn x && x.__esModule && Object.prototype.hasOwnProperty.call(x, 'default') ? x['default'] : x;\n}\n\nexport function createCommonjsModule(fn, module) {\n\treturn module = { exports: {} }, fn(module, module.exports), module.exports;\n}";
 
@@ -260,3 +260,7 @@ var PREFIX = '\0commonjs-proxy:';
 
-			if ( !name ) { name = "require$$" + (uid++); }
+			if ( !name ) {
+				do { name = "require$$" + (uid++); }
+				while ( scope.contains( name ) );
+			}
+
 			required[ source ] = { source: source, name: name, importsDefault: false };
@@ -546,6 +550,8 @@ }
 
-					namedExportDeclarations.push({
-						str: declaration,
-						name: name
-					});
+					if ( name !== 'default' ) {
+						namedExportDeclarations.push({
+							str: declaration,
+							name: name
+						});
+					}
 
@@ -552,0 +558,0 @@ defaultExportPropertyAssignments.push( (moduleName + "." + name + " = " + deconflicted + ";") );

dist/rollup-plugin-commonjs.es.js

@@ -11,3 +11,3 @@ import { statSync } from 'fs';
 
-var HELPERS = "\nexport var commonjsGlobal = typeof window !== 'undefined' ? window : typeof global !== 'undefined' ? global : typeof self !== 'undefined' ? self : {};\n\nexport function commonjsRequire () {\n\tthrow new Error('Dynamic requires are not currently supported by rollup-plugin-commonjs');\n}\n\nexport function unwrapExports (x) {\n\treturn x && x.__esModule ? x['default'] : x;\n}\n\nexport function createCommonjsModule(fn, module) {\n\treturn module = { exports: {} }, fn(module, module.exports), module.exports;\n}";
+var HELPERS = "\nexport var commonjsGlobal = typeof window !== 'undefined' ? window : typeof global !== 'undefined' ? global : typeof self !== 'undefined' ? self : {};\n\nexport function commonjsRequire () {\n\tthrow new Error('Dynamic requires are not currently supported by rollup-plugin-commonjs');\n}\n\nexport function unwrapExports (x) {\n\treturn x && x.__esModule && Object.prototype.hasOwnProperty.call(x, 'default') ? x['default'] : x;\n}\n\nexport function createCommonjsModule(fn, module) {\n\treturn module = { exports: {} }, fn(module, module.exports), module.exports;\n}";
 
@@ -256,3 +256,7 @@ var PREFIX = '\0commonjs-proxy:';
 
-			if ( !name ) { name = "require$$" + (uid++); }
+			if ( !name ) {
+				do { name = "require$$" + (uid++); }
+				while ( scope.contains( name ) );
+			}
+
 			required[ source ] = { source: source, name: name, importsDefault: false };
@@ -542,6 +546,8 @@ }
 
-					namedExportDeclarations.push({
-						str: declaration,
-						name: name
-					});
+					if ( name !== 'default' ) {
+						namedExportDeclarations.push({
+							str: declaration,
+							name: name
+						});
+					}
 
@@ -548,0 +554,0 @@ defaultExportPropertyAssignments.push( (moduleName + "." + name + " = " + deconflicted + ";") );

package.json

 {
   "name": "rollup-plugin-commonjs",
-  "version": "8.2.1",
+  "version": "8.2.3",
   "description": "Convert CommonJS modules to ES2015",
@@ -22,3 +22,3 @@ "main": "dist/rollup-plugin-commonjs.cjs.js",
   "dependencies": {
-    "acorn": "^5.1.1",
+    "acorn": "^5.1.2",
     "estree-walker": "^0.5.0",
@@ -30,12 +30,12 @@ "magic-string": "^0.22.4",
   "devDependencies": {
-    "eslint": "^4.5.0",
+    "eslint": "^4.8.0",
     "locate-character": "^2.0.1",
-    "mocha": "^3.5.0",
+    "mocha": "^4.0.1",
     "require-relative": "^0.8.7",
-    "rollup": "^0.48.2",
-    "rollup-plugin-buble": "^0.15.0",
+    "rollup": "^0.50.0",
+    "rollup-plugin-buble": "^0.16.0",
     "rollup-plugin-node-resolve": "^3.0.0",
     "shx": "^0.2.2",
-    "source-map": "^0.5.7",
-    "source-map-support": "^0.4.16"
+    "source-map": "^0.6.1",
+    "source-map-support": "^0.5.0"
   },
@@ -42,0 +42,0 @@ "repository": "rollup/rollup-plugin-commonjs",

README.md

@@ -26,5 +26,7 @@ # rollup-plugin-commonjs [![Build Status][travis-img]][travis]
 export default {
-  entry: 'main.js',
-  dest: 'bundle.js',
-  format: 'iife',
+  input: 'main.js',
+  output: {
+    file: 'bundle.js',
+    format: 'iife'
+  },
   plugins: [
@@ -31,0 +33,0 @@ nodeResolve({

src/helpers.js

@@ -11,3 +11,3 @@ export const HELPERS_ID = '\0commonjsHelpers';
 export function unwrapExports (x) {
-	return x && x.__esModule ? x['default'] : x;
+	return x && x.__esModule && Object.prototype.hasOwnProperty.call(x, 'default') ? x['default'] : x;
 }
@@ -14,0 +14,0 @@

src/transform.js

@@ -94,3 +94,7 @@ import acorn from 'acorn';
 
-			if ( !name ) name = `require$$${uid++}`;
+			if ( !name ) {
+				do name = `require$$${uid++}`;
+				while ( scope.contains( name ) );
+			}
+
 			required[ source ] = { source, name, importsDefault: false };
@@ -378,6 +382,8 @@ }
 
-					namedExportDeclarations.push({
-						str: declaration,
-						name
-					});
+					if ( name !== 'default' ) {
+						namedExportDeclarations.push({
+							str: declaration,
+							name
+						});
+					}
 
@@ -384,0 +390,0 @@ defaultExportPropertyAssignments.push( `${moduleName}.${name} = ${deconflicted};` );

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

New author

Supply chain risk

A new npm collaborator published a version of the package for the first time. New collaborators are usually benign additions to a project, but do indicate a change to the security surface area of a package.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

183467

increased by0.56%

Lines of code

1761

increased by0.97%

Number of lines in readme file

116

increased by1.75%

Worsened metrics

Number of medium supply chain risk alerts

1

increased byInfinity%

Dependency changes

• Updatedacorn@^5.1.2