@jaybeeuu/agent-cortex 1.37.1 → 1.38.1

package/test/cli.test.mjs

@@ -1,6 +1,8 @@
 import { describe, it } from "node:test";
 import assert from "node:assert/strict";
 import { execFile } from "node:child_process";
+import { mkdtempSync, readFileSync, rmSync } from "node:fs";
+import { tmpdir } from "node:os";
 import { fileURLToPath } from "node:url";
 import { dirname, join } from "node:path";
 import { parseArgs, buildHelpText, validateHarness } from "../lib/cli.mjs";
@@ -56,6 +58,84 @@ describe("parseArgs", () => {
     assert.deepStrictEqual(result, { command: "install", harness: null });
   });
 
+  it("returns output for install claude --output <dir>", () => {
+    const result = parseArgs(["install", "claude", "--output", "/tmp/x"]);
+    assert.deepStrictEqual(result, { command: "install", harness: "claude", output: "/tmp/x" });
+  });
+
+  it("returns output for install claude --output=<dir>", () => {
+    const result = parseArgs(["install", "claude", "--output=/tmp/x"]);
+    assert.deepStrictEqual(result, { command: "install", harness: "claude", output: "/tmp/x" });
+  });
+
+  it("returns dryRun for install claude --dry-run", () => {
+    const result = parseArgs(["install", "claude", "--dry-run"]);
+    assert.deepStrictEqual(result, { command: "install", harness: "claude", dryRun: true });
+  });
+
+  it("combines --dry-run with --output", () => {
+    const result = parseArgs(["install", "claude", "--output", "/tmp/x", "--dry-run"]);
+    assert.deepStrictEqual(result, { command: "install", harness: "claude", output: "/tmp/x", dryRun: true });
+  });
+
+  it("reports an unknown option for claude", () => {
+    const result = parseArgs(["install", "claude", "--bogus"]);
+    assert.equal(result.command, "install");
+    assert.equal(result.harness, "claude");
+    assert.ok(result.optionError.includes("--bogus"));
+  });
+
+  it("reports a missing value for --output for claude", () => {
+    const result = parseArgs(["install", "claude", "--output"]);
+    assert.equal(result.command, "install");
+    assert.equal(result.harness, "claude");
+    assert.ok(result.optionError.includes("--output"));
+  });
+
+  it("parses --dry-run for install", () => {
+    const result = parseArgs(["install", "pi", "--dry-run"]);
+    assert.deepStrictEqual(result, { command: "install", harness: "pi", dryRun: true });
+  });
+
+  it("parses --output <dir> for install", () => {
+    const result = parseArgs(["install", "pi", "--output", "/tmp/out"]);
+    assert.deepStrictEqual(result, { command: "install", harness: "pi", output: "/tmp/out" });
+  });
+
+  it("parses --plugin-root <dir> for install", () => {
+    const result = parseArgs(["install", "pi", "--plugin-root", "/tmp/plugin"]);
+    assert.deepStrictEqual(result, { command: "install", harness: "pi", pluginRoot: "/tmp/plugin" });
+  });
+
+  it("combines multiple install options", () => {
+    const result = parseArgs(["install", "pi", "--dry-run", "--output", "/tmp/out", "--plugin-root", "/tmp/plugin"]);
+    assert.deepStrictEqual(result, {
+      command: "install",
+      harness: "pi",
+      dryRun: true,
+      output: "/tmp/out",
+      pluginRoot: "/tmp/plugin",
+    });
+  });
+
+  it("reports a missing value for --output", () => {
+    const result = parseArgs(["install", "pi", "--output"]);
+    assert.equal(result.command, "install");
+    assert.equal(result.harness, "pi");
+    assert.ok(result.optionError.includes("--output"));
+  });
+
+  it("reports an unknown option", () => {
+    const result = parseArgs(["install", "pi", "--bogus"]);
+    assert.equal(result.command, "install");
+    assert.equal(result.harness, "pi");
+    assert.ok(result.optionError.includes("--bogus"));
+  });
+
+  it("keeps the old shape when no options are given", () => {
+    assert.deepStrictEqual(parseArgs(["install", "pi"]), { command: "install", harness: "pi" });
+  });
+
   it("returns { command: 'unknown', name: 'foo' } for unknown command", () => {
     const result = parseArgs(["foo"]);
     assert.deepStrictEqual(result, { command: "unknown", name: "foo" });
@@ -117,8 +197,9 @@ describe("CLI integration", () => {
     assert.ok(stdout.includes("copilot"));
   });
 
-  it("exits 0 for install claude", async () => {
-    const { exitCode, stdout } = await runCli(["install", "claude"]);
+  it("exits 0 for install claude (into a temp output dir)", async () => {
+    const dir = mkdtempSync(join(tmpdir(), "agent-cortex-cli-"));
+    const { exitCode, stdout } = await runCli(["install", "claude", "--output", dir]);
     assert.equal(exitCode, 0);
     assert.ok(stdout.includes("claude"));
   });
@@ -135,6 +216,38 @@ describe("CLI integration", () => {
     assert.ok(stderr.includes("Missing harness"));
   });
 
+  it("exits 1 and prints error for install with an unknown option", async () => {
+    const { exitCode, stderr } = await runCli(["install", "pi", "--bogus"]);
+    assert.equal(exitCode, 1);
+    assert.ok(stderr.includes("--bogus"));
+  });
+
+  it("exits 1 and prints error for install with a missing option value", async () => {
+    const { exitCode, stderr } = await runCli(["install", "pi", "--output"]);
+    assert.equal(exitCode, 1);
+    assert.ok(stderr.includes("--output"));
+  });
+
+  it("exits 0 for install pi --dry-run and writes nothing", async () => {
+    const { exitCode, stdout } = await runCli(["install", "pi", "--dry-run"]);
+    assert.equal(exitCode, 0);
+    assert.ok(stdout.includes("ralph.agent.md"));
+    assert.ok(stdout.includes("dry-run"));
+  });
+
+  it("exits 0 for install pi --output <tmp> and writes agent files", async () => {
+    const outDir = mkdtempSync(join(tmpdir(), "cli-pi-install-"));
+    try {
+      const { exitCode, stdout } = await runCli(["install", "pi", "--output", outDir]);
+      assert.equal(exitCode, 0);
+      assert.ok(stdout.includes("ralph.agent.md"));
+      const ralph = readFileSync(join(outDir, "agents", "ralph.agent.md"), "utf-8");
+      assert.ok(ralph.includes("agent-cortex:ralph"));
+    } finally {
+      rmSync(outDir, { recursive: true, force: true });
+    }
+  });
+
   it("exits 1 and prints error for install with unknown harness", async () => {
     const { exitCode, stderr } = await runCli(["install", "docker"]);
     assert.equal(exitCode, 1);

package/test/copilot-installer.test.mjs

@@ -0,0 +1,298 @@
+// Integration tests for `agent-cortex install copilot`: the install-time
+// generator for the Copilot CLI flat agent files (agents/*.agent.md). All tests
+// install into mkdtemp dirs so the real repo agents/ is never touched.
+
+import { describe, it } from "node:test";
+import assert from "node:assert/strict";
+import { execFile } from "node:child_process";
+import { mkdtempSync, mkdirSync, writeFileSync, rmSync, readFileSync, existsSync, readdirSync, statSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { fileURLToPath } from "node:url";
+import { join, dirname } from "node:path";
+import { installCopilot } from "../bin/installers/copilot.mjs";
+
+const __dirname = dirname(fileURLToPath(import.meta.url));
+const ROOT = join(__dirname, "..");
+const CLI_PATH = join(ROOT, "bin", "agent-cortex.mjs");
+
+function runCli(args) {
+  return new Promise((resolve) => {
+    execFile(process.execPath, [CLI_PATH, ...args], (error, stdout, stderr) => {
+      resolve({ exitCode: error ? error.code ?? 1 : 0, stdout, stderr });
+    });
+  });
+}
+
+// ─── Fixtures ────────────────────────────────────────────────────────────────
+
+function tokenMap() {
+  return {
+    name: "fixture token map",
+    version: 1,
+    harnesses: ["copilot"],
+    tools: {
+      bash: { copilot: "bash" },
+      view: { copilot: "view" },
+      task: { copilot: "task" },
+      ask_user: { copilot: "ask_user" },
+    },
+    paths: {
+      plugin_root: { copilot: "~/.copilot/installed-plugins/_direct/agent-cortex" },
+      agents_dir: { copilot: { base: "plugin_root", relative: "agents" } },
+    },
+  };
+}
+
+function makeFixture() {
+  const root = mkdtempSync(join(tmpdir(), "copilot-installer-"));
+  return {
+    root,
+    output: join(root, "generated"),
+    cleanup: () => rmSync(root, { recursive: true, force: true }),
+  };
+}
+
+function writeFixture(fx, relPath, content) {
+  const p = join(fx.root, relPath);
+  mkdirSync(dirname(p), { recursive: true });
+  writeFileSync(p, content);
+}
+
+/** Seed a minimal but realistic package tree (agents + token-map). */
+function seedCopilotPackage(fx, opts = {}) {
+  const { agents = {} } = opts;
+  writeFixture(fx, "token-map.json", JSON.stringify(tokenMap()));
+  for (const [name, def] of Object.entries(agents)) {
+    writeFixture(fx, `agents/${name}/agent.md`, def.body ?? `# ${name}\nBody of ${name}.`);
+    writeFixture(
+      fx,
+      `agents/${name}/copilot/frontmatter.json`,
+      JSON.stringify(
+        def.frontmatter ?? {
+          name: `agent-cortex:${name}`,
+          description: `Description of ${name}`,
+          tools: ["bash", "view"],
+          argumentHint: `Hint for ${name}`,
+        },
+      ),
+    );
+    for (const [section, content] of Object.entries(def.sections ?? {})) {
+      writeFixture(fx, `agents/${name}/copilot/${section}.md`, content);
+    }
+  }
+}
+
+/** Deterministic snapshot of a directory: relative paths + byte content. */
+function snapshot(dir) {
+  const entries = [];
+  const walk = (p) => {
+    for (const e of readdirSync(p, { withFileTypes: true }).sort((a, b) => a.name.localeCompare(b.name))) {
+      const full = join(p, e.name);
+      if (e.isDirectory()) {
+        entries.push([e.name, "dir"]);
+        walk(full);
+      } else entries.push([e.name, "file", readFileSync(full, "utf-8")]);
+    }
+  };
+  walk(dir);
+  return entries;
+}
+
+// ─── Agent composition ───────────────────────────────────────────────────────
+
+describe("installCopilot — agent composition", () => {
+  it("writes one flat <name>.agent.md per composable agent dir with copilot frontmatter and no leftover tokens", () => {
+    const fx = makeFixture();
+    try {
+      seedCopilotPackage(fx, {
+        agents: {
+          alpha: {
+            body: "# alpha\nUse {{TOOL:view}} to read.\n{{SECTION:polling}}\nRead {{PATH:agents_dir}}.",
+            sections: { polling: "Spawn with {{TOOL:task}}." },
+            frontmatter: {
+              name: "agent-cortex:alpha",
+              description: "Reads things.",
+              tools: ["bash", "view"],
+              argumentHint: "Read something",
+            },
+          },
+        },
+      });
+
+      const result = installCopilot({ root: fx.root, output: fx.output });
+
+      const file = join(fx.output, "alpha.agent.md");
+      assert.ok(existsSync(file), "flat agent file written");
+      const content = readFileSync(file, "utf-8");
+      assert.ok(content.startsWith("---\n"), "yaml frontmatter");
+      assert.ok(content.includes("# GENERATED from agents/alpha/ by scripts/build-copilot-agents.mjs — DO NOT EDIT."));
+      // Committed flat-file field order: description, name, tools (YAML array), argument-hint.
+      const order = ["description:", "name: \"agent-cortex:alpha\"", "tools: [", "argument-hint:"].map((s) => content.indexOf(s));
+      assert.ok(order.every((i) => i !== -1), "all frontmatter fields present");
+      assert.deepEqual([...order].sort((a, b) => a - b), order, "frontmatter fields in committed order");
+      assert.match(content, /tools: \["bash", "view"\]/, "copilot tools kept as a YAML array (no translation)");
+      assert.ok(content.includes("Use view to read."), "{{TOOL:view}} substituted");
+      assert.ok(content.includes("Spawn with task."), "{{SECTION:polling}} composed from copilot/polling.md");
+      assert.ok(!/{{/.test(content), "no literal tokens remain");
+      assert.equal(result.agents.length, 1);
+      assert.deepEqual(result.agents, ["alpha"]);
+      assert.equal(result.output, fx.output);
+    } finally {
+      fx.cleanup();
+    }
+  });
+
+  it("skips non-composable entries in agents/ and never clobbers composable sources", () => {
+    const fx = makeFixture();
+    try {
+      seedCopilotPackage(fx, { agents: { alpha: { body: "# alpha" } } });
+      writeFixture(fx, "agents/README.md", "# agents dir docs\n");
+
+      const result = installCopilot({ root: fx.root, output: fx.output });
+
+      assert.deepEqual(readdirSync(fx.output).sort(), ["alpha.agent.md"], "no README, no subdirectories written");
+      assert.deepEqual(result.agents, ["alpha"]);
+      // The composable source dir next to the generated file stays untouched.
+      assert.ok(existsSync(join(fx.root, "agents", "alpha", "agent.md")), "composable source preserved");
+      assert.ok(existsSync(join(fx.root, "agents", "alpha", "copilot", "frontmatter.json")), "harness dir preserved");
+    } finally {
+      fx.cleanup();
+    }
+  });
+
+  it("throws when a composable dir lacks copilot/frontmatter.json", () => {
+    const fx = makeFixture();
+    try {
+      seedCopilotPackage(fx, { agents: { alpha: { body: "# alpha" } } });
+      rmSync(join(fx.root, "agents", "alpha", "copilot"), { recursive: true, force: true });
+      assert.throws(
+        () => installCopilot({ root: fx.root, output: fx.output }),
+        /without copilot\/frontmatter\.json/,
+      );
+    } finally {
+      fx.cleanup();
+    }
+  });
+
+  it("defaults to the plugin-scanned agents/ dir of the package root (plugin.json \"agents\": \"agents/\")", () => {
+    const fx = makeFixture();
+    try {
+      seedCopilotPackage(fx, { agents: { alpha: { body: "# alpha" } } });
+
+      const result = installCopilot({ root: fx.root });
+
+      assert.equal(result.output, join(fx.root, "agents"), "default output is the dir plugin.json scans");
+      assert.ok(existsSync(join(fx.root, "agents", "alpha.agent.md")), "flat file lands next to the composable dir");
+      assert.ok(existsSync(join(fx.root, "agents", "alpha", "agent.md")), "composable source untouched");
+    } finally {
+      fx.cleanup();
+    }
+  });
+
+  it("throws on an unknown tool token", () => {
+    const fx = makeFixture();
+    try {
+      seedCopilotPackage(fx, { agents: { alpha: { body: "# alpha\nUse {{TOOL:nope}} now." } } });
+      assert.throws(() => installCopilot({ root: fx.root, output: fx.output }), /unknown tool token {{TOOL:nope}}/);
+    } finally {
+      fx.cleanup();
+    }
+  });
+});
+
+// ─── Output / dry-run ────────────────────────────────────────────────────────
+
+describe("installCopilot — output & dry-run", () => {
+  it("dry-run reports the plan without writing anything", () => {
+    const fx = makeFixture();
+    try {
+      seedCopilotPackage(fx, { agents: { alpha: { body: "# alpha" } } });
+
+      const result = installCopilot({ root: fx.root, output: fx.output, dryRun: true });
+
+      assert.equal(result.dryRun, true);
+      assert.equal(existsSync(fx.output), false, "no output dir created in dry-run");
+      assert.deepEqual(result.agents, ["alpha"], "planned files still reported");
+    } finally {
+      fx.cleanup();
+    }
+  });
+
+  it("is deterministic across repeated installs into the same output", () => {
+    const fx = makeFixture();
+    try {
+      seedCopilotPackage(fx, {
+        agents: {
+          alpha: { body: "# alpha\n{{SECTION:polling}}", sections: { polling: "Poll with {{TOOL:task}}." } },
+          beta: { body: "# beta" },
+        },
+      });
+
+      installCopilot({ root: fx.root, output: fx.output });
+      const first = snapshot(fx.output);
+      installCopilot({ root: fx.root, output: fx.output });
+      assert.deepEqual(snapshot(fx.output), first);
+    } finally {
+      fx.cleanup();
+    }
+  });
+});
+
+// ─── Integration against the real repo ───────────────────────────────────────
+
+describe("installCopilot — real repo integration", () => {
+  it("regenerates byte-identical copies of the committed agents/*.agent.md files", () => {
+    const fx = makeFixture();
+    try {
+      const result = installCopilot({ root: ROOT, output: fx.output });
+
+      assert.deepEqual(result.agents.sort(), ["plan", "ralph", "ralph-plan", "strategy"]);
+      for (const name of result.agents) {
+        const generated = readFileSync(join(fx.output, `${name}.agent.md`), "utf-8");
+        const committed = readFileSync(join(ROOT, "agents", `${name}.agent.md`), "utf-8");
+        assert.equal(generated, committed, `${name}.agent.md must match the committed flat file (CI drift check)`);
+      }
+    } finally {
+      fx.cleanup();
+    }
+  });
+});
+
+// ─── CLI wiring ──────────────────────────────────────────────────────────────
+
+describe("CLI integration", () => {
+  it("exits 0 for install copilot --output <tmp> and writes the flat files", async () => {
+    const fx = makeFixture();
+    try {
+      const { exitCode, stdout } = await runCli(["install", "copilot", "--output", fx.output]);
+      assert.equal(exitCode, 0);
+      assert.ok(stdout.includes("copilot"), "prints the harness name");
+      assert.ok(stdout.includes("Generated"), "prints the generation summary");
+      assert.ok(existsSync(join(fx.output, "ralph.agent.md")), "agent files written");
+      assert.ok(existsSync(join(fx.output, "plan.agent.md")));
+    } finally {
+      fx.cleanup();
+    }
+  });
+
+  it("exits 0 for install copilot --dry-run and writes nothing", async () => {
+    const fx = makeFixture();
+    try {
+      const { exitCode, stdout } = await runCli(["install", "copilot", "--output", fx.output, "--dry-run"]);
+      assert.equal(exitCode, 0);
+      assert.ok(/dry-run/i.test(stdout));
+      assert.equal(existsSync(fx.output), false, "no files written in dry-run");
+    } finally {
+      fx.cleanup();
+    }
+  });
+
+  it("exits 0 for install copilot (default output) without touching committed files", async () => {
+    const before = readFileSync(join(ROOT, "agents", "ralph.agent.md"), "utf-8");
+    const { exitCode } = await runCli(["install", "copilot"]);
+    assert.equal(exitCode, 0);
+    // Default output regenerates the shared agents/ dir in place — output must
+    // stay byte-identical to what is committed.
+    assert.equal(readFileSync(join(ROOT, "agents", "ralph.agent.md"), "utf-8"), before);
+  });
+});

package/test/installer.test.mjs

@@ -0,0 +1,179 @@
+// Integration tests for `agent-cortex install claude`: the install-time
+// generator for the Claude Code plugin subtree. All tests install into
+// mkdtemp dirs so the real repo claude/ and ~/.claude are never touched.
+
+import { describe, it } from "node:test";
+import assert from "node:assert/strict";
+import { execFile } from "node:child_process";
+import { mkdtempSync, readFileSync, readdirSync, existsSync, lstatSync, readlinkSync, realpathSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { fileURLToPath } from "node:url";
+import { dirname, join, relative } from "node:path";
+
+const __dirname = dirname(fileURLToPath(import.meta.url));
+const ROOT = join(__dirname, "..");
+const CLI_PATH = join(ROOT, "bin", "agent-cortex.mjs");
+
+function runCli(args) {
+  return new Promise((resolve) => {
+    execFile(process.execPath, [CLI_PATH, ...args], (error, stdout, stderr) => {
+      resolve({ exitCode: error ? error.code ?? 1 : 0, stdout, stderr });
+    });
+  });
+}
+
+function makeTmp() {
+  return mkdtempSync(join(tmpdir(), "agent-cortex-installer-"));
+}
+
+/** Deterministic snapshot of a generated tree: relative paths, kind, and for
+ * files/symlinks their byte content / link target. */
+function snapshot(dir) {
+  const entries = [];
+  const walk = (p) => {
+    for (const e of readdirSync(p, { withFileTypes: true }).sort((a, b) => a.name.localeCompare(b.name))) {
+      const full = join(p, e.name);
+      const rel = relative(dir, full);
+      if (e.isSymbolicLink()) entries.push([rel, "link", readlinkSync(full)]);
+      else if (e.isDirectory()) {
+        entries.push([rel, "dir"]);
+        walk(full);
+      } else entries.push([rel, "file", readFileSync(full, "utf-8")]);
+    }
+  };
+  walk(dir);
+  return entries;
+}
+
+describe("install claude", () => {
+  it("composes agents from the composable sources (claude names, no tokens)", async () => {
+    const out = makeTmp();
+    const { exitCode } = await runCli(["install", "claude", "--output", out]);
+    assert.equal(exitCode, 0);
+
+    const plan = readFileSync(join(out, "agents", "plan.md"), "utf-8");
+    assert.match(plan, /^---\n# GENERATED from agents\/plan\//);
+    // provenance names the shared installer (no build-time script), so the
+    // committed subtree can only come from the install-time generator.
+    assert.match(plan, /bin\/installers\/claude\.mjs/);
+    assert.doesNotMatch(plan, /build-claude-agents/);
+    assert.match(plan, /\nname: plan\n/);
+    assert.doesNotMatch(plan, /\{\{(TOOL|PATH|SECTION):/);
+
+    const ralphPlan = readFileSync(join(out, "agents", "ralph-plan.md"), "utf-8");
+    assert.match(ralphPlan, /^---\n# GENERATED from agents\/ralph-plan\//);
+    assert.match(ralphPlan, /\nname: ralph-plan\n/);
+  });
+
+  it("copies native agents verbatim (ralph stays native, no composed counterpart)", async () => {
+    const out = makeTmp();
+    await runCli(["install", "claude", "--output", out]);
+    const native = readFileSync(join(ROOT, "agents-native", "ralph.md"), "utf-8");
+    assert.equal(readFileSync(join(out, "agents", "ralph.md"), "utf-8"), native);
+  });
+
+  it("symlinks flattened skills and excludes ralph-coupled skills", async () => {
+    const out = makeTmp();
+    await runCli(["install", "claude", "--output", out]);
+    const skills = join(out, "skills");
+    const names = readdirSync(skills);
+    assert.ok(names.includes("tdd"));
+    assert.ok(names.includes("plan"));
+    assert.ok(!names.includes("ralph"));
+    assert.ok(!names.includes("run-pipeline-stage"));
+    assert.ok(lstatSync(join(skills, "tdd")).isSymbolicLink());
+    assert.equal(realpathSync(join(skills, "tdd")), realpathSync(join(ROOT, "skills", "engineering", "tdd")));
+  });
+
+  it("generates plugin.json tracking the package version and referencing hooks.json", async () => {
+    const out = makeTmp();
+    await runCli(["install", "claude", "--output", out]);
+    const pkgVersion = JSON.parse(readFileSync(join(ROOT, "package.json"), "utf-8")).version;
+    const plugin = JSON.parse(readFileSync(join(out, ".claude-plugin", "plugin.json"), "utf-8"));
+    assert.equal(plugin.name, "agent-cortex");
+    assert.equal(plugin.version, pkgVersion);
+    assert.equal(plugin.hooks, "./hooks.json");
+  });
+
+  it("generates hooks.json from the canonical hooks/claude/ source", async () => {
+    const out = makeTmp();
+    await runCli(["install", "claude", "--output", out]);
+    const source = JSON.parse(readFileSync(join(ROOT, "hooks", "claude", "hooks.json"), "utf-8"));
+    const installed = JSON.parse(readFileSync(join(out, "hooks.json"), "utf-8"));
+    assert.deepStrictEqual(installed, source);
+    assert.ok(source.hooks.SessionStart);
+  });
+
+  it("matches the Notification hook only on documented notification types", async () => {
+    // The Notification matcher is an exact-string list over `notification_type` —
+    // there are no success/error types, so that matcher would never fire. Guard
+    // against it recurring (see docs/claude-hooks.md for the documented type list).
+    const out = makeTmp();
+    await runCli(["install", "claude", "--output", out]);
+    const installed = JSON.parse(readFileSync(join(out, "hooks.json"), "utf-8"));
+    const matcher = installed.hooks.Notification.flatMap((g) => g.hooks).length
+      ? installed.hooks.Notification[0].matcher
+      : "";
+    const documented = new Set([
+      "permission_prompt",
+      "idle_prompt",
+      "auth_success",
+      "elicitation_dialog",
+      "elicitation_url_dialog",
+      "elicitation_complete",
+      "elicitation_response",
+      "agent_needs_input",
+      "agent_completed",
+      "quota_auto_resume_fired",
+      "quota_auto_resume_stale",
+      "quota_auto_resume_disabled",
+    ]);
+    const types = matcher ? matcher.split("|") : [];
+    assert.ok(types.length > 0, "Notification hook should list at least one matcher type");
+    for (const t of types) {
+      assert.ok(documented.has(t), `Notification matcher references unknown type "${t}"`);
+    }
+  });
+
+  it("bundles hook support scripts into claude/hooks/ and references them via the plugin root", async () => {
+    const out = makeTmp();
+    await runCli(["install", "claude", "--output", out]);
+
+    const canonical = readFileSync(join(ROOT, "hooks", "claude", "scripts", "notify.mjs"), "utf-8");
+    const bundled = readFileSync(join(out, "hooks", "scripts", "notify.mjs"), "utf-8");
+    assert.equal(bundled, canonical);
+
+    const installed = JSON.parse(readFileSync(join(out, "hooks.json"), "utf-8"));
+    const notify = installed.hooks.Notification.flatMap((g) => g.hooks)
+      .filter((h) => h.command?.includes("notify.mjs"));
+    assert.ok(notify.length === 1, "hooks.json should reference the notification script");
+    assert.match(notify[0].command, /^node "\$\{CLAUDE_PLUGIN_ROOT\}\/hooks\/scripts\/notify\.mjs"$/);
+  });
+
+  it("is deterministic across repeated installs into the same output", async () => {
+    const out = makeTmp();
+    await runCli(["install", "claude", "--output", out]);
+    const first = snapshot(out);
+    await runCli(["install", "claude", "--output", out]);
+    assert.deepStrictEqual(snapshot(out), first);
+  });
+
+  it("--dry-run reports the plan without writing anything", async () => {
+    const out = makeTmp();
+    const { exitCode, stdout } = await runCli(["install", "claude", "--output", out, "--dry-run"]);
+    assert.equal(exitCode, 0);
+    assert.ok(/would/i.test(stdout));
+    assert.equal(existsSync(join(out, "agents")), false);
+    assert.equal(existsSync(join(out, "skills")), false);
+    assert.equal(existsSync(join(out, ".claude-plugin")), false);
+  });
+
+  it("leaves the committed claude/ subtree untouched when --output is given", async () => {
+    const out = makeTmp();
+    const beforeSkills = readdirSync(join(ROOT, "claude", "skills")).sort();
+    const beforePlugin = readFileSync(join(ROOT, "claude", ".claude-plugin", "plugin.json"), "utf-8");
+    await runCli(["install", "claude", "--output", out]);
+    assert.deepStrictEqual(readdirSync(join(ROOT, "claude", "skills")).sort(), beforeSkills);
+    assert.equal(readFileSync(join(ROOT, "claude", ".claude-plugin", "plugin.json"), "utf-8"), beforePlugin);
+  });
+});