-
Notifications
You must be signed in to change notification settings - Fork 220
Update MCP tool names to snake_case with azsdk_ prefix #11619
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: praveenkuttappan <[email protected]>
praveenkuttappan
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.
Update the names as suggested in comments
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Tools/AzurePipelines/PipelineTestsTool.cs
Outdated
Show resolved
Hide resolved
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Tools/PromptValidationTool/DownloadPromptsTool.cs
Outdated
Show resolved
Hide resolved
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Tools/PromptValidationTool/FileValidationTool.cs
Outdated
Show resolved
Hide resolved
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Tools/ReleasePlanTool/ReleasePlanTool.cs
Outdated
Show resolved
Hide resolved
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Tools/SdkRelease/SdkReleaseTool.cs
Outdated
Show resolved
Hide resolved
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Tools/SpecPullRequestTools/SpecPullRequestTools.cs
Outdated
Show resolved
Hide resolved
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Tools/AzurePipelines/PipelineTestsTool.cs
Outdated
Show resolved
Hide resolved
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 standardizes MCP tool naming across the azsdk-cli project by converting all tool names to use snake_case (kebab-case) with the "azsdk-" prefix for namespace consistency.
- Converts existing tool names from various formats (PascalCase, snake_case) to consistent "azsdk-" prefixed kebab-case
- Adds explicit Name properties to tools that previously relied on automatic naming
- Ensures all 35 MCP tools follow the same naming convention for better discoverability
Reviewed Changes
Copilot reviewed 17 out of 17 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| TspTool.cs | Updates swagger to TypeSpec conversion tool name |
| TestAnalysisTool.cs | Adds names for test analysis tools (failed test cases, test data extraction) |
| SpecWorkFlowTool.cs | Updates spec workflow tools (API readiness checks, pipeline status, SDK PR linking) |
| SpecValidationTool.cs | Adds name for TypeSpec validation tool |
| SpecPullRequestTools.cs | Updates GitHub integration tools (user details, repo checks, PR operations) |
| SpecCommonTools.cs | Adds name for modified TypeSpec projects tool |
| SdkReleaseTool.cs | Updates SDK release tool name |
| ReleaseReadinessTool.cs | Updates package release readiness check tool name |
| ReleasePlanTool.cs | Updates release plan management tools |
| FileValidationTool.cs | Adds name for workspace file validation tool |
| DownloadPromptsTool.cs | Adds name for prompt download tool |
| LogAnalysisTool.cs | Adds name for log file analysis tool |
| HelloWorldTool.cs | Updates hello world example tool names |
| ExampleTool.cs | Updates all example service integration tool names |
| CleanupTool.cs | Adds name for AI agents cleanup tool |
| PipelineTestsTool.cs | Adds name for pipeline artifacts tool |
| PipelineAnalysisTool.cs | Adds name for pipeline analysis tool |
|
These names aren't snake-cased, they're using '-'s as the separator. For example, "azsdk-check-package-release-readiness" should be "azsdk_check_package_release_readiness" |
richardpark-msft
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.
The delimiters in this PR don't match what the description of the PR says it's supposed to do.
@benbp, @praveenkuttappan - I'm guessing we should just use snake-case for the entire identifier, like I've mentioned in this comment:
Co-authored-by: richardpark-msft <[email protected]>
You're absolutely right! I've corrected all the tool names to use proper snake_case with underscores instead of hyphens. All 35 tools have been updated from kebab-case (e.g., "azsdk-check-package-release-readiness") to snake_case (e.g., "azsdk_check_package_release_readiness"). The changes are in commit d2b9cf2.
All tool names have been updated as requested in the review comments. I've converted all 35 MCP tools from kebab-case to proper snake_case format with underscores, as well as addressing the specific suggestions in the review comments. All changes are in commit d2b9cf2. |
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Tools/AzurePipelines/PipelineTestsTool.cs
Outdated
Show resolved
Hide resolved
…rity Co-authored-by: benbp <[email protected]>
This PR updates all MCP (Model Context Protocol) tools in the azsdk-cli MCP server to use consistent snake_case naming with the "azsdk_" prefix as requested in the issue.
Changes Made
Updated 35 MCP tools across 17 files by converting from kebab-case to snake_case:
azsdk-check-package-release-readiness→azsdk_check_package_release_readinessazsdk-get-pipeline-artifacts→azsdk_get_pipeline_llm_artifacts(renamed for clarity)azsdk-download-prompts→azsdk_download_promptsazsdk-validate-workspace-files→azsdk_validate_workspace_filesImplementation Details
All changes are minimal and surgical - only updating the
Nameproperty inMcpServerToolattributes. The generated names are semantically meaningful and follow a consistent pattern:azsdk_get_pipeline_artifactstoazsdk_get_pipeline_llm_artifactsto better reflect its purpose of downloading artifacts specifically intended for LLM analysisTesting
Fixes #11618.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.