Conversation
There was a problem hiding this comment.
Pull request overview
This PR restructures the Azure deployment guidance by splitting “create/configure” from “deploy”, and updates reference documentation to standardize deployment workflows around azd.
Changes:
- Introduces a new
azure-create-appskill focused on discovery, architecture planning, file generation, and validation forazdprojects. - Simplifies
azure-deployskill to anazd-centric execution flow (env/subscription/location → deploy → troubleshoot). - Updates multiple Azure service reference guides to emphasize
azdworkflows and de-emphasize directazCLI usage.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| plugin/skills/azure-deploy/reference/static-web-apps.md | Updates SWA prerequisites and quick start to an azd-first workflow. |
| plugin/skills/azure-deploy/reference/functions.md | Reworks Functions deployment guidance to focus on azd + Bicep patterns. |
| plugin/skills/azure-deploy/reference/container-apps.md | Removes Azure CLI auth prereq and standardizes on azd auth checks. |
| plugin/skills/azure-deploy/reference/app-service.md | Shifts App Service guidance toward “Always use azd” and updates prerequisites. |
| plugin/skills/azure-deploy/reference/aks.md | Shifts AKS guidance toward “Always use azd”, updates prerequisites and kubectl install. |
| plugin/skills/azure-deploy/SKILL.md | Replaces the prior broad workflow with a 6-step azd-focused deployment flow. |
| plugin/skills/azure-create-app/SKILL.md | Adds a new skill for preparing an application for Azure (azure.yaml, infra/, validation). |
Comments suppressed due to low confidence (3)
plugin/skills/azure-deploy/reference/functions.md:392
$FUNCTION_APPis used as if it’s a predefined variable, but this guide no longer defines it anywhere (the previous az-based resource creation section that set it is gone). Add a short note defining what$FUNCTION_APPshould be (and how to obtain it when deploying with azd), or replace it with a consistent placeholder like<function-app-name>.
```bash
# Deploy to Azure (from project root)
func azure functionapp publish $FUNCTION_APP
plugin/skills/azure-deploy/reference/functions.md:657
- These two statements conflict: if
azd pipeline config“automatically creates the necessary secrets and workflow configuration”, there shouldn’t also be a manual step to “Add the output asAZURE_CREDENTIALS”. Clarify whatazd pipeline configgenerates vs what the user must still add manually (and keep it consistent with the workflow example above).
This automatically creates the necessary secrets and workflow configuration.
Add the output as `AZURE_CREDENTIALS` secret in GitHub repository settings.
plugin/skills/azure-deploy/SKILL.md:72
- The tool name
mcp_azure_mcp_subscription_listdoesn’t match the Azure MCP tool naming used elsewhere in this repo (e.g.,azure__subscription_listinplugin/skills/azure-mcp/SKILL.md:64,121). If this is meant to be the standard subscription listing tool, update the doc to use the correct tool name so the workflow is executable.
1. Call the `mcp_azure_mcp_subscription_list` tool to get available subscriptions:
```json
{
"tool": "mcp_azure_mcp_subscription_list",
"parameters": {}
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 7 out of 7 changed files in this pull request and generated 7 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
paulyuk
left a comment
There was a problem hiding this comment.
please see comments. I'm not sure this will work for services other than ACA
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 13 out of 13 changed files in this pull request and generated 2 comments.
Comments suppressed due to low confidence (4)
plugin/skills/azure-create-app/reference/functions.md:590
- This section reintroduces an
az functionapp ...command (slot swap) even though the guide positionsazdas the required workflow and no longer lists Azure CLI as a prerequisite. Either mark this Azure CLI step as legacy/optional and document the Azure CLI requirement (or a Portal-based alternative), or adjust earlier wording so it’s clear whenazis still needed.
plugin/skills/azure-create-app/reference/static-web-apps.md:39 - The command
curl -fsSL https://aka.ms/install-azd.sh | bashdownloads and executes a remote script without any integrity verification or version pinning, which introduces a supply chain risk. If theaka.msendpoint or its DNS/TLS channel is compromised, an attacker could execute arbitrary code in the developer or CI environment where this command is run, potentially accessing credentials or modifying build artifacts. Prefer installingazdvia a package manager or a pinned, checksum-verified artifact instead of piping a mutable remote script directly intobash.
plugin/skills/azure-create-app/reference/app-service.md:89 - The command
curl -fsSL https://aka.ms/install-azd.sh | bashdownloads and executes a remote script without checksum/signature verification or version pinning, which is a supply chain risk. If theaka.msendpoint or its DNS/TLS channel is compromised, an attacker could execute arbitrary code in the environment where this command is run, including CI agents or dev machines with access to secrets. Prefer installingazdvia a package manager or a pinned, integrity-checked artifact instead of piping a mutable remote script directly intobash.
plugin/skills/azure-create-app/reference/aks.md:92 - The Linux installation command
curl -fsSL https://aka.ms/install-azd.sh | bashintroduces a supply chain vulnerability by executing a remote script without integrity verification or version pinning. If that URL or its hosting infrastructure were compromised, an attacker could run arbitrary code in developer or CI environments, gaining access to credentials or modifying Kubernetes-related build artifacts. Replace this pattern with an installation approach that uses a package manager or a pinned artifact with checksum/signature verification instead of piping a mutable remote script directly intobash.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary
Split
azure-deployskill into two focused skills and standardize onazdCLINew Skill:
azure-create-appmcp_azure_mcp_azdcommandsSimplified:
azure-deployazd config get defaultsto surface existing defaults in promptsazure-create-appskill when azure.yaml is missingReference Files Updated
azCLI commands, standardized onazdfor deploymentsaz functionappcommands, replaced with azd/Bicep patternsdistfolder requirement, replaced Quick Start with azd workflow