pgsql-deparser 17.18.1 → 17.18.3

package/deparser.js

@@ -658,10 +658,15 @@ class Deparser {
                     return context.format([leftExpr, operator, rightExpr]);
                 }
                 else if (rexpr) {
-                    return context.format([
-                        this.deparseOperatorName(name, context),
-                        this.visit(rexpr, context)
-                    ]);
+                    // Unary operator (e.g., unary minus: - expr)
+                    const operator = this.deparseOperatorName(name, context);
+                    let rightExpr = this.visit(rexpr, context);
+                    // Wrap in parentheses if rexpr is a binary A_Expr to preserve semantics
+                    // e.g., -(a - b) must stay -(a - b), not become -a - b
+                    if (rexpr && 'A_Expr' in rexpr && rexpr.A_Expr?.kind === 'AEXPR_OP' && rexpr.A_Expr?.lexpr) {
+                        rightExpr = context.parens(rightExpr);
+                    }
+                    return context.format([operator, rightExpr]);
                 }
                 break;
             case 'AEXPR_OP_ANY':
@@ -1312,8 +1317,13 @@ class Deparser {
         }
         // Handle EXTRACT function with SQL syntax
         if (node.funcformat === 'COERCE_SQL_SYNTAX' && name === 'pg_catalog.extract' && args.length >= 2) {
-            const field = this.visit(args[0], context);
+            let field = this.visit(args[0], context);
             const source = this.visit(args[1], context);
+            // EXTRACT requires an unquoted uppercase keyword (EPOCH, YEAR, etc.)
+            // but A_Const sval deparsing wraps it in single quotes — strip and uppercase
+            if (field.startsWith("'") && field.endsWith("'")) {
+                field = field.slice(1, -1).toUpperCase();
+            }
             return `EXTRACT(${field} FROM ${source})`;
         }
         // Handle TRIM function with SQL syntax (TRIM TRAILING/LEADING/BOTH)

package/esm/deparser.js

@@ -655,10 +655,15 @@ export class Deparser {
                     return context.format([leftExpr, operator, rightExpr]);
                 }
                 else if (rexpr) {
-                    return context.format([
-                        this.deparseOperatorName(name, context),
-                        this.visit(rexpr, context)
-                    ]);
+                    // Unary operator (e.g., unary minus: - expr)
+                    const operator = this.deparseOperatorName(name, context);
+                    let rightExpr = this.visit(rexpr, context);
+                    // Wrap in parentheses if rexpr is a binary A_Expr to preserve semantics
+                    // e.g., -(a - b) must stay -(a - b), not become -a - b
+                    if (rexpr && 'A_Expr' in rexpr && rexpr.A_Expr?.kind === 'AEXPR_OP' && rexpr.A_Expr?.lexpr) {
+                        rightExpr = context.parens(rightExpr);
+                    }
+                    return context.format([operator, rightExpr]);
                 }
                 break;
             case 'AEXPR_OP_ANY':
@@ -1309,8 +1314,13 @@ export class Deparser {
         }
         // Handle EXTRACT function with SQL syntax
         if (node.funcformat === 'COERCE_SQL_SYNTAX' && name === 'pg_catalog.extract' && args.length >= 2) {
-            const field = this.visit(args[0], context);
+            let field = this.visit(args[0], context);
             const source = this.visit(args[1], context);
+            // EXTRACT requires an unquoted uppercase keyword (EPOCH, YEAR, etc.)
+            // but A_Const sval deparsing wraps it in single quotes — strip and uppercase
+            if (field.startsWith("'") && field.endsWith("'")) {
+                field = field.slice(1, -1).toUpperCase();
+            }
             return `EXTRACT(${field} FROM ${source})`;
         }
         // Handle TRIM function with SQL syntax (TRIM TRAILING/LEADING/BOTH)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pgsql-deparser",
-  "version": "17.18.1",
+  "version": "17.18.3",
   "author": "Constructive <developers@constructive.io>",
   "description": "PostgreSQL AST Deparser",
   "main": "index.js",
@@ -60,5 +60,5 @@
     "@pgsql/quotes": "17.1.0",
     "@pgsql/types": "^17.6.2"
   },
-  "gitHead": "58ddefe11a5d002db6645fa7bc2f284123d48470"
+  "gitHead": "16d4dced1144372f2debc270ea07e9d842cc50c9"
 }