@putout/plugin-putout 23.1.0 → 23.2.1

package/README.md

@@ -33,6 +33,7 @@ npm i @putout/plugin-putout -D
 - ✅ [apply-processors-destructuring](#apply-processors-destructuring);
 - ✅ [apply-remove](#apply-remove);
 - ✅ [apply-rename](#apply-rename);
+- ✅ [apply-parens](#apply-parens);
 - ✅ [apply-short-processors](#apply-short-processors);
 - ✅ [check-match](#check-match);
 - ✅ [check-replace-code](#check-replace-code);
@@ -92,6 +93,7 @@ npm i @putout/plugin-putout -D
         "putout/apply-declare": "on",
         "putout/apply-processors-destructuring": "on",
         "putout/apply-rename": "on",
+        "putout/apply-parens": "on",
         "putout/apply-remove": "on",
         "putout/apply-insert-before": "on",
         "putout/apply-insert-after": "on",
@@ -215,6 +217,35 @@ export const fix = ({path, from, to}) => {
 };
 ```
 
+## apply-parens
+
+Better use [`addParens(path)`](https://github.com/coderaiser/putout/tree/master/packages/operate#addparenspath-path) method of `operator` instead of using `path.node.extra` to have support of both printers:
+
+- ✅ `@putout/printer`;
+- ✅ `babel`;
+
+Check out in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/d325a84e26807d46b5ec14db179c38f4/8bcd22d6dbe880c0fd70992a1a36b9c609b53b22).
+
+### ❌ Example of incorrect code
+
+```js
+path.node.extra.parenthesized = false;
+path.node.extra.parenthesized = true;
+
+if (!path.node.extra?.parenthesized)
+    return;
+```
+
+### ✅ Example of correct code
+
+```js
+removeParens(path);
+addParens(path);
+
+if (!hasParens(path))
+    return;
+```
+
 ## apply-remove
 
 Better to use [`remove(path)`](https://github.com/coderaiser/putout/tree/master/packages/operate#removepath) method of `operator`.

package/lib/apply-fixture-name-to-message/index.js

@@ -57,7 +57,7 @@ const check = ({__a}, path) => {
     const regEnd = RegExp(`: ${name}$`);
     const regMiddle = RegExp(`: ${name}: .*`);
     
-    return !(regEnd.test(value) || regMiddle.test(value));
+    return !regEnd.test(value) && !regMiddle.test(value);
 };
 
 const transform = ({__a}, path) => {

package/lib/apply-parens/index.js

@@ -0,0 +1,25 @@
+'use strict';
+
+module.exports.report = (path) => {
+    if (path.isOptionalMemberExpression())
+        return `Use 'hasParens(path)' instead of 'path.node.extra'`;
+    
+    const rightPath = path.get('right');
+    const {value} = rightPath.node;
+    const methodName = value ? 'add' : 'remove';
+    
+    return `Use '${methodName}Parens(path)' instead of 'path.node.extra'`;
+};
+
+const ASSIGN_OBJECT = `
+    __a.node.extra = {
+        parenthesized: true,
+    }
+`;
+
+module.exports.replace = () => ({
+    '__a.node.extra.parenthesized = false': 'removeParens(__a)',
+    '__a.node.extra.parenthesized = true': 'addParens(__a)',
+    '__a.node.extra?.parenthesized': 'hasParens(__a)',
+    [ASSIGN_OBJECT]: 'addParens(__a)',
+});

package/lib/declare/operator/index.js

@@ -3,11 +3,13 @@
 const filesystem = require('./filesystem');
 const json = require('./json');
 const keyword = require('./keyword');
+const parens = require('./parens');
 
 module.exports = {
     ...filesystem,
     ...json,
     ...keyword,
+    ...parens,
     operator: `import {operator} from 'putout'`,
     compare: 'const {compare} = operator',
     compareAll: 'const {compareAll} = operator',
@@ -40,6 +42,4 @@ module.exports = {
     setLiteralValue: 'const {setLiteralValue} = operator',
     matchFiles: 'const {matchFiles} = operator',
     ignore: 'const {ignore} = operator',
-    addParens: 'const {addParens} = operator',
-    removeParens: 'const {removeParens} = operator',
 };

package/lib/declare/operator/parens.js

@@ -0,0 +1,7 @@
+'use strict';
+
+module.exports = {
+    addParens: 'const {addParens} = operator',
+    removeParens: 'const {removeParens} = operator',
+    hasParens: 'const {hasParens} = operator',
+};

package/lib/index.js

@@ -60,6 +60,7 @@ const applyFixtureNameToMessage = require('./apply-fixture-name-to-message');
 const applyVars = require('./apply-vars');
 const declareTemplateVariables = require('./declare-template-variables');
 const declarePathVariable = require('./declare-path-variable');
+const applyParens = require('./apply-parens');
 
 module.exports.rules = {
     'apply-processors-destructuring': applyProcessorsDestructuring,
@@ -122,4 +123,5 @@ module.exports.rules = {
     'apply-vars': applyVars,
     'declare-template-variables': declareTemplateVariables,
     'declare-path-variable': declarePathVariable,
+    'apply-parens': applyParens,
 };

package/lib/simplify-replace-template/index.js

@@ -8,7 +8,7 @@ const parentNodesList = [
     'export const replace = __',
 ];
 
-module.exports.report = () => `Simplify replce template`;
+module.exports.report = () => `Simplify replace template`;
 
 module.exports.fix = (path) => {
     const {body} = path.node;

package/package.json

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