karajan-code 4.70.4 → 5.0.0

package/src/ci/pr-diff.js

@@ -1,26 +0,0 @@
-/**
- * Read PR diff via gh CLI for Karajan CI flow.
- * The reviewer reads the PR diff instead of local git diff.
- */
-
-import { runCommand } from "../utils/process.js";
-
-/**
- * Get the diff of a PR via `gh pr diff <number>`.
- * Returns the diff string or throws on failure.
- */
-export async function getPrDiff(prNumber) {
-  if (!prNumber) throw new Error("prNumber is required");
-
-  const res = await runCommand("gh", [
-    "pr",
-    "diff",
-    String(prNumber)
-  ]);
-
-  if (res.exitCode !== 0) {
-    throw new Error(`gh pr diff ${prNumber} failed: ${res.stderr || res.stdout}`);
-  }
-
-  return res.stdout;
-}

package/src/cli/register-standby.js

@@ -1,26 +0,0 @@
-// KJC-TSK-0414 PR2: registra `kj standby list` + `kj standby resume <id>`.
-// `resume` es subcomando de standby para no colisionar con el `kj resume`
-// pre-existente (Solomon pauses con --answer). Modelos mentales distintos:
-//   - `kj resume <id> --answer X`  → Solomon pauseado (existente)
-//   - `kj standby resume <id>`     → run hibernada por quota
-import { resumeCommand, standbyListCommand } from "../commands/standby.js";
-
-export function registerStandby(program) {
-  const standby = program.command("standby").description("Sesiones hibernadas (QUOTA_EXHAUSTED_DAILY etc.)");
-  standby
-    .command("list", { isDefault: true })
-    .description("Lista sesiones pendientes de reanudar")
-    .option("--json", "Output JSON")
-    .action(async (flags) => {
-      await standbyListCommand({ json: flags.json });
-    });
-
-  standby
-    .command("resume")
-    .description("Reanuda una sesión hibernada (re-spawn comando original)")
-    .argument("<sessionId>")
-    .option("--force", "Reanudar aunque cooldownUntil todavía esté en el futuro (no recomendado)")
-    .action(async (sessionId, flags) => {
-      await resumeCommand({ sessionId, force: Boolean(flags.force) });
-    });
-}

package/src/commands/architect.js

@@ -1,111 +0,0 @@
-import { createAgent } from "../agents/index.js";
-import { assertAgentsAvailable } from "../agents/availability.js";
-import { resolveRole } from "../config.js";
-import { buildArchitectPrompt, parseArchitectOutput } from "../prompts/architect.js";
-import { createCliProgressReporter } from "../utils/cli-progress.js";
-import { withBrainRecovery } from "../brain/with-brain-recovery.js";
-
-// KJC-BUG-0313 (#2061): the agent answers each list with plain strings or with
-// objects; both render, and a section with neither prints nothing, never "undefined".
-function section(title, items, render, lines) {
-  if (!items?.length) return;
-  lines.push(`### ${title}`);
-  for (const item of items) lines.push(...(typeof item === "string" ? [`- ${item}`] : render(item)));
-  lines.push("");
-}
-
-const named = (o, detail) => [`- **${o.name ?? o.decision ?? JSON.stringify(o)}**${detail ? `: ${detail}` : ""}`];
-const contract = (c) => [`- \`${c.method || "GET"} ${c.endpoint ?? c.path ?? ""}\``];
-const tradeoff = (t) => [...named(t, t.rationale), ...(t.alternatives?.length ? [`  Alternatives: ${t.alternatives.join(", ")}`] : [])];
-const entity = (e) => named(e, Array.isArray(e.fields) ? e.fields.join(", ") : e.description);
-
-function formatArchitecture(arch, lines) {
-  if (arch.type) lines.push(`**Type:** ${arch.type}`, "");
-  section("Layers", arch.layers, (l) => named(l, l.responsibility), lines);
-  section("Patterns", arch.patterns, (p) => [`- ${p.name ?? JSON.stringify(p)}`], lines);
-  section("Data Model", arch.dataModel?.entities, entity, lines);
-  // A contract given as a string ("GET /x") is code too: same backticks as an object's.
-  section("API Contracts", arch.apiContracts?.map((c) => (typeof c === "string" ? { raw: c } : c)), (c) => (c.raw ? [`- \`${c.raw}\``] : contract(c)), lines);
-  section("Dependencies", arch.dependencies, (d) => named(d, d.reason), lines);
-  section("Tradeoffs", arch.tradeoffs, tradeoff, lines);
-}
-
-export function formatArchitect(result) {
-  const lines = [];
-  lines.push(`## Architecture Design`);
-  lines.push(`**Verdict:** ${result.verdict || "unknown"}`, "");
-
-  if (result.architecture) formatArchitecture(result.architecture, lines);
-
-  if (result.questions?.length) {
-    lines.push("### Clarification Questions");
-    for (const q of result.questions) lines.push(`- ${q.question || q}`);
-    lines.push("");
-  }
-
-  if (result.summary) lines.push(`---\n${result.summary}`);
-  return lines.join("\n");
-}
-
-export async function architectCommand({ task, config, logger, context, json }) {
-  const { withCliRunLog } = await import("../utils/cli-run-log.js");
-  return withCliRunLog("architect", { projectDir: config?.projectDir, logger }, async ({ runLog }) => {
-    const architectRole = resolveRole(config, "architect");
-    await assertAgentsAvailable([architectRole.provider]);
-    logger.info(`Architect (${architectRole.provider}) starting...`);
-    runLog.logText(`[architect] provider=${architectRole.provider}`);
-
-    const agent = createAgent(architectRole.provider, config, logger);
-    const prompt = await buildArchitectPrompt({ task, researchContext: context });
-    const progress = createCliProgressReporter({ role: "architect" });
-    let result;
-    try {
-      result = await withBrainRecovery({
-        agent, taskArgs: { prompt, onOutput: progress.onOutput, role: "architect" },
-        role: "architect", provider: architectRole.provider, logger,
-      });
-      progress.finish(result.ok ? "done" : (result.action || "failed"));
-    } catch (err) { progress.finish("failed"); throw err; }
-
-    if (!result.ok) {
-      throw new Error(result.error || result.output || "Architect failed");
-    }
-
-    const parsed = parseArchitectOutput(result.output);
-
-    if (json) {
-      console.log(JSON.stringify(parsed || result.output, null, 2));
-      return { ok: true };
-    }
-
-    if (parsed?.verdict) {
-      console.log(formatArchitect(parsed));
-    } else {
-      console.log(result.output);
-    }
-
-    // Persist ADRs from the architect's tradeoffs[] (v2.7.5). One ADR
-    // per tradeoff in Nygard format under `~/.kj/plans/_loose/adrs/`.
-    // Coder/reviewer in subsequent runs can pull the active ADRs into
-    // their prompt for context — done in a follow-up PR.
-    if (parsed?.architecture && config?.projectDir) {
-      try {
-        const { persistAdrsFromArchitecture } = await import("../plan/adr-generator.js");
-        const { adrs, paths } = await persistAdrsFromArchitecture({
-          projectDir: config.projectDir,
-          planId: null,                  // standalone architect → "_loose" bucket
-          architecture: parsed.architecture,
-        });
-        if (paths.length > 0) {
-          console.log(`\n${adrs.length} ADR${adrs.length === 1 ? "" : "s"} written:`);
-          for (const p of paths) console.log(`  - ${p}`);
-        }
-      } catch (err) {
-        logger.warn(`ADR write skipped: ${err.message}`);
-      }
-    }
-
-    logger.info("Architect completed.");
-    return { ok: true };
-  });
-}

package/src/commands/autorun.js

@@ -1,49 +0,0 @@
-/**
- * `kj autorun <spec>` — the unattended delivery chain (KJC-TSK-0574, épica
- * KJC-PCS-0062, design in docs/specs/autonomous-delivery.md §7).
- *
- * One command: spec → plan → run all HUs → outcome. It reuses `kj plan` and
- * `kj run --plan` rather than reimplementing them, and defaults to the
- * `autonomous` level so the Arbiter resolves every gate (no human in the loop).
- * The rich outcome report lands in AUTO-F; here we chain + propagate exit code.
- */
-
-import { planGenerateCommand } from "./plan/generate.js";
-import { runCommandHandler } from "./run.js";
-import { buildOutcome, formatOutcomeReport } from "../autonomy/outcome-report.js";
-
-export async function autorunCommand({
-  task,
-  config,
-  logger = console,
-  flags = {},
-  planFn = planGenerateCommand,
-  runFn = runCommandHandler,
-} = {}) {
-  // autorun implies unattended: default to autonomous unless the user picked a
-  // level explicitly (so `kj autorun --autonomy assisted` still works).
-  const autoFlags = flags.autonomy || flags.autonomous ? flags : { ...flags, autonomous: true };
-
-  logger.info?.("kj autorun — planning…");
-  const plan = await planFn({ task, config, logger, flags: autoFlags });
-  if (!plan?.ok || !plan.planId) {
-    logger.error?.(`kj autorun: planning failed${plan?.reason ? ` (${plan.reason})` : ""}.`);
-    return { ok: false, stage: "plan", reason: plan?.reason ?? "plan-failed" };
-  }
-
-  logger.info?.(`kj autorun — running plan ${plan.ref ?? plan.planId} (${plan.huCount ?? "?"} HUs)…`);
-  const run = await runFn({ config, logger, flags: { ...autoFlags, plan: plan.planId } });
-
-  const ok = run?.ok === true;
-  // AUTO-F: emit the auditable outcome — delivered/incomplete, HUs that met the
-  // ask, Arbiter decisions and residual defects (enriched as the run surfaces
-  // them; baseline today is delivered + plan ref).
-  const outcome = buildOutcome({
-    planRef: plan.ref ?? plan.planId,
-    delivered: ok,
-    hus: run?.hus ?? [],
-    decisions: run?.decisions ?? [],
-  });
-  for (const line of formatOutcomeReport(outcome)) logger.info?.(line);
-  return { ok, stage: ok ? "done" : "run", planId: plan.planId, ref: plan.ref ?? plan.planId, outcome };
-}

package/src/commands/researcher.js

@@ -1,54 +0,0 @@
-import { createAgent } from "../agents/index.js";
-import { assertAgentsAvailable } from "../agents/availability.js";
-import { resolveRole } from "../config.js";
-import { createCliProgressReporter } from "../utils/cli-progress.js";
-import { withBrainRecovery } from "../brain/with-brain-recovery.js";
-
-const SUBAGENT_PREAMBLE = [
-  "IMPORTANT: You are running as a Karajan sub-agent.",
-  "Do NOT ask about using Karajan, do NOT mention Karajan, do NOT suggest orchestration.",
-  "Do NOT use any MCP tools. Focus only on researching the codebase."
-].join(" ");
-
-function buildResearchPrompt(task) {
-  return [
-    SUBAGENT_PREAMBLE,
-    "Investigate the codebase for the following task.",
-    "Identify affected files, patterns, constraints, prior decisions, risks, and test coverage.",
-    "Return a single valid JSON object with your findings and nothing else.",
-    '{"affected_files":[string],"patterns":[string],"constraints":[string],"prior_decisions":[string],"risks":[string],"test_coverage":string}',
-    `## Task\n${task}`
-  ].join("\n\n");
-}
-
-export async function researcherCommand({ task, config, logger }) {
-  const { withCliRunLog } = await import("../utils/cli-run-log.js");
-  return withCliRunLog("researcher", { projectDir: config?.projectDir, logger }, async ({ runLog }) => {
-    const researcherRole = resolveRole(config, "researcher");
-    await assertAgentsAvailable([researcherRole.provider]);
-    logger.info(`Researcher (${researcherRole.provider}) starting...`);
-    runLog.logText(`[researcher] provider=${researcherRole.provider}`);
-
-    const agent = createAgent(researcherRole.provider, config, logger);
-    const prompt = buildResearchPrompt(task);
-    const progress = createCliProgressReporter({ role: "researcher" });
-    let result;
-    try {
-      result = await withBrainRecovery({
-        agent, taskArgs: { prompt, onOutput: progress.onOutput, role: "researcher" },
-        role: "researcher", provider: researcherRole.provider, logger,
-      });
-      progress.finish(result.ok ? "done" : (result.action || "failed"));
-    } catch (err) { progress.finish("failed"); throw err; }
-
-    if (!result.ok) {
-      throw new Error(result.error || result.output || "Researcher failed");
-    }
-
-    if (result.output) {
-      console.log(result.output);
-    }
-    logger.info("Researcher completed.");
-    return { ok: true };
-  });
-}

package/src/commands/resume.js

@@ -1,72 +0,0 @@
-import { EventEmitter } from "node:events";
-import { resumeFlow } from "../orchestrator.js";
-import { sweepOnResume } from "./steward.js";
-import { createActivityLog } from "../activity-log.js";
-import { printEvent } from "../utils/display/event-handlers.js";
-import { withCliRunLog } from "../utils/cli-run-log.js";
-import { registerRun, unregisterRun } from "../utils/run-registry.js";
-import { createCliAskQuestion } from "../utils/cli-ask-question.js";
-
-export async function resumeCommand({ sessionId, answer, config, logger, flags }) {
-  const jsonMode = flags?.json;
-  const quietMode = config.output?.quiet !== false;
-  // STW-E (KJC-TSK-0793): resuming work is when someone is in front to review
-  // the state — sweep if the report went stale; adoption stays explicit.
-  await sweepOnResume({ config, logger });
-
-  // Same wrapper as every other CLI command — without it `.kj/run.log`
-  // is never opened during the resume and `kj-tail` stays silent
-  // (every line of the resumed run goes only to the per-session
-  // activity log + the local console). Also registers the run so the
-  // HU Board shows it as live and can offer its Stop button.
-  return withCliRunLog("resume", { projectDir: config?.projectDir, logger }, async ({ forwardProgress }) => {
-    const runId = registerRun({
-      pid: process.pid,
-      planId: flags?.plan || null,
-      huIds: null,
-      projectDir: config?.projectDir || process.cwd(),
-      source: "cli-resume",
-    });
-    const cleanupRegistry = () => { if (runId) unregisterRun(runId); };
-    process.once("exit", cleanupRegistry);
-    process.once("SIGTERM", () => { cleanupRegistry(); process.exit(143); });
-    process.once("SIGINT", () => { cleanupRegistry(); process.exit(130); });
-
-    const emitter = new EventEmitter();
-    let activityLog = null;
-    forwardProgress(emitter);
-
-    emitter.on("progress", (event) => {
-      if (!activityLog && event.sessionId) {
-        activityLog = createActivityLog(event.sessionId);
-        logger.onLog((entry) => activityLog.write(entry));
-      }
-      if (activityLog) {
-        activityLog.writeEvent(event);
-      }
-      if (!jsonMode) {
-        printEvent(event, { quiet: quietMode });
-      }
-    });
-
-    // KJC-BUG-0081 round 2 — propagate `flags` so the helper honours
-    // `--yes` here too. Before this fix, `kj resume <id> --yes` in
-    // non-TTY mode still routed every prompt to the HU Board bridge
-    // and hung, exactly the bug the original PR #995 fixed for `kj run`.
-    const askQuestion = createCliAskQuestion({ sessionId, flags });
-    const result = await resumeFlow({
-      sessionId,
-      answer: answer || null,
-      config,
-      logger,
-      flags: flags || {},
-      emitter,
-      askQuestion
-    });
-
-    if (jsonMode || !answer) {
-      console.log(JSON.stringify(result, null, 2));
-    }
-    return result;
-  });
-}

package/src/commands/review.js

@@ -1,89 +0,0 @@
-import { createAgent } from "../agents/index.js";
-import { assertAgentsAvailable } from "../agents/availability.js";
-import { computeBaseRef, generateDiff } from "../review/diff-generator.js";
-import { buildReviewerPrompt } from "../prompts/reviewer.js";
-import { resolveRole } from "../config.js";
-import { resolveReviewProfile } from "../review/profiles.js";
-import { withCliRunLog } from "../utils/cli-run-log.js";
-import { createCliProgressReporter } from "../utils/cli-progress.js";
-
-export async function reviewCommand({ task, config, logger, baseRef }) {
-  return withCliRunLog("review", { projectDir: config?.projectDir, logger }, async ({ runLog }) => {
-    const reviewerRole = resolveRole(config, "reviewer");
-    await assertAgentsAvailable([reviewerRole.provider]);
-    logger.info(`Reviewer (${reviewerRole.provider}) starting...`);
-    runLog.logText(`[reviewer] provider=${reviewerRole.provider} baseRef=${baseRef || "(auto)"}`);
-    const reviewer = createAgent(reviewerRole.provider, config, logger);
-
-  let diff;
-  if (config.ci?.enabled) {
-    // CI mode: read diff from open PR
-    const { detectRepo, detectPrNumber } = await import("../ci/repo.js");
-    const { getPrDiff } = await import("../ci/pr-diff.js");
-    const repo = await detectRepo();
-    const prNumber = await detectPrNumber();
-    if (!prNumber) {
-      throw new Error("CI enabled but no open PR found for current branch. Create a PR first or disable CI.");
-    }
-    logger.info(`CI: reading PR diff #${prNumber}`);
-    diff = await getPrDiff(prNumber);
-    // Store for dispatch later
-    config._ci_pr = { repo, prNumber };
-  } else {
-    const resolvedBase = await computeBaseRef({ baseBranch: config.base_branch, baseRef });
-    diff = await generateDiff({ baseRef: resolvedBase });
-  }
-
-  const { rules } = await resolveReviewProfile({ mode: config.review_mode, projectDir: process.cwd() });
-
-  const prompt = await buildReviewerPrompt({ task, diff, reviewRules: rules, mode: config.review_mode });
-  const progress = createCliProgressReporter({ role: "reviewer" });
-  let result;
-  try {
-    result = await reviewer.reviewTask({ prompt, onOutput: progress.onOutput, role: "reviewer" });
-    progress.finish(result.ok ? "done" : "failed");
-  } catch (err) { progress.finish("failed"); throw err; }
-  if (!result.ok) {
-    if (result.error) logger.error(result.error);
-    throw new Error(result.error || result.output || `Reviewer failed (exit ${result.exitCode})`);
-  }
-  console.log(result.output);
-  logger.info(`Reviewer completed (exit ${result.exitCode})`);
-
-  // CI: dispatch review result
-  if (config.ci?.enabled && config._ci_pr) {
-    try {
-      const { dispatchReview, dispatchComment } = await import("../ci/dispatch.js");
-      const { repo, prNumber } = config._ci_pr;
-      const bc = config.ci;
-
-      // Try to parse structured review from output
-      let review;
-      try {
-        review = JSON.parse(result.output);
-      } catch { /* output is not JSON */
-        review = { approved: true, summary: result.output };
-      }
-
-      const event = review.approved ? "APPROVE" : "REQUEST_CHANGES";
-      await dispatchReview({
-        repo, prNumber, event,
-        body: review.summary || result.output.slice(0, 500),
-        agent: "Reviewer", ciConfig: bc
-      });
-
-      await dispatchComment({
-        repo, prNumber, agent: "Reviewer",
-        body: `Standalone review: ${event}\n\n${review.summary || result.output.slice(0, 1000)}`,
-        ciConfig: bc
-      });
-
-      logger.info(`CI: dispatched review for PR #${prNumber}`);
-    } catch (err) {
-      logger.warn(`CI dispatch failed (non-blocking): ${err.message}`);
-    }
-  }
-  runLog.logText(`[reviewer] finished (exit=${result.exitCode})`);
-  return { ok: true };
-  });
-}

package/src/commands/run.js

@@ -1,185 +0,0 @@
-import { EventEmitter } from "node:events";
-import { runFlow } from "../orchestrator.js";
-import { assertAgentsAvailable } from "../agents/availability.js";
-import { createActivityLog } from "../activity-log.js";
-import { withCliRunLog } from "../utils/cli-run-log.js";
-import { registerRun, unregisterRun } from "../utils/run-registry.js";
-import { printHeader } from "../utils/display/header.js";
-import { printEvent } from "../utils/display/event-handlers.js";
-import { printResumeHint } from "../utils/display/resume-hint.js";
-import { resolveRole } from "../config.js";
-import { parseCardId } from "../planning-game/adapter.js";
-import { confirmCwd } from "../utils/cwd-confirm.js";
-import { runSpecReview } from "../spec-review/run-spec-review.js";
-import { maybeAutoUpdate } from "../rag/auto-update.js";
-
-// Re-export the shared helper so existing imports (tests, MCP wrappers)
-// keep working. KJC-BUG-0081 round 2 split the body into utils/ because
-// resume.js also needs the same contract.
-import { createCliAskQuestion } from "../utils/cli-ask-question.js";
-import { buildDecider } from "../autonomy/decider.js";
-import { createAutonomyAskQuestion } from "../autonomy/stage-ask.js";
-export { createCliAskQuestion };
-
-export async function runCommandHandler({ task, config, logger, flags }) {
-  // PR-F (cwd confirmation): when launching from an interactive
-  // terminal, surface the working directory so the user can abort if
-  // they typed `kj run` from the wrong place. The user's words:
-  // "todas las que lo lancé desde terminal te garantizo que estaba
-  // en la carpeta del proyecto … aún así preguntar por si acaso".
-  //
-  // Skip the prompt when:
-  //   - --yes / -y is set (CI / scripted runs),
-  //   - --json mode is on (machine-readable output, no TTY assumed),
-  //   - stdin is not a TTY (piped, board-spawned subprocess, MCP).
-  const projectDir = config?.projectDir || process.cwd();
-  const canPrompt = process.stdin.isTTY && process.stdout.isTTY
-    && !flags?.yes && !flags?.json;
-  if (canPrompt) {
-    const ok = await confirmCwd(projectDir);
-    if (!ok) {
-      console.log("Aborted by user.");
-      return { ok: false, aborted: true };
-    }
-  }
-
-  // KJC-TSK-0455 — Pre-run drift check. Default ON; opt out with
-  // --no-rag-update (flag) or config.rag.autoUpdate.onRun=false. If the
-  // local index is behind HEAD we re-embed the delta before the agents
-  // start reading from the RAG store; on any failure we log and move on
-  // so a broken vector store never blocks a run.
-  await maybeAutoUpdate({ projectDir, config, logger, flags });
-
-  // Best-effort session cleanup before starting
-  try {
-    const { cleanupExpiredSessions } = await import("../session-cleanup.js");
-    await cleanupExpiredSessions({ logger });
-  } catch { /* non-blocking */ }
-
-  const requiredProviders = [
-    resolveRole(config, "coder").provider
-  ];
-  if (config.pipeline?.reviewer?.enabled !== false) {
-    requiredProviders.push(resolveRole(config, "reviewer").provider);
-  }
-  if (config.pipeline?.triage?.enabled) requiredProviders.push(resolveRole(config, "triage").provider);
-  if (config.pipeline?.planner?.enabled) requiredProviders.push(resolveRole(config, "planner").provider);
-  if (config.pipeline?.refactorer?.enabled) requiredProviders.push(resolveRole(config, "refactorer").provider);
-  if (config.pipeline?.researcher?.enabled) requiredProviders.push(resolveRole(config, "researcher").provider);
-  if (config.pipeline?.tester?.enabled) requiredProviders.push(resolveRole(config, "tester").provider);
-  if (config.pipeline?.security?.enabled) requiredProviders.push(resolveRole(config, "security").provider);
-  await assertAgentsAvailable(requiredProviders);
-
-  // --- Planning Game: resolve card ID ---
-  const pgCardId = flags?.pgTask || parseCardId(task);
-  const pgProject = flags?.pgProject || config.planning_game?.project_id || null;
-
-  const jsonMode = flags?.json;
-  // Quiet mode is the default; --verbose disables it
-  const quietMode = config.output?.quiet !== false;
-
-  return withCliRunLog("run", { projectDir: config?.projectDir, logger }, async ({ runLog, forwardProgress }) => {
-    runLog.logText(`[kj_run] task="${String(task).slice(0, 80)}..."`);
-
-    // KJC-TSK-0396 extensión: registrar el PID + planId + huIds en
-    // ~/.karajan/runs/<runId>.json para que el HU Board pueda mostrar
-    // el botón ⏹ Stop incluso cuando el run se lanza desde terminal
-    // externa (no solo desde el botón ▶ Run del board). Best-effort:
-    // si la escritura falla, el run sigue su curso normal. Limpieza
-    // garantizada en el finally del withCliRunLog vía registry.unregisterRun.
-    const huIdsFromFlag = typeof flags?.hu === "string"
-      ? flags.hu.split(",").map((s) => s.trim()).filter(Boolean)
-      : null;
-    const runId = registerRun({
-      pid: process.pid,
-      planId: flags?.plan || null,
-      huIds: huIdsFromFlag,
-      projectDir: config?.projectDir || process.cwd(),
-      source: "cli",
-    });
-    const cleanupRegistry = () => { if (runId) unregisterRun(runId); };
-    // Cubrir todos los caminos de salida del proceso.
-    process.once("exit", cleanupRegistry);
-    process.once("SIGTERM", () => { cleanupRegistry(); process.exit(143); });
-    process.once("SIGINT", () => { cleanupRegistry(); process.exit(130); });
-
-    const emitter = new EventEmitter();
-    let activityLog = null;
-
-    // Mirror every progress event into .kj/run.log so kj-tail works identically
-    // whether the command came from the MCP server or the CLI (KJC-TSK-0327 follow-up).
-    forwardProgress(emitter);
-
-    emitter.on("progress", (event) => {
-      if (!activityLog && event.sessionId) {
-        activityLog = createActivityLog(event.sessionId);
-        logger.onLog((entry) => activityLog.write(entry));
-      }
-
-      if (activityLog) {
-        activityLog.writeEvent(event);
-      }
-
-      if (!jsonMode) {
-        printEvent(event, { quiet: quietMode });
-      }
-    });
-
-    if (!jsonMode) {
-      printHeader({ task: task, config });
-    }
-
-    const askQuestion = createCliAskQuestion({ flags });
-
-    // Autonomy (KJC-PCS-0062): wire the level to a resolver. In autonomous mode
-    // every gate is decided by the Arbiter instead of the human; interactive
-    // keeps the human prompts. The decider is threaded into the gates below.
-    const { autonomy, resolve: decide } = buildDecider({ config, flags, ask: askQuestion, logger });
-
-    // Spec-reviewer pre-pipeline audit (KJC-PCS-0048). Runs BEFORE
-    // anything else — surfaces ambiguity / missing scope / missing AC.
-    // In autonomous mode the Arbiter decides (never aborts); interactive
-    // lets the user bail. Bypass with --skip-spec-review.
-    const reviewResult = await runSpecReview({ spec: task, config, logger, askQuestion, flags, resolve: decide, autonomy });
-    if (!reviewResult.proceed) {
-      logger.info("Aborted by user after spec review.");
-      cleanupRegistry();
-      return { ok: false, aborted: true, reason: "spec-review-cancelled" };
-    }
-    // If the user refined the spec via [r]efine, downstream runs the
-    // REFINED text (not the original) so the pipeline acts on what the
-    // user actually approved.
-    const effectiveTask = reviewResult.refined && reviewResult.finalSpec ? reviewResult.finalSpec : task;
-
-    // AUTO-E: in autonomous mode the stages never ask a human or block on the
-    // board — they auto-continue with the least-bad choice. interactive/assisted
-    // keep the human prompts.
-    const stageAsk = createAutonomyAskQuestion({ baseAsk: askQuestion, autonomy, logger });
-
-    let result;
-    try {
-      result = await runFlow({ task: effectiveTask, config, logger, flags, emitter, askQuestion: stageAsk, pgTaskId: pgCardId || null, pgProject: pgProject || null });
-    } finally {
-      // KJC-TSK-0396: limpia el registro pase lo que pase (éxito,
-      // throw, o paused). Los handlers de SIGINT/SIGTERM/exit son
-      // backstop para casos donde el throw no llega aquí (segfault,
-      // kill -9, etc.).
-      cleanupRegistry();
-    }
-
-    if (jsonMode) {
-      console.log(JSON.stringify(result, null, 2));
-    } else {
-      // Last line printed: if the run stopped (hibernated / paused /
-      // failed), tell the user the exact command to resume it.
-      printResumeHint(result);
-    }
-    // Surface per-HU results so `kj autorun` can build a real outcome report
-    // (KJC-BUG-0090: it showed "0/0 met the ask" because only { ok } came back).
-    // The plan run returns them as `huResults` (run-hu-batch finalResult).
-    const hus = Array.isArray(result?.huResults)
-      ? result.huResults.map((r) => ({ id: r.huId, met: r.approved === true }))
-      : [];
-    return { ok: !result?.paused && result?.approved !== false, hus };
-  });
-}