feat(skills): support user-invocable on claudecode and at the rulesync root with per-target overrides - #1961
Merged
Conversation
…om root - Add user-invocable to ClaudecodeSkillFrontmatterSchema with bidirectional passthrough via the rulesync claudecode section (Claude Code documents this field at https://code.claude.com/docs/en/skills). - Promote user-invocable to the rulesync skill frontmatter root and propagate to claudecode, qwencode, vibe, factorydroid (the four targets whose upstream spec recognizes the field). Per-target sections can still override the root default. - Introduce resolveUserInvocable helper mirroring resolveDisableModelInvocation; wire it into the four compatible tools' fromRulesyncSkill conversions. - Fix factorydroid passthrough that was schema-only (the field was accepted but dropped on conversion to/from rulesync). - Document the root field and override semantics in docs/reference/file-formats.md and the synced skills/rulesync/file-formats.md.
Owner
|
Merged. Thanks! Filed #1967 to track the minor docs follow-ups (vibe example, factorydroid verification). |
shaneholloman
pushed a commit
to shaneholloman/rulesync
that referenced
this pull request
Jun 20, 2026
Follow-up from PR dyoshikawa#1961: the skills frontmatter example listed user-invocable for claudecode/factorydroid/qwencode but had no vibe: section, even though vibe supports user-invocable (and allowed-tools). Add a vibe example for consistency. Also confirmed Factory Droid honors user-invocable per its official docs (https://docs.factory.ai/cli/configuration/skills), so no schema change is needed there.
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
2 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.
Claude Code's skill spec documents a
user-invocable: falseflag that hides a skill from the/menu while keeping it model-invokable (https://code.claude.com/docs/en/skills). rulesync already routed this field throughqwencode,vibe, andfactorydroid, but it was missing fromclaudecodeitself — and from the rulesync root.Users can now set
user-invocableonce at the rulesync skill frontmatter root and have it applied to every supported tool (claudecode,qwencode,vibe,factorydroid). A value set inside a tool section still wins, so single-target overrides remain possible — same pattern asdisable-model-invocation.Also fixes a small
factorydroidbug whereuser-invocablewas accepted in the SKILL.md schema but silently dropped on conversion to/from rulesync.