sync: 171 file da Baida98/AI@4df3de8e (2026-08-23 17:33 UTC)

models/ai_client.py

@@ -113,11 +113,26 @@ class AIClient:
         model-selection variables deliberately win so emergency model migrations
         do not require reading or mutating provider secrets in the database.
         """
-        database_model = str(row.get("default_model", ""))
+        database_model = str(row.get("default_model", "")).strip()
         row_base_url = str(row.get("base_url", "")).rstrip("/")
         for definition in _PROVIDER_DEFS:
             if row_base_url == definition["base_url"].rstrip("/"):
-                return os.getenv(definition["model_env"], database_model)
+                # Explicit runtime configuration always wins over persisted DB values.
+                configured_model = os.getenv(definition["model_env"], "").strip()
+                if configured_model:
+                    return configured_model
+                # Supabase may retain a model retired by the provider. Do not let a
+                # stale row override the tested repository default after deployment.
+                if (
+                    definition["name"] == "groq"
+                    and database_model in {
+                        "llama-3.3-70b-versatile",
+                        "llama-3.1-70b-versatile",
+                        "llama-3.1-8b-instant",
+                    }
+                ):
+                    return definition["default_model"]
+                return database_model or definition["default_model"]
         return database_model
 
     @staticmethod