@putout/plugin-conditions 8.3.0 → 8.5.0

package/README.md

@@ -16,6 +16,7 @@ npm i @putout/plugin-conditions -D
 - ✅ [add-return](#add-return);
 - ✅ [apply-comparison-order](#apply-comparison-order);
 - ✅ [apply-consistent-blocks](#apply-consistent-blocks);
+- ✅ [apply-equal](#apply-equal);
 - ✅ [apply-if](#apply-if);
 - ✅ [convert-comparison-to-boolean](#convert-comparison-to-boolean);
 - ✅ [convert-equal-to-strict-equal](#convert-equal-to-strict-equal);
@@ -40,6 +41,7 @@ npm i @putout/plugin-conditions -D
     "rules": {
         "conditions/apply-consistent-blocks": "on",
         "conditions/apply-comparison-order": "on",
+        "conditions/apply-equal": "on",
         "conditions/apply-if": "on",
         "conditions/add-return": "on",
         "conditions/convert-comparison-to-boolean": "on",
@@ -141,6 +143,22 @@ Linter | Rule | Fix
 🐊 **Putout**| [`conditions/apply-comparison-order`](https://github.com/coderaiser/putout/tree/master/packages/plugin-conditions/#apply-comparison-order)| ✅
 ⏣ **ESLint** | [`yoda`](https://eslint.org/docs/rules/yoda) | ½
 
+## apply-equal
+
+Checkout in 🐊[**Putout Editor**](https://putout.cloudcmd.io/#/gist/7ec940ad79df7e12a1d9111136b4a168/2ccea607e3232721eb950c18195168f8d9b4ef6a).
+
+### ❌ Example of incorrect code
+
+```js
+return a.b = c;
+```
+
+### ✅ Example of correct code
+
+```js
+return a.b === c;
+```
+
 ## apply-if
 
 ### ❌ Example of incorrect code

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

@@ -12,7 +12,7 @@ export const report = () => `Use consistent blocks`;
 export const fix = (path) => {
     const paths = getAllNodes(path);
     
-    if (isAllBlocks(paths))
+    if (isAllBlocks(paths)) {
         for (const path of paths) {
             if (isBlockStatement(path))
                 continue;
@@ -20,14 +20,17 @@ export const fix = (path) => {
             const {node} = path;
             replaceWith(path, blockStatement([node]));
         }
-    else
-        for (const path of paths) {
-            if (!isBlockStatement(path))
-                continue;
-            
-            const [node] = path.node.body;
-            replaceWith(path, node);
-        }
+        
+        return;
+    }
+    
+    for (const path of paths) {
+        if (!isBlockStatement(path))
+            continue;
+        
+        const [node] = path.node.body;
+        replaceWith(path, node);
+    }
 };
 
 function isAllBlocks(paths) {
@@ -80,7 +83,7 @@ export const filter = (path) => {
                 continue;
             
             if (first.leadingComments?.length)
-                continue;
+                return false;
         }
         
         blocks.push(is);

package/lib/apply-equal/index.js

@@ -0,0 +1,5 @@
+export const report = () => `Use '===' instead of '=' when return a value`;
+
+export const replace = () => ({
+    'return __a = __b': 'return __a === __b',
+});

package/lib/index.js

@@ -1,3 +1,4 @@
+import * as applyEqual from './apply-equal/index.js';
 import * as applyConsistentBlocks from './apply-consistent-blocks/index.js';
 import * as applyComparisonOrder from './apply-comparison-order/index.js';
 import * as applyIf from './apply-if/index.js';
@@ -6,7 +7,7 @@ import * as convertComparisonToBoolean from './convert-comparison-to-boolean/ind
 import * as convertEqualToStrictEqual from './convert-equal-to-strict-equal/index.js';
 import * as mergeIfStatements from './merge-if-statements/index.js';
 import * as removeBoolean from './remove-boolean/index.js';
-import removeZero from './remove-zero/index.js';
+import * as removeZero from './remove-zero/index.js';
 import * as removeUselessElse from './remove-useless-else/index.js';
 import * as simplify from './simplify/index.js';
 import * as removeSameValuesCondition from './remove-same-values-condition/index.js';
@@ -36,4 +37,5 @@ export const rules = {
     'wrap-with-block': wrapWithBlock,
     'remove-useless-loop-condition': removeUselessLoopCondition,
     'merge-if-with-else': mergeIfWithElse,
+    'apply-equal': applyEqual,
 };

package/lib/remove-zero/index.js

@@ -1,3 +1,7 @@
 import {createAvoidInAssertions} from '../avoid-in-assertions.js';
 
-export default createAvoidInAssertions(0);
+export const {
+    report,
+    match,
+    replace,
+} = createAvoidInAssertions(0);

package/package.json

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