Skip to content

[Onboard][Part One] Mcp Server for Azure DocumentDB (Index)#1968

Open
xingfan-git wants to merge 20 commits intomainfrom
users/xingfan/onboardingvcoremcp_connection
Open

[Onboard][Part One] Mcp Server for Azure DocumentDB (Index)#1968
xingfan-git wants to merge 20 commits intomainfrom
users/xingfan/onboardingvcoremcp_connection

Conversation

@xingfan-git
Copy link

@xingfan-git xingfan-git commented Mar 9, 2026

What does this PR do?

This PR added MCP tools for managing Azure DocumentDB (with MongoDB compatibility) index:

  • Index tools (5): Create index, List indexes, Drop index, Get index stats, Get current operations

[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

  • 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
    • Validate README.md changes using script at eng/scripts/Process-PackageReadMe.ps1. See Package README
    • Updated command list in /servers/Azure.Mcp.Server/docs/azmcp-commands.md and/or /docs/fabric-commands.md
    • Run .\eng\scripts\Update-AzCommandsMetadata.ps1 to update tool metadata in azmcp-commands.md (required for CI)
    • 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 tools with new names, including new tools or renamed tools, update consolidated-tools.json
    • 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

@xingfan-git xingfan-git requested review from a team as code owners March 9, 2026 13:03
Copilot AI review requested due to automatic review settings March 9, 2026 13:03
@xingfan-git xingfan-git requested a review from a team as a code owner March 9, 2026 13:03
@github-project-automation github-project-automation bot moved this to Untriaged in Azure MCP Server Mar 9, 2026
Copy link
Contributor

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 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.DocumentDb tool 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.ps1 to 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

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@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.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why don't we see a CI failure for this ?

@xingfan-git xingfan-git changed the title [Onboard][Part One] Mcp Server for Azure DocumentDB (Connection) [Onboard][Part One] Mcp Server for Azure DocumentDB (Index) Mar 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Untriaged

Development

Successfully merging this pull request may close these issues.

4 participants