Skip to content

Sync .github directory with azure-sdk-tools for PR 15210#2453

Open
azure-sdk wants to merge 4 commits intomainfrom
sync-.github-fix/azsdk-common-skill-rename-sync-15210
Open

Sync .github directory with azure-sdk-tools for PR 15210#2453
azure-sdk wants to merge 4 commits intomainfrom
sync-.github-fix/azsdk-common-skill-rename-sync-15210

Conversation

@azure-sdk
Copy link
Copy Markdown
Collaborator

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

@azure-sdk azure-sdk requested a review from l0lawrence April 20, 2026 19:04
@azure-sdk azure-sdk added EngSys This issue is impacting the engineering system. Central-EngSys This issue is owned by the Engineering System team. labels Apr 20, 2026
@azure-sdk azure-sdk force-pushed the sync-.github-fix/azsdk-common-skill-rename-sync-15210 branch 3 times, most recently from 0b9fd6b to be39505 Compare April 24, 2026 19:19
l0lawrence and others added 4 commits April 24, 2026 22:31
Re-creates PR #15062 on a fresh branch off current upstream/main to avoid
sync-pipeline CI issues on the original branch. Original branch
squad/15049-sync-shared-skills and PR #15062 are preserved unchanged for
comparison.

Makes skills the single source of truth by consolidating instruction file
content into the corresponding skill SKILL.md files, then deleting the
redundant instruction files. Also renames 5 shared skill directories with
the 'azsdk-common-' prefix required by the sync pipeline.

7 instruction files deleted (content merged into skills):
- local-sdk-workflow.instructions.md -> azsdk-common-generate-sdk-locally
- create-release-plan.instructions.md -> azsdk-common-prepare-release-plan
- sdk-details-in-release-plan.instructions.md -> azsdk-common-prepare-release-plan
- verify-namespace-approval.instructions.md -> azsdk-common-prepare-release-plan
- check-package-readiness.instructions.md -> azsdk-common-sdk-release
- check-package-validation.instructions.md -> azsdk-common-generate-sdk-locally
- copilot/sdk-release.instructions.md -> azsdk-common-sdk-release

3 skills enriched with details from instruction files:
- generate-sdk-locally: commit checkpoints, language selection, config file
  identification, SDK project path, metadata update tools, validation params
- prepare-release-plan: critical LLM instructions (NextSteps field), Release
  Planner Tool link, per-language package name extraction rules,
  valid/invalid examples table, JSON structure, namespace approval flow
- sdk-release: readiness check details, interaction flow, do-not-ask-for-PR
  guard, case-sensitive language note

5 shared skill directories renamed with 'azsdk-common-' prefix:
  apiview-feedback-resolution, generate-sdk-locally,
  pipeline-troubleshooting, prepare-release-plan, sdk-release

typespec-to-sdk orchestrator updated to reference skills directly. Removed
all related_skill frontmatter and blockquotes.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…lease-plan; only change is name prefix

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…efix

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Fix wrong MCP tool name: azsdk_typespec_customized_code_update
  -> azsdk_customized_code_update (apiview-feedback-resolution SKILL.md)
- Add azure-sdk-mcp: prefix to tool table in generate-sdk-locally
  SKILL.md for consistency with step references
- Narrow overly broad regex_not_match patterns in
  generate-sdk-locally eval.yaml (remove 'failed'/'error occurred'
  which cause false failures for build troubleshooting tasks)
- Fix YAML indentation in 4 evals/eval.yaml files (graders section
  not properly nested under graders: key)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@azure-sdk azure-sdk force-pushed the sync-.github-fix/azsdk-common-skill-rename-sync-15210 branch from be39505 to 0453c0f Compare April 24, 2026 22:31
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.

3 participants