@putout/plugin-putout 13.0.0 → 13.1.0

package/README.md

@@ -19,8 +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/apply-declare": "on",
+        "putout/apply-remove": "on",
+        "putout/apply-insert-after": "on",
         "putout/add-args": "on",
         "putout/add-push": "on",
         "putout/check-match": "on",
@@ -96,6 +97,31 @@ export const fix = (path) => {
 };
 ```
 
+## 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 +244,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-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-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.js

@@ -16,6 +16,7 @@ module.exports = {
     isSimpleRegExp: `const {isSimpleRegExp} = operator`,
     getTemplateValues: `const {getTemplateValues} = operator`,
     addArgs: `const {addArgs} = operator`,
+    insertAfter: 'const {insertAfter} = operator',
     replaceWith: `const {replaceWith} = operator`,
     replaceWithMultiple: `const {replaceWithMultiple} = operator`,
     remove: 'const {remove} = operator',

package/lib/index.js

@@ -9,6 +9,7 @@ module.exports.rules = {
     ...getRule('apply-async-formatter'),
     ...getRule('apply-create-test'),
     ...getRule('apply-remove'),
+    ...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.1.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"