Skip to content

Sync .github/skills directory with azure-sdk-tools for PR 16634 - #50577

Merged
Crystal YU (chunyu3) merged 28 commits into
mainfrom
sync-.github-skills-yuc/sdk-quality-skill-16634
Sep 24, 2026
Merged

Crystal YU (chunyu3) merged 28 commits into
mainfrom
sync-.github-skills-yuc/sdk-quality-skill-16634

Conversation

@azure-sdk-automation

Copy link
Copy Markdown
Contributor

Sync .github/skills directory with azure-sdk-tools for PR Azure/azure-sdk-tools#16634 See eng/common workflow

Crystal YU (chunyu3) and others added 28 commits September 24, 2026 17:54
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>
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>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: chunyu3 <10458143+chunyu3@users.noreply.github.com>
Copilot AI lite review requested due to automatic review settings September 24, 2026 18:04
@azure-sdk-automation
azure-sdk-automation Bot requested a review from a team as a code owner September 24, 2026 18:04
@azure-sdk-automation azure-sdk-automation Bot added EngSys This issue is impacting the engineering system. Central-EngSys This issue is owned by the Engineering System team. labels Sep 24, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
36 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

Unresolved moderate issues affect workflow routing, duplicate generation, and evaluation coverage.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 2 Medium severity

Open (2)
What changed in this PR

Syncs SDK workflow skills with azure-sdk-tools PR 16634, adding breaking-change coverage and updating local generation guidance.

Changes:

  • Adds breaking-change detection and mitigation documentation.
  • Adds TypeSpec fixtures and workflow evaluations.
  • Updates SDK generation guidance and trigger coverage.
File Summary
.github/​skills/​azsdk-common-sdk-breaking-change/​SKILL.md Adds breaking-change workflow; fix triggers and duplicate generation handling need updates.
.github/​skills/​azsdk-common-sdk-breaking-change/​fixtures/​detect-sdk-breaking-changes-for-service-001/​models.tsp Adds TypeSpec detection fixture.
.github/​skills/​azsdk-common-sdk-breaking-change/​evals/​trigger.eval.yaml Adds breaking-change trigger evaluation.
.github/​skills/​azsdk-common-sdk-breaking-change/​evals/​eval.yaml Adds integration evaluation; mitigation and result validation need stronger coverage.
.github/​skills/​azsdk-common-generate-sdk-locally/​SKILL.md Updates generation workflow; references an unavailable skill.
.github/​skills/​azsdk-common-generate-sdk-locally/​evals/​trigger.eval.yaml Adds generation trigger evaluations.
.github/​skills/​azsdk-common-generate-sdk-locally/​evals/​eval.yaml Adds generation workflow evaluation coverage.

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@chunyu3
Crystal YU (chunyu3) merged commit 51c9f8e into main Sep 24, 2026
21 of 22 checks passed
@chunyu3
Crystal YU (chunyu3) deleted the sync-.github-skills-yuc/sdk-quality-skill-16634 branch September 24, 2026 23:19
browndav-msft pushed a commit to browndav-msft/azure-sdk-for-java that referenced this pull request Oct 14, 2026
…e#50577)

* add sdk quality assureance skill

* iterative assurance sdk quality

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* update the reference

* remove breaking change resolve from generate-sdk-locally skill

* format the breaking change display

* remove unused tool

* format and resolve comments

* add evals

* refine the instruction

* typo

* add eval test

* resolve comments

* format eval yaml

* resolve comments

* Sync .github/skills directory with azure-sdk-tools repository for Tools PR 16634

* use editScope name in skill

* add start mcp server instruction

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* Add generate-only eval coverage for local SDK generation

Co-authored-by: chunyu3 <10458143+chunyu3@users.noreply.github.com>

* update the eval case

* update the timeout

* set editCode CutomCode when in SDK repo

* Sync .github/skills directory with azure-sdk-tools repository for Tools PR 16634

* Sync .github/skills directory with azure-sdk-tools repository for Tools PR 16634

---------

Co-authored-by: FAREAST\chunyu <chunyu@microsoft.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: azure-sdk <azuresdk@microsoft.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: chunyu3 <10458143+chunyu3@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants