{Agent} Move X agent skills into repository policy - #34033
{Agent} Move X agent skills into repository policy#34033Aditya Pujara (a0x1ab) wants to merge 6 commits into
Conversation
Copilot-Session: da851f42-f383-454b-83ea-398b7c4c972a
There was a problem hiding this comment.
🟡 Changes recommended
The updated Tester charter omits the required repo_full_name="Azure/azure-cli" argument when instructing calls to infer_target_for_repo, which would cause the workflow to fail at runtime.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
This PR migrates several Azure CLI agent capabilities (target inference, changed-test selection, regression coverage, AAZ generation-source routing, and CLI Extensions handoff) from the shared/base skill set into repository-owned custom skills under .x/skills, and updates the agent charters and .x/x.yml wiring accordingly.
Changes:
- Adds repository-owned custom skills for changed-test selection, target inference, regression coverage, AAZ source promotion/routing, and extensions handoff.
- Updates
.x/x.ymlto map custom skill entrypoints and to include the required scope-bound shared primitives. - Updates agent role charters (
.x/*.md) to explicitly reference repository-owned custom skills.
File summaries
| File | Description |
|---|---|
| .x/x.yml | Removes prior base-skill entries and maps the new repository-owned custom skills; adds needed shared primitives for the wrappers. |
| .x/tester.md | Updates live-test dispatch flow to use repository-owned changed_test_files + infer_target_for_repo. |
| .x/reviewer.md | Updates review gating flow to call repository-owned get_pr_regression_coverage_summary. |
| .x/fixer.md | Updates target inference and extensions handoff guidance to call repository-owned custom skills. |
| .x/coordinator.md | Updates routing order to use repository-owned AAZ source routing/promotion wrappers. |
| .x/skills/README.md | Documents the intent and boundaries of repository-owned custom skills vs base primitives. |
| .x/skills/changed_test_files.py | Implements changed test-file stem selection for Azure CLI PR file lists. |
| .x/skills/find_aaz_fork_prs_ready_for_promotion.py | Wrapper that binds generation-source discovery to Azure CLI scope. |
| .x/skills/find_promoted_aaz_source_pr.py | Wrapper that binds promoted generation-source lookup to Azure CLI scope. |
| .x/skills/get_pr_regression_coverage_summary.py | Implements command-module regression coverage gap detection from PR file changes. |
| .x/skills/infer_target_for_repo.py | Implements module/extension inference using trusted repository directory listings and PR/text heuristics. |
| .x/skills/promote_aaz_fork_pr.py | Wrapper that binds generation-source promotion to Azure CLI scope. |
| .x/skills/start_aaz_source_task.py | Wrapper that binds durable generation-source task creation to Azure CLI scope. |
| .x/skills/start_extension_tracker_task.py | Wrapper that binds CLI-to-Extensions handoff to the central handoff primitive. |
Review details
- Files reviewed: 14/14 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Copilot-Session: da851f42-f383-454b-83ea-398b7c4c972a
Copilot-Session: da851f42-f383-454b-83ea-398b7c4c972a
|
Agent |
Copilot-Session: da851f42-f383-454b-83ea-398b7c4c972a
🤖 PR Validation — ️✔️ All clear
Summary
.x/skills..x/x.ymlto map the custom functions and approve only the scope bound shared primitives they require.Security