@nk070281sjv/cli 2.3.6 → 2.3.7

package/dist/index.js

@@ -30779,7 +30779,7 @@ ${hint}
 }
 
 // src/lib/version.ts
-var CLI_VERSION = true ? "2.3.6" : createRequire(import.meta.url)("../../package.json").version;
+var CLI_VERSION = true ? "2.3.7" : createRequire(import.meta.url)("../../package.json").version;
 
 // src/lib/deps.ts
 init_src();
@@ -31821,9 +31821,9 @@ var NodeFsHandler = class {
     if (this.fsw.closed) {
       return;
     }
-    const dirname12 = sysPath.dirname(file);
+    const dirname13 = sysPath.dirname(file);
     const basename9 = sysPath.basename(file);
-    const parent = this.fsw._getWatchedDir(dirname12);
+    const parent = this.fsw._getWatchedDir(dirname13);
     let prevStats = stats;
     if (parent.has(basename9))
       return;
@@ -31850,7 +31850,7 @@ var NodeFsHandler = class {
             prevStats = newStats2;
           }
         } catch (error) {
-          this.fsw._remove(dirname12, basename9);
+          this.fsw._remove(dirname13, basename9);
         }
       } else if (parent.has(basename9)) {
         const at = newStats.atimeMs;
@@ -37409,6 +37409,10 @@ async function writeRoundArtifact(roundDir, relativePath, content) {
 import { mkdir as mkdir2, readFile, writeFile as writeFile2 } from "node:fs/promises";
 import { existsSync as existsSync21 } from "node:fs";
 import { dirname as dirname10, join as join25 } from "node:path";
+import { execFile } from "node:child_process";
+import { promisify } from "node:util";
+var execFileAsync = promisify(execFile);
+var GIT_SNAPSHOT_LIMIT = 2e4;
 async function prepareReviewContext(input) {
   const roundDir = join25(input.sessionDir, "rounds", `round-${input.round}`);
   await mkdir2(roundDir, { recursive: true });
@@ -37425,6 +37429,7 @@ async function prepareReviewContext(input) {
     "# Review Context\n\nNo shared review context has been recorded yet.\n"
   );
   const reviewBriefPath = join25(roundDir, "review-brief.md");
+  const gitSnapshot = input.cwd ? await collectGitSnapshot(input.cwd) : void 0;
   await writeFile2(
     reviewBriefPath,
     [
@@ -37434,6 +37439,37 @@ async function prepareReviewContext(input) {
       `Round: ${input.round}`,
       `Discovered standards: ${discoveredStandardsPath}`,
       `Shared context: ${contextPath}`,
+      ...input.cwd ? [`Repository root: ${input.cwd}`] : [],
+      "",
+      "## 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.",
+      "",
+      ...gitSnapshot ? [
+        "## Git Status",
+        "",
+        fenced("text", gitSnapshot.status || "No working tree changes reported."),
+        "",
+        "## Changed Files",
+        "",
+        fenced("text", gitSnapshot.changedFiles || "No changed files reported."),
+        "",
+        "## Diff Stat",
+        "",
+        fenced("text", gitSnapshot.diffStat || "No diff stat available."),
+        ""
+      ] : [
+        "## Git Snapshot",
+        "",
+        "No repository snapshot was captured by the CLI.",
+        ""
+      ],
+      "## Reviewer Guidance",
+      "",
+      "- Read this brief first, then inspect relevant changed files and surrounding code as needed.",
+      "- Prefer targeted `git diff -- <path>` and file reads over broad repository scans.",
+      "- Report only findings supported by code evidence and concrete file references.",
       ""
     ].join("\n"),
     "utf-8"
@@ -37478,6 +37514,49 @@ async function prepareReviewContext(input) {
     requirementsExpected: Boolean(requirementsPath)
   };
 }
+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"])
+  ]);
+  return {
+    status,
+    changedFiles: uniqueLines(`${cachedFiles}
+${worktreeFiles}`),
+    diffStat
+  };
+}
+async function runGit(cwd, args) {
+  try {
+    const { stdout, stderr } = await execFileAsync("git", args, {
+      cwd,
+      maxBuffer: GIT_SNAPSHOT_LIMIT * 2
+    });
+    return truncate(`${stdout}${stderr ? `
+${stderr}` : ""}`.trim());
+  } catch (error) {
+    if (error && typeof error === "object" && "stdout" in error && typeof error.stdout === "string") {
+      return truncate(error.stdout.trim());
+    }
+    return "";
+  }
+}
+function uniqueLines(value) {
+  const lines = value.split(/\r?\n/).map((line) => line.trim()).filter(Boolean);
+  return [...new Set(lines)].join("\n");
+}
+function truncate(value) {
+  if (value.length <= GIT_SNAPSHOT_LIMIT) return value;
+  return `${value.slice(0, GIT_SNAPSHOT_LIMIT)}
+[truncated by OCR CLI]`;
+}
+function fenced(language, value) {
+  return `\`\`\`${language}
+${value}
+\`\`\``;
+}
 async function ensureMarkdownFile(path2, fallbackContent) {
   if (existsSync21(path2)) {
     await readFile(path2, "utf-8");
@@ -37554,7 +37633,7 @@ var OpenCodeProcessRunner = class {
   constructor(options = {}) {
     this.spawn = options.spawn ?? spawnBinary;
     this.promptArgvLimit = options.promptArgvLimit ?? DEFAULT_PROMPT_ARGV_LIMIT;
-    this.serveReadyTimeoutMs = options.serveReadyTimeoutMs ?? 15e3;
+    this.serveReadyTimeoutMs = options.serveReadyTimeoutMs ?? 12e4;
     this.pure = options.pure ?? true;
   }
   async run(request, signal) {
@@ -37764,7 +37843,7 @@ function isRecord(value) {
 
 // src/lib/agent-orchestrator/prompt-writer.ts
 import { mkdir as mkdir3, writeFile as writeFile3 } from "node:fs/promises";
-import { join as join26, relative as relative3 } from "node:path";
+import { dirname as dirname11, join as join26, relative as relative3 } from "node:path";
 async function writePromptSnapshots(input) {
   const promptsDir = join26(input.context.roundDir, "prompts");
   const reviewsDir = join26(input.context.roundDir, "reviews");
@@ -37833,6 +37912,7 @@ function reviewerPrompt(context, reviewer, promptPath, reviewPath) {
     "Phase: reviews",
     "",
     "Input files to read before reviewing:",
+    `- ${reviewerPersonaPath(context, reviewer)}`,
     `- ${context.discoveredStandardsPath}`,
     `- ${context.contextPath}`,
     `- ${context.reviewBriefPath}`,
@@ -37846,8 +37926,14 @@ function reviewerPrompt(context, reviewer, promptPath, reviewPath) {
     `  - ${relative3(context.roundDir, context.discoveredStandardsPath)}`,
     `  - ${relative3(context.roundDir, context.contextPath)}`,
     `  - ${relative3(context.roundDir, context.reviewBriefPath)}`,
+    `  - ${relative3(context.roundDir, reviewerPersonaPath(context, reviewer))}`,
     ...context.requirementsPath ? [`  - ${relative3(context.roundDir, context.requirementsPath)}`] : [],
     "",
+    "Review focus:",
+    "- Apply the reviewer persona file as your primary review lens.",
+    "- Use the review brief as the source of truth for changed files and scope.",
+    "- Inspect surrounding source code only when needed to validate a finding.",
+    "",
     "Output contract:",
     `- Return review markdown through stdout for ${reviewPath}.`,
     "- Do not write files directly.",
@@ -37862,6 +37948,10 @@ function reviewerPrompt(context, reviewer, promptPath, reviewPath) {
     ""
   ].join("\n");
 }
+function reviewerPersonaPath(context, reviewer) {
+  const ocrDir = dirname11(dirname11(context.sessionDir));
+  return join26(ocrDir, "skills", "references", "reviewers", `${reviewer.persona}.md`);
+}
 function pipelinePrompt(context, stage, agent, promptPath, artifactPath) {
   return [
     `# Pipeline Agent: ${stage}`,
@@ -37938,6 +38028,18 @@ async function runReviewerPhase(input) {
     };
   });
   const requests = reviewerRequests.map((item) => item.request);
+  const running = new Set(requests.map((request) => request.id));
+  const completed = /* @__PURE__ */ new Set();
+  const progressTimer = setInterval(() => {
+    emit({
+      event: "reviews:progress",
+      phase: "reviews",
+      completed: completed.size,
+      total: requests.length,
+      running: [...running].sort()
+    });
+  }, 3e4);
+  progressTimer.unref?.();
   const promises = reviewerRequests.map(async ({ reviewer, request }) => {
     const agentSessionId = await journal.startInstance({
       workflowId: input.context.sessionId,
@@ -37959,6 +38061,8 @@ async function runReviewerPhase(input) {
     });
     const result = await input.runner.run(request, controller.signal);
     const logPaths = await writeProcessLogs(input.context.roundDir, request, result);
+    running.delete(request.id);
+    completed.add(request.id);
     emit({
       event: "agent:complete",
       phase: "reviews",
@@ -37977,15 +38081,19 @@ async function runReviewerPhase(input) {
     return result;
   });
   const results = [];
-  await Promise.all(
-    promises.map(async (promise) => {
-      const result = await promise;
-      results.push(result);
-      if (result.exitCode !== 0) {
-        controller.abort();
-      }
-    })
-  );
+  try {
+    await Promise.all(
+      promises.map(async (promise) => {
+        const result = await promise;
+        results.push(result);
+        if (result.exitCode !== 0) {
+          controller.abort();
+        }
+      })
+    );
+  } finally {
+    clearInterval(progressTimer);
+  }
   const failed = results.find((result) => result.exitCode !== 0);
   if (failed) {
     const failedRequest = requests.find((request) => request.id === failed.id);
@@ -38052,7 +38160,8 @@ async function runOpenCodeProcessAgentReview(input) {
       sessionId: input.sessionId,
       sessionDir: input.sessionDir,
       round: input.round,
-      requirements: input.requirements
+      requirements: input.requirements,
+      cwd: input.cwd
     });
     const snapshot = await writePromptSnapshots({
       context,
@@ -38254,6 +38363,9 @@ async function writeProcessLogs(roundDir, request, result) {
   const stdoutPath = `${prefix}.stdout.ndjson`;
   const stderrPath = `${prefix}.stderr.log`;
   const metaPath = `${prefix}.meta.json`;
+  const absoluteMetaPath = join27(roundDir, metaPath);
+  const startedAt = readStartedAt(absoluteMetaPath) ?? (/* @__PURE__ */ new Date()).toISOString();
+  const completedAt = (/* @__PURE__ */ new Date()).toISOString();
   await writeRoundArtifact(roundDir, stdoutPath, result.stdout);
   await writeRoundArtifact(roundDir, stderrPath, result.stderr);
   await writeRoundArtifact(
@@ -38267,7 +38379,9 @@ async function writeProcessLogs(roundDir, request, result) {
         model: request.model,
         prompt_path: request.promptPath,
         status: "completed",
-        completed_at: (/* @__PURE__ */ new Date()).toISOString(),
+        started_at: startedAt,
+        completed_at: completedAt,
+        duration_ms: Math.max(0, Date.parse(completedAt) - Date.parse(startedAt)),
         exit_code: result.exitCode,
         output_text_bytes: Buffer.byteLength(processOutput(result), "utf-8"),
         stdout_bytes: Buffer.byteLength(result.stdout, "utf-8"),
@@ -38280,6 +38394,16 @@ async function writeProcessLogs(roundDir, request, result) {
   );
   return { stdout: stdoutPath, stderr: stderrPath, meta: metaPath };
 }
+function readStartedAt(path2) {
+  if (!existsSync22(path2)) return void 0;
+  try {
+    const parsed = JSON.parse(readFileSync16(path2, "utf-8"));
+    if (!isRecord2(parsed)) return void 0;
+    return typeof parsed.started_at === "string" ? parsed.started_at : void 0;
+  } catch {
+    return void 0;
+  }
+}
 function processOutput(result) {
   return result.outputText ?? result.stdout;
 }
@@ -38760,12 +38884,12 @@ var updateCommand = new Command("update").description("Update OCR assets after p
 
 // src/commands/dashboard.ts
 import { existsSync as existsSync25 } from "node:fs";
-import { join as join30, dirname as dirname11 } from "node:path";
+import { join as join30, dirname as dirname12 } from "node:path";
 import { fileURLToPath } from "node:url";
 init_src();
 init_db();
 var __filename = fileURLToPath(import.meta.url);
-var __dirname = dirname11(__filename);
+var __dirname = dirname12(__filename);
 function resolveServerPath() {
   return join30(__dirname, "dashboard", "server.js");
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nk070281sjv/cli",
-  "version": "2.3.6",
+  "version": "2.3.7",
   "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.6",
+    "@nk070281sjv/agents": "2.3.7",
     "chalk": "^5.4.1",
     "chokidar": "^4.0.3",
     "commander": "^13.0.0",