diff options
| author | Bryan Morgan <[email protected]> | 2025-06-08 19:07:05 -0400 |
|---|---|---|
| committer | GitHub <[email protected]> | 2025-06-08 19:07:05 -0400 |
| commit | 152af28a347186517c46fc0d9ae88eb6fa883267 (patch) | |
| tree | 258dc6fdc04e744c11faceb2212056a020d87d2c | |
| parent | 31c14ea78f19b3e2720150943855ea8d15fdeaff (diff) | |
Bryanmorgan/add tool source description (#856)
| -rw-r--r-- | packages/cli/src/ui/hooks/slashCommandProcessor.test.ts | 6 | ||||
| -rw-r--r-- | packages/cli/src/ui/hooks/slashCommandProcessor.ts | 2 | ||||
| -rw-r--r-- | packages/core/src/tools/mcp-tool.ts | 2 |
3 files changed, 5 insertions, 5 deletions
diff --git a/packages/cli/src/ui/hooks/slashCommandProcessor.test.ts b/packages/cli/src/ui/hooks/slashCommandProcessor.test.ts index 3fcdff97..aa1e701f 100644 --- a/packages/cli/src/ui/hooks/slashCommandProcessor.test.ts +++ b/packages/cli/src/ui/hooks/slashCommandProcessor.test.ts @@ -423,8 +423,8 @@ Add any other context about the problem here. it('should display only Gemini CLI tools (filtering out MCP tools)', async () => { // Create mock tools - some with serverName property (MCP tools) and some without (Gemini CLI tools) const mockTools = [ - { name: 'tool1' }, - { name: 'tool2' }, + { name: 'tool1', displayName: 'Tool1' }, + { name: 'tool2', displayName: 'Tool2' }, { name: 'mcp_tool1', serverName: 'mcp-server1' }, { name: 'mcp_tool2', serverName: 'mcp-server1' }, ]; @@ -447,7 +447,7 @@ Add any other context about the problem here. 2, expect.objectContaining({ type: MessageType.INFO, - text: 'Available Gemini CLI tools:\n\ntool1\ntool2', + text: 'Available Gemini CLI tools:\n\nTool1\nTool2', }), expect.any(Number), ); diff --git a/packages/cli/src/ui/hooks/slashCommandProcessor.ts b/packages/cli/src/ui/hooks/slashCommandProcessor.ts index 85ae825e..daec0379 100644 --- a/packages/cli/src/ui/hooks/slashCommandProcessor.ts +++ b/packages/cli/src/ui/hooks/slashCommandProcessor.ts @@ -350,7 +350,7 @@ export const useSlashCommandProcessor = ( // Filter out MCP tools by checking if they have a serverName property const geminiTools = tools.filter((tool) => !('serverName' in tool)); - const geminiToolList = geminiTools.map((tool) => tool.name); + const geminiToolList = geminiTools.map((tool) => tool.displayName); addMessage({ type: MessageType.INFO, diff --git a/packages/core/src/tools/mcp-tool.ts b/packages/core/src/tools/mcp-tool.ts index ffe12cf7..d5a8ccb5 100644 --- a/packages/core/src/tools/mcp-tool.ts +++ b/packages/core/src/tools/mcp-tool.ts @@ -32,7 +32,7 @@ export class DiscoveredMCPTool extends BaseTool<ToolParams, ToolResult> { ) { super( name, - name, + `${serverToolName} (${serverName} MCP Server)`, description, parameterSchema, true, // isOutputMarkdown |
