refactor(routes): improve conversation page loading and error handling

Author: nsarrazin

src/routes/+layout.ts

@@ -57,7 +57,6 @@ export const load = async ({ depends, fetch }) => {
 								avatarUrl: fetch(`${base}/api/v2/assistants/${conv.assistantId}`)
 									.then((res) => res.json() as Promise<Serialize<Assistant>>)
 									.then((assistant) => {
-										console.log(assistant);
 										if (!assistant.avatar) {
 											return undefined;
 										}

src/routes/conversation/[id]/+page.ts

@@ -1,19 +1,17 @@
 import { base } from "$app/paths";
 import type { GETConversationResponse } from "$lib/server/api/routes/groups/conversations.js";
 import { UrlDependency } from "$lib/types/UrlDependency";
-import { error } from "@sveltejs/kit";
+import { fetchJSON } from "$lib/utils/fetchJSON.js";
+import { redirect } from "@sveltejs/kit";
 
 export const load = async ({ params, depends, fetch }) => {
 	depends(UrlDependency.Conversation);
 
-	const r = await fetch(`${base}/api/v2/conversations/${params.id}`);
-
-	if (!r.ok) {
-		console.log({ r });
-		error(r.status, "Failed to fetch conversation");
+	try {
+		return await fetchJSON<GETConversationResponse>(`${base}/api/v2/conversations/${params.id}`, {
+			fetch,
+		});
+	} catch {
+		redirect(302, "/");
 	}
-
-	const data = await r.json();
-
-	return data as GETConversationResponse;
 };