Reduce context token usage by 79% — fix silent component exclusion (#161)
* Update create-agent-skills to match 2026 official docs, add /triage-prs command - Rewrite SKILL.md to document that commands and skills are now merged - Add new frontmatter fields: disable-model-invocation, user-invocable, context, agent - Add invocation control table and dynamic context injection docs - Fix skill-structure.md: was incorrectly recommending XML tags over markdown headings - Update official-spec.md with complete 2026 specification - Add local /triage-prs command for PR triage workflow - Add PR triage plan document Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * [2.31.0] Reduce context token usage by 79%, include recent community contributions The plugin was consuming 316% of Claude Code's description character budget (~50,500 chars vs 16,000 limit), causing components to be silently excluded. Now at 65% (~10,400 chars) with all components visible. Changes: - Trim all 29 agent descriptions (move examples to body) - Add disable-model-invocation to 18 manual commands - Add disable-model-invocation to 6 manual skills - Include recent community contributions in changelog - Fix component counts (29 agents, 24 commands, 18 skills) Contributors: @trevin, @terryli, @robertomello, @zacwilliams, @aarnikoskela, @samxie, @davidalley Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix: keep disable-model-invocation off commands called by /lfg, rename xcode-test - Remove disable-model-invocation from test-browser, feature-video, resolve_todo_parallel — these are called programmatically by /lfg and /slfg - Rename xcode-test to test-xcode to match test-browser naming convention Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Fix: keep git-worktree skill auto-invocable (used by /workflows:work) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * feat(converter): support disable-model-invocation frontmatter Parse disable-model-invocation from command and skill frontmatter. Commands/skills with this flag are excluded from OpenCode command maps and Codex prompt/skill generation, matching Claude Code behavior where these components are user-only invocable. Bump converter version to 0.3.0. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -15,8 +15,8 @@ describe("loadClaudePlugin", () => {
|
||||
|
||||
expect(plugin.manifest.name).toBe("compound-engineering")
|
||||
expect(plugin.agents.length).toBe(2)
|
||||
expect(plugin.commands.length).toBe(6)
|
||||
expect(plugin.skills.length).toBe(1)
|
||||
expect(plugin.commands.length).toBe(7)
|
||||
expect(plugin.skills.length).toBe(2)
|
||||
expect(plugin.hooks).toBeDefined()
|
||||
expect(plugin.mcpServers).toBeDefined()
|
||||
|
||||
@@ -55,6 +55,28 @@ describe("loadClaudePlugin", () => {
|
||||
expect(plugin.mcpServers?.context7?.url).toBe("https://mcp.context7.com/mcp")
|
||||
})
|
||||
|
||||
test("parses disable-model-invocation from commands", async () => {
|
||||
const plugin = await loadClaudePlugin(fixtureRoot)
|
||||
|
||||
const disabledCommand = plugin.commands.find((command) => command.name === "deploy-docs")
|
||||
expect(disabledCommand).toBeDefined()
|
||||
expect(disabledCommand?.disableModelInvocation).toBe(true)
|
||||
|
||||
const normalCommand = plugin.commands.find((command) => command.name === "workflows:review")
|
||||
expect(normalCommand?.disableModelInvocation).toBeUndefined()
|
||||
})
|
||||
|
||||
test("parses disable-model-invocation from skills", async () => {
|
||||
const plugin = await loadClaudePlugin(fixtureRoot)
|
||||
|
||||
const disabledSkill = plugin.skills.find((skill) => skill.name === "disabled-skill")
|
||||
expect(disabledSkill).toBeDefined()
|
||||
expect(disabledSkill?.disableModelInvocation).toBe(true)
|
||||
|
||||
const normalSkill = plugin.skills.find((skill) => skill.name === "skill-one")
|
||||
expect(normalSkill?.disableModelInvocation).toBeUndefined()
|
||||
})
|
||||
|
||||
test("loads MCP servers from .mcp.json when manifest is empty", async () => {
|
||||
const plugin = await loadClaudePlugin(mcpFixtureRoot)
|
||||
expect(plugin.mcpServers?.remote?.url).toBe("https://example.com/stream")
|
||||
|
||||
@@ -172,6 +172,44 @@ Don't confuse with file paths like /tmp/output.md or /dev/null.`,
|
||||
expect(parsed.body).toContain("/dev/null")
|
||||
})
|
||||
|
||||
test("excludes commands with disable-model-invocation from prompts and skills", () => {
|
||||
const plugin: ClaudePlugin = {
|
||||
...fixturePlugin,
|
||||
commands: [
|
||||
{
|
||||
name: "normal-command",
|
||||
description: "Normal command",
|
||||
body: "Normal body.",
|
||||
sourcePath: "/tmp/plugin/commands/normal.md",
|
||||
},
|
||||
{
|
||||
name: "disabled-command",
|
||||
description: "Disabled command",
|
||||
disableModelInvocation: true,
|
||||
body: "Disabled body.",
|
||||
sourcePath: "/tmp/plugin/commands/disabled.md",
|
||||
},
|
||||
],
|
||||
agents: [],
|
||||
skills: [],
|
||||
}
|
||||
|
||||
const bundle = convertClaudeToCodex(plugin, {
|
||||
agentMode: "subagent",
|
||||
inferTemperature: false,
|
||||
permissions: "none",
|
||||
})
|
||||
|
||||
// Only normal command should produce a prompt
|
||||
expect(bundle.prompts).toHaveLength(1)
|
||||
expect(bundle.prompts[0].name).toBe("normal-command")
|
||||
|
||||
// Only normal command should produce a generated skill
|
||||
const commandSkills = bundle.generatedSkills.filter((s) => s.name === "normal-command" || s.name === "disabled-command")
|
||||
expect(commandSkills).toHaveLength(1)
|
||||
expect(commandSkills[0].name).toBe("normal-command")
|
||||
})
|
||||
|
||||
test("truncates generated skill descriptions to Codex limits and single line", () => {
|
||||
const longDescription = `Line one\nLine two ${"a".repeat(2000)}`
|
||||
const plugin: ClaudePlugin = {
|
||||
|
||||
@@ -168,4 +168,19 @@ describe("convertClaudeToOpenCode", () => {
|
||||
const parsed = parseFrontmatter(agentFile!.content)
|
||||
expect(parsed.data.mode).toBe("primary")
|
||||
})
|
||||
|
||||
test("excludes commands with disable-model-invocation from command map", async () => {
|
||||
const plugin = await loadClaudePlugin(fixtureRoot)
|
||||
const bundle = convertClaudeToOpenCode(plugin, {
|
||||
agentMode: "subagent",
|
||||
inferTemperature: false,
|
||||
permissions: "none",
|
||||
})
|
||||
|
||||
// deploy-docs has disable-model-invocation: true, should be excluded
|
||||
expect(bundle.config.command?.["deploy-docs"]).toBeUndefined()
|
||||
|
||||
// Normal commands should still be present
|
||||
expect(bundle.config.command?.["workflows:review"]).toBeDefined()
|
||||
})
|
||||
})
|
||||
|
||||
7
tests/fixtures/sample-plugin/commands/disabled-command.md
vendored
Normal file
7
tests/fixtures/sample-plugin/commands/disabled-command.md
vendored
Normal file
@@ -0,0 +1,7 @@
|
||||
---
|
||||
name: deploy-docs
|
||||
description: Deploy documentation site
|
||||
disable-model-invocation: true
|
||||
---
|
||||
|
||||
Deploy docs body.
|
||||
7
tests/fixtures/sample-plugin/skills/disabled-skill/SKILL.md
vendored
Normal file
7
tests/fixtures/sample-plugin/skills/disabled-skill/SKILL.md
vendored
Normal file
@@ -0,0 +1,7 @@
|
||||
---
|
||||
name: disabled-skill
|
||||
description: A skill with model invocation disabled
|
||||
disable-model-invocation: true
|
||||
---
|
||||
|
||||
Disabled skill body.
|
||||
Reference in New Issue
Block a user