@putout/plugin-putout 14.5.0 → 14.7.0

package/README.md

@@ -26,6 +26,7 @@ npm i @putout/plugin-putout -D
         "putout/apply-namaspace-specifier": "on",
         "putout/add-args": "on",
         "putout/add-push": "on",
+        "putout/add-index-to-import": "on",
         "putout/check-match": "on",
         "putout/check-replace-code": "on",
         "putout/convert-putout-test-to-create-test": "on",
@@ -661,6 +662,29 @@ module.exports.traverse = ({push}) => ({
 });
 ```
 
+## add-index-to-import
+
+ESM doesn't add `index.js`, so it can be left after [`@putout/plugin-convert-esm-to-commonjs`](https://github.com/coderaiser/putout/blob/master/packages/plugin-convert-esm-to-commonjs#readme).
+Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/b7c489710767efee95ecf3dd16e232a2/9f974f0a345ef4d0cb39b011097dff82e6c32b75).
+
+### ❌ Example of incorrect code
+
+```js
+import insertRust from './insert-rust.js';
+import addAction from './add-action.js';
+
+export const rules = {};
+```
+
+### ✅ Example of correct code
+
+```js
+import insertRust from './insert-rust/index.js';
+import addAction from './add-action/index.js';
+
+export const rules = {};
+```
+
 ## convert-add-argument-to-add-args
 
 ### ❌ Example of incorrect code

package/lib/add-index-to-import/index.js

@@ -0,0 +1,46 @@
+'use strict';
+
+const {operator} = require('putout');
+const {
+    traverse,
+    setLiteralValue,
+} = operator;
+
+module.exports.report = () => `Add 'index.js' to nested import`;
+
+module.exports.fix = (path) => {
+    const {source} = path.node;
+    const {value} = source;
+    
+    if (value.endsWith('js')) {
+        setLiteralValue(source, value.replace('.js', '/index.js'));
+        return;
+    }
+    
+    setLiteralValue(source, `${value}/index.js`);
+};
+
+module.exports.traverse = ({push, listStore}) => ({
+    'export const rules = __object': listStore,
+    'Program': {
+        exit: (path) => {
+            const rules = listStore();
+            
+            if (!rules.length)
+                return;
+            
+            traverse(path, {
+                ImportDeclaration: createImportVisitor(push),
+            });
+        },
+    },
+});
+
+const createImportVisitor = (push) => (path) => {
+    const {value} = path.node.source;
+    
+    if (value.endsWith('index.js'))
+        return;
+    
+    push(path);
+};

package/lib/apply-namaspace-specifier/index.js

@@ -1,15 +1,19 @@
 'use strict';
 
+const {parseImportSpecifiers} = require('parse-import-specifiers');
+const noop = () => {};
+
 module.exports.report = () => `Use 'import * as plugin' instead of 'import plugin'`;
 
 module.exports.fix = ({first}) => {
     first.node.type = 'ImportNamespaceSpecifier';
 };
 
-module.exports.traverse = ({push, listStore}) => ({
+module.exports.traverse = ({push, listStore, pathStore}) => ({
     'export const rules = __object': listStore,
     'import {createTest} from "@putout/test"': listStore,
     ...createImportVisitor({
+        pathStore,
         push,
         names: [
             './index.js',
@@ -20,7 +24,10 @@ module.exports.traverse = ({push, listStore}) => ({
         exit(path) {
             const rules = listStore();
             
-            if (!rules.length)
+            if (rules.length !== 1)
+                return;
+            
+            if (pathStore().length > 2)
                 return;
             
             path.traverse(createImportVisitor({
@@ -31,10 +38,13 @@ module.exports.traverse = ({push, listStore}) => ({
     },
 });
 
-const createImportVisitor = ({push, names}) => ({
+const createImportVisitor = ({push, names, pathStore = noop}) => ({
     ImportDeclaration(path) {
+        pathStore(path);
+        
         const {value} = path.node.source;
-        const first = path.get('specifiers.0');
+        const specifiers = path.get('specifiers');
+        const [first] = specifiers;
         
         if (!first)
             return;
@@ -45,6 +55,12 @@ const createImportVisitor = ({push, names}) => ({
         if (first.isImportSpecifier())
             return;
         
+        const {defaults, imports} = parseImportSpecifiers(specifiers);
+        
+        if (defaults.length && imports.length) {
+            return;
+        }
+        
         for (const name of names) {
             if (value === name || name === 'any') {
                 push({

package/lib/index.js

@@ -40,6 +40,7 @@ const includer = require('./includer');
 const createTest = require('./create-test');
 const applyNamaspaceSpecifier = require('./apply-namaspace-specifier');
 const convertGetRuleToRequire = require('./convert-get-rule-to-require');
+const addIndexToImport = require('./add-index-to-import');
 
 module.exports.rules = {
     'apply-processors-destructuring': applyProcessorsDestructuring,
@@ -82,4 +83,5 @@ module.exports.rules = {
     'create-test': createTest,
     'apply-namaspace-specifier': applyNamaspaceSpecifier,
     'convert-get-rule-to-require': convertGetRuleToRequire,
+    'add-index-to-import': addIndexToImport,
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-putout",
-  "version": "14.5.0",
+  "version": "14.7.0",
   "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin helps with plugins development",
@@ -27,6 +27,7 @@
   "dependencies": {
     "fullstore": "^3.0.0",
     "just-camel-case": "^6.0.1",
+    "parse-import-specifiers": "^1.0.2",
     "try-catch": "^3.0.0"
   },
   "keywords": [