@putout/printer 1.119.0 → 1.121.0

package/ChangeLog

@@ -1,3 +1,14 @@
+2023.05.30, v1.121.0
+
+feature:
+- 26071a8 @putout/printer: add ability to traverse existing path
+
+2023.05.29, v1.120.0
+
+feature:
+- da33669 @putout/printer: ObjectProperty: improve support when one line inside LogicalExpression
+- db06c37 @putout/printer: ObjectPattern: improve RestElement support
+
 2023.05.29, v1.119.0
 
 feature:

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

@@ -92,6 +92,8 @@ module.exports.ObjectProperty = (path, {print, maybe}) => {
         computed,
     } = path.node;
     
+    const properties = path.parentPath.get('properties');
+    const isLast = path === properties.at(-1);
     const manyLines = !isOneLine(path.parentPath);
     
     maybe.print(computed, '[');
@@ -104,7 +106,10 @@ module.exports.ObjectProperty = (path, {print, maybe}) => {
         print('__value');
     }
     
-    maybe.print(manyLines, ',');
+    if (manyLines)
+        print(',');
+    else if (!isLast && properties.length)
+        print(', ');
 };
 
 const ONE_LINE = true;

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

@@ -1,6 +1,9 @@
 'use strict';
 
-const {isIdentifier} = require('@babel/types');
+const {
+    isIdentifier,
+    isObjectPattern,
+} = require('@babel/types');
 
 const {
     isForOf,
@@ -22,14 +25,15 @@ module.exports.ObjectPattern = {
         const properties = path.get('properties');
         const n = properties.length - 1;
         const is = shouldAddNewline(path);
+        const hasObject = n && hasObjectPattern(properties);
         
         maybe.print(is, '\n');
         
         for (const [i, property] of properties.entries()) {
             if (property.isRestElement()) {
-                maybe.indent(n > 1);
+                maybe.indent(hasObject);
                 print(property);
-                maybe.print.newline(n > 1);
+                maybe.print.newline(hasObject);
                 continue;
             }
             
@@ -62,7 +66,7 @@ module.exports.ObjectPattern = {
                 maybe.print.newline(couple);
             }
             
-            if (is) {
+            if (is || hasObject) {
                 print(',');
                 print.newline();
                 
@@ -83,7 +87,10 @@ module.exports.ObjectPattern = {
         if (!path.parentPath.isObjectProperty())
             return false;
         
-        return isOneParentProperty(path);
+        if (isOneParentProperty(path))
+            return true;
+        
+        return false;
     },
     after(path, {print}) {
         print.newline();
@@ -104,6 +111,15 @@ function checkLength(properties) {
     return false;
 }
 
+function hasObjectPattern(properties) {
+    for (const property of properties) {
+        if (isObjectPattern(property.node.value))
+            return true;
+    }
+    
+    return false;
+}
+
 function shouldAddNewline(path) {
     const properties = path.get('properties');
     const n = properties.length - 1;
@@ -116,4 +132,3 @@ function shouldAddNewline(path) {
     
     return false;
 }
-

package/lib/tokenize/statements/variable-declaration/variable-declaration.js

@@ -23,7 +23,9 @@ module.exports.VariableDeclaration = {
     print(path, {maybe, store, write, traverse}) {
         maybe.indent(isInsideBlock(path));
         write(String(path.node.kind));
-        maybeSpaceAfterKeyword(path, {write});
+        maybeSpaceAfterKeyword(path, {
+            write,
+        });
         
         const declarations = path.get('declarations');
         const n = declarations.length - 1;
@@ -150,4 +152,3 @@ const isNextAssign = (path) => {
         .get('expression')
         .isAssignmentExpression();
 };
-

package/lib/tokenize/tokenize.js

@@ -177,12 +177,15 @@ module.exports.tokenize = (ast, overrides = {}) => {
         ...overrides.visitors,
     };
     
-    babelTraverse(maybeFile(ast), {
-        Program(path) {
-            traverse(path);
-            path.stop();
-        },
-    });
+    if (ast.parentPath)
+        pathTraverse(ast, traverse);
+    else
+        babelTraverse(maybeFile(ast), {
+            Program(path) {
+                traverse(path);
+                path.stop();
+            },
+        });
     
     function traverse(path) {
         const {type} = path;
@@ -266,3 +269,14 @@ const computePath = (path, maybeLine) => {
     
     return maybeLine;
 };
+
+function pathTraverse(ast, traverse) {
+    ast.parentPath.traverse({
+        enter(path) {
+            if (path === ast) {
+                traverse(path);
+                path.stop();
+            }
+        },
+    });
+}

package/package.json

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