-
Notifications
You must be signed in to change notification settings - Fork 196
Add --namespaces
switch to azmcp tools list
#496
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
base: main
Are you sure you want to change the base?
Conversation
--namespaces
switch to azmcp tools list
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.
Pull Request Overview
This PR adds a new --namespaces
option to the azmcp tools list
command that returns top-level service namespaces with their subcommands instead of a flat list of all commands. It also introduces a count
field to provide quantitative information about namespaces and commands.
- Added
--namespaces
flag to list top-level service namespaces with their subcommands - Introduced
SubcommandsCount
field toCommandInfo
andResultsCount
field toCommandResponse
- Updated documentation and comprehensive test coverage for the new functionality
Reviewed Changes
Copilot reviewed 5 out of 7 changed files in this pull request and generated 2 comments.
Show a summary per file
File | Description |
---|---|
ToolsListCommand.cs | Implements the new --namespaces option logic and count field population |
CommandInfo.cs | Adds SubcommandsCount field to track number of subcommands |
CommandResponse.cs | Adds ResultsCount field to track total number of results |
ToolsListCommandTests.cs | Comprehensive test coverage for new namespace functionality and count fields |
azmcp-commands.md | Documentation updates for the new option and count fields |
core/Azure.Mcp.Core/src/Areas/Tools/Commands/ToolsListCommand.cs
Outdated
Show resolved
Hide resolved
core/Azure.Mcp.Core/src/Areas/Tools/Commands/ToolsListCommand.cs
Outdated
Show resolved
Hide resolved
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
core/Azure.Mcp.Core/src/Areas/Tools/Commands/ToolsListCommand.cs
Outdated
Show resolved
Hide resolved
...Azure.Mcp.Core/tests/Azure.Mcp.Core.UnitTests/Areas/Tools/UnitTests/ToolsListCommandTests.cs
Outdated
Show resolved
Hide resolved
…tion time" message to console output
…p individual tool prompts to the consolidated ones
What does this PR do?
This pull request adds a new
--namespaces
option to theazmcp tools list
command, allowing users to list top-level service namespaces instead of individual commands. It also introduces acount
field in the output to indicate the number of top-level namespaces and their subcommands. Tests and documentation have been updated accordingly.GitHub issue number?
N/A
Pre-merge Checklist
servers/Azure.Mcp.Server/CHANGELOG.md
and/orservers/Fabric.Mcp.Server/CHANGELOG.md
for product changes (features, bug fixes, UI/UX, updated dependencies
)servers/Azure.Mcp.Server/README.md
and/orservers/Fabric.Mcp.Server/README.md
documentation/docs/azmcp-commands.md
and/or/docs/fabric-commands.md
ToolDescriptionEvaluator
and obtained a score of0.4
or more and a top 3 ranking for all related test prompts/docs/e2eTestPrompts.md
crypto mining, spam, data exfiltration, etc.
)/azp run mcp - pullrequest - live
to run Live Test Pipeline