@dev.fast/whiteboard 0.2.3 → 0.2.4

package/dist/server/desktop-host.js

@@ -1,20 +1,20 @@
 #!/usr/bin/env node
 import { a as diffFileSummaries, b as resolveRevision, i as diff, x as splitGitPatchFiles } from "../src-CmdiBL20.js";
 import { c as jsonString, i as jsonBoolean, n as isJsonObject, p as isObjectValue, s as jsonProperty, u as parseJsonText } from "../src-X9phtB2j.js";
-import { i as SharedReviewStore } from "../client-D01H2KOV.js";
-import { c as ReviewBugReportRequestSchema, l as parseReviewBugReportResponse, o as parseZod, s as ReviewBugReportMetaV2Schema, t as parseReviewCliInstallApplyRequest, v as reviewDiffrSummarizerInputSchema } from "../src-DHiDJQDy.js";
-import { C as resourceReferences, p as documentSchema } from "../stored-document-migration-T9Uj5MkN.js";
+import { i as SharedReviewStore } from "../client-BFClfQLO.js";
+import { c as ReviewBugReportRequestSchema, l as parseReviewBugReportResponse, o as parseZod, s as ReviewBugReportMetaV2Schema, t as parseReviewCliInstallApplyRequest, v as reviewDiffrSummarizerInputSchema } from "../src-WCbabvdS.js";
+import { C as resourceReferences, p as documentSchema } from "../stored-document-migration-BxZWQQjv.js";
 import { t as ReviewInputError } from "../input-error-OLgB_h31.js";
 import { A as devReviewHome, P as writePrivateJsonAtomic } from "../s3-config-CAxhO9u_.js";
 import { U as shellQuote, i as reviewInstanceDiscoveryPath, o as reviewLegacyDiscoveryPath } from "../review-home-paths-6zZH-1c9.js";
 import { i as readReviewPackageVersion, t as findReviewPackageRoot } from "../package-paths-B6-zxvIO.js";
-import { c as reviewInstanceIdentity, t as REVIEW_INSTANCE_ENV } from "../desktop-discovery-DK2HoBy5.js";
-import { t as openReviewProfile } from "../profile-WzZu7Spt.js";
-import { r as reviewTelemetryChannel, t as ReviewTelemetry } from "../review-telemetry-Pp4ZsVrh.js";
-import { o as REVIEW_TELEMETRY_TABS } from "../request-origin-DlHTFCrQ.js";
-import { c as removeLegacyReviewSkills, d as skipCliInstall, i as finishCliInstallUpdate, l as resetCliInstall, n as applyCliInstall, r as declineCliInstall, s as removeCliInstall, t as aliasInstallationToAccount, u as resolveCliInstallStatus } from "../account-alias-DLE9XZhi.js";
-import { _ as setDiffrConfigValue, a as serverJson, c as recordClientError, d as HttpJsonError, f as ReviewServerError, g as saveDiffrSummarizer, h as readDiffrConfig, i as createWhiteboardCore, l as createNodeRequestListener, m as migrateDiffrConfig, n as installProcessErrorTelemetry, o as captureSanitizedUiTelemetry, p as GlobalReviewDesktopVerbRelay, r as serverCrashReportPath, s as clientErrorsForSession, t as drainServerCrashReport, u as readBoundedRequestJson, v as testDiffrSummarizer } from "../process-error-telemetry-mzRG988B.js";
-import { c as readQuerySchemas } from "../authoring-tools-F_q8cC7J.js";
+import { c as reviewInstanceIdentity, t as REVIEW_INSTANCE_ENV } from "../desktop-discovery-C-Qz54wO.js";
+import { t as openReviewProfile } from "../profile-RDEioF8b.js";
+import { r as reviewTelemetryChannel, t as ReviewTelemetry } from "../review-telemetry-DBy3eVCo.js";
+import { o as REVIEW_TELEMETRY_TABS } from "../request-origin-DA4K1qWY.js";
+import { c as removeLegacyReviewSkills, d as skipCliInstall, i as finishCliInstallUpdate, l as resetCliInstall, n as applyCliInstall, r as declineCliInstall, s as removeCliInstall, t as aliasInstallationToAccount, u as resolveCliInstallStatus } from "../account-alias-_1_7vROx.js";
+import { _ as setDiffrConfigValue, a as serverJson, c as recordClientError, d as HttpJsonError, f as ReviewServerError, g as saveDiffrSummarizer, h as readDiffrConfig, i as createWhiteboardCore, l as createNodeRequestListener, m as migrateDiffrConfig, n as installProcessErrorTelemetry, o as captureSanitizedUiTelemetry, p as GlobalReviewDesktopVerbRelay, r as serverCrashReportPath, s as clientErrorsForSession, t as drainServerCrashReport, u as readBoundedRequestJson, v as testDiffrSummarizer } from "../process-error-telemetry-Dzh5a3s_.js";
+import { c as readQuerySchemas } from "../authoring-tools-DvIO906l.js";
 import crypto, { createHash, randomBytes, randomUUID } from "node:crypto";
 import { z } from "zod";
 import fs, { existsSync } from "node:fs";

package/dist/sharing/index.d.ts

@@ -67,7 +67,7 @@ interface LocalVcsCommitSummary {
   deletions: number;
 }
 //#endregion
-//#region ../../node_modules/@dev.fast/diffr/dist/contract.d.ts
+//#region ../../diffr/diffr-ts/dist/contract.d.ts
 /** A pairing always has at least one side; additions/deletions omit the other. */
 type StructuralPairing<T> = {
   lhs: T;
@@ -171,6 +171,8 @@ type StructuralRegion = {
 type StructuralNode = {
   kind: "leaf";
   alignment_id: number;
+  /** The partner leaf's `id` on the other side; absent when added or deleted. */
+  pair?: number;
   changed?: StructuralSpan[];
 } | {
   kind: "fold";
@@ -310,7 +312,7 @@ declare const fileLineRangeSchema: z.ZodObject<{
 type FileLineRange = z.infer<typeof fileLineRangeSchema>;
 //#endregion
 //#region src/lens-selection.d.ts
-type AlignmentRow$1 = readonly [number | null, number | null];
+type AlignmentRow = readonly [number | null, number | null];
 //#endregion
 //#region src/viewed-coverage.d.ts
 /** Zero-based, half-open intervals; always normalized at the storage boundary. */
@@ -319,7 +321,7 @@ interface Coverage {
   base: LineInterval[];
   head: LineInterval[];
 }
-interface CoverageFile$1 {
+interface CoverageFile {
   path: string;
   previousPath?: string;
   fingerprint: string;
@@ -1038,9 +1040,11 @@ interface WrittenComponent {
 //#region src/review-api/comparison-coverage.d.ts
 type CoverageMode = "structural" | "textual";
 interface ComparisonCoverage {
-  files: CoverageFile$1[];
+  /** All Git changes are counted, even if folding is still loading. */
+  counted: boolean;
+  files: CoverageFile[];
   fileSources: Map<string, FileLineRange[]>;
-  alignments: Map<string, readonly AlignmentRow$1[]>;
+  alignments: Map<string, readonly AlignmentRow[]>;
 }
 //#endregion
 //#region src/review-api/pull-request.d.ts
@@ -1694,6 +1698,7 @@ declare class LocalReviewData {
   coverageSnapshot(reviewId: string, pins: Pins, mode: CoverageMode): {
     pending: boolean;
     comparison: ComparisonCoverage | {
+      counted: false;
       files: never[];
       fileSources: Map<any, any>;
       alignments: Map<any, any>;
@@ -1701,14 +1706,10 @@ declare class LocalReviewData {
   };
   private coverageNotification;
   private notifyCoverage;
-  coverage(reviewId: string, pins: Pins, mode: CoverageMode): Promise<{
-    files: CoverageFile[];
-    fileSources: Map<string, FileLineRange[]>;
-    alignments: Map<string, readonly AlignmentRow[]>;
-  }>;
+  coverage(reviewId: string, pins: Pins, mode: CoverageMode): Promise<ComparisonCoverage>;
   changes(pins: Pins): Promise<LocalVcsDiffFileSummary[]>;
-  changes(pins: Pins, file: string): Promise<string>;
-  changes(pins: Pins, file?: string): Promise<LocalVcsDiffFileSummary[] | string>;
+  changes(pins: Pins, file: string, previousPath?: string): Promise<string>;
+  changes(pins: Pins, file?: string, previousPath?: string): Promise<LocalVcsDiffFileSummary[] | string>;
   private summaries;
   /** Raw Git patch text for exact filenames, or every change. */
   private rawPatch;

package/dist/sharing/index.js

@@ -1,2 +1,2 @@
-import { a as sharedReviewId, c as verifyShareRepository, i as SharedReviewStore, l as exportShare, n as ShareClient, o as validateShareBundle } from "../client-D01H2KOV.js";
+import { a as sharedReviewId, c as verifyShareRepository, i as SharedReviewStore, l as exportShare, n as ShareClient, o as validateShareBundle } from "../client-BFClfQLO.js";
 export { ShareClient, SharedReviewStore, exportShare, sharedReviewId, validateShareBundle, verifyShareRepository };

package/dist/{src-DHiDJQDy.js → src-WCbabvdS.js}

@@ -1,7 +1,7 @@
 import { n as isJsonObject } from "./src-X9phtB2j.js";
 import { _ as ReviewAgentTraceEventSchema, v as ReviewAgentTraceSessionSchema } from "./src-CGP5ytbV.js";
 import { z } from "zod";
-//#region ../../node_modules/@dev.fast/diffr/dist/contract.js
+//#region ../../diffr/diffr-ts/dist/contract.js
 /**
 * Mirrors crates/diffr-core/src/protocol/mod.rs and crates/diffr-core/src/pairing.rs. See ../README.md for sync checks.
 * Import only zod: Review embeds this file in generated source.
@@ -128,6 +128,7 @@ const StructuralRegionSchema = z.lazy(() => z.object({
 const StructuralNodeSchema = z.lazy(() => z.union([z.object({
 	kind: z.literal("leaf"),
 	alignment_id: structuralU32,
+	pair: structuralU32.optional(),
 	changed: z.array(StructuralSpanSchema).optional()
 }), z.object({
 	kind: z.literal("fold"),
@@ -158,7 +159,7 @@ const StructuralLineCountsSchema = z.lazy(() => z.object({
 	removed: structuralU32
 }));
 //#endregion
-//#region ../../node_modules/@dev.fast/diffr/dist/stream.js
+//#region ../../diffr/diffr-ts/dist/stream.js
 function decodeStructuralDiffEvent(line) {
 	try {
 		return StructuralDiffEventSchema.parse(JSON.parse(line));

package/dist/{stored-document-migration-T9Uj5MkN.js → stored-document-migration-BxZWQQjv.js}

@@ -1,5 +1,5 @@
 import { f as isNumberValue, l as jsonValueSchema, m as isStringValue, n as isJsonObject } from "./src-X9phtB2j.js";
-import "./src-DHiDJQDy.js";
+import "./src-WCbabvdS.js";
 import { t as ReviewInputError } from "./input-error-OLgB_h31.js";
 import { z } from "zod";
 import { fromMarkdown } from "mdast-util-from-markdown";

package/dist/{tool-failure-Djeeax6q.js → tool-failure-DJmTieh5.js}

@@ -1,5 +1,5 @@
 import { m as isStringValue, n as isJsonObject } from "./src-X9phtB2j.js";
-import { n as callAuthoringTool, s as ReviewApiError, t as ToolText } from "./agent-client-DoaODH6I.js";
+import { n as callAuthoringTool, s as ReviewApiError, t as ToolText } from "./agent-client-B_b1GtUc.js";
 import { t as ReviewInputError } from "./input-error-OLgB_h31.js";
 //#region src/review-api/public-tools.ts
 function publicDescription(text) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dev.fast/whiteboard",
-  "version": "0.2.3",
+  "version": "0.2.4",
   "private": false,
   "license": "MIT",
   "repository": {
@@ -43,7 +43,6 @@
     "@agentclientprotocol/claude-agent-acp": "0.84.0",
     "@agentclientprotocol/codex-acp": "2.0.0",
     "@agentclientprotocol/sdk": "1.5.1",
-    "@dev.fast/diffr": "0.1.15",
     "@hono/node-server": "2.0.12",
     "@img/sharp-wasm32": "0.35.5",
     "@modelcontextprotocol/sdk": "1.31.0",
@@ -59,7 +58,8 @@
     "semver": "7.8.5",
     "sharp": "0.35.5",
     "smol-toml": "1.9.0",
-    "zod": "4.6.5"
+    "zod": "4.6.5",
+    "@dev.fast/diffr": "0.1.17"
   },
   "devDependencies": {
     "@dev.fast/local-vcs": "^0.1.0",
@@ -85,14 +85,14 @@
     "repo": "devdotfast/review",
     "docs": "packages/review/instructions/authoring.md"
   },
-  "gitHead": "29a00b6b1c647529bfe19e0571cae2336a44356b",
+  "gitHead": "5b00f0e66d034e187a29d270b3690a37b57ccf7b",
   "scripts": {
     "prebuild": "pnpm run build:workspace-deps",
     "build": "node --max-old-space-size=8192 node_modules/tsdown/dist/run.mjs",
     "build:tutorial-assets": "tsx scripts/build-tutorial-assets.ts",
     "ensure:tutorial-assets": "tsx scripts/ensure-tutorial-assets.ts",
     "check:tutorial": "tsx scripts/check-tutorial.ts",
-    "build:workspace-deps": "pnpm --filter @dev.fast/json --filter @dev.fast/local-vcs --filter @dev.fast/review-protocol --filter @dev.fast/review-share-protocol --filter @dev.fast/trace-protocol --filter @dev.fast/trace-core build",
+    "build:workspace-deps": "pnpm --filter @dev.fast/diffr --filter @dev.fast/json --filter @dev.fast/local-vcs --filter @dev.fast/review-protocol --filter @dev.fast/review-share-protocol --filter @dev.fast/trace-protocol --filter @dev.fast/trace-core build",
     "pretypecheck": "pnpm run ensure:tutorial-assets",
     "pretest": "pnpm run ensure:tutorial-assets",
     "pretest:node": "pnpm run ensure:tutorial-assets",

package/src/review-api/comparison-coverage.ts

@@ -29,6 +29,8 @@ const hash = (parts: (string | null)[]) =>
   createHash("sha256").update(JSON.stringify(parts)).digest("hex");
 
 export interface ComparisonCoverage {
+  /** All Git changes are counted, even if folding is still loading. */
+  counted: boolean;
   files: CoverageFile[];
   fileSources: Map<string, FileLineRange[]>;
   alignments: Map<string, readonly AlignmentRow[]>;
@@ -42,196 +44,221 @@ export async function comparisonCoverage(
   mode: CoverageMode,
   signal: AbortSignal,
   publish?: (coverage: ComparisonCoverage) => void,
-) {
+): Promise<ComparisonCoverage> {
   const fileSources = new Map<string, FileLineRange[]>();
   const alignments = new Map<string, readonly AlignmentRow[]>();
 
   const files: CoverageFile[] = [];
 
-  if (mode === "textual") {
-    for (const file of await data.changes(pins)) {
-      signal.throwIfAborted();
-      const patch = await data.changes(pins, file.path);
-      const changed = emptyCoverage();
-
-      for (const hunk of parseUnifiedPatch(file.path, patch))
-        for (const line of hunk.lines) {
-          if (line.kind === "add")
-            changed.head.push([line.newLine! - 1, line.newLine!]);
-
-          if (line.kind === "remove")
-            changed.base.push([line.oldLine! - 1, line.oldLine!]);
-        }
-
-      changed.base = unionIntervals(changed.base);
-      changed.head = unionIntervals(changed.head);
-
-      const [base, head] = await Promise.all([
-        file.status === "added"
-          ? null
-          : data
-              .file(pins, "base", file.previousPath ?? file.path)
-              .then((value) => value.text)
-              .catch(() => null),
-        file.status === "deleted"
-          ? null
-          : data
-              .file(pins, "head", file.path)
-              .then((value) => value.text)
-              .catch(() => null),
-      ]);
-
-      alignments.set(
-        file.path,
-        textualRows(
-          file.path,
-          patch,
-          base === null ? 0 : base.split("\n").length,
-          head === null ? 0 : head.split("\n").length,
-        ),
-      );
+  for (const file of await data.changes(pins)) {
+    signal.throwIfAborted();
+    const patch = await data.changes(pins, file.path, file.previousPath);
+    const changed = emptyCoverage();
 
-      const readable =
-        (file.status === "added" || base !== null) &&
-        (file.status === "deleted" || head !== null);
-
-      const fingerprint = hash([
-        pins.repositoryId,
-        base,
-        head,
-        ...(readable ? [] : [pins.base, pins.head, patch]),
-      ]);
-
-      fileSources.set(file.path, [
-        ...(base !== null
-          ? [
-              {
-                side: "base" as const,
-                file: file.previousPath ?? file.path,
-                fromLine: 1,
-                toLine: base.split("\n").length,
-              },
-            ]
-          : []),
-        ...(head !== null
-          ? [
-              {
-                side: "head" as const,
-                file: file.path,
-                fromLine: 1,
-                toLine: head.split("\n").length,
-              },
-            ]
-          : []),
-      ]);
-      files.push({
-        path: file.path,
-        previousPath: file.previousPath,
-        fingerprint,
-        changed,
-        viewed: emptyCoverage(),
-      });
+    for (const hunk of parseUnifiedPatch(file.path, patch))
+      for (const line of hunk.lines) {
+        if (line.kind === "add")
+          changed.head.push([line.newLine! - 1, line.newLine!]);
+
+        if (line.kind === "remove")
+          changed.base.push([line.oldLine! - 1, line.oldLine!]);
+      }
+
+    changed.base = unionIntervals(changed.base);
+    changed.head = unionIntervals(changed.head);
+
+    const [base, head] = await Promise.all([
+      file.status === "added"
+        ? null
+        : data
+            .file(pins, "base", file.previousPath ?? file.path)
+            .then((value) => value.text)
+            .catch(() => null),
+      file.status === "deleted"
+        ? null
+        : data
+            .file(pins, "head", file.path)
+            .then((value) => value.text)
+            .catch(() => null),
+    ]);
+
+    alignments.set(
+      file.path,
+      textualRows(
+        file.path,
+        patch,
+        base === null ? 0 : base.split("\n").length,
+        head === null ? 0 : head.split("\n").length,
+      ),
+    );
+
+    const readable =
+      (file.status === "added" || base !== null) &&
+      (file.status === "deleted" || head !== null);
+
+    const fingerprint = hash([
+      pins.repositoryId,
+      base,
+      head,
+      ...(readable ? [] : [pins.base, pins.head, patch]),
+    ]);
+
+    fileSources.set(file.path, [
+      ...(base !== null
+        ? [
+            {
+              side: "base" as const,
+              file: file.previousPath ?? file.path,
+              fromLine: 1,
+              toLine: base.split("\n").length,
+            },
+          ]
+        : []),
+      ...(head !== null
+        ? [
+            {
+              side: "head" as const,
+              file: file.path,
+              fromLine: 1,
+              toLine: head.split("\n").length,
+            },
+          ]
+        : []),
+    ]);
+    files.push({
+      path: file.path,
+      previousPath: file.previousPath,
+      fingerprint,
+      changed,
+      viewed: emptyCoverage(),
+    });
+
+    if (mode === "textual")
       publish?.({
+        counted: false,
         files: [...files],
         fileSources: new Map(fileSources),
         alignments: new Map(alignments),
       });
-    }
-  } else {
-    const remaining = new Set<string>();
-
-    for await (const event of data.structuralChanges({
-      reviewId,
-      pins,
-      signal,
-    })) {
-      if (event.type === "start") {
-        for (const entry of event.files)
-          remaining.add((entry.file.rhs ?? entry.file.lhs)!.path);
-
-        if (!remaining.size) break;
-        continue;
-      }
+  }
 
-      if (event.type === "complete" && (event.failed || event.aborted))
-        throw new Error(
-          event.aborted?.message ?? "Structural coverage is incomplete.",
-        );
+  const coverage: ComparisonCoverage = {
+    files,
+    fileSources,
+    alignments,
+    counted: true,
+  };
 
-      if (event.type !== "file") continue;
+  if (mode === "textual") return coverage;
 
-      const path = (event.file.rhs ?? event.file.lhs)!.path;
+  // Keep Git's totals and file identities while structural results add folding.
+  const publishCurrent = () =>
+    publish?.({ ...coverage, files: [...coverage.files] });
 
-      if (!remaining.delete(path))
-        throw new Error(`Unexpected structural result: ${path}`);
+  publishCurrent();
 
-      if (event.error) {
-        if (event.error.code !== "unsupported_file_type")
-          throw new Error(`Cannot count ${path}: ${event.error.message}`);
+  const baseFiles = new Map(
+    coverage.files.map((file, index) => [
+      file.previousPath ?? file.path,
+      index,
+    ]),
+  );
 
-        if (!remaining.size) break;
-        continue;
-      }
+  const headFiles = new Map(
+    coverage.files.map((file, index) => [file.path, index]),
+  );
 
-      const diff = event.diff;
+  const remaining = new Set<string>();
 
-      const previousPath =
-        event.file.lhs?.path !== path ? event.file.lhs?.path : undefined;
-
-      const base = diff.type === "text" ? (diff.lhs?.text ?? null) : null;
-      const head = diff.type === "text" ? (diff.rhs?.text ?? null) : null;
-
-      const fingerprint = hash([
-        pins.repositoryId,
-        base,
-        head,
-        ...(diff.type === "binary"
-          ? [event.file.lhs?.oid ?? null, event.file.rhs?.oid ?? null]
-          : []),
-      ]);
-
-      const sources: FileLineRange[] = [];
-
-      if (base !== null)
-        sources.push({
-          side: "base",
-          file: event.file.lhs!.path,
-          fromLine: 1,
-          toLine: base.split("\n").length,
-        });
-
-      if (head !== null)
-        sources.push({
-          side: "head",
-          file: path,
-          fromLine: 1,
-          toLine: head.split("\n").length,
-        });
-      fileSources.set(path, sources);
-
-      if (diff.type === "text") alignments.set(path, structuralRows(diff));
-      files.push({
-        path,
-        previousPath,
-        fingerprint,
-        changed:
-          diff.type === "text" ? diff.structural_changes : emptyCoverage(),
-        folded: foldedChanges(diff),
-        viewed: emptyCoverage(),
-      });
-      publish?.({
-        files: [...files],
-        fileSources: new Map(fileSources),
-        alignments: new Map(alignments),
-      });
+  for await (const event of data.structuralChanges({
+    reviewId,
+    pins,
+    signal,
+  })) {
+    if (event.type === "start") {
+      for (const entry of event.files)
+        remaining.add((entry.file.rhs ?? entry.file.lhs)!.path);
 
       if (!remaining.size) break;
+      continue;
     }
 
-    if (remaining.size) throw new Error("Structural coverage is incomplete.");
+    if (event.type === "complete" && (event.failed || event.aborted))
+      throw new Error(
+        event.aborted?.message ?? "Structural coverage is incomplete.",
+      );
+
+    if (event.type !== "file") continue;
+
+    const path = (event.file.rhs ?? event.file.lhs)!.path;
+
+    if (!remaining.delete(path))
+      throw new Error(`Unexpected structural result: ${path}`);
+
+    if (event.error) {
+      if (
+        event.error.code !== "unsupported_file_type" &&
+        event.error.code !== "not_utf8"
+      )
+        throw new Error(`Cannot count ${path}: ${event.error.message}`);
+    } else if (event.diff.type === "text") {
+      const diff = event.diff;
+      const folded = foldedChanges(diff);
+
+      // Match each side separately: diffr can split a Git rename into two files.
+      for (const side of ["base", "head"] as const) {
+        const source = side === "base" ? event.file.lhs : event.file.rhs;
+
+        const index =
+          source && (side === "base" ? baseFiles : headFiles).get(source.path);
+
+        if (index === undefined) continue;
+        const file = coverage.files[index];
+
+        const visible = subtractIntervals(
+          diff.structural_changes[side],
+          folded[side],
+        );
+
+        coverage.files[index] = {
+          ...file,
+          folded: {
+            ...(file.folded ?? emptyCoverage()),
+            [side]: subtractIntervals(file.changed[side], visible),
+          },
+        };
+      }
+
+      const baseIndex = event.file.lhs && baseFiles.get(event.file.lhs.path);
+      const headIndex = event.file.rhs && headFiles.get(event.file.rhs.path);
+      const index = headIndex ?? baseIndex;
+
+      if (index !== undefined) {
+        const sources =
+          coverage.fileSources.get(coverage.files[index].path) ?? [];
+
+        if (
+          sources.every(
+            (source) =>
+              source.file ===
+              (source.side === "base" ? event.file.lhs : event.file.rhs)?.path,
+          )
+        )
+          coverage.alignments = new Map(coverage.alignments).set(
+            coverage.files[index].path,
+            structuralRows(diff),
+          );
+      }
+
+      publishCurrent();
+    }
+
+    if (!remaining.size) break;
   }
 
-  return { files, fileSources, alignments };
+  if (remaining.size) throw new Error("Structural coverage is incomplete.");
+
+  return coverage;
 }
 
 /**

package/src/review-api/http.ts

@@ -389,12 +389,22 @@ export function createReviewApi(
 
     const snapshot = store.read(id);
 
+    const pins = snapshot.pins
+      ? (await data.resolveSource(snapshot)).pins
+      : undefined;
+
+    const coverage = pins && data.coverageSnapshot(id, pins, input.mode);
+
+    const partial =
+      coverage && coverage.pending ? coverage.comparison : undefined;
+
     const progress = await reviewProgress(
       store,
       data,
       snapshot,
       context.req.raw.signal,
       input.mode,
+      partial,
     );
 
     const files = input.files.map((update) => {
@@ -427,6 +437,7 @@ export function createReviewApi(
         store.read(id, input.version),
         context.req.raw.signal,
         input.mode,
+        partial,
       ),
     );
   });

package/src/review-api/local-data.ts

@@ -1409,6 +1409,7 @@ export class LocalReviewData {
     return {
       pending,
       comparison: entry.partial ?? {
+        counted: false,
         files: [],
         fileSources: new Map(),
         alignments: new Map(),
@@ -1496,16 +1497,21 @@ export class LocalReviewData {
   }
 
   changes(pins: Pins): Promise<LocalVcsDiffFileSummary[]>;
-  changes(pins: Pins, file: string): Promise<string>;
+  changes(pins: Pins, file: string, previousPath?: string): Promise<string>;
   changes(
     pins: Pins,
     file?: string,
+    previousPath?: string,
   ): Promise<LocalVcsDiffFileSummary[] | string>;
-  async changes(pins: Pins, file?: string) {
+  async changes(pins: Pins, file?: string, previousPath?: string) {
     if (file === undefined) return this.summaries(pins);
     checkRelativePath(file);
 
-    return this.rawPatch(pins, { paths: [file] });
+    if (previousPath) checkRelativePath(previousPath);
+
+    return this.rawPatch(pins, {
+      paths: previousPath ? [previousPath, file] : [file],
+    });
   }
   private async summaries(pins: Pins) {
     if (pins.worktreeRevision) {