npm - pgsql-deparser - Versions diffs - 17.16.0 → 17.17.0 - Mend

pgsql-deparser 17.16.0 → 17.17.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (3) hide show

package/deparser.js CHANGED Viewed

@@ -508,7 +508,9 @@ class Deparser {
                 output.push('VALUES');
                 const lists = list_utils_1.ListUtils.unwrapList(node.valuesLists).map(list => {
                     const values = list_utils_1.ListUtils.unwrapList(list).map(val => this.visit(val, context));
-                    return context.parens(values.join(', '));
+                    // Put each value on its own line for pretty printing
+                    const indentedValues = values.map(val => context.indent(val));
+                    return '(\n' + indentedValues.join(',\n') + '\n)';
                 });
                 const indentedTuples = lists.map(tuple => {
                     if (this.containsMultilineStringLiteral(tuple)) {
@@ -1021,7 +1023,14 @@ class Deparser {
                     else {
                         const updateContext = context.spawn('UpdateStmt', { update: true });
                         const targets = targetList.map(target => this.visit(target, updateContext));
-                        output.push(targets.join(', '));
+                        if (context.isPretty()) {
+                            // Put each assignment on its own line for pretty printing
+                            const indentedTargets = targets.map(target => context.indent(target));
+                            output.push('\n' + indentedTargets.join(',\n'));
+                        }
+                        else {
+                            output.push(targets.join(', '));
+                        }
                     }
                 }
                 if (node.onConflictClause.whereClause) {
@@ -2055,7 +2064,7 @@ class Deparser {
                 if (size != null) {
                     return 'char';
                 }
-                return 'pg_catalog.bpchar';
+                return 'bpchar';
             case 'varchar':
                 return 'varchar';
             case 'numeric':
@@ -2069,7 +2078,7 @@ class Deparser {
             case 'int8':
                 return 'bigint';
             case 'real':
-                return 'pg_catalog.float4';
+                return 'float4';
             case 'time':
                 return 'time';
             case 'timestamp':
@@ -2079,7 +2088,7 @@ class Deparser {
             case 'bit':
                 return 'bit';
             default:
-                return `pg_catalog.${typeName}`;
+                return typeName;
         }
     }
     isPgCatalogType(typeName) {

package/esm/deparser.js CHANGED Viewed

@@ -505,7 +505,9 @@ export class Deparser {
                 output.push('VALUES');
                 const lists = ListUtils.unwrapList(node.valuesLists).map(list => {
                     const values = ListUtils.unwrapList(list).map(val => this.visit(val, context));
-                    return context.parens(values.join(', '));
+                    // Put each value on its own line for pretty printing
+                    const indentedValues = values.map(val => context.indent(val));
+                    return '(\n' + indentedValues.join(',\n') + '\n)';
                 });
                 const indentedTuples = lists.map(tuple => {
                     if (this.containsMultilineStringLiteral(tuple)) {
@@ -1018,7 +1020,14 @@ export class Deparser {
                     else {
                         const updateContext = context.spawn('UpdateStmt', { update: true });
                         const targets = targetList.map(target => this.visit(target, updateContext));
-                        output.push(targets.join(', '));
+                        if (context.isPretty()) {
+                            // Put each assignment on its own line for pretty printing
+                            const indentedTargets = targets.map(target => context.indent(target));
+                            output.push('\n' + indentedTargets.join(',\n'));
+                        }
+                        else {
+                            output.push(targets.join(', '));
+                        }
                     }
                 }
                 if (node.onConflictClause.whereClause) {
@@ -2052,7 +2061,7 @@ export class Deparser {
                 if (size != null) {
                     return 'char';
                 }
-                return 'pg_catalog.bpchar';
+                return 'bpchar';
             case 'varchar':
                 return 'varchar';
             case 'numeric':
@@ -2066,7 +2075,7 @@ export class Deparser {
             case 'int8':
                 return 'bigint';
             case 'real':
-                return 'pg_catalog.float4';
+                return 'float4';
             case 'time':
                 return 'time';
             case 'timestamp':
@@ -2076,7 +2085,7 @@ export class Deparser {
             case 'bit':
                 return 'bit';
             default:
-                return `pg_catalog.${typeName}`;
+                return typeName;
         }
     }
     isPgCatalogType(typeName) {

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "pgsql-deparser",
-  "version": "17.16.0",
+  "version": "17.17.0",
   "author": "Constructive <developers@constructive.io>",
   "description": "PostgreSQL AST Deparser",
   "main": "index.js",
@@ -60,5 +60,5 @@
   "dependencies": {
     "@pgsql/types": "^17.6.2"
   },
-  "gitHead": "a1fd1b05dfdf557fad8d5c7d952754353b01e9b6"
+  "gitHead": "529f487216d0ca3d93b1dea2e6411a1e5432d183"
 }