-
Notifications
You must be signed in to change notification settings - Fork 5
Add expandable tool details showing descriptions and input schemas #48
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: garland3 <1162675+garland3@users.noreply.github.com>
Co-authored-by: garland3 <1162675+garland3@users.noreply.github.com>
…tion Co-authored-by: garland3 <1162675+garland3@users.noreply.github.com>
|
@copilot Ok. It seems like the is no longer needed and the drop down since the user can see more information by click on the "i" and also just click on the tool or prompt to load or unload it. Please remove this drop down. |
Co-authored-by: garland3 <1162675+garland3@users.noreply.github.com>
Removed the dropdown expand/collapse section in commit 9cdff5d. Users can now interact with tools directly by clicking them to enable/disable, and use the info icon to see details. |
garland3
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tested this and it seems to work like expected.
| def test_tools_detailed_includes_description_and_schema(mock_mcp_manager): | ||
| """Test that tools_detailed field contains description and inputSchema.""" | ||
| server_tools = mock_mcp_manager.available_tools["test_server"]["tools"] | ||
| server_config = mock_mcp_manager.available_tools["test_server"]["config"] |
Check notice
Code scanning / CodeQL
Unused local variable Note test
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix
AI 23 days ago
The best way to fix this problem is to simply remove the assignment to server_config at line 64. Since the assigned value is not used and the assignment itself does not cause any side effects, this removal is safe and will not affect the functionality of the test. The removal should only affect this single line in the code (server_config = ...), and no imports or further changes are needed.
| @@ -61,7 +61,6 @@ | ||
| def test_tools_detailed_includes_description_and_schema(mock_mcp_manager): | ||
| """Test that tools_detailed field contains description and inputSchema.""" | ||
| server_tools = mock_mcp_manager.available_tools["test_server"]["tools"] | ||
| server_config = mock_mcp_manager.available_tools["test_server"]["config"] | ||
|
|
||
| # Simulate what the config endpoint does | ||
| tools_detailed = [] |

MCP tool users need visibility into tool capabilities—what each tool does and what arguments it accepts—without cluttering the default UI.
Changes
Backend (
config_routes.py)tools_detailedarray to/api/configresponse containing name, description, and inputSchema for each toolFrontend (
ToolsPanel.jsx)Example API response structure:
{ "tools_detailed": [{ "name": "canvas", "description": "Display final rendered content...", "inputSchema": { "type": "object", "properties": { "content": { "type": "string", "description": "The content to display..." } }, "required": ["content"] } }] }UI
Default state: Tools display as simple buttons with info icons
Expanded state: Click info icon reveals description and all input arguments with types and requirement indicators
Interaction: Click tool buttons directly to enable/disable, no dropdown needed
Original prompt
<issue_description>When using the mcp servers, the atlas-ui-3 should surface to the frontend the ability to see the specific description and input args about individual functions. This should be hidden by default, but allow the user to see more information about what the tool does and what input args and arg descriptions. This should not be expanded by default. </issue_description>
Comments on the Issue (you are @copilot in this section)
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.