fix: optional mcp tools

Author: NicoleMGomes

src/sap_cloud_sdk/agentgateway/converters.py

@@ -74,7 +74,10 @@ async def run(**kwargs) -> str:
 
     # Build args schema from input_schema
     properties = mcp_tool.input_schema.get("properties", {})
-    fields: dict[str, Any] = {k: (str, ...) for k in properties}
+    required = set(mcp_tool.input_schema.get("required", []))
+    fields: dict[str, Any] = {
+        k: (str, ...) if k in required else (str | None, None) for k in properties
+    }
     args_schema = create_model(f"{mcp_tool.name}_args", **fields) if fields else None
 
     return StructuredTool.from_function(

tests/agentgateway/unit/test_converters.py

@@ -78,6 +78,31 @@ def test_handles_empty_input_schema(self):
         assert result.name == "simple_tool"
         assert result.args_schema is not None
 
+    def test_optional_fields_not_required_in_args_schema(self):
+        """Fields absent from 'required' must be optional in the generated Pydantic model."""
+        tool = MCPTool(
+            name="get_supplier_bid",
+            server_name="ariba",
+            description="Gets all supplier bids for the specified event",
+            input_schema={
+                "type": "object",
+                "required": ["eventid"],
+                "properties": {
+                    "eventid": {"description": "Unique identifier of the event"},
+                    "showdeclinedreason": {"description": "Show supplier decline reason"},
+                    "datafetchmode": {"description": "Level of detail for the response"},
+                },
+            },
+            url="https://example.com/mcp",
+        )
+
+        result = mcp_tool_to_langchain(tool, AsyncMock(return_value="result"), lambda: "token")
+
+        fields = result.args_schema.model_fields
+        assert fields["eventid"].is_required(), "eventid should be required"
+        assert not fields["showdeclinedreason"].is_required(), "showdeclinedreason should be optional"
+        assert not fields["datafetchmode"].is_required(), "datafetchmode should be optional"
+
     def test_handles_input_schema_without_properties(self):
         """Handle MCPTool with input schema but no properties."""
         tool = MCPTool(