Conversation
There was a problem hiding this comment.
Pull request overview
Prepares the Azure MCP Server for the 2.0.0 release by updating the server package version and adding consolidated 2.0.0 release notes to the relevant changelogs.
Changes:
- Bumped
Azure.Mcp.Serverpackage version to2.0.0. - Added a new
2.0.0release section to the server changelog, including high-level highlights and a “since 1.0.0” summary. - Added a new
2.0.0release section to the VS Code changelog with installation/documentation links.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| servers/Azure.Mcp.Server/vscode/CHANGELOG.md | Adds top-level 2.0.0 release notes for the VS Code-facing changelog. |
| servers/Azure.Mcp.Server/src/Azure.Mcp.Server.csproj | Updates the server/package version to 2.0.0. |
| servers/Azure.Mcp.Server/CHANGELOG.md | Adds top-level 2.0.0 release notes and a consolidated “since 1.0.0” section. |
jongio
left a comment
There was a problem hiding this comment.
Two items not yet flagged: (1) the Azure AI Foundry branding joshfree caught in the main CHANGELOG also appears in the VS Code CHANGELOG (lines 19, 21) and needs the same fix, (2) the VS Code CHANGELOG is missing the breaking changes summary that the main CHANGELOG includes - users upgrading from 1.x won't have a consolidated view of what changed. Also a nit: over 40 beta iterations in both changelogs - there are exactly 40 (beta.1 through beta.40).
jongio
left a comment
There was a problem hiding this comment.
Nit: the breaking changes section mixes regular hyphens and en-dash characters as separators between the Removed and use descriptions. Cosmos DB and Key Vault lines use the hyphen, while the other 9 lines use the en-dash character. Same in both changelogs. Not blocking - just a consistency thing if you are doing another pass.
|
|
||
| - **Cosmos DB/MySQL/PostgreSQL**: Consolidated **list** commands: | ||
| - Removed `cosmos account list`, `cosmos database list`, and `cosmos database container list` - use `cosmos list` with the appropriate parameters to route hierarchically | ||
| - Removed `postgres server list`, `postgres database list`, `postgres table list` – use `postgres list` with the appropriate parameters to route hierarchically |
There was a problem hiding this comment.
Minor: this line and the following service entries use the en-dash character as a separator, but the Cosmos DB entry above and Key Vault entries below use a regular hyphen. Consider picking one for consistency across the section.
What does this PR do?
Updated changelogs and version ahead of release.
GitHub issue number?
N/A
Pre-merge Checklist
servers/Azure.Mcp.Server/README.mdand/orservers/Fabric.Mcp.Server/README.mddocumentationREADME.mdchanges running the script./eng/scripts/Process-PackageReadMe.ps1. See Package READMEToolDescriptionEvaluatorand obtained a score of0.4or more and a top 3 ranking for all related test promptsconsolidated-tools.jsonbreaking-changelabelservers/Azure.Mcp.Server/docs/azmcp-commands.md./eng/scripts/Update-AzCommandsMetadata.ps1to update tool metadata inazmcp-commands.md(required for CI)servers/Azure.Mcp.Server/docs/e2eTestPrompts.mdcrypto mining, spam, data exfiltration, etc.)/azp run mcp - pullrequest - liveto run Live Test Pipeline