fix(tool): handle empty additionalProperties in schema

Updated ToolSchema to correctly handle cases where additionalProperties is an empty schema, ensuring that the original semantics are preserved. Added unit tests to verify this behavior.

修复(tool)：处理 schema 中的空 additionalProperties
更新 ToolSchema，以正确处理 additionalProperties 为空 schema 的情况，确保保留原语义。添加单元测试以验证此行为。

Change-Id: Idf4c5fd6a530da24a9a691765c44c61be7831713
Signed-off-by: OhYee <oyohyee@oyohyee.com>

Author: OhYee

agentrun/toolset/model.py

@@ -181,9 +181,13 @@ def from_any_openapi_schema(cls, schema: Any) -> "ToolSchema":
 
         additional_properties_raw = pg(schema, "additionalProperties")
         additional_properties = (
-            cls.from_any_openapi_schema(additional_properties_raw)
-            if isinstance(additional_properties_raw, dict)
-            else additional_properties_raw
+            cls()
+            if additional_properties_raw == {}
+            else (
+                cls.from_any_openapi_schema(additional_properties_raw)
+                if isinstance(additional_properties_raw, dict)
+                else additional_properties_raw
+            )
         )
 
         return cls(

tests/unittests/tool/test_model.py

@@ -377,6 +377,30 @@ def test_from_any_openapi_schema_additional_properties_schema(self):
             == "integer"
         )
 
+    def test_from_any_openapi_schema_empty_additional_properties_schema(self):
+        """测试 additionalProperties 为空 schema 时保留原语义"""
+        openapi_schema = {
+            "type": "object",
+            "properties": {
+                "metadata": {
+                    "type": "object",
+                    "additionalProperties": {},
+                }
+            },
+        }
+
+        schema = ToolSchema.from_any_openapi_schema(openapi_schema)
+
+        assert schema.properties is not None
+        metadata_schema = schema.properties["metadata"]
+        assert metadata_schema.additional_properties is not None
+        assert metadata_schema.additional_properties.type is None
+
+        json_schema = schema.to_json_schema()
+        assert (
+            json_schema["properties"]["metadata"]["additionalProperties"] == {}
+        )
+
     def test_to_json_schema_simple(self):
         """测试转换为 JSON Schema - 简单情况"""
         schema = ToolSchema(

tests/unittests/toolset/test_model.py

@@ -487,6 +487,30 @@ def test_from_any_openapi_schema_with_schema_additional_properties(self):
             == "integer"
         )
 
+    def test_from_any_openapi_schema_with_empty_additional_properties(self):
+        """测试 additionalProperties 为空 schema 时保留原语义"""
+        openapi_schema = {
+            "type": "object",
+            "properties": {
+                "metadata": {
+                    "type": "object",
+                    "additionalProperties": {},
+                }
+            },
+        }
+
+        schema = ToolSchema.from_any_openapi_schema(openapi_schema)
+
+        assert schema.properties is not None
+        metadata_schema = schema.properties["metadata"]
+        assert metadata_schema.additional_properties is not None
+        assert metadata_schema.additional_properties.type is None
+
+        json_schema = schema.to_json_schema()
+        assert (
+            json_schema["properties"]["metadata"]["additionalProperties"] == {}
+        )
+
     def test_from_any_openapi_schema_with_items(self):
         """测试从带 items 的数组 schema 创建"""
         openapi_schema = {