@putout/plugin-putout 11.0.1 → 11.1.0

package/README.md

@@ -20,6 +20,7 @@ npm i @putout/plugin-putout -D
         "putout/apply-processors-destructuring": "on",
         "putout/apply-async-formatter": "on",
         "putout/add-args": "on",
+        "putout/add-push": "on",
         "putout/convert-putout-test-to-create-test": "on",
         "putout/convert-to-no-transform-code": "on",
         "putout/convert-replace-with": "on",
@@ -435,6 +436,28 @@ test('', ({comparePlaces}) => {
 });
 ```
 
+## add-push
+
+### ❌ Example of incorrect code
+
+```js
+module.exports.traverse = () => ({
+    '__a.replace(/__b/g, __c)': (path) => {
+        push(path);
+    },
+});
+```
+
+### ✅ Example of correct code
+
+```js
+module.exports.traverse = ({push}) => ({
+    '__a.replace(/__b/g, __c)': (path) => {
+        push(path);
+    },
+});
+```
+
 ## convert-add-argument-to-add-args
 
 ```js

package/lib/add-push/index.js

@@ -0,0 +1,41 @@
+'use strict';
+
+const {
+    types,
+    operator,
+} = require('putout');
+
+const {traverse} = operator;
+
+const {
+    ObjectProperty,
+    ObjectPattern,
+    Identifier,
+} = types;
+
+module.exports.report = () => `Add 'push' argument to 'traverse'`;
+
+module.exports.fix = (path) => {
+    const computed = false;
+    const shorthand = true;
+    const name = Identifier('push');
+    
+    path.node.right.params.push(ObjectPattern([
+        ObjectProperty(name, name, computed, shorthand),
+    ]));
+};
+
+module.exports.traverse = ({push}) => ({
+    'module.exports.traverse = (__args) => __': (traversePath) => {
+        const paramsPaths = traversePath.get('right.params');
+        
+        if (paramsPaths.length)
+            return;
+        
+        traverse(traversePath, {
+            'push(__)': () => {
+                push(traversePath);
+            },
+        });
+    },
+});

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

@@ -1,7 +1,10 @@
 'use strict';
 
 const {operator} = require('putout');
-const {contains} = operator;
+const {
+    contains,
+    traverse,
+} = operator;
 
 module.exports.report = () => 'Replacer should be used instead of Traverser (https://git.io/JqcMn)';
 
@@ -15,6 +18,9 @@ module.exports.match = () => ({
         if (withFix)
             return false;
         
+        if (hasPushCall(path))
+            return false;
+        
         if (!__args.length)
             return true;
         
@@ -22,6 +28,9 @@ module.exports.match = () => ({
             'push',
         ]);
         
+        if (withPush)
+            return false;
+        
         return !withPush;
     },
 });
@@ -30,3 +39,16 @@ module.exports.replace = () => ({
     'module.exports.traverse = (__args) => __a': 'module.exports.replace = (__args) => __a',
 });
 
+function hasPushCall(path) {
+    let is = false;
+    
+    traverse(path, {
+        'push(__a)': (path) => {
+            is = true;
+            path.stop();
+        },
+    });
+    
+    return is;
+}
+

package/lib/index.js

@@ -31,6 +31,7 @@ module.exports.rules = {
     ...getRule('check-replace-code'),
     ...getRule('declare'),
     ...getRule('add-args'),
+    ...getRule('add-push'),
     ...getRule('move-require-on-top-level'),
     ...getRule('includer'),
 };

package/package.json

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