fix: issues with page loading & assistant avatars

Author: nsarrazin

src/lib/components/NavConversationItem.svelte

@@ -40,11 +40,13 @@
 		{/if}
 		{#if conv.avatarUrl}
 			{#await conv.avatarUrl then avatarUrl}
-				<img
-					src="{base}{avatarUrl}"
-					alt="Assistant avatar"
-					class="mr-1.5 inline size-4 flex-none rounded-full object-cover"
-				/>
+				{#if avatarUrl}
+					<img
+						src="{base}{avatarUrl}"
+						alt="Assistant avatar"
+						class="mr-1.5 inline size-4 flex-none rounded-full object-cover"
+					/>
+				{/if}
 			{/await}
 			{conv.title.replace(/\p{Emoji}/gu, "")}
 		{:else if conv.assistantId}

src/lib/types/ConvSidebar.ts

@@ -4,5 +4,5 @@ export interface ConvSidebar {
 	updatedAt: Date;
 	model?: string;
 	assistantId?: string;
-	avatarUrl?: string | Promise<string>;
+	avatarUrl?: string | Promise<string | undefined>;
 }

src/lib/utils/serialize.ts

@@ -1,4 +1,4 @@
-import { ObjectId } from "mongodb";
+import type { ObjectId } from "mongodb";
 
 export type Serialize<T> = T extends ObjectId | Date
 	? string

src/routes/+layout.ts

@@ -56,10 +56,14 @@ export const load = async ({ depends, fetch }) => {
 								assistantId: conv.assistantId.toString(),
 								avatarUrl: fetch(`${base}/api/v2/assistants/${conv.assistantId}`)
 									.then((res) => res.json() as Promise<Serialize<Assistant>>)
-									.then(
-										(assistant) =>
-											`/settings/assistants/${conv.assistantId}/avatar.jpg?hash=${assistant.avatar}`
-									),
+									.then((assistant) => {
+										console.log(assistant);
+										if (!assistant.avatar) {
+											return undefined;
+										}
+
+										return `/settings/assistants/${conv.assistantId}/avatar.jpg?hash=${assistant.avatar}`;
+									}),
 							}
 						: {}),
 				} satisfies ConvSidebar;