Skip to content

fix(docs): update skills page to load from both skills/ and .claude/skills/ directories#2451

Merged
huacnlee merged 3 commits into
mainfrom
copilot/fix-skilled-page-links
Jun 9, 2026
Merged

fix(docs): update skills page to load from both skills/ and .claude/skills/ directories#2451
huacnlee merged 3 commits into
mainfrom
copilot/fix-skilled-page-links

Conversation

Copilot AI commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

The Skills page linked to stale paths from before the skills consolidation in 89f7662. The data loader only scanned .claude/skills/, missing the two skills moved to skills/gpui and skills/gpui-component.

Description

  • Updated docs/data/skills.data.js to scan both skills/ and .claude/skills/ directories instead of only .claude/skills/
  • Used the full relative path (from repo root) as the skill id to prevent collisions if the same directory name appears in both locations
- const skillsDir = join(__dirname, "../../.claude/skills");
+ const skillsDirs = [
+   join(repoRoot, "skills"),
+   join(repoRoot, ".claude/skills"),
+ ];

Skills now resolved:

ID Path
skills/gpui skills/gpui/SKILL.md
skills/gpui-component skills/gpui-component/SKILL.md
.claude/skills/gpui-component-dev .claude/skills/gpui-component-dev/SKILL.md

Screenshot

Before After
Links pointed to removed paths (e.g. .claude/skills/gpui-action/SKILL.md) Links point to existing skill entry points

How to Test

Load the Skills page (/skills) and verify all skill cards link to existing GitHub paths.

Checklist

  • I have read the CONTRIBUTING document and followed the guidelines.
  • Reviewed the changes in this PR and confirmed AI generated code (If any) is accurate.
  • Passed cargo run for story tests related to the changes.
  • Tested macOS, Windows and Linux platforms performance (if the change is platform-specific)

Copilot AI changed the title [WIP] Fix broken links on skills page fix(docs): update skills page to load from both skills/ and .claude/skills/ directories Jun 9, 2026
Copilot AI requested a review from huacnlee June 9, 2026 05:37
@huacnlee huacnlee marked this pull request as ready for review June 9, 2026 06:24
@huacnlee huacnlee enabled auto-merge (squash) June 9, 2026 07:03
@huacnlee huacnlee merged commit daffa6d into main Jun 9, 2026
4 checks passed
@huacnlee huacnlee deleted the copilot/fix-skilled-page-links branch June 9, 2026 07:07
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.

docs: skills page links point to removed skill paths

2 participants