Make model picker search and header labels truthful

Author: 1jehuang

src/tui/app/tests/state_model_poke_03.rs

@@ -585,6 +585,34 @@ fn test_login_smoke_model_picker_renders_unstacked_provider_rows() {
     );
 }
 
+#[test]
+fn test_model_picker_filter_text_includes_provider_and_method() {
+    let entry = crate::tui::PickerEntry {
+        name: "glm-51-nvfp4".to_string(),
+        options: vec![crate::tui::PickerOption {
+            provider: "Comtegra GPU Cloud".to_string(),
+            api_method: "openai-compatible:comtegra".to_string(),
+            available: true,
+            detail: "https://llm.comtegra.cloud/v1".to_string(),
+            estimated_reference_cost_micros: None,
+        }],
+        action: crate::tui::PickerAction::Model,
+        selected_option: 0,
+        is_current: false,
+        is_default: false,
+        recommended: false,
+        recommendation_rank: usize::MAX,
+        old: false,
+        created_date: None,
+        effort: None,
+    };
+
+    let filter_text = crate::tui::PickerKind::Model.filter_text(&entry);
+    assert!(filter_text.contains("glm-51-nvfp4"));
+    assert!(filter_text.contains("Comtegra GPU Cloud"));
+    assert!(filter_text.contains("openai-compatible:comtegra"));
+}
+
 #[test]
 fn test_login_picker_preview_stays_open_and_updates_filter() {
     let mut app = create_test_app();

src/tui/mod.rs

@@ -704,7 +704,13 @@ impl PickerKind {
                     .unwrap_or("");
                 format!("{} {} {} {}", entry.name, status, window, detail)
             }
-            Self::Model => entry.name.clone(),
+            Self::Model => {
+                let route = entry.active_option();
+                let provider = route.map(|option| option.provider.as_str()).unwrap_or("");
+                let method = route.map(|option| option.api_method.as_str()).unwrap_or("");
+                let detail = route.map(|option| option.detail.as_str()).unwrap_or("");
+                format!("{} {} {} {}", entry.name, provider, method, detail)
+            }
         }
     }
 }

src/tui/ui_header.rs

@@ -231,15 +231,12 @@ fn header_provider_auth_tag(name: &str, auth: &AuthStatus) -> &'static str {
                 ""
             }
         }
-        "openai" => {
-            if auth.openai_has_oauth {
-                "oauth"
-            } else if auth.openai_has_api_key {
-                "api-key"
-            } else {
-                ""
-            }
-        }
+        "openai" => match (auth.openai_has_oauth, auth.openai_has_api_key) {
+            (true, true) => "oauth+key",
+            (true, false) => "oauth",
+            (false, true) => "api-key",
+            (false, false) => "",
+        },
         "copilot" => {
             if auth.copilot_has_api_token {
                 "oauth"
@@ -835,15 +832,15 @@ mod tests {
     }
 
     #[test]
-    fn header_provider_auth_tag_prefers_openai_oauth_over_api_key() {
+    fn header_provider_auth_tag_reports_openai_oauth_and_api_key() {
         let auth = AuthStatus {
             openai: AuthState::Available,
             openai_has_oauth: true,
             openai_has_api_key: true,
             ..AuthStatus::default()
         };
 
-        assert_eq!(header_provider_auth_tag("openai", &auth), "oauth");
+        assert_eq!(header_provider_auth_tag("openai", &auth), "oauth+key");
     }
 
     #[test]