pjandro/OmniRoute
1
0
Fork 0
mirror of https://github.com/diegosouzapw/OmniRoute.git synced 2026-07-31 04:12:10 +03:00
Code Issues Packages Projects Releases Wiki Activity

fix(codex): preserve namespace MCP tools forwarded to Codex Responses API (#1483)

Browse Source 
Integrated into release/v3.7.0
This commit is contained in:
vanminhph
2026-04-22 03:18:26 +07:00
committed by GitHub
parent 3502c5afd1
commit 9cd36af3e8
2 changed files with 17 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
open-sse/executors/codex.ts
View File

@@ -336,6 +336,20 @@ function normalizeCodexTools(body: Record<string, unknown>): void {
}
const tool = toolValue as Record<string, unknown>;
// Preserve namespace tools (MCP tool groups used by Codex/OpenAI Responses API).
// Codex API supports them natively; register sub-tool names for tool_choice validation.
if (tool.type === "namespace" && Array.isArray(tool.tools)) {
for (const st of tool.tools as unknown[]) {
if (st && typeof st === "object" && !Array.isArray(st)) {
const subTool = st as Record<string, unknown>;
const name = typeof subTool.name === "string" ? subTool.name.trim() : "";
if (name) validToolNames.add(name);
}
}
return true;
}
if (tool.type !== "function") {
return false;
}

4
open-sse/translator/request/openai-responses.ts
View File

@@ -55,12 +55,14 @@ export function openaiResponsesToOpenAIRequest(
for (const toolValue of tools) {
const tool = toRecord(toolValue);
const toolType = toString(tool.type);
// Allow: function tools, tools already in Chat format (have .function property), and CLI subagent tools
// Allow: function tools, tools already in Chat format (have .function property), CLI subagent tools,
// and namespace tools (MCP tool groups used by Codex/OpenAI Responses API).
if (
toolType &&
toolType !== "function" &&
toolType !== "custom" &&
toolType !== "command" &&
toolType !== "namespace" &&
!tool.function
) {
throw unsupportedFeature(