@putout/printer 6.10.0 → 6.11.1

package/ChangeLog

@@ -1,3 +1,13 @@
+2023.12.01, v6.11.1
+
+feature:
+- 39e3a0c ArrayExpression: simplify
+
+2023.11.30, v6.11.0
+
+feature:
+- 6390281 @putout/printer: improve support of comments
+
 2023.11.30, v6.10.0
 
 feature:

package/lib/tokenize/comment/parse-trailing-comments.js

@@ -29,6 +29,11 @@ function isCommentOnNextLine(path) {
     const [comment] = trailingComments;
     const {line} = comment.loc.start;
     
+    const next = path.getNextSibling();
+    
+    if (line < next.node?.loc.start.line)
+        return false;
+    
     return line === loc.start.line + 1;
 }
 

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

@@ -9,10 +9,10 @@ const {
 } = require('../../is');
 
 const {
-    isNewlineBetweenElements,
+    isMultiLine,
     isIncreaseIndent,
     isCurrentNewLine,
-} = require('./new-line');
+} = require('./newline');
 
 const {types} = require('@putout/babel');
 const {
@@ -67,7 +67,7 @@ module.exports.ArrayExpression = {
         } = semantics;
         
         const elements = path.get('elements');
-        const shouldIncreaseIndent = !isIncreaseIndent(path);
+        const shouldIncreaseIndent = isIncreaseIndent(path);
         
         print('[');
         
@@ -76,7 +76,7 @@ module.exports.ArrayExpression = {
         if (indented)
             maybe.indent.inc(shouldIncreaseIndent);
         
-        const isNewLine = isNewlineBetweenElements(path, {
+        const isNewLine = isMultiLine(path, {
             elements,
             maxElementsInOneLine,
         });

package/lib/tokenize/expressions/array-expression/{new-line.js → newline.js}

@@ -47,7 +47,7 @@ const isSimpleAndObject = ([a, b]) => isSimple(a) && isObjectExpression(b);
 const ONE_LINE = false;
 const MULTI_LINE = true;
 
-module.exports.isNewlineBetweenElements = (path, {elements, maxElementsInOneLine}) => {
+module.exports.isMultiLine = (path, {elements, maxElementsInOneLine}) => {
     if (elements.length > 3 && !isObjectExpression(elements[0]))
         return MULTI_LINE;
     
@@ -63,7 +63,7 @@ module.exports.isNewlineBetweenElements = (path, {elements, maxElementsInOneLine
     if (isCallInsideArrow(path))
         return ONE_LINE;
     
-    if (isIncreaseIndent(path))
+    if (notIncreaseIndent(path))
         return ONE_LINE;
     
     if (isInsideLoop(path))
@@ -102,7 +102,7 @@ module.exports.isNewlineBetweenElements = (path, {elements, maxElementsInOneLine
     if (isSimpleAndObject(elements))
         return ONE_LINE;
     
-    if (isStringAndString(elements) && path.parentPath.isArrayExpression() && isArrayExpression(path.parentPath.node.elements[0]))
+    if (isStringAndString(elements) && isParentIsArrayWithFirstArrayElement(path))
         return ONE_LINE;
     
     if (tooLong(path) || isCoupleLines(path) || !isNumbers(elements) && !isForOf(path) && isLastArg(path) && !isParentProperty(path))
@@ -111,6 +111,15 @@ module.exports.isNewlineBetweenElements = (path, {elements, maxElementsInOneLine
     return ONE_LINE;
 };
 
+const isParentIsArrayWithFirstArrayElement = ({parentPath}) => {
+    if (!isArrayExpression(parentPath))
+        return false;
+    
+    const [first] = parentPath.node.elements;
+    
+    return isArrayExpression(first);
+};
+
 const isForOf = ({parentPath}) => parentPath.isForOfStatement();
 
 const isStringAndString = ([a, b]) => isStringLiteral(a) && isStringLiteral(b);
@@ -235,23 +244,31 @@ function isParentProperty(path) {
     return path.find(isObjectProperty);
 }
 
-module.exports.isIncreaseIndent = isIncreaseIndent;
-function isIncreaseIndent(path) {
-    const elements = path.get('elements');
+const not = (fn) => (...a) => !fn(...a);
+
+const isFirstObject = (path) => isObjectExpression(path.node.elements[0]);
+const isSecondSpread = (path) => isSpreadElement(path.node.elements[1]);
+
+const isStringAndObject = (path) => {
+    const {elements} = path.node;
+    const first = elements.at(0);
+    const last = elements.at(-1);
     
-    if (!elements.length)
+    return isStringLiteral(first) && isObjectExpression(last);
+};
+
+module.exports.isIncreaseIndent = not(notIncreaseIndent);
+function notIncreaseIndent(path) {
+    if (isInsideCallLoop(path))
         return false;
     
-    if (isInsideCallLoop(path))
+    if (isSecondSpread(path))
         return false;
     
-    if (elements[0].isObjectExpression())
+    if (isStringAndObject(path))
         return true;
     
-    if (isSpreadElement(elements[1]))
-        return false;
-    
-    return isStringAndObject(elements);
+    return isFirstObject(path);
 }
 
 function isInsideCallLoop(path) {
@@ -261,13 +278,6 @@ function isInsideCallLoop(path) {
     return path.parentPath.parentPath.isForOfStatement();
 }
 
-const isStringAndObject = (elements) => {
-    const first = elements.at(0);
-    const last = elements.at(-1);
-    
-    return isStringLiteral(first) && isObjectExpression(last);
-};
-
 module.exports.isCurrentNewLine = (path) => {
     if (path.isSpreadElement())
         return true;

package/package.json

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