Skip to content

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

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

Replace disallowed ⛔ emoji with ❌ in azure-prepare SKILL.md#1103
kvenkatrajan merged 2 commits intofix/skill-invocation-claude45-v2from
copilot/sub-pr-1080

Conversation

Copy link
Contributor

Copilot AI commented Mar 3, 2026

Skill authoring guidelines restrict emojis to status indicators only (✅, ❌, ⚠️), but plugin/skills/azure-prepare/SKILL.md used ⛔ in multiple places.

Changes

  • plugin/skills/azure-prepare/SKILL.md: Replaced all 5 occurrences of ⛔ with ❌ — covering Rule 7, two section headings (PLAN-FIRST WORKFLOW, STEP 0), one planning table cell, and one STOP HERE callout block.

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Co-authored-by: kvenkatrajan <102772054+kvenkatrajan@users.noreply.github.com>
Copilot AI changed the title [WIP] Update skill routing improvements based on feedback Replace disallowed ⛔ emoji with ❌ in azure-prepare SKILL.md Mar 3, 2026
@kvenkatrajan kvenkatrajan marked this pull request as ready for review March 3, 2026 00:27
Copilot AI review requested due to automatic review settings March 3, 2026 00:27
@kvenkatrajan kvenkatrajan merged commit c335be7 into fix/skill-invocation-claude45-v2 Mar 3, 2026
2 checks passed
@kvenkatrajan kvenkatrajan deleted the copilot/sub-pr-1080 branch March 3, 2026 00:27
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 updates the azure-prepare skill documentation to comply with the repository’s skill authoring guidelines by removing a disallowed emoji (⛔) and replacing it with an allowed status indicator (❌).

Changes:

  • Replaced 5 occurrences of ⛔ with ❌ in plugin/skills/azure-prepare/SKILL.md (rules list, section headings, planning table, and a STOP callout).

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