You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
PR #3143 ("POCO-based options follow-ups") removed the WorkspaceFolder / --workspace-folder parameter from tools/Azure.Mcp.Tools.Deploy/src/Options/Plan/GetOptions.cs. The property and all test references to --workspace-folder were deleted, making --project-name the first (and only required) parameter.
Gaps Found
azmcp deploy plan get: --workspace-folder <workspace-folder> is still shown as the first required parameter in azmcp-commands.md (line ~2386), but the parameter no longer exists in the code.
Files to Update
servers/Azure.Mcp.Server/docs/azmcp-commands.md
Context
Current docs (lines ~2383–2393 of azmcp-commands.md):
# Get a deployment plan for a specific project
azmcp deploy plan get --workspace-folder <workspace-folder> \
--project-name <project-name> \
--target-app-service <target-app-service> \
--provisioning-tool <provisioning-tool> \
--source-type <source-type> \
[--iac-options <iac-options>] \
[--deploy-option <deploy-option>] \
[--resource-group <resource-group>] \
[--subscription <subscription>]
After the PR, GetOptions.cs only has: ProjectName (required), TargetAppService, ProvisioningTool, SourceType, DeployOption, IacOptions, ResourceGroup, Subscription. No WorkspaceFolder.
📐 Implementation Guide
This section contains step-by-step instructions for a coding agent to implement the changes described above.
Step 1: Modify files
File:servers/Azure.Mcp.Server/docs/azmcp-commands.md Action: Edit the ### Azure Deploy Operations section — remove --workspace-folder from the deploy plan get command signature.
Replace:
# Get a deployment plan for a specific project# ❌ Destructive | ✅ Idempotent | ❌ OpenWorld | ✅ ReadOnly | ❌ Secret | ❌ LocalRequired
azmcp deploy plan get --workspace-folder <workspace-folder> \
--project-name <project-name> \
--target-app-service <target-app-service> \
--provisioning-tool <provisioning-tool> \
--source-type <source-type> \
[--iac-options <iac-options>] \
[--deploy-option <deploy-option>] \
[--resource-group <resource-group>] \
[--subscription <subscription>]
With:
# Get a deployment plan for a specific project# ❌ Destructive | ✅ Idempotent | ❌ OpenWorld | ✅ ReadOnly | ❌ Secret | ❌ LocalRequired
azmcp deploy plan get --project-name <project-name> \
[--target-app-service <target-app-service>] \
[--provisioning-tool <provisioning-tool>] \
[--source-type <source-type>] \
[--iac-options <iac-options>] \
[--deploy-option <deploy-option>] \
[--resource-group <resource-group>] \
[--subscription <subscription>]
Note: --target-app-service, --provisioning-tool, and --source-type are string? (optional) in GetOptions.cs and should use bracket notation. This also supersedes issue #3116's fix for this command.
Step 2: Verify documentation structure
servers/Azure.Mcp.Server/docs/azmcp-commands.md — optional parameters use [--param <value>] notation, required parameters use bare --param <value>.
Step 3: Validate
dotnet build servers/Azure.Mcp.Server/ — confirms the server project compiles cleanly
dotnet build tools/Azure.Mcp.Tools.Deploy/src/ — confirms the affected toolset compiles
dotnet test tools/Azure.Mcp.Tools.Deploy/tests/Azure.Mcp.Tools.Deploy.Tests/ --filter "TestType!=Live" — runs unit tests
.\eng\common\spelling\Invoke-Cspell.ps1 — checks spelling in modified documentation
Next Steps
Tip
Ready for automated implementation? Assign this issue to @copilot to have Copilot coding agent implement the changes described in the Implementation Guide above
Documentation Gap
Server:
Azure.Mcp.ServerTool directory:
tools/Azure.Mcp.Tools.DeployTriggered by: commit
50309cc24695a3d641f48d53e9d44bb14d55c623(PR #3143) by@alzimmermsftChanged files:
tools/Azure.Mcp.Tools.Deploy/src/Options/Plan/GetOptions.cstools/Azure.Mcp.Tools.Deploy/tests/Azure.Mcp.Tools.Deploy.Tests/Commands/Plan/GetCommandTests.csWhat Changed
PR #3143 ("POCO-based options follow-ups") removed the
WorkspaceFolder/--workspace-folderparameter fromtools/Azure.Mcp.Tools.Deploy/src/Options/Plan/GetOptions.cs. The property and all test references to--workspace-folderwere deleted, making--project-namethe first (and only required) parameter.Gaps Found
azmcp deploy plan get:--workspace-folder <workspace-folder>is still shown as the first required parameter inazmcp-commands.md(line ~2386), but the parameter no longer exists in the code.Files to Update
servers/Azure.Mcp.Server/docs/azmcp-commands.mdContext
Current docs (lines ~2383–2393 of
azmcp-commands.md):After the PR,
GetOptions.csonly has:ProjectName(required),TargetAppService,ProvisioningTool,SourceType,DeployOption,IacOptions,ResourceGroup,Subscription. NoWorkspaceFolder.📐 Implementation Guide
This section contains step-by-step instructions for a coding agent to implement the changes described above.
Step 1: Modify files
File:
servers/Azure.Mcp.Server/docs/azmcp-commands.mdAction: Edit the
### Azure Deploy Operationssection — remove--workspace-folderfrom thedeploy plan getcommand signature.Replace:
With:
Step 2: Verify documentation structure
servers/Azure.Mcp.Server/docs/azmcp-commands.md— optional parameters use[--param <value>]notation, required parameters use bare--param <value>.Step 3: Validate
dotnet build servers/Azure.Mcp.Server/— confirms the server project compiles cleanlydotnet build tools/Azure.Mcp.Tools.Deploy/src/— confirms the affected toolset compilesdotnet test tools/Azure.Mcp.Tools.Deploy/tests/Azure.Mcp.Tools.Deploy.Tests/ --filter "TestType!=Live"— runs unit tests.\eng\common\spelling\Invoke-Cspell.ps1— checks spelling in modified documentationNext Steps
Tip
Ready for automated implementation? Assign this issue to
@copilotto have Copilot coding agent implement the changes described in the Implementation Guide above