Skip to content

fix: remove azure-compliance skill triggers in skill body - #3118

Open
JasonYeMSFT (JasonYeMSFT) wants to merge 1 commit into
microsoft:mainfrom
JasonYeMSFT:chuye/polish-azure-compliance
Open

fix: remove azure-compliance skill triggers in skill body#3118
JasonYeMSFT (JasonYeMSFT) wants to merge 1 commit into
microsoft:mainfrom
JasonYeMSFT:chuye/polish-azure-compliance

Conversation

@JasonYeMSFT

Copy link
Copy Markdown
Member

Description

Remove the redundant "When to Use This Skill" and "Skill Activation Triggers" sections from the Azure compliance skill. Trigger guidance remains in the skill frontmatter, reducing duplication and keeping the skill body focused on execution guidance.

Checklist

  • Tests pass locally (cd tests && npm test)
  • Title has one of the prefixes: fix:, feat:, feature:, chore:, misc:, test:, eval:
  • If modifying skill descriptions: verified routing correctness with integration tests (In tests/, npm run test:vally -- --plugin <plugin-dirname> --skill <skill>)

Related Issues

None.

Copilot AI lite review requested due to automatic review settings August 27, 2026 20:20
@JasonYeMSFT JasonYeMSFT (JasonYeMSFT) changed the title feature: streamline Azure compliance skill triggers fix: remove azure-compliance skill triggers in skill body Aug 27, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

Review tier: Lite
Findings: 1 Medium severity

New issues introduced by this change (1)
Severity Finding
Medium severity plugins/​azure-skills/​skills/​azure-compliance/​SKILL.mdSKILL.md files under plugins/*/skills/** are expected to include a **"## When to Use This…
What changed in this PR

This PR updates the azure-compliance skill documentation to reduce duplicated trigger guidance by removing the “When to Use This Skill” and “Skill Activation Triggers” sections from the skill body, relying on the frontmatter description for activation wording.

Changes:

  • Removed the “When to Use This Skill” section from azure-compliance SKILL.md.
  • Removed the “Skill Activation Triggers” section from azure-compliance SKILL.md.
File Description
plugins/​azure-skills/​skills/​azure-compliance/​SKILL.md Removes duplicated trigger/activation sections to keep the skill body focused on execution guidance.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines 16 to 20
| Best for | Compliance scans, security audits, Key Vault expiration checks |
| Primary capabilities | Comprehensive Resources Assessment, Key Vault Expiration Monitoring |
| MCP tools | azqr, subscription and resource group listing, Key Vault item inspection |

## When to Use This Skill

- Run azqr or Azure Quick Review for compliance assessment
- Validate Azure resource configuration against best practices
- Identify orphaned or misconfigured resources
- Audit Key Vault keys, secrets, and certificates for expiration

## Skill Activation Triggers

Activate this skill when user wants to:
- Check Azure compliance or best practices
- Assess Azure resources for configuration issues
- Run azqr or Azure Quick Review
- Identify orphaned or misconfigured resources
- Review Azure security posture
- "Show me expired certificates/keys/secrets in my Key Vault"
- "Check what's expiring in the next 30 days"
- "Audit my Key Vault for compliance"
- "Find secrets without expiration dates"
- "Check certificate expiration dates"

## Prerequisites
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.

2 participants