fix: getInstances refactor for API rate limiting

packages/dashboard/src/components/Navbar.svelte

@@ -4,13 +4,9 @@
   import MediaQuery from '$components/MediaQuery.svelte'
   import { DOCS_URL } from '$src/env'
   import { handleLogoutAndRedirect } from '$util/database'
-  import { getInstances } from '$util/getInstances'
   import { globalInstancesStore } from '$util/stores'
   import { values } from '@s-libs/micro-dash'
 
-  // This will query the database for all instances and then update the global state
-  getInstances()
-
   const linkClasses =
     'font-medium text-xl text-base-content btn btn-ghost capitalize justify-start'
   const subLinkClasses =

packages/dashboard/src/routes/+layout.svelte

@@ -7,7 +7,10 @@
   import '../app.css'
   import '../services'
 
+  import { getInstances } from '$util/getInstances'
   import { isUserLoggedIn } from '$util/stores'
+
+  getInstances()
 </script>
 
 <Meta />

packages/dashboard/src/routes/app/instances/[instanceId]/+layout.svelte

@@ -2,18 +2,15 @@
   import { page } from '$app/stores'
   import AuthStateGuard from '$components/helpers/AuthStateGuard.svelte'
   import { getSingleInstance } from '$util/getInstances'
-  import { createCleanupManager } from '@pockethost/common'
+  import { assertTruthy } from '@pockethost/common'
-  import { onDestroy } from 'svelte'
   import { instance } from './store'
 
-  const cm = createCleanupManager()
-
   // Run anytime the page params changes
   $: {
-    getSingleInstance($page.params.instanceId)
+    const { instanceId } = $page.params
+    assertTruthy(instanceId)
+    getSingleInstance(instanceId)
   }
-
-  onDestroy(() => cm.shutdown())
 </script>
 
 <AuthStateGuard>

packages/dashboard/src/util/getInstances.ts

@@ -1,7 +1,7 @@
 import { browser } from '$app/environment'
 import { client } from '$src/pocketbase'
 import { instance } from '$src/routes/app/instances/[instanceId]/store'
-import { globalInstancesStore } from '$util/stores'
+import { globalInstancesStore, isUserLoggedIn } from '$util/stores'
 import {
   LoggerService,
   assertExists,
@@ -13,16 +13,17 @@ import { onDestroy, onMount } from 'svelte'
 export const getInstances = async () => {
   const { error } = LoggerService()
 
-  onMount(() => {
+  const cm = createCleanupManager()
-    if (browser) {
+  onMount(async () => {
-      ;(async () => {
+    const unsub = isUserLoggedIn.subscribe(async (isLoggedIn) => {
+      if (!isLoggedIn) return
       const { getAllInstancesById } = client()
 
       const instances = await getAllInstancesById()
 
       globalInstancesStore.set(instances)
 
-        client()
+      const unsub = await client()
         .client.collection('instances')
         .subscribe<InstanceFields>('*', (data) => {
           globalInstancesStore.update((instances) => {
@@ -30,15 +31,14 @@ export const getInstances = async () => {
             return instances
           })
         })
-      })().catch(error)
+      cm.add(unsub)
-    }
+    })
+    cm.add(unsub)
   })
 
   // Stop listening to the db if this component unmounts
   onDestroy(() => {
-    if (browser) {
+    cm.shutdown().catch(console.error)
-      client().client.collection('instances').unsubscribe('*').catch(error)
-    }
   })
 }