feature: @putout/plugin-conditions: apply-consistent-blocks: leading comments

Author: coderaiser

packages/plugin-conditions/lib/apply-consistent-blocks/fixture/comment-else.js

@@ -0,0 +1,14 @@
+export const get = () => {
+    let a = 0;
+
+    if (b) {
+        // hello
+        a = 0;
+    } else if (c) {
+        // world
+        const x = 1;
+        a = create(x);
+    }
+
+    return a;
+};

packages/plugin-conditions/lib/apply-consistent-blocks/index.js

@@ -12,22 +12,25 @@ 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;
 
             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);

packages/plugin-conditions/lib/apply-consistent-blocks/index.spec.js

@@ -72,6 +72,11 @@ test('putout: apply-consistent-blocks: no report: comment', (t) => {
     t.end();
 });
 
+test('putout: apply-consistent-blocks: no report: comment-else', (t) => {
+    t.noReport('comment-else');
+    t.end();
+});
+
 test('putout: apply-consistent-blocks: no report after transform: nested', (t) => {
     t.noReportAfterTransform('nested');
     t.end();