Closed
Conversation
️✔️AzureCLI-FullTest
|
️✔️AzureCLI-BreakingChangeTest
|
Collaborator
|
Thank you for your contribution! We will review the pull request and get back to you soon. |
|
The git hooks are available for azure-cli and azure-cli-extensions repos. They could help you run required checks before creating the PR. Please sync the latest code with latest dev branch (for azure-cli) or main branch (for azure-cli-extensions). pip install azdev --upgrade
azdev setup -c <your azure-cli repo path> -r <your azure-cli-extensions repo path>
|
Update template spec operations to extract and use the subscription ID from the resource ID if provided. Add tests to verify correct handling of subscription IDs in show, update, delete, and export commands for template specs.
Deleted test cases and YAML recordings related to template spec operations using a fake subscription ID. These tests were unnecessary as the functionality is already tested in other tests.
Removed redundant subscription_id variable assignments in template spec-related functions. Now, subscription_id is passed directly from parsed resource ID when template_spec is provided, improving code clarity and reducing duplication.
Introduces the 'tags' argument to SQL server creation commands in _params.py. Adds a new test and recording to verify server creation with tags in test_sql_commands.py and corresponding YAML recording.
Refreshed test recording for SQL server creation with tags to reflect new API versions, updated timestamps, and changed response metadata. Ensures test compatibility with latest Azure CLI and service API updates.
Updated devcontainer setup scripts for improved repo management and environment initialization. Added helpers, models, and refactored main entry for aaz-flow to support interactive Azure CLI code generation workflows. Updated dependencies and configuration for fastmcp and related packages.
Introduces a new workflow that generates tests for newly created Azure CLI modules/extensions using LLM sampling. Updates README to reflect new features, adds the generate_tests function to helpers.py, and integrates test generation into the main code generation flow in main.py.
a0x1ab
force-pushed
the
aaz-mcp-server
branch
from
90271e5 to
087622f
Compare
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Contributor
|
Hi @a0x1ab |
Contributor
|
Could you please narrow down the changes to include only what is necessary? |
necusjz
reviewed
Sep 9, 2025
| "storage": "128gb" | ||
| } | ||
| } | ||
| "workspaceFolder": "/workspaces/azure-cli", |
Member
There was a problem hiding this comment.
the 4 folders (azure-cli, azure-cli-extensions, azure-rest-api-specs，aaz) should at the same level. why do we need to change it?
Member
Author
There was a problem hiding this comment.
So the previous configuration of the devcontainer does not work on my machine - the workspace configuration and host requirements both fail and need manual intervention to work. I have moved the workspace configuration to the init script and it is now done automatically.
Contributor
There was a problem hiding this comment.
What environment did you test the changes? You can create a devcontainer and test in it.
Member
There was a problem hiding this comment.
perhaps we need to follow the same level structure. although our devcontainer is in azure-cli, users usually contribute code to azure-cli-extensions and aaz, which belong to the same abstraction layer.
necusjz
reviewed
Sep 9, 2025
yanzhudd
reviewed
Sep 9, 2025
necusjz
reviewed
Sep 9, 2025
Eliminates the mcp package from dependencies and updates fastmcp to version 2.12.2. Also modifies service fetching logic in helpers.py to use local directories instead of GitHub API, improving reliability in offline scenarios.
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
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.
Related command
NA
Description
This PR consists of several changes for the internally used AAZ MCP server to assist 1P teams to generate CLI cmdlets using AAZ through the VSCode Copilot Chat.
Testing Guide
NA
The PR title and description has followed the guideline in Submitting Pull Requests.
I adhere to the Command Guidelines.
I adhere to the Error Handling Guidelines.