@dev.fast/whiteboard 0.2.3 → 0.2.4

package/src/review-api/review-progress.ts

@@ -79,7 +79,14 @@ export async function reviewProgress(
   // each resolve against the comparison their own pins name.
   const comparison: ComparisonCoverage = pins
     ? (partial ?? (await data.coverage(snapshot.reviewId, pins, mode)))
-    : { files: [], fileSources: new Map(), alignments: new Map() };
+    : {
+        counted: true,
+        files: [],
+        fileSources: new Map(),
+        alignments: new Map(),
+      };
+
+  const counting = !comparison.counted;
 
   signal.throwIfAborted();
 
@@ -98,7 +105,10 @@ export async function reviewProgress(
     if (partial) {
       const state = data.coverageSnapshot(snapshot.reviewId, own, mode);
 
-      if (state.pending) return undefined;
+      if (state.pending)
+        return state.comparison.counted
+          ? Promise.resolve(state.comparison)
+          : undefined;
     }
 
     loading = data.coverage(snapshot.reviewId, own, mode);
@@ -162,7 +172,7 @@ export async function reviewProgress(
 
     // Context-only files are absent from the changed-file stream. Read both pins
     // and require equality rather than inventing correspondence for missing diffs.
-    if (!file && partial && !own) {
+    if (!file && counting && !own) {
       pendingSources.add(key);
 
       return [];
@@ -228,9 +238,9 @@ export async function reviewProgress(
         return {
           ...lens,
           ...resolved,
-          pending: !!partial,
+          pending: counting,
           unavailable:
-            partial || resolved.fileCount
+            counting || resolved.fileCount
               ? undefined
               : "No files match these targets",
         };
@@ -257,7 +267,7 @@ export async function reviewProgress(
     ),
   );
 
-  const uncategorized = partial
+  const uncategorized = counting
     ? []
     : uncategorizedSources(
         files,
@@ -283,15 +293,15 @@ export async function reviewProgress(
           )?.path ?? source.file,
       ),
     ).size,
-    pending: !!partial,
+    pending: counting,
     unavailable:
-      partial || uncategorized.length
+      counting || uncategorized.length
         ? undefined
         : "All changed lines are covered by lenses",
   });
 
   const progress: ReviewProgress = {
-    complete: !partial,
+    complete: !counting,
     files,
     referenceFiles,
     lenses,

package/src/server/diffr-config.ts

@@ -51,7 +51,7 @@ function diffrArgv(args: DiffrArguments): string[] {
     case "show":
       return ["config", "show", "--json", ...(args.reveal ? ["--reveal"] : [])];
     case "set":
-      return ["config", "set", "--stdin", "--json"];
+      return ["config", "set", "-", "--json"];
     case "migrate":
       return ["config", "migrate", "--json"];
     case "diff":

package/src/server/structural-diff.ts

@@ -63,9 +63,11 @@ export async function* structuralDiff(
 
   args.push("--", ...(input.paths ?? []));
 
-  const idleAbort = new AbortController();
-  const idle = setTimeout(() => idleAbort.abort(), 120_000);
-  const signal = AbortSignal.any([idleAbort.signal, input.signal]);
+  // Plugins can exceed two minutes between records. Apply this timeout only
+  // until the manifest arrives; plugins enforce their own request timeouts.
+  const startupAbort = new AbortController();
+  const startup = setTimeout(() => startupAbort.abort(), 120_000);
+  const signal = AbortSignal.any([startupAbort.signal, input.signal]);
 
   // The host inherits its own environment and runs from the repository, so
   // diffr reads the user's config and keys exactly as it would from a shell.
@@ -103,13 +105,11 @@ export async function* structuralDiff(
 
   try {
     for await (const line of lines) {
-      // Large comparisons may take minutes while continuing to make progress.
-      idle.refresh();
-
       // Bound individual records, not the entire streamed comparison: a
       // directory move can legitimately contain thousands of small files.
-      if (Buffer.byteLength(line) > 64 * 1024 * 1024)
-        throw new Error("Structural diff record exceeded 64 MiB.");
+      // Generated parsers in diffr can exceed 64 MiB after JSON encoding.
+      if (Buffer.byteLength(line) > 96 * 1024 * 1024)
+        throw new Error("Structural diff record exceeded 96 MiB.");
 
       if (!line.trim()) continue;
       const event = decodeStructuralDiffEvent(line);
@@ -125,6 +125,7 @@ export async function* structuralDiff(
         }
 
         started = true;
+        clearTimeout(startup);
       } else if (event.type === "complete") {
         completed = true;
 
@@ -150,7 +151,7 @@ export async function* structuralDiff(
     if (code !== 0 && !(code === 2 && (failed > 0 || aborted !== undefined)))
       throw exitError(code);
   } finally {
-    clearTimeout(idle);
+    clearTimeout(startup);
     lines.close();
 
     if (child.exitCode === null) child.kill();

package/dist/agent-cli-50KJgMKi.js

@@ -1,2 +0,0 @@
-import { n as runReviewAgentCli } from "./agent-cli-DRRQfHQ-.js";
-export { runReviewAgentCli };

package/dist/profile-Cqztt-9-.js

@@ -1,2 +0,0 @@
-import { t as openReviewProfile } from "./profile-WzZu7Spt.js";
-export { openReviewProfile };