@putout/plugin-putout 12.1.0 → 12.1.1

package/README.md

@@ -87,6 +87,7 @@ export const fix = (path) => {
 
 ```js
 import {operator} from 'putout';
+
 const {remove} = operator;
 
 export const fix = (path) => {
@@ -153,6 +154,7 @@ const test = require('@putout/test')({
 
 ```js
 const {createTest} = require('@putout/test');
+
 const test = createTest({
     'remove-debugger': plugin,
 });
@@ -166,7 +168,9 @@ Add properties to `createTest` options, here is exmample of `.putout.json`:
 {
     "rules": {
         "putout/create-test": ["on", {
-            "add": ["printer", "putout"]
+            "add": [
+                ["printer", "putout"]
+            ]
         }]
     }
 }
@@ -285,7 +289,9 @@ module.exports.fix = (path) => {
 
 ```js
 module.exports.fix = (path) => {
-    path.replaceWithMultiple([Identifier('hello')]);
+    path.replaceWithMultiple([
+        Identifier('hello'),
+    ]);
 };
 ```
 
@@ -295,7 +301,9 @@ module.exports.fix = (path) => {
 const {replaceWithMultiple} = require('putout').operator;
 
 module.exports.fix = (path) => {
-    replaceWithMultiple(path, [Identifier('hello')]);
+    replaceWithMultiple(path, [
+        Identifier('hello'),
+    ]);
 };
 ```
 
@@ -365,10 +373,8 @@ const {
 
 ```js
 module.exports.replace = () => ({
-    'const __a = __b': ({}) => {
-    },
-    'const __c = __d': ({}, path) => {
-    },
+    'const __a = __b': ({}) => {},
+    'const __c = __d': ({}, path) => {},
 });
 ```
 
@@ -376,10 +382,8 @@ module.exports.replace = () => ({
 
 ```js
 module.exports.replace = () => ({
-    'const __a = __b': (vars) => {
-    },
-    'const __c = __d': (vars, path) => {
-    },
+    'const __a = __b': (vars) => {},
+    'const __c = __d': (vars, path) => {},
 });
 ```
 
@@ -474,8 +478,7 @@ module.exports.merge = (processedSource, list) => '';
 
 ```js
 module.exports.match = () => ({
-    'module.exports.traverse = __a'({}, path) {
-    },
+    'module.exports.traverse = __a'({}, path) {},
 });
 ```
 
@@ -483,8 +486,7 @@ module.exports.match = () => ({
 
 ```js
 module.exports.match = () => ({
-    'module.exports.traverse = __a': ({}, path) => {
-    },
+    'module.exports.traverse = __a': ({}, path) => {},
 });
 ```
 
@@ -518,7 +520,10 @@ isIdentifier(a);
 ### ✅ Example of correct code
 
 ```js
-const {operator, types} = require('putout');
+const {
+    operator,
+    types,
+} = require('putout');
 const {compare} = operator;
 const {isIdentifier} = types;
 
@@ -598,6 +603,7 @@ module.exports = addArgs({
 import {createTest} from '@putout/test';
 import plugin from '@putout/plugin-debugger';
 import {createSimport} from 'simport';
+
 const {__dirname} = createSimport(import.meta.url);
 
 const test = createTest(__dirname, {
@@ -675,6 +681,7 @@ test('remove debugger: report', (t) => {
 
 ```js
 const removeDebugger = require('..');
+
 const test = require('@putout/test')(__dirname, {
     'remove-debugger': removeDebugger,
 });
@@ -683,6 +690,7 @@ test('remove debugger: report', (t) => {
     const test = require('@putout/test')(__dirname, {
         'remove-debugger': removeDebugger,
     });
+    
     t.end();
 });
 ```
@@ -694,25 +702,37 @@ test('remove debugger: report', (t) => {
 ```js
 module.exports.include = () => 'cons __a = __b';
 module.exports.exclude = () => 'var __a = __b';
-
 module.exports.include = 'cons __a = __b';
 module.exports.exclude = 'var __a = __b';
-
-module.exports.include = ['cons __a = __b'];
-module.exports.exclude = ['var __a = __b'];
+module.exports.include = [
+    'cons __a = __b',
+];
+module.exports.exclude = [
+    'var __a = __b',
+];
 ```
 
 ### ✅ Example of correct code
 
 ```js
-module.exports.include = () => ['cons __a = __b'];
-module.exports.exclude = () => ['var __a = __b'];
-
-module.exports.include = () => ['cons __a = __b'];
-module.exports.exclude = () => ['var __a = __b'];
-
-module.exports.include = () => ['cons __a = __b'];
-module.exports.exclude = () => ['var __a = __b'];
+module.exports.include = () => [
+    'cons __a = __b',
+];
+module.exports.exclude = () => [
+    'var __a = __b',
+];
+module.exports.include = () => [
+    'cons __a = __b',
+];
+module.exports.exclude = () => [
+    'var __a = __b',
+];
+module.exports.include = () => [
+    'cons __a = __b',
+];
+module.exports.exclude = () => [
+    'var __a = __b',
+];
 ```
 
 ## replace-test-message

package/lib/add-args/index.js

@@ -8,21 +8,15 @@ module.exports = addArgs({
         'test("__a", async (__args) => __body)',
         'test.skip("__a", async (__args) => __body)',
         'test.only("__a", async (__args) => __body)',
-    ],
-    ],
-    
+    ]],
     process: ['{process}', [
         'test("__a", async (__args) => __body)',
         'test.skip("__a", async (__args) => __body)',
         'test.only("__a", async (__args) => __body)',
-    ],
-    ],
-    
+    ]],
     noProcess: ['{noProcess}', [
         'test("__a", async (__args) => __body)',
         'test.skip("__a", async (__args) => __body)',
         'test.only("__a", async (__args) => __body)',
-    ],
-    ],
+    ]],
 });
-

package/lib/apply-async-formatter/index.js

@@ -44,4 +44,3 @@ const create = (name) => (vars, path) => {
     
     return `await ${name}(__args)`;
 };
-

package/lib/apply-create-test/index.js

@@ -5,4 +5,3 @@ module.exports.report = () => `Apply 'createTest'`;
 module.exports.replace = () => ({
     'require("@putout/test")(__dirname, __a)': `createTest(__dirname, __a)`,
 });
-

package/lib/apply-declare/index.js

@@ -5,4 +5,3 @@ module.exports.report = () => `Use 'Declarator' instead of 'operator.declare()'`
 module.exports.replace = () => ({
     'module.exports = declare(__a)': 'module.exports.declare = () => __a',
 });
-

package/lib/apply-processors-destructuring/index.js

@@ -6,7 +6,6 @@ module.exports.replace = () => ({
     'async (t) => {await t.process(__args)}': 'async ({process}) => {await process(__args)}',
     'async (t) => {await t.noProcess(__args)}': 'async ({noProcess}) => {await noProcess(__args)}',
     'async (t) => {await t.comparePlaces(__args)}': 'async ({comparePlaces}) => {await comparePlaces(__args)}',
-    
     'async (t) => {await t.process(__args); t.end();}': 'async ({process}) => {await process(__args)}',
     'async (t) => {await t.noProcess(__args); t.end();}': 'async ({noProcess}) => {await noProcess(__args)}',
     'async (t) => {await t.comparePlaces(__args); t.end();}': 'async ({comparePlaces}) => {await comparePlaces(__args)}',

package/lib/apply-remove/index.js

@@ -5,4 +5,3 @@ module.exports.report = () => `Use 'remove(path)' instead of 'path.remove()'`;
 module.exports.replace = () => ({
     'path.remove()': 'remove(path)',
 });
-

package/lib/check-replace-code/generate-code.js

@@ -2,11 +2,15 @@
 
 const putout = require('putout');
 const tryCatch = require('try-catch');
-
 const noop = () => {};
 
-const {types, operator} = putout;
+const {
+    types,
+    operator,
+} = putout;
+
 const {replaceWith} = operator;
+
 const {
     ArrayPattern,
     ObjectPattern,
@@ -16,6 +20,7 @@ const {
 
 module.exports = (rootPath, key) => {
     const getVar = createVarStore(rootPath);
+    
     const [transformError, result] = tryCatch(putout, key, {
         fix: true,
         isTS: true,
@@ -75,7 +80,10 @@ module.exports = (rootPath, key) => {
         ],
     });
     
-    return [transformError, result?.code];
+    return [
+        transformError,
+        result?.code,
+    ];
 };
 
 function createVarStore(path) {
@@ -102,4 +110,3 @@ function objectify(path) {
     if (isAssign && parentPath.get('right') === path)
         return replaceWith(path, ObjectExpression([]));
 }
-

package/lib/check-replace-code/index.js

@@ -2,11 +2,8 @@
 
 const putout = require('putout');
 const tryCatch = require('try-catch');
-
 const generateCode = require('./generate-code');
-
 const noop = () => {};
-
 const {operator} = putout;
 const {
     compare,
@@ -15,7 +12,6 @@ const {
 } = operator;
 
 const name = '__putout_plugin_check_replace_code';
-
 const get = (path) => path[name];
 const set = (path) => path[name] = true;
 
@@ -61,6 +57,7 @@ module.exports.traverse = ({push}) => ({
                     mainPath: path,
                     path: propertyPath,
                 });
+                
                 return;
             }
             
@@ -73,6 +70,7 @@ module.exports.traverse = ({push}) => ({
                     mainPath: path,
                     path: propertyPath,
                 });
+                
                 return;
             }
             
@@ -95,6 +93,7 @@ module.exports.traverse = ({push}) => ({
                     mainPath: path,
                     path: propertyPath,
                 });
+                
                 return;
             }
             
@@ -116,13 +115,19 @@ function parseKey(propertyPath) {
     const [isComputed, key] = compute(keyPath);
     
     if (!isComputed)
-        return [Error(`Replace key cannot be computed: '${keyPath.toString()}'`)];
+        return [
+            Error(`Replace key cannot be computed: '${keyPath.toString()}'`),
+        ];
     
-    return [null, key];
+    return [
+        null,
+        key,
+    ];
 }
 
 function hasMatch(path) {
-    const {body} = path.scope.getProgramParent().path.node;
+    const {body} = path.scope
+        .getProgramParent().path.node;
     
     for (const current of body) {
         if (compare(current, 'module.exports.match = __a'))
@@ -131,4 +136,3 @@ function hasMatch(path) {
     
     return false;
 }
-

package/lib/convert-add-argument-to-add-args/index.js

@@ -8,4 +8,3 @@ module.exports.replace = () => ({
         return path;
     },
 });
-

package/lib/convert-babel-types/index.js

@@ -17,16 +17,17 @@ module.exports.report = () => {
 
 function isRequire(path) {
     return path
-        .get('callee')
-        .isIdentifier({name: 'require'});
+        .get('callee').isIdentifier({
+            name: 'require',
+        });
 }
 
 function isBabelTypes(path) {
     return path
-        .get('arguments.0')
-        .isStringLiteral({value: '@babel/types'});
+        .get('arguments.0').isStringLiteral({
+            value: '@babel/types',
+        });
 }
-
 module.exports.traverse = ({push}) => ({
     CallExpression(path) {
         if (!isRequire(path))
@@ -42,4 +43,3 @@ module.exports.traverse = ({push}) => ({
 module.exports.fix = (path) => {
     replaceWith(path, astRequire);
 };
-

package/lib/convert-dirname-to-url/index.js

@@ -12,4 +12,3 @@ module.exports.match = () => ({
 module.exports.replace = () => ({
     'createTest(__dirname, __a)': 'createTest(import.meta.url, __a)',
 });
-

package/lib/convert-find-to-traverse/index.js

@@ -1,6 +1,10 @@
 'use strict';
 
-const {types, operator} = require('putout');
+const {
+    types,
+    operator,
+} = require('putout');
+
 const {replaceWith} = operator;
 
 const {
@@ -24,11 +28,9 @@ module.exports.fix = fixType({
     isMemberExpression: (path) => {
         path.get('property').node.name = 'traverse';
     },
-    
     isFunction: (path) => {
         path.node.params = [path.node.params[1]];
     },
-    
     isCallExpression: (path) => {
         replaceWith(path, ReturnStatement(path.node.arguments[1]));
     },
@@ -46,7 +48,6 @@ module.exports.traverse = ({push}) => ({
             return;
         
         const traverseCallPath = getTraverseCall(rightPath);
-        
         push(traverseCallPath);
         push(leftPath);
         push(rightPath);
@@ -72,14 +73,16 @@ function getTraverseCall(path) {
     
     path.traverse({
         CallExpression(path) {
-            if (!path.get('callee').isIdentifier({name: 'traverse'}))
+            if (!path.get('callee').isIdentifier({
+                name: 'traverse',
+            }))
                 return;
             
             result = path;
+            
             path.stop();
         },
     });
     
     return result;
 }
-

package/lib/convert-match-to-function/index.js

@@ -5,4 +5,3 @@ module.exports.report = () => `'match' should be a function`;
 module.exports.replace = () => ({
     'module.exports.match= __object': 'module.exports.match = () => __object',
 });
-

package/lib/convert-method-to-property/index.js

@@ -10,7 +10,9 @@ const {ObjectProperty} = types;
 
 module.exports.report = () => 'Object Property should be used instead of Method';
 
-module.exports.include = () => ['ObjectMethod'];
+module.exports.include = () => [
+    'ObjectMethod',
+];
 
 module.exports.filter = (path) => {
     if (!path.node.params.length)
@@ -26,9 +28,9 @@ module.exports.filter = (path) => {
 
 module.exports.fix = (path) => {
     const keyPath = path.get('key');
+    
     path.node.type = 'ArrowFunctionExpression';
     path.node.id = null;
     
     replaceWith(path, ObjectProperty(keyPath.node, path.node));
 };
-

package/lib/convert-node-to-path-in-get-template-values/index.js

@@ -35,7 +35,6 @@ module.exports.traverse = ({push}) => ({
     'getTemplateValues(__a, __b)': (path) => {
         const {scope} = path;
         const {bindings} = scope;
-        
         const __aPath = path.get('arguments.0');
         
         if (__aPath.isMemberExpression()) {
@@ -70,4 +69,3 @@ module.exports.traverse = ({push}) => ({
         });
     },
 });
-

package/lib/convert-number-to-numeric/index.js

@@ -23,4 +23,3 @@ module.exports.traverse = ({push}) => ({
         push(path);
     },
 });
-

package/lib/convert-process-to-find/index.js

@@ -28,4 +28,3 @@ module.exports.replace = () => ({
         return 'module.exports.find = __a';
     },
 });
-

package/lib/convert-putout-test-to-create-test/index.js

@@ -3,7 +3,6 @@
 const {assign} = Object;
 
 module.exports.report = () => `Use 'createTest' instead of 'putoutTest'`;
-
 module.exports.filter = ({scope}) => !scope.bindings.createTest;
 
 module.exports.include = () => [
@@ -21,4 +20,3 @@ module.exports.fix = (path) => {
     
     path.scope.rename('putoutTest', 'createTest');
 };
-

package/lib/convert-replace-to-function/index.js

@@ -5,4 +5,3 @@ module.exports.report = () => `'replace' should be a function`;
 module.exports.replace = () => ({
     'module.exports.replace = __object': 'module.exports.replace = () => __object',
 });
-

package/lib/convert-replace-with/index.js

@@ -50,8 +50,7 @@ module.exports.fix = ({path, calleePath, property, object, program, isInserted})
     const id = Identifier('replaceWith');
     const varPath = getVarPath(bindings);
     
-    varPath.node.id.properties
-        .unshift(ObjectProperty(id, id, false, true));
+    varPath.node.id.properties.unshift(ObjectProperty(id, id, false, true));
 };
 
 function getVarPath(bindings) {
@@ -65,7 +64,6 @@ function getVarPath(bindings) {
     
     return insertAfter.path;
 }
-
 module.exports.traverse = ({push}) => {
     const isInserted = fullstore();
     
@@ -76,7 +74,10 @@ module.exports.traverse = ({push}) => {
             if (!calleePath.isMemberExpression())
                 return;
             
-            const {object, property} = calleePath.node;
+            const {
+                object,
+                property,
+            } = calleePath.node;
             
             if (property.name !== 'replaceWith')
                 return;
@@ -94,4 +95,3 @@ module.exports.traverse = ({push}) => {
         },
     };
 };
-

package/lib/convert-replace-with-multiple/index.js

@@ -38,10 +38,9 @@ module.exports.fix = ({path, calleePath, property, object, program}) => {
         return insertAfter(strictModePath, replaceWithAST);
     
     const id = Identifier('replaceWithMultiple');
-    
     const varPath = getVarPath(bindings);
-    varPath.node.id.properties
-        .push(ObjectProperty(id, id, false, true));
+    
+    varPath.node.id.properties.push(ObjectProperty(id, id, false, true));
 };
 
 function getVarPath(bindings) {
@@ -55,7 +54,6 @@ function getVarPath(bindings) {
     
     return insertAfter.path;
 }
-
 module.exports.traverse = ({push}) => ({
     CallExpression(path) {
         const calleePath = path.get('callee');
@@ -63,7 +61,10 @@ module.exports.traverse = ({push}) => ({
         if (!calleePath.isMemberExpression())
             return;
         
-        const {object, property} = calleePath.node;
+        const {
+            object,
+            property,
+        } = calleePath.node;
         
         if (property.name !== 'replaceWithMultiple')
             return;
@@ -79,4 +80,3 @@ module.exports.traverse = ({push}) => ({
         });
     },
 });
-

package/lib/convert-to-no-transform-code/index.js

@@ -16,7 +16,10 @@ module.exports.traverse = ({push}) => ({
         if (!calleePath.isMemberExpression())
             return;
         
-        const {object, property} = calleePath.node;
+        const {
+            object,
+            property,
+        } = calleePath.node;
         
         if (object.name !== 't' || property.name !== 'transformCode')
             return;
@@ -40,4 +43,3 @@ module.exports.fix = ({path, calleePath}) => {
     calleePath.node.property = Identifier('noTransformCode');
     path.node.arguments.pop();
 };
-

package/lib/convert-traverse-to-include/index.js

@@ -5,12 +5,14 @@ const {
     template,
     operator,
 } = require('putout');
+
 const {StringLiteral} = types;
 const {compare} = operator;
 
-const isPush = (path) => path.get('value').isIdentifier({
-    name: 'push',
-});
+const isPush = (path) => path
+    .get('value').isIdentifier({
+        name: 'push',
+    });
 
 module.exports.report = () => 'Includer should be used instead of Traverser';
 
@@ -69,4 +71,3 @@ function isBlock(path) {
     
     return compare(node, 'push(path)');
 }
-

package/lib/convert-traverse-to-replace/index.js

@@ -11,6 +11,7 @@ module.exports.report = () => 'Replacer should be used instead of Traverser (htt
 module.exports.match = () => ({
     'module.exports.traverse = (__args) => __a': ({__args}, path) => {
         const program = path.scope.getProgramParent().path;
+        
         const withFix = contains(program, [
             'module.exports.fix = __a',
         ]);
@@ -54,6 +55,7 @@ function check(path) {
                 return;
             
             hasTraverseMethod = true;
+            
             path.stop();
         },
         'push(__a)': (path) => {
@@ -64,4 +66,3 @@ function check(path) {
     
     return hasPushCall || hasTraverseMethod;
 }
-

package/lib/convert-url-to-dirname/index.js

@@ -12,4 +12,3 @@ module.exports.match = () => ({
 module.exports.replace = () => ({
     'createTest(import.meta.url, __a)': 'createTest(__dirname, __a)',
 });
-

package/lib/create-test/index.js

@@ -27,28 +27,39 @@ module.exports.include = () => [
 ];
 
 module.exports.fix = (path, {options}) => {
-    const [name, value] = options.add;
     const objectPath = path.get('arguments.1');
     
     if (!getProperty(objectPath, 'plugins'))
         convert(objectPath);
     
-    const property = ObjectProperty(Identifier(name), StringLiteral(value));
-    objectPath.node.properties.unshift(property);
+    for (const [name, value] of options.add) {
+        if (getProperty(objectPath, name))
+            continue;
+        
+        const property = ObjectProperty(Identifier(name), StringLiteral(value));
+        objectPath.node.properties.unshift(property);
+    }
 };
 
 module.exports.filter = (path, {options}) => {
     if (!options.add)
         return false;
     
-    const [name] = options.add;
     const objectPath = path.get('arguments.1');
     
-    return !getProperty(objectPath, name);
+    for (const [name] of options.add) {
+        if (!getProperty(objectPath, name))
+            return true;
+    }
+    
+    return false;
 };
 
 function convert(objectPath) {
-    const {key, value} = objectPath.node.properties[0];
+    const {
+        key,
+        value,
+    } = objectPath.node.properties[0];
     
     replaceWith(objectPath, ObjectExpression([
         ObjectProperty(Identifier('plugins'), ArrayExpression([
@@ -59,4 +70,3 @@ function convert(objectPath) {
         ])),
     ]));
 }
-

package/lib/declare/index.js

@@ -9,4 +9,3 @@ module.exports.declare = () => ({
     ...operator,
     ...types,
 });
-

package/lib/declare/operator.js

@@ -21,4 +21,3 @@ module.exports = {
     getProperty: `const {getProperty} = operator`,
     getProperties: `const {getProperties} = operator`,
 };
-

package/lib/declare/types.json

@@ -635,4 +635,4 @@
     "isRegexLiteral": "const {isRegexLiteral} = types",
     "isRestProperty": "const {isRestProperty} = types",
     "isSpreadProperty": "const {isSpreadProperty} = types"
-}
+}

package/lib/includer/index.js

@@ -5,10 +5,8 @@ module.exports.report = () => 'Includer functions should return array (https://g
 module.exports.replace = () => ({
     'module.exports.include = () => "__a"': 'module.exports.include = ["__a"]',
     'module.exports.exclude = () => "__a"': 'module.exports.exclude = ["__a"]',
-    
     'module.exports.include = ["__a"]': 'module.exports.include = () => ["__a"]',
     'module.exports.exclude = ["__a"]': 'module.exports.exclude = () => ["__a"]',
-    
     'module.exports.include = "__a"': 'module.exports.include = ["__a"]',
     'module.exports.exclude = "__a"': 'module.exports.exclude= ["__a"]',
 });

package/lib/index.js

@@ -41,4 +41,3 @@ module.exports.rules = {
     ...getRule('includer'),
     ...getRule('create-test'),
 };
-

package/lib/move-require-on-top-level/index.js

@@ -1,6 +1,7 @@
 'use strict';
 
 const justCamelCase = require('just-camel-case');
+
 const {
     types,
     template,
@@ -60,8 +61,11 @@ module.exports.replace = () => ({
 const buildRequire = template(`const NAME = REQUIRE`);
 
 function declareRequire({__a, __b}, path) {
-    const shortName = (__a.value || __a.name).split('/').pop();
+    const shortName = __a.value || __a.name
+        .split('/').pop();
+    
     const name = justCamelCase(shortName);
+    
     const requireNode = buildRequire({
         NAME: Identifier(name),
         REQUIRE: __b,
@@ -70,9 +74,10 @@ function declareRequire({__a, __b}, path) {
     if (path.scope.hasBinding(name))
         return name;
     
-    const programPath = path.scope.getProgramParent().path;
+    const programPath = path.scope
+        .getProgramParent().path;
+    
     programPath.node.body.unshift(requireNode);
     
     return name;
 }
-

package/lib/replace-operate-with-operator/index.js

@@ -5,4 +5,3 @@ module.exports.report = () => '"operator" should be used instead of "operate"';
 module.exports.replace = () => ({
     'const __object = require("putout").operate': 'const __object = require("putout").operator',
 });
-

package/lib/replace-test-message/index.js

@@ -58,16 +58,22 @@ function isCorrect({path, incorrect}) {
     const calleePath = path.findParent(isCallExpression);
     
     if (!calleePath)
-        return [CORRECT];
+        return [
+            CORRECT,
+        ];
     
     const messagePath = calleePath.get('arguments.0');
     
     if (!messagePath.isStringLiteral())
-        return [CORRECT];
+        return [
+            CORRECT,
+        ];
     
     const {value} = messagePath.node;
     const is = !incorrect.test(value);
     
-    return [is, messagePath];
+    return [
+        is,
+        messagePath,
+    ];
 }
-

package/lib/shorten-imports/index.js

@@ -6,4 +6,3 @@ module.exports.replace = () => ({
     'require("putout/lib/cli/process-file")': 'require("putout/process-file")',
     'require("putout/lib/parse-options")': 'require("putout/parse-options")',
 });
-

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-putout",
-  "version": "12.1.0",
+  "version": "12.1.1",
   "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin helps with plugins development",