feature: @putout/plugin-apply-consistent-blocks: add

Author: coderaiser

README.md

@@ -1254,28 +1254,26 @@ function world(a) {
 
 </details>
 
-<details><summary>convert <code>esm</code> to <code>commonjs</code> (disabled)</summary>
-
-```diff
--import hello from 'world';
-+const hello = require('world');
-```
-
-</details>
-<details><summary>convert <code>commonjs</code> to <code>esm</code> (disabled)</summary>
+<details><summary>convert <code>replace</code> to <code>replaceAll</code> (<a href=https://github.com/tc39/proposal-string-replaceall>stage-4</a>)</summary>
 
 ```diff
--const hello = require('world');
-+import hello from 'world';
+-'hello'.replace(/hello/g, 'world');
++'hello'.replaceAll('hello', 'world');
 ```
 
 </details>
 
-<details><summary>convert <code>replace</code> to <code>replaceAll</code> (<a href=https://github.com/tc39/proposal-string-replaceall>stage-4</a>)</summary>
+<details><summary>apply consistent-blocks</summary>
 
 ```diff
--'hello'.replace(/hello/g, 'world');
-+'hello'.replaceAll('hello', 'world');
+-if (a)
++if (a) {
+    b();
++} else {
+-else {
+    c();
+    d();
+}
 ```
 
 </details>
@@ -1740,6 +1738,24 @@ async () => {
 
 ### Node.js
 
+<details><summary>convert <code>esm</code> to <code>commonjs</code> (disabled)</summary>
+
+```diff
+-import hello from 'world';
++const hello = require('world');
+```
+
+</details>
+
+<details><summary>convert <code>commonjs</code> to <code>esm</code> (disabled)</summary>
+
+```diff
+-const hello = require('world');
++import hello from 'world';
+```
+
+</details>
+
 <details><summary>convert <code>fs.promises</code> to <code>fs/promises</code> for <a href=https://nodejs.org/dist/latest-v15.x/docs/api/fs.html#fs_fs_promises_api>node.js</a></summary>
 
 ```diff
@@ -1904,6 +1920,7 @@ It has a lot plugins divided by groups:
 
 | Package | Version |
 |--------|-------|
+| [`@putout/plugin-apply-consistent-blocks`](/packages/plugin-apply-consistent-blocks#readme) | [![npm](https://img.shields.io/npm/v/@putout/plugin-apply-consistent-blocks.svg?maxAge=86400)](https://www.npmjs.com/package/@putout/plugin-apply-consistent-blocks) |
 | [`@putout/plugin-apply-at`](/packages/plugin-apply-at#readme) | [![npm](https://img.shields.io/npm/v/@putout/plugin-apply-at.svg?maxAge=86400)](https://www.npmjs.com/package/@putout/plugin-apply-at) |
 | [`@putout/plugin-apply-dot-notation`](/packages/plugin-apply-dot-notation#readme) | [![npm](https://img.shields.io/npm/v/@putout/plugin-apply-dot-notation.svg?maxAge=86400)](https://www.npmjs.com/package/@putout/plugin-apply-dot-notation) |
 | [`@putout/plugin-apply-starts-with`](/packages/plugin-apply-starts-with#readme) | [![npm](https://img.shields.io/npm/v/@putout/plugin-apply-starts-with.svg?maxAge=86400)](https://www.npmjs.com/package/@putout/plugin-apply-starts-with) |

packages/cli-cache/lib/cache.js

@@ -114,9 +114,8 @@ const hash = (a) => murmur(a).result()
     .toString(36);
 
 const createGetOptionsCache = ({version}) => (options) => {
-    if (!optionsHashCache.has(options)) {
+    if (!optionsHashCache.has(options))
         optionsHashCache.set(options, hash(`${version}_${nodeVersion}_${stringify(options)}`));
-    }
 
     return optionsHashCache.get(options);
 };

packages/cli-cache/lib/is-no-definition.js

@@ -4,9 +4,8 @@ module.exports = (places) => {
     for (const {rule, message} of places) {
         const shortRule = rule.replace('eslint/', '');
 
-        if (message === `Definition for rule '${shortRule}' was not found.`) {
+        if (message === `Definition for rule '${shortRule}' was not found.`)
             return true;
-        }
     }
 
     return false;

packages/compare/lib/comparators.js

@@ -73,9 +73,8 @@ module.exports.runComparators = (node, template, {add, templateStore}) => {
     while (++i < n) {
         const compare = comparators[i];
 
-        if (compare(node, template, {add, templateStore})) {
+        if (compare(node, template, {add, templateStore}))
             return true;
-        }
     }
 
     return false;

packages/compare/lib/vars.js

@@ -102,9 +102,8 @@ function getValues({waysFrom, node}) {
             else if (isArgsStr(name) || isJSXChildrenStr(name) || isJSXAttributesStr(name))
                 way = way.replace(/\.0$/, '');
 
-            if (!isJSXElement(node)) {
+            if (!isJSXElement(node))
                 way = way.replace(/\.expression$/, '');
-            }
 
             result[name] = result[name] || parseExpression(jessy(way, node));
         }

packages/engine-loader/lib/processors/parse-processor-names.js

@@ -18,9 +18,8 @@ module.exports = (plugins) => {
         if (isArray(plugin)) {
             const [pluginName, fn] = plugin;
 
-            if (isOff(fn)) {
+            if (isOff(fn))
                 continue;
-            }
 
             if (isOn(fn)) {
                 result.push([pluginName]);

packages/engine-loader/test/load-plugins.js

@@ -32,9 +32,8 @@ test('putout: loader: user plugin', (t) => {
         if (!name.indexOf(`@putout/plugin-${rmVars}`))
             return false;
 
-        if (name === `putout-plugin-${rmVars}`) {
+        if (name === `putout-plugin-${rmVars}`)
             return name;
-        }
 
         return _findPath(name, paths);
     });

packages/engine-runner/lib/replace/index.js

@@ -149,9 +149,8 @@ const getFilter = (match = stubMatch, options) => (path) => {
     for (const [from, matchProperty] of all) {
         const nodeFrom = template.ast(from);
 
-        if (!compare(path.node, nodeFrom)) {
+        if (!compare(path.node, nodeFrom))
             continue;
-        }
 
         const waysFrom = findVarsWays(nodeFrom);
         const {node} = path;

packages/engine-runner/lib/replace/watermark.js

@@ -41,9 +41,8 @@ function create(from, to, path) {
 
 module.exports.init = init;
 function init({path, program}) {
-    if (path.node) {
+    if (path.node)
         path.node[name] = path.node[name] || new Set();
-    }
 
     program.node[name] = program.node[name] || new Set();
 }

packages/engine-runner/lib/run-fix.js

@@ -20,9 +20,8 @@ const tryToFix = (fix, {path, position, options}) => {
             .getProgramParent()
             .crawl();
 
-    if (!e) {
+    if (!e)
         return;
-    }
 
     e.loc = e.loc || position;
 

packages/eslint-plugin-putout/lib/putout/parse-error.js

@@ -4,9 +4,8 @@ module.exports.parseError = ({code, message, name}) => {
     if (message.includes('buildError'))
         return 'Parser error';
 
-    if (code === 'ERR_REQUIRE_ESM') {
+    if (code === 'ERR_REQUIRE_ESM')
         return `☝️ Looks like '${name}' is ESM, extend from 'plugin:putout/esm'`;
-    }
 
     return message;
 };

packages/eslint-plugin-putout/lib/single-property-destructuring/index.js

@@ -48,9 +48,8 @@ module.exports.filter = ({node, text, getText, getCommentsInside}) => {
 };
 
 module.exports.fix = ({text, node, getText}) => {
-    if (isImportDeclaration(node)) {
+    if (isImportDeclaration(node))
         return text.replace(NewLinesReg, '');
-    }
 
     const {id} = node;
     const idText = getText(id);

packages/eslint/lib/create-plugin/index.js

@@ -146,9 +146,8 @@ function isLastNodeInBody(node) {
         const {length} = node.parent.body;
         const n = length - 1;
 
-        if (node === node.parent?.body?.[n]) {
+        if (node === node.parent?.body?.[n])
             return true;
-        }
     }
 
     return false;

packages/operate/lib/compute.js

@@ -81,9 +81,8 @@ function isExtractable(path) {
     if (!path.isIdentifier() && !path.isLiteral())
         return false;
 
-    if (parentPath.isObjectProperty({computed})) {
+    if (parentPath.isObjectProperty({computed}))
         return !usedInAssignment(path);
-    }
 
     return false;
 }

packages/operate/lib/remove.js

@@ -40,9 +40,8 @@ const getComments = (path) => {
 
     const {parentPath} = path;
 
-    if (path.isVariableDeclarator() && isOneDeclaration(parentPath)) {
+    if (path.isVariableDeclarator() && isOneDeclaration(parentPath))
         return parentPath.node.comments;
-    }
 
     return [];
 };

packages/operator-filesystem/lib/filesystem.js

@@ -41,9 +41,8 @@ module.exports.findFile = (node, name) => {
     for (const filenamePath of traverseProperties(node, 'filename')) {
         const {value} = filenamePath.node.value;
 
-        if (value === name || getRegExp(name).test(value)) {
+        if (value === name || getRegExp(name).test(value))
             filePaths.push(filenamePath.parentPath);
-        }
     }
 
     return filePaths;

packages/plugin-apply-at/lib/apply-at.js

@@ -10,9 +10,8 @@ module.exports.filter = (path) => {
     if (parentPath.isAssignmentExpression())
         return false;
 
-    if (compare(path, '__a[length - __b]')) {
+    if (compare(path, '__a[length - __b]'))
         return compare(path.parentPath.parentPath.getPrevSibling(), 'const {length} = __a');
-    }
 
     return true;
 };

packages/plugin-apply-consistent-blocks/.eslintrc.json

@@ -0,0 +1,10 @@
+{
+    "extends": [
+        "plugin:n/recommended",
+        "plugin:putout/recommended"
+    ],
+    "plugins": [
+        "n",
+        "putout"
+    ]
+}

packages/plugin-apply-consistent-blocks/.gitignore

@@ -0,0 +1,6 @@
+node_modules
+*.swp
+yarn-error.log
+
+coverage
+.idea

packages/plugin-apply-consistent-blocks/.madrun.mjs

@@ -0,0 +1,12 @@
+import {run} from 'madrun';
+
+export default {
+    'test': () => `tape 'lib/**/*.spec.js' 'test/*.js'`,
+    'watch:test': async () => `nodemon -w lib -w test -x ${await run('test')}`,
+    'lint': () => `putout .`,
+    'fresh:lint': () => run('lint', '--fresh'),
+    'lint:fresh': () => run('lint', '--fresh'),
+    'fix:lint': () => run('lint', '--fix'),
+    'coverage': async () => `c8 ${await run('test')}`,
+    'report': () => 'c8 report --reporter=lcov',
+};

packages/plugin-apply-consistent-blocks/.npmignore

@@ -0,0 +1,7 @@
+.*
+test
+yarn-error.log
+fixture
+*.spec.js
+
+coverage

packages/plugin-apply-consistent-blocks/.nycrc.json

@@ -0,0 +1,14 @@
+{
+    "check-coverage": true,
+    "all": true,
+    "exclude": [
+        "**/*.spec.*",
+        "**/fixture",
+        "test",
+        ".*.*"
+    ],
+    "branches": 100,
+    "lines": 100,
+    "functions": 100,
+    "statements": 100
+}

packages/plugin-apply-consistent-blocks/.putout.json

@@ -0,0 +1,8 @@
+{
+    "printer": "putout",
+    "match": {
+        "README.md": {
+            "apply-at": "off"
+        }
+    }
+}

packages/plugin-apply-consistent-blocks/LICENSE

@@ -0,0 +1,21 @@
+The MIT License (MIT)
+
+Copyright (c) coderaiser
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.