{Misc.} Repository owned agents and tooling with a single x.yml - #34163
Open
Aditya Pujara (a0x1ab) wants to merge 4 commits into
Open
Aditya Pujara (a0x1ab) wants to merge 4 commits into
Aditya Pujara (a0x1ab) wants to merge 4 commits into
Conversation
Preserve existing agents, domain tooling, workflow settings, isolated execution, immutable job recovery and explicit rollback. Copilot-Session: 9b1d5c02-4be1-4677-ac69-1046a5632dbc
Collaborator
|
Thank you for your contribution! We will review the pull request and get back to you soon. |
Contributor
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Unresolved dispatch, validation, and review-policy defects can misdirect implementation work or produce incorrect review outcomes.
Review effort: Balanced
Findings: 1
Open (9)
Stop dispatch when the extension target is unresolved · New Handle paginated and incomplete AAZ source searches · New Validate extension tracker destination before writing · New Finalize handoff only after dispatch and reuse retries · New Include Azure CLI Core runtime files in production changes · New Use PR body and recognize valid History Notes · New Require validated AAZ source metadata · New Apply path-based risk signals to all changed files · New Fall back to component TestCase suites for test selection · New
What changed in this PR
Consolidates Azure CLI’s agent configuration in x.yml and moves repository-owned automation into discoverable definitions and tools.
Changes:
- Introduces a versioned repository profile and Python dependency pins.
- Adds role definitions and routing, review, testing, and validation tools.
- Removes legacy charters and skill wrappers; documents the package contract.
| File | Description |
|---|---|
.x/x.yml |
Defines repository profile and dependencies. |
.x/tools/validation/checks.py |
Adds unit-test and AAZ validation. |
.x/tools/tester/azure_cli/live_tests.py |
Selects tests and summarizes regression evidence. |
.x/tools/settings.py |
Centralizes domain patterns and constants. |
.x/tools/reviewer/policy/analysis.py |
Adds review checks and risk assessment. |
.x/tools/reviewer/azure_cli/review.py |
Defines CLI release and review policy. |
.x/tools/reviewer/azure_cli/failures.py |
Extracts and classifies test failures. |
.x/tools/fixer/title_gate/title_repair.py |
Repairs titles and History Notes. |
.x/tools/fixer/formatting/guidance.py |
Generates PR formatting guidance. |
.x/tools/fixer/azure_cli/targets.py |
Discovers module and extension targets. |
.x/tools/fixer/azure_cli/routing.py |
Creates extension issue trackers. |
.x/tools/fixer/azure_cli/guidance.py |
Supplies code-generation guidance. |
.x/tools/fixer/azure_cli/aaz.py |
Handles AAZ tasks and source lookup. |
.x/tester.md |
Removes legacy Tester charter. |
.x/skills/start_extension_tracker_task.py |
Removes extension handoff wrapper. |
.x/skills/start_aaz_source_task.py |
Removes AAZ task wrapper. |
.x/skills/promote_aaz_fork_pr.py |
Removes promotion wrapper. |
.x/skills/infer_target_for_repo.py |
Removes legacy target inference. |
.x/skills/get_pr_regression_coverage_summary.py |
Removes legacy coverage summary. |
.x/skills/find_promoted_aaz_source_pr.py |
Removes source lookup wrapper. |
.x/skills/find_aaz_fork_prs_ready_for_promotion.py |
Removes promotion discovery wrapper. |
.x/skills/changed_test_files.py |
Removes legacy test selector. |
.x/reviewer.md |
Removes legacy Reviewer charter. |
.x/README.md |
Documents discovery and validation contracts. |
.x/fixer.md |
Removes legacy Fixer charter. |
.x/definitions/tester.md |
Adds Tester workflow instructions. |
.x/definitions/reviewer.md |
Adds review and reporting instructions. |
.x/definitions/fixer.md |
Adds triage and dispatch instructions. |
.x/definitions/coordinator.md |
Defines coordination scope and safeguards. |
.x/coordinator.md |
Removes legacy Coordinator charter. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Copilot-Session: 9b1d5c02-4be1-4677-ac69-1046a5632dbc
Copilot-Session: 9b1d5c02-4be1-4677-ac69-1046a5632dbc
Copilot-Session: 9b1d5c02-4be1-4677-ac69-1046a5632dbc
Member
Author
|
/azp run |
|
Azure Pipelines: Successfully started running 3 pipeline(s). |
This branch has not been deployed
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.


Move repository owned agents, tooling and workflow configuration into .x.