[setup] check status once per page load

Author: kyleaupton

web/src/App.vue

@@ -1,21 +1,38 @@
 <script setup lang="ts">
 import { onMounted, ref } from 'vue'
-import { useRoute } from 'vue-router'
+import { useRouter, useRoute } from 'vue-router'
 import { useAuthStore } from '@/stores/auth'
 import AppSidebar from '@/components/AppSidebar.vue'
 import AppLayoutHeader from '@/components/AppLayoutHeader.vue'
 import DialogContainer from '@/components/DialogContainer.vue'
 import { SidebarInset, SidebarProvider } from '@/components/ui/sidebar'
 import { TooltipProvider } from '@/components/ui/tooltip'
+import { client } from '@/client/client.gen'
 
 import 'vue-sonner/style.css'
 import { Toaster } from '@/components/ui/sonner'
 
 const authStore = useAuthStore()
+const router = useRouter()
 const route = useRoute()
 const isCheckingAuth = ref(true)
+const isCheckingSetup = ref(true)
+const needsSetup = ref(false)
 
 onMounted(async () => {
+  // Check setup status once on page load
+  try {
+    const response = await client.get({ url: '/v1/setup/status' })
+    needsSetup.value = !(response as any).data.initialized
+    if (needsSetup.value && route.path !== '/setup') {
+      router.push('/setup')
+    }
+  } catch {
+    // If setup check fails, assume initialized
+    needsSetup.value = false
+  }
+  isCheckingSetup.value = false
+
   // Rehydrate auth token from localStorage
   if (!authStore.token) {
     await authStore.rehydrate()
@@ -28,7 +45,7 @@ onMounted(async () => {
   <TooltipProvider>
     <Toaster position="top-center" />
     <DialogContainer />
-    <div v-if="isCheckingAuth" class="flex min-h-svh items-center justify-center">
+    <div v-if="isCheckingAuth || isCheckingSetup" class="flex min-h-svh items-center justify-center">
       <div class="text-muted-foreground">Loading...</div>
     </div>
     <router-view v-else-if="route.meta.public" />

web/src/router/index.ts

@@ -1,6 +1,5 @@
 import { createRouter, createWebHistory } from 'vue-router'
 import { useAuthStore } from '@/stores/auth'
-import { client } from '@/client/client.gen'
 
 const router = createRouter({
   history: createWebHistory(import.meta.env.BASE_URL),
@@ -91,19 +90,6 @@ const router = createRouter({
   ],
 })
 
-// Check setup status
-async function checkSetupStatus(): Promise<boolean> {
-  try {
-    const response = await client.get<{ initialized: boolean }>({
-      url: '/v1/setup/status',
-    })
-    return (response as any).data.initialized
-  } catch {
-    // If setup check fails, assume initialized (safer default)
-    return true
-  }
-}
-
 router.beforeEach(async (to) => {
   const auth = useAuthStore()
 
@@ -112,20 +98,7 @@ router.beforeEach(async (to) => {
     await auth.rehydrate()
   }
 
-  // Check if setup is complete
-  const isInitialized = await checkSetupStatus()
-
-  // If not initialized, redirect everything to setup (except setup page itself)
-  if (!isInitialized && to.path !== '/setup') {
-    return { path: '/setup' }
-  }
-
-  // If initialized, block setup page
-  if (isInitialized && to.path === '/setup') {
-    return { path: '/login' }
-  }
-
-  // Public routes allowed
+  // Public routes (login, setup, auth callback)
   if (to.meta.public) {
     return true
   }