@valentinkolb/cloud 0.5.2 → 0.5.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@valentinkolb/cloud",
-  "version": "0.5.2",
+  "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/api/admin-core-settings.ts

@@ -11,10 +11,14 @@ import { z } from "zod";
 import { listApps } from "../_internal/registry";
 import { auth, v, type AuthContext } from "../server";
 import { settingsDeleteLegacyKeys, settingsListLegacyKeys } from "../services";
+import { sendEmail } from "../services/notifications/email";
 import * as settings from "../services/settings";
 import { SETTINGS_MAP } from "../services/settings/defaults";
 
 const BulkUpdateSchema = z.record(z.string(), z.unknown());
+const TestEmailSchema = z.object({
+  recipient: z.email(),
+});
 
 type FieldErrors = Record<string, string>;
 
@@ -28,6 +32,24 @@ const app = new Hono<AuthContext>()
   .delete("/legacy", auth.requireRole("admin"), async (c) => {
     return c.json(await settingsDeleteLegacyKeys(await liveSettingKeys()));
   })
+  .post("/test-email", auth.requireRole("admin"), v("json", TestEmailSchema), async (c) => {
+    const { recipient } = c.req.valid("json");
+    const sentAt = new Date().toISOString();
+
+    try {
+      await sendEmail(recipient, "Cloud test email", {
+        rawHtml: `
+          <p>This is a test email from Cloud.</p>
+          <p>If you received this message, SMTP delivery is configured correctly.</p>
+          <p style="margin-top:24px;color:#71717a;font-size:12px;">Sent at ${sentAt}</p>
+        `,
+      });
+      return c.json({ ok: true });
+    } catch (error) {
+      const message = error instanceof Error ? error.message : "Failed to send test email";
+      return c.json({ message }, 500);
+    }
+  })
   .put(
     "/",
     auth.requireRole("admin"),

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/notifications/index.ts

@@ -9,6 +9,11 @@ const log = logger("notifications");
 export type NotificationType = "email";
 export type NotificationStatus = "sent" | "pending" | "error";
 export type NotificationStatusSummary = Record<NotificationStatus, number>;
+export type NotificationSearchSummary = NotificationStatusSummary & {
+  total: number;
+  system: number;
+  latestCreatedAt: Date | null;
+};
 
 /**
  * Computes notification delivery status from sent/error timestamps.
@@ -63,6 +68,15 @@ const emptyStatusSummary = (): NotificationStatusSummary => ({
   error: 0,
 });
 
+const emptySearchSummary = (): NotificationSearchSummary => ({
+  total: 0,
+  sent: 0,
+  pending: 0,
+  error: 0,
+  system: 0,
+  latestCreatedAt: null,
+});
+
 type DbNotificationRow = {
   id: string;
   type: NotificationType;
@@ -316,6 +330,79 @@ export const getStatusSummary = async (options?: {
   return summary;
 };
 
+/**
+ * Count statuses across all entries matching a search term in the current access scope.
+ */
+export const getSearchSummary = async (options: {
+  search: string;
+  sentBy?: string;
+  isAdmin?: boolean;
+}): Promise<NotificationSearchSummary> => {
+  const { sentBy, isAdmin } = options;
+  const search = options.search.trim();
+  if (!search) return emptySearchSummary();
+
+  const searchPattern = `%${escapeLikePattern(search)}%`;
+  let rows: Array<{
+    total: number | string;
+    sent: number | string;
+    pending: number | string;
+    error: number | string;
+    system: number | string;
+    latest_created_at: Date | null;
+  }> = [];
+
+  if (isAdmin) {
+    rows = await sql`
+      SELECT
+        COUNT(*)::int AS total,
+        COUNT(*) FILTER (WHERE sent_at IS NOT NULL)::int AS sent,
+        COUNT(*) FILTER (WHERE sent_at IS NULL AND error IS NULL)::int AS pending,
+        COUNT(*) FILTER (WHERE sent_at IS NULL AND error IS NOT NULL)::int AS error,
+        COUNT(*) FILTER (WHERE sent_by IS NULL)::int AS system,
+        MAX(created_at) AS latest_created_at
+      FROM notifications.messages
+      WHERE subject ILIKE ${searchPattern} ESCAPE '\'
+        OR content ILIKE ${searchPattern} ESCAPE '\'
+        OR recipient ILIKE ${searchPattern} ESCAPE '\'
+    `;
+  } else if (sentBy) {
+    rows = await sql`
+      SELECT
+        COUNT(*)::int AS total,
+        COUNT(*) FILTER (WHERE sent_at IS NOT NULL)::int AS sent,
+        COUNT(*) FILTER (WHERE sent_at IS NULL AND error IS NULL)::int AS pending,
+        COUNT(*) FILTER (WHERE sent_at IS NULL AND error IS NOT NULL)::int AS error,
+        COUNT(*) FILTER (WHERE sent_by IS NULL)::int AS system,
+        MAX(created_at) AS latest_created_at
+      FROM notifications.messages
+      WHERE sent_by = ${sentBy}
+        AND (
+          subject ILIKE ${searchPattern} ESCAPE '\'
+          OR content ILIKE ${searchPattern} ESCAPE '\'
+          OR recipient ILIKE ${searchPattern} ESCAPE '\'
+        )
+    `;
+  }
+
+  const row = rows[0];
+  if (!row) return emptySearchSummary();
+
+  const toNumber = (value: number | string) => {
+    const parsed = typeof value === "string" ? Number.parseInt(value, 10) : value;
+    return Number.isFinite(parsed) ? parsed : 0;
+  };
+
+  return {
+    total: toNumber(row.total),
+    sent: toNumber(row.sent),
+    pending: toNumber(row.pending),
+    error: toNumber(row.error),
+    system: toNumber(row.system),
+    latestCreatedAt: row.latest_created_at,
+  };
+};
+
 /**
  * Get a single notification by ID.
  */
@@ -481,4 +568,5 @@ export const notifications = {
   getPendingSystemCount,
   sendAllPendingSystem,
   getStatusSummary,
+  getSearchSummary,
 };

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("{}");
+  });
+});