Skip to content

Keep skill links inside each skill directory (marketplace link validation) - #200

Merged
Nikola Metulev (nmetulev) merged 1 commit into
stagingfrom
nmetulev-winapp-0-7-migration
Sep 30, 2026
Merged

Nikola Metulev (nmetulev) merged 1 commit into
stagingfrom
nmetulev-winapp-0-7-migration

Conversation

@nmetulev

Copy link
Copy Markdown
Member

Description

awesome-copilot's external-plugin quality gate (github/awesome-copilot#4323) runs vally valid-refs. That check rejects markdown links in a SKILL.md that resolve outside the skill's own directory. Our skills had 22 cross-skill links, such as [winui-dev-workflow](../winui-dev-workflow/SKILL.md), across 7 skills, so the 0.7.0 listing update fails.

  • Replace those links with plain-text skill names (`winui-dev-workflow`) and name the shared reference as `winui-packaging`'s `references/sourcegen-patterns.md`. Agents load skills by name, so no capability is lost. vally only inspects markdown links, not inline code.
  • Add the same rule to the Validate Agent Plugins package job so CI catches regressions. It flags links outside the skill directory and links to missing files, and it ignores URLs, anchors, and code.

Verified: the new check reports exactly vally's 22 findings on current staging and passes on this branch. All eight skills pass skills-ref, and the PowerShell regressions pass.

This needs a 0.7.1 patch release so awesome-copilot can pin the fixed tag.

Type of Change

  • 🐛 Bug fix
  • 🔧 Config / build / CI

Checklist

  • SKILL.md frontmatter valid; skill references still resolve by name
  • Added a bullet to ## [Unreleased] in CHANGELOG.md
  • No version fields changed

Marketplace link validation (vally valid-refs, used by awesome-copilot) rejects markdown links that resolve outside a skill's own folder. Name sibling skills and the shared source-generator reference in plain text instead, and add the same check to the plugin validator so CI catches regressions.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@nmetulev
Nikola Metulev (nmetulev) merged commit 694301f into staging Sep 30, 2026
9 checks passed
@nmetulev
Nikola Metulev (nmetulev) deleted the nmetulev-winapp-0-7-migration branch September 30, 2026 23:02
This was referenced Sep 30, 2026
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.

1 participant