Improve error message for discoverTools function (#4157)

This commit is contained in:
Gal Zahavi
2025-07-30 17:16:21 -07:00
committed by GitHub
parent 8b645ff688
commit 152de2b6d8
2 changed files with 71 additions and 14 deletions

View File

@@ -428,21 +428,29 @@ export async function discoverTools(
const discoveredTools: DiscoveredMCPTool[] = [];
for (const funcDecl of tool.functionDeclarations) {
if (!isEnabled(funcDecl, mcpServerName, mcpServerConfig)) {
continue;
}
try {
if (!isEnabled(funcDecl, mcpServerName, mcpServerConfig)) {
continue;
}
discoveredTools.push(
new DiscoveredMCPTool(
mcpCallableTool,
mcpServerName,
funcDecl.name!,
funcDecl.description ?? '',
funcDecl.parametersJsonSchema ?? { type: 'object', properties: {} },
mcpServerConfig.timeout ?? MCP_DEFAULT_TIMEOUT_MSEC,
mcpServerConfig.trust,
),
);
discoveredTools.push(
new DiscoveredMCPTool(
mcpCallableTool,
mcpServerName,
funcDecl.name!,
funcDecl.description ?? '',
funcDecl.parametersJsonSchema ?? { type: 'object', properties: {} },
mcpServerConfig.timeout ?? MCP_DEFAULT_TIMEOUT_MSEC,
mcpServerConfig.trust,
),
);
} catch (error) {
console.error(
`Error discovering tool: '${
funcDecl.name
}' from MCP server '${mcpServerName}': ${(error as Error).message}`,
);
}
}
return discoveredTools;
} catch (error) {