@putout/printer 2.2.0 → 2.4.0

package/ChangeLog

@@ -1,3 +1,13 @@
+2023.06.12, v2.4.0
+
+feature:
+- 1aa1e9e @putout/printer: ObjectExpression: improve support of using as not last argument of CallExpression
+
+2023.06.12, v2.3.0
+
+feature:
+- 75353e2 @putout/printer: ArrayExpression: maxElementsInOneLine
+
 2023.06.12, v2.2.0
 
 feature:

package/README.md

@@ -158,8 +158,7 @@ if(a>3)console.log('ok');else console.log('not ok');
 
 Options used to configure logic of output, similar to ESLint rules:
 
-```js
-```
+- `maxElementsInOneLine` - count of `ArrayExpression` and `ArrayPattern` elements placed in one line.
 
 ### `write`
 

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

@@ -23,8 +23,6 @@ const {
     isStringAndMember,
 } = require('../../is');
 
-const MAX_ARRAY_ELEMENTS_WHEN_USED_AS_ARGUMENT = 4;
-
 const isForOf = ({parentPath}) => parentPath.isForOfStatement();
 
 const isInsideOneElementArray = ({parentPath}) => parentPath.node.elements.length === 1;
@@ -112,7 +110,8 @@ module.exports.ArrayExpression = {
     before(path, {print}) {
         print.breakline();
     },
-    print(path, {print, maybe, indent}) {
+    print(path, {print, maybe, indent}, semantics) {
+        const {maxElementsInOneLine} = semantics;
         const elements = path.get('elements');
         const shouldIncreaseIndent = !isIncreaseIndent(path);
         
@@ -123,6 +122,7 @@ module.exports.ArrayExpression = {
         
         const isNewLine = isNewlineBetweenElements(path, {
             elements,
+            maxElementsInOneLine,
         });
         
         const n = elements.length - 1;
@@ -256,7 +256,7 @@ function isOneSimple(path) {
     return first.isMemberExpression();
 }
 
-function isNewlineBetweenElements(path, {elements}) {
+function isNewlineBetweenElements(path, {elements, maxElementsInOneLine}) {
     if (elements.length > 3 && !isObjectExpression(elements[0]))
         return true;
     
@@ -287,7 +287,7 @@ function isNewlineBetweenElements(path, {elements}) {
     if (isSimpleAndCall(elements))
         return false;
     
-    if (isShortTwoSimplesInsideCall(path, MAX_ARRAY_ELEMENTS_WHEN_USED_AS_ARGUMENT))
+    if (isShortTwoSimplesInsideCall(path, maxElementsInOneLine))
         return false;
     
     if (isTwoStringsDifferentLength(elements))

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

@@ -79,6 +79,17 @@ const hasNextLeadingComment = (path) => {
     return hasLeadingComment(next);
 };
 
+const notLastArgInsideCall = (path) => {
+    const {parentPath} = path;
+    
+    if (!parentPath.isCallExpression())
+        return false;
+    
+    return path !== parentPath
+        .get('arguments')
+        .at(-1);
+};
+
 function shouldAddNewline(path) {
     if (!path.parentPath.isLogicalExpression())
         return false;
@@ -96,6 +107,9 @@ function isOneLine(path) {
     if (!length)
         return true;
     
+    if (notLastArgInsideCall(path))
+        return ONE_LINE;
+    
     if (isForOf(path))
         return ONE_LINE;
     

package/package.json

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