-
Notifications
You must be signed in to change notification settings - Fork 310
Fix XML doc namespace and update new-command.md for Microsoft.Mcp.Core migration #1319
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
…statements Co-authored-by: hallipr <[email protected]>
7ee723f to
ac1ede4
Compare
|
@copilot, the base PR has been updated with many using statement changes. Reset this PR and redo only the xml and markdown doc changes. |
I've reset the PR to commit 6cae57c and created a new commit (0c8d6d8) with only the XML documentation and markdown changes as requested. However, I'm unable to force push due to system constraints. The changes are ready and committed locally:
No tool file using statement changes are included. The branch needs a force push to update the remote. |
…e migration Co-authored-by: hallipr <[email protected]>
What does this PR do?
Completes namespace migration follow-up from #1318 for XML documentation and developer guidance. This PR addresses only the XML documentation and markdown documentation updates, excluding tool file using statement changes which are now handled in the base PR.
Changes
XML Documentation: Added full namespace qualifiers in
SingleProxyToolLoader.csto fix documentation generation and IDE tooling support:Microsoft.Mcp.Core.Commands.IBaseCommandMicrosoft.Mcp.Core.Areas.IAreaSetupDeveloper Documentation: Updated
new-command.mdusing statement examples to reflect the namespace migration:using Microsoft.Mcp.Core.Commands;TrimAnnotations.CommandAnnotationsExample
Note:
Azure.Mcp.Core.Commands.Subscriptioncorrectly remains in the Azure namespace.GitHub issue number?
Related to #1318
Pre-merge Checklist
servers/Azure.Mcp.Server/CHANGELOG.mdand/orservers/Fabric.Mcp.Server/CHANGELOG.mdfor product changes (features, bug fixes, UI/UX, updated dependencies)servers/Azure.Mcp.Server/README.mdand/orservers/Fabric.Mcp.Server/README.mddocumentationeng/scripts/Process-PackageReadMe.ps1. See Package README/servers/Azure.Mcp.Server/docs/azmcp-commands.mdand/or/docs/fabric-commands.md.\eng\scripts\Update-AzCommandsMetadata.ps1to update tool metadata in azmcp-commands.md (required for CI)ToolDescriptionEvaluatorand obtained a score of0.4or more and a top 3 ranking for all related test promptsconsolidated-tools.json/servers/Azure.Mcp.Server/docs/e2eTestPrompts.mdcrypto mining, spam, data exfiltration, etc.)/azp run mcp - pullrequest - liveto run Live Test Pipeline💡 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.