Merge branch '#242-routing-and-ssr'

src/lib/utils/websocket_utils.ts

@@ -22,13 +22,18 @@ export interface NostrFilter {
   limit?: number;
 }
 
-export async function fetchNostrEvent(filter: NostrFilter): Promise<NostrEvent> {
-  // TODO: Improve relay selection when relay management is implemented.
-  const ws = await WebSocketPool.instance.acquire("wss://thecitadel.nostr1.com");
-  const subId = crypto.randomUUID();
+type ResolveCallback<T> = (value: T | PromiseLike<T>) => void;
+type RejectCallback = (reason?: any) => void;
+type EventHandler = (ev: Event) => void;
+type EventHandlerReject = (reject: RejectCallback) => EventHandler; 
+type EventHandlerResolve<T> = (resolve: ResolveCallback<T>) => EventHandlerReject;
 
-  const res = new Promise<NostrEvent>((resolve, reject) => {
-    ws.addEventListener("message", (ev) => {
+function handleMessage(
+  ev: MessageEvent,
+  subId: string,
+  resolve: (event: NostrEvent) => void,
+  reject: (reason: any) => void
+) {
   const data = JSON.parse(ev.data);
 
   if (data[1] !== subId) {
@@ -52,12 +57,52 @@ export async function fetchNostrEvent(filter: NostrFilter): Promise<NostrEvent>
   }
 
   resolve(event);
-    });
+}
 
-    ws.addEventListener("error", (ev) => {
+function handleError(
+  ev: Event,
+  reject: (reason: any) => void
+) {
   reject(ev);
+}
+
+export async function fetchNostrEvent(filter: NostrFilter): Promise<NostrEvent> {
+  // TODO: Improve relay selection when relay management is implemented.
+  const ws = await WebSocketPool.instance.acquire("wss://thecitadel.nostr1.com");
+  const subId = crypto.randomUUID();
+
+  // AI-NOTE: Currying is used here to abstract the internal handler logic away from the WebSocket
+  // handling logic. The message and error handlers themselves can be refactored without affecting
+  // the WebSocket handling logic.
+  const curriedMessageHandler: (subId: string) => EventHandlerResolve<NostrEvent> =
+    (subId) =>
+      (resolve) =>
+        (reject) =>
+          (ev: MessageEvent) =>
+            handleMessage(ev, subId, resolve, reject);
+  const curriedErrorHandler: EventHandlerReject =
+    (reject) =>
+      (ev: Event) =>
+        handleError(ev, reject);
+
+  // AI-NOTE: These variables store references to partially-applied handlers so that the `finally`
+  // block receives the correct references to clean up the listeners.
+  let messageHandler: EventHandler;
+  let errorHandler: EventHandler;
+
+  const res = new Promise<NostrEvent>((resolve, reject) => {
+    messageHandler = curriedMessageHandler(subId)(resolve)(reject);
+    errorHandler = curriedErrorHandler(reject);
+
+    ws.addEventListener("message", messageHandler);
+    ws.addEventListener("error", errorHandler);
+  })
+  .withTimeout(2000)
+  .finally(() => {
+    ws.removeEventListener("message", messageHandler);
+    ws.removeEventListener("error", errorHandler);
+    WebSocketPool.instance.release(ws);
   });
-  }).withTimeout(2000);
 
   ws.send(JSON.stringify(["REQ", subId, filter]));
   return res;
@@ -70,14 +115,14 @@ export async function fetchEventById(id: string): Promise<NostrEvent> {
   try {
     const event = await fetchNostrEvent({ ids: [id], limit: 1 });
     if (!event) {
-      throw error(404, `Event not found for ID: ${id}`);
+      error(404, `Event not found for ID: ${id}`);
     }
     return event;
   } catch (err) {
     if (err && typeof err === "object" && "status" in err) {
       throw err;
     }
-    throw error(404, `Failed to fetch event by ID: ${err}`);
+    error(404, `Failed to fetch event by ID: ${err}`);
   }
 }
 
@@ -88,14 +133,14 @@ export async function fetchEventByDTag(dTag: string): Promise<NostrEvent> {
   try {
     const event = await fetchNostrEvent({ "#d": [dTag], limit: 1 });
     if (!event) {
-      throw error(404, `Event not found for d-tag: ${dTag}`);
+      error(404, `Event not found for d-tag: ${dTag}`);
     }
     return event;
   } catch (err) {
     if (err && typeof err === "object" && "status" in err) {
       throw err;
     }
-    throw error(404, `Failed to fetch event by d-tag: ${err}`);
+    error(404, `Failed to fetch event by d-tag: ${err}`);
   }
 }
 
@@ -112,14 +157,14 @@ export async function fetchEventByNaddr(naddr: string): Promise<NostrEvent> {
     };
     const event = await fetchNostrEvent(filter);
     if (!event) {
-      throw error(404, `Event not found for naddr: ${naddr}`);
+      error(404, `Event not found for naddr: ${naddr}`);
     }
     return event;
   } catch (err) {
     if (err && typeof err === "object" && "status" in err) {
       throw err;
     }
-    throw error(404, `Failed to fetch event by naddr: ${err}`);
+    error(404, `Failed to fetch event by naddr: ${err}`);
   }
 }
 
@@ -131,13 +176,13 @@ export async function fetchEventByNevent(nevent: string): Promise<NostrEvent> {
     const decoded = neventDecode(nevent);
     const event = await fetchNostrEvent({ ids: [decoded.id], limit: 1 });
     if (!event) {
-      throw error(404, `Event not found for nevent: ${nevent}`);
+      error(404, `Event not found for nevent: ${nevent}`);
     }
     return event;
   } catch (err) {
     if (err && typeof err === "object" && "status" in err) {
       throw err;
     }
-    throw error(404, `Failed to fetch event by nevent: ${err}`);
+    error(404, `Failed to fetch event by nevent: ${err}`);
   }
 }

src/routes/publication/+page.server.ts

@@ -25,17 +25,17 @@ export const load: PageServerLoad = ({ url }) => {
   if (id) {
     // Check if id is an naddr or nevent
     if (id.startsWith(IDENTIFIER_PREFIXES.NADDR)) {
-      throw redirect(301, `${ROUTES.NADDR}/${id}`);
+      redirect(301, `${ROUTES.NADDR}/${id}`);
     } else if (id.startsWith(IDENTIFIER_PREFIXES.NEVENT)) {
-      throw redirect(301, `${ROUTES.NEVENT}/${id}`);
+      redirect(301, `${ROUTES.NEVENT}/${id}`);
     } else {
       // Assume it's a hex ID
-      throw redirect(301, `${ROUTES.ID}/${id}`);
+      redirect(301, `${ROUTES.ID}/${id}`);
     }
   } else if (dTag) {
-    throw redirect(301, `${ROUTES.D_TAG}/${dTag}`);
+    redirect(301, `${ROUTES.D_TAG}/${dTag}`);
   }
 
   // If no query parameters, redirect to the start page
-  throw redirect(301, ROUTES.START);
+  redirect(301, ROUTES.START);
 }; 

src/routes/publication/[type]/[identifier]/+layout.server.ts

@@ -7,7 +7,7 @@ export const load: LayoutServerLoad = async ({ params, url }) => {
   // Validate the identifier type for SSR
   const validTypes = ['id', 'd', 'naddr', 'nevent'];
   if (!validTypes.includes(type)) {
-    throw error(400, `Unsupported identifier type: ${type}`);
+    error(400, `Unsupported identifier type: ${type}`);
   }
 
   // Provide basic metadata for SSR - actual fetching will happen on client

src/routes/publication/[type]/[identifier]/+layout.ts

@@ -4,6 +4,8 @@ import { fetchEventByDTag, fetchEventById, fetchEventByNaddr, fetchEventByNevent
 import type { NostrEvent } from "../../../../lib/utils/websocket_utils.ts";
 import { browser } from "$app/environment";
 
+export const ssr = true;
+
 export const load: LayoutLoad = async ({ params, url }) => {
   const { type, identifier } = params;
 
@@ -39,7 +41,7 @@ export const load: LayoutLoad = async ({ params, url }) => {
         indexEvent = await fetchEventByNevent(identifier);
         break;
       default:
-        throw error(400, `Unsupported identifier type: ${type}`);
+        error(400, `Unsupported identifier type: ${type}`);
     }
 
     // Extract metadata for meta tags
@@ -60,6 +62,6 @@ export const load: LayoutLoad = async ({ params, url }) => {
     };
   } catch (err) {
     console.error('Failed to fetch publication:', err);
-    throw error(404, `Failed to load publication: ${err}`);
+    error(404, `Failed to load publication: ${err}`);
   }
 };

src/routes/publication/[type]/[identifier]/+page.server.ts

@@ -7,7 +7,7 @@ export const load: PageServerLoad = async ({ params }) => {
   // Validate the identifier type for SSR
   const validTypes = ['id', 'd', 'naddr', 'nevent'];
   if (!validTypes.includes(type)) {
-    throw error(400, `Unsupported identifier type: ${type}`);
+    error(400, `Unsupported identifier type: ${type}`);
   }
 
   // Provide basic data for SSR - actual fetching will happen on client

src/routes/publication/[type]/[identifier]/+page.ts

@@ -34,11 +34,11 @@ export const load: PageLoad = async ({ params }) => {
         indexEvent = await fetchEventByNevent(identifier);
         break;
       default:
-        throw error(400, `Unsupported identifier type: ${type}`);
+        error(400, `Unsupported identifier type: ${type}`);
     }
 
     if (!indexEvent) {
-      throw error(404, `Event not found for ${type}: ${identifier}`);
+      error(404, `Event not found for ${type}: ${identifier}`);
     }
 
     const publicationType = indexEvent.tags.find((tag) => tag[0] === "type")?.[1] ?? "";
@@ -49,6 +49,6 @@ export const load: PageLoad = async ({ params }) => {
     };
   } catch (err) {
     console.error('Failed to fetch publication:', err);
-    throw error(404, `Failed to load publication: ${err}`);
+    error(404, `Failed to load publication: ${err}`);
   }
 };