@putout/printer 8.32.0 → 8.34.0

package/ChangeLog

@@ -1,3 +1,14 @@
+2024.05.07, v8.34.0
+
+feature:
+- 8a303f5 @putout/printer: jsx: JSXOpeningElement: indent
+
+2024.05.03, v8.33.0
+
+feature:
+- c8b9948 @putout/printer: ObjectPattern: improve support of maxPropertiesLengthInOneLine
+- 039c54d @putout/printer: ObjectPattern: indent: simplify
+
 2024.05.03, v8.32.0
 
 feature:

package/lib/tokenize/expressions/object-pattern/more-then-max-properties-length-in-one-line.js

@@ -1,11 +1,26 @@
 'use strict';
 
-const {isIdentifier} = require('@putout/babel').types;
+const {
+    isAssignmentPattern,
+    isIdentifier,
+} = require('@putout/babel').types;
 
 module.exports.moreThenMaxPropertiesLengthInOneLine = (path, {maxPropertiesLengthInOneLine}) => {
     const {properties} = path.node;
     
-    for (const {key} of properties) {
+    for (const {key, value} of properties) {
+        if (isAssignmentPattern(value)) {
+            const {left, right} = value;
+            
+            if (!isIdentifier(left) || !isIdentifier(right))
+                continue;
+            
+            const length = left.name.length + right.name.length;
+            
+            if (length >= maxPropertiesLengthInOneLine)
+                return true;
+        }
+        
         if (!isIdentifier(key))
             continue;
         

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

@@ -26,8 +26,18 @@ function isIndent(path) {
     return !path.parentPath.isArrayPattern();
 }
 
+const isCoupleProperties = ({path, valuePath, property}) => {
+    if (!isCoupleLines(valuePath))
+        return false;
+    
+    if (exists(property.getPrevSibling()))
+        return false;
+    
+    return !path.parentPath.isObjectProperty();
+};
+
 module.exports.ObjectPattern = {
-    print: maybeTypeAnnotation((path, {print, maybe}, semantics) => {
+    print: maybeTypeAnnotation((path, {print, maybe, indent}, semantics) => {
         const shouldIndent = isIndent(path);
         const {
             maxPropertiesInOneLine,
@@ -64,7 +74,11 @@ module.exports.ObjectPattern = {
             const isAssign = valuePath.isAssignmentPattern();
             
             const {shorthand, computed} = property.node;
-            const couple = isCoupleLines(valuePath) && !exists(property.getPrevSibling()) && !path.parentPath.isObjectProperty();
+            const couple = isCoupleProperties({
+                path,
+                property,
+                valuePath,
+            });
             
             maybe.indent(is);
             maybe.print.breakline(couple);
@@ -84,7 +98,7 @@ module.exports.ObjectPattern = {
                 maybe.print.newline(couple);
             }
             
-            if (is || hasObject) {
+            if (is || hasObject && !isAssign) {
                 print(',');
                 print.newline();
                 
@@ -97,9 +111,7 @@ module.exports.ObjectPattern = {
             }
         }
         
-        maybe.indent.dec(shouldIndent);
-        
-        maybe.indent.dec(!shouldIndent);
+        indent.dec();
         maybe.indent(is);
         maybe.indent.inc(!shouldIndent);
         print('}');
@@ -187,9 +199,7 @@ function shouldAddNewline(path, semantics) {
     return parentPath.isObjectProperty();
 }
 
-function isFunctionParam(path) {
-    const {parentPath} = path;
-    
+function isFunctionParam({parentPath}) {
     if (parentPath.isFunction())
         return true;
     

package/lib/tokenize/jsx/jsx-opening-element.js

@@ -3,6 +3,9 @@
 const {isCoupleLines} = require('../is');
 
 const isNotJSX = ({parentPath}) => {
+    if (parentPath.parentPath.isCallExpression())
+        return false;
+    
     if (parentPath.parentPath.isJSXElement())
         return false;
     

package/package.json

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