🐛 fix Multimodal duplicated added

Author: Phinease

backend/services/model_management_service.py

@@ -89,11 +89,11 @@ async def create_model_for_tenant(user_id: str, tenant_id: str, model_data: Dict
             create_model_record(embedding_data, user_id, tenant_id)
             logging.debug(
                 f"Embedding model {embedding_data['display_name']} created successfully")
-
-        # Non-multimodal
-        create_model_record(model_data, user_id, tenant_id)
-        logging.debug(
-            f"Model {model_data['display_name']} created successfully")
+        else:
+            # Non-multimodal
+            create_model_record(model_data, user_id, tenant_id)
+            logging.debug(
+                f"Model {model_data['display_name']} created successfully")
     except Exception as e:
         logging.error(f"Failed to create model: {str(e)}")
         raise Exception(f"Failed to create model: {str(e)}")

test/backend/services/test_model_management_service.py

@@ -329,7 +329,7 @@ async def test_create_model_for_tenant_conflict_raises():
 
 
 @pytest.mark.asyncio
-async def test_create_model_for_tenant_multi_embedding_creates_three_records():
+async def test_create_model_for_tenant_multi_embedding_creates_two_records():
     svc = import_svc()
 
     with mock.patch.object(svc, "get_model_by_display_name", return_value=None), \
@@ -346,8 +346,8 @@ async def test_create_model_for_tenant_multi_embedding_creates_three_records():
         }
 
         await svc.create_model_for_tenant(user_id, tenant_id, model_data)
-        # Per current implementation, it creates multi_embedding, embedding variant, and then one more general create
-        assert mock_create.call_count == 3
+        # Should create two records: multi_embedding and its embedding variant
+        assert mock_create.call_count == 2
 
 
 @pytest.mark.asyncio