fix: provider routing uses provider_type, Active badge, fmt

src-tauri/src/ai/provider.rs

@@ -15,7 +15,13 @@ pub trait Provider: Send + Sync {
 }
 
 pub fn create_provider(config: &ProviderConfig) -> Box<dyn Provider> {
-    match config.name.as_str() {
+    // Match on provider_type (the kind), falling back to name for legacy configs
+    let kind = if config.provider_type.is_empty() {
+        config.name.as_str()
+    } else {
+        config.provider_type.as_str()
+    };
+    match kind {
         "anthropic" => Box::new(crate::ai::anthropic::AnthropicProvider),
         "gemini" => Box::new(crate::ai::gemini::GeminiProvider),
         "mistral" => Box::new(crate::ai::mistral::MistralProvider),

src-tauri/src/ollama/installer.rs

@@ -11,9 +11,13 @@ pub async fn check_ollama() -> anyhow::Result<OllamaStatus> {
     let which_result = std::process::Command::new(which_cmd).arg("ollama").output();
 
     // Check common install paths explicitly — Tauri's process PATH may omit /usr/local/bin
-    let in_common_path = ["/usr/local/bin/ollama", "/opt/homebrew/bin/ollama", "/usr/bin/ollama"]
-        .iter()
-        .any(|p| std::path::Path::new(p).exists());
+    let in_common_path = [
+        "/usr/local/bin/ollama",
+        "/opt/homebrew/bin/ollama",
+        "/usr/bin/ollama",
+    ]
+    .iter()
+    .any(|p| std::path::Path::new(p).exists());
 
     let installed = which_result.map(|o| o.status.success()).unwrap_or(false) || in_common_path;
 

src-tauri/src/state.rs

@@ -5,6 +5,8 @@ use std::sync::{Arc, Mutex};
 #[derive(Debug, Clone, Serialize, Deserialize)]
 pub struct ProviderConfig {
     pub name: String,
+    #[serde(default)]
+    pub provider_type: String,
     pub api_url: String,
     pub api_key: String,
     pub model: String,

src/pages/Settings/AIProviders.tsx

@@ -136,7 +136,7 @@ export default function AIProviders() {
                   <span className="text-sm font-medium">{provider.name}</span>
                   <Badge variant="secondary">{provider.provider_type}</Badge>
                   {active_provider === provider.name && (
-                    <Badge className="bg-green-600 text-white">Active</Badge>
+                    <Badge variant="success">Active</Badge>
                   )}
                 </div>
                 <p className="text-xs text-muted-foreground">