refactor(ragkit) 0.1.4: drop commands/, skills user-invocable (fix Record-2 improvisation)#87
Merged
Merged
Conversation
…(fix Record-2 improvisation) Claude Code merged commands into skills — a default (no `user-invocable`) skill already provides both the `/ragkit:*` slash entry AND model auto-invocation. So: - deleted the 4 thin `commands/*.md` wrappers - removed `user-invocable: false` from the 4 skills (embed/query/status/eval) `/ragkit:query` etc. are unchanged for users — now served directly by the skill. This ROOT-FIXES the Record-2 failure: the old command body said "Follow the plugin skill at `skills/query/SKILL.md`" (a relative path), which led a weak model on CodeBuddy to Search the PROJECT dir for that file, find nothing (it's in the plugin cache), abandon the plugin, and reimplement retrieval manually (install numpy, call the embedding API, hand-roll cosine+RRF → wrong results). Dropping the command layer removes that "go find the file" indirection — the model can only invoke the skill by name. specode's by-name `ragkit:query` calls are unaffected; the `__env_probe__` probe remains (pending CodeBuddy run). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Co-Authored-By: qxbyte <126648675+qxbyte@users.noreply.github.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Claude Code merged commands into skills — a default skill already gives both the
/ragkit:*slash AND model auto-invocation. So this drops the redundant command layer:commands/*.mduser-invocable: falsefrom the 4 skillsRoot-fixes the Record-2 improvisation. The old command body said
Follow the plugin skill at skills/query/SKILL.md(a relative path) → a weak model on CodeBuddySearched the PROJECT dir for that file, found nothing (it lives in the plugin cache), abandoned the plugin, and reimplemented retrieval by hand (numpy + embedding API + hand-rolled cosine/RRF → wrong, non-reproducible results). No command layer → no "go find the file" indirection → the model can only invoke the skill by name./ragkit:queryetc. unchanged for users. specode's by-nameragkit:queryunaffected.__env_probe__probe retained (pending CodeBuddy run). Validation passes.🤖 Generated with Claude Code