Skip to content

Fix 122 broken symlinks in skills/ directory#225

Open
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-broken-symlinks-skills
Open

Fix 122 broken symlinks in skills/ directory#225
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-broken-symlinks-skills

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 27, 2026

All symlinks under skills/ were broken after skill directories moved from .github/skills/ to .github/plugins/<plugin>/skills/.

Updated every symlink target to reflect the new location:

# Before (broken)
skills/python/data/blob -> ../../../.github/skills/azure-storage-blob-py

# After
skills/python/data/blob -> ../../../.github/plugins/azure-sdk-python/skills/azure-storage-blob-py

Plugin mapping by language:

  • Python (azure-sdk-python) -- 38 symlinks
  • .NET (azure-sdk-dotnet) -- 28 symlinks
  • Java (azure-sdk-java) -- 25 symlinks
  • TypeScript (azure-sdk-typescript) -- 24 symlinks
  • Rust (azure-sdk-rust) -- 7 symlinks

All 122 symlinks now resolve and each target contains a valid SKILL.md.


💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

Copilot AI linked an issue Mar 27, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Fix broken symlinks in skills directory Fix 122 broken symlinks in skills/ directory Mar 27, 2026
Copilot AI requested a review from thegovind March 27, 2026 13:58
@scottaddie scottaddie marked this pull request as ready for review April 3, 2026 16:42
Copy link
Copy Markdown
Collaborator

@thegovind thegovind left a comment

Choose a reason for hiding this comment

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

Review summary

This PR is stale — the skills/ symlink directory it patches no longer exists on main. The work has been superseded by a different reorganization. Recommend closing.

Issues

🔴 Blocking

  • Branch is out of date and conflicting. mergeStateStatus: DIRTY, mergeable: CONFLICTING per gh pr view 225.
  • Target tree no longer exists. git ls-tree origin/main skills returns nothing — the entire top-level skills/ directory has been removed from main since this PR was filed (created 2026-03-27, last updated 2026-04-03). The symlink fix this PR proposes targets a path tree that's no longer in the repository.

🟡 Recommended

  • Symlink targets in the diff are correct for the architecture as it existed when this PR was opened — .github/plugins/<plugin>/skills/<skill> is the canonical source per AGENTS.md. So if the skills/ discovery path needs to come back, the same redirection pattern is the right one. But that's a follow-up question, not a fix that should land via this branch.

🟢 Nits

  • If the skills/ discovery layer is revived in the future, regenerate this PR fresh against current main rather than rebasing this branch — too much has shifted underneath it.

Recommend closing as superseded.

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.

Broken symlinks for skills

2 participants