@putout/plugin-putout 20.10.0 → 21.1.0

package/README.md

@@ -16,6 +16,7 @@ npm i @putout/plugin-putout -D
 - ✅ [add-await-to-progress](#add-await-to-progress);
 - ✅ [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-test-args](#add-test-args);
 - ✅ [add-traverse-args](#add-traverse-args);
 - ✅ [add-track-file](#add-track-file);
@@ -71,6 +72,13 @@ npm i @putout/plugin-putout -D
 ```json
 {
     "rules": {
+        "putout/add-places-to-compare-places": "on",
+        "putout/add-path-arg-to-fix": "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",
@@ -82,12 +90,6 @@ 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/add-places-to-compare-places": "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/check-match": "on",
         "putout/check-replace-code": ["on", {
             "once": true
@@ -820,6 +822,26 @@ comparePlaces('hello');
 comparePlaces('hello', []);
 ```
 
+## add-path-arg-to-fix
+
+Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/3d3fb6b8f5e5376cdd1674c9b19e0e27/4872f77e680a2fc443618f4bf5fae3ffab11046a).
+
+### ❌ Example of incorrect code
+
+```js
+export const fix = () => {
+    path.remove();
+};
+```
+
+### ✅ Example of correct code
+
+```js
+export const fix = (path) => {
+    path.remove();
+};
+```
+
 ## add-test-args
 
 ### ❌ Example of incorrect code

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

@@ -0,0 +1,8 @@
+'use strict';
+
+module.exports.report = () => `Add 'path' argument to 'fix'`;
+
+module.exports.replace = () => ({
+    'const fix = () => __a': 'const fix = (path) => __a',
+    'module.exports.fix = () => {}': 'module.exports.fix = (path) => {}',
+});

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

@@ -22,12 +22,14 @@ module.exports.match = () => ({
         if (__aPath.isBlockStatement())
             return false;
         
-        for (const propertyPath of __aPath.get('properties')) {
-            if (isPush(propertyPath) || isBlock(propertyPath))
-                return true;
+        const properties = __aPath.get('properties');
+        
+        for (const propertyPath of properties) {
+            if (!isPush(propertyPath) && !isBlock(propertyPath))
+                return false;
         }
         
-        return false;
+        return true;
     },
 });
 
@@ -35,8 +37,9 @@ module.exports.replace = () => ({
     'module.exports.traverse = __a': (vars, path) => {
         const node = template.ast.fresh('module.exports.include = () => []');
         const __aPath = path.get('right.body');
+        const properties = __aPath.get('properties');
         
-        for (const propertyPath of __aPath.get('properties')) {
+        for (const propertyPath of properties) {
             const name = getName(propertyPath);
             
             if (isPush(propertyPath) || isBlock(propertyPath)) {

package/lib/index.js

@@ -52,6 +52,7 @@ const removeUnusedGetPropertiesArgument = require('./remove-unused-get-propertie
 const simplifyReplaceTemplate = require('./simplify-replace-template');
 const removeEmptyArrayFromProcess = require('./remove-empty-array-from-process');
 const addPlacesToComparePlaces = require('./add-places-to-compare-places');
+const addPathArgToFix = require('./add-path-arg-to-fix');
 
 module.exports.rules = {
     'apply-processors-destructuring': applyProcessorsDestructuring,
@@ -106,4 +107,5 @@ module.exports.rules = {
     'simplify-replace-template': simplifyReplaceTemplate,
     'remove-empty-array-from-process': removeEmptyArrayFromProcess,
     'add-places-to-compare-places': addPlacesToComparePlaces,
+    'add-path-arg-to-fix': addPathArgToFix,
 };

package/lib/move-require-on-top-level/index.js

@@ -1,7 +1,6 @@
 'use strict';
 
 const justCamelCase = require('just-camel-case');
-
 const {types, template} = require('putout');
 
 const TEST = `

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-putout",
-  "version": "20.10.0",
+  "version": "21.1.0",
   "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin helps with plugins development",
@@ -26,7 +26,7 @@
   },
   "dependencies": {
     "fullstore": "^3.0.0",
-    "just-camel-case": "^6.0.1",
+    "just-camel-case": "^6.2.0",
     "parse-import-specifiers": "^1.0.2",
     "try-catch": "^3.0.0"
   },
@@ -38,11 +38,11 @@
   ],
   "devDependencies": {
     "@putout/plugin-tape": "*",
-    "@putout/test": "^10.0.0",
+    "@putout/test": "^11.0.0",
     "c8": "^10.0.0",
     "eslint": "^9.0.0",
     "eslint-plugin-n": "^17.0.0",
-    "eslint-plugin-putout": "^22.0.0",
+    "eslint-plugin-putout": "^23.0.0",
     "lerna": "^6.0.1",
     "madrun": "^10.0.0",
     "montag": "^1.2.1",
@@ -50,7 +50,7 @@
     "supertape": "^10.0.0"
   },
   "peerDependencies": {
-    "putout": ">=35"
+    "putout": ">=36"
   },
   "license": "MIT",
   "engines": {