Skip to content

Commit 4f3f630

Browse files
jongioCopilot
andcommitted
fix(pr-1092): address copilot trigger and emoji feedback
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 4edc352 commit 4f3f630

File tree

3 files changed

+10
-12
lines changed

3 files changed

+10
-12
lines changed

plugin/skills/azure-deploy/SKILL.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ description: "Deploy validated applications to Azure via azd, Bicep, or Terrafor
99
>
1010
> **PREREQUISITE**: The **azure-validate** skill **MUST** be invoked and completed with status `Validated` BEFORE executing this skill.
1111
12-
> ** STOP — PREREQUISITE CHECK REQUIRED**
12+
> **⚠️ STOP — PREREQUISITE CHECK REQUIRED**
1313
> Before proceeding, verify BOTH prerequisites are met:
1414
>
1515
> 1. **azure-prepare** was invoked and completed → `.azure/plan.md` exists
@@ -19,7 +19,7 @@ description: "Deploy validated applications to Azure via azd, Bicep, or Terrafor
1919
> - No plan? → Invoke **azure-prepare** skill first
2020
> - Status not `Validated`? → Invoke **azure-validate** skill first
2121
>
22-
> ** DO NOT MANUALLY UPDATE THE PLAN STATUS**
22+
> **⚠️ DO NOT MANUALLY UPDATE THE PLAN STATUS**
2323
>
2424
> You are **FORBIDDEN** from changing the plan status to `Validated` yourself. Only the **azure-validate** skill is authorized to set this status after running actual validation checks. If you update the status without running validation, deployments will fail.
2525
>
@@ -35,7 +35,7 @@ Activate this skill when user wants to:
3535
- Push updates to existing deployment
3636
- Run `azd up` or `az deployment`
3737
- Ship code to production
38-
- Deploy Azure Functions to the cloud
38+
- Deploy Azure serverless apps to the cloud
3939
- Deploy an application that already includes API Management (APIM) gateway infrastructure
4040

4141
> **APIM / AI Gateway**: Use this skill to deploy applications whose APIM/AI gateway infrastructure was already created during **azure-prepare**. For creating or changing APIM resources, see [APIM deployment guide](https://learn.microsoft.com/azure/api-management/get-started-create-service-instance). For AI governance policies, invoke **azure-aigateway** skill.
@@ -45,7 +45,7 @@ Activate this skill when user wants to:
4545
1. Run after azure-prepare and azure-validate
4646
2. `.azure/plan.md` must exist with status `Validated`
4747
3. **Pre-deploy checklist required**[Pre-Deploy Checklist](references/pre-deploy-checklist.md)
48-
4. **Destructive actions require `ask_user`**[global-rules](references/global-rules.md)
48+
4. ⚠️ **Destructive actions require `ask_user`**[global-rules](references/global-rules.md)
4949

5050
---
5151

@@ -61,7 +61,7 @@ Activate this skill when user wants to:
6161
| 6 | **Handle Errors** — See recipe's `errors.md` ||
6262
| 7 | **Verify Success** — Confirm deployment completed and endpoints are accessible | [Verify](references/recipes/azd/verify.md) |
6363

64-
> ** VALIDATION PROOF CHECK**
64+
> **⚠️ VALIDATION PROOF CHECK**
6565
>
6666
> When checking the plan, verify the **Validation Proof** section (Section 7) contains actual validation results with commands run and timestamps. If this section is empty, validation was bypassed — invoke **azure-validate** skill first.
6767

plugin/skills/azure-prepare/SKILL.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -29,11 +29,11 @@ Activate this skill when user wants to:
2929
4. **Update plan progressively** — Mark steps complete as you go
3030
5. **Validate before deploy** — Invoke azure-validate before azure-deploy
3131
6. **Confirm Azure context** — Use `ask_user` for subscription and location per [Azure Context](references/azure-context.md)
32-
7. **Destructive actions require `ask_user`**[Global Rules](references/global-rules.md)
32+
7. ⚠️ **Destructive actions require `ask_user`**[Global Rules](references/global-rules.md)
3333

3434
---
3535

36-
## PLAN-FIRST WORKFLOW — MANDATORY
36+
## ⚠️ PLAN-FIRST WORKFLOW — MANDATORY
3737

3838
> **YOU MUST CREATE A PLAN BEFORE DOING ANY WORK**
3939
>
@@ -46,7 +46,7 @@ Activate this skill when user wants to:
4646
4747
---
4848

49-
## STEP 0: Specialized Technology Check — MANDATORY FIRST ACTION
49+
## ⚠️ STEP 0: Specialized Technology Check — MANDATORY FIRST ACTION
5050

5151
**BEFORE starting Phase 1**, check if the user's prompt mentions a specialized technology that has a dedicated skill with tested templates. If matched, **invoke that skill FIRST** — then resume azure-prepare for validation and deployment.
5252

@@ -69,7 +69,7 @@ Create `.azure/plan.md` by completing these steps. Do NOT generate any artifacts
6969

7070
| # | Action | Reference |
7171
|---|--------|-----------|
72-
| 0 | ** Check Prompt for Specialized Tech** — If user mentions copilot SDK, Azure Functions, etc., invoke that skill first | [specialized-routing.md](references/specialized-routing.md) |
72+
| 0 | **⚠️ Check Prompt for Specialized Tech** — If user mentions copilot SDK, Azure Functions, etc., invoke that skill first | [specialized-routing.md](references/specialized-routing.md) |
7373
| 1 | **Analyze Workspace** — Determine mode: NEW, MODIFY, or MODERNIZE | [analyze.md](references/analyze.md) |
7474
| 2 | **Gather Requirements** — Classification, scale, budget | [requirements.md](references/requirements.md) |
7575
| 3 | **Scan Codebase** — Identify components, technologies, dependencies | [scan.md](references/scan.md) |
@@ -81,7 +81,7 @@ Create `.azure/plan.md` by completing these steps. Do NOT generate any artifacts
8181

8282
---
8383

84-
> ** STOP HERE** — Do NOT proceed to Phase 2 until the user approves the plan.
84+
> **⚠️ STOP HERE** — Do NOT proceed to Phase 2 until the user approves the plan.
8585
8686
---
8787

tests/azure-deploy/__snapshots__/triggers.test.ts.snap

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ exports[`azure-deploy - Trigger Tests Trigger Keywords Snapshot skill descriptio
1111
"cli",
1212
"deploy",
1313
"execution",
14-
"function",
1514
"identity",
1615
"live",
1716
"mcp",
@@ -36,7 +35,6 @@ exports[`azure-deploy - Trigger Tests Trigger Keywords Snapshot skill keywords m
3635
"cli",
3736
"deploy",
3837
"execution",
39-
"function",
4038
"identity",
4139
"live",
4240
"mcp",

0 commit comments

Comments
 (0)