mirror of
https://github.com/Wei-Shaw/sub2api.git
synced 2026-09-24 16:05:44 +08:00
fix(apicompat): 内置 tool_search 与同名工具撞名时显式拒绝请求
tool_search 服务端工具降级后的代理 function 不能改名(codex 的模型侧按 tool_search 这个名字调用),与客户端声明的同名 function/custom 工具无法 区分:去程产生重复声明,回程会把普通工具的调用劫持还原成 tool_search_call。 与 namespace 摊平撞名的处理一致,在请求转换阶段显式报错(网关 400); 重复声明 type=tool_search 去重后只产出一个代理,不拒绝。 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NWQyEgFKKbdve67G6qCoAU
This commit is contained in:
@@ -573,6 +573,7 @@ func responsesToolsToChatTools(tools []ResponsesTool) ([]ChatTool, error) {
|
||||
}
|
||||
}
|
||||
flatOwner := make(map[string]NamespacedToolName)
|
||||
toolSearchDeclared := false
|
||||
out := make([]ChatTool, 0, len(tools))
|
||||
for _, tool := range tools {
|
||||
switch tool.Type {
|
||||
@@ -598,6 +599,16 @@ func responsesToolsToChatTools(tools []ResponsesTool) ([]ChatTool, error) {
|
||||
},
|
||||
})
|
||||
case "tool_search":
|
||||
// 代理不能改名(codex 的模型侧按 tool_search 这个名字调用),与客户端
|
||||
// 声明的同名工具无法区分——回程会把普通工具的调用劫持成 tool_search_call,
|
||||
// 必须显式拒绝;重复声明 type=tool_search 去重即可。
|
||||
if topLevel[toolSearchProxyName] {
|
||||
return nil, fmt.Errorf("built-in tool_search conflicts with a declared tool named %q; this upstream cannot disambiguate them, rename the tool", toolSearchProxyName)
|
||||
}
|
||||
if toolSearchDeclared {
|
||||
continue
|
||||
}
|
||||
toolSearchDeclared = true
|
||||
out = append(out, toolSearchProxyChatTool())
|
||||
case "namespace":
|
||||
flattened, err := namespaceChildrenToChatTools(tool, topLevel, flatOwner)
|
||||
|
||||
@@ -536,6 +536,44 @@ func TestNamespaceToolNames_MapsFlattenedNames(t *testing.T) {
|
||||
assert.Nil(t, NamespaceToolNames(nil))
|
||||
}
|
||||
|
||||
// 内置 tool_search 降级后的代理 function 与客户端声明的同名工具无法区分:回程会把
|
||||
// 普通工具的调用劫持成 tool_search_call,必须显式拒绝(代理不能改名,codex 的模型
|
||||
// 侧按 tool_search 这个名字调用)。
|
||||
func TestResponsesToChatCompletionsRequest_RejectsToolSearchNameConflict(t *testing.T) {
|
||||
// 与顶层 function 工具同名。
|
||||
_, err := ResponsesToChatCompletionsRequest(&ResponsesRequest{
|
||||
Model: "glm-5.2",
|
||||
Input: json.RawMessage(`"hi"`),
|
||||
Tools: []ResponsesTool{
|
||||
{Type: "tool_search"},
|
||||
{Type: "function", Name: "tool_search"},
|
||||
},
|
||||
})
|
||||
require.Error(t, err, "与内置 tool_search 代理撞名的 function 工具必须拒绝")
|
||||
assert.Contains(t, err.Error(), "tool_search")
|
||||
|
||||
// 与顶层 custom 工具同名。
|
||||
_, err = ResponsesToChatCompletionsRequest(&ResponsesRequest{
|
||||
Model: "glm-5.2",
|
||||
Input: json.RawMessage(`"hi"`),
|
||||
Tools: []ResponsesTool{
|
||||
{Type: "custom", Name: "tool_search"},
|
||||
{Type: "tool_search"},
|
||||
},
|
||||
})
|
||||
require.Error(t, err, "与内置 tool_search 代理撞名的 custom 工具必须拒绝")
|
||||
|
||||
// 重复声明 type=tool_search 去重后只产出一个代理,不拒绝。
|
||||
out, err := ResponsesToChatCompletionsRequest(&ResponsesRequest{
|
||||
Model: "glm-5.2",
|
||||
Input: json.RawMessage(`"hi"`),
|
||||
Tools: []ResponsesTool{{Type: "tool_search"}, {Type: "tool_search"}},
|
||||
})
|
||||
require.NoError(t, err)
|
||||
require.Len(t, out.Tools, 1)
|
||||
assert.Equal(t, "tool_search", out.Tools[0].Function.Name)
|
||||
}
|
||||
|
||||
// 客户端请求在原生 Responses API 上合法(namespace 子工具按 namespace+name 路由),
|
||||
// 是摊平转换让名字产生歧义;歧义无法消除时必须显式拒绝整个请求(400),而不是
|
||||
// 静默降级——否则重复声明发给上游、回程还原到错误工具,问题只能靠抓包定位。
|
||||
|
||||
Reference in New Issue
Block a user