test(cli): add Jest test suite for PromptKit CLI

Add comprehensive test coverage for the CLI module:

- manifest.js: loadManifest, getTemplates, persona/protocol/format/
  taxonomy lookups, resolveTemplateDeps (16 tests)
- assemble.js: stripFrontmatter, loadComponent, full assembly with
  section ordering, param substitution, verbatim inclusion (15 tests)
- launch.js: detectCli priority order with mocked child_process,
  copyContentToTemp file/directory operations (9 tests)
- cli.js: integration tests for list, assemble, --version, error
  handling via subprocess execution (7 tests)

Tests validate against real repo content (personas/, protocols/,
formats/, templates/, manifest.yaml) rather than synthetic fixtures,
so they catch structural regressions without maintenance burden.

Infrastructure:
- Jest dev dependency with test and test:coverage scripts
- CI workflow (.github/workflows/cli-tests.yml) triggers on cli/ changes
- coverage/ added to .gitignore

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: abeltrano

.github/workflows/cli-tests.yml

@@ -0,0 +1,32 @@
+# SPDX-License-Identifier: MIT
+# Copyright (c) PromptKit Contributors
+
+name: CLI Tests
+
+on:
+  push:
+    paths:
+      - 'cli/**'
+  pull_request:
+    paths:
+      - 'cli/**'
+
+jobs:
+  test:
+    name: Run CLI tests
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Set up Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: '22'
+
+      - name: Install dependencies
+        working-directory: cli
+        run: npm ci
+
+      - name: Run tests
+        working-directory: cli
+        run: npm test

cli/.gitignore

@@ -1,2 +1,3 @@
 node_modules/
 content/
+coverage/

cli/__tests__/assemble.test.js

@@ -0,0 +1,167 @@
+// SPDX-License-Identifier: MIT
+
+const path = require("path");
+const {
+  assemble,
+  loadComponent,
+  stripFrontmatter,
+} = require("../lib/assemble");
+const { loadManifest, getTemplates } = require("../lib/manifest");
+
+// Use the real repo content — no synthetic fixtures
+const repoRoot = path.resolve(__dirname, "..", "..");
+
+describe("assemble", () => {
+  describe("stripFrontmatter", () => {
+    // Pure function tests use inline strings — no files needed
+    test("removes YAML frontmatter delimited by ---", () => {
+      const input = "---\nname: test\ntype: foo\n---\n# Body\n\nContent here.";
+      expect(stripFrontmatter(input)).toBe("# Body\n\nContent here.");
+    });
+
+    test("returns content unchanged when no frontmatter present", () => {
+      const input = "# Just a heading\n\nSome content.";
+      expect(stripFrontmatter(input)).toBe(input);
+    });
+
+    test("handles frontmatter with Windows-style line endings", () => {
+      const input = "---\r\nname: test\r\n---\r\n# Body";
+      expect(stripFrontmatter(input)).toBe("# Body");
+    });
+
+    test("trims leading/trailing whitespace after stripping", () => {
+      const input = "---\nname: test\n---\n\n  # Body  \n\n";
+      expect(stripFrontmatter(input)).toBe("# Body");
+    });
+  });
+
+  describe("loadComponent", () => {
+    test("loads a persona and strips HTML comments and frontmatter", () => {
+      const body = loadComponent(repoRoot, "personas/systems-engineer.md");
+      expect(body).not.toContain("SPDX-License-Identifier");
+      expect(body).toContain("# Persona: Senior Systems Engineer");
+    });
+
+    test("loads a protocol and strips HTML comments and frontmatter", () => {
+      const body = loadComponent(
+        repoRoot,
+        "protocols/guardrails/anti-hallucination.md"
+      );
+      expect(body).not.toContain("SPDX-License-Identifier");
+      expect(body).toContain("# Protocol: Anti-Hallucination Guardrails");
+    });
+
+    test("returns null for missing component and warns", () => {
+      const warnSpy = jest.spyOn(console, "warn").mockImplementation();
+      const result = loadComponent(repoRoot, "nonexistent.md");
+
+      expect(result).toBeNull();
+      expect(warnSpy).toHaveBeenCalledWith(
+        expect.stringContaining("nonexistent.md")
+      );
+      warnSpy.mockRestore();
+    });
+  });
+
+  describe("assemble (full)", () => {
+    let manifest;
+
+    beforeAll(() => {
+      manifest = loadManifest(repoRoot);
+    });
+
+    test("assembles a complete prompt with all sections in order", () => {
+      const templates = getTemplates(manifest);
+      const bug = templates.find((t) => t.name === "investigate-bug");
+      const result = assemble(repoRoot, manifest, bug);
+
+      // Check section headers are present in order
+      const identityPos = result.indexOf("# Identity");
+      const protocolsPos = result.indexOf("# Reasoning Protocols");
+      const formatPos = result.indexOf("# Output Format");
+      const taskPos = result.indexOf("# Task");
+
+      expect(identityPos).toBeGreaterThanOrEqual(0);
+      expect(protocolsPos).toBeGreaterThan(identityPos);
+      expect(formatPos).toBeGreaterThan(protocolsPos);
+      expect(taskPos).toBeGreaterThan(formatPos);
+    });
+
+    test("includes real persona body text verbatim", () => {
+      const templates = getTemplates(manifest);
+      const bug = templates.find((t) => t.name === "investigate-bug");
+      const result = assemble(repoRoot, manifest, bug);
+
+      expect(result).toContain("# Persona: Senior Systems Engineer");
+    });
+
+    test("includes all protocol bodies", () => {
+      const templates = getTemplates(manifest);
+      const bug = templates.find((t) => t.name === "investigate-bug");
+      const result = assemble(repoRoot, manifest, bug);
+
+      expect(result).toContain("# Protocol: Anti-Hallucination Guardrails");
+      expect(result).toContain("# Protocol: Root Cause Analysis");
+    });
+
+    test("includes taxonomy section when template references one", () => {
+      const templates = getTemplates(manifest);
+      const review = templates.find((t) => t.name === "review-code");
+      const result = assemble(repoRoot, manifest, review);
+
+      expect(result).toContain("# Classification Taxonomy");
+    });
+
+    test("includes format body text", () => {
+      const templates = getTemplates(manifest);
+      const bug = templates.find((t) => t.name === "investigate-bug");
+      const result = assemble(repoRoot, manifest, bug);
+
+      expect(result).toContain("# Format: Investigation Report");
+    });
+
+    test("substitutes parameters when provided", () => {
+      const templates = getTemplates(manifest);
+      const bug = templates.find((t) => t.name === "investigate-bug");
+      const result = assemble(repoRoot, manifest, bug, {
+        problem_description: "Use-after-free in socket handler",
+        code_context: "See socket.c line 42",
+        environment: "Linux x86_64",
+      });
+
+      expect(result).toContain("Use-after-free in socket handler");
+      expect(result).toContain("See socket.c line 42");
+      expect(result).not.toContain("{{problem_description}}");
+    });
+
+    test("leaves unfilled params as placeholders", () => {
+      const templates = getTemplates(manifest);
+      const bug = templates.find((t) => t.name === "investigate-bug");
+      const result = assemble(repoRoot, manifest, bug, {
+        problem_description: "memory leak",
+      });
+
+      expect(result).toContain("memory leak");
+      expect(result).toContain("{{code_context}}");
+    });
+
+    test("does not contain YAML frontmatter or SPDX headers", () => {
+      const templates = getTemplates(manifest);
+      const bug = templates.find((t) => t.name === "investigate-bug");
+      const result = assemble(repoRoot, manifest, bug);
+
+      expect(result).not.toContain("SPDX-License-Identifier");
+      // YAML frontmatter (---\nkey: val) should not appear; section dividers (---) are fine
+      expect(result).not.toMatch(/^---\r?\n\w+:/m);
+    });
+
+    test("separates sections with --- dividers", () => {
+      const templates = getTemplates(manifest);
+      const bug = templates.find((t) => t.name === "investigate-bug");
+      const result = assemble(repoRoot, manifest, bug);
+
+      const sections = result.split("\n\n---\n\n");
+      expect(sections.length).toBeGreaterThanOrEqual(4);
+    });
+  });
+});

cli/__tests__/cli.test.js

@@ -0,0 +1,125 @@
+// SPDX-License-Identifier: MIT
+
+const path = require("path");
+const { execFileSync } = require("child_process");
+
+const cliPath = path.resolve(__dirname, "..", "bin", "cli.js");
+
+// Integration tests run the actual CLI as a subprocess.
+// The CLI reads content copied from the repo root via the prepare script.
+
+describe("cli integration", () => {
+
+  describe("list command", () => {
+    test("lists templates as JSON", () => {
+      const result = runCli(["list", "--json"]);
+      const templates = JSON.parse(result);
+
+      expect(Array.isArray(templates)).toBe(true);
+      expect(templates.length).toBeGreaterThan(0);
+      expect(templates[0]).toHaveProperty("name");
+      expect(templates[0]).toHaveProperty("category");
+    });
+
+    test("lists templates in human-readable format", () => {
+      const result = runCli(["list"]);
+
+      expect(result).toContain("Available PromptKit templates:");
+      expect(result).toContain("investigate-bug");
+    });
+  });
+
+  describe("assemble command", () => {
+    const outputPath = path.join(
+      require("os").tmpdir(),
+      `promptkit-test-${Date.now()}.md`
+    );
+
+    afterAll(() => {
+      try {
+        require("fs").unlinkSync(outputPath);
+      } catch {
+        // ignore cleanup errors
+      }
+    });
+
+    test("assembles a template to a file", () => {
+      runCli(["assemble", "investigate-bug", "--output", outputPath]);
+
+      const fs = require("fs");
+      expect(fs.existsSync(outputPath)).toBe(true);
+
+      const content = fs.readFileSync(outputPath, "utf8");
+      expect(content).toContain("# Identity");
+      expect(content).toContain("# Task");
+    });
+
+    test("substitutes params via --param", () => {
+      runCli([
+        "assemble",
+        "investigate-bug",
+        "--output",
+        outputPath,
+        "--param",
+        "problem_description=test bug",
+        "--param",
+        "code_context=test context",
+        "--param",
+        "environment=test env",
+      ]);
+
+      const content = require("fs").readFileSync(outputPath, "utf8");
+      expect(content).toContain("test bug");
+      expect(content).toContain("test context");
+      expect(content).not.toContain("{{problem_description}}");
+    });
+
+    test("reports unfilled parameters", () => {
+      const result = runCli(["assemble", "investigate-bug", "--output", outputPath]);
+
+      expect(result).toContain("unfilled parameter");
+    });
+
+    test("exits with error for unknown template", () => {
+      expect(() => {
+        runCli(["assemble", "nonexistent-template"]);
+      }).toThrow();
+    });
+  });
+
+  describe("--version", () => {
+    test("prints the version number", () => {
+      const result = runCli(["--version"]);
+      expect(result.trim()).toMatch(/^\d+\.\d+\.\d+/);
+    });
+  });
+});
+
+/**
+ * Runs the CLI with the given arguments. Ensures the content/ directory
+ * is populated from the repo root via the prepare script before first use.
+ */
+function runCli(args) {
+  // The CLI reads content from ../content relative to bin/cli.js.
+  // We need the prepare script to have run, or we run it first.
+  const contentDir = path.resolve(__dirname, "..", "content");
+  const fs = require("fs");
+
+  if (!fs.existsSync(path.join(contentDir, "manifest.yaml"))) {
+    // Run prepare to copy content
+    execFileSync(process.execPath, [
+      path.resolve(__dirname, "..", "scripts", "copy-content.js"),
+    ], { cwd: path.resolve(__dirname, "..") });
+  }
+
+  const result = execFileSync(
+    process.execPath,
+    [cliPath, ...args],
+    {
+      encoding: "utf8",
+      cwd: path.resolve(__dirname, ".."),
+      timeout: 10000,
+    }
+  );
+  return result;
+}