Skip to content

fix: keep each skill's links inside its own folder - #12

Merged
hackernotfound merged 1 commit into
mainfrom
fix/self-contained-skill-links
Oct 1, 2026
Merged

hackernotfound merged 1 commit into
mainfrom
fix/self-contained-skill-links

Conversation

@hackernotfound

Copy link
Copy Markdown
Owner

Makes every skill pass awesome-copilot's vally lint, which requires each skill to reference only files inside its own folder. 18 skills linked into other skills (../poteto-mode/..., ../principle-*/SKILL.md). The build now rewrites those 37 links into plain pointers such as "see skills/poteto-mode/references/codex-tools.md in this plugin", leaving pstack's content otherwise unchanged.

Changes

  • tools/build.mjs rewrites cross-skill Markdown links on every build and fails if none are found, so an upstream style change is noticed.
  • tools/check.mjs fails if any skill links outside its own folder.
  • Version 1.1.1.

Verified:

  • @microsoft/vally 0.12.0 lint: 54/54 skills pass (was 36/54).
  • node tools/check.mjs: 22/22. With the rewrite disabled, the new check fails.
  • tools/eval.sh 10/10 and node tools/eval-vscode.mjs 10/10.

@hackernotfound
hackernotfound merged commit 3b2bfa2 into main Oct 1, 2026
4 checks passed
@hackernotfound
hackernotfound deleted the fix/self-contained-skill-links branch October 1, 2026 23:37
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