Improve tool display: show args in progress and clean MCP tool names #60
Closed
Improve tool display: show args in progress and clean MCP tool names
#60
Conversation
- generateProgressSummary now shows tool arguments in compact JSON format instead of the generic "Calling <func>..." message, giving users clearer visibility into what each tool call is doing - Add MCPManager.GetToolDisplayName to strip the internal "mcp_<server>_" prefix from MCP tool names, showing a cleaner "server:tool" format in both progress indicators and --show-commands output https://claude.ai/code/session_014PBU2AUUYoCw3UpcW7YFyS
There was a problem hiding this comment.
Pull request overview
This PR improves the user-facing display of tool calls in two ways: (1) the progress indicator now shows tool arguments in a compact JSON format instead of a generic "Calling <func>..." message, and (2) MCP tool names are displayed in a cleaner server:tool format by stripping the internal mcp_<server>_ prefix. These changes affect both the inline progress indicator (shown during tool execution) and the --show-commands output.
Changes:
generateProgressSummaryinapplication.gonow round-trips args through JSON unmarshal/marshal to produce compact, key-sorted JSON, truncated to 50 characters, and displays them alongside the function name.- A new
GetToolDisplayNamemethod onMCPManagerstrips themcp_<server>_prefix from tool names and returns aserver:tooldisplay format. handleMCPToolCalluses the new display name for both progress indicators and--show-commandsoutput.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
mcp.go |
Adds GetToolDisplayName method to MCPManager for cleaner MCP tool name display |
application.go |
Rewrites generateProgressSummary to show compact JSON args; uses GetToolDisplayName in MCP tool call handling |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
Owner
Author
|
While this fixes a few issues, not exactly doing what I want. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
instead of the generic "Calling ..." message, giving users clearer
visibility into what each tool call is doing
prefix from MCP tool names, showing a cleaner "server:tool" format in
both progress indicators and --show-commands output
https://claude.ai/code/session_014PBU2AUUYoCw3UpcW7YFyS