Skip to content

[Skills] Directory-based package skill discovery (drop registry table)#49478

Merged
samvaity merged 1 commit into
Azure:mainfrom
samvaity:savaity/directory-based-skill-discovery
Jun 12, 2026
Merged

[Skills] Directory-based package skill discovery (drop registry table)#49478
samvaity merged 1 commit into
Azure:mainfrom
samvaity:savaity/directory-based-skill-discovery

Conversation

@samvaity

Copy link
Copy Markdown
Member

What this does

Switches package-specific skill discovery from a hand-maintained registry table to directory-based discovery, mirroring the approach used in azure-sdk-for-js#38913 and azure-sdk-for-net.

Before

find-package-skill/SKILL.md contained a static Package Skills table mapping each package name to its SKILL.md path. Every new package skill required adding a row to this central table (the create-package-skill wizard had a dedicated "Register" phase for exactly this).

After

find-package-skill now instructs the agent to:

  1. Resolve the package directory from the file/package being modified.
  2. Check for a .github/skills/ directory inside that package directory.
  3. Read every SKILL.md (+ adjacent references/) found there.

Placement is registration — no central table to maintain.

Changes

  • find-package-skill/SKILL.md — replace the registry table with directory-based discovery instructions.
  • create-package-skill/SKILL.md — update description to reflect automatic discovery; Phase 4 label updated.
  • phases/03-validate.md — update Phase 4 reference from "Register" to "Finalize".
  • phases/04-register.md04-finalize.md — no longer edits a table, just confirms the skill sits in the discoverable location.

Cross-language alignment

Switch package-specific skill discovery from a hand-maintained registry table
to directory-based discovery, mirroring the approach in azure-sdk-for-js#38913
and azure-sdk-for-net.

Changes:
- find-package-skill/SKILL.md: replace registry table with directory-based
  discovery instructions (check for .github/skills/ inside the package dir)
- create-package-skill/SKILL.md: update description, Phase 4 label
- phases/03-validate.md: update Phase 4 reference wording
- phases/04-register.md -> 04-finalize.md: no central table to update,
  placement is registration

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@samvaity samvaity force-pushed the savaity/directory-based-skill-discovery branch from bc01532 to 78bb708 Compare June 12, 2026 19:08
@samvaity samvaity marked this pull request as ready for review June 12, 2026 19:09
@samvaity samvaity requested a review from a team as a code owner June 12, 2026 19:09
Copilot AI review requested due to automatic review settings June 12, 2026 19:09
@samvaity samvaity self-assigned this Jun 12, 2026

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.

Pull request overview

This PR updates the Azure SDK for Java GitHub Skills documentation to switch package-skill discovery from a centralized, hand-maintained registry table to directory-based discovery (skills are discovered by being placed under a package’s sdk/<service>/<package>/.github/skills/ directory).

Changes:

  • Replaces find-package-skill’s static “Package Skills” registry table with step-by-step directory-based discovery instructions.
  • Updates create-package-skill wizard messaging and phase naming to reflect automatic discovery (Phase 4: “Register” → “Finalize”).
  • Renames and rewrites the Phase 4 instructions to confirm discoverability rather than editing a central table.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
.github/skills/find-package-skill/SKILL.md Replaces registry-table lookup with directory-based discovery steps.
.github/skills/create-package-skill/SKILL.md Updates wizard description and Phase 4 label/link to “Finalize”.
.github/skills/create-package-skill/phases/03-validate.md Updates Phase 3 prompt and “Next” link to point to Finalize.
.github/skills/create-package-skill/phases/04-register.md Removes old “Register” phase content that edited the central table.
.github/skills/create-package-skill/phases/04-finalize.md Adds new “Finalize” phase to confirm location-based discovery and provide summary.

Comment thread .github/skills/create-package-skill/phases/04-finalize.md
@samvaity samvaity merged commit 3de29ca into Azure:main Jun 12, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants