devondragon / devondragon/ccmagic
Re-add disable-model-invocation to action skills once upstream bugs are fixed
Nobody has claimed this yet.
- Dominant language
- Shell
- Stars
- 6
- Forks
- 0
- Avg merge
- 28m
- Merged PRs (30d)
- 5
Description
Context
In v2.2.0 we removed disable-model-invocation: true from all 25 action skills (push, merge, init, etc.) because the flag is broken for plugin skills in Claude Code.
See commit 1eee461.
Upstream Bugs to Track
All of these need to be resolved before we can re-add the flag:
-
anthropics/claude-code#22345 —
disable-model-invocationis ignored for plugin skills. Skill descriptions are loaded into context regardless, so the flag provides no benefit. -
anthropics/claude-code#24042 — Skills with
disable-model-invocation: truethat appear in autocomplete fail when selected. Users get: "Skill X cannot be used with Skill tool due to disable-model-invocation". The flag blocks both model AND user invocation. -
anthropics/claude-code#18949 — Plugin skills don't appear in slash command autocomplete at all (only
commands/*.mdfiles do). Even typing the full skill name (e.g.,/ccmagic:push) returns "Unknown skill". This means users cannot directly invoke ANY plugin skills via slash commands — they must ask the model to invoke skills on their behalf via the Skill tool. -
anthropics/claude-code#26251 (closed as dup of #24042) — Additional report confirming that
disable-model-invocationprevents user invocation via slash command.
Current State (v2.2.0)
| Invocation Path | Before (with disable-model-invocation) |
After (removed in v2.2.0) |
|---|---|---|
User types /ccmagic:push |
"Unknown skill" (bug #18949) | Still "Unknown skill" (bug #18949) |
| User asks Claude to run a skill | Claude couldn't see it | Claude can now invoke via Skill tool |
| User says "use /ccmagic:push" | Claude refused to invoke | Claude can now invoke it |
Without disable-model-invocation, Claude may auto-trigger action skills like /push, /merge, /init that have side effects. This is the trade-off we accepted — user access to all skills is more important than preventing auto-invocation.
Action Items
- Monitor upstream issues for fixes
- Once #22345 and #24042 are both resolved, re-add
disable-model-invocation: trueto action skills - Once #18949 is resolved, verify that plugin skills appear in autocomplete and can be user-invoked via
/ - Skills to re-flag:
add-backlog,checkpoint,codex-review,complete-task,context-load,context-save,create-features,create-spike,create-tasks,debug,handoff,init,map-codebase,merge,plan,pr,push,quick-start,quick,resume,settings,start-spike,start-task,sync,verify
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing commit 1eee461 and the listed action skills: add-backlog, checkpoint, codex-review, complete-task, context-load, context-save, create-features, create-spike, create-tasks, debug, handoff, init, map-codebase, merge, plan, pr, push, quick-start, quick, resume, settings, start-spike, start-task, sync, and verify. Monitor upstream issues #22345, #24042, and #18949; done means the relevant fixes are available, the flag is restored to those skills, and user invocation and autocomplete work as expected.
Written by the indexing model from the issue text.
Assessment
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100