Skip to content

Fix dead links in synced core skills - #97

Merged
bketelsen merged 2 commits into
mainfrom
copilot/fix-dead-links-in-skills-sync
Aug 19, 2026
Merged

Fix dead links in synced core skills#97
bketelsen merged 2 commits into
mainfrom
copilot/fix-dead-links-in-skills-sync

Conversation

Copilot AI commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Summary

Default synced skills referenced core-only paths that dangle in consumer repositories.

  • Replace core ADR and script links with canonical GitHub URLs.
  • Correct the frostyard-design cross-skill path.
  • Reject skill links that escape .agents/skills/, with regression coverage.

Risk tier

Risk tier: 1 — shared documentation, skills, and validation only.

Docs housekeeping

  • New docs started from their category's TEMPLATE.md — N/A
  • Every new doc indexed in docs/README.md — N/A
  • Cross-links added in both directions (ADR ↔ design ↔ spec ↔ plan) — N/A
  • New significant decision recorded as an ADR first, in this PR — N/A
  • Conformance aliases (ADR-0029) untouched — canonical targets edited
    instead

Verification

  • node scripts/check-docs.mjs green
  • Scaffold changed? npm ci && npm test in
    .agents/skills/frostyard-docs-site/scaffold/ green
  • Workflows changed? Every action SHA-pinned (40 chars + # vX.Y.Z),
    permissions: {}, persist-credentials: false (ADR-0021) — N/A

Co-authored-by: bketelsen <37492+bketelsen@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix dead links in core skills sync Fix dead links in synced core skills Aug 19, 2026
Copilot AI requested a review from bketelsen August 19, 2026 16:04
@bketelsen
bketelsen marked this pull request as ready for review August 19, 2026 16:53
@bketelsen
bketelsen merged commit 451b380 into main Aug 19, 2026
2 checks passed
@bketelsen
bketelsen deleted the copilot/fix-dead-links-in-skills-sync branch August 19, 2026 16:57
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.

[core] Synced skills link to core ADRs with repo-relative paths that dangle in all 11 consuming repos

2 participants