feat(cline): add native skills install support#1033
Conversation
Ship Cline install docs, a symlink-based skills installer, and release component mapping so CE skills load through Cline's native discovery paths without a Bun converter target.
|
cc: @tmchow |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c5cb436147
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
|
@Ayush7614 please resolve conflicts |
Resolve conflicts with merged Antigravity install changes and address Codex review: omit skills with disable-model-invocation from the Cline symlink installer since Cline auto-activates from description matching.
|
@tmchow Conflicts resolved — merged latest Also addressed the Codex P2 on Docs updated in Validation: |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3563b96709
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c478a4b2a2
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
Address Codex reviews: default install skips manual-only skills without promising slash-command access; add --include-manual opt-in with auto-activation caveats. Default reruns now remove stale manual-only symlinks left from prior --include-manual installs.
Ayush7614
force-pushed
the
feat/cline-native-skills-install
branch
from
c478a4b to
90c84e4
Compare
|
cc: @tmchow |
| if [[ "$INCLUDE_MANUAL" != "true" ]]; then | ||
| target="$DEST/$name" | ||
| if [[ -L "$target" ]]; then | ||
| rm "$target" |
There was a problem hiding this comment.
One thing I think we should tighten before merging: this removes any symlink with a manual only skill name, even if that link was not created by this installer. For example, a user could already have ~/.cline/skills/lfg pointing at their own local skill, then a normal CE install would delete it.
Can we make pruning ownership aware? A manifest would be cleanest. A smaller fix would be to only remove the link when its target resolves under this checkout's skills/ directory. That keeps the safety gate for CE manual skills without touching someone else's Cline setup.
When omitting manual-only skills, remove stale links only when the symlink target resolves under this checkout's skills/ directory so unrelated ~/.cline/skills/<name> links are preserved. Add script tests.
|
Addressed @tmchow's review in Manual-only pruning is now ownership-aware. On a default install (without Added cc: @tmchow |
2 participants
Summary
--to clineconverter), following the same strategy used for Kimi and OpenCode.cline/scripts/install-skills.shto symlink CE'sskills/directories into~/.cline/skills/or.cline/skills/.cline/INSTALL.mdanddocs/specs/cline.mdApproach
Cline's extension and CLI load CE through on-demand
SKILL.mdskills, not through a separate plugin manifest. This PR intentionally avoids a converter target (similar to how #997 was superseded by native Kimi manifests in #998).Users enable Settings -> Features -> Enable Skills in Cline, run the install script from a checkout, and start a new task.
Test plan
./.cline/scripts/install-skills.sh --globallinks all CE skills (smoke-tested locally)bun run release:validatepassesbun test tests/release-components.test.tspasses