From a2fee6bdd3927a2a08520ab87dabc8fecdb6075b Mon Sep 17 00:00:00 2001 From: Olcan Date: Sun, 8 Jun 2025 21:52:11 -0700 Subject: fix mcp timeouts and missing description on mcp errors (#868) --- packages/cli/src/ui/hooks/useToolScheduler.test.ts | 13 +++---------- 1 file changed, 3 insertions(+), 10 deletions(-) (limited to 'packages/cli/src/ui/hooks/useToolScheduler.test.ts') diff --git a/packages/cli/src/ui/hooks/useToolScheduler.test.ts b/packages/cli/src/ui/hooks/useToolScheduler.test.ts index f5a3529c..8e3f139b 100644 --- a/packages/cli/src/ui/hooks/useToolScheduler.test.ts +++ b/packages/cli/src/ui/hooks/useToolScheduler.test.ts @@ -924,7 +924,7 @@ describe('mapToDisplay', () => { expectedStatus: ToolCallStatus.Error, expectedResultDisplay: 'Error display tool not found', expectedName: baseRequest.name, - expectedDescription: '', + expectedDescription: JSON.stringify(baseRequest.args), }, { name: 'error tool execution failed', @@ -940,7 +940,7 @@ describe('mapToDisplay', () => { expectedStatus: ToolCallStatus.Error, expectedResultDisplay: 'Execution failed display', expectedName: baseTool.displayName, // Changed from baseTool.name - expectedDescription: '', + expectedDescription: baseTool.getDescription(baseRequest.args), }, { name: 'cancelled', @@ -986,14 +986,7 @@ describe('mapToDisplay', () => { expect(toolDisplay.resultDisplay).toBe(expectedResultDisplay); expect(toolDisplay.name).toBe(expectedName); - - if (status === 'error' && !extraProps?.tool) { - expect(toolDisplay.description).toBe(''); - } else { - expect(toolDisplay.description).toBe( - expectedDescription ?? baseTool.getDescription(baseRequest.args), - ); - } + expect(toolDisplay.description).toBe(expectedDescription); expect(toolDisplay.renderOutputAsMarkdown).toBe( extraProps?.tool?.isOutputMarkdown ?? false, -- cgit v1.2.3