feat (suggestion): add shared skills settings#1085
feat (suggestion): add shared skills settings#1085JustYannicc wants to merge 3 commits intopingdotgg:mainfrom
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment Tip You can customize the tone of the review comments and chat replies.Configure the |
github-actions
bot
added
size:XXL
![macroscopeapp[bot]](https://avatars.githubusercontent.com/in/900172?s=60&v=4){kind=small}
JustYannicc
force-pushed
the
codex/issue-1081-shared-skills
branch
from
a78c30d to
1120ea8
Compare
|
out of scope / big, closing sorry. |
2 participants
What Changed
Implements shared skill management #1081.
This adds a
Skillssection in Settings, similar to Codex, where shared skills can be initialized and managed. On first use, skills are moved into a user-defined shared directory and symlinked back into the harness skill roots so Codex can keep working as usual without needing to know anything about the shared setup.The PR also adds:
Why
The goal is to make local skills more portable across harnesses while keeping Codex behavior unchanged.
I mainly built this for myself so I can keep skills in one shared location and eventually reuse the same setup across tools like Codex and Claude Code. From Codex’s perspective, it should still just see normal skill directories in the places it already expects, because the shared setup is handled through symlinks.
This is tied to #1081, but there is no expectation that this has to be merged. I’m opening it in case the implementation is useful, and I’m completely fine if you decide not to take it.
If there are any changes I have to make, let me know.
UI Changes
Settings Screen
Before:
After:
New Skills Setting Screen
Single Skill clicked in
Checklist
Note
Add shared skills management panel to settings
Skillstab to the settings page, toggled via?tab=skillsURL search param, rendering a newSkillsSettingsPanelcomponent.sharedSkills.tson the server, implementing full lifecycle management: scanning, migrating, symlinking, enabling/disabling, and uninstalling skills shared across Codex and~/.agents/skills.getSharedSkills,getSharedSkillDetail,initializeSharedSkills,setSharedSkillEnabled,uninstallSharedSkill) wired through contracts, the WS server, and the web API client.sharedSkillsReactQuery.tsto fetch state, initialize, toggle, and uninstall skills with automatic cache invalidation.Macroscope summarized f7a3870.