@putout/plugin-putout 22.3.0 → 22.5.0

package/README.md

@@ -28,6 +28,7 @@ npm i @putout/plugin-putout -D
 - ✅ [apply-fixture-name-to-message](#apply-fixture-name-to-message);
 - ✅ [apply-insert-after](#apply-insert-after);
 - ✅ [apply-insert-before](#apply-insert-before);
+- ✅ [apply-vars](#apply-vars);
 - ✅ [apply-namespace-specifier](#apply-namespace-specifier);
 - ✅ [apply-processors-destructuring](#apply-processors-destructuring);
 - ✅ [apply-remove](#apply-remove);
@@ -92,6 +93,7 @@ npm i @putout/plugin-putout -D
         "putout/apply-remove": "on",
         "putout/apply-insert-before": "on",
         "putout/apply-insert-after": "on",
+        "putout/apply-vars": "on",
         "putout/apply-short-processors": "on",
         "putout/apply-namespace-specifier": "on",
         "putout/apply-for-of-to-track-file": "on",
@@ -283,6 +285,42 @@ export const fix = (path) => {
 };
 ```
 
+## apply-vars
+
+Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/f96aaef6c54049ca91497a17f7a4b88a/b960916e73a7f51c1c0425ad9a3f304452c0d159).
+
+### ❌ Example of incorrect code
+
+```js
+export const replace = () => ({
+    '__a(__args': ({}, path) => {
+        return true;
+    },
+});
+
+export const match = () => ({
+    '__a(__args': ({}, path) => {
+        return '';
+    },
+});
+```
+
+### ✅ Example of correct code
+
+```js
+export const replace = () => ({
+    '__a(__args': (vars, path) => {
+        return true;
+    },
+});
+
+export const match = () => ({
+    '__a(__args': (vars, path) => {
+        return '';
+    },
+});
+```
+
 ## apply-declare
 
 Better to use [`Declareator`](https://github.com/coderaiser/putout/tree/master/packages/engine-runner#declarator) instead of `operator.declare()`.

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

@@ -33,7 +33,9 @@ module.exports.replace = () => ({
     't.noTransform(__a)': transform,
 });
 
-const isTest = (path) => compare(path, 'test(__a, (t) => __body)');
+const isTest = (path) => compare(path, 'test(__a, (t) => __body)', {
+    findUp: false,
+});
 
 const check = ({__a}, path) => {
     const name = __a.value;
@@ -41,20 +43,14 @@ const check = ({__a}, path) => {
     if (FIXTURE.includes(name))
         return false;
     
-    const testPath = path.find(isTest);
-    
-    if (!testPath)
-        return false;
-    
-    const [str] = testPath.parentPath.parentPath.node.arguments;
+    const str = getTestNodeArgument(name, path);
     
     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 str = getTestNodeArgument(name, path);
     
     const values = str.value.split(':');
     const last = values
@@ -68,3 +64,17 @@ const transform = ({__a}, path) => {
     
     return path;
 };
+
+const getTestNodeArgument = (value, path) => {
+    let testPath = path.find(isTest);
+    
+    if (!testPath)
+        return {
+            value,
+        };
+    
+    if (testPath.isExpressionStatement())
+        testPath = testPath.get('expression');
+    
+    return testPath.node.arguments[0];
+};

package/lib/apply-vars/index.js

@@ -0,0 +1,7 @@
+'use strict';
+
+module.exports.report = () => `Use 'var' instead of '{}'`;
+
+module.exports.replace = () => ({
+    '({}, path) => __body': '(vars, path) => __body',
+});

package/lib/index.js

@@ -57,6 +57,7 @@ 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');
+const applyVars = require('./apply-vars');
 
 module.exports.rules = {
     'apply-processors-destructuring': applyProcessorsDestructuring,
@@ -116,4 +117,5 @@ module.exports.rules = {
     'remove-useless-printer-option': removeUselessPrinterOption,
     'add-path-arg-to-visitors': addPathArgToVisitors,
     'apply-fixture-name-to-message': applyFixtureNameToMessage,
+    'apply-vars': applyVars,
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-putout",
-  "version": "22.3.0",
+  "version": "22.5.0",
   "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin helps with plugins development",
@@ -39,8 +39,7 @@
   "keywords": [
     "putout",
     "putout-plugin",
-    "plugin",
-    "putout"
+    "plugin"
   ],
   "devDependencies": {
     "@putout/plugin-nodejs": "*",