From 02ee6980ecc951b642db8ab980073361fb4e3082 Mon Sep 17 00:00:00 2001
From: DecafDev <40307803+decaf-dev@users.noreply.github.com>
Date: Tue, 10 Sep 2024 21:11:06 -0600
Subject: [PATCH] refactor: fix smiCache logs

---
 src/svelte/app/components/grid-card.svelte | 24 ++++++++++++++--------
 1 file changed, 15 insertions(+), 9 deletions(-)

diff --git a/src/svelte/app/components/grid-card.svelte b/src/svelte/app/components/grid-card.svelte
index caec80b..4051c11 100644
--- a/src/svelte/app/components/grid-card.svelte
+++ b/src/svelte/app/components/grid-card.svelte
@@ -27,7 +27,8 @@
 
 	type SocialMediaImageResult = {
 		status: "SUCCESS" | "NOT_FOUND" | "EXPIRED" | "NO_IMAGE";
-		url?: string; // Only present when status is 'SUCCESS'
+		websiteUrl: string;
+		smiUrl: string | null;
 	};
 
 	export let displayName: string;
@@ -184,24 +185,29 @@
 	async function getCachedSocialMediaImageUrl(
 		websiteUrl: string,
 	): Promise<SocialMediaImageResult> {
+		Logger.trace({
+			fileName: "grid-card.svelte",
+			functionName: "getCachedSocialMediaUrl",
+			message: "result",
+		});
 		const entry = await getSMICacheEntry(websiteUrl);
 
 		if (entry) {
-			const { socialMediaImageUrl } = entry;
+			const { smiUrl } = entry;
 
-			if (socialMediaImageUrl) {
+			if (smiUrl) {
 				const isExpired = await isSMICacheEntryExpired(entry);
 				if (!isExpired) {
-					return { status: "SUCCESS", url: socialMediaImageUrl };
+					return { status: "SUCCESS", websiteUrl, smiUrl };
 				} else {
-					return { status: "EXPIRED" }; // Image found but expired
+					return { status: "EXPIRED", websiteUrl, smiUrl }; // Image found but expired
 				}
 			} else {
-				return { status: "NO_IMAGE" }; // Social image was fetched but doesn't exist
+				return { status: "NO_IMAGE", websiteUrl, smiUrl }; // Image was previously cached but doesn't exist
 			}
 		}
 
-		return { status: "NOT_FOUND" }; // Image not cached
+		return { status: "NOT_FOUND", websiteUrl, smiUrl: null }; // Website URL doesn't have a cached image
 	}
 
 	$: if (imageUrl) {
@@ -218,9 +224,9 @@
 				},
 			);
 
-			const { status, url } = result;
+			const { status, smiUrl } = result;
 			if (status === "SUCCESS") {
-				imgSrc = url!;
+				imgSrc = smiUrl;
 			} else if (status === "EXPIRED" || status === "NOT_FOUND") {
 				imgSrc = imageUrl;
 			} else if (status === "NO_IMAGE") {