fix: desktop model list uses sidecar proxy (no CORS) and correct provider names

- Replace fetchProviderModels() (browser CORS block) with modelsApi.list()
(server-side sidecar proxy) to fetch NVIDIA/OpenRouter models
- Derive provider names from ~/.claude.json authProvider field, avoiding
sidecar's stale/corrupted names like "TUI: NVIDIA NIM"
- Fall back to buildDefaultCliModels() when sidecar returns empty
- Remove clearProviderModelCache() since we no longer use providerModels.ts
in settings/provider stores
- Fix ApiFormat type comparison ('openai' -> 'openai_chat'/'openai_responses')

Co-Authored-By: Claude Big Pickle <noreply@anthropic.com>

desktop/src/api/providerModels.ts

@@ -156,7 +156,7 @@ let nvCacheTime = 0
 
 async function fetchNvidiaModels(apiKey?: string | null, baseUrl?: string): Promise<ModelInfo[]> {
   if (nvCache && Date.now() - nvCacheTime < OR_CACHE_TTL) {
-    return nvCache
+    return nvCache.length > 0 ? nvCache : []
   }
 
   if (!apiKey) {

desktop/src/stores/providerStore.ts

@@ -6,12 +6,7 @@ import { useChatStore } from './chatStore'
 import { useSessionRuntimeStore } from './sessionRuntimeStore'
 import { useSettingsStore } from './settingsStore'
 import { OFFICIAL_DEFAULT_MODEL_ID } from '../constants/modelCatalog'
-import {
-  OPENAI_OFFICIAL_DEFAULT_MODEL_ID,
-  OPENAI_OFFICIAL_PROVIDER_ID,
-} from '../constants/openaiOfficialProvider'
 import { saveTuiConfigPatch, clearConfigCache } from '../api/config'
-import { clearProviderModelCache } from '../api/providerModels'
 import type {
   SavedProvider,
   CreateProviderInput,
@@ -58,8 +53,9 @@ function mapSidecarToTuiProvider(provider: SavedProvider): TuiProviderMapping {
   const name = (provider.name || '').toLowerCase()
 
   // Detect local provider (Ollama, LM Studio, vLLM, etc.)
+  const isOpenAiFormat = apiFormat === 'openai_chat' || apiFormat === 'openai_responses'
   if (
-    apiFormat === 'openai' &&
+    isOpenAiFormat &&
     (baseUrl.includes('localhost') || baseUrl.includes('127.0.0.1') || baseUrl.includes('0.0.0.0') ||
      name.includes('local') || name.includes('ollama'))
   ) {
@@ -74,7 +70,8 @@ function mapSidecarToTuiProvider(provider: SavedProvider): TuiProviderMapping {
   switch (apiFormat) {
     case 'anthropic':
       return { authProvider: 'anthropic', apiKeyField: 'anthropicApiKey', apiKey: provider.apiKey }
-    case 'openai': {
+    case 'openai_chat':
+    case 'openai_responses': {
       if (baseUrl.includes('nvidia') || name.includes('nvidia')) {
         return { authProvider: 'nvidia', apiKeyField: 'nvidiaApiKey', apiKey: provider.apiKey }
       }
@@ -228,11 +225,8 @@ export const useProviderStore = create<ProviderStore>((set, get) => ({
     patch.model = provider.models.main
 
     await saveTuiConfigPatch(patch)
-    clearProviderModelCache()
     clearConfigCache()
-
-    await settings.setModel(provider.models.main)
-    await settings.fetchAll()
+    await settings.syncFromConfig()
   },
 
   activateOfficial: async () => {
@@ -240,7 +234,6 @@ export const useProviderStore = create<ProviderStore>((set, get) => ({
     await get().fetchProviders()
     // 同步回 Anthropic first-party
     await saveTuiConfigPatch({ authProvider: 'anthropic' })
-    clearProviderModelCache()
     clearConfigCache()
     const settings = useSettingsStore.getState()
     await settings.setModel(OFFICIAL_DEFAULT_MODEL_ID)

desktop/src/stores/settingsStore.ts

@@ -1,8 +1,8 @@
 import { create } from 'zustand'
 import { ApiError } from '../api/client'
 import { settingsApi } from '../api/settings'
+import { modelsApi } from '../api/models'
 import { h5AccessApi } from '../api/h5Access'
-import { fetchProviderModels, clearProviderModelCache } from '../api/providerModels'
 import { getTuiConfig, saveTuiConfigPatch, clearConfigCache } from '../api/config'
 import {
   isThemeMode,
@@ -40,6 +40,30 @@ export const UI_ZOOM_STEP = APP_ZOOM_CONTROL_STEP
 export const UI_ZOOM_DEFAULT = DEFAULT_APP_ZOOM
 let desktopNotificationsSaveQueue: Promise<void> = Promise.resolve()
 
+function buildDefaultCliModels(providerKey: string): ModelInfo[] {
+  switch (providerKey) {
+    case 'nvidia':
+      return [
+        { id: 'nvidia/llama-3.1-nemotron-70b-instruct', name: 'Nemotron 70B', description: 'Default NVIDIA model', context: '' },
+      ]
+    case 'openai':
+      return [
+        { id: 'gpt-5.4-codex', name: 'GPT-5.4 Codex', description: 'Default OpenAI model', context: '' },
+        { id: 'gpt-5.4-mini', name: 'GPT-5.4 Mini', description: 'Fast OpenAI option', context: '' },
+      ]
+    case 'openrouter':
+      return [
+        { id: 'openrouter/default', name: 'OpenRouter Default', description: 'Set via OpenRouter dashboard', context: '' },
+      ]
+    case 'local':
+      return [
+        { id: 'local/default', name: 'Local Default', description: 'Check your local model server', context: '' },
+      ]
+    default:
+      return []
+  }
+}
+
 function getStoredLocale(): Locale {
   try {
     const stored = localStorage.getItem(LOCALE_STORAGE_KEY)
@@ -173,47 +197,46 @@ export const useSettingsStore = create<SettingsStore>((set, get) => ({
     set({ isLoading: true, error: null })
     try {
       const previousH5Access = get().h5Access
-      const [directModels, { mode }, config, userSettings, h5AccessResult] = await Promise.all([
-        // Fetch models directly from provider APIs (TUI-style)
-        fetchProviderModels().catch(() => ({ models: [] as ModelInfo[], provider: null as { id: string; name: string } | null })),
+      const [sidecarResult, { mode }, config, userSettings, h5AccessResult] = await Promise.all([
+        // Fetch models from sidecar proxy (server-side, bypasses CORS)
+        modelsApi.list().catch(() => ({ models: [] as ModelInfo[], provider: null as { id: string; name: string } | null })),
         settingsApi.getPermissionMode(),
-        // Read model + effort from ~/.claude.json (TUI-style), bypassing sidecar
+        // Read model + effort from ~/.claude.json (TUI-style)
         getTuiConfig(),
         settingsApi.getUser(),
         loadH5AccessSettings(previousH5Access),
       ])
 
-      // Use direct models if available (TUI-style), otherwise fall back to sidecar
+      // Derive provider info from ~/.claude.json authProvider (correct names).
+      // Sidecar's provider name may be stale/corrupted (e.g. "TUI: NVIDIA NIM").
+      const CLI_NAMES: Record<string, string> = {
+        nvidia: 'NVIDIA',
+        openrouter: 'OpenRouter',
+        opencode: 'OpenCode Zen',
+        openai: 'OpenAI',
+        local: 'Local',
+        anthropic: 'Anthropic',
+      }
+      const authProvider = config.authProvider as string | undefined
       let availableModels: ModelInfo[]
       let activeProviderId: string | null
       let activeProviderName: string | null
 
-      if (directModels.models.length > 0) {
-        availableModels = directModels.models
-        activeProviderId = directModels.provider?.id ?? null
-        activeProviderName = directModels.provider?.name ?? null
-      } else {
-        // No direct models — always derive from ~/.claude.json authProvider.
-        // Never use sidecar's provider name: it may be stale and corrupted
-        // (e.g. "TUI: NVIDIA NIM" instead of "NVIDIA").
-        const authProvider = config.authProvider as string | undefined
-        if (authProvider) {
-          const CLI_PROVIDER_NAMES: Record<string, string> = {
-            nvidia: 'NVIDIA',
-            openrouter: 'OpenRouter',
-            opencode: 'OpenCode Zen',
-            openai: 'OpenAI',
-            local: 'Local',
-            anthropic: 'Anthropic',
-          }
-          availableModels = []
-          activeProviderId = `cli-${authProvider}`
-          activeProviderName = CLI_PROVIDER_NAMES[authProvider] ?? authProvider
+      if (authProvider) {
+        activeProviderId = `cli-${authProvider}`
+        activeProviderName = CLI_NAMES[authProvider] ?? authProvider
+        // Use sidecar models if available (from proxy), else build defaults
+        if (sidecarResult.models.length > 0) {
+          availableModels = sidecarResult.models
         } else {
-          availableModels = []
-          activeProviderId = null
-          activeProviderName = null
+          availableModels = buildDefaultCliModels(authProvider)
         }
+      } else {
+        activeProviderId = sidecarResult.provider?.id ?? null
+        activeProviderName = sidecarResult.provider?.name ?? null
+        availableModels = sidecarResult.models.length > 0
+          ? sidecarResult.models
+          : []
       }
 
       // Derive currentModel from config's model field, matched against availableModels
@@ -301,10 +324,9 @@ export const useSettingsStore = create<SettingsStore>((set, get) => ({
   syncFromConfig: async () => {
     // Re-read ~/.claude.json and re-fetch the model list for the new provider.
     // This keeps settingsStore in sync when cliAuthStore writes to the config file.
-    clearProviderModelCache()
     clearConfigCache()
-    const config = await getTuiConfig().catch(() => ({}))
-    const authProvider = config?.authProvider as string | undefined
+    const config = await getTuiConfig().catch(() => ({ authProvider: null }))
+    const authProvider = (config?.authProvider ?? null) as string | null
     if (!authProvider) {
       set({ activeProviderId: null, activeProviderName: null, availableModels: [], currentModel: null })
       return
@@ -320,13 +342,20 @@ export const useSettingsStore = create<SettingsStore>((set, get) => ({
     const cliProviderId = `cli-${authProvider}`
     const cliProviderName = CLI_NAMES[authProvider] ?? authProvider
 
-    // Fetch fresh models for the new provider
-    const { models } = await fetchProviderModels().catch(() => ({ models: [] as ModelInfo[], provider: null }))
+    // Fetch models from sidecar proxy (server-side, bypasses CORS)
+    const sidecarResult = await modelsApi.list().catch(() => ({
+      models: [] as ModelInfo[],
+      provider: null as { id: string; name: string } | null,
+    }))
+
+    // Fallback chain: sidecar models → default CLI models
+    const defaults = buildDefaultCliModels(authProvider)
+    const availableModels = sidecarResult.models.length > 0 ? sidecarResult.models : defaults
     set({
       activeProviderId: cliProviderId,
       activeProviderName: cliProviderName,
-      availableModels: models,
-      currentModel: models.length > 0 ? models[0] : null,
+      availableModels,
+      currentModel: availableModels[0] ?? null,
     })
   },