From b2ec97889f4c2660a3c527670177a9e91502538a Mon Sep 17 00:00:00 2001 From: bmottin Date: Sun, 13 Sep 2026 17:21:30 -0400 Subject: [PATCH] chore(make): drop disable-model-invocation from the new-skill template MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit BDR-019 stripped the key from all 27 skills because `true` blocks model AND orchestrator routing — silently breaking the routing CLAUDE.md describes — while `false` was a no-op noise line. The scaffold kept injecting `true`, so every skill created with `make new-skill` would have shipped unroutable. doctor.sh already covers the other half upstream: its check was inverted to warn on REINTRODUCTION rather than on absence, which is what surfaced this — a new skill from the template would now trip that very guard. --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 7002ce1..3fa4aa3 100644 --- a/Makefile +++ b/Makefile @@ -71,7 +71,7 @@ new-skill: ## Create a new skill scaffold (usage: make new-skill name=myskill) echo "✅ Created agents/$(name).md"; \ else echo "⚠️ agents/$(name).md already exists"; fi @if [ ! -f skills/$(name)/SKILL.md ]; then \ - printf -- '---\nname: $(name)\ndescription: \nargument-hint: \ndisable-model-invocation: true\nallowed-tools: Read, Grep, Glob, Bash\n---\n\nLoad and follow strictly:\n- .claude/agents/$(name).md\n\nExecute on:\n\n$$ARGUMENTS\n' > skills/$(name)/SKILL.md; \ + printf -- '---\nname: $(name)\ndescription: \nargument-hint: \nallowed-tools: Read, Grep, Glob, Bash\n---\n\nLoad and follow strictly:\n- .claude/agents/$(name).md\n\nExecute on:\n\n$$ARGUMENTS\n' > skills/$(name)/SKILL.md; \ echo "✅ Created skills/$(name)/SKILL.md"; \ else echo "⚠️ skills/$(name)/SKILL.md already exists"; fi @echo " Edit both files, then run: bash link.sh"