karajan-code 3.12.1 → 3.12.3

package/node_modules/@karajan/core/src/vec-store.js

@@ -1,253 +0,0 @@
-// KJC-PCS-0049 Step 1 — Vector store sobre better-sqlite3 + sqlite-vec.
-// El RAG indexer (Step 4) escribe; el retriever (Step 5) lee. Una sola
-// DB en `~/.karajan/rag.db` por instalación.
-import { mkdirSync, existsSync } from "node:fs";
-import { dirname, join } from "node:path";
-import Database from "better-sqlite3";
-import * as sqliteVec from "sqlite-vec";
-
-import { getKarajanHome } from "./paths.js";
-
-const DEFAULT_DIM = 768;
-
-export function dbPath() {
-  return process.env.KJ_RAG_DB || join(getKarajanHome(), "rag.db");
-}
-
-/**
- * Open the rag.db, load the sqlite-vec extension, and ensure the
- * chunks + vec_chunks tables exist. Idempotent: a second call against
- * the same file is a no-op past the schema creation.
- *
- * Returns the open Database handle. Caller owns close().
- *
- * @param {Object} [opts]
- * @param {number} [opts.dim=768] — embedding dimension. Must match
- *   the embedder configured for the project (Step 2). Changing it
- *   later requires a fresh DB; the constant is captured in the
- *   virtual table DDL.
- */
-export function openVecStore({ dim = DEFAULT_DIM, path = dbPath() } = {}) {
-  if (!existsSync(dirname(path))) mkdirSync(dirname(path), { recursive: true });
-  const db = new Database(path);
-  sqliteVec.load(db);
-  db.pragma("journal_mode = WAL");
-  db.pragma("foreign_keys = ON");
-  db.exec(`
-    CREATE TABLE IF NOT EXISTS chunks (
-      id INTEGER PRIMARY KEY AUTOINCREMENT,
-      source TEXT NOT NULL,
-      kind TEXT NOT NULL CHECK (kind IN ('plan', 'code', 'onboarding')),
-      text TEXT NOT NULL,
-      metadata TEXT,
-      created_at TEXT NOT NULL DEFAULT (datetime('now'))
-    );
-    CREATE INDEX IF NOT EXISTS chunks_by_source ON chunks(source);
-    CREATE INDEX IF NOT EXISTS chunks_by_kind ON chunks(kind);
-  `);
-  // KJC-TSK-0438 — Project isolation. project_slug = basename of projectDir
-  // normalized; chunks indexed before the migration carry NULL and remain
-  // queryable when `--project all` (or no slug detected at query time).
-  const cols = db.prepare("PRAGMA table_info(chunks)").all().map((c) => c.name);
-  if (!cols.includes("project_slug")) {
-    db.exec("ALTER TABLE chunks ADD COLUMN project_slug TEXT;");
-    db.exec("CREATE INDEX IF NOT EXISTS chunks_by_project ON chunks(project_slug);");
-  }
-  // KJC-TSK-0484 — Content-hash dedup. SHA-256 of the chunk text lets the
-  // indexer skip embedding bodies it has already seen for the same project.
-  // Older rows keep NULL; the lookup ignores NULLs so they neither dedup
-  // nor get deduped until they are re-indexed and stamped.
-  if (!cols.includes("content_hash")) {
-    db.exec("ALTER TABLE chunks ADD COLUMN content_hash TEXT;");
-    db.exec("CREATE INDEX IF NOT EXISTS chunks_by_hash ON chunks(content_hash, project_slug);");
-  }
-  // KJC-TSK-0455 — Auto-update by commit diff. Per-project metadata tracks
-  // the last commit whose changes were reflected in the index, so subsequent
-  // `kj rag index --since auto` invocations (and the post-merge hook / pre-run
-  // drift check landed in PR2) only re-embed the actual delta instead of the
-  // whole project.
-  db.exec(`
-    CREATE TABLE IF NOT EXISTS vec_store_meta (
-      project_slug TEXT PRIMARY KEY,
-      last_indexed_commit TEXT,
-      last_indexed_at TEXT NOT NULL DEFAULT (datetime('now'))
-    );
-  `);
-  db.exec(`CREATE VIRTUAL TABLE IF NOT EXISTS vec_chunks USING vec0(embedding float[${dim}]);`);
-  // KJC-TSK-0443 — FTS5 keyword index for BM25 hybrid scoring. content='chunks'
-  // makes it a contentless mirror linked by id (no double storage); triggers
-  // sync the FTS5 rows on insert/update/delete. Rebuilds idempotently when
-  // the FTS5 table exists but is empty (e.g. created post-migration on a
-  // DB with pre-v2.28 chunks).
-  db.exec(`
-    CREATE VIRTUAL TABLE IF NOT EXISTS chunks_fts USING fts5(text, content='chunks', content_rowid='id');
-    CREATE TRIGGER IF NOT EXISTS chunks_ai AFTER INSERT ON chunks BEGIN
-      INSERT INTO chunks_fts(rowid, text) VALUES (new.id, new.text);
-    END;
-    CREATE TRIGGER IF NOT EXISTS chunks_ad AFTER DELETE ON chunks BEGIN
-      INSERT INTO chunks_fts(chunks_fts, rowid, text) VALUES ('delete', old.id, old.text);
-    END;
-    CREATE TRIGGER IF NOT EXISTS chunks_au AFTER UPDATE ON chunks BEGIN
-      INSERT INTO chunks_fts(chunks_fts, rowid, text) VALUES ('delete', old.id, old.text);
-      INSERT INTO chunks_fts(rowid, text) VALUES (new.id, new.text);
-    END;
-  `);
-  const ftsCount = db.prepare("SELECT COUNT(*) AS n FROM chunks_fts").get().n;
-  const chunksCount = db.prepare("SELECT COUNT(*) AS n FROM chunks").get().n;
-  if (ftsCount === 0 && chunksCount > 0) {
-    db.exec("INSERT INTO chunks_fts(chunks_fts) VALUES('rebuild');");
-  }
-  return db;
-}
-
-/**
- * Persist one chunk. The chunks.id and vec_chunks.rowid stay in sync
- * because the virtual table rowid is set explicitly to the freshly
- * minted chunks.id. Returns that id.
- */
-export function insertChunk(db, { source, kind, text, metadata = null, embedding, project = null, contentHash = null }) {
-  if (!Array.isArray(embedding) && !ArrayBuffer.isView(embedding)) {
-    throw new Error("insertChunk: embedding must be an array or typed array of floats");
-  }
-  const meta = metadata == null ? null : (typeof metadata === "string" ? metadata : JSON.stringify(metadata));
-  const ins = db.prepare("INSERT INTO chunks (source, kind, text, metadata, project_slug, content_hash) VALUES (?, ?, ?, ?, ?, ?)");
-  const info = ins.run(source, kind, text, meta, project, contentHash);
-  // sqlite-vec's vec0 virtual table requires the rowid to arrive as a BigInt
-  // even for values that comfortably fit in a Number. Better-sqlite3 returns
-  // lastInsertRowid as BigInt only when safeIntegers mode is on; force it.
-  const id = typeof info.lastInsertRowid === "bigint" ? info.lastInsertRowid : BigInt(info.lastInsertRowid);
-  const buf = embedding instanceof Float32Array ? embedding : Float32Array.from(embedding);
-  db.prepare("INSERT INTO vec_chunks(rowid, embedding) VALUES (?, ?)").run(id, Buffer.from(buf.buffer));
-  return Number(id);
-}
-
-/**
- * Cosine-ordered nearest-neighbour search. Returns `[{ id, source, kind,
- * text, metadata, distance }]` for the topK best matches; lower
- * distance = closer. Returns `[]` when the store is empty.
- */
-export function searchSimilar(db, embedding, topK = 5, { kind = null, project = null, whereSql = "", whereParams = [] } = {}) {
-  const buf = embedding instanceof Float32Array ? embedding : Float32Array.from(embedding);
-  const kindClause = kind ? "AND c.kind = ?" : "";
-  const projectClause = project ? "AND c.project_slug = ?" : "";
-  const sql = `
-    SELECT c.id, c.source, c.kind, c.text, c.metadata, c.project_slug, v.distance
-    FROM vec_chunks v
-    JOIN chunks c ON c.id = v.rowid
-    WHERE v.embedding MATCH ? AND k = ? ${kindClause} ${projectClause}${whereSql}
-    ORDER BY v.distance
-  `;
-  const params = [Buffer.from(buf.buffer), topK];
-  if (kind) params.push(kind);
-  if (project) params.push(project);
-  if (whereParams.length) params.push(...whereParams);
-  const rows = db.prepare(sql).all(...params);
-  return rows.map((r) => ({ ...r, metadata: r.metadata ? safeParse(r.metadata) : null }));
-}
-
-/**
- * Normalize a projectDir into a stable slug used for project isolation.
- * Basename → lowercased → non-alphanumeric stripped to dashes.
- * KJC-TSK-0438.
- */
-export function projectSlug(projectDir) {
-  if (!projectDir) return null;
-  const base = String(projectDir).replace(/\/+$/, "").split("/").pop();
-  if (!base) return null;
-  return base.toLowerCase().replace(/[^a-z0-9._-]+/g, "-").replace(/^-+|-+$/g, "");
-}
-
-function safeParse(s) { try { return JSON.parse(s); } catch { return s; } }
-
-/**
- * Remove every chunk that came from a given source path. Both tables
- * are touched explicitly because vec0 virtual tables do not honour
- * SQLite foreign-key cascades.
- */
-export function deleteChunksBySource(db, source) {
-  const ids = db.prepare("SELECT id FROM chunks WHERE source = ?").all(source).map((r) => r.id);
-  if (ids.length === 0) return 0;
-  const placeholders = ids.map(() => "?").join(",");
-  db.prepare(`DELETE FROM vec_chunks WHERE rowid IN (${placeholders})`).run(...ids);
-  db.prepare(`DELETE FROM chunks WHERE id IN (${placeholders})`).run(...ids);
-  return ids.length;
-}
-
-// KJC-TSK-0455 — Per-project commit watermark for `kj rag index --since auto`
-// and the post-merge / pre-run drift hooks landing in PR2.
-export function getLastIndexedCommit(db, projectSlug) {
-  if (!projectSlug) return null;
-  const row = db.prepare("SELECT last_indexed_commit FROM vec_store_meta WHERE project_slug = ?").get(projectSlug);
-  return row?.last_indexed_commit ?? null;
-}
-
-export function setLastIndexedCommit(db, projectSlug, commit) {
-  if (!projectSlug || !commit) return;
-  db.prepare(`
-    INSERT INTO vec_store_meta (project_slug, last_indexed_commit, last_indexed_at)
-    VALUES (?, ?, datetime('now'))
-    ON CONFLICT(project_slug) DO UPDATE SET
-      last_indexed_commit = excluded.last_indexed_commit,
-      last_indexed_at = excluded.last_indexed_at
-  `).run(projectSlug, commit);
-}
-
-// KJC-TSK-0484 PR-B — Bulk fetch embeddings for an explicit set of chunk ids.
-// Returns Map<id, Float32Array>. Used by MMR diversification in retriever.js.
-export function getEmbeddingsByIds(db, ids) {
-  const out = new Map();
-  if (!ids?.length) return out;
-  const placeholders = ids.map(() => "?").join(",");
-  const rows = db.prepare(`SELECT rowid AS id, embedding FROM vec_chunks WHERE rowid IN (${placeholders})`).all(...ids);
-  for (const row of rows) {
-    if (!row.embedding) continue;
-    const buf = Buffer.isBuffer(row.embedding) ? row.embedding : Buffer.from(row.embedding);
-    out.set(Number(row.id), new Float32Array(buf.buffer, buf.byteOffset, buf.byteLength / 4));
-  }
-  return out;
-}
-
-// KJC-TSK-0484 — Lookup an already-indexed chunk by content_hash. Scoped to a
-// project so two repos with the same boilerplate file don't cross-pollute.
-// Returns `{ id, source }` or null. Pass `project=null` to match NULL slugs.
-export function findChunkByHash(db, hash, project = null) {
-  if (!hash) return null;
-  const sql = project
-    ? "SELECT id, source FROM chunks WHERE content_hash = ? AND project_slug = ? LIMIT 1"
-    : "SELECT id, source FROM chunks WHERE content_hash = ? AND project_slug IS NULL LIMIT 1";
-  const row = project ? db.prepare(sql).get(hash, project) : db.prepare(sql).get(hash);
-  return row || null;
-}
-
-export function countChunks(db, { kind = null } = {}) {
-  if (kind) return db.prepare("SELECT COUNT(*) AS n FROM chunks WHERE kind = ?").get(kind).n;
-  return db.prepare("SELECT COUNT(*) AS n FROM chunks").get().n;
-}
-
-/**
- * KJC-TSK-0443 — BM25 keyword search over chunks_fts. Returns
- * [{ id, source, kind, text, metadata, project_slug, bm25 }] ordered by
- * relevance (lower bm25 = better in SQLite FTS5). `query` is sanitized for
- * the FTS5 grammar: special chars are stripped, words split on whitespace.
- */
-export function searchBM25(db, queryText, topK = 10, { kind = null, project = null, whereSql = "", whereParams = [] } = {}) {
-  const cleaned = String(queryText || "").replace(/["'()]/g, " ").split(/\s+/).filter(Boolean).join(" OR ");
-  if (!cleaned) return [];
-  const kindClause = kind ? "AND c.kind = ?" : "";
-  const projectClause = project ? "AND c.project_slug = ?" : "";
-  const sql = `
-    SELECT c.id, c.source, c.kind, c.text, c.metadata, c.project_slug, bm25(chunks_fts) AS bm25
-    FROM chunks_fts JOIN chunks c ON c.id = chunks_fts.rowid
-    WHERE chunks_fts MATCH ? ${kindClause} ${projectClause}${whereSql}
-    ORDER BY bm25 LIMIT ?
-  `;
-  const params = [cleaned];
-  if (kind) params.push(kind);
-  if (project) params.push(project);
-  if (whereParams.length) params.push(...whereParams);
-  params.push(topK);
-  try {
-    const rows = db.prepare(sql).all(...params);
-    return rows.map((r) => ({ ...r, metadata: r.metadata ? safeParse(r.metadata) : null }));
-  } catch { return []; }
-}

package/node_modules/@karajan/core/tests/atomic-write.test.js

@@ -1,30 +0,0 @@
-// Smoke coverage for the @karajan/core/atomic-write extraction
-// (KJC-TSK-0511 PR1). The full behavioural matrix already lives next to
-// the CLI; this file just proves the new package entry point works
-// end-to-end so a future regression in workspace wiring fails loudly.
-import { mkdtemp, readFile } from "node:fs/promises";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { describe, it, expect } from "vitest";
-
-import { writeJsonAtomic, writeJsonAtomicSync } from "../src/atomic-write.js";
-
-async function tmpDir() {
-  return mkdtemp(join(tmpdir(), "core-atomic-"));
-}
-
-describe("@karajan/core/atomic-write", () => {
-  it("writeJsonAtomic persists pretty JSON and leaves no .tmp", async () => {
-    const dir = await tmpDir();
-    const target = join(dir, "plan.json");
-    await writeJsonAtomic(target, { hu: 42 });
-    expect(await readFile(target, "utf-8")).toBe('{\n  "hu": 42\n}');
-  });
-
-  it("writeJsonAtomicSync persists pretty JSON synchronously", async () => {
-    const dir = await tmpDir();
-    const target = join(dir, "state.json");
-    writeJsonAtomicSync(target, { ok: true });
-    expect(await readFile(target, "utf-8")).toBe('{\n  "ok": true\n}');
-  });
-});

package/node_modules/@karajan/core/tests/cost/index.test.js

@@ -1,89 +0,0 @@
-import { describe, it, expect } from "vitest";
-import { aggregateRunCost } from "../../src/cost/index.js";
-
-describe("aggregateRunCost — happy path", () => {
-  it("aggregates a single-role single-model run", () => {
-    const r = aggregateRunCost({
-      events: [
-        { role: "coder", model: "claude-opus-4-7", tokensIn: 1_000_000, tokensOut: 100_000 },
-      ],
-    });
-    // 1M * $15 + 0.1M * $75 = 15 + 7.5
-    expect(r.totalUsd).toBe(22.5);
-    expect(r.byRole).toEqual({ coder: 22.5 });
-    expect(r.byModel).toEqual({ "claude-opus-4-7": 22.5 });
-    expect(r.hasUnknown).toBe(false);
-    expect(r.currency).toBe("USD");
-  });
-
-  it("splits cost by role across multiple models and accumulates same-key events", () => {
-    const r = aggregateRunCost({
-      events: [
-        { role: "coder", model: "claude-opus-4-7", tokensIn: 200_000, tokensOut: 50_000 },
-        { role: "coder", model: "claude-opus-4-7", tokensIn: 100_000, tokensOut: 0 },
-        { role: "reviewer", model: "gpt-4o", tokensIn: 100_000, tokensOut: 20_000 },
-      ],
-    });
-    // coder: (0.2*15 + 0.05*75) + (0.1*15) = 6.75 + 1.5 = 8.25
-    // reviewer: 0.1*2.5 + 0.02*10 = 0.45
-    expect(r.byRole.coder).toBe(8.25);
-    expect(r.byRole.reviewer).toBe(0.45);
-    expect(r.totalUsd).toBe(8.7);
-    expect(Object.keys(r.byModel).sort()).toEqual(["claude-opus-4-7", "gpt-4o"]);
-  });
-});
-
-describe("aggregateRunCost — unknown models", () => {
-  it("routes unknown tokens to unknownModelTokens, sorts + dedupes, calls logger", () => {
-    const calls = [];
-    const r = aggregateRunCost({
-      events: [
-        { role: "coder", model: "zzz-llm", tokensIn: 500, tokensOut: 200 },
-        { role: "coder", model: "aaa-llm", tokensIn: 100, tokensOut: 100 },
-        { role: "coder", model: "zzz-llm", tokensIn: 1, tokensOut: 1 },
-        { role: "reviewer", model: "claude-opus-4-7", tokensIn: 100_000, tokensOut: 10_000 },
-      ],
-      logger: (level, msg, ctx) => calls.push({ level, ctx }),
-    });
-    expect(r.hasUnknown).toBe(true);
-    expect(r.unknownModelTokens.tokensIn).toBe(601);
-    expect(r.unknownModelTokens.tokensOut).toBe(301);
-    expect(r.unknownModelTokens.models).toEqual(["aaa-llm", "zzz-llm"]);
-    expect(r.byRole.coder).toBeUndefined();
-    expect(r.byRole.reviewer).toBeGreaterThan(0);
-    expect(calls.length).toBe(3);
-    expect(calls[0].level).toBe("warn");
-  });
-});
-
-describe("aggregateRunCost — edge cases", () => {
-  it("returns zero for empty / missing events", () => {
-    expect(aggregateRunCost({ events: [] }).totalUsd).toBe(0);
-    expect(aggregateRunCost({}).totalUsd).toBe(0);
-  });
-
-  it("skips events with zero tokens and coerces invalid fields", () => {
-    const r = aggregateRunCost({
-      events: [
-        { role: "coder", model: "claude-opus-4-7", tokensIn: 0, tokensOut: 0 },
-        { role: "coder", model: "claude-opus-4-7", tokensIn: NaN, tokensOut: 10_000 },
-        { role: "", model: "gpt-4o", tokensIn: 10_000, tokensOut: 1_000 },
-        null,
-        "not-an-object",
-      ],
-    });
-    // First skipped, second: tokensIn=0, 10k output * 75 = 0.75
-    expect(r.byRole.coder).toBe(0.75);
-    expect(r.byRole.unknown).toBeGreaterThan(0);
-  });
-
-  it("rounds to 4 decimals", () => {
-    const r = aggregateRunCost({
-      events: [{ role: "coder", model: "gemini-1.5-flash", tokensIn: 100_000, tokensOut: 100_000 }],
-    });
-    // 0.1*0.075 + 0.1*0.30 = 0.0075 + 0.03 = 0.0375
-    expect(r.totalUsd).toBe(0.0375);
-    const frac = String(r.totalUsd).split(".")[1] || "";
-    expect(frac.length).toBeLessThanOrEqual(4);
-  });
-});

package/node_modules/@karajan/core/tests/hu-snapshot.test.js

@@ -1,39 +0,0 @@
-import { describe, it, expect, vi } from "vitest";
-import {
-  snapshotRefForHu, createHuSnapshot, hasHuSnapshot,
-  restoreHuSnapshot, removeHuSnapshot,
-} from "../src/hu-snapshot.js";
-
-describe("@karajan/core/hu-snapshot", () => {
-  it("snapshotRefForHu sanitises and prefixes", () => {
-    expect(snapshotRefForHu("hu_001")).toBe("refs/kj-snapshots/hu_001");
-    expect(snapshotRefForHu("hu/with..weird:chars")).toBe("refs/kj-snapshots/hu_with__weird_chars");
-    expect(() => snapshotRefForHu("")).toThrow(/huId/);
-  });
-
-  it("createHuSnapshot calls rev-parse then update-ref", async () => {
-    const runner = vi.fn()
-      .mockResolvedValueOnce({ exitCode: 0, stdout: "abc123\n", stderr: "" })
-      .mockResolvedValueOnce({ exitCode: 0, stdout: "", stderr: "" });
-    const r = await createHuSnapshot({ projectDir: "/x", huId: "hu_001", runner });
-    expect(r).toEqual({ ok: true, ref: "refs/kj-snapshots/hu_001", sha: "abc123" });
-  });
-
-  it("hasHuSnapshot returns sha when ref exists", async () => {
-    const runner = vi.fn().mockResolvedValue({ exitCode: 0, stdout: "deadbeef\n", stderr: "" });
-    expect(await hasHuSnapshot({ projectDir: "/x", huId: "h", runner })).toEqual({ ok: true, sha: "deadbeef" });
-  });
-
-  it("restoreHuSnapshot reports missing snapshot", async () => {
-    const runner = vi.fn().mockResolvedValue({ exitCode: 1, stdout: "", stderr: "" });
-    const r = await restoreHuSnapshot({ projectDir: "/x", huId: "missing", runner });
-    expect(r.ok).toBe(false);
-    expect(r.error).toMatch(/no encontrado/);
-  });
-
-  it("removeHuSnapshot is idempotent (git -d returns 0 even if absent)", async () => {
-    const runner = vi.fn().mockResolvedValue({ exitCode: 0, stdout: "", stderr: "" });
-    const r = await removeHuSnapshot({ projectDir: "/x", huId: "absent", runner });
-    expect(r.ok).toBe(true);
-  });
-});

package/node_modules/@karajan/core/tests/paths.test.js

@@ -1,51 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach } from "vitest";
-import os from "node:os";
-import path from "node:path";
-import {
-  resolveHome,
-  getKarajanHome,
-  getSessionRoot,
-  getRunsDir,
-  getPromptsDir,
-  __resetKjHomeWarningForTests,
-} from "../src/paths.js";
-
-describe("@karajan/core/paths", () => {
-  let originalKarajanHome;
-  let originalKjHome;
-
-  beforeEach(() => {
-    originalKarajanHome = process.env.KARAJAN_HOME;
-    originalKjHome = process.env.KJ_HOME;
-    delete process.env.KARAJAN_HOME;
-    delete process.env.KJ_HOME;
-    __resetKjHomeWarningForTests();
-  });
-
-  afterEach(() => {
-    if (originalKarajanHome !== undefined) process.env.KARAJAN_HOME = originalKarajanHome;
-    if (originalKjHome !== undefined) process.env.KJ_HOME = originalKjHome;
-  });
-
-  it("KARAJAN_HOME takes precedence", () => {
-    process.env.KARAJAN_HOME = "/tmp/explicit-home";
-    expect(resolveHome()).toBe(path.resolve("/tmp/explicit-home"));
-  });
-
-  it("getKarajanHome defaults to .karajan segment", () => {
-    process.env.KARAJAN_HOME = "/tmp/root";
-    expect(getKarajanHome()).toBe(path.resolve("/tmp/root"));
-  });
-
-  it("derived dirs hang off getKarajanHome", () => {
-    process.env.KARAJAN_HOME = "/tmp/root";
-    const home = path.resolve("/tmp/root");
-    expect(getSessionRoot()).toBe(path.join(home, "sessions"));
-    expect(getRunsDir()).toBe(path.join(home, "runs"));
-    expect(getPromptsDir()).toBe(path.join(home, "prompts"));
-  });
-
-  it("VITEST env auto-isolates to tmpdir", () => {
-    expect(resolveHome().startsWith(os.tmpdir())).toBe(true);
-  });
-});

package/node_modules/@karajan/core/tests/plan-hu-ops.test.js

@@ -1,68 +0,0 @@
-import { describe, it, expect } from "vitest";
-import {
-  addHu,
-  removeHu,
-  updateHu,
-  updateHuStatus,
-  certifyAllHus,
-  assertPlanRunnable,
-  computePlanOutcome,
-} from "../src/plan-hu-ops.js";
-
-const newPlan = () => ({ planId: "plan-T", hus: [], updatedAt: null });
-
-describe("@karajan/core/plan-hu-ops", () => {
-  it("addHu assigns sequential ID + defaults", () => {
-    const plan = newPlan();
-    const hu = addHu(plan, { title: "T1" });
-    expect(hu.id).toBe("hu_plan-T_001");
-    expect(hu.status).toBe("pending");
-    expect(plan.hus).toHaveLength(1);
-  });
-
-  it("removeHu cleans blocked_by + reuse refs", () => {
-    const plan = newPlan();
-    addHu(plan, { title: "T1" });
-    addHu(plan, { title: "T2", blocked_by: ["hu_plan-T_001"], reuse: ["hu_plan-T_001"] });
-    expect(removeHu(plan, "hu_plan-T_001")).toBe(true);
-    expect(plan.hus[0].blocked_by).toEqual([]);
-    expect(plan.hus[0].reuse).toEqual([]);
-  });
-
-  it("updateHu only writes allowed fields", () => {
-    const plan = newPlan();
-    addHu(plan, { title: "T1" });
-    updateHu(plan, "hu_plan-T_001", { title: "T1!", id: "evil" });
-    expect(plan.hus[0].title).toBe("T1!");
-    expect(plan.hus[0].id).toBe("hu_plan-T_001");
-  });
-
-  it("updateHuStatus + certifyAllHus + assertPlanRunnable", () => {
-    const plan = newPlan();
-    addHu(plan, { title: "T1" });
-    expect(updateHuStatus(plan, "hu_plan-T_001", "in_progress")).toBe(true);
-    addHu(plan, { title: "T2" });
-    expect(certifyAllHus(plan)).toBe(1);
-    expect(plan.status).toBe("ready");
-    expect(() => assertPlanRunnable(plan, "plan-T")).not.toThrow();
-  });
-
-  it("assertPlanRunnable throws when no HU certified", () => {
-    const plan = newPlan();
-    addHu(plan, { title: "T1" });
-    expect(() => assertPlanRunnable(plan, "plan-T")).toThrow(/0 HU/);
-  });
-
-  it("computePlanOutcome rolls up counts + dedupes", () => {
-    const plan = newPlan();
-    addHu(plan, { title: "T1" });
-    addHu(plan, { title: "T2" });
-    plan.hus[0].outcome = { status: "done", pr_url: "u", blockers: [] };
-    plan.hus[1].outcome = { status: "failed", pr_url: "u", blockers: ["x"] };
-    const o = computePlanOutcome(plan, { duration_ms: 1 });
-    expect(o.status).toBe("partial");
-    expect(o.counts).toEqual({ done: 1, failed: 1, blocked: 0, pending: 0 });
-    expect(o.prs).toEqual(["u"]);
-    expect(o.blockers).toEqual(["x"]);
-  });
-});

package/node_modules/@karajan/core/tests/plan-id.test.js

@@ -1,21 +0,0 @@
-import { describe, it, expect } from "vitest";
-import { generatePlanId, generateHuId, normaliseAlias } from "../src/plan-id.js";
-
-describe("@karajan/core/plan-id", () => {
-  it("generatePlanId returns plan-<ts>-<rand>", () => {
-    const id = generatePlanId();
-    expect(id).toMatch(/^plan-\d{14}-[a-z0-9]{4}$/);
-  });
-
-  it("generateHuId formats sequence zero-padded", () => {
-    expect(generateHuId("plan-X", 1)).toBe("hu_plan-X_001");
-    expect(generateHuId("plan-X", 42)).toBe("hu_plan-X_042");
-  });
-
-  it("normaliseAlias slugifies + caps to 60 chars", () => {
-    expect(normaliseAlias("Greta App")).toBe("greta-app");
-    expect(normaliseAlias("Café — Ñoño")).toBe("cafe-nono");
-    expect(normaliseAlias("")).toBeNull();
-    expect(normaliseAlias(null)).toBeNull();
-  });
-});

package/node_modules/@karajan/core/tests/plan-validation.test.js

@@ -1,39 +0,0 @@
-import { describe, it, expect } from "vitest";
-import { validateBlockedByChange } from "../src/plan-validation.js";
-
-const plan = () => ({
-  hus: [
-    { id: "a", blocked_by: [] },
-    { id: "b", blocked_by: ["a"] },
-    { id: "c", blocked_by: ["b"] },
-  ],
-});
-
-describe("@karajan/core/plan-validation", () => {
-  it("ok when the new blocked_by is a valid subset", () => {
-    expect(validateBlockedByChange(plan(), "c", ["a"]).ok).toBe(true);
-  });
-
-  it("rejects self-dependency", () => {
-    const r = validateBlockedByChange(plan(), "a", ["a"]);
-    expect(r.ok).toBe(false);
-    expect(r.error).toMatch(/auto-dependencia/);
-  });
-
-  it("rejects unknown HU reference", () => {
-    const r = validateBlockedByChange(plan(), "a", ["zzz"]);
-    expect(r.ok).toBe(false);
-    expect(r.error).toMatch(/inexistente/);
-  });
-
-  it("rejects cycle creation", () => {
-    const r = validateBlockedByChange(plan(), "a", ["c"]);
-    expect(r.ok).toBe(false);
-    expect(r.cycle).toBeDefined();
-    expect(r.error).toMatch(/Ciclo/);
-  });
-
-  it("rejects non-array blocked_by", () => {
-    expect(validateBlockedByChange(plan(), "a", "nope").ok).toBe(false);
-  });
-});

package/node_modules/@karajan/core/tests/port-check.test.js

@@ -1,47 +0,0 @@
-// Smoke coverage for the @karajan/core/port-check extraction
-// (KJC-TSK-0511 PR2). Real socket binding — tiny but fast.
-import { createServer } from "node:net";
-import { describe, it, expect } from "vitest";
-
-import { isPortAvailable, findAvailablePort } from "../src/port-check.js";
-
-function occupy() {
-  return new Promise((resolve) => {
-    const srv = createServer();
-    srv.listen(0, () => resolve({ srv, port: srv.address().port }));
-  });
-}
-
-describe("@karajan/core/port-check", () => {
-  it("isPortAvailable returns false for a port we already bound", async () => {
-    const { srv, port } = await occupy();
-    try {
-      expect(await isPortAvailable(port)).toBe(false);
-    } finally {
-      srv.close();
-    }
-  });
-
-  it("findAvailablePort skips busy ports and returns the next free one", async () => {
-    const { srv, port } = await occupy();
-    try {
-      const got = await findAvailablePort(port, 5);
-      expect(got).toBeGreaterThan(port);
-    } finally {
-      srv.close();
-    }
-  });
-
-  it("findAvailablePort throws when the range is exhausted", async () => {
-    const occupied = [];
-    for (let i = 0; i < 3; i += 1) occupied.push(await occupy());
-    const ports = occupied.map((o) => o.port).sort((a, b) => a - b);
-    try {
-      const gap = ports[2] - ports[0];
-      if (gap > 2) return;
-      await expect(findAvailablePort(ports[0], gap + 1)).rejects.toThrow(/No available port/);
-    } finally {
-      for (const { srv } of occupied) srv.close();
-    }
-  });
-});