Sync .github/skills directory with azure-sdk-tools for PR 16603 - #10189
Open
helen229 wants to merge 2 commits into
Open
Sync .github/skills directory with azure-sdk-tools for PR 16603#10189helen229 wants to merge 2 commits into
helen229 wants to merge 2 commits into
Conversation
|
Azure Pipelines: 1 pipeline(s) were filtered out due to trigger conditions. There may be pipelines that require an authorized user to comment /azp run to run. |
Contributor
There was a problem hiding this comment.
Pull request overview
Syncs shared skill guidance and evaluations from azure-sdk-tools#16603.
Changes:
- Adds explicit skill environments and cross-skill routing checks.
- Improves evaluation prompts and tool assertions.
- Updates SDK generation and release workflow guidance.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
azsdk-common-apiview-feedback-resolution/evals/eval.yaml |
Expands APIView capability evaluations. |
azsdk-common-apiview-feedback-resolution/evals/trigger.eval.yaml |
Adds explicit routing environments. |
azsdk-common-generate-sdk-locally/SKILL.md |
Adds pipeline-generation guidance. |
azsdk-common-generate-sdk-locally/evals/eval.yaml |
Makes local-generation scenarios actionable. |
azsdk-common-pipeline-analysis/evals/trigger.eval.yaml |
Strengthens analysis routing boundaries. |
azsdk-common-pipeline-fixer/evals/trigger.eval.yaml |
Strengthens fixer routing boundaries. |
azsdk-common-prepare-release-plan/SKILL.md |
Corrects release-plan creation guidance. |
azsdk-common-prepare-release-plan/evals/trigger.eval.yaml |
Expands release-plan routing tests. |
azsdk-common-sdk-release/SKILL.md |
Adds informational release triggers. |
azsdk-common-sdk-release/evals/trigger.eval.yaml |
Expands SDK-release routing tests. |
Comments suppressed due to low confidence (1)
.github/skills/azsdk-common-generate-sdk-locally/SKILL.md:88
- This repeats the incomplete direct-tool route for all-language generation.
azsdk_run_generate_sdkmust be invoked once per target language after obtaining release-plan details, so this troubleshooting advice should delegate to the pipeline-generation skill rather than imply one direct call handles the workflow.
- For "generate SDK for all languages", pipeline-based generation, or when no local SDK clone exists, call `azure-sdk-mcp:azsdk_run_generate_sdk` instead of the local generate flow, and never substitute `azure-sdk-mcp:azsdk_get_sdk_pull_request_link` / `azure-sdk-mcp:azsdk_get_pull_request` for generation.
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.
Summary
Manually syncs
.github/skillscontent from Azure/azure-sdk-tools#16603 ("Add environment.skills to workflow-scenario eval specs", a clean rebuild of #16380) into this repo, since the automated sync pipeline currently fails with agit am -3"sha1 information is lacking or useless" error onazsdk-common-generate-sdk-locally/SKILL.md.That failure is caused by two files in this repo having drifted from
azure-sdk-toolsmain (left over from the #16380/#10150/#10165 sync-then-revert history):.github/skills/azsdk-common-generate-sdk-locally/SKILL.md.github/skills/azsdk-common-prepare-release-plan/SKILL.mdThis PR brings all 10 files touched by azure-sdk-tools#16603 to match that branch's content exactly (verified via direct content diff against azure-sdk-tools main + the PR branch), which both catches up the drifted baseline and applies #16603's
environment.skillschanges in one step.Files synced
azsdk-common-apiview-feedback-resolution/evals/{eval,trigger.eval}.yamlazsdk-common-generate-sdk-locally/{SKILL.md,evals/eval.yaml}azsdk-common-pipeline-analysis/evals/trigger.eval.yamlazsdk-common-pipeline-fixer/evals/trigger.eval.yamlazsdk-common-prepare-release-plan/{SKILL.md,evals/trigger.eval.yaml}azsdk-common-sdk-release/{SKILL.md,evals/trigger.eval.yaml}Related: Azure/azure-sdk-tools#16603 (supersedes Azure/azure-sdk-tools#16380).