mirror of
https://github.com/EveryInc/compound-engineering-plugin.git
synced 2026-06-19 15:41:46 +02:00
Trevin Chow
9c3954a46a
Standardize all 37 compound-engineering skills and 49 agents on a `ce-` hyphen prefix, eliminating namespace collisions with other plugins and removing the colon character that required filesystem sanitization. Key changes: - `ce:plan` → `ce-plan`, `git-commit` → `ce-commit`, etc. (29 skill directory renames + 8 frontmatter-only updates) - `adversarial-reviewer` → `ce-adversarial-reviewer`, etc. (49 agent file renames + frontmatter updates) - `ce-review` → `ce-code-review` (clearer purpose) - `ce-document-review` → `ce-doc-review` (more concise) - Agent references simplified: `compound-engineering:review:X` → `review:ce-X` - All bare agent names in prose now carry `ce-` prefix - Codex converter: remove workflow prompt wrappers (skills are directly invocable), replace `isCanonicalCodexWorkflowSkill` prefix check with direct skill copying, map deprecated `workflows:*` aliases to skill targets instead of dead prompt targets - Remove `codexPrompt` field from types/parser (no longer needed) - Path sanitization tests use non-CE examples to preserve colon coverage - New invariant test: no CE skill name contains a colon BREAKING CHANGE: All skill and agent names changed. `/ce:plan` is now `/ce-plan`, `/ce:review` is now `/ce-code-review`, agent references use `<category>:ce-<name>` format instead of `compound-engineering:<category>:<name>`. Closes #337 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
47 lines
1.5 KiB
TypeScript
47 lines
1.5 KiB
TypeScript
import { describe, expect, test } from "bun:test"
|
|
import path from "path"
|
|
import { loadClaudePlugin } from "../src/parsers/claude"
|
|
import { sanitizePathName } from "../src/utils/files"
|
|
|
|
const pluginRoot = path.join(process.cwd(), "plugins", "compound-engineering")
|
|
|
|
describe("sanitizePathName", () => {
|
|
test("replaces colons with hyphens", () => {
|
|
expect(sanitizePathName("other:skill")).toBe("other-skill")
|
|
expect(sanitizePathName("other:tool")).toBe("other-tool")
|
|
})
|
|
|
|
test("no CE skill name contains a colon", async () => {
|
|
const plugin = await loadClaudePlugin(pluginRoot)
|
|
for (const skill of plugin.skills) {
|
|
expect(skill.name).not.toContain(":")
|
|
}
|
|
})
|
|
|
|
test("passes through names without colons", () => {
|
|
expect(sanitizePathName("frontend-design")).toBe("frontend-design")
|
|
})
|
|
|
|
test("handles multiple colons", () => {
|
|
expect(sanitizePathName("a:b:c")).toBe("a-b-c")
|
|
})
|
|
})
|
|
|
|
describe("path sanitization collision detection", () => {
|
|
test("no two skill names collide after sanitization", async () => {
|
|
const plugin = await loadClaudePlugin(pluginRoot)
|
|
const sanitized = plugin.skills.map((skill) => sanitizePathName(skill.name))
|
|
const unique = new Set(sanitized)
|
|
|
|
expect(unique.size).toBe(sanitized.length)
|
|
})
|
|
|
|
test("no two agent names collide after sanitization", async () => {
|
|
const plugin = await loadClaudePlugin(pluginRoot)
|
|
const sanitized = plugin.agents.map((agent) => sanitizePathName(agent.name))
|
|
const unique = new Set(sanitized)
|
|
|
|
expect(unique.size).toBe(sanitized.length)
|
|
})
|
|
})
|