feature: retire azure-rbac skill#2779
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR retires the azure-rbac skill from the plugin by removing the skill source/evals and cleaning up references in test registration, CI, and related documentation so the remaining skills and scheduled runs no longer include azure-rbac.
Changes:
- Removed
azure-rbacfrom the registered skills list and the scheduled integration test matrix. - Updated dependent skill frontmatter descriptions to remove references to the retired
azure-rbacskill. - Removed
azure-rbacreferences from CI workflow inputs, CODEOWNERS, and internal meta-skill docs.
Show a summary per file
| File | Description |
|---|---|
| tests/skills.json | Removes azure-rbac from the skill registry and integration schedule list. |
| plugin/skills/entra-app-registration/SKILL.md | Drops azure-rbac mention from anti-triggers in the skill description. |
| plugin/skills/entra-agent-id/SKILL.md | Drops azure-rbac mention from anti-triggers in the skill description. |
| plugin/skills/azure-rbac/version.json | Deletes per-skill version configuration as part of retiring the skill. |
| plugin/skills/azure-rbac/SKILL.md | Removes the retired skill definition. |
| evals/azure-rbac/eval.yaml | Removes the retired skill’s Vally integration eval suite. |
| .github/workflows/test-all-integration.yml | Removes azure-rbac from examples and from VALLY_SKILLS. |
| .github/skills/sensei/references/SCORING.md | Removes azure-rbac from the scoring reference list. |
| .github/skills/file-test-bug/SKILL.md | Updates example skill names away from azure-rbac. |
| .github/skills/analyze-skill-issues/references/blob-structure.md | Removes azure-rbac from skill name mapping documentation. |
| .github/CODEOWNERS | Removes ownership entry for the retired skill path. |
Review details
- Files reviewed: 11/11 changed files
- Comments generated: 1
- Review effort level: Low
JasonYeMSFT
force-pushed
the
chuye/retire-azure-rbac
branch
from
July 13, 2026 19:50
56f682d to
1227c99
Compare
tmeschter
previously approved these changes
Jul 15, 2026
JasonYeMSFT
force-pushed
the
chuye/retire-azure-rbac
branch
from
July 15, 2026 17:09
1227c99 to
a4a4626
Compare
tmeschter
approved these changes
Jul 15, 2026
msalaman
approved these changes
Jul 15, 2026
msalaman
approved these changes
Jul 15, 2026
RickWinter
approved these changes
Jul 16, 2026
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.
Description
Retires the
azure-rbacskill by removing its plugin definition, evals, tests, and references throughout the codebase. Updates dependent skills (entra-agent-id, entra-app-registration) and CI workflows accordingly.Checklist
cd tests && npm test)fix:,feat:,feature:,chore:,misc:,test:,eval:tests/,npm run test:integration -- <skill>ornpm run test:vally -- --skill <skill>)Related Issues
Closes #2778