Repository navigation
fix(registry): never link a Skill whose source is gone - #634
Merged
Merged
Conversation
A Skill whose SKILL.md is gone upstream answers 410. #389 kept the Skill page from linking such a sibling, but the file page render still read every Skill name in the repository. Related Skills and the featured sections on /skills read Skill cards without the source check too. The Skill page and the file page now share one read of linkable sibling names. The registry card reads share one linkable Skill predicate. The asset cache key moves to v6 so cached renders drop stale links.
Collaborator
Author
|
Agent note (Harlan Agent Kit). Checked by hand on 2026-10-13, since CI cannot cover it:
|
harlan-zw
added
the
harlan-agent-review
harlan-github-agent
Bot
added
harlan-agent-review-required
Contributor
🤖 MERGED
GitHub merged this pull request. No material findings were recorded. The pull request closed. |
harlan-github-agent
Bot
removed
harlan-agent-running
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
❓ Type of change
📚 Description
A Skill whose SKILL.md is gone upstream answers 410. Nuxt SEO's "Broken internal link · 410 Gone" Action flagged 17 skilld pages that linked such a Skill or one of its files. #389 stopped the Skill page from linking them, but three reads still could:
/gh/<owner>/<repo>/<name>/-/<file>, linked a/namemention to any Skill in the repository, gone ones included./skillslisted and counted gone Skills.The Skill page and the file page now share one read of linkable sibling names,
loadLinkableRepoSkillNames. The registry card reads share one predicate,LINKABLE_SKILL_SQL. The CLI name lookup behind/api/skills/resolvestill matches gone Skills, because it renders no link. The file render cache key moves tov6, so cached renders with stale links drop out.