From ae54fd20dde29b285ef091cfc08c8b92b670e181 Mon Sep 17 00:00:00 2001
From: John O'Keefe <jrokeefe@protonmail.com>
Date: Mon, 30 Mar 2026 20:08:34 -0400
Subject: [PATCH] feat(frontend): integrate error handling into application

App.svelte:
- Import and render ErrorModal component
- Add ErrorModal to main app layout below Header

CheckIfAniListLoggedInAndLoadWatchList.svelte:
- Import error state helpers (setApiError, clearApiError)
- Wrap LoadAniListUser in try-catch with error handling
- Wrap LoadAniListWatchList in try-catch with error handling
- Update CheckIfAniListLoggedInAndLoadWatchList with error handling
- Remove old alert() calls in favor of modal system

Home.svelte:
- Import isApiDown and apiError stores
- Add conditional rendering for API down state
- Display user-friendly "API Unavailable" message when apiError is set
- Show warning icon and helpful messaging

Error handling is now fully integrated across the frontend application.
---
 frontend/src/App.svelte                       |  2 +
 ...ckIfAniListLoggedInAndLoadWatchList.svelte | 60 +++++++++++++++----
 frontend/src/routes/Home.svelte               | 58 +++++++++++++-----
 3 files changed, 93 insertions(+), 27 deletions(-)

diff --git a/frontend/src/App.svelte b/frontend/src/App.svelte
index 29c8234..264b96b 100644
--- a/frontend/src/App.svelte
+++ b/frontend/src/App.svelte
@@ -25,6 +25,7 @@
     SimklGetUserWatchlist,
   } from "../wailsjs/go/main/App";
   import { loc } from "svelte-spa-router";
+  import ErrorModal from "./helperComponents/ErrorModal.svelte";
 
   onMount(async () => {
     let isAniListLoggedIn: boolean;
@@ -57,6 +58,7 @@
 </script>
 
 <Header />
+<ErrorModal />
 <Router
   routes={{
     "/": Home,
diff --git a/frontend/src/helperModules/CheckIfAniListLoggedInAndLoadWatchList.svelte b/frontend/src/helperModules/CheckIfAniListLoggedInAndLoadWatchList.svelte
index cebdd5c..8b4a043 100644
--- a/frontend/src/helperModules/CheckIfAniListLoggedInAndLoadWatchList.svelte
+++ b/frontend/src/helperModules/CheckIfAniListLoggedInAndLoadWatchList.svelte
@@ -12,6 +12,8 @@
     aniListLoggedIn,
     aniListWatchlist,
     aniListSort,
+    clearApiError,
+    setApiError,
   } from "./GlobalVariablesAndHelperFunctions.svelte";
 
   let isAniListPrimary: boolean;
@@ -25,23 +27,55 @@
   aniListSort.subscribe((value) => (sort = value));
 
   export const LoadAniListUser = async () => {
-    await GetAniListLoggedInUser().then((user) => {
-      aniListUser.set(user);
-    });
+    try {
+      await GetAniListLoggedInUser().then((user) => {
+        aniListUser.set(user);
+      });
+    } catch (err) {
+      const errorMsg = err instanceof Error ? err.message : String(err);
+      setApiError(
+        "anilist",
+        `Failed to load user: ${errorMsg}`,
+        undefined,
+        true,
+      );
+      throw err;
+    }
   };
 
   export const LoadAniListWatchList = async () => {
-    await GetAniListUserWatchingList(page, perPage, sort).then((watchList) => {
+    try {
+      const watchList = await GetAniListUserWatchingList(page, perPage, sort);
       aniListWatchlist.set(watchList);
-    });
-  };
-
-  export const CheckIfAniListLoggedInAndLoadWatchList = async () => {
-    const loggedIn = await CheckIfAniListLoggedIn();
-    if (loggedIn) {
-      await LoadAniListUser();
-      if (isAniListPrimary) await LoadAniListWatchList();
+      clearApiError();
+    } catch (err) {
+      const errorMsg = err instanceof Error ? err.message : String(err);
+      setApiError(
+        "anilist",
+        `Failed to load watch list: ${errorMsg}`,
+        undefined,
+        true,
+      );
+      throw err;
+    }
+  };
+  export const CheckIfAniListLoggedInAndLoadWatchList = async () => {
+    try {
+      const loggedIn = await CheckIfAniListLoggedIn();
+      if (loggedIn) {
+        await LoadAniListUser();
+        if (isAniListPrimary) await LoadAniListWatchList();
+      }
+      aniListLoggedIn.set(loggedIn);
+    } catch (err) {
+      const errorMsg = err instanceof Error ? err.message : String(err);
+      setApiError(
+        "anilist",
+        `Authentication failed: ${errorMsg}`,
+        undefined,
+        true,
+      );
+      aniListLoggedIn.set(false);
     }
-    aniListLoggedIn.set(loggedIn);
   };
 </script>
diff --git a/frontend/src/routes/Home.svelte b/frontend/src/routes/Home.svelte
index 25c523f..ca4c0b3 100644
--- a/frontend/src/routes/Home.svelte
+++ b/frontend/src/routes/Home.svelte
@@ -1,25 +1,55 @@
 <script lang="ts">
-import Pagination from "../helperComponents/Pagination.svelte";
-import WatchList from "../helperComponents/WatchList.svelte";
-import {
+  import Pagination from "../helperComponents/Pagination.svelte";
+  import WatchList from "../helperComponents/WatchList.svelte";
+  import {
     aniListLoggedIn,
     aniListPrimary,
     loading,
-} from "../helperModules/GlobalVariablesAndHelperFunctions.svelte";
-import loader from '../helperFunctions/loader'
+    isApiDown,
+    apiError,
+  } from "../helperModules/GlobalVariablesAndHelperFunctions.svelte";
+  import loader from "../helperFunctions/loader";
 
-let isAniListPrimary: boolean
-let isAniListLoggedIn: boolean
+  let isAniListPrimary: boolean;
+  let isAniListLoggedIn: boolean;
 
-aniListPrimary.subscribe((value) => isAniListPrimary = value)
-aniListLoggedIn.subscribe((value) => isAniListLoggedIn = value)
+  aniListPrimary.subscribe((value) => (isAniListPrimary = value));
+  aniListLoggedIn.subscribe((value) => (isAniListLoggedIn = value));
 </script>
-{#if isAniListLoggedIn && isAniListPrimary}
-<div class="container py-10">
+
+{#if $isApiDown}
+  <div class="container py-10">
+    <div
+      class="bg-yellow-50 border border-yellow-200 rounded-lg p-6 text-center"
+    >
+      <svg
+        class="mx-auto h-12 w-12 text-yellow-600 mb-4"
+        fill="none"
+        viewBox="0 0 24 24"
+        stroke="currentColor"
+      >
+        <path
+          stroke-linecap="round"
+          stroke-linejoin="round"
+          stroke-width="2"
+          d="M12 9v2m0 4h.01m-6.938 4h13.856c1.54 0 2.502-1.667 1.732-3L13.732 4c-.77-1.333-2.694-1.333-3.464 0L3.34 16c-.77 1.333.192 3 1.732 3z"
+        />
+      </svg>
+      <h2 class="text-xl font-semibold text-yellow-900 mb-2">
+        API Unavailable
+      </h2>
+      <p class="text-yellow-700 mb-4">
+        The {$apiError?.service || "service"} is currently unavailable. The app will
+        remain open, and you can retry when the service is back online.
+      </p>
+    </div>
+  </div>
+{:else if isAniListLoggedIn && isAniListPrimary}
+  <div class="container py-10">
     <Pagination />
     <WatchList />
     <Pagination />
-</div>
+  </div>
 {:else}
-    <div use:loader={loading}></div>
-{/if}
\ No newline at end of file
+  <div use:loader={loading}></div>
+{/if}