[FEATURE] add kernels skills add to the cli
#278
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
julien-c
changed the title
kernels skill add to the clikernels skills add to the cli
sayakpaul
reviewed
Feb 12, 2026
Member
sayakpaul
left a comment
sayakpaul
left a comment
There was a problem hiding this comment.
Thanks for working on this! Very interesting.
My main feedback is on the doc:
- Let's make what the current skill can do clearer.
- Let's supplement the results we got after applying the kernels.
- Example on how the generated kernels can be used with
get_kernel()so that the integration message it clear. - A note on the output quality (and potentially showing the outputs).
|
|
||
| def add_skill(args: Namespace) -> None: | ||
| if not (args.claude or args.codex or args.opencode or args.dest): | ||
| print( |
Member
There was a problem hiding this comment.
We should be able to do it via exposing a vendor option in the args and then restricting its choices to claude, codex, etc. for example? Do you reckon any disadvantages? The vendor could be an nargs to allow lists as an input.
Author
There was a problem hiding this comment.
It's a makes sense, but this in consistent with other cli tools. like hf and skills
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds a new cli command for installing an agent compatible skill. Things to note:
kernelswill need to vendor this skill. I have added it to docs but not ToC. I think this is the easiest way, but I'm open to suggestions.You can test this out with:
kernels skills --helpkernels skills add --claudeetc.