From 0ede4c72601e776cdd4da6390800e50d4c1bd37b Mon Sep 17 00:00:00 2001
From: trafficlunar <hello@trafficlunar.net>
Date: Sun, 26 Apr 2026 15:59:14 +0100
Subject: [PATCH] fix: profile picture null causes crash

---
 frontend/src/components/header.tsx    | 8 +++++++-
 frontend/src/pages/profile/layout.tsx | 6 ++----
 frontend/src/pages/report/user.tsx    | 2 +-
 3 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/frontend/src/components/header.tsx b/frontend/src/components/header.tsx
index c3d8493..9da9bb4 100644
--- a/frontend/src/components/header.tsx
+++ b/frontend/src/components/header.tsx
@@ -69,7 +69,13 @@ export default function Header() {
 								data-tooltip="Your Profile"
 							>
 								<img
-									src={$session.user.image.startsWith("/profile") ? `${import.meta.env.VITE_API_URL}${$session.user.image}` : $session.user.image}
+									src={
+										$session?.user?.image
+											? $session.user.image.startsWith("/profile")
+												? `${import.meta.env.VITE_API_URL}${$session.user.image}`
+												: $session.user.image
+											: "/guest.png"
+									}
 									onError={(e) => {
 										e.currentTarget.onerror = null; // Prevent infinite loops
 										e.currentTarget.src = "/guest.png";
diff --git a/frontend/src/pages/profile/layout.tsx b/frontend/src/pages/profile/layout.tsx
index 4c6f3a0..a1f4dda 100644
--- a/frontend/src/pages/profile/layout.tsx
+++ b/frontend/src/pages/profile/layout.tsx
@@ -51,9 +51,7 @@ export default function ProfileLayout() {
 	const joinDate = new Date(user.createdAt).toLocaleDateString("en-US", { month: "long", year: "numeric" });
 	const metaTitle = `${user.name} - TomodachiShare`;
 	const metaDescription = `View ${user.name}'s profile on TomodachiShare. Creator of ${user._count.miis} Miis. Member since ${joinDate}.`;
-	const metaImage = user.image.startsWith("/profile")
-		? `${import.meta.env.VITE_API_URL}${user.image}`
-		: (user.image ?? `${import.meta.env.VITE_API_URL}/guest.png`);
+	const metaImage = user.image ? (user.image.startsWith("/profile") ? `${import.meta.env.VITE_API_URL}${user.image}` : user.image) : "/guest.png";
 
 	return (
 		<div>
@@ -81,7 +79,7 @@ export default function ProfileLayout() {
 					{/* Profile picture */}
 					<Link to={`/profile/${user.id}`} className="size-28 aspect-square">
 						<img
-							src={user.image.startsWith("/profile") ? `${import.meta.env.VITE_API_URL}${user.image}` : user.image}
+							src={user.image ? (user.image.startsWith("/profile") ? `${import.meta.env.VITE_API_URL}${user.image}` : user.image) : "/guest.png"}
 							onError={(e) => {
 								e.currentTarget.onerror = null; // Prevent infinite loops
 								e.currentTarget.src = "/guest.png";
diff --git a/frontend/src/pages/report/user.tsx b/frontend/src/pages/report/user.tsx
index 4574eb6..4346590 100644
--- a/frontend/src/pages/report/user.tsx
+++ b/frontend/src/pages/report/user.tsx
@@ -65,7 +65,7 @@ export default function ReportUserPage() {
 
 				<div className="bg-orange-100 rounded-xl border-2 border-orange-400 flex p-4 gap-4">
 					<img
-						src={user.image.startsWith("/profile") ? `${import.meta.env.VITE_API_URL}${user.image}` : user.image}
+						src={user.image ? (user.image.startsWith("/profile") ? `${import.meta.env.VITE_API_URL}${user.image}` : user.image) : "/guest.png"}
 						onError={(e) => {
 							e.currentTarget.onerror = null; // Prevent infinite loops
 							e.currentTarget.src = "/guest.png";