Updating new-command documentation to reflect new repo structure #63
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jairmyree
requested review from
JasonYeMSFT,
JonathanCrd,
alzimmermsft,
sandeep-sen,
tmeschter,
wbreza and
xiangyan99
There was a problem hiding this comment.
Pull Request Overview
This PR updates the new command documentation to reflect the migrated repository structure, specifically changing references from the old areas/{area-name} directory structure to the new tools/Azure.Mcp.Tools.{AreaName} structure and updating project references from AzureMcp.Cli to Azure.Mcp.Server.
Key Changes:
- Updates directory paths from
areas/{area-name}totools/Azure.Mcp.Tools.{AreaName}throughout documentation - Changes project references from
AzureMcp.ClitoAzure.Mcp.Server - Updates namespace and class naming conventions from
AzureMcp.{Area}toAzure.Mcp.Tools.{AreaName}
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| eng/scripts/Test-Code.ps1 | Updates project paths and pattern matching for the new repository structure |
| eng/scripts/Deploy-TestResources.ps1 | Changes test resources directory path to use new tools structure |
| docs/new-command.md | Comprehensive documentation update reflecting all path, namespace, and project reference changes |
|
Due to this PR: #68, I'll remove updates to the test-deployment within this PR. |
|
@hallipr and I spoke about what to rename "Areas" to within the documentation. In the repo directory "areas" has been replaced with "Tools", but since we also refer to MCP commands as individual tools this creates ambiguity in documentation. So far, we have decided to go with "toolset" in documentation that was previously referring to Areas. |
…ture and documentation change of Area name to Toolset
jairmyree
force-pushed
the
update-new_command-doc
branch
from
dd0394c to
2092023
Compare
alzimmermsft
reviewed
Sep 2, 2025
hallipr
reviewed
Sep 2, 2025
hallipr
reviewed
Sep 2, 2025
hallipr
reviewed
Sep 2, 2025
Co-authored-by: Alan Zimmer <[email protected]>
Co-authored-by: Alan Zimmer <[email protected]>
Co-authored-by: Patrick Hallisey <[email protected]>
Co-authored-by: Alan Zimmer <[email protected]>
alzimmermsft
approved these changes
Sep 2, 2025
feiskyer
pushed a commit
to feiskyer/microsoft-mcp
that referenced
this pull request
Sep 8, 2025
colbytimm
pushed a commit
to colbytimm/microsoft-mcp
that referenced
this pull request
Sep 27, 2025
…rosoft#63) * Updating new-command.md and Program.cs to reflect new directory structure and documentation change of Area name to Toolset * Updating cspell.json * Update docs/new-command.md Co-authored-by: Alan Zimmer <[email protected]> * Update docs/new-command.md Co-authored-by: Alan Zimmer <[email protected]> * Update docs/new-command.md Co-authored-by: Patrick Hallisey <[email protected]> * Update docs/new-command.md Co-authored-by: Alan Zimmer <[email protected]> * Updating new-command and removing changes to Program.cs --------- Co-authored-by: Alan Zimmer <[email protected]> Co-authored-by: Patrick Hallisey <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR updates the new command documentation to reflect the migrated repo structure. This PR is not a comprehensive documentation update and other references to the old structure need to be updated as well. This PR is to unblock using the new-command.md to create new tools.