@putout/plugin-esm 10.0.2 → 10.3.0

package/README.md

@@ -22,6 +22,7 @@ npm i putout @putout/plugin-esm -D
 - ✅ [apply-export-from](#apply-export-from);
 - ✅ [apply-import-import](#apply-import-attributes);
 - ✅ [convert-assert-to-with](#convert-assert-to-with);
+- ✅ [convert-const-to-import](#convert-const-to-import);
 - ✅ [declare-imports-first](#declare-imports-first);
 - ✅ [group-imports-by-source](#group-imports-by-source);
 - ✅ [merge-duplicate-imports](#merge-duplicate-imports);
@@ -51,6 +52,8 @@ npm i putout @putout/plugin-esm -D
         "esm/apply-export-from": "on",
         "esm/apply-import-attirbutes": "on",
         "esm/declare-imports-first": "on",
+        "esm/convert-assert-to-with": "on",
+        "esm/convert-const-to-import": "on",
         "esm/group-imports-by-source": "on",
         "esm/merge-duplicate-imports": "on",
         "esm/merge-declaration-with-export": "on",
@@ -61,9 +64,9 @@ npm i putout @putout/plugin-esm -D
             "ignore": []
         }],
         "esm/sort-imports-by-specifiers": "on",
-        "esm/apply-js-imported-file": "off",
         "esm/resolve-imported-file": "off",
         "esm/shorten-imported-file": "off",
+        "esm/apply-js-imported-file": "off",
         "esm/apply-name-to-imported-file": "off",
         "esm/apply-namespace-to-imported-file": "off",
         "esm/apply-privately-imported-file": "off",
@@ -433,6 +436,22 @@ import('foo.json', {
 });
 ```
 
+### convert-const-to-import
+
+Check out in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/2682a3899fe6916e22073abf0b56b13c/f9d7ca3ddb65331bb836af78825d80fd258b3f94).
+
+#### ❌ Example of incorrect code
+
+```
+const {Server} from 'socket.io';
+```
+
+#### ✅ Example of correct code
+
+```js
+import {Server} from 'socket.io';
+```
+
 ## File Rules
 
 ### apply-namespace-to-imported-file

package/lib/apply-privately-imported-file/index.js

@@ -3,6 +3,7 @@ import {
     parse,
     print,
     transform,
+    findPlaces,
     operator,
 } from 'putout';
 import * as getImports from '#get-imports';
@@ -20,6 +21,7 @@ const getMessage = (a) => a.message;
 export const report = (file, {from, to, filename}) => {
     return `Apply private import: '${from}' -> '${to}' in '${filename}'`;
 };
+
 export const fix = (file, {ast, from, to}) => {
     transform(ast, {
         rules: {
@@ -48,10 +50,6 @@ export const scan = (rootPath, {push, trackFile}) => {
     
     for (const file of trackFile(rootPath, mask)) {
         const content = readFileContent(file);
-        
-        if (!content.includes('import'))
-            continue;
-        
         const privateImports = getPrivateImports(file);
         
         if (!privateImports.size)
@@ -59,7 +57,7 @@ export const scan = (rootPath, {push, trackFile}) => {
         
         const ast = parse(content);
         
-        const places = transform(ast, {
+        const places = findPlaces(ast, {
             plugins: [
                 ['get-imports', getImports],
             ],

package/lib/convert-const-to-import/index.js

@@ -0,0 +1,68 @@
+import {
+    template,
+    types,
+    operator,
+} from 'putout';
+
+const {remove, replaceWith} = operator;
+const {
+    importSpecifier,
+    isObjectPattern,
+    isIdentifier,
+} = types;
+
+export const report = () => `Use 'import' instead of 'const'`;
+
+export const fix = ({path, second, third}) => {
+    const {value} = third.node.expression;
+    const {id} = path.node.declarations[0];
+    
+    if (isIdentifier(id)) {
+        const nodeImport = template.ast(`import ${id.name} from '${value}'`);
+        replaceWith(path, nodeImport);
+        
+        return;
+    }
+    
+    if (isObjectPattern(id)) {
+        const nodeImport = template.ast.fresh(`import {} from '${value}'`);
+        
+        for (const {key, value} of id.properties) {
+            nodeImport.specifiers.push(importSpecifier(key, value));
+        }
+        
+        replaceWith(path, nodeImport);
+    }
+    
+    remove(second);
+    remove(third);
+};
+
+export const traverse = ({push}) => ({
+    VariableDeclaration(path) {
+        if (path.node.declarations[0].init)
+            return;
+        
+        const second = path.getNextSibling();
+        
+        if (!second.isExpressionStatement())
+            return;
+        
+        if (!second.get('expression').isIdentifier({name: 'from'}))
+            return;
+        
+        const third = second.getNextSibling();
+        
+        if (!third.isExpressionStatement())
+            return;
+        
+        if (!third.get('expression').isStringLiteral())
+            return;
+        
+        push({
+            path,
+            second,
+            third,
+        });
+    },
+});

package/lib/index.js

@@ -1,3 +1,4 @@
+import * as convertConstToImport from './convert-const-to-import/index.js';
 import * as applyImportAttributes from './apply-import-attributes/index.js';
 import * as applyDefaultImport from './apply-default-import/index.js';
 import * as applyExportFrom from './apply-export-from/index.js';
@@ -41,4 +42,5 @@ export const rules = {
     'apply-js-imported-file': ['off', applyJsImportedFile],
     'apply-default-import': applyDefaultImport,
     'apply-import-attributes': applyImportAttributes,
+    'convert-const-to-import': convertConstToImport,
 };

package/lib/resolve-imported-file/change-imports/index.js

@@ -11,6 +11,8 @@ export const replace = ({options}) => {
     if (!to.endsWith('json'))
         return {
             [`import __imports from '${from}'`]: `import __imports from '${to}'`,
+            [`export __exports from '${from}'`]: `export __exports from '${to}'`,
+            [`export * from '${from}'`]: `export * from '${to}'`,
             [`import('${from}')`]: `import('${to}')`,
         };
     

package/lib/resolve-imported-file/get-imports/index.js

@@ -5,6 +5,8 @@ export const fix = () => {};
 export const include = () => [
     'ImportDeclaration',
     'import("__a")',
+    'export * from "__a"',
+    'export __exports from "__a"',
 ];
 
 export const filter = (path) => {

package/lib/resolve-imported-file/index.js

@@ -4,6 +4,7 @@ import {
     print,
     transform,
     operator,
+    findPlaces,
 } from 'putout';
 import * as changeImports from '#change-imports';
 import * as getImports from './get-imports/index.js';
@@ -47,13 +48,9 @@ export const scan = (rootPath, {push, trackFile, crawlFile}) => {
     
     for (const file of trackFile(rootPath, mask)) {
         const content = readFileContent(file);
-        
-        if (!content.includes('import'))
-            continue;
-        
         const ast = parse(content);
         
-        const places = transform(ast, {
+        const places = findPlaces(ast, {
             plugins: [
                 ['get-imports-source', getImports],
             ],

package/lib/shorten-imported-file/get-imports/index.js

@@ -5,6 +5,8 @@ export const fix = () => {};
 export const include = () => [
     'ImportDeclaration',
     'import("__a")',
+    'export __exports from "__a"',
+    'export * from "__a"',
 ];
 
 export const filter = (path) => {

package/lib/shorten-imported-file/index.js

@@ -51,10 +51,6 @@ export const scan = (rootPath, {push, trackFile}) => {
     
     for (const file of trackFile(rootPath, mask)) {
         const content = readFileContent(file);
-        
-        if (!content.includes('import'))
-            continue;
-        
         const [error, ast] = tryCatch(parse, content);
         
         if (error)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-esm",
-  "version": "10.0.2",
+  "version": "10.3.0",
   "type": "module",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin improves ability to transform ESM code",