cjs-es - npm Package Compare versions

Comparing version 0.8.0 to 0.8.1

lib/import-writer.js

@@ -37,6 +37,6 @@ const {pathToName} = require("./util");
         context.finalImportType[id] = preferDefault ? "default" : "named";
-        if (nodes.length === 1 && nodes[0].declarator &&
-          (!preferDefault || nodes[0].declarator.isSingleBinding)
+        if (nodes.every(n => n.declarator) &&
+          (!preferDefault || nodes.every(n => n.declarator.isSingleBinding))
         ) {
-          return writeDeclaredRequires(nodes[0], preferDefault);
+          return writeDeclaredRequires(nodes, preferDefault);
         }
@@ -47,3 +47,9 @@ return writeHoistRequires(id, nodes, preferDefault);
   
-  function writeDeclaredRequires(node, preferDefault) {
+  function writeDeclaredRequires(nodes, preferDefault) {
+    for (const node of nodes) {
+      writeDeclareImport(node, preferDefault);
+    }
+  }
+    
+  function writeDeclareImport(node, preferDefault) {
     if (node.declarator.isSingleBinding) {
@@ -50,0 +56,0 @@ // const foo = require("foo")

package.json

 {
   "name": "cjs-es",
-  "version": "0.8.0",
+  "version": "0.8.1",
   "description": "Transform CommonJS module into ES module.",
@@ -5,0 +5,0 @@ "keywords": [

README.md

@@ -316,2 +316,6 @@ cjs-es
 
+* 0.8.1 (Jun 18, 2019)
+
+  - Fix: don't hoist duplicated imports.
+
 * 0.8.0 (Jun 13, 2019)
@@ -318,0 +322,0 @@

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

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

54809

increased by0.4%

Lines of code

1355

increased by0.37%

Number of lines in readme file

448

increased by0.9%

No dependency changes