@putout/plugin-putout 23.16.0 → 24.1.0

package/README.md

@@ -25,6 +25,7 @@ npm i @putout/plugin-putout -D
 - ✅ [apply-create-test](#apply-create-test);
 - ✅ [apply-create-nested-directory](#apply-create-nested-directory);
 - ✅ [apply-declare](#apply-declare);
+- ✅ [apply-exports-to-add-args](#apply-exports-to-add-args);
 - ✅ [apply-for-of-to-track-file](#apply-for-of-to-track-file);
 - ✅ [apply-fixture-name-to-message](#apply-fixture-name-to-message);
 - ✅ [apply-insert-after](#apply-insert-after);
@@ -96,6 +97,7 @@ npm i @putout/plugin-putout -D
         "putout/apply-create-nested-directory": "on",
         "putout/apply-async-formatter": "on",
         "putout/apply-declare": "on",
+        "putout/apply-exports-to-add-args": "on",
         "putout/apply-report": "on",
         "putout/apply-processors-destructuring": "on",
         "putout/apply-rename": "on",
@@ -456,6 +458,32 @@ module.exports.declare = () => ({
 });
 ```
 
+## apply-exports-to-add-args
+
+Check out in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/cf35de5e80e8f7aad866358a50c5eded/0af6142fc9c9e71ac2a2aa96cb85613dd95c9fbf).
+
+### ❌ Example of incorrect code
+
+```js
+export default addArgs(__args);
+```
+
+### ✅ Example of correct code
+
+```js
+const {
+    report,
+    fix,
+    traverse,
+} = addArgs(__args);
+
+export {
+    report,
+    fix,
+    traverse,
+};
+```
+
 ## apply-async-formatter
 
 ### ❌ Example of incorrect code

package/lib/add-await-to-progress/index.js

@@ -1,6 +1,5 @@
-'use strict';
+import {types, operator} from 'putout';
 
-const {types, operator} = require('putout');
 const {
     objectProperty,
     objectPattern,
@@ -17,14 +16,14 @@ const checkAwait = (vars, path) => {
     return !path.parentPath.isAwaitExpression();
 };
 
-module.exports.report = () => `Add 'await' to operator 'progress()'`;
+export const report = () => `Add 'await' to operator 'progress()'`;
 
-module.exports.match = () => ({
+export const match = () => ({
     'progress(__args)': checkAwait,
     't.progress(__args)': checkAwait,
 });
 
-module.exports.replace = () => ({
+export const replace = () => ({
     'test(__a, async (t, {progress}) => __body)': 'test(__a, async ({progress}) => __body)',
     'progress(__args)': addAwait,
     't.progress(__args)': addAwait,

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

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

package/lib/add-path-arg-to-fix/index.js

@@ -1,6 +1,5 @@
-'use strict';
+import {operator} from 'putout';
 
-const {operator} = require('putout');
 const {compare} = operator;
 const check = ({__a}) => {
     if (__a.body)
@@ -9,14 +8,14 @@ const check = ({__a}) => {
     return compare(__a, 'path.__()');
 };
 
-module.exports.report = () => `Add 'path' argument to 'fix'`;
+export const report = () => `Add 'path' argument to 'fix'`;
 
-module.exports.match = () => ({
+export const match = () => ({
     'const fix = () => __a': check,
     'module.exports.fix = () => __a': check,
 });
 
-module.exports.replace = () => ({
+export const replace = () => ({
     'const fix = () => __a': 'const fix = (path) => __a',
     'module.exports.fix = () => __a': 'module.exports.fix = (path) => __a',
 });

package/lib/add-path-arg-to-visitors/index.js

@@ -1,17 +1,16 @@
-'use strict';
+import {types} from 'putout';
 
-const {types} = require('putout');
 const {identifier} = types;
 
-module.exports.report = () => `Add 'path' argument to 'traverse' visitors`;
+export const report = () => `Add 'path' argument to 'traverse' visitors`;
 
 const TRAVERSE = '(__args) => __object';
 
-module.exports.fix = (path) => {
+export const fix = (path) => {
     path.node.params.push(identifier('path'));
 };
 
-module.exports.traverse = ({push}) => ({
+export const traverse = ({push}) => ({
     [`export const traverse = ${TRAVERSE}`]: traverseMethods({
         where: 'declaration.declarations.0.init',
         push,

package/lib/add-places-to-compare-places/index.js

@@ -1,7 +1,5 @@
-'use strict';
+export const report = () => `Add 'places' array to 'comparePlaces()'`;
 
-module.exports.report = () => `Add 'places' array to 'comparePlaces()'`;
-
-module.exports.replace = () => ({
+export const replace = () => ({
     'comparePlaces(__a)': 'comparePlaces(__a, [])',
 });

package/lib/add-test-args/index.js

@@ -1,9 +1,12 @@
-'use strict';
+import {operator} from 'putout';
 
-const {operator} = require('putout');
 const {addArgs} = operator;
 
-module.exports = addArgs({
+const {
+    report,
+    fix,
+    traverse,
+} = addArgs({
     comparePlaces: ['{comparePlaces}', [
         'test("__a", async (__args) => __body)',
         'test.skip("__a", async (__args) => __body)',
@@ -25,3 +28,9 @@ module.exports = addArgs({
         'test.only("__a", async (__args) => __body)',
     ]],
 });
+
+export {
+    report,
+    fix,
+    traverse,
+};

package/lib/add-track-file/index.js

@@ -1,11 +1,20 @@
-'use strict';
+import {operator} from 'putout';
 
-const {operator} = require('putout');
 const {addArgs} = operator;
 
-module.exports = addArgs({
+const {
+    report,
+    fix,
+    traverse,
+} = addArgs({
     trackFile: ['{trackFile}', [
         '(__a, __b) => __body',
         '(__a) => __body',
     ]],
 });
+
+export {
+    report,
+    fix,
+    traverse,
+};

package/lib/add-traverse-args/index.js

@@ -1,6 +1,5 @@
-'use strict';
+import {types, operator} from 'putout';
 
-const {types, operator} = require('putout');
 const {
     objectPattern,
     objectProperty,
@@ -8,7 +7,7 @@ const {
     isObjectPattern,
 } = types;
 
-const {traverse, getBinding} = operator;
+const {getBinding} = operator;
 
 const defaultNames = [
     'push',
@@ -18,9 +17,9 @@ const defaultNames = [
     'options',
 ];
 
-module.exports.report = ({name}) => `Add '${name}' argument to 'traverse'`;
+export const report = ({name}) => `Add '${name}' argument to 'traverse'`;
 
-module.exports.fix = ({name, fn}) => {
+export const fix = ({name, fn}) => {
     const computed = false;
     const shorthand = true;
     const id = identifier(name);
@@ -34,7 +33,7 @@ module.exports.fix = ({name, fn}) => {
     fn.params[0].properties.push(property);
 };
 
-module.exports.traverse = ({push, options}) => {
+export const traverse = ({push, options}) => {
     const {
         names = defaultNames,
     } = options;
@@ -67,7 +66,7 @@ const isArgExists = (mainName, fn) => {
 const checkArgs = (names, push) => (path) => {
     const fn = parseFn(path);
     
-    traverse(path, {
+    operator.traverse(path, {
         ReferencedIdentifier(path) {
             const {name} = path.node;
             

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

@@ -1,6 +1,5 @@
-'use strict';
+import {operator, types} from 'putout';
 
-const {operator, types} = require('putout');
 const {
     objectProperty,
     objectPattern,
@@ -12,9 +11,9 @@ const shorthand = true;
 
 const {compare} = operator;
 
-module.exports.report = () => 'Use Async API to test Formatter';
+export const report = () => 'Use Async API to test Formatter';
 
-module.exports.replace = () => ({
+export const replace = () => ({
     't.format(__args)': create('format'),
     't.noFormat(__args)': create('noFormat'),
     't.formatMany(__args)': create('formatMany'),

package/lib/apply-create-nested-directory/index.js

@@ -1,6 +1,5 @@
-'use strict';
+import {types} from 'putout';
 
-const {types} = require('putout');
 const {isStringLiteral} = types;
 
 const NESTED = {
@@ -8,12 +7,12 @@ const NESTED = {
     createNestedDirectory: 'createDirectory',
 };
 
-module.exports.report = (path) => {
+export const report = (path) => {
     const {name} = path.node.callee;
     return `Use '${NESTED[name]}()' instead of '${name}()'`;
 };
 
-module.exports.match = () => ({
+export const match = () => ({
     'createDirectory(__a, __b)': ({__b}) => {
         if (!isStringLiteral(__b))
             return false;
@@ -28,7 +27,7 @@ module.exports.match = () => ({
     },
 });
 
-module.exports.replace = () => ({
+export const replace = () => ({
     'createDirectory(__a, __b)': 'createNestedDirectory(__a, __b)',
     'createNestedDirectory(__a, __b)': 'createDirectory(__a, __b)',
 });

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

@@ -1,7 +1,5 @@
-'use strict';
+export const report = () => `Apply 'createTest'`;
 
-module.exports.report = () => `Apply 'createTest'`;
-
-module.exports.replace = () => ({
+export const replace = () => ({
     'require("@putout/test")(__dirname, __a)': `createTest(__dirname, __a)`,
 });

package/lib/apply-declare/index.js

@@ -1,7 +1,5 @@
-'use strict';
+export const report = () => `Use 'Declarator' instead of 'operator.declare()'`;
 
-module.exports.report = () => `Use 'Declarator' instead of 'operator.declare()'`;
-
-module.exports.replace = () => ({
+export const replace = () => ({
     'module.exports = declare(__a)': 'module.exports.declare = () => __a',
 });

package/lib/apply-exports-to-add-args/index.js

@@ -0,0 +1,12 @@
+export const report = () => `Apply exports to 'addArgs'`;
+
+export const replace = () => ({
+    'export default addArgs(__args)': `{
+         const {report, fix, traverse} = addArgs(__args);
+         export {
+            report,
+            fix,
+            traverse,
+        }
+    }`,
+});

package/lib/apply-fixture-name-to-message/index.js

@@ -1,6 +1,5 @@
-'use strict';
+import {operator} from 'putout';
 
-const {operator} = require('putout');
 const {
     setLiteralValue,
     compare,
@@ -17,9 +16,9 @@ const NAMES = [
     'no transform',
 ];
 
-module.exports.report = () => `Apply 'fixture' name to 'message'`;
+export const report = () => `Apply 'fixture' name to 'message'`;
 
-module.exports.match = () => ({
+export const match = () => ({
     't.noReport(__a)': check,
     't.report(__a, __b)': check,
     't.transform(__a)': check,
@@ -27,7 +26,7 @@ module.exports.match = () => ({
     't.noTransform(__a)': check,
 });
 
-module.exports.replace = () => ({
+export const replace = () => ({
     't.noReport(__a)': transform,
     't.report(__a, __b)': transform,
     't.transform(__a)': transform,

package/lib/apply-for-of-to-track-file/index.js

@@ -1,7 +1,5 @@
-'use strict';
+export const report = () => `Use 'if condition' instead of 'ternary expression'`;
 
-module.exports.report = () => `Use 'if condition' instead of 'ternary expression'`;
-
-module.exports.replace = () => ({
+export const replace = () => ({
     'trackFile(__a, __b).map(push)': 'for (const file of trackFile( __a, __b)) {push(file)}',
 });

package/lib/apply-insert-after/index.js

@@ -1,7 +1,5 @@
-'use strict';
+export const report = () => `Use 'insertAfter(a, b)' instead of 'a.insertAfter(b)'`;
 
-module.exports.report = () => `Use 'insertAfter(a, b)' instead of 'a.insertAfter(b)'`;
-
-module.exports.replace = () => ({
+export const replace = () => ({
     '__a.insertAfter(__b)': 'insertAfter(__a, __b)',
 });

package/lib/apply-insert-before/index.js

@@ -1,7 +1,5 @@
-'use strict';
+export const report = () => `Use 'insertBefore(a, b)' instead of 'a.insertBefore(b)'`;
 
-module.exports.report = () => `Use 'insertBefore(a, b)' instead of 'a.insertBefore(b)'`;
-
-module.exports.replace = () => ({
+export const replace = () => ({
     '__a.insertBefore(__b)': 'insertBefore(__a, __b)',
 });

package/lib/apply-lowercase-to-node-builders/index.js

@@ -1,18 +1,17 @@
-'use strict';
+import {types} from 'putout';
 
-const {types} = require('putout');
 const {isIdentifier} = types;
 
-module.exports.report = () => `Use lowercased node builders`;
+export const report = () => `Use lowercased node builders`;
 
-module.exports.fix = (path) => {
+export const fix = (path) => {
     const [first] = path.node.name;
     const other = path.node.name.slice(1);
     
     path.node.name = first.toLowerCase() + other;
 };
 
-module.exports.traverse = ({push}) => ({
+export const traverse = ({push}) => ({
     CallExpression(path) {
         const calleePath = path.get('callee');
         

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

@@ -1,18 +1,16 @@
-'use strict';
+import {types} from 'putout';
+import {parseImportSpecifiers} from 'parse-import-specifiers';
 
-const {types} = require('putout');
-
-const {parseImportSpecifiers} = require('parse-import-specifiers');
 const {isImportDeclaration} = types;
 const noop = () => {};
 
-module.exports.report = () => `Use 'import * as plugin' instead of 'import plugin'`;
+export const report = () => `Use 'import * as plugin' instead of 'import plugin'`;
 
-module.exports.fix = ({first}) => {
+export const fix = ({first}) => {
     first.node.type = 'ImportNamespaceSpecifier';
 };
 
-module.exports.traverse = ({push, listStore, pathStore}) => ({
+export const traverse = ({push, listStore, pathStore}) => ({
     'export const rules = __object': listStore,
     'import {createTest} from "@putout/test"': listStore,
     ...createImportVisitor({

package/lib/apply-parens/index.js

@@ -1,6 +1,4 @@
-'use strict';
-
-module.exports.report = (path) => {
+export const report = (path) => {
     if (path.isOptionalMemberExpression())
         return `Use 'hasParens(path)' instead of 'path.node.extra'`;
     
@@ -17,7 +15,7 @@ const ASSIGN_OBJECT = `
     }
 `;
 
-module.exports.replace = () => ({
+export const replace = () => ({
     '__a.node.extra.parenthesized = false': 'removeParens(__a)',
     '__a.node.extra.parenthesized = true': 'addParens(__a)',
     '__a.node.extra?.parenthesized': 'hasParens(__a)',

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

@@ -1,8 +1,6 @@
-'use strict';
+export const report = () => 'Test operator should be destructured';
 
-module.exports.report = () => 'Test operator should be destructured';
-
-module.exports.replace = () => ({
+export const 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)}',

package/lib/apply-remove/index.js

@@ -1,7 +1,5 @@
-'use strict';
+export const report = () => `Use 'remove(path)' instead of 'path.remove()'`;
 
-module.exports.report = () => `Use 'remove(path)' instead of 'path.remove()'`;
-
-module.exports.replace = () => ({
+export const replace = () => ({
     '__a.remove()': 'remove(__a)',
 });

package/lib/apply-rename/index.js

@@ -1,7 +1,5 @@
-'use strict';
+export const report = () => `Use 'rename(path, from, to)' instead of 'path.scope.rename(from, to)'`;
 
-module.exports.report = () => `Use 'rename(path, from, to)' instead of 'path.scope.rename(from, to)'`;
-
-module.exports.replace = () => ({
+export const replace = () => ({
     '__a.scope.rename(__b, __c)': 'rename(__a, __b, __c)',
 });

package/lib/apply-report/index.js

@@ -1,6 +1,5 @@
-'use strict';
+import {operator} from 'putout';
 
-const {operator} = require('putout');
 const {compare} = operator;
 const TYPES = {
     noReport: 'noReportWithOptions',
@@ -8,7 +7,7 @@ const TYPES = {
     noReportWithOptions: 'noReport',
 };
 
-module.exports.report = (path) => {
+export const report = (path) => {
     const {name} = path.node.callee.property;
     
     if (compare(path, 't.noReport(__a, "__b")'))
@@ -20,7 +19,7 @@ module.exports.report = (path) => {
     return `Use 't.${TYPES[name]}()' instead of 't.${name}()'`;
 };
 
-module.exports.replace = () => ({
+export const replace = () => ({
     't.noReport(__a, __object)': 't.noReportWithOptions(__a, __object)',
     't.noReport(__a, __b)': 't.noReport(__a)',
     't.report(__a)': 't.noReport(__a)',

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

@@ -1,6 +1,4 @@
-'use strict';
-
-const {types, operator} = require('putout');
+import {types, operator} from 'putout';
 
 const {compute, rename} = operator;
 const {identifier} = types;
@@ -14,12 +12,12 @@ const getShortName = (a) => {
         .replace('putout_processor_', '');
 };
 
-module.exports.report = ({value}) => {
+export const report = ({value}) => {
     const name = getShortName(value);
     return `Use '[${name}]' instead of '${value}'`;
 };
 
-module.exports.fix = ({path, value}) => {
+export const fix = ({path, value}) => {
     const name = getShortName(value);
     
     path.node.key = identifier(name);
@@ -27,7 +25,7 @@ module.exports.fix = ({path, value}) => {
     rename(path.get('value'), '__a', chooseType(name));
 };
 
-module.exports.traverse = ({push}) => ({
+export const traverse = ({push}) => ({
     ObjectProperty(path) {
         const [is, value] = compute(path.get('key'));
         

package/lib/apply-vars/index.js

@@ -1,15 +1,14 @@
-'use strict';
+import {operator} from 'putout';
 
-const {operator} = require('putout');
 const {compareAny} = operator;
 
-module.exports.report = () => `Use 'vars' instead of '{}'`;
+export const report = () => `Use 'vars' instead of '{}'`;
 
-module.exports.match = () => ({
+export const match = () => ({
     '({}, path) => __a': isInsideReplacer,
 });
 
-module.exports.replace = () => ({
+export const replace = () => ({
     '({}, path) => __a': '(vars, path) => __a',
 });
 

package/lib/check-declare/index.js

@@ -1,11 +1,8 @@
-'use strict';
-
-const tryCatch = require('try-catch');
-const putout = require('putout');
+import tryCatch from 'try-catch';
+import putout from 'putout';
 
 const {types, operator} = putout;
 
-const noop = () => {};
 const {getTemplateValues} = operator;
 
 const DECLARE_ESM = 'export const declare = () => __object';
@@ -17,10 +14,10 @@ const {
     isStringLiteral,
 } = types;
 
-module.exports.report = ({message}) => message;
+export const report = ({message}) => message;
 
-module.exports.fix = noop;
-module.exports.traverse = ({push}) => ({
+export const fix = () => {};
+export const traverse = ({push}) => ({
     [DECLARE_ESM]: createCheck(push, DECLARE_ESM),
     [DECLARE_COMMONJS]: createCheck(push, DECLARE_COMMONJS),
     [COMMONJS]: createCheck(push, COMMONJS),

package/lib/check-match/index.js

@@ -1,6 +1,4 @@
-'use strict';
-
-const {operator, types} = require('putout');
+import {operator, types} from 'putout';
 
 const {
     getTemplateValues,
@@ -13,13 +11,13 @@ const PATTERN_MATCH = 'module.exports.match = () => __object';
 const PATTERN_REPLACE = 'module.exports.replace = () => __object';
 const PATTERN_REPLACE_RETURN = 'module.exports.replace = (__args) => __body';
 
-module.exports.report = () => `☝️ Looks like 'match()' template absent in 'replace()'`;
+export const report = () => `☝️ Looks like 'match()' template absent in 'replace()'`;
 
-module.exports.replace = () => ({
+export const replace = () => ({
     [PATTERN_MATCH]: PATTERN_MATCH,
 });
 
-module.exports.match = () => ({
+export const match = () => ({
     [PATTERN_MATCH]: ({__object}, path) => {
         const namesMatch = getNames(__object);
         const namesReplace = [];