postgresdk 0.16.4 → 0.16.6

package/dist/cli.js

@@ -4682,6 +4682,22 @@ const log = {
   error: (...args: any[]) => console.error("[sdk]", ...args),
 };
 
+/**
+ * Prepare query parameters for PostgreSQL.
+ * The pg library should handle JSONB automatically, but there are edge cases
+ * where explicit stringification is needed (e.g., certain pg versions or when
+ * objects have been through serialization/deserialization).
+ */
+function prepareParams(params: any[]): any[] {
+  return params.map(p => {
+    if (p === null || p === undefined) return p;
+    // Stringify objects/arrays for JSONB - while pg should handle this automatically,
+    // we've observed cases where it fails without explicit stringification
+    if (typeof p === 'object') return JSON.stringify(p);
+    return p;
+  });
+}
+
 /**
  * CREATE operation - Insert a new record
  */
@@ -4701,17 +4717,28 @@ export async function createRecord(
     const text = \`INSERT INTO "\${ctx.table}" (\${cols.map(c => '"' + c + '"').join(", ")})
                    VALUES (\${placeholders})
                    RETURNING *\`;
-    
+
     log.debug("SQL:", text, "vals:", vals);
-    const { rows } = await ctx.pg.query(text, vals);
-    
+    const { rows } = await ctx.pg.query(text, prepareParams(vals));
+
     return { data: rows[0] ?? null, status: rows[0] ? 201 : 500 };
   } catch (e: any) {
-    log.error(\`POST \${ctx.table} error:\`, e?.stack ?? e);
-    return { 
-      error: e?.message ?? "Internal error", 
+    // Enhanced logging for JSON validation errors
+    const errorMsg = e?.message ?? "";
+    const isJsonError = errorMsg.includes("invalid input syntax for type json");
+
+    if (isJsonError) {
+      log.error(\`POST \${ctx.table} - Invalid JSON input detected!\`);
+      log.error("Input data that caused error:", JSON.stringify(data, null, 2));
+      log.error("PostgreSQL error:", errorMsg);
+    } else {
+      log.error(\`POST \${ctx.table} error:\`, e?.stack ?? e);
+    }
+
+    return {
+      error: e?.message ?? "Internal error",
       ...(DEBUG ? { stack: e?.stack } : {}),
-      status: 500 
+      status: 500
     };
   }
 }
@@ -4731,8 +4758,8 @@ export async function getByPk(
     
     const text = \`SELECT * FROM "\${ctx.table}" WHERE \${wherePkSql} LIMIT 1\`;
     log.debug(\`GET \${ctx.table} by PK:\`, pkValues, "SQL:", text);
-    
-    const { rows } = await ctx.pg.query(text, pkValues);
+
+    const { rows } = await ctx.pg.query(text, prepareParams(pkValues));
     
     if (!rows[0]) {
       return { data: null, status: 404 };
@@ -5116,14 +5143,14 @@ export async function listRecords(
     // Get total count for pagination
     const countText = \`SELECT COUNT(*) FROM "\${ctx.table}" \${countWhereSQL}\`;
     log.debug(\`LIST \${ctx.table} COUNT SQL:\`, countText, "params:", countParams);
-    const countResult = await ctx.pg.query(countText, countParams);
+    const countResult = await ctx.pg.query(countText, prepareParams(countParams));
     const total = parseInt(countResult.rows[0].count, 10);
 
     // Get paginated data
     const text = \`SELECT \${selectClause} FROM "\${ctx.table}" \${whereSQL} \${orderBySQL} LIMIT \${limitParam} OFFSET \${offsetParam}\`;
     log.debug(\`LIST \${ctx.table} SQL:\`, text, "params:", allParams);
 
-    const { rows } = await ctx.pg.query(text, allParams);
+    const { rows } = await ctx.pg.query(text, prepareParams(allParams));
 
     // Calculate hasMore
     const hasMore = offset + limit < total;
@@ -5142,7 +5169,18 @@ export async function listRecords(
     log.debug(\`LIST \${ctx.table} result: \${rows.length} rows, \${total} total, hasMore=\${hasMore}\`);
     return metadata;
   } catch (e: any) {
-    log.error(\`LIST \${ctx.table} error:\`, e?.stack ?? e);
+    // Enhanced logging for JSON validation errors
+    const errorMsg = e?.message ?? "";
+    const isJsonError = errorMsg.includes("invalid input syntax for type json");
+
+    if (isJsonError) {
+      log.error(\`LIST \${ctx.table} - Invalid JSON input detected in query!\`);
+      log.error("WHERE clause:", JSON.stringify(params.where, null, 2));
+      log.error("PostgreSQL error:", errorMsg);
+    } else {
+      log.error(\`LIST \${ctx.table} error:\`, e?.stack ?? e);
+    }
+
     return {
       error: e?.message ?? "Internal error",
       ...(DEBUG ? { stack: e?.stack } : {}),
@@ -5177,12 +5215,12 @@ export async function updateRecord(
     const setSql = Object.keys(filteredData)
       .map((k, i) => \`"\${k}" = $\${i + pkValues.length + 1}\`)
       .join(", ");
-    
+
     const text = \`UPDATE "\${ctx.table}" SET \${setSql} WHERE \${wherePkSql} RETURNING *\`;
     const params = [...pkValues, ...Object.values(filteredData)];
-    
+
     log.debug(\`PATCH \${ctx.table} SQL:\`, text, "params:", params);
-    const { rows } = await ctx.pg.query(text, params);
+    const { rows } = await ctx.pg.query(text, prepareParams(params));
     
     if (!rows[0]) {
       return { data: null, status: 404 };
@@ -5190,11 +5228,25 @@ export async function updateRecord(
     
     return { data: rows[0], status: 200 };
   } catch (e: any) {
-    log.error(\`PATCH \${ctx.table} error:\`, e?.stack ?? e);
-    return { 
-      error: e?.message ?? "Internal error", 
+    // Enhanced logging for JSON validation errors
+    const errorMsg = e?.message ?? "";
+    const isJsonError = errorMsg.includes("invalid input syntax for type json");
+
+    if (isJsonError) {
+      log.error(\`PATCH \${ctx.table} - Invalid JSON input detected!\`);
+      log.error("Input data that caused error:", JSON.stringify(updateData, null, 2));
+      log.error("Filtered data (sent to DB):", JSON.stringify(Object.fromEntries(
+        Object.entries(updateData).filter(([k]) => !ctx.pkColumns.includes(k))
+      ), null, 2));
+      log.error("PostgreSQL error:", errorMsg);
+    } else {
+      log.error(\`PATCH \${ctx.table} error:\`, e?.stack ?? e);
+    }
+
+    return {
+      error: e?.message ?? "Internal error",
       ...(DEBUG ? { stack: e?.stack } : {}),
-      status: 500 
+      status: 500
     };
   }
 }
@@ -5215,9 +5267,9 @@ export async function deleteRecord(
     const text = ctx.softDeleteColumn
       ? \`UPDATE "\${ctx.table}" SET "\${ctx.softDeleteColumn}" = NOW() WHERE \${wherePkSql} RETURNING *\`
       : \`DELETE FROM "\${ctx.table}" WHERE \${wherePkSql} RETURNING *\`;
-    
+
     log.debug(\`DELETE \${ctx.softDeleteColumn ? '(soft)' : ''} \${ctx.table} SQL:\`, text, "pk:", pkValues);
-    const { rows } = await ctx.pg.query(text, pkValues);
+    const { rows } = await ctx.pg.query(text, prepareParams(pkValues));
     
     if (!rows[0]) {
       return { data: null, status: 404 };

package/dist/index.js

@@ -3853,6 +3853,22 @@ const log = {
   error: (...args: any[]) => console.error("[sdk]", ...args),
 };
 
+/**
+ * Prepare query parameters for PostgreSQL.
+ * The pg library should handle JSONB automatically, but there are edge cases
+ * where explicit stringification is needed (e.g., certain pg versions or when
+ * objects have been through serialization/deserialization).
+ */
+function prepareParams(params: any[]): any[] {
+  return params.map(p => {
+    if (p === null || p === undefined) return p;
+    // Stringify objects/arrays for JSONB - while pg should handle this automatically,
+    // we've observed cases where it fails without explicit stringification
+    if (typeof p === 'object') return JSON.stringify(p);
+    return p;
+  });
+}
+
 /**
  * CREATE operation - Insert a new record
  */
@@ -3872,17 +3888,28 @@ export async function createRecord(
     const text = \`INSERT INTO "\${ctx.table}" (\${cols.map(c => '"' + c + '"').join(", ")})
                    VALUES (\${placeholders})
                    RETURNING *\`;
-    
+
     log.debug("SQL:", text, "vals:", vals);
-    const { rows } = await ctx.pg.query(text, vals);
-    
+    const { rows } = await ctx.pg.query(text, prepareParams(vals));
+
     return { data: rows[0] ?? null, status: rows[0] ? 201 : 500 };
   } catch (e: any) {
-    log.error(\`POST \${ctx.table} error:\`, e?.stack ?? e);
-    return { 
-      error: e?.message ?? "Internal error", 
+    // Enhanced logging for JSON validation errors
+    const errorMsg = e?.message ?? "";
+    const isJsonError = errorMsg.includes("invalid input syntax for type json");
+
+    if (isJsonError) {
+      log.error(\`POST \${ctx.table} - Invalid JSON input detected!\`);
+      log.error("Input data that caused error:", JSON.stringify(data, null, 2));
+      log.error("PostgreSQL error:", errorMsg);
+    } else {
+      log.error(\`POST \${ctx.table} error:\`, e?.stack ?? e);
+    }
+
+    return {
+      error: e?.message ?? "Internal error",
       ...(DEBUG ? { stack: e?.stack } : {}),
-      status: 500 
+      status: 500
     };
   }
 }
@@ -3902,8 +3929,8 @@ export async function getByPk(
     
     const text = \`SELECT * FROM "\${ctx.table}" WHERE \${wherePkSql} LIMIT 1\`;
     log.debug(\`GET \${ctx.table} by PK:\`, pkValues, "SQL:", text);
-    
-    const { rows } = await ctx.pg.query(text, pkValues);
+
+    const { rows } = await ctx.pg.query(text, prepareParams(pkValues));
     
     if (!rows[0]) {
       return { data: null, status: 404 };
@@ -4287,14 +4314,14 @@ export async function listRecords(
     // Get total count for pagination
     const countText = \`SELECT COUNT(*) FROM "\${ctx.table}" \${countWhereSQL}\`;
     log.debug(\`LIST \${ctx.table} COUNT SQL:\`, countText, "params:", countParams);
-    const countResult = await ctx.pg.query(countText, countParams);
+    const countResult = await ctx.pg.query(countText, prepareParams(countParams));
     const total = parseInt(countResult.rows[0].count, 10);
 
     // Get paginated data
     const text = \`SELECT \${selectClause} FROM "\${ctx.table}" \${whereSQL} \${orderBySQL} LIMIT \${limitParam} OFFSET \${offsetParam}\`;
     log.debug(\`LIST \${ctx.table} SQL:\`, text, "params:", allParams);
 
-    const { rows } = await ctx.pg.query(text, allParams);
+    const { rows } = await ctx.pg.query(text, prepareParams(allParams));
 
     // Calculate hasMore
     const hasMore = offset + limit < total;
@@ -4313,7 +4340,18 @@ export async function listRecords(
     log.debug(\`LIST \${ctx.table} result: \${rows.length} rows, \${total} total, hasMore=\${hasMore}\`);
     return metadata;
   } catch (e: any) {
-    log.error(\`LIST \${ctx.table} error:\`, e?.stack ?? e);
+    // Enhanced logging for JSON validation errors
+    const errorMsg = e?.message ?? "";
+    const isJsonError = errorMsg.includes("invalid input syntax for type json");
+
+    if (isJsonError) {
+      log.error(\`LIST \${ctx.table} - Invalid JSON input detected in query!\`);
+      log.error("WHERE clause:", JSON.stringify(params.where, null, 2));
+      log.error("PostgreSQL error:", errorMsg);
+    } else {
+      log.error(\`LIST \${ctx.table} error:\`, e?.stack ?? e);
+    }
+
     return {
       error: e?.message ?? "Internal error",
       ...(DEBUG ? { stack: e?.stack } : {}),
@@ -4348,12 +4386,12 @@ export async function updateRecord(
     const setSql = Object.keys(filteredData)
       .map((k, i) => \`"\${k}" = $\${i + pkValues.length + 1}\`)
       .join(", ");
-    
+
     const text = \`UPDATE "\${ctx.table}" SET \${setSql} WHERE \${wherePkSql} RETURNING *\`;
     const params = [...pkValues, ...Object.values(filteredData)];
-    
+
     log.debug(\`PATCH \${ctx.table} SQL:\`, text, "params:", params);
-    const { rows } = await ctx.pg.query(text, params);
+    const { rows } = await ctx.pg.query(text, prepareParams(params));
     
     if (!rows[0]) {
       return { data: null, status: 404 };
@@ -4361,11 +4399,25 @@ export async function updateRecord(
     
     return { data: rows[0], status: 200 };
   } catch (e: any) {
-    log.error(\`PATCH \${ctx.table} error:\`, e?.stack ?? e);
-    return { 
-      error: e?.message ?? "Internal error", 
+    // Enhanced logging for JSON validation errors
+    const errorMsg = e?.message ?? "";
+    const isJsonError = errorMsg.includes("invalid input syntax for type json");
+
+    if (isJsonError) {
+      log.error(\`PATCH \${ctx.table} - Invalid JSON input detected!\`);
+      log.error("Input data that caused error:", JSON.stringify(updateData, null, 2));
+      log.error("Filtered data (sent to DB):", JSON.stringify(Object.fromEntries(
+        Object.entries(updateData).filter(([k]) => !ctx.pkColumns.includes(k))
+      ), null, 2));
+      log.error("PostgreSQL error:", errorMsg);
+    } else {
+      log.error(\`PATCH \${ctx.table} error:\`, e?.stack ?? e);
+    }
+
+    return {
+      error: e?.message ?? "Internal error",
       ...(DEBUG ? { stack: e?.stack } : {}),
-      status: 500 
+      status: 500
     };
   }
 }
@@ -4386,9 +4438,9 @@ export async function deleteRecord(
     const text = ctx.softDeleteColumn
       ? \`UPDATE "\${ctx.table}" SET "\${ctx.softDeleteColumn}" = NOW() WHERE \${wherePkSql} RETURNING *\`
       : \`DELETE FROM "\${ctx.table}" WHERE \${wherePkSql} RETURNING *\`;
-    
+
     log.debug(\`DELETE \${ctx.softDeleteColumn ? '(soft)' : ''} \${ctx.table} SQL:\`, text, "pk:", pkValues);
-    const { rows } = await ctx.pg.query(text, pkValues);
+    const { rows } = await ctx.pg.query(text, prepareParams(pkValues));
     
     if (!rows[0]) {
       return { data: null, status: 404 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "postgresdk",
-  "version": "0.16.4",
+  "version": "0.16.6",
   "description": "Generate a typed server/client SDK from a Postgres schema (includes, Zod, Hono).",
   "type": "module",
   "bin": {