@putout/plugin-putout 11.0.0 → 11.1.1

package/README.md

@@ -19,7 +19,9 @@ npm i @putout/plugin-putout -D
         "putout/apply-create-test": "on",
         "putout/apply-processors-destructuring": "on",
         "putout/apply-async-formatter": "on",
+        "putout/apply-remove": "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",
@@ -64,6 +66,30 @@ test('', async ({process}) => {
 });
 ```
 
+## apply-remove
+
+Better to use [`remove(path)`](https://github.com/coderaiser/putout/tree/master/packages/operate#removepath) method of `operator`.
+It helps to preserve comments.
+
+### ❌ Example of incorrect code
+
+```js
+export const fix = (path) => {
+    path.remove();
+};
+```
+
+### ✅ Example of correct code
+
+```js
+import {operator} from 'putout';
+const {remove} = operator;
+
+export const fix = (path) => {
+    remove(path);
+};
+```
+
 ## apply-async-formatter
 
 ### ❌ Example of incorrect code
@@ -102,7 +128,7 @@ const test = createTest({
 });
 ```
 
-## convert-putout-test-to-create-test"
+## convert-putout-test-to-create-test
 
 Fixes results of [@putout/convert-commonjs-to-esm](https://github.com/coderaiser/putout/tree/master/packages/plugin-convert-commonjs-to-esm) work.
 
@@ -119,7 +145,7 @@ const test = putoutTest(__dirname, {
 ### ✅ Example of correct code
 
 ```js
-import createTest from '@putout/test';
+import {createTest} from '@putout/test';
 
 const test = createTest(__dirname, {
     'remove-unused-variables': rmVars,
@@ -435,8 +461,32 @@ 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
 
+### ❌ Example of incorrect code
+
 ```js
 const {operator} = require('putout');
 const {addArgument} = operator;
@@ -459,6 +509,8 @@ module.exports = addArgs({
 
 ## convert-dirname-to-url
 
+### ❌ Example of incorrect code
+
 ```js
 import {createTest} from '@putout/test';
 import plugin from '@putout/plugin-debugger';
@@ -483,11 +535,13 @@ const test = createTest(import.meta.url, {
 
 ## convert-url-to-dirname-
 
+### ❌ Example of incorrect code
+
 ```js
 const {createTest} = require('@putout/test');
 const plugin = require('@putout/plugin-debugger');
 
-const test = createTest(import.meta.url, {
+const test = createTest(__dirname, {
     'remove-debugger': plugin,
 });
 ```

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/apply-remove/index.js

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

package/lib/convert-putout-test-to-create-test/index.js

@@ -1,5 +1,7 @@
 'use strict';
 
+const {assign} = Object;
+
 module.exports.report = () => `Use 'createTest' instead of 'putoutTest'`;
 
 module.exports.filter = ({scope}) => !scope.bindings.createTest;
@@ -9,6 +11,14 @@ module.exports.include = () => [
 ];
 
 module.exports.fix = (path) => {
+    const [first] = path.node.specifiers;
+    
+    assign(first, {
+        type: 'ImportSpecifier',
+        kind: 'value',
+        imported: first.local,
+    });
+    
     path.scope.rename('putoutTest', 'createTest');
 };
 

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/declare/operator.js

@@ -10,9 +10,10 @@ module.exports = {
     declare: `const {declare} = operator`,
     isSimpleRegExp: `const {isSimpleRegExp} = operator`,
     getTemplateValues: `const {getTemplateValues} = operator`,
-    addArgument: `const {addArgument} = operator`,
+    addArgs: `const {addArgs} = operator`,
     replaceWith: `const {replaceWith} = operator`,
     replaceWithMultiple: `const {replaceWithMultiple} = operator`,
+    remove: 'const {remove} = operator',
     isESM: `const {isESM} = operator`,
     getProperty: `const {getProperty} = operator`,
     getProperties: `const {getProperties} = operator`,

package/lib/index.js

@@ -8,6 +8,7 @@ module.exports.rules = {
     ...getRule('apply-processors-destructuring'),
     ...getRule('apply-async-formatter'),
     ...getRule('apply-create-test'),
+    ...getRule('apply-remove'),
     ...getRule('convert-putout-test-to-create-test'),
     ...getRule('convert-to-no-transform-code'),
     ...getRule('convert-find-to-traverse'),
@@ -31,6 +32,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,9 +1,9 @@
 {
   "name": "@putout/plugin-putout",
-  "version": "11.0.0",
+  "version": "11.1.1",
   "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
-  "description": "putout plugin helps with plugins development",
+  "description": "🐊Putout plugin helps with plugins development",
   "homepage": "https://github.com/coderaiser/putout/tree/master/packages/plugin-putout#readme",
   "main": "lib/index.js",
   "release": false,
@@ -40,7 +40,7 @@
     "c8": "^7.5.0",
     "eslint": "^8.0.1",
     "eslint-plugin-node": "^11.0.0",
-    "eslint-plugin-putout": "^14.0.0",
+    "eslint-plugin-putout": "^15.0.0",
     "lerna": "^4.0.0",
     "madrun": "^9.0.0",
     "montag": "^1.2.1",