Pipeline analysis - #16300
Conversation
📊 GEPA Skill Quality Scores
10/13 skills at quality ≥ 0.80 How to improve# Score a specific skill
python .github/skills/sensei/scripts/gepa/auto_evaluator.py score --skill <name> --skills-dir .github/skills --tests-dir tests
# Optimize a skill with GEPA
python .github/skills/sensei/scripts/gepa/auto_evaluator.py optimize --skill <name> --skills-dir .github/skills --tests-dir tests |
There was a problem hiding this comment.
Pull request overview
This PR expands the Azure.Sdk.Tools.Vally evaluation coverage for pipeline troubleshooting/fixing by adding a new “quality” tier, updating the canonical mock pipeline failure fixture to a real-world storage version-parsing bug, and splitting pipeline capabilities into analysis-only vs fix-and-verify skills.
Changes:
- Add output-quality pipeline evals (analyze + fix) and include them in the Vally PR-gate suite.
- Update the mock
azsdk_analyze_pipelinehandler and related Vally scenarios/fixtures to use the QueueClientOptions service-version parsing failure. - Introduce
azsdk-common-pipeline-fixerand adjust the existing troubleshooting skill to be analysis-only with a structured output format.
Reviewed changes
Copilot reviewed 21 out of 21 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| tools/azsdk-cli/Azure.Sdk.Tools.Vally/README.md | Documents the new evals/quality tier and updated scenario inventory/layout. |
| tools/azsdk-cli/Azure.Sdk.Tools.Vally/fixtures/analyze-pipeline/QueueClientOptionsTests/code/QueueClientOptionsTests.cs | Adds the canonical failing test fixture for the pipeline “fix” eval. |
| tools/azsdk-cli/Azure.Sdk.Tools.Vally/fixtures/analyze-pipeline/QueueClientOptionsTests/code/QueueClientOptions.cs | Adds the canonical failing source fixture (missing switch cases) used by pipeline evals. |
| tools/azsdk-cli/Azure.Sdk.Tools.Vally/fixtures/.gitkeep | Updates fixture path guidance for environment.files overlays. |
| tools/azsdk-cli/Azure.Sdk.Tools.Vally/evals/workflow-scenarios/mock/analyze-failed-pipeline.eval.yaml | Aligns the mock workflow-scenario grader substring with the new canonical failure. |
| tools/azsdk-cli/Azure.Sdk.Tools.Vally/evals/tools/prompt-to-tool-pipeline.eval.yaml | Adds pipeline-failure routing stimuli (required/disallowed tool routing). |
| tools/azsdk-cli/Azure.Sdk.Tools.Vally/evals/quality/pipeline/fix-pipeline.eval.yaml | New output-quality eval that requires applying the code fix and verifying via package MCP tools. |
| tools/azsdk-cli/Azure.Sdk.Tools.Vally/evals/quality/pipeline/analyze-pipeline.eval.yaml | New output-quality eval that requires a correct root-cause diagnosis from build ref. |
| tools/azsdk-cli/Azure.Sdk.Tools.Vally/.vally.yaml | Adds a quality suite and includes it in pr-gate. |
| tools/azsdk-cli/Azure.Sdk.Tools.Mock/Handlers/Pipeline/AnalyzeAndArtifactsHandlers.cs | Updates the mock azsdk_analyze_pipeline output to the canonical fixture failure. |
| tools/azsdk-cli/Azure.Sdk.Tools.Cli/Helpers/PipelineIdentifierHelper.cs | Adjusts project-name resolution logic for DevOps build references. |
| .github/skills/README.md | Documents the new pipeline fixer skill and refines troubleshooting description. |
| .github/skills/azsdk-common-pipeline-troubleshooting/SKILL.md | Repositions the skill as analysis-only (“Pipeline Analysis”) and updates tools/steps. |
| .github/skills/azsdk-common-pipeline-troubleshooting/references/output-format.md | Adds a required markdown output structure for pipeline diagnoses. |
| .github/skills/azsdk-common-pipeline-troubleshooting/references/failure-patterns.md | Reworks failure pattern guidance into categorized tables (tests/build/validation/infra). |
| .github/skills/azsdk-common-pipeline-troubleshooting/evals/trigger.eval.yaml | Updates trigger prompts and adds an anti-trigger for auto-fix requests. |
| .github/skills/azsdk-common-pipeline-troubleshooting/evals/eval.yaml | Removes the old capability suite file. |
| .github/skills/azsdk-common-pipeline-fixer/SKILL.md | Introduces a new skill for applying and verifying pipeline fixes. |
| .github/skills/azsdk-common-pipeline-fixer/references/language-notes.md | Adds language-specific guidance for what build/check cover and what’s out-of-scope. |
| .github/skills/azsdk-common-pipeline-fixer/evals/trigger.eval.yaml | Adds trigger/anti-trigger evals for the new fixer skill. |
| .github/skills/.vally.yaml | Registers the new fixer skill’s eval path for skill-eval discovery. |
Removed unnecessary evaluation files and fixtures for pipeline analysis and fixing. Updated and added skills files for pipeline analysis and fixing.
Updated skills/tools descriptions. Deleted redundant eval in prompt-to-tool-pipeline.eval.yaml
99e2f30 to
6c1c525
Compare
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
- Normalize mock eval scoring.weights to sum to 1.0 (matching the repo convention on main), while adding completed + skill-invocation to the weighted graders so skill routing is actually gated in both the analyze and fix mock evals. - Correct stale doc references: mock handler evals path (evals/quality -> evals/workflow-scenarios/mock), Vally README double-slash link, and the ResolveProjectNameAsync XML summary to reflect that untrusted non-GUID names are rejected. - Prettier-format the .github/skills/README.md skill table.
|
The following pipelines have been queued for testing: |
Sync .github/skills directory with azure-sdk-tools for PR Azure/azure-sdk-tools#16300 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) --------- Co-authored-by: ReilleyMilne <reilleymilne@gmail.com> Co-authored-by: ReilleyMilne <91291100+ReilleyMilne@users.noreply.github.com>
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#16300 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) Co-authored-by: ReilleyMilne <reilleymilne@gmail.com>
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#16300 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) Co-authored-by: ReilleyMilne <reilleymilne@gmail.com>
|
/check-enforcer override |
azsdk-common-pipeline-troublshootingskill into two separate skills one for analysis and one for fixing.azsdk-common-pipeline-fixer, which automatically applies and verifies fixes for Azure SDK CI/CD pipeline failures.PipelineIdentiferHelperto fail early when a valid project name is not given.Fixes: #16200
Fixes: #16204