From 522c11a7632944aa889252e9a044dea6219475d4 Mon Sep 17 00:00:00 2001 From: Grace <88872231+gr4ceG@users.noreply.github.com> Date: Wed, 17 Dec 2025 19:06:56 -0800 Subject: [PATCH] Revert "Omit args and params in tool function def and calls (#13516)" (#13518) This reverts commit 0fadeffaee76c50d84f21e81400c606fcc08c4f8. --- api/types.go | 4 ++-- server/routes_debug_test.go | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/api/types.go b/api/types.go index d24178d1..3ccf3ce2 100644 --- a/api/types.go +++ b/api/types.go @@ -341,7 +341,7 @@ type ToolFunctionParameters struct { Defs any `json:"$defs,omitempty"` Items any `json:"items,omitempty"` Required []string `json:"required,omitempty"` - Properties map[string]ToolProperty `json:"properties,omitempty"` + Properties map[string]ToolProperty `json:"properties"` } func (t *ToolFunctionParameters) String() string { @@ -352,7 +352,7 @@ func (t *ToolFunctionParameters) String() string { type ToolFunction struct { Name string `json:"name"` Description string `json:"description,omitempty"` - Parameters ToolFunctionParameters `json:"parameters,omitempty"` + Parameters ToolFunctionParameters `json:"parameters"` } func (t *ToolFunction) String() string { diff --git a/server/routes_debug_test.go b/server/routes_debug_test.go index 012125b5..6f9104c3 100644 --- a/server/routes_debug_test.go +++ b/server/routes_debug_test.go @@ -363,7 +363,7 @@ func TestChatDebugRenderOnly(t *testing.T) { DebugRenderOnly: true, }, expectDebug: true, - expectTemplate: "[{\"type\":\"function\",\"function\":{\"name\":\"get_weather\",\"description\":\"Get weather information\",\"parameters\":{\"type\":\"\"}}}]user: Get the weather\n", + expectTemplate: "[{\"type\":\"function\",\"function\":{\"name\":\"get_weather\",\"description\":\"Get weather information\",\"parameters\":{\"type\":\"\",\"properties\":null}}}]user: Get the weather\n", }, }