diff options
Diffstat (limited to 'packages/core/src/tools')
| -rw-r--r-- | packages/core/src/tools/tool-registry.test.ts | 6 | ||||
| -rw-r--r-- | packages/core/src/tools/tool-registry.ts | 25 |
2 files changed, 27 insertions, 4 deletions
diff --git a/packages/core/src/tools/tool-registry.test.ts b/packages/core/src/tools/tool-registry.test.ts index ab337252..de355a98 100644 --- a/packages/core/src/tools/tool-registry.test.ts +++ b/packages/core/src/tools/tool-registry.test.ts @@ -312,7 +312,7 @@ describe('ToolRegistry', () => { return mockChildProcess as any; }); - await toolRegistry.discoverTools(); + await toolRegistry.discoverAllTools(); const discoveredTool = toolRegistry.getTool('tool-with-bad-format'); expect(discoveredTool).toBeDefined(); @@ -338,7 +338,7 @@ describe('ToolRegistry', () => { }; vi.spyOn(config, 'getMcpServers').mockReturnValue(mcpServerConfigVal); - await toolRegistry.discoverTools(); + await toolRegistry.discoverAllTools(); expect(mockDiscoverMcpTools).toHaveBeenCalledWith( mcpServerConfigVal, @@ -360,7 +360,7 @@ describe('ToolRegistry', () => { }; vi.spyOn(config, 'getMcpServers').mockReturnValue(mcpServerConfigVal); - await toolRegistry.discoverTools(); + await toolRegistry.discoverAllTools(); expect(mockDiscoverMcpTools).toHaveBeenCalledWith( mcpServerConfigVal, diff --git a/packages/core/src/tools/tool-registry.ts b/packages/core/src/tools/tool-registry.ts index a6742c06..b72ed9a5 100644 --- a/packages/core/src/tools/tool-registry.ts +++ b/packages/core/src/tools/tool-registry.ts @@ -153,8 +153,9 @@ export class ToolRegistry { /** * Discovers tools from project (if available and configured). * Can be called multiple times to update discovered tools. + * This will discover tools from the command line and from MCP servers. */ - async discoverTools(): Promise<void> { + async discoverAllTools(): Promise<void> { // remove any previously discovered tools for (const tool of this.tools.values()) { if (tool instanceof DiscoveredTool || tool instanceof DiscoveredMCPTool) { @@ -174,6 +175,28 @@ export class ToolRegistry { } /** + * Discovers tools from project (if available and configured). + * Can be called multiple times to update discovered tools. + * This will NOT discover tools from the command line, only from MCP servers. + */ + async discoverMcpTools(): Promise<void> { + // remove any previously discovered tools + for (const tool of this.tools.values()) { + if (tool instanceof DiscoveredMCPTool) { + this.tools.delete(tool.name); + } + } + + // discover tools using MCP servers, if configured + await discoverMcpTools( + this.config.getMcpServers() ?? {}, + this.config.getMcpServerCommand(), + this, + this.config.getDebugMode(), + ); + } + + /** * Discover or re-discover tools for a single MCP server. * @param serverName - The name of the server to discover tools from. */ |
