diff options
| author | Bryan Morgan <[email protected]> | 2025-06-07 18:30:56 -0400 |
|---|---|---|
| committer | GitHub <[email protected]> | 2025-06-07 18:30:56 -0400 |
| commit | e95a6086fc89191888f2a4e41a3c875273311d96 (patch) | |
| tree | 907073862c859af9eb2222578b637f455325e6e2 /packages/core/src/tools/mcp-tool.test.ts | |
| parent | dd08582f81b1cab4d3ded0f21fc4359ac88b231e (diff) | |
Bryanmorgan/add mcp description support (#825)
Diffstat (limited to 'packages/core/src/tools/mcp-tool.test.ts')
| -rw-r--r-- | packages/core/src/tools/mcp-tool.test.ts | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/packages/core/src/tools/mcp-tool.test.ts b/packages/core/src/tools/mcp-tool.test.ts index fc6ce6be..d972efdb 100644 --- a/packages/core/src/tools/mcp-tool.test.ts +++ b/packages/core/src/tools/mcp-tool.test.ts @@ -52,7 +52,7 @@ describe('DiscoveredMCPTool', () => { }); describe('constructor', () => { - it('should set properties correctly and augment description (non-generic server)', () => { + it('should set properties correctly (non-generic server)', () => { const tool = new DiscoveredMCPTool( mockCallableToolInstance, serverName, // serverName is 'mock-mcp-server', not 'mcp' @@ -64,14 +64,13 @@ describe('DiscoveredMCPTool', () => { expect(tool.name).toBe(toolNameForModel); expect(tool.schema.name).toBe(toolNameForModel); - const expectedDescription = `${baseDescription}\n\nThis MCP tool named '${serverToolName}' was discovered from an MCP server.`; - expect(tool.schema.description).toBe(expectedDescription); + expect(tool.schema.description).toBe(baseDescription); expect(tool.schema.parameters).toEqual(inputSchema); expect(tool.serverToolName).toBe(serverToolName); expect(tool.timeout).toBeUndefined(); }); - it('should set properties correctly and augment description (generic "mcp" server)', () => { + it('should set properties correctly (generic "mcp" server)', () => { const genericServerName = 'mcp'; const tool = new DiscoveredMCPTool( mockCallableToolInstance, @@ -81,8 +80,7 @@ describe('DiscoveredMCPTool', () => { inputSchema, serverToolName, ); - const expectedDescription = `${baseDescription}\n\nThis MCP tool named '${serverToolName}' was discovered from '${genericServerName}' MCP server.`; - expect(tool.schema.description).toBe(expectedDescription); + expect(tool.schema.description).toBe(baseDescription); }); it('should accept and store a custom timeout', () => { |
