feature: @putout/plugin-parens: remove-useless: add

Author: coderaiser

packages/plugin-parens/.eslintrc.json

@@ -4,6 +4,7 @@
         "rules": {
             "@typescript-eslint/no-unused-expressions": "off",
             "no-unsafe-optional-chaining": "off",
+            "@stylistic/ts/no-extra-parens": "off",
             "@typescript-eslint/no-non-null-asserted-optional-chain": "off"
         }
     }],

packages/plugin-parens/README.md

@@ -20,12 +20,15 @@ npm i @putout/plugin-parens
 ```json
 {
     "rules": {
-        "parens/add-missing": "on"
+        "parens/add-missing": "on",
+        "parens/remove-useless": "on"
     }
 }
 ```
 
-## assign
+## add-missing
+
+### assign
 
 To disable use:
 
@@ -37,14 +40,12 @@ To disable use:
 }
 ```
 
-## ❌ Example of incorrect code
-
 ```diff
 -a && b = a;
 +a && (b = a);
 ```
 
-## await
+### await
 
 To disable use:
 
@@ -56,19 +57,19 @@ To disable use:
 }
 ```
 
-## ❌ Example of incorrect code
+#### ❌ Example of incorrect code
 
 ```ts
 await asyncFn().filter(Boolean);
 ```
 
-## ✅ Example of correct code
+#### ✅ Example of correct code
 
 ```js
 (await asyncFn()).filter(Boolean);
 ```
 
-## template
+#### template
 
 Checkout in 🐊[**Putout Editor**](https://putout.vercel.app/#/gist/ef3f1e198a8d5ebeb9dd3fd1fef8f305/c6b46a34037f5cb095b5419b748a24b6dc8e2933).
 
@@ -80,22 +81,40 @@ Checkout in 🐊[**Putout Editor**](https://putout.vercel.app/#/gist/ef3f1e198a8
 }
 ```
 
-## ❌ Example of incorrect code
+#### ❌ Example of incorrect code
 
 ```ts
 getConsoleLog?.()``;
 String?.raw``;
 String?.raw!``;
 ```
 
-## ✅ Example of correct code
+#### ✅ Example of correct code
 
 ```ts
 (getConsoleLog?.())``;
 (String?.raw)``;
 (String?.raw)!``;
 ```
 
+## remove-useless
+
+### await
+
+Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/3800b0c52a199dd49a089ed4b9b37566/e2dddb75cb68811883cae640a22b340b8e1afa73).
+
+#### ❌ Example of incorrect code
+
+```ts
+const s = (await m());
+```
+
+#### ✅ Example of correct code
+
+```ts
+const s = await m();
+```
+
 ## License
 
 MIT

packages/plugin-parens/lib/add-missing-for-await/index.js

@@ -5,6 +5,7 @@ const {
     types,
     operator,
 } = require('putout');
+
 const {AwaitExpression} = types;
 const {replaceWith} = operator;
 

packages/plugin-parens/lib/index.js

@@ -3,9 +3,11 @@
 const addMissingForAwait = require('./add-missing-for-await');
 const addMissingForTemplate = require('./add-missing-for-template');
 const addMissingForAssign = require('./add-missing-for-assign');
+const removeUselessForAwait = require('./remove-useless-for-await');
 
 module.exports.rules = {
     'add-missing-for-awai': addMissingForAwait,
     'add-missing-for-template': addMissingForTemplate,
     'add-missing-for-assign': addMissingForAssign,
+    'remove-useless-for-await': removeUselessForAwait,
 };

packages/plugin-parens/lib/remove-useless-for-await/fixture/remove-useless-for-await-fix.js

@@ -0,0 +1 @@
+var s = await m();

packages/plugin-parens/lib/remove-useless-for-await/fixture/remove-useless-for-await.js

@@ -0,0 +1,2 @@
+var s = (await m());
+

packages/plugin-parens/lib/remove-useless-for-await/index.js

@@ -0,0 +1,16 @@
+'use strict';
+
+module.exports.report = () => `Remove useless parens around 'await'`;
+
+module.exports.fix = (path) => {
+    path.node.extra = {
+        parenthesized: false,
+    };
+};
+
+module.exports.traverse = ({push}) => ({
+    AwaitExpression(path) {
+        if (path.parentPath.isVariableDeclarator())
+            push(path);
+    },
+});

packages/plugin-parens/lib/remove-useless-for-await/index.spec.js

@@ -0,0 +1,21 @@
+'use strict';
+
+const {createTest} = require('@putout/test');
+const plugin = require('.');
+
+const test = createTest(__dirname, {
+    printer: 'putout',
+    plugins: [
+        ['remove-useless-for-await', plugin],
+    ],
+});
+
+test('parens: remove-useless-for-await: report', (t) => {
+    t.report('remove-useless-for-await', `Remove useless parens around 'await'`);
+    t.end();
+});
+
+test('parens: remove-useless-for-await: transform', (t) => {
+    t.transform('remove-useless-for-await');
+    t.end();
+});

packages/plugin-parens/test/fixture/remove-useless-for-await-fix.js

@@ -0,0 +1 @@
+var s = await m();

packages/plugin-parens/test/fixture/remove-useless-for-await.js

@@ -0,0 +1,2 @@
+var s = (await m());
+