@putout/printer 6.11.5 → 6.11.6

package/ChangeLog

@@ -1,3 +1,8 @@
+2023.12.01, v6.11.6
+
+feature:
+- 60ae143 @putout/printer: BlockStatement: siplify
+
 2023.12.01, v6.11.5
 
 feature:

package/lib/tokenize/expressions/array-expression/newline.js

@@ -9,6 +9,7 @@ const isTwoStringsTupleAndParentIsArrayWithFirstArrayElement = (path) => {
     
     return isParentIsArrayWithFirstArrayElement(path);
 };
+
 const isLotsOfElementsFirstNotObject = (path) => {
     const {elements} = path.node;
     const [first] = elements;
@@ -291,4 +292,3 @@ module.exports.isCurrentNewLine = (path) => {
     
     return !path.isObjectExpression();
 };
-

package/lib/tokenize/statements/block-statement/block-statement.js

@@ -5,6 +5,8 @@ const {
     isParentProgram,
     isLast,
     exists,
+    satisfy,
+
 } = require('../../is');
 
 const {
@@ -12,6 +14,8 @@ const {
     isObjectMethod,
     isFunctionDeclaration,
     isExportDeclaration,
+    isDoWhileStatement,
+    isBlockStatement,
 } = require('@putout/babel').types;
 
 const {markAfter} = require('../../mark');
@@ -77,35 +81,62 @@ module.exports.BlockStatement = {
     },
 };
 
-function shouldAddNewlineAfter(path) {
-    const {parentPath} = path;
-    
-    if (parentPath.isDoWhileStatement())
+const isTopLevelWithNoNext = (path) => {
+    if (isNext(path))
         return false;
     
-    if (parentPath.isBlockStatement())
-        return true;
+    return !isNext(path.parentPath) && isParentProgram(path.parentPath);
+};
+
+const isInsideIfWithoutElseInsideFn = (path) => {
+    return parentIfWithoutElse(path) && path.find(isMethodOrArrow);
+};
+
+const isEmptyBodyNoNext = (path) => {
+    const {parentPath} = path;
+    return parentPath.isStatement() && !path.node.body.length && !isNext(parentPath);
+};
+
+const isLooksLikeInsideFn = ({parentPath}) => {
+    return /FunctionExpression/.test(parentPath.type);
+};
+
+const NEWLINE = true;
+const NO_NEWLINE = false;
+
+const isInsideDoWhile = ({parentPath}) => isDoWhileStatement(parentPath);
+const isInsideBlock = ({parentPath}) => isBlockStatement(parentPath);
+
+const isNoNewline = satisfy([
+    isInsideDoWhile,
+    isTopLevelWithNoNext,
+    insideIfWithNoBody,
+]);
+
+function shouldAddNewlineAfter(path) {
+    if (isInsideBlock(path))
+        return NEWLINE;
     
-    if (!isNext(path) && !isNext(path.parentPath) && isParentProgram(path.parentPath))
-        return false;
+    if (isNoNewline(path))
+        return NO_NEWLINE;
     
-    if (insideIfWithNoBody(path))
-        return false;
+    if (isInsideIfWithoutElseInsideFn(path))
+        return NEWLINE;
     
-    if (parentIfWithoutElse(path) && path.find(isMethodOrArrow))
-        return true;
+    if (isEmptyBodyNoNext(path))
+        return NO_NEWLINE;
     
-    if (parentPath.isStatement() && !path.node.body.length && !isNext(parentPath))
-        return false;
+    if (isTry(path))
+        return NO_NEWLINE;
     
-    if (isTry(path) || /FunctionExpression/.test(path.parentPath.type))
-        return false;
+    if (isLooksLikeInsideFn(path))
+        return NO_NEWLINE;
     
     if (isLast(path))
-        return false;
+        return NO_NEWLINE;
     
     if (isExportFunction(path))
-        return false;
+        return NO_NEWLINE;
     
     return !isNextIfAlternate(path);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/printer",
-  "version": "6.11.5",
+  "version": "6.11.6",
   "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "Simplest possible opinionated Babel AST printer for 🐊Putout",