@valentinkolb/cloud 0.5.3 → 0.5.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@valentinkolb/cloud",
-  "version": "0.5.3",
+  "version": "0.5.4",
   "description": "Modular Hono+SolidJS framework for building per-app docker services behind a dynamic gateway. Powers cloud.stuve-ulm.de.",
   "license": "MIT",
   "repository": {

package/src/server/services/access.ts

@@ -1,5 +1,6 @@
 import { err, fail, ok, type Result } from "@valentinkolb/stdlib";
 import { sql } from "bun";
+import { toPgUuidArray } from "../../services/postgres";
 
 // ==========================
 // Permission Levels
@@ -85,14 +86,6 @@ type DbAccessUser = {
 // Helper Functions
 // ==========================
 
-/**
- * Converts UUID strings into a PostgreSQL uuid[] literal for relation queries.
- */
-const toPgUuidArray = (values: string[] | null | undefined): string => {
-  if (!Array.isArray(values) || values.length === 0) return "{}";
-  return `{${values.join(",")}}`;
-};
-
 const uniqueIds = (values: string[] | null | undefined): string[] => [...new Set((values ?? []).filter(Boolean))];
 
 const escapeLikePattern = (value: string): string => value.replace(/[\\%_]/g, (match) => `\\${match}`);

package/src/server/services/freeipa/util.ts

@@ -1,3 +1,5 @@
+import { toPgTextArray as serializePgTextArray } from "../../../services/postgres";
+
 export class IpaError extends Error {
   constructor(
     message: string,
@@ -54,7 +56,4 @@ export const mapIpaErrorCode = (code: number): 400 | 401 | 403 => {
   return 400;
 };
 
-export const toPgTextArray = (values: string[] | null | undefined): string => {
-  if (!Array.isArray(values) || values.length === 0) return "{}";
-  return `{${values.map((value) => `"${value.replace(/\\/g, "\\\\").replace(/"/g, '\\"')}"`).join(",")}}`;
-};
+export const toPgTextArray = serializePgTextArray;

package/src/services/postgres.test.ts

@@ -0,0 +1,18 @@
+import { describe, expect, test } from "bun:test";
+import { toPgTextArray, toPgUuidArray } from "./postgres";
+
+describe("Postgres array helpers", () => {
+  test("serializes UUID arrays and treats non-arrays as empty arrays", () => {
+    expect(toPgUuidArray(["11111111-1111-4111-8111-111111111111", "22222222-2222-4222-8222-222222222222"])).toBe(
+      "{11111111-1111-4111-8111-111111111111,22222222-2222-4222-8222-222222222222}",
+    );
+    expect(toPgUuidArray([])).toBe("{}");
+    expect(toPgUuidArray("{}" as unknown as string[])).toBe("{}");
+  });
+
+  test("serializes text arrays with escaping and treats non-arrays as empty arrays", () => {
+    expect(toPgTextArray(["alpha", "has space", 'has "quote"', "has\\slash"])).toBe('{"alpha","has space","has \\"quote\\"","has\\\\slash"}');
+    expect(toPgTextArray([])).toBe("{}");
+    expect(toPgTextArray("{}" as unknown as string[])).toBe("{}");
+  });
+});