@putout/printer 2.1.0 → 2.3.0

package/ChangeLog

@@ -1,3 +1,13 @@
+2023.06.12, v2.3.0
+
+feature:
+- 75353e2 @putout/printer: ArrayExpression: maxElementsInOneLine
+
+2023.06.12, v2.2.0
+
+feature:
+- 5be9df9 printer: comments
+
 2023.06.12, v2.1.0
 
 feature:

package/README.md

@@ -104,7 +104,8 @@ write(ast, {
 
 ### `format`
 
-With help of `format` you can override next options:
+Options related to visuals and not related to logic of output can be changed with help of `format`,
+you can override next options:
 
 ```js
 const overrides = {
@@ -112,8 +113,9 @@ const overrides = {
         indent: '    ',
         newline: '\n',
         space: ' ',
-        comments: true,
         splitter: '\n',
+        roundBraceOpen: '(',
+        roundBraceClose: ')',
     },
 };
 ```
@@ -121,8 +123,8 @@ const overrides = {
 - `indent` - use two spaces, tabs, or anything you want;
 - `newline` - symbol for used for line separation;
 - `space` - default symbol used for space character;
-- `comments` - produce comments in output or skip them;
 - `splitter` - mandatory symbol that used inside of statements like this:
+- `roundBraceOpen` and `roundBraceClose` symbols to to output braces in a  single argument arrow function expressions: `(a) => {}`.
 
 Default options produce:
 
@@ -148,12 +150,15 @@ const overrides = {
 
 And have minified code:
 
-```js
-if (a > 3)
-    console.log('ok');
-else
-    console.log('not ok');
 ```
+if(a>3)console.log('ok');else console.log('not ok');
+```
+
+### Semantics
+
+Options used to configure logic of output, similar to ESLint rules:
+
+- `maxElementsInOneLine` - count of `ArrayExpression` and `ArrayPattern` elements placed in one line.
 
 ### `write`
 

package/lib/tokenize/comments.js

@@ -104,7 +104,7 @@ module.exports.parseComments = (path, {write}, semantics) => {
 };
 
 function isSameLine(path, loc) {
-    return path.node.loc.start.line === loc.start.line;
+    return path.node.loc.start.line === loc.start.line || path.node.loc.end.line === loc.end.line;
 }
 
 function maybeInsideFn(insideFn, {print, indent}) {

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/package.json

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