fix: race condition get Assistant (#5161)

web-app/src/providers/DataProvider.tsx

@@ -28,9 +28,16 @@ export function DataProvider() {
       getProviders().then(setProviders)
     })
     getMCPConfig().then((data) => setServers(data.mcpServers ?? []))
-    getAssistants().then((data) =>
+    getAssistants()
-      setAssistants((data as unknown as Assistant[]) ?? [])
+      .then((data) => {
-    )
+        // Only update assistants if we have valid data
+        if (data && Array.isArray(data) && data.length > 0) {
+          setAssistants(data as unknown as Assistant[])
+        }
+      })
+      .catch((error) => {
+        console.warn('Failed to load assistants, keeping default:', error)
+      })
     migrateData()
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [])

web-app/src/services/assistants.ts

@@ -6,9 +6,16 @@ import { Assistant, AssistantExtension, ExtensionTypeEnum } from '@janhq/core'
  * @returns A promise that resolves to the assistants.
  */
 export const getAssistants = async () => {
-  return ExtensionManager.getInstance()
+  const extension = ExtensionManager.getInstance().get<AssistantExtension>(
-    .get<AssistantExtension>(ExtensionTypeEnum.Assistant)
+    ExtensionTypeEnum.Assistant
-    ?.getAssistants()
+  )
+
+  if (!extension) {
+    console.warn('AssistantExtension not found')
+    return null
+  }
+
+  return extension.getAssistants()
 }
 
 /**