Skip to content

fix: add azure-cloud-migrate to azure-prepare SKILL.md Step 0 routing table#1096

Merged
kvenkatrajan merged 2 commits intofix/skill-invocation-claude45-v2from
copilot/sub-pr-1080-again
Mar 2, 2026
Merged

fix: add azure-cloud-migrate to azure-prepare SKILL.md Step 0 routing table#1096
kvenkatrajan merged 2 commits intofix/skill-invocation-claude45-v2from
copilot/sub-pr-1080-again

Conversation

Copy link
Contributor

Copilot AI commented Mar 2, 2026

SKILL.md's inline Step 0 routing table was missing the azure-cloud-migrate row that exists in specialized-routing.md, creating a documentation inconsistency — the description text correctly excluded cross-cloud migration but the table didn't route it anywhere.

Changes

  • plugin/skills/azure-prepare/SKILL.md: Added azure-cloud-migrate as the first row in the Step 0 routing table, matching its Priority 1 position in specialized-routing.md
Prompt keywords Invoke FIRST
Lambda, AWS Lambda, migrate AWS, migrate GCP, Lambda to Functions, migrate from AWS, migrate from GCP azure-cloud-migrate
copilot SDK, copilot app, ... azure-hosted-copilot-sdk
... ...

Both routing documents now consistently direct Lambda/AWS/GCP migration prompts to azure-cloud-migrate before any other skill check.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: kvenkatrajan <102772054+kvenkatrajan@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback on skill routing improvements for Claude 4.5 fix: add azure-cloud-migrate to azure-prepare SKILL.md Step 0 routing table Mar 2, 2026
@kvenkatrajan kvenkatrajan marked this pull request as ready for review March 2, 2026 22:19
Copilot AI review requested due to automatic review settings March 2, 2026 22:19
@kvenkatrajan kvenkatrajan merged commit b07f85f into fix/skill-invocation-claude45-v2 Mar 2, 2026
4 checks passed
@kvenkatrajan kvenkatrajan deleted the copilot/sub-pr-1080-again branch March 2, 2026 22:19
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a documentation inconsistency in plugin/skills/azure-prepare/SKILL.md where the inline Step 0 routing table was missing the azure-cloud-migrate row, even though the reference file specialized-routing.md listed it as the highest-priority (Priority 1) routing entry. The change ensures the inline summary table matches the authoritative routing reference.

Changes:

  • Added azure-cloud-migrate as the first row in the Step 0 routing table in SKILL.md, with keywords (Lambda, AWS Lambda, migrate AWS, migrate GCP, Lambda to Functions, migrate from AWS, migrate from GCP) exactly matching those in specialized-routing.md.

kvenkatrajan added a commit that referenced this pull request Mar 3, 2026
* Improve skill descriptions for Claude 4.5 routing

- azure-deploy: capability-claim description ('This skill runs azd up...'),
  narrowed triggers to already-prepared apps, added scope rule
- azure-prepare: added 'Preparation ONLY' scope boundary, rule 8 for
  scope enforcement, handoff marker in Phase 2
- azure-validate: added handoff language to azure-deploy in description
- Tests: updated 2 deploy prompts with anti-routing signals
- Updated snapshots for all 3 skills

* Reduce RUNS_PER_PROMPT from 5 to 1 for skill-invocation tests

CI runs 3x/day across scheduled runs, providing statistical signal over time.
Running each prompt 5x per CI job was redundant cost with no added reliability.

* fix: explicitly use CLI foundation system prompt in SDK sessions

Set systemMessage to { mode: 'append' } by default instead of undefined,
ensuring the Copilot CLI built-in system prompt is always included in
integration test sessions.

* fix: append skill-preference system prompt to CLI foundation

Appends 'When a relevant skill is available, prefer using it instead of doing the task manually.' to the CLI system prompt via mode: append, nudging the model to invoke skills rather than executing tasks directly.

* fix: remove cross-skill keyword pollution from frontmatter descriptions

- azure-prepare: Remove handoff sentence ('After preparation, hand off to
  azure-validate then azure-deploy') and 'MUST be invoked FIRST' sentence
  that injected azure-deploy/azure-validate as competing keywords.
  Add explicit WHEN triggers for Terraform, App Service, Container Apps,
  Static Web Apps, and brownfield patterns.
- azure-deploy: Remove 'provision infrastructure' competing trigger,
  add DO NOT USE WHEN guidance for create/build patterns.

* fix: add terraform plan/apply triggers and update snapshots

* chore: strengthen azure-prepare triggers

* test: add yolo mode for skill-invocation

* restore: azure-cloud-migrate files

* fix: validate frontmatter for multiple skills

* restore: revert unintended deletions

* chore: update waza registry URLs

* test: update azure-validate trigger snapshots

* test: default agent system prompt

* feat: tighten azure-prepare triggers

* feat: add managed identity/key vault triggers

* feat: expand azure-prepare trigger phrases

* feat: update azure-prepare triggers and snapshots

* chore: trim azure-prepare description and refresh snapshots

* chore: add license and metadata to skills

* Update plugin/skills/azure-prepare/SKILL.md

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

* Update plugin/skills/azure-deploy/SKILL.md

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

* fix: replace ⛔ with ⚠️ in azure-prepare SKILL.md (#1095)

* Initial plan

* fix: replace ⛔ emoji with ⚠️ in azure-prepare SKILL.md per authoring guidelines

Co-authored-by: kvenkatrajan <102772054+kvenkatrajan@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: kvenkatrajan <102772054+kvenkatrajan@users.noreply.github.com>

* fix: add azure-cloud-migrate to azure-prepare SKILL.md Step 0 routing table (#1096)

* Initial plan

* fix: add azure-cloud-migrate to SKILL.md Step 0 routing table

Co-authored-by: kvenkatrajan <102772054+kvenkatrajan@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: kvenkatrajan <102772054+kvenkatrajan@users.noreply.github.com>

* feat: add azure-prepare trigger phrases

* chore: bump azure-deploy version

* Replace disallowed ⛔ emoji with ❌ in azure-prepare SKILL.md (#1103)

* Initial plan

* Replace disallowed ⛔ emoji with ❌ in azure-prepare SKILL.md

Co-authored-by: kvenkatrajan <102772054+kvenkatrajan@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: kvenkatrajan <102772054+kvenkatrajan@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
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.

3 participants