Skip to content

[codex] fix(skills): load symlinked markdown skills#12025

Open
yzlu0917 wants to merge 1 commit intocontinuedev:mainfrom
yzlu0917:codex/issue-11985-symlinked-skills
Open

[codex] fix(skills): load symlinked markdown skills#12025
yzlu0917 wants to merge 1 commit intocontinuedev:mainfrom
yzlu0917:codex/issue-11985-symlinked-skills

Conversation

@yzlu0917
Copy link
Copy Markdown

@yzlu0917 yzlu0917 commented Apr 3, 2026

Summary

  • load project and global markdown skills from symlinked skill directories in both the CLI and core loaders
  • keep read_skill working by enumerating first-level skill directories instead of relying on recursive directory walks that skip symlinks
  • add regression tests covering symlinked .continue/skills/<skill> directories for both loaders

Why

npx skills add ... --skill <name> can install skills into a project via symlink. Before this change, those symlinked skill directories were skipped during skill discovery, so CLI skill listings and the read_skill tool could not find the installed skill.

Validation

  • ran npm test -- src/util/loadMarkdownSkills.test.ts in extensions/cli
  • ran npm run vitest -- config/markdown/loadMarkdownSkills.vitest.ts in core
  • ran npm run lint in extensions/cli (passes with 2 pre-existing warnings)
  • ran git diff --check

Closes #11985


Summary by cubic

Include symlinked markdown skills in discovery for both core and the CLI so skills installed via symlink show up and are readable by read_skill. Adds tests to prevent regressions.

  • Bug Fixes
    • Discover SKILL.md in symlinked directories under project and global skills dirs.
    • Switch to first-level directory enumeration (includes symlinks) instead of recursive walks that skipped them.
    • Add core and CLI tests for symlinked .continue/skills/<skill> and verify read_skill returns content and files.

Written for commit 3a93696. Summary will update on new commits.

@yzlu0917 yzlu0917 requested a review from a team as a code owner April 3, 2026 08:48
@yzlu0917 yzlu0917 requested review from sestinj and removed request for a team April 3, 2026 08:48
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Apr 3, 2026
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 4 files

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

Failed to read skill when it was installed with symlink

1 participant