ilia/nanobot
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

fix: codex tool calling failed unexpectedly

Browse Source
This commit is contained in:
pinhua33 2026-02-06 11:52:03 +08:00
parent f20afc8d2f
commit b639192e46
1 changed files with 23 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
nanobot/providers/openai_codex_provider.py
View File

@ -123,11 +123,19 @@ def _convert_tools(tools: list[dict[str, Any]]) -> list[dict[str, Any]]:
# Nanobot tool definitions already use the OpenAI function schema. # Nanobot tool definitions already use the OpenAI function schema.
converted: list[dict[str, Any]] = [] converted: list[dict[str, Any]] = []
for tool in tools: for tool in tools:
name = tool.get("name") fn = tool.get("function") if isinstance(tool, dict) and tool.get("type") == "function" else None
if fn and isinstance(fn, dict):
name = fn.get("name")
desc = fn.get("description")
params = fn.get("parameters")
else:
name = tool.get("name")
desc = tool.get("description")
params = tool.get("parameters")
if not isinstance(name, str) or not name: if not isinstance(name, str) or not name:
# Skip invalid tools to avoid Codex rejection. # Skip invalid tools to avoid Codex rejection.
continue continue
params = tool.get("parameters") or {} params = params or {}
if not isinstance(params, dict): if not isinstance(params, dict):
# Parameters must be a JSON Schema object. # Parameters must be a JSON Schema object.
params = {} params = {}
@ -135,7 +143,7 @@ def _convert_tools(tools: list[dict[str, Any]]) -> list[dict[str, Any]]:
{ {
"type": "function", "type": "function",
"name": name, "name": name,
"description": tool.get("description") or "", "description": desc or "",
"parameters": params, "parameters": params,
} }
) )
@ -173,8 +181,9 @@ def _convert_messages(messages: list[dict[str, Any]]) -> tuple[str, list[dict[st
# Then handle tool calls. # Then handle tool calls.
for tool_call in msg.get("tool_calls", []) or []: for tool_call in msg.get("tool_calls", []) or []:
fn = tool_call.get("function") or {} fn = tool_call.get("function") or {}
call_id = tool_call.get("id") or f"call_{idx}" call_id, item_id = _split_tool_call_id(tool_call.get("id"))
item_id = f"fc_{idx}" call_id = call_id or f"call_{idx}"
item_id = item_id or f"fc_{idx}"
input_items.append( input_items.append(
{ {
"type": "function_call", "type": "function_call",
@ -226,6 +235,15 @@ def _extract_call_id(tool_call_id: Any) -> str:
return "call_0" return "call_0"
def _split_tool_call_id(tool_call_id: Any) -> tuple[str, str | None]:
if isinstance(tool_call_id, str) and tool_call_id:
if "|" in tool_call_id:
call_id, item_id = tool_call_id.split("|", 1)
return call_id, item_id or None
return tool_call_id, None
return "call_0", None
def _prompt_cache_key(messages: list[dict[str, Any]]) -> str: def _prompt_cache_key(messages: list[dict[str, Any]]) -> str:
raw = json.dumps(messages, ensure_ascii=True, sort_keys=True) raw = json.dumps(messages, ensure_ascii=True, sort_keys=True)
return hashlib.sha256(raw.encode("utf-8")).hexdigest() return hashlib.sha256(raw.encode("utf-8")).hexdigest()