From 73fbf2e0be181cf84537fcd464852373a989a0d1 Mon Sep 17 00:00:00 2001
From: quexeky <git@quexeky.dev>
Date: Wed, 12 Feb 2025 09:29:03 +1100
Subject: [PATCH] feat(library): Reactive library updating

Signed-off-by: quexeky <git@quexeky.dev>
---
 desktop/components/Header.vue              |  4 +-
 desktop/composables/current-page-engine.ts |  4 +-
 desktop/pages/library.vue                  | 74 ++++++++++++----------
 desktop/pages/settings.vue                 |  8 +--
 desktop/src-tauri/src/games/commands.rs    |  1 -
 desktop/src-tauri/src/games/library.rs     | 11 ++--
 6 files changed, 57 insertions(+), 45 deletions(-)

diff --git a/desktop/components/Header.vue b/desktop/components/Header.vue
index 75537eb1..c8a82d3f 100644
--- a/desktop/components/Header.vue
+++ b/desktop/components/Header.vue
@@ -11,7 +11,7 @@
               v-for="(nav, navIdx) in navigation"
               :class="[
                 'transition  uppercase font-display font-semibold text-md',
-                navIdx === currentPageIndex
+                navIdx === currentNavigation
                   ? 'text-zinc-100'
                   : 'text-zinc-400 hover:text-zinc-200',
               ]"
@@ -78,7 +78,7 @@ const navigation: Array<NavigationItem> = [
   },
 ];
 
-const currentPageIndex = useCurrentNavigationIndex(navigation);
+const {currentNavigation} = useCurrentNavigationIndex(navigation);
 
 const quickActions: Array<QuickActionNav> = [
   {
diff --git a/desktop/composables/current-page-engine.ts b/desktop/composables/current-page-engine.ts
index 463ad197..0990195c 100644
--- a/desktop/composables/current-page-engine.ts
+++ b/desktop/composables/current-page-engine.ts
@@ -26,5 +26,7 @@ export const useCurrentNavigationIndex = (
     currentNavigation.value = calculateCurrentNavIndex(to);
   });
 
-  return currentNavigation;
+  return {currentNavigation, recalculateNavigation: () => {
+    currentNavigation.value = calculateCurrentNavIndex(route);
+  }};
 };
diff --git a/desktop/pages/library.vue b/desktop/pages/library.vue
index 182699dd..6c8f6d3f 100644
--- a/desktop/pages/library.vue
+++ b/desktop/pages/library.vue
@@ -1,28 +1,17 @@
 <template>
   <div class="flex flex-row h-full">
-    <div
-      class="flex-none max-h-full overflow-y-auto w-64 bg-zinc-950 px-2 py-1"
-    >
+    <div class="flex-none max-h-full overflow-y-auto w-64 bg-zinc-950 px-2 py-1">
       <ul class="flex flex-col gap-y-1">
-        <NuxtLink
-          v-for="(nav, navIdx) in navigation"
-          :key="nav.route"
-          :class="[
-            'transition-all duration-200 rounded-lg flex items-center py-1.5 px-3',
-            navIdx === currentNavigationIndex
-              ? 'bg-zinc-800 text-zinc-100'
-              : nav.isInstalled.value
+        <NuxtLink v-for="(nav, navIdx) in navigation" :key="nav.route" :class="[
+          'transition-all duration-200 rounded-lg flex items-center py-1.5 px-3',
+          navIdx === currentNavigation
+            ? 'bg-zinc-800 text-zinc-100'
+            : nav.isInstalled.value
               ? 'text-zinc-300 hover:bg-zinc-800/90 hover:text-zinc-200'
               : 'text-zinc-500 hover:bg-zinc-800/70 hover:text-zinc-300',
-          ]"
-          :href="nav.route"
-        >
+        ]" :href="nav.route">
           <div class="flex items-center w-full gap-x-3">
-            <img
-              class="size-6 flex-none object-cover bg-zinc-900 rounded"
-              :src="icons[navIdx]"
-              alt=""
-            />
+            <img class="size-6 flex-none object-cover bg-zinc-900 rounded" :src="icons[nav.id]" alt="" />
             <p class="truncate text-sm font-display leading-6 flex-1">
               {{ nav.label }}
             </p>
@@ -31,35 +20,47 @@
       </ul>
     </div>
     <div class="grow overflow-y-auto">
-      <NuxtPage :libraryDownloadError = "libraryDownloadError" />
+      <NuxtPage :libraryDownloadError="libraryDownloadError" />
     </div>
   </div>
 </template>
 
 <script setup lang="ts">
 import { invoke } from "@tauri-apps/api/core";
-import { GameStatusEnum, type Game, type NavigationItem } from "~/types";
+import { listen } from "@tauri-apps/api/event";
+import { GameStatusEnum, type Game, type GameStatus, type NavigationItem } from "~/types";
 
 let libraryDownloadError = false;
 
-async function calculateGames(): Promise<Game[]> {
+const games: { [key: string]: { game: Game, status: Ref<GameStatus, GameStatus> } } = {};
+const icons: { [key: string]: string } = {};
+
+const rawGames: Ref<Game[], Game[]> = ref([])
+
+async function calculateGames() {
   try {
-    return await invoke("fetch_library");
+    rawGames.value = await invoke("fetch_library");
+    for (const game of rawGames.value) {
+      if (games[game.id]) continue;
+      games[game.id] = await useGame(game.id);
+    }
+    for (const game of rawGames.value) {
+      if (icons[game.id]) continue;
+      icons[game.id] = await useObject(game.mIconId);
+    }
   }
-  catch(e) {
+  catch (e) {
     console.log(e)
     libraryDownloadError = true;
     return new Array();
   }
 }
 
-const rawGames: Array<Game> = await calculateGames();
-const games = await Promise.all(rawGames.map((e) => useGame(e.id)));
-const icons = await Promise.all(
-  games.map(({ game, status }) => useObject(game.mIconId))
-);
+await calculateGames();
+
+const navigation = computed(() => rawGames.value.map((game) => {
+  const status = games[game.id].status;
 
-const navigation = games.map(({ game, status }) => {
   const isInstalled = computed(
     () =>
       status.value.type == GameStatusEnum.Installed ||
@@ -71,9 +72,18 @@ const navigation = games.map(({ game, status }) => {
     route: `/library/${game.id}`,
     prefix: `/library/${game.id}`,
     isInstalled,
+    id: game.id,
   };
   return item;
-});
+})
+);
+const { currentNavigation, recalculateNavigation } = useCurrentNavigationIndex(navigation.value);
+
+
+listen("update_library", async (event) => {
+  console.log("Updating library");
+  await calculateGames()
+  recalculateNavigation();
+})
 
-const currentNavigationIndex = useCurrentNavigationIndex(navigation);
 </script>
diff --git a/desktop/pages/settings.vue b/desktop/pages/settings.vue
index 3c39c653..2b92ff94 100644
--- a/desktop/pages/settings.vue
+++ b/desktop/pages/settings.vue
@@ -10,13 +10,13 @@
         <ul role="list" class="-mx-2 space-y-1">
           <li v-for="(item, itemIdx) in navigation" :key="item.prefix">
             <NuxtLink :href="item.route" :class="[
-              itemIdx === currentPageIndex
+              itemIdx === currentNavigation
                 ? 'bg-zinc-800/50 text-zinc-100'
                 : 'text-zinc-400 hover:bg-zinc-800/30 hover:text-zinc-200',
               'transition group flex gap-x-3 rounded-md p-2 pr-12 text-sm font-semibold leading-6',
             ]">
               <component :is="item.icon" :class="[
-                itemIdx === currentPageIndex
+                itemIdx === currentNavigation
                   ? 'text-zinc-100'
                   : 'text-zinc-400 group-hover:text-zinc-200',
                 'transition h-6 w-6 shrink-0',
@@ -112,10 +112,10 @@ const navigation = computed(() => [
 const currentPlatform = platform();
 
 // Use .value to unwrap the computed ref
-const currentPageIndex = useCurrentNavigationIndex(navigation.value);
+const {currentNavigation} = useCurrentNavigationIndex(navigation.value);
 
 // Watch for navigation changes and update currentPageIndex
 watch(navigation, (newNav) => {
-  currentPageIndex.value = useCurrentNavigationIndex(newNav).value;
+  currentNavigation.value = useCurrentNavigationIndex(newNav).currentNavigation.value;
 });
 </script>
diff --git a/desktop/src-tauri/src/games/commands.rs b/desktop/src-tauri/src/games/commands.rs
index 88eb7530..cf4f573a 100644
--- a/desktop/src-tauri/src/games/commands.rs
+++ b/desktop/src-tauri/src/games/commands.rs
@@ -38,7 +38,6 @@ pub fn uninstall_game(game_id: String, app_handle: AppHandle) -> Result<(), Libr
         Some(data) => data,
         None => return Err(LibraryError::MetaNotFound(game_id)),
     };
-    println!("{:?}", meta);
     uninstall_game_logic(meta, &app_handle);
 
     Ok(())
diff --git a/desktop/src-tauri/src/games/library.rs b/desktop/src-tauri/src/games/library.rs
index 20a5a524..1be4a1ee 100644
--- a/desktop/src-tauri/src/games/library.rs
+++ b/desktop/src-tauri/src/games/library.rs
@@ -103,7 +103,6 @@ pub fn fetch_library_logic(
     drop(db_handle);
     cache_object("library", &games)?;
 
-
     Ok(games)
 }
 pub fn fetch_library_logic_offline(
@@ -137,8 +136,7 @@ pub fn fetch_game_logic(
             status,
         };
 
-
-        cache_object(id, &data)?;    
+        cache_object(id, &data)?;
 
         return Ok(data);
     }
@@ -261,8 +259,6 @@ pub fn uninstall_game_logic(meta: DownloadableMetadata, app_handle: &AppHandle)
             .entry(meta.clone())
             .and_modify(|v| *v = ApplicationTransientStatus::Uninstalling {});
 
-        db_handle.applications.installed_game_version.remove(&meta.id);
-        
         drop(db_handle);
 
         let app_handle = app_handle.clone();
@@ -273,6 +269,10 @@ pub fn uninstall_game_logic(meta: DownloadableMetadata, app_handle: &AppHandle)
             Ok(_) => {
                 let mut db_handle = borrow_db_mut_checked();
                 db_handle.applications.transient_statuses.remove(&meta);
+                db_handle
+                    .applications
+                    .installed_game_version
+                    .remove(&meta.id);
                 db_handle
                     .applications
                     .game_statuses
@@ -282,6 +282,7 @@ pub fn uninstall_game_logic(meta: DownloadableMetadata, app_handle: &AppHandle)
                 save_db();
 
                 debug!("uninstalled game id {}", &meta.id);
+                app_handle.emit("update_library", {}).unwrap();
 
                 push_game_update(
                     &app_handle,