@nk070281sjv/cli 2.3.18 → 2.3.20

package/dist/index.js

@@ -30779,7 +30779,7 @@ ${hint}
 }
 
 // src/lib/version.ts
-var CLI_VERSION = true ? "2.3.18" : createRequire(import.meta.url)("../../package.json").version;
+var CLI_VERSION = true ? "2.3.20" : createRequire(import.meta.url)("../../package.json").version;
 
 // src/lib/deps.ts
 init_src();
@@ -37432,7 +37432,6 @@ import { execFile } from "node:child_process";
 import { promisify } from "node:util";
 var execFileAsync = promisify(execFile);
 var GIT_SUMMARY_LIMIT = 2e4;
-var GIT_PATCH_LIMIT = 8e4;
 var REVIEW_SNAPSHOT_EXCLUDES = [".ocr", ".opencode"];
 async function prepareReviewContext(input) {
   const roundDir = join25(input.sessionDir, "rounds", `round-${input.round}`);
@@ -37451,11 +37450,11 @@ async function prepareReviewContext(input) {
   );
   const reviewBriefPath = join25(roundDir, "review-brief.md");
   const gitSnapshot = input.cwd ? await collectGitSnapshot(input.cwd, input.target) : void 0;
-  const diffPatchPath = gitSnapshot ? join25(roundDir, "diff.patch") : void 0;
-  if (gitSnapshot && diffPatchPath) {
+  const changedFilesPath = gitSnapshot ? join25(roundDir, "changed-files.md") : void 0;
+  if (gitSnapshot && changedFilesPath) {
     await writeFile2(
-      diffPatchPath,
-      gitSnapshot.diffPatch || "No textual diff patch was captured by the CLI.\n",
+      changedFilesPath,
+      changedFilesArtifact(gitSnapshot, input.cwd),
       "utf-8"
     );
   }
@@ -37468,7 +37467,7 @@ async function prepareReviewContext(input) {
       `Round: ${input.round}`,
       `Discovered standards: ${discoveredStandardsPath}`,
       `Shared context: ${contextPath}`,
-      ...diffPatchPath ? [`Diff patch: ${diffPatchPath}`] : [],
+      ...changedFilesPath ? [`Changed files artifact: ${changedFilesPath}`] : [],
       ...input.cwd ? [`Repository root: ${input.cwd}`] : [],
       "",
       "## Scope",
@@ -37529,7 +37528,7 @@ async function prepareReviewContext(input) {
           discovered_standards: discoveredStandardsPath,
           context: contextPath,
           review_brief: reviewBriefPath,
-          ...diffPatchPath ? { diff_patch: diffPatchPath } : {},
+          ...changedFilesPath ? { changed_files: changedFilesPath } : {},
           ...requirementsPath ? { requirements: requirementsPath } : {}
         }
       },
@@ -37546,7 +37545,7 @@ async function prepareReviewContext(input) {
     discoveredStandardsPath,
     contextPath,
     reviewBriefPath,
-    diffPatchPath,
+    changedFilesPath,
     requirementsPath,
     metadataPath,
     requirementsExpected: Boolean(requirementsPath)
@@ -37564,9 +37563,7 @@ async function collectGitSnapshot(cwd, target = ".") {
     untrackedFiles,
     branchFiles,
     worktreeDiffStat,
-    branchDiffStat,
-    worktreeDiffPatch,
-    branchDiffPatch
+    branchDiffStat
   ] = await Promise.all([
     runGit(cwd, ["status", "--short"]),
     runGit(cwd, ["diff", "--cached", "--name-only", "--", ...gitReviewPathspec()]),
@@ -37574,12 +37571,9 @@ async function collectGitSnapshot(cwd, target = ".") {
     runGit(cwd, ["ls-files", "--others", "--exclude-standard", "--", ...gitReviewPathspec()]),
     branchDiffArgs ? runGit(cwd, ["diff", "--name-only", ...branchDiffArgs]) : Promise.resolve(""),
     runGit(cwd, ["diff", "--stat", "HEAD", "--", ...gitReviewPathspec()]),
-    branchDiffArgs ? runGit(cwd, ["diff", "--stat", ...branchDiffArgs]) : Promise.resolve(""),
-    runGit(cwd, ["diff", "--patch", "HEAD", "--", ...gitReviewPathspec()], GIT_PATCH_LIMIT),
-    branchDiffArgs ? runGit(cwd, ["diff", "--patch", ...branchDiffArgs], GIT_PATCH_LIMIT) : Promise.resolve("")
+    branchDiffArgs ? runGit(cwd, ["diff", "--stat", ...branchDiffArgs]) : Promise.resolve("")
   ]);
   const untrackedFileList = filterIgnoredPaths(uniqueLines(untrackedFiles));
-  const untrackedDiffPatch = await collectUntrackedDiffPatch(cwd, untrackedFileList);
   const changedFiles = filterIgnoredPaths(uniqueLines([
     branchFiles,
     cachedFiles,
@@ -37592,44 +37586,31 @@ ${branchDiffStat}` : "",
     worktreeDiffStat ? `Working tree diff (HEAD):
 ${worktreeDiffStat}` : ""
   ]);
-  const diffPatch = truncate(uniqueSections([
-    branchDiffPatch ? `Branch diff (${baseRef}...HEAD):
-${branchDiffPatch}` : "",
-    worktreeDiffPatch ? `Working tree diff (HEAD):
-${worktreeDiffPatch}` : "",
-    untrackedDiffPatch ? `Untracked files diff:
-${untrackedDiffPatch}` : ""
-  ]), GIT_PATCH_LIMIT);
   return {
     status: filterIgnoredStatusLines(status),
     changedFiles,
     diffStat,
-    diffPatch,
     baseRef
   };
 }
-async function collectUntrackedDiffPatch(cwd, files) {
-  const patches = [];
-  let remaining = GIT_PATCH_LIMIT;
-  for (const file of files.split(/\r?\n/).filter(Boolean)) {
-    if (remaining <= 0) {
-      patches.push("[truncated by OCR CLI]");
-      break;
-    }
-    const patch = await runGit(cwd, ["diff", "--no-index", "--", "/dev/null", file], remaining);
-    if (patch) {
-      patches.push(patch);
-      remaining -= patch.length;
-    }
-  }
-  return patches.join("\n\n");
-}
 function gitReviewPathspec() {
   return [".", ...REVIEW_SNAPSHOT_EXCLUDES.flatMap((path2) => [
     `:(exclude)${path2}`,
     `:(exclude)${path2}/**`
   ])];
 }
+function changedFilesArtifact(snapshot, cwd) {
+  return [
+    "# Changed Files",
+    "",
+    ...cwd ? [`Repository root: ${cwd}`, ""] : [],
+    ...snapshot.baseRef ? [`Branch base: ${snapshot.baseRef}`, ""] : [],
+    "Only file paths are captured here. Reviewer agents should inspect the listed files and run targeted diffs when needed.",
+    "",
+    fenced("text", snapshot.changedFiles || "No changed files reported."),
+    ""
+  ].join("\n");
+}
 async function resolveDefaultRemoteBase(cwd) {
   const originHead = await runGit(cwd, ["symbolic-ref", "--quiet", "--short", "refs/remotes/origin/HEAD"]);
   if (originHead) return originHead;
@@ -38095,13 +38076,13 @@ async function reviewerSharedPrefix(context) {
     `- Discovered standards: ${context.discoveredStandardsPath}`,
     `- Shared context: ${context.contextPath}`,
     `- Review brief: ${context.reviewBriefPath}`,
-    ...context.diffPatchPath ? [`- Diff patch: ${context.diffPatchPath}`] : [],
+    ...context.changedFilesPath ? [`- Changed files: ${context.changedFilesPath}`] : [],
     ...context.requirementsPath ? [`- Requirements: ${context.requirementsPath}`] : [],
     "",
     embeddedFileSection("Discovered Standards", context.discoveredStandardsPath, await readTextFile(context.discoveredStandardsPath)),
     embeddedFileSection("Shared Review Context", context.contextPath, await readTextFile(context.contextPath)),
     embeddedFileSection("Review Brief", context.reviewBriefPath, await readTextFile(context.reviewBriefPath)),
-    ...context.diffPatchPath ? [embeddedFileSection("Diff Patch", context.diffPatchPath, await readTextFile(context.diffPatchPath))] : [],
+    ...context.changedFilesPath ? [embeddedFileSection("Changed Files", context.changedFilesPath, await readTextFile(context.changedFilesPath))] : [],
     ...context.requirementsPath ? [embeddedFileSection("Requirements", context.requirementsPath, await readTextFile(context.requirementsPath))] : [],
     ""
   ];
@@ -38748,10 +38729,15 @@ async function runUkrainianTranslation(input) {
     });
     return result;
   }
-  const translationErrors = validateUkrainianTranslationOutput(
-    readFileSync16(finalPath, "utf-8"),
+  const sourceFinalMarkdown = readFileSync16(finalPath, "utf-8");
+  const translatedMarkdown = normalizeUkrainianTranslationOutput(
+    sourceFinalMarkdown,
     processOutput(result)
   );
+  const translationErrors = validateUkrainianTranslationOutput(
+    sourceFinalMarkdown,
+    translatedMarkdown
+  );
   if (translationErrors.length > 0) {
     await patchProcessMeta(input.context.roundDir, logPaths.meta, {
       status: "failed",
@@ -38788,7 +38774,7 @@ async function runUkrainianTranslation(input) {
     });
     return { ...result, exitCode: 1 };
   }
-  await writeRoundArtifact(input.context.roundDir, ref.artifactPath, processOutput(result));
+  await writeRoundArtifact(input.context.roundDir, ref.artifactPath, translatedMarkdown);
   await journal.endInstance(agentSessionId, { exitCode: 0 });
   emit({
     event: "translation:complete",
@@ -38819,6 +38805,19 @@ function validateUkrainianTranslationOutput(sourceMarkdown, translatedMarkdown)
   }
   return [];
 }
+function normalizeUkrainianTranslationOutput(sourceMarkdown, translatedMarkdown) {
+  const sourceBlock = extractOcrJsonFence(sourceMarkdown);
+  if (!sourceBlock) return translatedMarkdown;
+  const translatedWithoutJson = translatedMarkdown.replace(/```ocr-json[\s\S]*?```/g, "").trimEnd();
+  return `${translatedWithoutJson}
+
+${sourceBlock}
+`;
+}
+function extractOcrJsonFence(markdown) {
+  const matches = markdown.match(/```ocr-json[\s\S]*?```/g);
+  return matches?.length === 1 ? matches[0] : void 0;
+}
 function sortResults(results, requests) {
   const order = new Map(requests.map((request, index) => [request.id, index]));
   return [...results].sort((a, b) => (order.get(a.id) ?? 0) - (order.get(b.id) ?? 0));

package/package.json

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