@putout/plugin-conditions 6.0.0 → 6.2.0

package/README.md

@@ -22,6 +22,7 @@ npm i @putout/plugin-conditions -D
 - ✅ [convert-arrow-to-condition](#convert-arrow-to-condition);
 - ✅ [evaluate](#evaluate);
 - ✅ [merge-if-statements](#merge-if-statements);
+- ✅ [reverse](#reverse);
 - ✅ [remove-boolean](#remove-boolean);
 - ✅ [remove-constant](#remove-constant);
 - ✅ [remove-same-values-condition](hremove-same-values-condition);
@@ -42,6 +43,7 @@ npm i @putout/plugin-conditions -D
         "conditions/convert-equal-to-strict-equal": "on",
         "conditions/convert-arrow-to-condition": "on",
         "conditions/evaluate": "on",
+        "conditions/reverse": "on",
         "conditions/remove-boolean": "on",
         "conditions/remove-constant": "on",
         "conditions/remove-zero": "on",
@@ -245,6 +247,22 @@ const c = a;
 console.log(a);
 ```
 
+## reverse
+
+Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/fabfc16a5a5d1002c721fc2dfc31474b/8d1de98533ef439e56e8784f54a34c3928a074fa).
+
+### ❌ Example of incorrect code
+
+```js
+const check = (references) => !(references > 3);
+```
+
+### ✅ Example of correct code
+
+```js
+const check = (references) => references <= 3;
+```
+
 ## remove-boolean
 
 ### ❌ Example of incorrect code

package/lib/apply-consistent-blocks/index.js

@@ -5,6 +5,8 @@ const {replaceWith} = operator;
 const {
     isBlockStatement,
     BlockStatement,
+    isExpressionStatement,
+    isIfStatement,
 } = types;
 
 module.exports.report = () => `Use consistent blocks`;
@@ -63,11 +65,20 @@ module.exports.filter = (path) => {
     if (path === path.parentPath.get('alternate'))
         return false;
     
-    const nodes = getAllNodes(path);
+    const paths = getAllNodes(path);
     const blocks = [];
     
-    for (const node of nodes) {
-        const is = isBlockStatement(node);
+    for (const path of paths) {
+        const is = isBlockStatement(path);
+        const {body} = path.node;
+        
+        if (is && body.length === 1) {
+            const [first] = body;
+            
+            if (!isExpressionStatement(first) && !isIfStatement(first))
+                continue;
+        }
+        
         blocks.push(is);
     }
     
@@ -78,8 +89,8 @@ module.exports.filter = (path) => {
     
     const couple = [];
     
-    for (const node of nodes) {
-        const is = isBlockStatement(node) && node.node.body.length > 1;
+    for (const path of paths) {
+        const is = isBlockStatement(path) && path.node.body.length > 1;
         couple.push(is);
     }
     
@@ -88,10 +99,10 @@ module.exports.filter = (path) => {
     if (!coupleCount)
         return true;
     
-    if (coupleCount === nodes.length)
+    if (coupleCount === paths.length)
         return false;
     
-    return !(count !== 1 && count === nodes.length);
+    return !(count !== 1 && count === paths.length);
 };
 
 function getAllNodes(path, nodes = []) {

package/lib/index.js

@@ -14,6 +14,7 @@ const simplify = require('./simplify');
 const removeSameValuesCondition = require('./remove-same-values-condition');
 const addReturn = require('./add-return');
 const convertArrowToCondition = require('./convert-arrow-to-condition');
+const reverseCondition = require('./reverse-condition');
 
 module.exports.rules = {
     'apply-comparison-order': applyComparisonOrder,
@@ -30,4 +31,5 @@ module.exports.rules = {
     'remove-same-values-condition': removeSameValuesCondition,
     'add-return': addReturn,
     'convert-arrow-to-condition': convertArrowToCondition,
+    'reverse-condition': reverseCondition,
 };

package/lib/reverse-condition/index.js

@@ -0,0 +1,7 @@
+'use strict';
+
+module.exports.report = () => `Avoid useless '!'`;
+
+module.exports.replace = () => ({
+    '!(__a > __b)': '__a <= __b',
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/plugin-conditions",
-  "version": "6.0.0",
+  "version": "6.2.0",
   "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout plugin adds support of conditions transformations",