@nk070281sjv/cli 2.3.7 → 2.3.8

package/dist/index.js

@@ -30779,7 +30779,7 @@ ${hint}
 }
 
 // src/lib/version.ts
-var CLI_VERSION = true ? "2.3.7" : createRequire(import.meta.url)("../../package.json").version;
+var CLI_VERSION = true ? "2.3.8" : createRequire(import.meta.url)("../../package.json").version;
 
 // src/lib/deps.ts
 init_src();
@@ -37218,7 +37218,15 @@ var teamCommand = new Command("team").description("Resolve and persist team comp
 // src/commands/review.ts
 init_src();
 import { join as join28 } from "node:path";
-import { existsSync as existsSync23, readFileSync as readFileSync17 } from "node:fs";
+import {
+  closeSync,
+  existsSync as existsSync23,
+  mkdirSync as mkdirSync8,
+  openSync,
+  readdirSync as readdirSync9,
+  readFileSync as readFileSync17,
+  writeFileSync as writeFileSync9
+} from "node:fs";
 init_db();
 
 // ../shared/config/src/pipeline-config.ts
@@ -37413,6 +37421,7 @@ import { execFile } from "node:child_process";
 import { promisify } from "node:util";
 var execFileAsync = promisify(execFile);
 var GIT_SNAPSHOT_LIMIT = 2e4;
+var REVIEW_SNAPSHOT_EXCLUDES = [".ocr", ".opencode"];
 async function prepareReviewContext(input) {
   const roundDir = join25(input.sessionDir, "rounds", `round-${input.round}`);
   await mkdir2(roundDir, { recursive: true });
@@ -37444,7 +37453,7 @@ async function prepareReviewContext(input) {
       "## Scope",
       "",
       "Review the current working tree changes. Focus on changed files and the unchanged code needed to validate those changes.",
-      "Do not review generated OCR session artifacts under `.ocr/sessions/` unless they are explicitly part of the requested change.",
+      "Ignore local AI tooling/config churn under `.ocr/` and `.opencode/` unless the user explicitly asks to review OCR/OpenCode setup changes.",
       "",
       ...gitSnapshot ? [
         "## Git Status",
@@ -37517,17 +37526,20 @@ async function prepareReviewContext(input) {
 async function collectGitSnapshot(cwd) {
   const [status, cachedFiles, worktreeFiles, diffStat] = await Promise.all([
     runGit(cwd, ["status", "--short"]),
-    runGit(cwd, ["diff", "--cached", "--name-only"]),
-    runGit(cwd, ["diff", "--name-only", "HEAD"]),
-    runGit(cwd, ["diff", "--stat", "HEAD"])
+    runGit(cwd, ["diff", "--cached", "--name-only", "--", ...gitReviewPathspec()]),
+    runGit(cwd, ["diff", "--name-only", "HEAD", "--", ...gitReviewPathspec()]),
+    runGit(cwd, ["diff", "--stat", "HEAD", "--", ...gitReviewPathspec()])
   ]);
   return {
-    status,
-    changedFiles: uniqueLines(`${cachedFiles}
-${worktreeFiles}`),
+    status: filterIgnoredStatusLines(status),
+    changedFiles: filterIgnoredPaths(uniqueLines(`${cachedFiles}
+${worktreeFiles}`)),
     diffStat
   };
 }
+function gitReviewPathspec() {
+  return [".", ...REVIEW_SNAPSHOT_EXCLUDES.map((path2) => `:(exclude)${path2}`)];
+}
 async function runGit(cwd, args) {
   try {
     const { stdout, stderr } = await execFileAsync("git", args, {
@@ -37547,6 +37559,21 @@ function uniqueLines(value) {
   const lines = value.split(/\r?\n/).map((line) => line.trim()).filter(Boolean);
   return [...new Set(lines)].join("\n");
 }
+function filterIgnoredStatusLines(value) {
+  return value.split(/\r?\n/).filter((line) => {
+    const pathPart = line.slice(3).trim();
+    return !isIgnoredReviewPath(pathPart);
+  }).join("\n");
+}
+function filterIgnoredPaths(value) {
+  return value.split(/\r?\n/).filter((line) => !isIgnoredReviewPath(line.trim())).join("\n");
+}
+function isIgnoredReviewPath(path2) {
+  const normalized = path2.replace(/^"|"$/g, "");
+  return REVIEW_SNAPSHOT_EXCLUDES.some(
+    (ignored) => normalized === ignored || normalized.startsWith(`${ignored}/`)
+  );
+}
 function truncate(value) {
   if (value.length <= GIT_SNAPSHOT_LIMIT) return value;
   return `${value.slice(0, GIT_SNAPSHOT_LIMIT)}
@@ -38038,7 +38065,7 @@ async function runReviewerPhase(input) {
       total: requests.length,
       running: [...running].sort()
     });
-  }, 3e4);
+  }, 15e3);
   progressTimer.unref?.();
   const promises = reviewerRequests.map(async ({ reviewer, request }) => {
     const agentSessionId = await journal.startInstance({
@@ -38174,7 +38201,12 @@ async function runOpenCodeProcessAgentReview(input) {
       round: input.round,
       reviewers: snapshot.resolvedTeam.reviewers.length
     });
-    emit({ event: "reviews:start", session_id: input.sessionId, round: input.round });
+    emit({
+      event: "reviews:start",
+      session_id: input.sessionId,
+      round: input.round,
+      reviewers: snapshot.resolvedTeam.reviewers.length
+    });
     const reviewerResult = await runReviewerPhase({
       context,
       resolvedTeam: snapshot.resolvedTeam,
@@ -38268,7 +38300,21 @@ async function runPipelineStages(input) {
       prompt_path: request.promptPath,
       meta_log: startLogPaths.meta
     });
-    const result = await input.runner.run(request, new AbortController().signal);
+    const progressTimer = setInterval(() => {
+      emit({
+        event: "agent:progress",
+        phase: stage,
+        id: request.id,
+        status: "running"
+      });
+    }, 15e3);
+    progressTimer.unref?.();
+    let result;
+    try {
+      result = await input.runner.run(request, new AbortController().signal);
+    } finally {
+      clearInterval(progressTimer);
+    }
     const logPaths = await writeProcessLogs(input.context.roundDir, request, result);
     emit({
       event: "agent:complete",
@@ -38542,12 +38588,15 @@ function newSessionId() {
   const date = (/* @__PURE__ */ new Date()).toISOString().slice(0, 10);
   return `${date}-opencode-process-review-${Date.now()}`;
 }
-var runAgentsSubcommand = new Command("run-agents").description("Run OCR review through OpenCode process agents").argument("[target]", "Target path for fresh dashboard mode", ".").option("--session-id <workflow-session-id>", "Prepared workflow session id").option("--round <n>", "Prepared round number", (value) => Number.parseInt(value, 10)).option("--fresh", "Create a fresh workflow session before running process agents").option("--requirements <text-or-path>", "Inline requirements text or path to a requirements file").option("--team <json>", "JSON ReviewerInstance[] override for this run").action(
+var runAgentsSubcommand = new Command("run-agents").description("Run OCR review through OpenCode process agents").argument("[target]", "Target path for fresh dashboard mode", ".").option("--session-id <workflow-session-id>", "Prepared workflow session id").option("--round <n>", "Prepared round number", (value) => Number.parseInt(value, 10)).option("--fresh", "Create a fresh workflow session before running process agents").option("--requirements <text-or-path>", "Inline requirements text or path to a requirements file").option("--team <json>", "JSON ReviewerInstance[] override for this run").option("--foreground", "Run in the current process instead of detaching a background worker").addOption(new Option("--worker", "Internal detached worker mode").hideHelp()).action(
   async (target, options) => {
     const targetDir = process.cwd();
     requireOcrSetup(targetDir);
     const ocrDir = join28(targetDir, ".ocr");
     const db = await ensureDatabase(ocrDir);
+    let sessionDirForMeta;
+    let roundForMeta;
+    const modeForMeta = options.worker ? "worker" : options.foreground ? "foreground" : "detached";
     try {
       let sessionId = options.sessionId;
       let round = options.round;
@@ -38578,6 +38627,33 @@ var runAgentsSubcommand = new Command("run-agents").description("Run OCR review
         round = round ?? session.current_round;
         sessionDir = session.session_dir;
       }
+      sessionDirForMeta = sessionDir;
+      roundForMeta = round;
+      if (!options.foreground && !options.worker) {
+        const spawned = spawnDetachedRunAgents({
+          targetDir,
+          target,
+          sessionId,
+          sessionDir,
+          round,
+          requirements: options.requirements,
+          team: options.team
+        });
+        console.log(`OCR review started in background.`);
+        console.log(`Session: ${sessionId}`);
+        console.log(`Round: ${round}`);
+        console.log(`PID: ${spawned.pid}`);
+        console.log(`Progress: ocr review watch --session-id ${sessionId}`);
+        console.log(`Status: ocr review status --session-id ${sessionId}`);
+        console.log(`Artifacts: ${sessionDir}`);
+        return;
+      }
+      writeRunMeta(sessionDir, round, {
+        status: "running",
+        mode: options.worker ? "worker" : "foreground",
+        pid: process.pid,
+        started_at: (/* @__PURE__ */ new Date()).toISOString()
+      });
       const { team } = loadTeamConfig(ocrDir);
       const override = options.team ? parseTeamOverride(options.team) : void 0;
       const reviewers = resolveTeamComposition(team, override);
@@ -38591,18 +38667,226 @@ var runAgentsSubcommand = new Command("run-agents").description("Run OCR review
         pipelineAgents,
         requirements: readRequirements(options.requirements),
         journal: new PersistenceAgentLifecycleJournal(ocrDir),
-        emit: (event) => console.log(JSON.stringify(event))
+        emit: (event) => {
+          console.log(JSON.stringify(event));
+          renderRunAgentsProgress(event);
+        }
       });
       if (!result.ok) {
+        writeRunMeta(sessionDir, round, {
+          status: "failed",
+          mode: options.worker ? "worker" : "foreground",
+          pid: process.pid,
+          completed_at: (/* @__PURE__ */ new Date()).toISOString(),
+          errors: result.errors
+        });
         for (const error of result.errors) console.error(source_default.red(error));
         process.exit(1);
       }
+      writeRunMeta(sessionDir, round, {
+        status: "completed",
+        mode: options.worker ? "worker" : "foreground",
+        pid: process.pid,
+        completed_at: (/* @__PURE__ */ new Date()).toISOString()
+      });
       console.log(JSON.stringify({ event: "review:complete", session_id: sessionId, round }));
     } catch (error) {
+      if (sessionDirForMeta && roundForMeta) {
+        writeRunMeta(sessionDirForMeta, roundForMeta, {
+          status: "failed",
+          mode: modeForMeta,
+          pid: process.pid,
+          completed_at: (/* @__PURE__ */ new Date()).toISOString(),
+          errors: [error instanceof Error ? error.message : "Failed to run OpenCode process-agent review"]
+        });
+      }
       fail3(error instanceof Error ? error.message : "Failed to run OpenCode process-agent review");
     }
   }
 );
+var statusSubcommand2 = new Command("status").description("Show process-agent review status").requiredOption("--session-id <workflow-session-id>", "Workflow session id").option("--round <n>", "Round number", (value) => Number.parseInt(value, 10)).action(async (options) => {
+  const targetDir = process.cwd();
+  requireOcrSetup(targetDir);
+  const ocrDir = join28(targetDir, ".ocr");
+  const db = await ensureDatabase(ocrDir);
+  const session = getSession(db, options.sessionId);
+  if (!session) fail3(`Workflow session not found: ${options.sessionId}`);
+  renderReviewStatus(session.session_dir, options.round ?? session.current_round);
+});
+var watchSubcommand = new Command("watch").description("Watch process-agent review progress").requiredOption("--session-id <workflow-session-id>", "Workflow session id").option("--round <n>", "Round number", (value) => Number.parseInt(value, 10)).option("--interval <seconds>", "Refresh interval", (value) => Number.parseInt(value, 10), 2).action(async (options) => {
+  const targetDir = process.cwd();
+  requireOcrSetup(targetDir);
+  const ocrDir = join28(targetDir, ".ocr");
+  const db = await ensureDatabase(ocrDir);
+  const session = getSession(db, options.sessionId);
+  if (!session) fail3(`Workflow session not found: ${options.sessionId}`);
+  const round = options.round ?? session.current_round;
+  renderReviewStatus(session.session_dir, round);
+  setInterval(() => {
+    process.stdout.write("\x1Bc");
+    renderReviewStatus(session.session_dir, round);
+  }, Math.max(1, options.interval) * 1e3);
+});
+function spawnDetachedRunAgents(input) {
+  const roundDir = join28(input.sessionDir, "rounds", `round-${input.round}`);
+  mkdirSync8(join28(roundDir, "process-logs"), { recursive: true });
+  const stdoutPath = join28(roundDir, "process-logs", "run-agents-worker.stdout.log");
+  const stderrPath = join28(roundDir, "process-logs", "run-agents-worker.stderr.log");
+  const outFd = openSync(stdoutPath, "a");
+  const errFd = openSync(stderrPath, "a");
+  const entrypoint = process.argv[1];
+  if (!entrypoint) fail3("Cannot locate OCR CLI entrypoint for background worker.");
+  const args = [
+    entrypoint,
+    "review",
+    "run-agents",
+    input.target,
+    "--session-id",
+    input.sessionId,
+    "--round",
+    String(input.round),
+    "--worker",
+    "--foreground",
+    ...input.requirements ? ["--requirements", input.requirements] : [],
+    ...input.team ? ["--team", input.team] : []
+  ];
+  const child = spawnBinary(process.execPath, args, {
+    cwd: input.targetDir,
+    detached: true,
+    stdio: ["ignore", outFd, errFd]
+  });
+  child.unref();
+  closeSync(outFd);
+  closeSync(errFd);
+  writeRunMeta(input.sessionDir, input.round, {
+    status: "spawned",
+    mode: "detached",
+    pid: child.pid,
+    started_at: (/* @__PURE__ */ new Date()).toISOString(),
+    stdout_log: stdoutPath,
+    stderr_log: stderrPath
+  });
+  return { pid: child.pid };
+}
+function writeRunMeta(sessionDir, round, patch) {
+  const roundDir = join28(sessionDir, "rounds", `round-${round}`);
+  mkdirSync8(roundDir, { recursive: true });
+  const path2 = join28(roundDir, "run-meta.json");
+  let current = { schema_version: 1 };
+  if (existsSync23(path2)) {
+    try {
+      const parsed = JSON.parse(readFileSync17(path2, "utf-8"));
+      if (parsed && typeof parsed === "object" && !Array.isArray(parsed)) {
+        current = parsed;
+      }
+    } catch {
+      current = { schema_version: 1 };
+    }
+  }
+  writeFileSync9(path2, JSON.stringify({ ...current, ...patch }, null, 2), "utf-8");
+}
+function renderReviewStatus(sessionDir, round) {
+  const roundDir = join28(sessionDir, "rounds", `round-${round}`);
+  const processLogsDir = join28(roundDir, "process-logs");
+  const runMeta = readJson(join28(roundDir, "run-meta.json"));
+  console.log(source_default.bold(`OCR review status: ${sessionDir}`));
+  console.log(`Round: ${round}`);
+  if (runMeta) {
+    const pid = typeof runMeta.pid === "number" ? runMeta.pid : void 0;
+    console.log(
+      `Runner: ${String(runMeta.status ?? "unknown")}${pid ? ` pid=${pid} live=${isPidLive(pid) ? "yes" : "no"}` : ""}`
+    );
+  }
+  const metas = existsSync23(processLogsDir) ? readdirMetaFiles(processLogsDir).map((path2) => readJson(path2)).filter(isRecord3) : [];
+  const reviewers = metas.filter((meta) => meta.phase === "reviews");
+  const completedReviewers = reviewers.filter((meta) => meta.status === "completed");
+  const runningReviewers = reviewers.filter((meta) => meta.status === "running");
+  console.log(`Reviews: ${completedReviewers.length}/${reviewers.length || "?"} complete`);
+  if (runningReviewers.length > 0) {
+    console.log(`Running reviewers: ${runningReviewers.map((meta) => meta.id).join(", ")}`);
+  }
+  for (const phase of ["aggregation", "validation", "synthesis"]) {
+    const meta = metas.find((item) => item.id === phase);
+    console.log(`Pipeline ${phase}: ${String(meta?.status ?? "pending")}`);
+  }
+  if (existsSync23(join28(roundDir, "final.md"))) {
+    console.log(source_default.green(`Final: ${join28(roundDir, "final.md")}`));
+  }
+}
+function readdirMetaFiles(dir) {
+  return existsSync23(dir) ? Array.from(new Set(readdirSync9(dir))).filter((name) => name.endsWith(".meta.json")).sort().map((name) => join28(dir, name)) : [];
+}
+function readJson(path2) {
+  if (!existsSync23(path2)) return void 0;
+  try {
+    const parsed = JSON.parse(readFileSync17(path2, "utf-8"));
+    return isRecord3(parsed) ? parsed : void 0;
+  } catch {
+    return void 0;
+  }
+}
+function isPidLive(pid) {
+  try {
+    process.kill(pid, 0);
+    return true;
+  } catch {
+    return false;
+  }
+}
+function isRecord3(value) {
+  return typeof value === "object" && value !== null && !Array.isArray(value);
+}
+function renderRunAgentsProgress(event) {
+  const name = typeof event.event === "string" ? event.event : "";
+  if (name === "reviews:start") {
+    console.error(
+      source_default.cyan(
+        `OCR reviewers started: ${String(event.reviewers ?? "?")} agents running in parallel.`
+      )
+    );
+    return;
+  }
+  if (name === "agent:start") {
+    console.error(
+      source_default.dim(
+        `OCR agent started: ${String(event.id ?? "?")} (${String(event.phase ?? "?")}, ${String(event.model ?? "?")})`
+      )
+    );
+    return;
+  }
+  if (name === "reviews:progress") {
+    const running = Array.isArray(event.running) ? event.running.join(", ") : "?";
+    console.error(
+      source_default.cyan(
+        `OCR reviews progress: ${String(event.completed ?? "?")}/${String(event.total ?? "?")} complete; running: ${running || "none"}`
+      )
+    );
+    return;
+  }
+  if (name === "agent:complete") {
+    console.error(
+      source_default.green(
+        `OCR agent completed: ${String(event.id ?? "?")} exit=${String(event.exit_code ?? "?")}`
+      )
+    );
+    return;
+  }
+  if (name === "agent:progress") {
+    console.error(
+      source_default.cyan(
+        `OCR agent still running: ${String(event.id ?? "?")} (${String(event.phase ?? "?")})`
+      )
+    );
+    return;
+  }
+  if (name === "pipeline:start") {
+    console.error(source_default.cyan("OCR pipeline started: aggregation -> validation -> synthesis."));
+    return;
+  }
+  if (name === "pipeline:complete") {
+    console.error(source_default.green("OCR pipeline complete."));
+  }
+}
 var CONTROL_PROMPT = "Resume this OCR review: run `ocr state status --json` and act on `next_action`, continuing forward from `current_phase` without redoing completed phases.";
 var reviewCommand = new Command("review").description("Run or resume an OCR review").option("--resume <workflow-id>", "Resume a prior review by its workflow session id").action(async (options) => {
   if (!options.resume) {
@@ -38723,6 +39007,8 @@ var reviewCommand = new Command("review").description("Run or resume an OCR revi
   });
 });
 reviewCommand.addCommand(runAgentsSubcommand);
+reviewCommand.addCommand(statusSubcommand2);
+reviewCommand.addCommand(watchSubcommand);
 
 // src/commands/update.ts
 import { existsSync as existsSync24 } from "node:fs";
@@ -39409,7 +39695,7 @@ var pruneBackupsSubcommand = new Command("prune-backups").description("Delete ol
 var dbCommand = new Command("db").description("Inspect and maintain the OCR SQLite database").addCommand(doctorSubcommand).addCommand(vacuumSubcommand).addCommand(pruneSubcommand).addCommand(pruneBackupsSubcommand);
 
 // src/commands/reviewers.ts
-import { writeFileSync as writeFileSync9, renameSync as renameSync2 } from "node:fs";
+import { writeFileSync as writeFileSync10, renameSync as renameSync2 } from "node:fs";
 import { join as join33 } from "node:path";
 init_src();
 async function readStdin3() {
@@ -39518,7 +39804,7 @@ var syncSubcommand2 = new Command("sync").description("Sync reviewers-meta.json
       }
       const metaPath = join33(ocrDir, "reviewers-meta.json");
       const tmpPath = metaPath + ".tmp";
-      writeFileSync9(tmpPath, JSON.stringify(meta, null, 2) + "\n");
+      writeFileSync10(tmpPath, JSON.stringify(meta, null, 2) + "\n");
       renameSync2(tmpPath, metaPath);
       const tierCounts = meta.reviewers.reduce(
         (acc, r) => {
@@ -39548,7 +39834,7 @@ var syncSubcommand2 = new Command("sync").description("Sync reviewers-meta.json
     const meta = validateReviewersMeta(parsed);
     const metaPath = join33(ocrDir, "reviewers-meta.json");
     const tmpPath = metaPath + ".tmp";
-    writeFileSync9(tmpPath, JSON.stringify(meta, null, 2) + "\n");
+    writeFileSync10(tmpPath, JSON.stringify(meta, null, 2) + "\n");
     renameSync2(tmpPath, metaPath);
     const tierCounts = meta.reviewers.reduce(
       (acc, r) => {
@@ -39623,7 +39909,7 @@ var hostCommand = new Command("host").description("Inspect host (AI CLI) capabil
 // src/lib/update-check.ts
 import { homedir } from "node:os";
 import { join as join34 } from "node:path";
-import { readFileSync as readFileSync19, writeFileSync as writeFileSync10, mkdirSync as mkdirSync8 } from "node:fs";
+import { readFileSync as readFileSync19, writeFileSync as writeFileSync11, mkdirSync as mkdirSync9 } from "node:fs";
 var PACKAGE_NAME = "@open-code-review/cli";
 var REGISTRY_URL = `https://registry.npmjs.org/${PACKAGE_NAME}/latest`;
 var CACHE_DIR2 = join34(homedir(), ".ocr");
@@ -39639,8 +39925,8 @@ function readCache(cacheFile) {
 }
 function writeCache(cacheFile, cache2) {
   try {
-    mkdirSync8(join34(cacheFile, ".."), { recursive: true });
-    writeFileSync10(cacheFile, JSON.stringify(cache2));
+    mkdirSync9(join34(cacheFile, ".."), { recursive: true });
+    writeFileSync11(cacheFile, JSON.stringify(cache2));
   } catch {
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nk070281sjv/cli",
-  "version": "2.3.7",
+  "version": "2.3.8",
   "description": "CLI for Open Code Review - Multi-environment setup and progress tracking",
   "type": "module",
   "bin": {
@@ -37,7 +37,7 @@
   },
   "dependencies": {
     "@inquirer/prompts": "^7.2.0",
-    "@nk070281sjv/agents": "2.3.7",
+    "@nk070281sjv/agents": "2.3.8",
     "chalk": "^5.4.1",
     "chokidar": "^4.0.3",
     "commander": "^13.0.0",