[Onboard][Part One] Mcp Server for Azure DocumentDB (Index)#1968
[Onboard][Part One] Mcp Server for Azure DocumentDB (Index)#1968xingfan-git wants to merge 20 commits intomainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR onboards a new MCP tool area for Azure DocumentDB (with MongoDB compatibility), adding three connection management tools: ConnectionToggle (connect/disconnect), and GetConnectionStatus. It introduces the full tooling stack: commands, service interface/implementation, options, JSON context, helpers, unit tests, live tests, and supporting infrastructure.
Changes:
- New
Azure.Mcp.Tools.DocumentDbtool package with connection management commands (connect, disconnect, get_connection_status) - Server registration, consolidated tools JSON, documentation updates (README, azmcp-commands.md, e2eTestPrompts.md, CHANGELOG)
- Build infrastructure changes in
New-BuildInfo.ps1to disable trimming when MongoDB.Driver is present
Reviewed changes
Copilot reviewed 34 out of 34 changed files in this pull request and generated 13 comments.
Show a summary per file
| File | Description |
|---|---|
tools/Azure.Mcp.Tools.DocumentDb/src/** |
New DocumentDB tool package: commands, service, options, models, helpers, JSON context |
tools/Azure.Mcp.Tools.DocumentDb/tests/Azure.Mcp.Tools.DocumentDb.UnitTests/** |
Unit tests for connection commands |
tools/Azure.Mcp.Tools.DocumentDb/tests/Azure.Mcp.Tools.DocumentDb.LiveTests/** |
Live/recorded tests for connection commands |
servers/Azure.Mcp.Server/src/Program.cs |
Registers new DocumentDbSetup area |
servers/Azure.Mcp.Server/src/Resources/consolidated-tools.json |
Adds DocumentDB consolidated tool mappings |
servers/Azure.Mcp.Server/docs/azmcp-commands.md |
Adds DocumentDB CLI command docs (with incorrect syntax) |
servers/Azure.Mcp.Server/docs/e2eTestPrompts.md |
Adds DocumentDB test prompts |
servers/Azure.Mcp.Server/README.md |
Adds DocumentDB section |
servers/Azure.Mcp.Server/CHANGELOG.md + changelog-entries/*.yaml |
Changelog entries (duplicate entries present) |
servers/Azure.Mcp.Server/Azure.Mcp.Server.slnx |
Adds DocumentDB projects to solution |
eng/scripts/New-BuildInfo.ps1 |
Disables trimming based on MongoDB dependency detection (regex is broken) |
Directory.Packages.props |
Adds MongoDB.Driver 3.2.0 package version |
.github/CODEOWNERS |
Adds DocumentDB code owners |
...re.Mcp.Tools.DocumentDb/tests/Azure.Mcp.Tools.DocumentDb.LiveTests/DocumentDbCommandTests.cs
Outdated
Show resolved
Hide resolved
tools/Azure.Mcp.Tools.DocumentDb/src/Services/DocumentDbService.cs
Outdated
Show resolved
Hide resolved
tools/Azure.Mcp.Tools.DocumentDb/src/Services/DocumentDbService.cs
Outdated
Show resolved
Hide resolved
...re.Mcp.Tools.DocumentDb/tests/Azure.Mcp.Tools.DocumentDb.LiveTests/DocumentDbCommandTests.cs
Outdated
Show resolved
Hide resolved
tools/Azure.Mcp.Tools.DocumentDb/src/Services/IDocumentDbService.cs
Outdated
Show resolved
Hide resolved
tools/Azure.Mcp.Tools.DocumentDb/src/Commands/DocumentDbOptionDefinitions.cs
Show resolved
Hide resolved
tools/Azure.Mcp.Tools.DocumentDb/src/Commands/Connection/ConnectionToggleCommand.cs
Outdated
Show resolved
Hide resolved
tools/Azure.Mcp.Tools.DocumentDb/src/Commands/Connection/ConnectionToggleCommand.cs
Outdated
Show resolved
Hide resolved
tools/Azure.Mcp.Tools.DocumentDb/src/Commands/Connection/ConnectionToggleCommand.cs
Outdated
Show resolved
Hide resolved
tools/Azure.Mcp.Tools.DocumentDb/src/Commands/Connection/ConnectionToggleCommand.cs
Outdated
Show resolved
Hide resolved
tools/Azure.Mcp.Tools.DocumentDb/src/Commands/Connection/ConnectionToggleCommand.cs
Outdated
Show resolved
Hide resolved
tools/Azure.Mcp.Tools.DocumentDb/src/Commands/DocumentDbHelpers.cs
Outdated
Show resolved
Hide resolved
tools/Azure.Mcp.Tools.DocumentDb/src/Commands/DocumentDbOptionDefinitions.cs
Show resolved
Hide resolved
tools/Azure.Mcp.Tools.DocumentDb/src/Services/DocumentDbService.cs
Outdated
Show resolved
Hide resolved
tools/Azure.Mcp.Tools.DocumentDb/src/Commands/Connection/ConnectionToggleCommand.cs
Outdated
Show resolved
Hide resolved
tools/Azure.Mcp.Tools.DocumentDb/src/Commands/Connection/GetConnectionStatusCommand.cs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
@hallipr could you review this change, as I don't know the trimmed build usage well enough and this PR is added a dependency which doesn't support trimming.
There was a problem hiding this comment.
Why don't we see a CI failure for this ?
What does this PR do?
This PR added MCP tools for managing Azure DocumentDB (with MongoDB compatibility) index:
[Any additional context, screenshots, or information that helps reviewers]Documentation:
Azure DocumentDB (with MongoDB compatibility) documentation
GitHub issue number?
[MCP Server Request] DocumentDB previously known as Azure Cosmos DB for MongoDB (vCore)
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