Skip to content

Comments

feat: Follow symlinked skills#8370

Closed
philipp-spiess wants to merge 1 commit intoopenai:mainfrom
philipp-spiess:follow-symlinked-skills
Closed

feat: Follow symlinked skills#8370
philipp-spiess wants to merge 1 commit intoopenai:mainfrom
philipp-spiess:follow-symlinked-skills

Conversation

@philipp-spiess
Copy link

Closes #8369

Skills are an open standard and as such it should be easily possible to share skills across different agents. In my case, that involves setting up skills via symlinks.

This PR tracks symlinks in the skills traversal instead of skipping it. Note that symlinks can be recursive so we have to keep track of visited directories during the skills scan.

Disclaimer that the code is 100% written by GPT-5.2-Codex though :) But I did test it locally:

Before

CleanShot 2025-12-20 at 23 46 02@2x

After

CleanShot 2025-12-20 at 23 50 44@2x

I also ran one of the skills to ensure they work alright!

@github-actions
Copy link
Contributor

github-actions bot commented Dec 20, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@philipp-spiess
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

github-actions bot added a commit that referenced this pull request Dec 20, 2025
@etraut-openai
Copy link
Collaborator

Thanks for the contribution. We've updated our contribution guidelines to clarify that we're currently accepting contributions for bugs and security fixes, but we're not generally accepting new features at this time. We need to make sure that all new features compose well with both existing and upcoming features and fit into our roadmap. If you would like to propose a new feature, please file or upvote an enhancement request in the issue tracker. We will generally prioritize new features based on community feedback.

@philipp-spiess
Copy link
Author

@etraut-openai Hey! Do you have any insights into why this change could be causing issues? Seems like this hardly justifies being called a feature. Codex already supports symlinks for ~/.codex/AGENTS.md so this just seems like an oversight?

@lherron
Copy link

lherron commented Jan 3, 2026

Seriously why is this closed and not merged? Why wouldn't we be allowed to symlink to skills?

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.

Allow symlinked Skill folders/files

3 participants