@putout/plugin-putout 22.1.0 → 22.3.0

package/README.md

@@ -17,6 +17,7 @@ npm i @putout/plugin-putout -D
 - ✅ [add-index-to-import](#add-index-to-import);
 - ✅ [add-places-to-compare-places](#add-places-to-compare-places);
 - ✅ [add-path-arg-to-fix](#add-path-arg-to-fix);
+- ✅ [add-path-arg-to-visitors](#add-path-arg-to-visitors);
 - ✅ [add-test-args](#add-test-args);
 - ✅ [add-traverse-args](#add-traverse-args);
 - ✅ [add-track-file](#add-track-file);
@@ -24,6 +25,7 @@ npm i @putout/plugin-putout -D
 - ✅ [apply-create-test](#apply-create-test);
 - ✅ [apply-declare](#apply-declare);
 - ✅ [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);
 - ✅ [apply-insert-before](#apply-insert-before);
 - ✅ [apply-namespace-specifier](#apply-namespace-specifier);
@@ -76,15 +78,16 @@ npm i @putout/plugin-putout -D
     "rules": {
         "putout/add-places-to-compare-places": "on",
         "putout/add-path-arg-to-fix": "on",
+        "putout/add-path-arg-to-visitors": "on",
         "putout/add-test-args": "on",
         "putout/add-traverse-args": "on",
         "putout/add-track-file": "on",
         "putout/add-await-to-progress": "on",
         "putout/add-index-to-import": "on",
         "putout/apply-create-test": "on",
-        "putout/apply-processors-destructuring": "on",
         "putout/apply-async-formatter": "on",
         "putout/apply-declare": "on",
+        "putout/apply-processors-destructuring": "on",
         "putout/apply-rename": "on",
         "putout/apply-remove": "on",
         "putout/apply-insert-before": "on",
@@ -92,6 +95,7 @@ npm i @putout/plugin-putout -D
         "putout/apply-short-processors": "on",
         "putout/apply-namespace-specifier": "on",
         "putout/apply-for-of-to-track-file": "on",
+        "putout/apply-fixture-name-to-message": "on",
         "putout/check-match": "on",
         "putout/check-replace-code": ["on", {
             "once": true
@@ -866,6 +870,30 @@ export const fix = (path) => {
 };
 ```
 
+## add-path-arg-to-visitors
+
+Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/e20eb16668b2ebbceca1a03cde859e93/fee08d6dbb6aa4d835adacc8e9de4d994fd34848).
+
+### ❌ Example of incorrect code
+
+```js
+export const traverse = () => ({
+    TSUnionType() {
+        console.log(path);
+    },
+});
+```
+
+### ✅ Example of correct code
+
+```js
+export const traverse = () => ({
+    TSUnionType(path) {
+        console.log(path);
+    },
+});
+```
+
 ## add-test-args
 
 ### ❌ Example of incorrect code
@@ -1393,6 +1421,26 @@ module.exports.replace = () => ({
 });
 ```
 
+## apply-fixture-name-to-message"
+
+Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/66eb493ee966c2465a9fc783cd3ca362/f42ec60e393181b6d6aaf6df81640e93fabe8783).
+
+```js
+test('flatlint: convert-comma-to-semicolon: no report: xxx', (t) => {
+    t.noReport('array');
+    t.end();
+});
+```
+
+### ✅ Example of correct code
+
+```js
+test('flatlint: convert-comma-to-semicolon: no report: array', (t) => {
+    t.noReport('array');
+    t.end();
+});
+```
+
 ## License
 
 MIT

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

@@ -0,0 +1,46 @@
+'use strict';
+
+const {types} = require('putout');
+const {Identifier} = types;
+
+module.exports.report = () => `Add 'path' argument to 'traverse' visitors`;
+
+const TRAVERSE = '(__args) => __object';
+
+module.exports.fix = (path) => {
+    path.node.params.push(Identifier('path'));
+};
+
+module.exports.traverse = ({push}) => ({
+    [`export const traverse = ${TRAVERSE}`]: traverseMethods({
+        where: 'declaration.declarations.0.init',
+        push,
+    }),
+    [`module.exports.traverse = ${TRAVERSE}`]: traverseMethods({
+        where: 'right',
+        push,
+    }),
+});
+
+const traverseMethods = ({where, push}) => (path) => {
+    const initPath = path.get(where);
+    const objectPath = initPath.get('body');
+    
+    for (let prop of objectPath.get('properties')) {
+        if (prop.isObjectProperty())
+            prop = prop.get('value');
+        
+        if (!prop.isFunction())
+            continue;
+        
+        if (prop.node.params.length)
+            continue;
+        
+        prop.traverse({
+            ReferencedIdentifier(path) {
+                if (path.node.name === 'path')
+                    push(prop);
+            },
+        });
+    }
+};

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

@@ -107,10 +107,6 @@ function parseFn(path) {
     return path.get('declaration.declarations.0.init').node;
 }
 
-function isCallee(name, {parentPath}) {
-    return parentPath
-        .get('callee')
-        .isIdentifier({
-            name,
-        });
-}
+const isCallee = (name, {parentPath}) => parentPath.get('callee').isIdentifier({
+    name,
+});

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

@@ -0,0 +1,70 @@
+'use strict';
+
+const {operator} = require('putout');
+const {
+    setLiteralValue,
+    compare,
+} = operator;
+
+const FIXTURE = [
+    'report',
+    'transform',
+];
+
+const NAMES = [
+    ...FIXTURE,
+    'no report',
+    'no transform',
+];
+
+module.exports.report = () => `Apply 'fixture' name to 'message'`;
+
+module.exports.match = () => ({
+    't.noReport(__a)': check,
+    't.report(__a, __b)': check,
+    't.transform(__a)': check,
+    't.noTransform(__a)': check,
+});
+
+module.exports.replace = () => ({
+    't.noReport(__a)': transform,
+    't.report(__a, __b)': transform,
+    't.transform(__a)': transform,
+    't.noTransform(__a)': transform,
+});
+
+const isTest = (path) => compare(path, 'test(__a, (t) => __body)');
+
+const check = ({__a}, path) => {
+    const name = __a.value;
+    
+    if (FIXTURE.includes(name))
+        return false;
+    
+    const testPath = path.find(isTest);
+    
+    if (!testPath)
+        return false;
+    
+    const [str] = testPath.parentPath.parentPath.node.arguments;
+    
+    return !str.value.includes(name);
+};
+
+const transform = ({__a}, path) => {
+    const name = __a.value;
+    const testPath = path.find(isTest);
+    const [str] = testPath.parentPath.parentPath.node.arguments;
+    
+    const values = str.value.split(':');
+    const last = values
+        .at(-1)
+        .trim();
+    
+    if (!NAMES.includes(last))
+        values.pop();
+    
+    setLiteralValue(str, `${values.join(':')}: ${name}`);
+    
+    return path;
+};

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

@@ -12,21 +12,13 @@ module.exports.report = () => {
     return `"putout.types" should be used instead of "@babel/types"`;
 };
 
-function isRequire(path) {
-    return path
-        .get('callee')
-        .isIdentifier({
-            name: 'require',
-        });
-}
-
-function isBabelTypes(path) {
-    return path
-        .get('arguments.0')
-        .isStringLiteral({
-            value: '@babel/types',
-        });
-}
+const isRequire = (path) => path.get('callee').isIdentifier({
+    name: 'require',
+});
+
+const isBabelTypes = (path) => path.get('arguments.0').isStringLiteral({
+    value: '@babel/types',
+});
 
 module.exports.traverse = ({push}) => ({
     CallExpression(path) {

package/lib/convert-progress-to-track-file/index.js

@@ -29,11 +29,7 @@ module.exports.traverse = ({push}) => ({
         if (!statement)
             return;
         
-        const forOfCount = statement
-            .parentPath
-            .get('body')
-            .filter(isForOfStatement)
-            .length;
+        const forOfCount = statement.parentPath.get('body').filter(isForOfStatement).length;
         
         if (forOfCount > 1)
             return;

package/lib/index.js

@@ -55,6 +55,8 @@ const addPlacesToComparePlaces = require('./add-places-to-compare-places');
 const addPathArgToFix = require('./add-path-arg-to-fix');
 const convertIncludeToTraverse = require('./convert-include-to-traverse');
 const removeUselessPrinterOption = require('./remove-useless-printer-option');
+const addPathArgToVisitors = require('./add-path-arg-to-visitors');
+const applyFixtureNameToMessage = require('./apply-fixture-name-to-message');
 
 module.exports.rules = {
     'apply-processors-destructuring': applyProcessorsDestructuring,
@@ -112,4 +114,6 @@ module.exports.rules = {
     'add-path-arg-to-fix': addPathArgToFix,
     'convert-include-to-traverse': convertIncludeToTraverse,
     'remove-useless-printer-option': removeUselessPrinterOption,
+    'add-path-arg-to-visitors': addPathArgToVisitors,
+    'apply-fixture-name-to-message': applyFixtureNameToMessage,
 };

package/lib/remove-useless-printer-option/index.js

@@ -5,6 +5,7 @@ const {
     isCallExpression,
     isIdentifier,
 } = types;
+
 const {remove} = operator;
 
 module.exports.report = () => `Avoid useless 'printer' option`;

package/package.json

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