Skip to content

Sync plugin files from GitHub-Copilot-for-Azure (run 32783422210) - #415

Merged
JasonYeMSFT (JasonYeMSFT) merged 1 commit into
mainfrom
sync/copilot-for-azure-32783422210
Aug 24, 2026
Merged

Sync plugin files from GitHub-Copilot-for-Azure (run 32783422210)#415
JasonYeMSFT (JasonYeMSFT) merged 1 commit into
mainfrom
sync/copilot-for-azure-32783422210

Conversation

@ghcp4a-bot

@ghcp4a-bot ghcp4a-bot Bot commented Aug 24, 2026

Copy link
Copy Markdown

Automated sync of plugin/ from GitHub-Copilot-for-Azure at commit d1b5b04af31cb12866062e4914b12ca9dacb0c8b.

Copilot AI balanced review requested due to automatic review settings August 24, 2026 22:12
@JasonYeMSFT
JasonYeMSFT (JasonYeMSFT) merged commit 6a2bf7b into main Aug 24, 2026
4 checks passed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Syncs Azure plugin updates from the upstream repository.

Changes:

  • Adds referenced-workload planning and hardened IaC validation.
  • Adds plugin-version telemetry.
  • Bumps Azure plugin versions and changelog.

Reviewed changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
azure-enterprise-infra-planner/SKILL.md Documents the expanded planning pipeline.
references/workflow.md Adds referenced-mode and security gates.
references/terraform-generation.md Adds Terraform correctness checks.
references/schema.md Models existing-resource insights.
references/referenced-workload.md Defines brownfield integration behavior.
references/phases/6-generate-iac.md Hardens IaC generation and validation.
references/phases/1-extract-insights.md Merges existing resources into insights.
references/bicep-generation.md Adds Bicep correctness checks.
azure-skills/hooks/scripts/track-telemetry.sh Adds plugin-version telemetry.
azure-skills/hooks/scripts/track-telemetry.ps1 Adds plugin-version telemetry.
azure-skills/CHANGELOG.md Records release 1.2.34.
azure-skills/.plugin/plugin.json Bumps plugin version.
azure-skills/.cursor-plugin/plugin.json Bumps Cursor plugin version.
azure-skills/.claude-plugin/plugin.json Bumps Claude plugin version.
azure-kusto-graph-skills/hooks/scripts/track-telemetry.sh Updates telemetry paths and plugin version.
azure-kusto-graph-skills/hooks/scripts/track-telemetry.ps1 Updates telemetry paths and plugin version.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines +299 to +300
# Check if a path matches any known azure-skills folder structure
# Returns 0 (true) if matched, 1 (false) otherwise
- You must execute the seven phases in sequential order. Follow the instructions precisely as defined. Do not continue to the next phase until the current phase is complete.
- You must stop on all "gate" conditions and only continue when the conditions have been met.
- Destructive actions require explicit user confirmation.
- **Confirmation gate vs. answer-first.** Always present a plain-language summary of your understanding plus an explicit "confirm before I proceed" checkpoint *before deploying* (Phase 7). In referenced mode you still generate the plan and IaC in the same turn (answer-first) — the gate governs *deployment*, not whether you produce the artifacts. Never end a turn with only a question, and never deploy without an explicit, risk-acknowledged go-ahead.
Comment on lines +35 to +39
proceed to deploy"), and **then** generate the complete, deployable IaC in the SAME response using the
recommended default. Order within the turn: (1) summary of what exists + what you'll add, (2) explicit
"confirm before I proceed" gate, (3) the generated IaC, (4) the "which to incorporate?" choice and any
assumptions. Never end a turn with only a summary or a question, and never deploy before the user
confirms. If you lack a value (region, an existing resource's ID/name), **declare a parameter and
Comment on lines +40 to +44
- **Referenced (brownfield)** — the user supplies something that already exists (a live resource /
resource group / subscription, IaC or an infra plan, or a requirements doc). The same phases run, plus
[referenced-workload.md](references/referenced-workload.md): existing resources are inventoried and
referenced (never recreated), the new workload is wired into them, and **Phase 7 deploys additively**
(incremental only — never modifying or destroying the referenced resources).
Comment on lines +98 to +99
3. **Existing resources use `data`/`import`, not new `resource`.** Reference pre-existing infra via
`data` sources (or `import`), and wire new resources to them — never recreate them.
Comment on lines +67 to +69
P6 --> VAL{Validate<br/>az bicep build /<br/>terraform validate}
VAL -- "errors" --> P6
VAL -- "clean" --> P7{Phase 7 · Deploy<br/>risk-ack confirm?}
Comment on lines +292 to 295
# Azure-skills path patterns per client (used for SKILL.md and file-reference matching)
$pathPatternCopilot = '\.copilot/installed-plugins/[^/]+/azure/skills/'
$pathPatternClaude = '\.claude/plugins/cache/(azure-skills|claude-plugins-official)/azure/[0-9.]+/skills/'
$pathPatternVscodeAgentPlugins = 'agent-plugins/github\.com/microsoft/azure-skills/\.github/plugins/azure-skills/skills/'
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants