feature: Feature/azure compute/enablemachinemanagement by rakal-dyh · Pull Request #2069 · microsoft/GitHub-Copilot-for-Azure

rakal-dyh · 2026-04-27T22:17:07Z

Description

Checklist

[Y] Tests pass locally (cd tests && npm test)
[Y] If modifying skill descriptions: verified routing correctness with integration tests (npm run test:skills:integration -- <skill>)
[Y] If modifying skill USE FOR / DO NOT USE FOR / PREFER OVER clauses: confirmed no routing regressions for competing skills
[Y] Version bumped in skill frontmatter (if skill files changed)

Related Issues

- Add 'Should Trigger - Essential Machine Management' section with 7 EMM prompts - Add 'Essential Machine Management Workflow' unit tests (sections, routing, API, errors) - Add 'EMM Reference Files' unit tests (all 4 refs exist, content assertions) - Update router test to verify EMM description keywords and routing - Add 2 EMM integration test cases (enable flow, enrollment status) - Update snapshot with new EMM keywords Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…r-Azure into feature/azure-compute/enablemachinemanagement

rakal-dyh · 2026-04-27T22:28:14Z

verify the integration test now

Tom Meschter (tmeschter) · 2026-04-27T22:53:55Z

Update the PR title to start with "feature:" to have this automatically included in the CHANGELOG.md.

Jon Gallant (jongio)

Two things to address in this EMM sub-workflow addition:

1. Contradictory "Portal-only" statement (emm-overview.md, Key Characteristics section)

The overview states:

Portal-only during preview: No CLI/API-only enrollment flow is officially supported yet

But emm-enable-flow.md documents a complete 8-step Copilot-guided flow using REST API calls (PUT to Microsoft.ManagedOps/managedOps/default). If the agent reads the overview first, it'll tell users EMM is portal-only and skip the API flow - which is the whole point of this skill.

Suggestion: rephrase to clarify that the REST API exists and the Copilot-guided flow uses it, even if Microsoft's official docs focus on the portal experience.

2. Offboard instructions split with different approaches

essential-machine-management.md says offboard via portal UI (navigate, select, click Offboard). emm-enable-flow.md documents a DELETE API call for the same action. The parent workflow's routing doesn't point to the API-based offboard path. Consider consolidating - either add the API path to the parent routing tree, or remove the duplicate from the enable-flow reference.

…lability The 'portal-only' statement contradicted emm-enable-flow.md which documents a full Copilot-guided REST API enrollment flow via Microsoft.ManagedOps. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- Remove 'Via Portal' sub-section from Browse (keep Copilot-guided API approach) - Point Offboard to emm-enable-flow.md#disable-emm-offboard instead of portal steps - Keep portal routing entry for users who explicitly mention portal Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Update all references in SKILL.md, essential-machine-management.md, and unit tests. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

The #disable-emm-offboard anchor doesn't work when the agent reads files. Reference the file directly and mention the section name in text. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Jon Gallant (jongio)

Addresses my previous feedback - the "portal-only" contradiction is fixed and offboard instructions are consolidated. Clean changes.

Two new issues in the latest commits:

integration.test.ts: The describe block was renamed from "skill-invocation" to "11skill-invocation11". This looks like accidental debug characters left in. It'll affect test output/filtering.
.playwright-mcp/page-2026-04-20T21-13-59-247Z.yml was committed - looks like an accidental Playwright MCP artifact. Should be removed and probably added to .gitignore.

Jon Gallant (jongio) · 2026-04-28T14:48:43Z

.playwright-mcp/page-2026-04-20T21-13-59-247Z.yml was committed - looks like an accidental Playwright MCP artifact. Should be removed and .playwright-mcp/ added to .gitignore.

rakal-dyh · 2026-04-28T22:34:12Z

adress the latest comment, remove the mistakenly committed file

Jon Gallant (jongio)

Addresses my previous feedback. Both items fixed:

"11skill-invocation11" reverted to "skill-invocation"
.playwright-mcp/ artifact removed

No new issues found in the incremental diff. All CI green.

…r-Azure into feature/azure-compute/enablemachinemanagement

Copilot

Pull request overview

Adds Essential Machine Management (EMM) coverage to the azure-compute skill by introducing a new EMM workflow + reference docs and extending unit/trigger/integration tests to validate routing and keyword matching.

Changes:

Added an EMM workflow (essential-machine-management.md) with routing, browse/offboard guidance, and error handling.
Added EMM reference docs (overview, prerequisites, enable flow, portal enable flow).
Updated azure-compute skill metadata/routing and extended tests + snapshots to cover EMM prompts and routing.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
tests/azure-compute/unit.test.ts	Adds unit assertions for EMM description keywords, routing link presence, and validates new workflow/reference file content.
tests/azure-compute/triggers.test.ts	Adds trigger prompts ensuring EMM queries trigger the `azure-compute` skill.
tests/azure-compute/integration.test.ts	Adds integration prompts to verify EMM queries route to the new workflow at/above the invocation threshold.
tests/azure-compute/snapshots/triggers.test.ts.snap	Updates snapshot to include EMM in description and extracted keywords.
plugin/skills/azure-compute/workflows/essential-machine-management/essential-machine-management.md	New workflow router for EMM intents (enable/portal/overview/prereqs/browse/offboard/troubleshoot).
plugin/skills/azure-compute/workflows/essential-machine-management/references/emm-enable-flow.md	New Copilot-guided enablement flow and API/CLI guidance for enrolling/offboarding EMM.
plugin/skills/azure-compute/workflows/essential-machine-management/references/emm-enable-flow-portal-guidance.md	New portal-specific enablement walkthrough for users explicitly requesting portal steps.
plugin/skills/azure-compute/workflows/essential-machine-management/references/emm-overview.md	New EMM feature/tier/pricing overview reference.
plugin/skills/azure-compute/workflows/essential-machine-management/references/emm-prerequisites.md	New prerequisites (resources, roles, RPs) reference.
plugin/skills/azure-compute/SKILL.md	Extends skill description + routing to include EMM and link to the new workflow and references.

Jon Gallant (jongio)

Latest commits address the Copilot bot findings:

MCP tool names fixed to mcp_azure_mcp_subscription_list (both router and enable-flow)
azure-monitor MCP tool refs replaced with explicit CLI commands for Log Analytics and Azure Monitor workspaces
Role check tip and 403 error handling now consistent - both correctly explain that assignedTo is self-scoped but requires at least one role on the subscription

All prior review items resolved. CI green. No new issues.

…r-Azure into feature/azure-compute/enablemachinemanagement

Yinghui Dong and others added 3 commits April 21, 2026 13:57

emm subskill

e1ee149

Merge branch 'main' of https://github.com/microsoft/GitHub-Copilot-fo…

8e1d90e

…r-Azure into feature/azure-compute/enablemachinemanagement

rakal-dyh requested review from Alex Thompson (alex-thompson), Jon Gallant (jongio) and joybb as code owners April 27, 2026 22:17

update

d07b990

Tom Meschter (tmeschter) previously approved these changes Apr 27, 2026

View reviewed changes

rakal-dyh changed the title ~~Feature/azure compute/enablemachinemanagement~~ feature: Feature/azure compute/enablemachinemanagement Apr 27, 2026

Jon Gallant (jongio) reviewed Apr 27, 2026

View reviewed changes

Comment thread plugin/skills/azure-compute/workflows/essential-machine-management/references/emm-overview.md Outdated

Yinghui Dong and others added 3 commits April 27, 2026 17:07

rename: emm-enable-flow-portal.md → emm-enable-flow-portal-guidance.md

526977d

Update all references in SKILL.md, essential-machine-management.md, and unit tests. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

rakal-dyh dismissed Tom Meschter (tmeschter)’s stale review via 526977d April 28, 2026 00:23

fix(azure-compute): remove invalid anchor from offboard reference

4e2e759

The #disable-emm-offboard anchor doesn't work when the agent reads files. Reference the file directly and mention the section name in text. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Jon Gallant (jongio) reviewed Apr 28, 2026

View reviewed changes

Comment thread tests/azure-compute/integration.test.ts Outdated

Yinghui Dong added 2 commits April 28, 2026 15:30

remove unrelated change

95e9e93

revert local change

90591a3

Jon Gallant (jongio) reviewed Apr 28, 2026

View reviewed changes

Merge branch 'main' of https://github.com/microsoft/GitHub-Copilot-fo…

7a07d7e

…r-Azure into feature/azure-compute/enablemachinemanagement

Copilot AI review requested due to automatic review settings April 30, 2026 05:08

Copilot started reviewing on behalf of rakal-dyh April 30, 2026 05:08 View session

Copilot AI reviewed Apr 30, 2026

View reviewed changes

Yinghui Dong added 3 commits April 29, 2026 22:21

update emm mcp tool name mapping

1fa2c64

update mcp to clo guide

5f57873

update error handling for user role check

d719a0b

update tips for user roll query

82a7dba

Jon Gallant (jongio) reviewed Apr 30, 2026

View reviewed changes

Yinghui Dong added 3 commits April 30, 2026 14:07

Merge branch 'main' of https://github.com/microsoft/GitHub-Copilot-fo…

40db451

…r-Azure into feature/azure-compute/enablemachinemanagement

Merge branch 'main' of https://github.com/microsoft/GitHub-Copilot-fo…

d877761

…r-Azure into feature/azure-compute/enablemachinemanagement

remove user contributor role is good statement

2b79bf5

Alex Thompson (alex-thompson) approved these changes May 5, 2026

View reviewed changes

rakal-dyh merged commit 7612caa into microsoft:main May 5, 2026
12 checks passed

github-actions Bot mentioned this pull request May 8, 2026

[repo-status] Weekly Repo Status — May 2–8, 2026 #2201

Closed

Navigation Menu

Uh oh!

feature: Feature/azure compute/enablemachinemanagement - #2069

feature: Feature/azure compute/enablemachinemanagement#2069
rakal-dyh merged 18 commits into
microsoft:mainfrom
rakal-dyh:feature/azure-compute/enablemachinemanagement

rakal-dyh commented Apr 27, 2026 •

edited

Loading

Uh oh!

rakal-dyh commented Apr 27, 2026

Uh oh!

Tom Meschter (tmeschter) commented Apr 27, 2026

Uh oh!

Jon Gallant (jongio) left a comment

Uh oh!

Uh oh!

Jon Gallant (jongio) left a comment

Uh oh!

Uh oh!

Jon Gallant (jongio) commented Apr 28, 2026

Uh oh!

rakal-dyh commented Apr 28, 2026

Uh oh!

Jon Gallant (jongio) left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Jon Gallant (jongio) left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Uh oh!

Conversation

rakal-dyh commented Apr 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Checklist

Related Issues

Uh oh!

rakal-dyh commented Apr 27, 2026

Uh oh!

Tom Meschter (tmeschter) commented Apr 27, 2026

Uh oh!

Jon Gallant (jongio) left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Jon Gallant (jongio) left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Jon Gallant (jongio) commented Apr 28, 2026

Uh oh!

rakal-dyh commented Apr 28, 2026

Uh oh!

Jon Gallant (jongio) left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Jon Gallant (jongio) left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

rakal-dyh commented Apr 27, 2026 •

edited

Loading