Skip to content
This repository was archived by the owner on Sep 4, 2025. It is now read-only.

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Jul 30, 2025

The documentation was out of sync with the breaking change introduced in PR #704, which changed the default behavior of azmcp server start from exposing all individual tools (~128 tools) to namespace-level tools (~25 tools).

What was fixed

Before this fix:

  • Documentation incorrectly described the default mode as exposing "all tools individually"
  • Manual install examples were inconsistent and confusing
  • Server mode documentation referenced outdated behavior
  • No clear migration path for users wanting the old behavior

After this fix:

  • All documentation correctly describes namespace mode as the default
  • Clear examples showing both default behavior and --mode all for individual tools
  • Added prominent notice about the v0.5.0 breaking change with reference link
  • Consistent terminology across all documentation files

Files updated

  • README.md: Added breaking change notice with reference link, restructured manual install examples, updated custom client examples
  • CONTRIBUTING.md: Fixed server modes section to reflect correct default behavior and standardized terminology
  • docs/azmcp-commands.md: Removed redundant sections, clarified default behavior, added version context
  • TROUBLESHOOTING.md: Updated server mode examples to reflect new defaults

Key improvements

The documentation now clearly explains that:

  • azmcp server start uses namespace mode by default (since v0.5.0)
  • azmcp server start --mode all restores pre-v0.5.0 behavior with individual tools
  • The change helps prevent VS Code from hitting the 128 tool limit
  • Users have clear migration paths depending on their needs
  • Consistent terminology distinguishes "Namespace Mode" (default) from "Namespace Filtering" (service-specific filtering)

Fixes #781.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

@Copilot Copilot AI changed the title [WIP] Documentation is out of sync from code Fix documentation sync after PR #704 default mode change to namespace Jul 30, 2025
@Copilot Copilot AI requested a review from wbreza July 30, 2025 17:54
Copilot finished work on behalf of wbreza July 30, 2025 17:54
@wbreza wbreza marked this pull request as ready for review July 30, 2025 21:02
@Copilot Copilot AI review requested due to automatic review settings July 30, 2025 21:02
@wbreza wbreza requested a review from a team as a code owner July 30, 2025 21:02
Copy link
Contributor

@Copilot 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 fixes documentation that became out of sync with the breaking change introduced in PR #704, which changed the default behavior of azmcp server start from exposing all individual tools (~128 tools) to namespace-level tools (~25 tools).

  • Updated documentation to correctly reflect namespace mode as the default behavior
  • Added prominent breaking change notices and migration guidance for users
  • Restructured examples to show both default behavior and --mode all for individual tools

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
docs/azmcp-commands.md Clarified default mode descriptions, added version context, removed redundant sections
TROUBLESHOOTING.md Updated server mode examples to reflect new defaults
README.md Added breaking change notice, restructured manual install examples with clear configuration options
CONTRIBUTING.md Fixed server modes section terminology and updated example configurations
Comments suppressed due to low confidence (2)

docs/azmcp-commands.md:27

  • [nitpick] The heading uses inconsistent capitalization. 'Namespace-level' should use lowercase 'l' for consistency with other headings in the file.
#### Default Mode (Namespace-level Tools)

CONTRIBUTING.md:176

  • [nitpick] The terminology is inconsistent with the docs/azmcp-commands.md file which uses 'Namespace-level Tools'. Consider using consistent naming across documentation files.
**Default Mode (Namespace)** - namespace-level tools (default behavior):

Copy link
Member

@joshfree joshfree left a comment

Choose a reason for hiding this comment

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

Left some consistency suggestions

@Copilot Copilot AI requested a review from joshfree July 30, 2025 21:12
Copilot finished work on behalf of joshfree July 30, 2025 21:12
@Copilot Copilot AI requested a review from wbreza July 31, 2025 17:14
Copilot finished work on behalf of wbreza July 31, 2025 17:14
@Copilot Copilot AI requested a review from wbreza July 31, 2025 17:18
Copilot finished work on behalf of wbreza July 31, 2025 17:18
@github-project-automation github-project-automation bot moved this from Untriaged to In Progress in Azure MCP Server (OLD) Aug 4, 2025
@jongio
Copy link
Member

jongio commented Aug 25, 2025

Sorry for the inconvenience, but we have moved the Azure MCP Server source code to https://github.com/microsoft/mcp. This change allows us to build any Microsoft MCP server with the same engineering system and allows us to brand any MCP server with either Azure or Microsoft branding. This repo change means that you need to move this PR to that new repo. Please do so and then add a link here to that new PR. We'll close this PR now and we look forward to seeing this over on the new repo.

@jongio jongio closed this Aug 25, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in Azure MCP Server (OLD) Aug 25, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Documentation is out of sync from code
6 participants