feature: @putout/plugin-esm: group-imports-by-source: add support of hash

Author: coderaiser

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

@@ -0,0 +1,11 @@
+import {test} from 'supertape';
+import {createFilesystem} from '#filesystem';
+import {traverse} from '#fatlint';
+import {
+    parse,
+    print,
+    operator,
+    types,
+} from 'putout';
+import {createDisk} from '../fatdisk.js';
+import {readAST, writeAST} from '../fsast.js';

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

@@ -0,0 +1,11 @@
+import {createFilesystem} from '#filesystem';
+import {test} from 'supertape';
+import {traverse} from '#fatlint';
+import {createDisk} from '../fatdisk.js';
+import {
+    parse,
+    print,
+    operator,
+    types,
+} from 'putout';
+import {readAST, writeAST} from '../fsast.js';

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

@@ -31,6 +31,7 @@ module.exports.traverse = ({pathStore, push}) => ({
             const external = [];
             const internal = [];
             const builtin = [];
+            const hashed = [];
             const all = pathStore().filter(isImportDeclaration);
 
             if (!all.length)
@@ -49,12 +50,18 @@ module.exports.traverse = ({pathStore, push}) => ({
                     continue;
                 }
 
+                if (value.startsWith('#')) {
+                    hashed.push(current);
+                    continue;
+                }
+                
                 external.push(current);
             }
 
             const grouped = [
                 ...builtin,
                 ...external,
+                ...hashed,
                 ...internal,
             ];
 

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

@@ -5,6 +5,7 @@ const {createTest} = require('@putout/test');
 const nodejs = require('@putout/plugin-nodejs');
 const plugin = require('..');
 
+const sortImportsBySpecifiers = require('../sort-imports-by-specifiers');
 const convertEsmToCommonJS = nodejs.rules['convert-esm-to-commonjs'];
 
 const test = createTest(__dirname, {
@@ -39,3 +40,10 @@ test('putout: group-imports-by-source: transform: convert-esm-to-commonjs', (t)
     });
     t.end();
 });
+
+test('putout: group-imports-by-source: transform: sort-imports-by-specifiers', (t) => {
+    t.transform('sort-imports-by-specifiers', {
+        sortImportsBySpecifiers,
+    });
+    t.end();
+});

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

@@ -0,0 +1,9 @@
+[
+    ["1"], {
+        "key": "2",
+        "value": "3"
+    },     "index.js",{
+        "size": 1745,
+        "mtime": 1739812065844
+    }
+]

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

@@ -0,0 +1,7 @@
+import {
+    parse,
+    print,
+    operator,
+    types,
+} from 'putout';
+import {createDisk} from '../fatdisk.js';

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

@@ -29,13 +29,18 @@ module.exports.traverse = ({push}) => ({
         if (nextPath.node.specifiers.length !== 1)
             return;
 
-        if (!source.value.startsWith('./') && nextPath.node.source.value.startsWith('./'))
-            return;
-        
-        if (source.value.startsWith('node:') && !nextPath.node.source.value.startsWith('node:'))
-            return;
+        const is = isExcluded(source, nextPath, {
+            direct: [
+                ['node:', 'node:'],
+                ['#', '#'],
+            ],
+            reversed: [
+                ['./', './'],
+                ['../', '../'],
+            ],
+        });
 
-        if (source.value.startsWith('#') && !nextPath.node.source.value.startsWith('#'))
+        if (is)
             return;
 
         push({
@@ -44,3 +49,17 @@ module.exports.traverse = ({push}) => ({
         });
     },
 });
+
+function isExcluded(source, nextPath, {direct, reversed}) {
+    for (const [current, next] of direct) {
+        if (source.value.startsWith(current) && !nextPath.node.source.value.startsWith(next))
+            return true;
+    }
+    
+    for (const [current, next] of reversed) {
+        if (!source.value.startsWith(current) && nextPath.node.source.value.startsWith(next))
+            return true;
+    }
+    
+    return false;
+}

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

@@ -38,3 +38,8 @@ test('putout: sort-imports-by-specifiers: three', (t) => {
     t.noTransform('three');
     t.end();
 });
+
+test('putout: sort-imports-by-specifiers: external-internal', (t) => {
+    t.noTransform('external-internal');
+    t.end();
+});