fix: replace ⛔ with ⚠️ in azure-prepare SKILL.md#1095
Merged
kvenkatrajan merged 2 commits intofix/skill-invocation-claude45-v2from Mar 2, 2026
Merged
fix: replace ⛔ with ⚠️ in azure-prepare SKILL.md#1095kvenkatrajan merged 2 commits intofix/skill-invocation-claude45-v2from
kvenkatrajan merged 2 commits intofix/skill-invocation-claude45-v2from
Conversation
…guidelines Co-authored-by: kvenkatrajan <102772054+kvenkatrajan@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Address feedback on skill routing improvements
fix: replace ⛔ with ⚠️ in azure-prepare SKILL.md
Mar 2, 2026
kvenkatrajan
approved these changes
Mar 2, 2026
kvenkatrajan
merged commit e71bfb7
into
fix/skill-invocation-claude45-v2
3 checks passed
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the azure-prepare skill documentation to align with the repo’s skill authoring guideline that restricts emoji usage to status indicators (✅, ❌,
Changes:
- Replaced the Phase 2 workflow step label from
⛔ Hand Offto⚠️ Hand Offinazure-prepare’s execution steps table.
| | 4 | **Harden Security** — Apply security best practices | [security.md](references/security.md) | | ||
| | 5 | **Update Plan** — Mark steps complete, set status to `Ready for Validation` | `.azure/plan.md` | | ||
| | 6 | **⛔ Hand Off** — Invoke **azure-validate** skill. Your preparation work is done. Deployment execution is handled by azure-deploy. | — | | ||
| | 6 | **⚠️ Hand Off** — Invoke **azure-validate** skill. Your preparation work is done. Deployment execution is handled by azure-deploy. | — | |
There was a problem hiding this comment.
The PR description/title implies replacing disallowed ⛔ emojis to comply with the skill authoring guidelines, but azure-prepare/SKILL.md still contains multiple ⛔ occurrences (e.g., Phase 1 Step 0 and the "STOP HERE" callout). Either replace the remaining ⛔ in this file with allowed status indicators (✅/❌/
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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The "Hand Off" step in the azure-prepare workflow used ⛔, which violates the repo's skill authoring guidelines restricting emojis to status indicators (✅, ❌,⚠️ ) only.
Change
plugin/skills/azure-prepare/SKILL.md: replaced⛔ Hand Off→⚠️ Hand Offon the workflow steps table row🔒 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.