Add Azure DocumentDB control-plane MCP proposal - #102
Conversation
Copilot-Session: f35a45d0-bef6-4341-91c5-7327b9667f06
German Eichberger (xgerman)
left a comment
There was a problem hiding this comment.
Summary
The architecture direction is understandable, but the proposal often states it in implementation shorthand rather than decision-oriented language. These non-blocking edits clarify committed versus conditional scope, potentially misleading undo/redo terminology, undefined acronyms, and the full-feature exit criteria. Together with the concrete replacement text in the existing threads, they should make the proposal easier for product, security, and engineering approvers to evaluate consistently.
Copilot-Session: f35a45d0-bef6-4341-91c5-7327b9667f06
German Eichberger (xgerman)
left a comment
There was a problem hiding this comment.
Summary
The update addresses the earlier clarity feedback well: it now separates target from validated scope, explains the native versus orchestrated paths with examples, makes the Azure CLI POC-only, adds a glossary, and turns acceptance into a checklist. I found four remaining non-blocking consistency issues where wording or examples conflict with definitions elsewhere in the proposal.
Copilot-Session: f35a45d0-bef6-4341-91c5-7327b9667f06
| `microsoft/mcp` is the Azure MCP Server source and contribution repository. | ||
| `mcp/com.microsoft/azure` is its GitHub MCP Registry publication and discovery | ||
| page, which links to the source repository and released installation options. | ||
| DocumentDB tool code is developed in `microsoft/mcp`, not in the registry |
There was a problem hiding this comment.
There may be one step missing from the proposal. The question is: can we put all work in microsoft/mcp alone and deploy it from there? We need an end-to-end proposal before diving into feature details
Summary