Update model_registry.rs

Author: BunsDev

src-rust/crates/api/src/model_registry.rs

@@ -719,11 +719,11 @@ impl ModelRegistry {
             models = self.list_by_provider(provider_id);
         }
         if models.is_empty() {
-            // Codex models are not on models.dev (OAuth-only), so the bundled
-            // snapshot has no entries for them. Fall back to the canonical
-            // constant in claurst_core::codex_oauth so `/model` and the
-            // effective-model resolver still pick a real id for Codex users.
-            return codex_models_fallback(provider_id, /* small = */ false);
+            // Codex and GitHub Copilot models may be absent from models.dev
+            // (OAuth/subscription-backed providers). Fall back to the curated
+            // GPT agent model list so `/model` and the effective-model resolver
+            // still pick real ids offline.
+            return gpt_agent_models_fallback(provider_id, /* small = */ false);
         }
 
         let priority_patterns = flagship_patterns_for(provider_id);
@@ -769,8 +769,8 @@ impl ModelRegistry {
             models = self.list_by_provider(provider_id);
         }
         if models.is_empty() {
-            // See best_model_for_provider for the Codex rationale.
-            return codex_models_fallback(provider_id, /* small = */ true);
+            // See best_model_for_provider for the GPT-agent fallback rationale.
+            return gpt_agent_models_fallback(provider_id, /* small = */ true);
         }
 
         let small_priority = small_patterns_for(provider_id);
@@ -963,7 +963,7 @@ impl Default for ModelRegistry {
 /// Earlier entries score higher.
 fn flagship_patterns_for(provider_id: &str) -> &'static [&'static str] {
     match provider_id {
-        "anthropic" | "amazon-bedrock" | "github-copilot" | "azure" | "google-vertex" => &[
+        "anthropic" | "amazon-bedrock" | "azure" | "google-vertex" => &[
             "claude-opus-4",
             "claude-sonnet-4",
             "claude-3-5-sonnet",
@@ -1006,7 +1006,7 @@ fn flagship_patterns_for(provider_id: &str) -> &'static [&'static str] {
         ],
         "zai" => &["glm-5.1", "glm-5", "glm-4.7"],
         "minimax" => &["minimax-m2"],
-        "codex" | "openai-codex" => &[
+        "codex" | "openai-codex" | "github-copilot" => &[
             "gpt-5.6-sol",
             "gpt-5.6-terra",
             "gpt-5.6-luna",
@@ -1037,13 +1037,13 @@ fn flagship_patterns_for(provider_id: &str) -> &'static [&'static str] {
 }
 
 /// Substring patterns marking a model as the lightweight/cheap default.
-/// Codex is OAuth-only and absent from the models.dev bundled snapshot, so
-/// `list_by_provider("codex")` always returns empty. This fallback returns
-/// the canonical default / small ids from
+/// Codex and GitHub Copilot can be absent from the models.dev bundled snapshot,
+/// so `list_by_provider(...)` may return empty. This fallback returns the
+/// canonical GPT agent default / small ids from
 /// `claurst_core::codex_oauth::CODEX_MODELS` so `/model` and `/fast` keep
-/// working for Codex users.
-fn codex_models_fallback(provider_id: &str, small: bool) -> Option<String> {
-    if !matches!(provider_id, "codex" | "openai-codex") {
+/// working for subscription-backed GPT providers.
+fn gpt_agent_models_fallback(provider_id: &str, small: bool) -> Option<String> {
+    if !matches!(provider_id, "codex" | "openai-codex" | "github-copilot") {
         return None;
     }
     if small {
@@ -1062,7 +1062,9 @@ fn codex_models_fallback(provider_id: &str, small: bool) -> Option<String> {
 fn small_patterns_for(provider_id: &str) -> &'static [&'static str] {
     match provider_id {
         "anthropic" => &["claude-haiku-4", "claude-haiku-3-5", "claude-haiku"],
-        "codex" | "openai" | "openai-codex" => &["gpt-5.6-luna", "gpt-5.4-mini", "gpt-5-mini"],
+        "codex" | "openai" | "openai-codex" | "github-copilot" => {
+            &["gpt-5.6-luna", "gpt-5.4-mini", "gpt-5-mini"]
+        }
         _ => &["mini", "haiku", "flash", "lite", "small", "nano"],
     }
 }
@@ -1221,6 +1223,14 @@ mod tests {
         // The alias `openai-codex` must take the same path.
         assert!(reg.best_model_for_provider("openai-codex").is_some());
         assert!(reg.best_small_model_for_provider("openai-codex").is_some());
+        let copilot_best = reg
+            .best_model_for_provider("github-copilot")
+            .expect("github-copilot best model must fall back to GPT agent default");
+        assert_eq!(copilot_best, claurst_core::codex_oauth::DEFAULT_CODEX_MODEL);
+        let copilot_small = reg
+            .best_small_model_for_provider("github-copilot")
+            .expect("github-copilot small model must fall back to a GPT agent small id");
+        assert_eq!(copilot_small, "gpt-5.6-luna");
         // Unrelated providers continue to return None when absent.
         assert!(reg
             .best_model_for_provider("definitely-not-a-provider")