@putout/plugin-putout 13.0.0 → 13.2.0

package/README.md

@@ -19,8 +19,10 @@ 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/apply-declare": "on",
+        "putout/apply-remove": "on",
+        "putout/apply-insert-before": "on",
+        "putout/apply-insert-after": "on",
         "putout/add-args": "on",
         "putout/add-push": "on",
         "putout/check-match": "on",
@@ -96,6 +98,55 @@ export const fix = (path) => {
 };
 ```
 
+## apply-insert-before
+
+Better to use [`insertBefore(a, b)`](https://github.com/coderaiser/putout/tree/master/packages/operate#insert-before) method of `operator`.
+
+### ❌ Example of incorrect code
+
+```js
+export const fix = (path) => {
+    path.insertBefore(path.get('init'));
+};
+```
+
+### ✅ Example of correct code
+
+```js
+import {operator} from 'putout';
+
+const {insertBefore} = operator;
+
+export const fix = (path) => {
+    insertBefore(path, path.get('init'));
+};
+```
+
+## apply-insert-after
+
+Better to use [`insertAfter(a, b)`](https://github.com/coderaiser/putout/tree/master/packages/operate#insert-after) method of `operator`.
+It helps to avoid duplication of comments.
+
+### ❌ Example of incorrect code
+
+```js
+export const fix = (path) => {
+    path.insertAfter(path.get('init'));
+};
+```
+
+### ✅ Example of correct code
+
+```js
+import {operator} from 'putout';
+
+const {insertAfter} = operator;
+
+export const fix = (path) => {
+    insertAfter(path, path.get('init'));
+};
+```
+
 ## apply-declare
 
 Better to use [`Declareator`](https://github.com/coderaiser/putout/tree/master/packages/engine-runner#declarator) instead of `operator.declare()`.
@@ -218,7 +269,7 @@ isNumericLiteral(node);
 
 ## 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.
+Fixes results of [@putout/convert-commonjs-to-esm](https://github.com/coderaiser/putout/tree/master/packages/plugin-convert-commonjs-to-esm#readme) work.
 
 ### ❌ Example of incorrect code
 

package/lib/apply-insert-after/index.js

@@ -0,0 +1,7 @@
+'use strict';
+
+module.exports.report = () => `Use 'insertAfter(a, b)' instead of 'a.insertAfter(b)'`;
+
+module.exports.replace = () => ({
+    '__a.insertAfter(__b)': 'insertAfter(__a, __b)',
+});

package/lib/apply-insert-before/index.js

@@ -0,0 +1,7 @@
+'use strict';
+
+module.exports.report = () => `Use 'insertBefore(a, b)' instead of 'a.insertBefore(b)'`;
+
+module.exports.replace = () => ({
+    '__a.insertBefore(__b)': 'insertBefore(__a, __b)',
+});

package/lib/apply-remove/index.js

@@ -3,5 +3,5 @@
 module.exports.report = () => `Use 'remove(path)' instead of 'path.remove()'`;
 
 module.exports.replace = () => ({
-    'path.remove()': 'remove(path)',
+    '__a.remove()': 'remove(__a)',
 });

package/lib/convert-replace-with/index.js

@@ -9,7 +9,11 @@ const {
 const fullstore = require('fullstore');
 
 const {Identifier, ObjectProperty} = types;
-const {replaceWith, insertAfter} = operator;
+const {
+    replaceWith,
+    insertAfter,
+    insertBefore,
+} = operator;
 
 const isRecast = (program) => program.get('body.0.expression').isStringLiteral({
     value: 'use strict',
@@ -42,7 +46,7 @@ module.exports.fix = ({path, calleePath, property, object, program, isInserted})
         else if (types)
             insertAfter(pathToInsert, replaceWithAST);
         else
-            pathToInsert.insertBefore(replaceWithAST);
+            insertBefore(pathToInsert, replaceWithAST);
         
         isInserted(true);
         

package/lib/convert-replace-with-multiple/index.js

@@ -6,7 +6,12 @@ const {
     types,
 } = require('putout');
 
-const {insertAfter, replaceWith} = operator;
+const {
+    insertAfter,
+    replaceWith,
+    insertBefore,
+} = operator;
+
 const {Identifier, ObjectProperty} = types;
 
 const isRecast = (program) => program.get('body.0').get('expression')
@@ -36,7 +41,7 @@ module.exports.fix = ({path, calleePath, property, object, program}) => {
         if (isRecast(program))
             return insertAfter(first, replaceWithAST);
         
-        return first.insertBefore(replaceWithAST);
+        return insertBefore(first, replaceWithAST);
     }
     
     const id = Identifier('replaceWithMultiple');

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

@@ -7,7 +7,7 @@ const {
 } = require('putout');
 
 const {StringLiteral} = types;
-const {compare} = operator;
+const {compare, remove} = operator;
 
 const isPush = (path) => path.get('value').isIdentifier({
     name: 'push',
@@ -42,7 +42,7 @@ module.exports.replace = () => ({
             
             if (isPush(propertyPath) || isBlock(propertyPath)) {
                 node.right.body.elements.push(StringLiteral(name));
-                propertyPath.remove();
+                remove(propertyPath);
             }
         }
         

package/lib/declare/operator.json

@@ -0,0 +1,27 @@
+{
+    "operator": "import {operator} from 'putout'",
+    "compare": "const {compare} = operator",
+    "compareAll": "const {compareAll} = operator",
+    "compareAny": "const {compareAny} = operator",
+    "compute": "const {compute} = operator",
+    "contains": "const {contains} = operator",
+    "declare": "const {declare} = operator",
+    "rename": "const {rename} = operator",
+    "renameProperty": "const {renameProperty} = operator",
+    "extract": "const {extract} = operator",
+    "getPathAfterImports": "const {getPathAfterImports} = operator",
+    "traverse": "const {traverse} = operator",
+    "isSimpleRegExp": "const {isSimpleRegExp} = operator",
+    "getTemplateValues": "const {getTemplateValues} = operator",
+    "addArgs": "const {addArgs} = operator",
+    "insertBefore": "const {insertBefore} = operator",
+    "insertAfter": "const {insertAfter} = 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",
+    "isSimple": "const {isSimple} = operator",
+    "setLiteralValue": "const {setLiteralValue} = operator"
+}

package/lib/index.js

@@ -9,6 +9,8 @@ module.exports.rules = {
     ...getRule('apply-async-formatter'),
     ...getRule('apply-create-test'),
     ...getRule('apply-remove'),
+    ...getRule('apply-insert-before'),
+    ...getRule('apply-insert-after'),
     ...getRule('apply-declare'),
     ...getRule('check-replace-code'),
     ...getRule('check-match'),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-putout",
-  "version": "13.0.0",
+  "version": "13.2.0",
   "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin helps with plugins development",
@@ -44,7 +44,7 @@
     "lerna": "^6.0.1",
     "madrun": "^9.0.0",
     "montag": "^1.2.1",
-    "nodemon": "^2.0.1"
+    "nodemon": "^3.0.1"
   },
   "peerDependencies": {
     "putout": ">=30"

package/lib/declare/operator.js

@@ -1,26 +0,0 @@
-'use strict';
-
-module.exports = {
-    operator: `import {operator} from 'putout'`,
-    compare: `const {compare} = operator`,
-    compareAll: `const {compareAll} = operator`,
-    compareAny: `const {compareAny} = operator`,
-    compute: `const {compute} = operator`,
-    contains: `const {contains} = operator`,
-    declare: `const {declare} = operator`,
-    rename: `const {rename} = operator`,
-    renameProperty: `const {renameProperty} = operator`,
-    extract: `const {extract} = operator`,
-    getPathAfterImports: `const {getPathAfterImports} = operator`,
-    traverse: `const {traverse} = operator`,
-    isSimpleRegExp: `const {isSimpleRegExp} = operator`,
-    getTemplateValues: `const {getTemplateValues} = 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`,
-    isSimple: `const {isSimple} = operator`,
-};