Files
claude-engineering-plugin/tests/claude-parser.test.ts
Kieran Klaassen f744b797ef 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>
2026-02-08 22:28:51 -06:00

112 lines
4.9 KiB
TypeScript

import { describe, expect, test } from "bun:test"
import path from "path"
import { loadClaudePlugin } from "../src/parsers/claude"
const fixtureRoot = path.join(import.meta.dir, "fixtures", "sample-plugin")
const mcpFixtureRoot = path.join(import.meta.dir, "fixtures", "mcp-file")
const customPathsRoot = path.join(import.meta.dir, "fixtures", "custom-paths")
const invalidCommandPathRoot = path.join(import.meta.dir, "fixtures", "invalid-command-path")
const invalidHooksPathRoot = path.join(import.meta.dir, "fixtures", "invalid-hooks-path")
const invalidMcpPathRoot = path.join(import.meta.dir, "fixtures", "invalid-mcp-path")
describe("loadClaudePlugin", () => {
test("loads manifest, agents, commands, skills, hooks", async () => {
const plugin = await loadClaudePlugin(fixtureRoot)
expect(plugin.manifest.name).toBe("compound-engineering")
expect(plugin.agents.length).toBe(2)
expect(plugin.commands.length).toBe(7)
expect(plugin.skills.length).toBe(2)
expect(plugin.hooks).toBeDefined()
expect(plugin.mcpServers).toBeDefined()
const researchAgent = plugin.agents.find((agent) => agent.name === "repo-research-analyst")
expect(researchAgent?.capabilities).toEqual(["Capability A", "Capability B"])
const reviewCommand = plugin.commands.find((command) => command.name === "workflows:review")
expect(reviewCommand?.allowedTools).toEqual([
"Read",
"Write",
"Edit",
"Bash(ls:*)",
"Bash(git:*)",
"Grep",
"Glob",
"List",
"Patch",
"Task",
])
const planReview = plugin.commands.find((command) => command.name === "plan_review")
expect(planReview?.allowedTools).toEqual(["Read", "Edit"])
const skillCommand = plugin.commands.find((command) => command.name === "create-agent-skill")
expect(skillCommand?.allowedTools).toEqual(["Skill(create-agent-skills)"])
const modelCommand = plugin.commands.find((command) => command.name === "workflows:work")
expect(modelCommand?.allowedTools).toEqual(["WebFetch"])
const patternCommand = plugin.commands.find((command) => command.name === "report-bug")
expect(patternCommand?.allowedTools).toEqual(["Read(.env)", "Bash(git:*)"])
const planCommand = plugin.commands.find((command) => command.name === "workflows:plan")
expect(planCommand?.allowedTools).toEqual(["Question", "TodoWrite", "TodoRead"])
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")
})
test("merges default and custom component paths", async () => {
const plugin = await loadClaudePlugin(customPathsRoot)
expect(plugin.agents.map((agent) => agent.name).sort()).toEqual(["custom-agent", "default-agent"])
expect(plugin.commands.map((command) => command.name).sort()).toEqual(["custom-command", "default-command"])
expect(plugin.skills.map((skill) => skill.name).sort()).toEqual(["custom-skill", "default-skill"])
expect(plugin.hooks?.hooks.PreToolUse?.[0]?.hooks[0]?.command).toBe("echo default")
expect(plugin.hooks?.hooks.PostToolUse?.[0]?.hooks[0]?.command).toBe("echo custom")
})
test("rejects custom component paths that escape the plugin root", async () => {
await expect(loadClaudePlugin(invalidCommandPathRoot)).rejects.toThrow(
"Invalid commands path: ../outside-commands. Paths must stay within the plugin root.",
)
})
test("rejects hook paths that escape the plugin root", async () => {
await expect(loadClaudePlugin(invalidHooksPathRoot)).rejects.toThrow(
"Invalid hooks path: ../outside-hooks.json. Paths must stay within the plugin root.",
)
})
test("rejects MCP paths that escape the plugin root", async () => {
await expect(loadClaudePlugin(invalidMcpPathRoot)).rejects.toThrow(
"Invalid mcpServers path: ../outside-mcp.json. Paths must stay within the plugin root.",
)
})
})