fix: llama.cpp backend shows blank list sometime

core/src/browser/extension.ts

@@ -90,18 +90,18 @@
    * @property {Array} platform
    */
   compatibility(): Compatibility | undefined {
     return undefined
   }
 
   /**
    * Registers models - it persists in-memory shared ModelManager instance's data map.
    * @param models
    */
   async registerModels(models: Model[]): Promise<void> {
     for (const model of models) {
       ModelManager.instance().register(model)
     }
   }
 
   /**
    * Register settings for the extension.
@@ -110,9 +110,9 @@
    */
   async registerSettings(settings: SettingComponentProps[]): Promise<void> {
     if (!this.name) {
       console.error('Extension name is not defined')
       return
     }
 
     settings.forEach((setting) => {
       setting.extensionName = this.name
@@ -121,19 +121,23 @@
       const oldSettingsJson = localStorage.getItem(this.name)
       // Persists new settings
       if (oldSettingsJson) {
         const oldSettings = JSON.parse(oldSettingsJson)
         settings.forEach((setting) => {
           // Keep setting value
           if (setting.controllerProps && Array.isArray(oldSettings))
             setting.controllerProps.value = oldSettings.find(
               (e: any) => e.key === setting.key
             )?.controllerProps?.value
+          if ('options' in setting.controllerProps)
+            setting.controllerProps.options = setting.controllerProps.options?.length
+              ? setting.controllerProps.options
+              : oldSettings.find((e: any) => e.key === setting.key)?.controllerProps?.options
         })
       }
       localStorage.setItem(this.name, JSON.stringify(settings))
     } catch (err) {
       console.error(err)
     }
   }
 
   /**
@@ -167,18 +171,18 @@
    * @returns
    */
   async getSettings(): Promise<SettingComponentProps[]> {
     if (!this.name) return []
 
     try {
       const settingsString = localStorage.getItem(this.name)
       if (!settingsString) return []
       const settings: SettingComponentProps[] = JSON.parse(settingsString)
       return settings
     } catch (err) {
       console.warn(err)
       return []
     }
   }
 
   /**
    * Update the settings for the extension.

web-app/src/hooks/useModelProvider.ts

@@ -28,24 +28,28 @@
       selectedModel: null,
       deletedModels: [],
       getModelBy: (modelId: string) => {
         const provider = get().providers.find(
           (provider) => provider.provider === get().selectedProvider
         )
         if (!provider) return undefined
         return provider.models.find((model) => model.id === modelId)
       },
       setProviders: (providers) =>
         set((state) => {
-          const existingProviders = state.providers.map((provider) => {
-            return {
-              ...provider,
-              models: provider.models.filter(
-                (e) =>
-                  ('id' in e || 'model' in e) &&
-                  typeof (e.id ?? e.model) === 'string'
-              ),
-            }
-          })
+          const existingProviders = state.providers
+            // Filter out legacy llama.cpp provider for migration
+            // Can remove after a couple of releases
+            .filter((e) => e.provider !== 'llama.cpp')
+            .map((provider) => {
+              return {
+                ...provider,
+                models: provider.models.filter(
+                  (e) =>
+                    ('id' in e || 'model' in e) &&
+                    typeof (e.id ?? e.model) === 'string'
+                ),
+              }
+            })
           // Ensure deletedModels is always an array
           const currentDeletedModels = Array.isArray(state.deletedModels)
             ? state.deletedModels