@putout/plugin-putout 22.8.0 → 22.10.0

package/README.md

@@ -62,6 +62,7 @@ npm i @putout/plugin-putout -D
 - ✅ [create-test](#create-test);
 - ✅ [declare](#declare);
 - ✅ [declare-template-variables](#declare-template-variables);
+- ✅ [declare-path-variable](#declare-path-variable);
 - ✅ [includer](#includer);
 - ✅ [move-require-on-top-level](#move-require-on-top-level);
 - ✅ [remove-empty-array-from-process](#remove-empty-array-from-process);
@@ -129,6 +130,7 @@ npm i @putout/plugin-putout -D
         "putout/shorten-imports": "on",
         "putout/declare": "on",
         "putout/declare-template-variables": "on",
+        "putout/declare-path-variable": "on",
         "putout/includer": "on",
         "putout/move-require-on-top-level": "on",
         "putout/replace-test-message": "on",
@@ -900,6 +902,36 @@ export const match = () => ({
 });
 ```
 
+## declare-path-variable
+
+Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/b485bf55785ea836408d0d45c492c3e4/2a0b465076242ebf6a0d8015a90ccf7b0b660a42).
+
+### ❌ Example of incorrect code
+
+```js
+export const match = () => ({
+    '__x __a __expr': ({__x}) => {
+        if (path.isNextPunctuator(assign))
+            return false;
+        
+        return isOneOfKeywords(__x, ['const', 'let', 'var']);
+    },
+});
+```
+
+### ✅ Example of correct code
+
+```js
+export const match = () => ({
+    '__x __a __expr': ({__x}) => {
+        if (path.isNextPunctuator(assign))
+            return false;
+        
+        return isOneOfKeywords(__x, ['const', 'let', 'var']);
+    },
+});
+```
+
 ## add-places-to-compare-places"
 
 ### ❌ Example of incorrect code

package/lib/declare-path-variable/index.js

@@ -0,0 +1,31 @@
+'use strict';
+
+const {operator} = require('putout');
+const {getBindingPath} = operator;
+
+module.exports.report = () => `Declare 'path' variable`;
+
+module.exports.match = () => ({
+    '(__a) => __body': (vars, path) => {
+        let is = false;
+        
+        path.traverse({
+            ReferencedIdentifier(refPath) {
+                if (refPath.node.name !== 'path')
+                    return;
+                
+                if (getBindingPath(refPath, 'path'))
+                    return;
+                
+                is = true;
+                path.stop();
+            },
+        });
+        
+        return is;
+    },
+});
+
+module.exports.replace = () => ({
+    '(__a) => __body': '(__a, path) => __body',
+});

package/lib/declare-template-variables/index.js

@@ -1,6 +1,7 @@
 'use strict';
 
-const {types} = require('putout');
+const {types, operator} = require('putout');
+const {getBindingPath} = operator;
 const {
     ObjectPattern,
     isIdentifier,
@@ -28,23 +29,29 @@ module.exports.fix = ({path, node}) => {
 };
 
 module.exports.traverse = ({push}) => ({
-    '(vars, path) => __body': (path) => {
-        path.traverse({
-            ReferencedIdentifier(refPath) {
-                const {node} = refPath;
-                const {name} = node;
-                
-                if (!name.startsWith('__'))
-                    return;
-                
-                if (refPath.scope.bindings[name])
-                    return;
-                
-                push({
-                    path,
-                    node,
-                });
-            },
-        });
-    },
+    '(__a, path) => __body': process(push),
+    '(__a) => __body': process(push),
 });
+
+const process = (push) => (path) => {
+    if (!path.parentPath.isObjectProperty())
+        return;
+    
+    path.traverse({
+        ReferencedIdentifier(refPath) {
+            const {node} = refPath;
+            const {name} = node;
+            
+            if (!name.startsWith('__'))
+                return;
+            
+            if (getBindingPath(refPath, name))
+                return;
+            
+            push({
+                path,
+                node,
+            });
+        },
+    });
+};

package/lib/index.js

@@ -59,6 +59,7 @@ const addPathArgToVisitors = require('./add-path-arg-to-visitors');
 const applyFixtureNameToMessage = require('./apply-fixture-name-to-message');
 const applyVars = require('./apply-vars');
 const declareTemplateVariables = require('./declare-template-variables');
+const declarePathVariable = require('./declare-path-variable');
 
 module.exports.rules = {
     'apply-processors-destructuring': applyProcessorsDestructuring,
@@ -120,4 +121,5 @@ module.exports.rules = {
     'apply-fixture-name-to-message': applyFixtureNameToMessage,
     'apply-vars': applyVars,
     'declare-template-variables': declareTemplateVariables,
+    'declare-path-variable': declarePathVariable,
 };

package/package.json

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