Add extension cli generate command #461
Open
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
JasonYeMSFT
force-pushed
the
chuye/cligenerate-command
branch
from
376474e to
191a74e
Compare
JasonYeMSFT
commented
Sep 16, 2025
| var accessToken = await cliGenerateService.GetAzCliGenerateTokenAsync(); | ||
|
|
||
| // GHCP4A Prod app | ||
| const string url = "https://aiservice.ghcpaz-prod.azure.com/api/azurecli/generate"; |
There was a problem hiding this comment.
This API hasn't been deployed to Prod yet. The implementation has been merged and will be deployed soon.
JasonYeMSFT
force-pushed
the
chuye/cligenerate-command
branch
from
8873f2b to
5ddc234
Compare
JasonYeMSFT
requested review from
Copilot,
tmeschter,
xiangyan99,
JonathanCrd,
sandeep-sen and
jairmyree
There was a problem hiding this comment.
Pull Request Overview
This PR adds a new cligenerate command to the Azure MCP Server that generates Azure CLI commands using an LLM-powered service. The command takes user intent as input and produces validated Azure CLI commands through an external API service.
Key changes:
- Implements a new CLI generation service that communicates with an external Azure API endpoint
- Adds comprehensive command structure with options validation and error handling
- Provides unit tests to ensure the command functionality works correctly
Reviewed Changes
Copilot reviewed 16 out of 16 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| CliGenerateCommand.cs | Main command implementation with validation and Azure CLI generation logic |
| ICliGenerateService.cs | Service interface for CLI generation operations |
| CliGenerateService.cs | Service implementation that handles HTTP requests to external API |
| ExtensionOptionDefinitions.cs | Command option definitions for intent and CLI type parameters |
| CliGenerateOptions.cs | Options model for binding command line arguments |
| CliGenerateResult.cs | Result model for returning generated CLI commands |
| CliGenerateRequest.cs | Request models for API communication |
| ExtensionSetup.cs | Service registration and command setup |
| Constants.cs | Azure CLI type constant definition |
| ExtensionJsonContext.cs | JSON serialization context updates |
| Azure.Mcp.Tools.Extension.csproj | Project file cleanup |
| README.md | Documentation updates |
| CHANGELOG.md | Feature addition notes |
| e2eTestPrompts.md | Test prompts for the new functionality |
| azmcp-commands.md | Command documentation |
| CliGenerateCommandTests.cs | Unit tests for the CLI generate command |
tools/Azure.Mcp.Tools.Extension/src/Options/CliGenerateOptions.cs
Outdated
Show resolved
Hide resolved
tools/Azure.Mcp.Tools.Extension/src/Models/CliGenerateRequest.cs
Outdated
Show resolved
Hide resolved
tools/Azure.Mcp.Tools.Extension/src/Models/CliGenerateRequest.cs
Outdated
Show resolved
Hide resolved
tools/Azure.Mcp.Tools.Extension/src/Commands/CliGenerateCommand.cs
Outdated
Show resolved
Hide resolved
tools/Azure.Mcp.Tools.Extension/src/Commands/CliGenerateCommand.cs
Outdated
Show resolved
Hide resolved
JasonYeMSFT
commented
Sep 17, 2025
JasonYeMSFT
changed the title
jongio
requested changes
Sep 17, 2025
tools/Azure.Mcp.Tools.Extension/src/Commands/CliGenerateCommand.cs
Outdated
Show resolved
Hide resolved
tools/Azure.Mcp.Tools.Extension/src/Commands/CliGenerateCommand.cs
Outdated
Show resolved
Hide resolved
JasonYeMSFT
force-pushed
the
chuye/cligenerate-command
branch
from
1761780 to
4842872
Compare
JasonYeMSFT
force-pushed
the
chuye/cligenerate-command
branch
from
ca2f40d to
666d803
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
What does this PR do?
Add a new cligenerate command that can generate Azure Cli commands using a LLM powered services. This service can produce higher quality commands than what the LLM can generate on its own and guarantees the returned commands are valid through validation.
GitHub issue number?
#203
Pre-merge Checklist
servers/Azure.Mcp.Server/CHANGELOG.mdand/orservers/Fabric.Mcp.Server/CHANGELOG.mdfor product changes (features, bug fixes, UI/UX, updated dependencies)servers/Azure.Mcp.Server/README.mdand/orservers/Fabric.Mcp.Server/README.mddocumentation/docs/azmcp-commands.mdand/or/docs/fabric-commands.mdToolDescriptionEvaluatorand obtained a score of0.4or more and a top 3 ranking for all related test prompts/docs/e2eTestPrompts.mdcrypto mining, spam, data exfiltration, etc.)/azp run mcp - pullrequest - liveto run Live Test Pipeline