fix: @putout/plugin-esm: sort-imports-by-specifiers: report after transform

Author: coderaiser

packages/plugin-esm/lib/sort-imports-by-specifiers/.putoutcache.json

@@ -0,0 +1,16 @@
+import {e1} from '../../../../../../hello3';
+import {d1} from '../../../../Hello4';
+import {c1} from '../../../../Hello5';
+import {
+    a2,
+    b2,
+    c2,
+    d2,
+} from '../../../../../../hello';
+import {b1} from '../../../../Hello6';
+import {a1} from '../../../../abc/hello';
+import e from '../../../Hello/Hello';
+import d from '../../../Hello/Hello1';
+import b from '../../../Hello/Hello5';
+import a from '../../../Hello/HelloA';
+import f from '../../Hello/Hello1';

packages/plugin-esm/lib/sort-imports-by-specifiers/fixture/lots-fix.js

@@ -0,0 +1,17 @@
+import {
+    a2,
+    b2,
+    c2,
+    d2,
+} from '../../../../../../hello';
+
+import { e1 } from '../../../../../../hello3';
+import { d1 } from '../../../../Hello4';
+import { c1 } from '../../../../Hello5';
+import { b1 } from '../../../../Hello6';
+import { a1 } from '../../../../abc/hello';
+import e from '../../../Hello/Hello';
+import d from '../../../Hello/Hello1';
+import b from '../../../Hello/Hello5';
+import a from '../../../Hello/HelloA';
+import f from '../../Hello/Hello1';

packages/plugin-esm/lib/sort-imports-by-specifiers/fixture/lots.js

@@ -1,7 +1,8 @@
 import {parseImportSpecifiers} from 'parse-import-specifiers';
-import {operator} from 'putout';
+import {operator, types} from 'putout';
 
 const {insertBefore, remove} = operator;
+const {isImportDeclaration} = types;
 
 export const report = () => `Sort imports by specifiers count`;
 
@@ -12,44 +13,52 @@ export const fix = ({path, nextPath}) => {
 };
 
 export const traverse = ({push}) => ({
-    ImportDeclaration(path) {
-        const {node} = path;
-        const {source, specifiers} = node;
-        const {imports} = parseImportSpecifiers(specifiers);
+    Program(mainPath) {
+        const paths = mainPath.get('body').filter(isImportDeclaration);
 
-        if (imports.length < 4)
-            return;
-        
-        const nextPath = path.getNextSibling();
-        
-        if (!nextPath.isImportDeclaration())
-            return;
-        
-        if (nextPath.node.specifiers.length !== 1)
-            return;
-        
-        const first = source.value;
-        const second = nextPath.node.source.value;
-        
-        const is = isExcluded(first, second, {
-            direct: [
-                ['node:', 'node:'],
-                ['#', '#'],
-            ],
-            reversed: [
-                ['./', './'],
-                ['../', '../'],
-                ['#', '#'],
-            ],
-        });
-        
-        if (is)
-            return;
-        
-        push({
-            path,
-            nextPath,
-        });
+        for (const path of paths) {
+            const {node} = path;
+            
+            if (!node)
+                continue;
+            
+            const {source, specifiers} = node;
+            const {imports} = parseImportSpecifiers(specifiers);
+            
+            if (imports.length < 4)
+                continue;
+            
+            const nextPath = path.getNextSibling();
+            
+            if (!nextPath.isImportDeclaration())
+                continue;
+            
+            if (nextPath.node.specifiers.length !== 1)
+                continue;
+            
+            const first = source.value;
+            const second = nextPath.node.source.value;
+            
+            const is = isExcluded(first, second, {
+                direct: [
+                    ['node:', 'node:'],
+                    ['#', '#'],
+                ],
+                reversed: [
+                    ['./', './'],
+                    ['../', '../'],
+                    ['#', '#'],
+                ],
+            });
+            
+            if (is)
+                continue;
+            
+            push({
+                path,
+                nextPath,
+            });
+        }
     },
 });
 

packages/plugin-esm/lib/sort-imports-by-specifiers/index.js

@@ -46,3 +46,8 @@ test('putout: sort-imports-by-specifiers: external-hashed', (t) => {
     t.noReport('external-hashed');
     t.end();
 });
+
+test('putout: sort-imports-by-specifiers: lots', (t) => {
+    t.transform('lots');
+    t.end();
+});