From 8497c955f9a5ee782ca3b30bb04cba33decea615 Mon Sep 17 00:00:00 2001 From: miao <1468212639@qq.com> Date: Fri, 10 Apr 2026 13:31:09 +0800 Subject: [PATCH] fix(tools): make image_understand image_url optional and remove unused operation variable Change image_url from required to optional in both operation_tool.py and tool_service.py for image_understand operation, avoiding parameter validation conflict with uploaded_files priority logic. Remove unused operation variable from OpenClawTool.execute(). --- api/app/core/tools/builtin/openclaw_tool.py | 1 - api/app/core/tools/builtin/operation_tool.py | 4 ++-- api/app/services/tool_service.py | 4 ++-- 3 files changed, 4 insertions(+), 5 deletions(-) diff --git a/api/app/core/tools/builtin/openclaw_tool.py b/api/app/core/tools/builtin/openclaw_tool.py index 161769f1..80b993f0 100644 --- a/api/app/core/tools/builtin/openclaw_tool.py +++ b/api/app/core/tools/builtin/openclaw_tool.py @@ -122,7 +122,6 @@ class OpenClawTool(BuiltinTool): start_time = time.time() try: message = kwargs.get("message", "") - operation = kwargs.get("operation", "unknown") if not message: return ToolResult.error_result( error="message 参数不能为空", diff --git a/api/app/core/tools/builtin/operation_tool.py b/api/app/core/tools/builtin/operation_tool.py index a81ff951..d6692769 100644 --- a/api/app/core/tools/builtin/operation_tool.py +++ b/api/app/core/tools/builtin/operation_tool.py @@ -253,8 +253,8 @@ class OperationTool(BaseTool): ToolParameter( name="image_url", type=ParameterType.STRING, - description="必须提供,要分析的图片 URL 或 base64 data URI", - required=True + description="要分析的图片 URL 或 base64 data URI", + required=False ) ] else: diff --git a/api/app/services/tool_service.py b/api/app/services/tool_service.py index 20961119..1025a7e3 100644 --- a/api/app/services/tool_service.py +++ b/api/app/services/tool_service.py @@ -730,8 +730,8 @@ class ToolService: { "name": "image_url", "type": "string", - "description": "必须提供,要分析的图片 URL 或 base64 data URI", - "required": True + "description": "要分析的图片 URL 或 base64 data URI", + "required": False } ] else: