From 96e9a6d38bbfac8b47cbdb6751943931ecc59830 Mon Sep 17 00:00:00 2001
From: Claude <claude@anthropic.com>
Date: Mon, 23 Mar 2026 06:33:00 -0700
Subject: [PATCH] Fix Ollama: remove duplicate stale configMap in AIChatPanel

AIChatPanel had its own hardcoded configMap with the old llama-server
URL (localhost:8080) and field names (local_model_path). Every chat
message reconfigured the provider with these wrong values, overriding
the correct settings applied at startup.

Fix: replace the duplicate with a call to the shared configureAIProvider().
Also strip trailing slashes from ollama_url before appending /v1 to
prevent double-slash URLs (http://localhost:11434//v1).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 src/lib/components/AIChatPanel.svelte | 14 ++------------
 src/lib/stores/settings.ts            |  2 +-
 2 files changed, 3 insertions(+), 13 deletions(-)

diff --git a/src/lib/components/AIChatPanel.svelte b/src/lib/components/AIChatPanel.svelte
index 8e4ac28..e5ba88a 100644
--- a/src/lib/components/AIChatPanel.svelte
+++ b/src/lib/components/AIChatPanel.svelte
@@ -1,7 +1,7 @@
 <script lang="ts">
   import { invoke } from '@tauri-apps/api/core';
   import { segments, speakers } from '$lib/stores/transcript';
-  import { settings } from '$lib/stores/settings';
+  import { settings, configureAIProvider } from '$lib/stores/settings';
 
   interface ChatMessage {
     role: 'user' | 'assistant';
@@ -45,17 +45,7 @@
       }));
 
       // Ensure the provider is configured with current credentials before chatting
-      const s = $settings;
-      const configMap: Record<string, Record<string, string>> = {
-        openai: { api_key: s.openai_api_key, model: s.openai_model },
-        anthropic: { api_key: s.anthropic_api_key, model: s.anthropic_model },
-        litellm: { api_key: s.litellm_api_key, api_base: s.litellm_api_base, model: s.litellm_model },
-        local: { model: s.local_model_path, base_url: 'http://localhost:8080' },
-      };
-      const config = configMap[s.ai_provider];
-      if (config) {
-        await invoke('ai_configure', { provider: s.ai_provider, config });
-      }
+      await configureAIProvider($settings);
 
       const result = await invoke<{ response: string }>('ai_chat', {
         messages: chatMessages,
diff --git a/src/lib/stores/settings.ts b/src/lib/stores/settings.ts
index 9b011a9..dda80df 100644
--- a/src/lib/stores/settings.ts
+++ b/src/lib/stores/settings.ts
@@ -57,7 +57,7 @@ export async function configureAIProvider(s: AppSettings): Promise<void> {
     openai: { api_key: s.openai_api_key, model: s.openai_model },
     anthropic: { api_key: s.anthropic_api_key, model: s.anthropic_model },
     litellm: { api_key: s.litellm_api_key, api_base: s.litellm_api_base, model: s.litellm_model },
-    local: { model: s.ollama_model, base_url: s.ollama_url + '/v1' },
+    local: { model: s.ollama_model, base_url: s.ollama_url.replace(/\/+$/, '') + '/v1' },
   };
   const config = configMap[s.ai_provider];
   if (config) {