feature: @putout/plugin-esm: group-imports-by-source: sort-imports-by-specifiers: couple

Author: coderaiser

packages/plugin-esm/lib/group-imports-by-source/fixture/sort-imports-by-specifiers-couple.js

@@ -0,0 +1,18 @@
+import {hello11} from '@hello11';
+import {
+    abc,
+    def,
+    hello,
+    world,
+} from '@hello';
+import {
+    abc1,
+    def1,
+    hello1,
+    world1,
+} from '@hello1';
+import {
+    Iabc,
+    Idef,
+    Ihello,
+} from '../hello.js';

packages/plugin-esm/lib/group-imports-by-source/index.js

@@ -23,54 +23,51 @@ export const fix = ({grouped}) => {
     replaceWithMultiple(first, nodes);
 };
 
-export const traverse = ({pathStore, push}) => ({
-    ImportDeclaration: pathStore,
-    Program: {
-        exit(path) {
-            const external = [];
-            const internal = [];
-            const builtin = [];
-            const hashed = [];
-            const all = pathStore().filter(isImportDeclaration);
+export const traverse = ({push}) => ({
+    Program(path) {
+        const external = [];
+        const internal = [];
+        const builtin = [];
+        const hashed = [];
+        const all = path.get('body').filter(isImportDeclaration);
+        
+        if (!all.length)
+            return;
+        
+        for (const current of all) {
+            const {value} = current.node.source;
 
-            if (!all.length)
-                return;
-            
-            for (const current of all) {
-                const {value} = current.node.source;
-                
-                if (value.startsWith('.')) {
-                    internal.push(current);
-                    continue;
-                }
-                
-                if (value.startsWith('node:')) {
-                    builtin.push(current);
-                    continue;
-                }
-                
-                if (value.startsWith('#')) {
-                    hashed.push(current);
-                    continue;
-                }
-                
-                external.push(current);
+            if (value.startsWith('.')) {
+                internal.push(current);
+                continue;
             }
 
-            const grouped = [
-                ...builtin,
-                ...external,
-                ...hashed,
-                ...internal,
-            ];
+            if (value.startsWith('node:')) {
+                builtin.push(current);
+                continue;
+            }
 
-            if (isDeepStrictEqual(all, grouped))
-                return;
+            if (value.startsWith('#')) {
+                hashed.push(current);
+                continue;
+            }
 
-            push({
-                path,
-                grouped,
-            });
-        },
+            external.push(current);
+        }
+        
+        const grouped = [
+            ...builtin,
+            ...external,
+            ...hashed,
+            ...internal,
+        ];
+        
+        if (isDeepStrictEqual(all, grouped))
+            return;
+        
+        push({
+            path,
+            grouped,
+        });
     },
 });

packages/plugin-esm/lib/group-imports-by-source/index.spec.js

@@ -49,3 +49,10 @@ test('putout: group-imports-by-source: no report after transform: sort-imports-b
     });
     t.end();
 });
+
+test('putout: group-imports-by-source: no report: sort-imports-by-specifiers-couple', (t) => {
+    t.noReport('sort-imports-by-specifiers-couple', [
+        ['sort-imports-by-specifiers', sortImportsBySpecifiers],
+    ]);
+    t.end();
+});

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

@@ -4,18 +4,18 @@ import {
     unlink,
     mkdir,
 } from 'node:path';
+import a1 from 'a1';
+import {test} from 'supertape';
 import {
     a,
     b,
     c,
     d,
 } from 'd';
-import {test} from 'supertape';
 import {
     hello,
     world,
     something,
     else1,
 } from 'hello';
-import a1 from 'a1';
 import tryCatch from './index.js';