Skip to content

Conversation

fanyang-mono
Copy link
Contributor

@fanyang-mono fanyang-mono commented Oct 8, 2025

What does this PR do?

Add a command listing all the tool names. This will come handy when using --tool option to specify tools to be included.

GitHub issue number?

N/A

Pre-merge Checklist

  • Required for All PRs
    • Read contribution guidelines
    • PR title clearly describes the change
    • Commit history is clean with descriptive messages (cleanup guide)
    • Added comprehensive tests for new/modified functionality
    • Updated servers/Azure.Mcp.Server/CHANGELOG.md and/or servers/Fabric.Mcp.Server/CHANGELOG.md for product changes (features, bug fixes, UI/UX, updated dependencies)
  • For MCP tool changes:
    • One tool per PR: This PR adds or modifies only one MCP tool for faster review cycles
    • Updated servers/Azure.Mcp.Server/README.md and/or servers/Fabric.Mcp.Server/README.md documentation
    • Updated command list in /servers/Azure.Mcp.Server/docs/azmcp-commands.md and/or /docs/fabric-commands.md
    • For new or modified tool descriptions, ran ToolDescriptionEvaluator and obtained a score of 0.4 or more and a top 3 ranking for all related test prompts
    • For new tools associated with Azure services or publicly available tools/APIs/products, add URL to documentation in the PR description
  • Extra steps for Azure MCP Server tool changes:
    • Updated test prompts in /servers/Azure.Mcp.Server/docs/e2eTestPrompts.md
    • 👉 For Community (non-Microsoft team member) PRs:
      • Security review: Reviewed code for security vulnerabilities, malicious code, or suspicious activities before running tests (crypto mining, spam, data exfiltration, etc.)
      • Manual tests run: added comment /azp run mcp - pullrequest - live to run Live Test Pipeline

@Copilot Copilot AI review requested due to automatic review settings October 8, 2025 17:41
@fanyang-mono fanyang-mono requested a review from a team as a code owner October 8, 2025 17:41
@github-project-automation github-project-automation bot moved this to Untriaged in Azure MCP Server Oct 8, 2025
Copy link
Contributor

@Copilot Copilot AI left a 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 command to list all available tool names in the Azure MCP server. The command provides a simple enumeration of tool names without descriptions or metadata, making it useful for quick discovery and automated tool enumeration, particularly when using the --tool option to specify tools.

  • Introduces a new list-names command under the tools area
  • Adds necessary infrastructure for JSON serialization and dependency injection
  • Creates minimal options class as the command requires no additional parameters

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
core/Azure.Mcp.Core/src/Models/ModelsJsonContext.cs Adds JSON serialization support for the new command's result type
core/Azure.Mcp.Core/src/Areas/Tools/ToolsSetup.cs Registers the new command in the dependency injection container and command group
core/Azure.Mcp.Core/src/Areas/Tools/Options/ToolsListNamesOptions.cs Creates empty options class for the new command
core/Azure.Mcp.Core/src/Areas/Tools/Commands/ToolsListNamesCommand.cs Implements the main command logic to list all tool names

@github-project-automation github-project-automation bot moved this from Untriaged to Done in Azure MCP Server Oct 9, 2025
@fanyang-mono fanyang-mono reopened this Oct 9, 2025
@github-project-automation github-project-automation bot moved this from Done to Untriaged in Azure MCP Server Oct 9, 2025
@github-project-automation github-project-automation bot moved this from Untriaged to In Progress in Azure MCP Server Oct 9, 2025
@fanyang-mono fanyang-mono enabled auto-merge (squash) October 9, 2025 20:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

3 participants