eslint-plugin-simple-import-sort - npm Package Compare versions

Comparing version 8.0.0 to 10.0.0

exports.js

@@ -17,17 +17,35 @@ "use strict";
   },
-  create: (context) => ({
-    Program: (programNode) => {
-      const sourceCode = context.getSourceCode();
-      for (const chunk of shared.extractChunks(programNode, (node, lastNode) =>
-        isPartOfChunk(node, lastNode, sourceCode)
-      )) {
-        maybeReportChunkSorting(chunk, context);
+  create: (context) => {
+    const parents = new Set();
+
+    const addParent = (node) => {
+      if (isExportFrom(node)) {
+        parents.add(node.parent);
       }
-    },
-    ExportNamedDeclaration: (node) => {
-      if (node.source == null && node.declaration == null) {
-        maybeReportExportSpecifierSorting(node, context);
-      }
-    },
-  }),
+    };
+
+    return {
+      ExportNamedDeclaration: (node) => {
+        if (node.source == null && node.declaration == null) {
+          maybeReportExportSpecifierSorting(node, context);
+        } else {
+          addParent(node);
+        }
+      },
+
+      ExportAllDeclaration: addParent,
+
+      "Program:exit": () => {
+        const sourceCode = context.getSourceCode();
+        for (const parent of parents) {
+          for (const chunk of shared.extractChunks(parent, (node, lastNode) =>
+            isPartOfChunk(node, lastNode, sourceCode)
+          )) {
+            maybeReportChunkSorting(chunk, context);
+          }
+        }
+        parents.clear();
+      },
+    };
+  },
 };
@@ -34,0 +52,0 @@

imports.js

@@ -51,12 +51,23 @@ "use strict";
     const { groups: rawGroups = defaultGroups } = context.options[0] || {};
+
     const outerGroups = rawGroups.map((groups) =>
       groups.map((item) => RegExp(item, "u"))
     );
+
+    const parents = new Set();
+
     return {
-      Program: (programNode) => {
-        for (const chunk of shared.extractChunks(programNode, (node) =>
-          isImport(node) ? "PartOfChunk" : "NotPartOfChunk"
-        )) {
-          maybeReportChunkSorting(chunk, context, outerGroups);
+      ImportDeclaration: (node) => {
+        parents.add(node.parent);
+      },
+
+      "Program:exit": () => {
+        for (const parent of parents) {
+          for (const chunk of shared.extractChunks(parent, (node) =>
+            isImport(node) ? "PartOfChunk" : "NotPartOfChunk"
+          )) {
+            maybeReportChunkSorting(chunk, context, outerGroups);
+          }
         }
+        parents.clear();
       },
@@ -63,0 +74,0 @@ };

package.json

 {
   "name": "eslint-plugin-simple-import-sort",
-  "version": "8.0.0",
+  "version": "10.0.0",
   "license": "MIT",
@@ -5,0 +5,0 @@ "author": "Simon Lydell",

shared.js

@@ -5,3 +5,3 @@ "use strict";
 // and whitespace between.
-function extractChunks(programNode, isPartOfChunk) {
+function extractChunks(parentNode, isPartOfChunk) {
   const chunks = [];
@@ -11,3 +11,3 @@ let chunk = [];
 
-  for (const node of programNode.body) {
+  for (const node of parentNode.body) {
     const result = isPartOfChunk(node, lastNode);
@@ -735,8 +735,3 @@ switch (result) {
     (itemA, itemB) =>
-      // Put type imports/exports before regular ones.
-      compare(
-        getImportExportKind(itemA.node),
-        getImportExportKind(itemB.node)
-      ) ||
-      // Then compare by imported or exported name (external interface name).
+      // Compare by imported or exported name (external interface name).
       // import { a as b } from "a"
@@ -756,2 +751,7 @@ //          ^
       compare(itemA.node.local.name, itemB.node.local.name) ||
+      // Then put type specifiers before regular ones.
+      compare(
+        getImportExportKind(itemA.node),
+        getImportExportKind(itemB.node)
+      ) ||
       // Keep the original order if the names are the same. It’s not worth
@@ -758,0 +758,0 @@ // trying to compare anything else, `import {a, a} from "mod"` is a syntax

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

37285

increased by1.48%

Lines of code

1005

increased by2.13%

No dependency changes