@every-env/compound-plugin 0.9.0 → 2.34.2

package/tests/cli.test.ts

@@ -426,4 +426,184 @@ describe("CLI", () => {
     expect(await exists(path.join(piRoot, "prompts", "workflows-review.md"))).toBe(true)
     expect(await exists(path.join(piRoot, "extensions", "compound-engineering-compat.ts"))).toBe(true)
   })
+
+  test("install --to opencode uses permissions:none by default", async () => {
+    const tempRoot = await fs.mkdtemp(path.join(os.tmpdir(), "cli-perms-none-"))
+    const fixtureRoot = path.join(import.meta.dir, "fixtures", "sample-plugin")
+
+    const proc = Bun.spawn([
+      "bun",
+      "run",
+      "src/index.ts",
+      "install",
+      fixtureRoot,
+      "--to",
+      "opencode",
+      "--output",
+      tempRoot,
+    ], {
+      cwd: path.join(import.meta.dir, ".."),
+      stdout: "pipe",
+      stderr: "pipe",
+    })
+
+    const exitCode = await proc.exited
+    const stdout = await new Response(proc.stdout).text()
+    const stderr = await new Response(proc.stderr).text()
+
+    if (exitCode !== 0) {
+      throw new Error(`CLI failed (exit ${exitCode}).\nstdout: ${stdout}\nstderr: ${stderr}`)
+    }
+
+    expect(stdout).toContain("Installed compound-engineering")
+
+    const opencodeJsonPath = path.join(tempRoot, "opencode.json")
+    const content = await fs.readFile(opencodeJsonPath, "utf-8")
+    const json = JSON.parse(content)
+
+    expect(json).not.toHaveProperty("permission")
+    expect(json).not.toHaveProperty("tools")
+  })
+
+  test("install --to opencode --permissions broad writes permission block", async () => {
+    const tempRoot = await fs.mkdtemp(path.join(os.tmpdir(), "cli-perms-broad-"))
+    const fixtureRoot = path.join(import.meta.dir, "fixtures", "sample-plugin")
+
+    const proc = Bun.spawn([
+      "bun",
+      "run",
+      "src/index.ts",
+      "install",
+      fixtureRoot,
+      "--to",
+      "opencode",
+      "--permissions",
+      "broad",
+      "--output",
+      tempRoot,
+    ], {
+      cwd: path.join(import.meta.dir, ".."),
+      stdout: "pipe",
+      stderr: "pipe",
+    })
+
+    const exitCode = await proc.exited
+    const stdout = await new Response(proc.stdout).text()
+    const stderr = await new Response(proc.stderr).text()
+
+    if (exitCode !== 0) {
+      throw new Error(`CLI failed (exit ${exitCode}).\nstdout: ${stdout}\nstderr: ${stderr}`)
+    }
+
+    expect(stdout).toContain("Installed compound-engineering")
+
+    const opencodeJsonPath = path.join(tempRoot, "opencode.json")
+    const content = await fs.readFile(opencodeJsonPath, "utf-8")
+    const json = JSON.parse(content)
+
+    expect(json).toHaveProperty("permission")
+    expect(json.permission).not.toBeNull()
+  })
+
+  test("sync --target all detects new sync targets and ignores stale cursor directories", async () => {
+    const tempHome = await fs.mkdtemp(path.join(os.tmpdir(), "cli-sync-home-"))
+    const tempCwd = await fs.mkdtemp(path.join(os.tmpdir(), "cli-sync-cwd-"))
+    const repoRoot = path.join(import.meta.dir, "..")
+    const fixtureSkillDir = path.join(import.meta.dir, "fixtures", "sample-plugin", "skills", "skill-one")
+    const claudeSkillsDir = path.join(tempHome, ".claude", "skills", "skill-one")
+    const claudeCommandsDir = path.join(tempHome, ".claude", "commands", "workflows")
+
+    await fs.mkdir(path.dirname(claudeSkillsDir), { recursive: true })
+    await fs.cp(fixtureSkillDir, claudeSkillsDir, { recursive: true })
+    await fs.mkdir(claudeCommandsDir, { recursive: true })
+    await fs.writeFile(
+      path.join(claudeCommandsDir, "plan.md"),
+      [
+        "---",
+        "name: workflows:plan",
+        "description: Plan work",
+        "argument-hint: \"[goal]\"",
+        "---",
+        "",
+        "Plan the work.",
+      ].join("\n"),
+    )
+    await fs.writeFile(
+      path.join(tempHome, ".claude", "settings.json"),
+      JSON.stringify({
+        mcpServers: {
+          local: { command: "echo", args: ["hello"] },
+          remote: { url: "https://example.com/mcp" },
+          legacy: { type: "sse", url: "https://example.com/sse" },
+        },
+      }, null, 2),
+    )
+
+    await fs.mkdir(path.join(tempHome, ".config", "opencode"), { recursive: true })
+    await fs.mkdir(path.join(tempHome, ".codex"), { recursive: true })
+    await fs.mkdir(path.join(tempHome, ".pi"), { recursive: true })
+    await fs.mkdir(path.join(tempHome, ".factory"), { recursive: true })
+    await fs.mkdir(path.join(tempHome, ".copilot"), { recursive: true })
+    await fs.mkdir(path.join(tempHome, ".gemini"), { recursive: true })
+    await fs.mkdir(path.join(tempHome, ".codeium", "windsurf"), { recursive: true })
+    await fs.mkdir(path.join(tempHome, ".kiro"), { recursive: true })
+    await fs.mkdir(path.join(tempHome, ".qwen"), { recursive: true })
+    await fs.mkdir(path.join(tempHome, ".openclaw"), { recursive: true })
+    await fs.mkdir(path.join(tempCwd, ".cursor"), { recursive: true })
+
+    const proc = Bun.spawn([
+      "bun",
+      "run",
+      path.join(repoRoot, "src", "index.ts"),
+      "sync",
+      "--target",
+      "all",
+    ], {
+      cwd: tempCwd,
+      stdout: "pipe",
+      stderr: "pipe",
+      env: {
+        ...process.env,
+        HOME: tempHome,
+      },
+    })
+
+    const exitCode = await proc.exited
+    const stdout = await new Response(proc.stdout).text()
+    const stderr = await new Response(proc.stderr).text()
+
+    if (exitCode !== 0) {
+      throw new Error(`CLI failed (exit ${exitCode}).\nstdout: ${stdout}\nstderr: ${stderr}`)
+    }
+
+    expect(stdout).toContain("Synced to codex")
+    expect(stdout).toContain("Synced to opencode")
+    expect(stdout).toContain("Synced to pi")
+    expect(stdout).toContain("Synced to droid")
+    expect(stdout).toContain("Synced to windsurf")
+    expect(stdout).toContain("Synced to kiro")
+    expect(stdout).toContain("Synced to qwen")
+    expect(stdout).toContain("Synced to openclaw")
+    expect(stdout).toContain("Synced to copilot")
+    expect(stdout).toContain("Synced to gemini")
+    expect(stdout).not.toContain("cursor")
+
+    expect(await exists(path.join(tempHome, ".config", "opencode", "commands", "workflows:plan.md"))).toBe(true)
+    expect(await exists(path.join(tempHome, ".codex", "config.toml"))).toBe(true)
+    expect(await exists(path.join(tempHome, ".codex", "prompts", "workflows-plan.md"))).toBe(true)
+    expect(await exists(path.join(tempHome, ".codex", "skills", "workflows-plan", "SKILL.md"))).toBe(true)
+    expect(await exists(path.join(tempHome, ".pi", "agent", "prompts", "workflows-plan.md"))).toBe(true)
+    expect(await exists(path.join(tempHome, ".factory", "commands", "plan.md"))).toBe(true)
+    expect(await exists(path.join(tempHome, ".codeium", "windsurf", "mcp_config.json"))).toBe(true)
+    expect(await exists(path.join(tempHome, ".codeium", "windsurf", "global_workflows", "workflows-plan.md"))).toBe(true)
+    expect(await exists(path.join(tempHome, ".kiro", "settings", "mcp.json"))).toBe(true)
+    expect(await exists(path.join(tempHome, ".kiro", "skills", "workflows-plan", "SKILL.md"))).toBe(true)
+    expect(await exists(path.join(tempHome, ".qwen", "settings.json"))).toBe(true)
+    expect(await exists(path.join(tempHome, ".qwen", "commands", "workflows", "plan.md"))).toBe(true)
+    expect(await exists(path.join(tempHome, ".copilot", "mcp-config.json"))).toBe(true)
+    expect(await exists(path.join(tempHome, ".copilot", "skills", "workflows-plan", "SKILL.md"))).toBe(true)
+    expect(await exists(path.join(tempHome, ".gemini", "settings.json"))).toBe(true)
+    expect(await exists(path.join(tempHome, ".gemini", "commands", "workflows", "plan.toml"))).toBe(true)
+    expect(await exists(path.join(tempHome, ".openclaw", "skills", "skill-one"))).toBe(true)
+  })
 })

package/tests/converter.test.ts

@@ -8,7 +8,7 @@ import type { ClaudePlugin } from "../src/types/claude"
 const fixtureRoot = path.join(import.meta.dir, "fixtures", "sample-plugin")
 
 describe("convertClaudeToOpenCode", () => {
-  test("maps commands, permissions, and agents", async () => {
+  test("from-command mode: map allowedTools to global permission block", async () => {
     const plugin = await loadClaudePlugin(fixtureRoot)
     const bundle = convertClaudeToOpenCode(plugin, {
       agentMode: "subagent",
@@ -16,8 +16,9 @@ describe("convertClaudeToOpenCode", () => {
       permissions: "from-commands",
     })
 
-    expect(bundle.config.command?.["workflows:review"]).toBeDefined()
-    expect(bundle.config.command?.["plan_review"]).toBeDefined()
+    expect(bundle.config.command).toBeUndefined()
+    expect(bundle.commandFiles.find((f) => f.name === "workflows:review")).toBeDefined()
+    expect(bundle.commandFiles.find((f) => f.name === "plan_review")).toBeDefined()
 
     const permission = bundle.config.permission as Record<string, string | Record<string, string>>
     expect(Object.keys(permission).sort()).toEqual([
@@ -71,8 +72,10 @@ describe("convertClaudeToOpenCode", () => {
     expect(parsed.data.model).toBe("anthropic/claude-sonnet-4-20250514")
     expect(parsed.data.temperature).toBe(0.1)
 
-    const modelCommand = bundle.config.command?.["workflows:work"]
-    expect(modelCommand?.model).toBe("openai/gpt-4o")
+    const modelCommand = bundle.commandFiles.find((f) => f.name === "workflows:work")
+    expect(modelCommand).toBeDefined()
+    const commandParsed = parseFrontmatter(modelCommand!.content)
+    expect(commandParsed.data.model).toBe("openai/gpt-4o")
   })
 
   test("resolves bare Claude model aliases to full IDs", () => {
@@ -199,7 +202,7 @@ describe("convertClaudeToOpenCode", () => {
     expect(parsed.data.mode).toBe("primary")
   })
 
-  test("excludes commands with disable-model-invocation from command map", async () => {
+  test("excludes commands with disable-model-invocation from commandFiles", async () => {
     const plugin = await loadClaudePlugin(fixtureRoot)
     const bundle = convertClaudeToOpenCode(plugin, {
       agentMode: "subagent",
@@ -208,10 +211,10 @@ describe("convertClaudeToOpenCode", () => {
     })
 
     // deploy-docs has disable-model-invocation: true, should be excluded
-    expect(bundle.config.command?.["deploy-docs"]).toBeUndefined()
+    expect(bundle.commandFiles.find((f) => f.name === "deploy-docs")).toBeUndefined()
 
     // Normal commands should still be present
-    expect(bundle.config.command?.["workflows:review"]).toBeDefined()
+    expect(bundle.commandFiles.find((f) => f.name === "workflows:review")).toBeDefined()
   })
 
   test("rewrites .claude/ paths to .opencode/ in command bodies", () => {
@@ -240,10 +243,11 @@ Run \`/compound-engineering-setup\` to create a settings file.`,
       permissions: "none",
     })
 
-    const template = bundle.config.command?.["review"]?.template ?? ""
+    const commandFile = bundle.commandFiles.find((f) => f.name === "review")
+    expect(commandFile).toBeDefined()
 
     // Tool-agnostic path in project root — no rewriting needed
-    expect(template).toContain("compound-engineering.local.md")
+    expect(commandFile!.content).toContain("compound-engineering.local.md")
   })
 
   test("rewrites .claude/ paths in agent bodies", () => {
@@ -273,4 +277,33 @@ Run \`/compound-engineering-setup\` to create a settings file.`,
     // Tool-agnostic path in project root — no rewriting needed
     expect(agentFile!.content).toContain("compound-engineering.local.md")
   })
+
+  test("command .md files include description in frontmatter", () => {
+    const plugin: ClaudePlugin = {
+      root: "/tmp/plugin",
+      manifest: { name: "fixture", version: "1.0.0" },
+      agents: [],
+      commands: [
+        {
+          name: "test-cmd",
+          description: "Test description",
+          body: "Do the thing",
+          sourcePath: "/tmp/plugin/commands/test-cmd.md",
+        },
+      ],
+      skills: [],
+    }
+
+    const bundle = convertClaudeToOpenCode(plugin, {
+      agentMode: "subagent",
+      inferTemperature: false,
+      permissions: "none",
+    })
+
+    const commandFile = bundle.commandFiles.find((f) => f.name === "test-cmd")
+    expect(commandFile).toBeDefined()
+    const parsed = parseFrontmatter(commandFile!.content)
+    expect(parsed.data.description).toBe("Test description")
+    expect(parsed.body).toContain("Do the thing")
+  })
 })

package/tests/detect-tools.test.ts

@@ -0,0 +1,119 @@
+import { describe, expect, test } from "bun:test"
+import { promises as fs } from "fs"
+import path from "path"
+import os from "os"
+import { detectInstalledTools, getDetectedTargetNames } from "../src/utils/detect-tools"
+
+describe("detectInstalledTools", () => {
+  test("detects tools when config directories exist", async () => {
+    const tempHome = await fs.mkdtemp(path.join(os.tmpdir(), "detect-tools-"))
+    const tempCwd = await fs.mkdtemp(path.join(os.tmpdir(), "detect-tools-cwd-"))
+
+    // Create directories for some tools
+    await fs.mkdir(path.join(tempHome, ".codex"), { recursive: true })
+    await fs.mkdir(path.join(tempHome, ".codeium", "windsurf"), { recursive: true })
+    await fs.mkdir(path.join(tempHome, ".gemini"), { recursive: true })
+    await fs.mkdir(path.join(tempHome, ".copilot"), { recursive: true })
+
+    const results = await detectInstalledTools(tempHome, tempCwd)
+
+    const codex = results.find((t) => t.name === "codex")
+    expect(codex?.detected).toBe(true)
+    expect(codex?.reason).toContain(".codex")
+
+    const windsurf = results.find((t) => t.name === "windsurf")
+    expect(windsurf?.detected).toBe(true)
+    expect(windsurf?.reason).toContain(".codeium/windsurf")
+
+    const gemini = results.find((t) => t.name === "gemini")
+    expect(gemini?.detected).toBe(true)
+    expect(gemini?.reason).toContain(".gemini")
+
+    const copilot = results.find((t) => t.name === "copilot")
+    expect(copilot?.detected).toBe(true)
+    expect(copilot?.reason).toContain(".copilot")
+
+    // Tools without directories should not be detected
+    const opencode = results.find((t) => t.name === "opencode")
+    expect(opencode?.detected).toBe(false)
+
+    const droid = results.find((t) => t.name === "droid")
+    expect(droid?.detected).toBe(false)
+
+    const pi = results.find((t) => t.name === "pi")
+    expect(pi?.detected).toBe(false)
+  })
+
+  test("returns all tools with detected=false when no directories exist", async () => {
+    const tempHome = await fs.mkdtemp(path.join(os.tmpdir(), "detect-empty-"))
+    const tempCwd = await fs.mkdtemp(path.join(os.tmpdir(), "detect-empty-cwd-"))
+
+    const results = await detectInstalledTools(tempHome, tempCwd)
+
+    expect(results.length).toBe(10)
+    for (const tool of results) {
+      expect(tool.detected).toBe(false)
+      expect(tool.reason).toBe("not found")
+    }
+  })
+
+  test("detects home-based tools", async () => {
+    const tempHome = await fs.mkdtemp(path.join(os.tmpdir(), "detect-home-"))
+    const tempCwd = await fs.mkdtemp(path.join(os.tmpdir(), "detect-home-cwd-"))
+
+    await fs.mkdir(path.join(tempHome, ".config", "opencode"), { recursive: true })
+    await fs.mkdir(path.join(tempHome, ".factory"), { recursive: true })
+    await fs.mkdir(path.join(tempHome, ".pi"), { recursive: true })
+    await fs.mkdir(path.join(tempHome, ".openclaw"), { recursive: true })
+
+    const results = await detectInstalledTools(tempHome, tempCwd)
+
+    expect(results.find((t) => t.name === "opencode")?.detected).toBe(true)
+    expect(results.find((t) => t.name === "droid")?.detected).toBe(true)
+    expect(results.find((t) => t.name === "pi")?.detected).toBe(true)
+    expect(results.find((t) => t.name === "openclaw")?.detected).toBe(true)
+  })
+
+  test("detects copilot from project-specific skills without generic .github false positives", async () => {
+    const tempHome = await fs.mkdtemp(path.join(os.tmpdir(), "detect-copilot-home-"))
+    const tempCwd = await fs.mkdtemp(path.join(os.tmpdir(), "detect-copilot-cwd-"))
+
+    await fs.mkdir(path.join(tempCwd, ".github"), { recursive: true })
+
+    let results = await detectInstalledTools(tempHome, tempCwd)
+    expect(results.find((t) => t.name === "copilot")?.detected).toBe(false)
+
+    await fs.mkdir(path.join(tempCwd, ".github", "skills"), { recursive: true })
+
+    results = await detectInstalledTools(tempHome, tempCwd)
+    expect(results.find((t) => t.name === "copilot")?.detected).toBe(true)
+    expect(results.find((t) => t.name === "copilot")?.reason).toContain(".github/skills")
+  })
+})
+
+describe("getDetectedTargetNames", () => {
+  test("returns only names of detected tools", async () => {
+    const tempHome = await fs.mkdtemp(path.join(os.tmpdir(), "detect-names-"))
+    const tempCwd = await fs.mkdtemp(path.join(os.tmpdir(), "detect-names-cwd-"))
+
+    await fs.mkdir(path.join(tempHome, ".codex"), { recursive: true })
+    await fs.mkdir(path.join(tempHome, ".gemini"), { recursive: true })
+
+    const names = await getDetectedTargetNames(tempHome, tempCwd)
+
+    expect(names).toContain("codex")
+    expect(names).toContain("gemini")
+    expect(names).not.toContain("opencode")
+    expect(names).not.toContain("droid")
+    expect(names).not.toContain("pi")
+    expect(names).not.toContain("cursor")
+  })
+
+  test("returns empty array when nothing detected", async () => {
+    const tempHome = await fs.mkdtemp(path.join(os.tmpdir(), "detect-none-"))
+    const tempCwd = await fs.mkdtemp(path.join(os.tmpdir(), "detect-none-cwd-"))
+
+    const names = await getDetectedTargetNames(tempHome, tempCwd)
+    expect(names).toEqual([])
+  })
+})

package/tests/openclaw-converter.test.ts

@@ -0,0 +1,200 @@
+import { describe, expect, test } from "bun:test"
+import { convertClaudeToOpenClaw } from "../src/converters/claude-to-openclaw"
+import { parseFrontmatter } from "../src/utils/frontmatter"
+import type { ClaudePlugin } from "../src/types/claude"
+
+const fixturePlugin: ClaudePlugin = {
+  root: "/tmp/plugin",
+  manifest: { name: "compound-engineering", version: "1.0.0", description: "A plugin" },
+  agents: [
+    {
+      name: "security-reviewer",
+      description: "Security-focused agent",
+      capabilities: ["Threat modeling", "OWASP"],
+      model: "claude-sonnet-4-20250514",
+      body: "Focus on vulnerabilities in ~/.claude/settings.",
+      sourcePath: "/tmp/plugin/agents/security-reviewer.md",
+    },
+  ],
+  commands: [
+    {
+      name: "workflows:plan",
+      description: "Planning command",
+      argumentHint: "[FOCUS]",
+      model: "inherit",
+      allowedTools: ["Read"],
+      body: "Plan the work. See ~/.claude/settings for config.",
+      sourcePath: "/tmp/plugin/commands/workflows/plan.md",
+    },
+    {
+      name: "disabled-cmd",
+      description: "Disabled command",
+      model: "inherit",
+      allowedTools: [],
+      body: "Should be excluded.",
+      disableModelInvocation: true,
+      sourcePath: "/tmp/plugin/commands/disabled-cmd.md",
+    },
+  ],
+  skills: [
+    {
+      name: "existing-skill",
+      description: "Existing skill",
+      sourceDir: "/tmp/plugin/skills/existing-skill",
+      skillPath: "/tmp/plugin/skills/existing-skill/SKILL.md",
+    },
+  ],
+  hooks: undefined,
+  mcpServers: {
+    local: { command: "npx", args: ["-y", "some-mcp-server"] },
+    remote: { url: "https://mcp.example.com/api", headers: { Authorization: "Bearer token" } },
+  },
+}
+
+const defaultOptions = {
+  agentMode: "subagent" as const,
+  inferTemperature: false,
+  permissions: "none" as const,
+}
+
+describe("convertClaudeToOpenClaw", () => {
+  test("converts agents to skill files with SKILL.md content", () => {
+    const bundle = convertClaudeToOpenClaw(fixturePlugin, defaultOptions)
+
+    const skill = bundle.skills.find((s) => s.name === "security-reviewer")
+    expect(skill).toBeDefined()
+    expect(skill!.dir).toBe("agent-security-reviewer")
+    const parsed = parseFrontmatter(skill!.content)
+    expect(parsed.data.name).toBe("security-reviewer")
+    expect(parsed.data.description).toBe("Security-focused agent")
+    expect(parsed.data.model).toBe("claude-sonnet-4-20250514")
+    expect(parsed.body).toContain("Focus on vulnerabilities")
+  })
+
+  test("converts commands to skill files (excluding disableModelInvocation)", () => {
+    const bundle = convertClaudeToOpenClaw(fixturePlugin, defaultOptions)
+
+    const cmdSkill = bundle.skills.find((s) => s.name === "workflows:plan")
+    expect(cmdSkill).toBeDefined()
+    expect(cmdSkill!.dir).toBe("cmd-workflows:plan")
+
+    const disabledSkill = bundle.skills.find((s) => s.name === "disabled-cmd")
+    expect(disabledSkill).toBeUndefined()
+  })
+
+  test("commands list excludes disableModelInvocation commands", () => {
+    const bundle = convertClaudeToOpenClaw(fixturePlugin, defaultOptions)
+
+    const cmd = bundle.commands.find((c) => c.name === "workflows-plan")
+    expect(cmd).toBeDefined()
+    expect(cmd!.description).toBe("Planning command")
+    expect(cmd!.acceptsArgs).toBe(true)
+
+    const disabled = bundle.commands.find((c) => c.name === "disabled-cmd")
+    expect(disabled).toBeUndefined()
+  })
+
+  test("command colons are replaced with dashes in command registrations", () => {
+    const bundle = convertClaudeToOpenClaw(fixturePlugin, defaultOptions)
+
+    const cmd = bundle.commands.find((c) => c.name === "workflows-plan")
+    expect(cmd).toBeDefined()
+    expect(cmd!.name).not.toContain(":")
+  })
+
+  test("manifest includes plugin id, display name, and skills list", () => {
+    const bundle = convertClaudeToOpenClaw(fixturePlugin, defaultOptions)
+
+    expect(bundle.manifest.id).toBe("compound-engineering")
+    expect(bundle.manifest.name).toBe("Compound Engineering")
+    expect(bundle.manifest.kind).toBe("tool")
+    expect(bundle.manifest.skills).toContain("skills/agent-security-reviewer")
+    expect(bundle.manifest.skills).toContain("skills/cmd-workflows:plan")
+    expect(bundle.manifest.skills).toContain("skills/existing-skill")
+  })
+
+  test("package.json uses plugin name and version", () => {
+    const bundle = convertClaudeToOpenClaw(fixturePlugin, defaultOptions)
+
+    expect(bundle.packageJson.name).toBe("openclaw-compound-engineering")
+    expect(bundle.packageJson.version).toBe("1.0.0")
+    expect(bundle.packageJson.type).toBe("module")
+  })
+
+  test("skillDirCopies includes original skill directories", () => {
+    const bundle = convertClaudeToOpenClaw(fixturePlugin, defaultOptions)
+
+    const copy = bundle.skillDirCopies.find((s) => s.name === "existing-skill")
+    expect(copy).toBeDefined()
+    expect(copy!.sourceDir).toBe("/tmp/plugin/skills/existing-skill")
+  })
+
+  test("stdio MCP servers included in openclaw config", () => {
+    const bundle = convertClaudeToOpenClaw(fixturePlugin, defaultOptions)
+
+    expect(bundle.openclawConfig).toBeDefined()
+    const mcp = (bundle.openclawConfig!.mcpServers as Record<string, unknown>)
+    expect(mcp.local).toBeDefined()
+    expect((mcp.local as any).type).toBe("stdio")
+    expect((mcp.local as any).command).toBe("npx")
+  })
+
+  test("HTTP MCP servers included as http type in openclaw config", () => {
+    const bundle = convertClaudeToOpenClaw(fixturePlugin, defaultOptions)
+
+    const mcp = (bundle.openclawConfig!.mcpServers as Record<string, unknown>)
+    expect(mcp.remote).toBeDefined()
+    expect((mcp.remote as any).type).toBe("http")
+    expect((mcp.remote as any).url).toBe("https://mcp.example.com/api")
+  })
+
+  test("paths are rewritten from .claude/ to .openclaw/ in skill content", () => {
+    const bundle = convertClaudeToOpenClaw(fixturePlugin, defaultOptions)
+
+    const agentSkill = bundle.skills.find((s) => s.name === "security-reviewer")
+    expect(agentSkill!.content).toContain("~/.openclaw/settings")
+    expect(agentSkill!.content).not.toContain("~/.claude/settings")
+
+    const cmdSkill = bundle.skills.find((s) => s.name === "workflows:plan")
+    expect(cmdSkill!.content).toContain("~/.openclaw/settings")
+    expect(cmdSkill!.content).not.toContain("~/.claude/settings")
+  })
+
+  test("generateEntryPoint uses JSON.stringify for safe string escaping", () => {
+    const plugin: ClaudePlugin = {
+      ...fixturePlugin,
+      commands: [
+        {
+          name: "tricky-cmd",
+          description: 'Has "quotes" and \\backslashes\\ and\nnewlines',
+          model: "inherit",
+          allowedTools: [],
+          body: "body",
+          sourcePath: "/tmp/cmd.md",
+        },
+      ],
+    }
+    const bundle = convertClaudeToOpenClaw(plugin, defaultOptions)
+
+    // Entry point must be valid JS/TS — JSON.stringify handles all special chars
+    expect(bundle.entryPoint).toContain('"tricky-cmd"')
+    expect(bundle.entryPoint).toContain('\\"quotes\\"')
+    expect(bundle.entryPoint).toContain("\\\\backslashes\\\\")
+    expect(bundle.entryPoint).toContain("\\n")
+    // No raw unescaped newline inside a string literal
+    const lines = bundle.entryPoint.split("\n")
+    const nameLine = lines.find((l) => l.includes("tricky-cmd") && l.includes("name:"))
+    expect(nameLine).toBeDefined()
+  })
+
+  test("generateEntryPoint emits typed skills record", () => {
+    const bundle = convertClaudeToOpenClaw(fixturePlugin, defaultOptions)
+    expect(bundle.entryPoint).toContain("const skills: Record<string, string> = {}")
+  })
+
+  test("plugin without MCP servers has no openclawConfig", () => {
+    const plugin: ClaudePlugin = { ...fixturePlugin, mcpServers: undefined }
+    const bundle = convertClaudeToOpenClaw(plugin, defaultOptions)
+    expect(bundle.openclawConfig).toBeUndefined()
+  })
+})