updated mllm vendors

Author: digitallysavvy

src/agora_agent/agentkit/vendors/mllm.py

@@ -179,12 +179,11 @@ def to_config(self) -> Dict[str, Any]:
         if self.options.http_options is not None:
             params["http_options"] = self.options.http_options
 
-        params["project_id"] = self.options.project_id
-        params["location"] = self.options.location
-        params["adc_credentials_string"] = self.options.adc_credentials_string
-
         config: Dict[str, Any] = {
             "vendor": "vertexai",
+            "project_id": self.options.project_id,
+            "location": self.options.location,
+            "adc_credentials_string": self.options.adc_credentials_string,
             "params": params,
         }
 

tests/custom/test_agentkit_vendors.py

@@ -79,10 +79,12 @@ def test_vertex_ai_explicit_fields_override_additional_params():
     ).to_config()
 
     assert config["vendor"] == "vertexai"
+    # routing fields are top-level, not inside params
+    assert config["project_id"] == "explicit-project"
+    assert config["location"] == "explicit-region"
+    assert config["adc_credentials_string"] == "{}"
+    # model and extra_key live inside params
     assert config["params"]["model"] == "explicit-model"
-    assert config["params"]["project_id"] == "explicit-project"
-    assert config["params"]["location"] == "explicit-region"
-    assert config["params"]["adc_credentials_string"] == "{}"
     assert config["params"]["extra_key"] == "kept"