scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-10-10 03:28:53 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 85

fix(responses-bridge): drop Codex tool_search/local_shell tools for chat-only providers

Browse source
This commit is contained in:
Devin AI 2026-07-17 22:13:39 +00:00
parent c5b4456401
commit 6c7c5770ca
2 changed files with 59 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
litellm/responses/litellm_completion_transformation/transformation.py
View file

@ -1307,7 +1307,14 @@ class LiteLLMCompletionResponsesConfig:
chat_completion_tools.append(converted)
else:
_tool_type = tool.get("type")
if _tool_type in ("computer_use", "image_generation", "namespace", "shell"):
if _tool_type in (
"computer_use",
"image_generation",
"local_shell",
"namespace",
"shell",
"tool_search",
):
# Drop unsupported Responses-API-only tool types that have no
# Chat Completions equivalent. Passing them through verbatim
# causes providers to reject the request with "'function' is a

51
tests/test_litellm/responses/litellm_completion_transformation/test_litellm_completion_responses.py
View file

@ -1095,6 +1095,57 @@ class TestToolTransformation:
assert len(result_tools) == 0
assert web_search_options is None
def test_transform_codex_namespace_and_tool_search_tools_dropped(self):
"""Codex CLI (Responses wire) sends `namespace` and `tool_search` tools that
have no Chat Completions equivalent. Forwarding them verbatim made chat-only
providers such as Nvidia NIM reject the whole request with
`unknown variant 'namespace', expected 'function'` (issue #33779). The bridge
must drop them while keeping real function tools."""
tools = [
{
"type": "function",
"name": "shell",
"description": "run a shell command",
"parameters": {"type": "object", "properties": {}},
},
{
"type": "namespace",
"name": "mcp__server",
"description": "grouped tools",
"tools": [
{
"type": "function",
"name": "do_thing",
"description": "d",
"strict": False,
"parameters": {"type": "object", "properties": {}},
}
],
},
{
"type": "tool_search",
"execution": "client",
"description": "search available tools",
"parameters": {"type": "object", "properties": {}},
},
{"type": "local_shell"},
]
(
result_tools,
web_search_options,
) = LiteLLMCompletionResponsesConfig.transform_responses_api_tools_to_chat_completion_tools(
tools=tools
)
assert web_search_options is None
assert len(result_tools) == 1
assert result_tools[0]["type"] == "function"
assert result_tools[0]["function"]["name"] == "shell"
assert all(t.get("type") != "namespace" for t in result_tools)
assert all(t.get("type") != "tool_search" for t in result_tools)
assert all(t.get("type") != "local_shell" for t in result_tools)
def test_transform_custom_tools_to_function_tools(self):
"""Test that custom (freeform/grammar) tools are converted to function tools"""
custom_tool = {