Skip to content

Commit 884b38e

Browse files
committed
feature: retire azure-hosted-copilot-sdk skill
1 parent 20e7a21 commit 884b38e

45 files changed

Lines changed: 55 additions & 1947 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.github/CODEOWNERS

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@
2020
/plugin/skills/azure-deploy/ @tmeschter @wbreza @kvenkatrajan @paulyuk @RickWinter
2121
/plugin/skills/azure-diagnostics/ @tmeschter @saikoumudi @RickWinter
2222
/plugin/skills/azure-enterprise-infra-planner/ @Jbrocket @micha31r @arunrab @RickWinter
23-
/plugin/skills/azure-hosted-copilot-sdk/ @tmeschter @JasonYeMSFT @RickWinter
2423
/plugin/skills/azure-kubernetes/ @saikoumudi @chandraneel @gambtho @RickWinter
2524
/plugin/skills/azure-kusto/ @saikoumudi @RickWinter
2625
/plugin/skills/azure-messaging/ @kashifkhan @RickWinter

.github/skills/sensei/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -178,7 +178,7 @@ metadata:
178178
179179
> ⚠️ **"DO NOT USE FOR:" carries context-dependent risk.** In multi-skill environments (10+ skills with overlapping domains), anti-trigger clauses introduce the very keywords that cause wrong-skill activation on Claude Sonnet and fast-pattern-matching models ([evidence](https://gist.github.com/kvenkatrajan/52e6e77f5560ca30640490b4cc65d109)). For small, isolated skill sets (1-5 skills), the risk is low. When in doubt, use positive routing with `WHEN:` and distinctive quoted phrases.
180180
>
181-
> **Exception:** `DO NOT USE FOR:` is **REQUIRED** when a specialized skill's triggers overlap with a broader skill (e.g., `azure-hosted-copilot-sdk` vs. `azure-prepare` on "deploy to Azure"). Without the negative discriminator, the broader skill captures prompts that should route to the specialized one. Always run integration tests before removing a `DO NOT USE FOR:` clause.
181+
> **Exception:** `DO NOT USE FOR:` is **REQUIRED** when a specialized skill's triggers overlap with a broader skill (e.g., `azure-prepare` on "deploy to Azure"). Without the negative discriminator, the broader skill captures prompts that should route to the specialized one. Always run integration tests before removing a `DO NOT USE FOR:` clause.
182182
183183
## Test Scaffolding
184184

.github/skills/sensei/references/SCORING.md

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,7 @@ Per the [agentskills.io spec](https://agentskills.io/specification), the `name`
166166
| Single skill or small set (1-5 skills) with clear domain boundaries | Low | Anti-triggers are low-risk — domain boundaries are obvious |
167167
| Medium skill set (5-15 skills) with some overlap | Moderate | Anti-trigger keywords start competing with other skills' triggers |
168168
| Large skill set (15+ skills) with overlapping domains | **High** | Keyword contamination is measurable — negative keywords become activation keywords on fast-pattern-matching models |
169-
| **Specialized skill with trigger overlap against a broader skill** | **REQUIRED** | Anti-triggers are the **only** disambiguation mechanism when a specialized skill (e.g., `azure-hosted-copilot-sdk`) competes with a broader skill (e.g., `azure-prepare`) on shared trigger phrases like "deploy to Azure". Removing them causes routing regressions. |
169+
| **Specialized skill with trigger overlap against a broader skill** | **REQUIRED** | Anti-triggers are the **only** disambiguation mechanism when a specialized skill competes with a broader skill (e.g., `azure-prepare`) on shared trigger phrases like "deploy to Azure". Removing them causes routing regressions. |
170170

171171
**Why large skill sets are risky:** On Claude Sonnet and similar models that use fast pattern matching (first ~20 words), `DO NOT USE FOR: Function apps` causes Sonnet to key on "Function apps" and **activate** the skill for Functions queries. This was empirically demonstrated across 24 Azure skills ([analysis](https://gist.github.com/kvenkatrajan/52e6e77f5560ca30640490b4cc65d109)). Anthropic's own published skills confirm this pattern — 4 of 5 skills in `anthropics/skills` use positive-only routing.
172172

@@ -327,7 +327,6 @@ function scoreSkill(skill):
327327

328328
# Warn on anti-triggers (keyword contamination risk)
329329
# Exception: DO NOT warn if skill has trigger overlap with a broader skill
330-
# (e.g., azure-hosted-copilot-sdk overlaps with azure-prepare on "deploy")
331330
# In that case, DO NOT USE FOR is REQUIRED for disambiguation
332331
if containsAntiTriggers(skill.description):
333332
if hasOverlappingTriggersWithBroaderSkill(skill):

.github/skills/skill-authoring/references/guidelines/frontmatter.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,4 +56,4 @@ description: "Helps with Azure stuff."
5656
5757
> ⚠️ **Do NOT add "DO NOT USE FOR:" clauses** unless the skill has trigger overlap with a broader skill (e.g., `azure-prepare`). For most skills, `DO NOT USE FOR:` causes keyword contamination on Claude Sonnet and similar models — use positive routing with distinctive `WHEN:` phrases instead.
5858
>
59-
> **Exception — disambiguation-critical skills:** When a specialized skill shares trigger phrases with a broader skill (e.g., both `azure-hosted-copilot-sdk` and `azure-prepare` match "deploy to Azure"), `DO NOT USE FOR:` is **REQUIRED** to prevent the broader skill from capturing the specialized skill's prompts. Removing it causes routing regressions. Always run integration tests before removing a `DO NOT USE FOR:` clause.
59+
> **Exception — disambiguation-critical skills:** When a specialized skill shares trigger phrases with a broader skill, `DO NOT USE FOR:` is **REQUIRED** to prevent the broader skill from capturing the specialized skill's prompts. Removing it causes routing regressions. Always run integration tests before removing a `DO NOT USE FOR:` clause.

docs/spec/azure-hosted-copilot-sdk.md

Lines changed: 0 additions & 174 deletions
This file was deleted.

0 commit comments

Comments
 (0)