🐛 Fixes the model addition issue in ModelEngine. (#3026)

[Specification Detail]
1. Fixed the issue where the model list could not be retrieved when adding models in batches on the ME platform.
2. When adding a single model, check /open/router to modify model_factory.

Author: YehongPan

backend/services/model_management_service.py

@@ -55,6 +55,10 @@ async def create_model_for_tenant(user_id: str, tenant_id: str, model_data: Dict
             model_data["ssl_verify"] = False
         else:
             model_data["ssl_verify"] = True
+
+        # Set model_factory to modelengine when using open/router URL
+        if "open/router" in model_base_url:
+            model_data["model_factory"] = "modelengine"
         # Split model_name into repo and name
         model_repo, model_name = split_repo_name(
             model_data["model_name"]) if model_data.get("model_name") else ("", "")

frontend/app/[locale]/models/components/model/ModelAddDialog.tsx

@@ -281,8 +281,8 @@ export const ModelAddDialog = ({
   let getModelList;
   let getProviderSelectedModalList;
 
-// 2. 根据条件赋值
-  if (form.provider === "silicon") {
+  // Use silicon hook for silicon and modelengine providers (both use the same API pattern)
+  if (form.provider === "silicon" || form.provider === "modelengine") {
     ({ getModelList, getProviderSelectedModalList } = siliconHook);
   } else if (form.provider === "dashscope") {
     ({ getModelList, getProviderSelectedModalList } = dashscopeHook);

test/backend/services/test_model_management_service.py

@@ -359,7 +359,7 @@ async def test_create_model_for_tenant_success_llm():
 
 @pytest.mark.asyncio
 async def test_create_model_for_tenant_open_router_disables_ssl():
-    """When base_url contains 'open/router' ssl_verify should be set to False."""
+    """When base_url contains 'open/router' ssl_verify should be set to False and model_factory to 'modelengine'."""
     svc = import_svc()
 
     with mock.patch.object(svc, "get_model_by_display_name", return_value=None), \
@@ -381,6 +381,8 @@ async def test_create_model_for_tenant_open_router_disables_ssl():
         assert mock_create.call_count == 1
         create_args = mock_create.call_args[0][0]
         assert create_args["ssl_verify"] is False
+        # model_factory should be set to modelengine when open/router URL is used
+        assert create_args["model_factory"] == "modelengine"
 
 
 @pytest.mark.asyncio