Skip to content

Add Azure Local skills - #2163

Open
Tim Frankland (tfrankland2) wants to merge 14 commits into
microsoft:mainfrom
tfrankland2:feature/azure-local-skill
Open

Add Azure Local skills#2163
Tim Frankland (tfrankland2) wants to merge 14 commits into
microsoft:mainfrom
tfrankland2:feature/azure-local-skill

Conversation

@tfrankland2

@tfrankland2 Tim Frankland (tfrankland2) commented May 5, 2026

Copy link
Copy Markdown

Summary

  • add a dedicated Azure Local skill with workflows and references for deployment, operations, workloads, networking/security, updates, and troubleshooting
  • register the new skill in skill test coverage
  • update README coverage for Azure Local scenarios

Fixes #2164

Validation

  • npm install
  • npm run build
  • npm run tokens check (existing repo baseline still reports 90 over-limit files; new azure-local/SKILL.md is under the limit)
  • npm run build && scripts reference validation
  • tests unit suite
  • git diff --check

@tfrankland2

Tim Frankland (tfrankland2) commented May 5, 2026

Copy link
Copy Markdown
Author

Tim Frankland (@tfrankland2) please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.

@microsoft-github-policy-service agree [company="{your company}"]

Options:

  • (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
@microsoft-github-policy-service agree
  • (when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.
@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree company="Microsoft"

@tfrankland2 Tim Frankland (tfrankland2) changed the title Add Azure Local skill Add Azure Local skills May 5, 2026
@tfrankland2
Tim Frankland (tfrankland2) marked this pull request as ready for review May 5, 2026 15:36
Copilot AI review requested due to automatic review settings May 5, 2026 15:36

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

Adds a new azure-local skill to the Azure skills plugin, providing Azure Local (Azure Stack HCI) deployment/operations/workload/security/update/troubleshooting guidance, and wires it into the scheduled integration test skill list.

Changes:

  • Added a new plugin/skills/azure-local/ skill with core routing plus workflow and reference markdown.
  • Registered azure-local in tests/skills.json and included it in the integration schedule.
  • Updated plugin/README.md to mention Azure Local under Compute & Containers.

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
tests/skills.json Adds azure-local to the skill registry and scheduled integration run list.
plugin/skills/azure-local/SKILL.md Introduces the Azure Local skill frontmatter + top-level routing/rules/refs.
plugin/skills/azure-local/version.json Adds per-skill NBGV version configuration.
plugin/skills/azure-local/workflows/plan-and-deploy/plan-and-deploy.md Deployment planning/intake and guarded deployment sequencing.
plugin/skills/azure-local/workflows/operate-and-update/operate-and-update.md Ops + lifecycle update flow, including update safety guidance and inventory patterns.
plugin/skills/azure-local/workflows/workload-management/workload-management.md VM/AKS hybrid/workload management routing and safety checks.
plugin/skills/azure-local/workflows/networking-and-security/networking-and-security.md Networking + security flows, SDN routing, and guardrails.
plugin/skills/azure-local/workflows/troubleshooting/troubleshooting.md Triage flow, scenario routing, evidence checklist, and safety rules for remediation.
plugin/skills/azure-local/references/docs-map.md Curated Microsoft Learn URL map for version-aware Azure Local documentation.
plugin/skills/azure-local/references/mcp-and-cli-tools.md Guidance on MCP/ARG-first discovery plus safe CLI/PowerShell usage boundaries.
plugin/skills/azure-local/references/resource-types.md Azure Resource Graph resource type patterns and baseline KQL inventory queries.
plugin/skills/azure-local/references/safety-rules.md Centralized “ask before doing” and high-risk component handling rules.
plugin/README.md Mentions Azure Local in the “Available Tools” coverage list.

Comment thread tests/skills.json
Comment thread plugin/skills/azure-local/SKILL.md Outdated
Comment thread plugin/skills/azure-local/references/docs-map.md Outdated

@jongio Jon Gallant (jongio) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Clean new skill addition for Azure Local with good workflow separation and safety guardrails. A few suggestions to bring the SKILL.md into full compliance with the repo's skill-files.instructions.md.

Comment thread plugin/skills/azure-local/SKILL.md
Comment thread plugin/skills/azure-local/SKILL.md Outdated
Comment thread plugin/skills/azure-local/SKILL.md
Comment thread plugin/skills/azure-local/references/docs-map.md Outdated

@jongio Jon Gallant (jongio) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Addresses my previous feedback. The SKILL.md sections now follow the repo's table format conventions and docs-map.md uses latest-version URLs by default. Christian Esqueda (@cesquedamsft) ready for your final approval.

@cesquedamsft

Copy link
Copy Markdown
Member

Looks good to me. Approved.

Comment thread plugin/README.md Outdated
@jongio

Copy link
Copy Markdown
Collaborator

Tim Frankland (@tfrankland2) this has merge conflicts now - can you rebase on main? All my prior feedback is addressed and CI is green, so just the conflict left to sort out.

Tim Frankland (tfrankland2) and others added 11 commits May 13, 2026 15:39
Add a dedicated Azure Local skill with workflows and references for deployment, operations, workloads, networking, security, updates, and troubleshooting.

Update adjacent Azure skills to route Azure Local, Arc VM, AKS hybrid, and lifecycle scenarios to the new skill.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Add azure-local to the skill test schedule, update affected trigger snapshots, and remove cross-skill relative markdown links that escaped skill directories.

Shorten the azure-upgrade description so its metadata unit test remains within the existing word limit.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Remove extra plugin manifest keywords and avoid treating custom locations as an Azure Local-owned routing signal in adjacent skills.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Remove the azure-upgrade routing change and clarify that Azure MCP support for Azure Local is partial through generic Azure surfaces rather than a dedicated Azure Local tool namespace.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Keep the Azure Local PR focused on adding the new skill and required registration, without modifying existing skill routing.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Align Azure Local Resource Graph examples with existing resource lookup guidance, which describes ARG as a KQL subset.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Address PR review feedback by adding Azure Local unit, trigger, and integration tests, standardizing the top-level skill sections, and keeping the skill entrypoint under the token limit.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Use an inline quoted description so the skill passes the frontmatter validator used by CI.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@tfrankland2

Copy link
Copy Markdown
Author

Oops sorry if you got tagged for review. Forced pushed some changes by accident while rebasing.

Should be fixed now Jon Gallant (@jongio) JasonYeMSFT (@JasonYeMSFT)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

@jongio Jon Gallant (jongio) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

All prior feedback addressed - tables for Quick Reference/MCP Tools/Error Handling, version pinning removed from docs-map, README wording reverted. Skill structure looks solid with good progressive disclosure across references and workflows. Safety rules are thorough.

Only remaining blocker is the merge conflict from the rebase - once that's resolved this is good to go.

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.

Add Azure Local skill coverage

4 participants