@dev.fast/whiteboard 0.5.2-preview.20261104.147 → 0.5.2-preview.20261109.149

package/src/viewed-coverage.ts

@@ -1,17 +1,14 @@
-import { structuralChangeCounts } from "@dev.fast/review-protocol";
-import {
-  type FileLineRange,
-  type LineInterval,
-  unionIntervals,
-} from "@dev.fast/whiteboard-document";
 import { z } from "zod";
 
-export { type LineInterval, unionIntervals };
-
-export interface Coverage {
-  base: LineInterval[];
-  head: LineInterval[];
-}
+export {
+  type CoverageFile,
+  type CoverageProgress,
+  coverageProgress,
+  coverageSources,
+  emptyCoverage,
+  mergeCoverageProgress,
+  scopedCoverage,
+} from "@dev.fast/whiteboard-document";
 
 export const coverageSchema = z.object({
   base: z.array(
@@ -21,243 +18,3 @@ export const coverageSchema = z.object({
     z.tuple([z.number().int().nonnegative(), z.number().int().nonnegative()]),
   ),
 });
-
-export interface ChangeCounts {
-  additions: number;
-  deletions: number;
-}
-
-/**
- * Changed lines that diffr folds by default count as done, the way the
- * reader's viewed marks do. `folded` is how many of the scope's lines are
- * done only because they are folded; `remaining` excludes them.
- * "folded" is the state of a scope with nothing left to read and nothing the
- * reader marked: all of it starts folded.
- */
-export interface CoverageProgress {
-  state: "unread" | "partial" | "viewed" | "folded";
-  total: ChangeCounts;
-  remaining: ChangeCounts;
-  folded: ChangeCounts;
-}
-
-export interface CoverageFile {
-  path: string;
-  previousPath?: string;
-  fingerprint: string;
-  changed: Coverage;
-  /** The changed lines diffr folds by default: every changed line of a file
-   * it hides, or those under a region that starts collapsed. Absent without
-   * structural data, where every changed line is left to read. */
-  folded?: Coverage;
-  viewed: Coverage;
-}
-
-export const emptyCoverage = (): Coverage => ({ base: [], head: [] });
-
-export function intersectIntervals(
-  left: readonly LineInterval[],
-  right: readonly LineInterval[],
-): LineInterval[] {
-  const result: LineInterval[] = [];
-
-  const a = unionIntervals(left),
-    b = unionIntervals(right);
-
-  let i = 0,
-    j = 0;
-
-  while (i < a.length && j < b.length) {
-    const start = Math.max(a[i][0], b[j][0]),
-      end = Math.min(a[i][1], b[j][1]);
-
-    if (start < end) result.push([start, end]);
-
-    if (a[i][1] < b[j][1]) i++;
-    else j++;
-  }
-
-  return result;
-}
-
-export function subtractIntervals(
-  left: readonly LineInterval[],
-  right: readonly LineInterval[],
-): LineInterval[] {
-  const result: LineInterval[] = [];
-  const removed = unionIntervals(right);
-
-  for (const [start, end] of unionIntervals(left)) {
-    let cursor = start;
-
-    for (const [from, to] of removed) {
-      if (to <= cursor) continue;
-
-      if (from >= end) break;
-
-      if (from > cursor) result.push([cursor, from]);
-      cursor = Math.min(end, Math.max(cursor, to));
-    }
-
-    if (cursor < end) result.push([cursor, end]);
-  }
-
-  return result;
-}
-
-export function updateCoverage(
-  current: Coverage,
-  scope: Coverage,
-  viewed: boolean,
-): Coverage {
-  const apply = (side: "base" | "head") =>
-    viewed
-      ? unionIntervals([...current[side], ...scope[side]])
-      : subtractIntervals(current[side], scope[side]);
-
-  return { base: apply("base"), head: apply("head") };
-}
-
-export function scopedCoverage(
-  file: CoverageFile,
-  sources?: readonly FileLineRange[],
-): Coverage {
-  if (!sources) return file.changed;
-
-  const side = (side: "base" | "head") =>
-    intersectIntervals(
-      file.changed[side],
-      sources.flatMap((source): LineInterval[] =>
-        source.side === side &&
-        source.file ===
-          (side === "base" ? (file.previousPath ?? file.path) : file.path)
-          ? [[source.fromLine - 1, source.toLine]]
-          : [],
-      ),
-    );
-
-  return { base: side("base"), head: side("head") };
-}
-
-/** Each file's own sources, grouped once. */
-export function sourcesForFile(
-  sources?: readonly FileLineRange[],
-): (file: CoverageFile) => FileLineRange[] | undefined {
-  if (!sources) return () => undefined;
-  const byFile = Map.groupBy(sources, (source) => source.file);
-
-  return (file) => {
-    const base = byFile.get(file.previousPath ?? file.path) ?? [];
-
-    return file.previousPath && file.previousPath !== file.path
-      ? [...base, ...(byFile.get(file.path) ?? [])]
-      : base;
-  };
-}
-
-export function coverageProgress(
-  files: readonly CoverageFile[],
-  sources?: readonly FileLineRange[],
-): CoverageProgress {
-  const total = { additions: 0, deletions: 0 },
-    remaining = { additions: 0, deletions: 0 },
-    folded = { additions: 0, deletions: 0 };
-
-  const own = sourcesForFile(sources);
-
-  for (const file of files) {
-    const scope = scopedCoverage(file, own(file));
-    const counts = structuralChangeCounts(scope);
-
-    const side = (side: "base" | "head") => {
-      const unviewed = subtractIntervals(scope[side], file.viewed[side]);
-
-      return {
-        unread: subtractIntervals(unviewed, file.folded?.[side] ?? []),
-        folded: intersectIntervals(unviewed, file.folded?.[side] ?? []),
-      };
-    };
-
-    const base = side("base"),
-      head = side("head");
-
-    const unread = structuralChangeCounts({
-      base: base.unread,
-      head: head.unread,
-    });
-
-    const foldedCounts = structuralChangeCounts({
-      base: base.folded,
-      head: head.folded,
-    });
-
-    total.additions += counts.added;
-    total.deletions += counts.removed;
-    remaining.additions += unread.added;
-    remaining.deletions += unread.removed;
-    folded.additions += foldedCounts.added;
-    folded.deletions += foldedCounts.removed;
-  }
-
-  return {
-    total,
-    remaining,
-    folded,
-    state: progressState(total, remaining, folded),
-  };
-}
-
-/** Progress over several comparisons at once: the sums, and the same state
- * rule as one. */
-export function mergeCoverageProgress(
-  parts: readonly CoverageProgress[],
-): CoverageProgress {
-  const total = { additions: 0, deletions: 0 },
-    remaining = { additions: 0, deletions: 0 },
-    folded = { additions: 0, deletions: 0 };
-
-  for (const part of parts) {
-    total.additions += part.total.additions;
-    total.deletions += part.total.deletions;
-    remaining.additions += part.remaining.additions;
-    remaining.deletions += part.remaining.deletions;
-    folded.additions += part.folded.additions;
-    folded.deletions += part.folded.deletions;
-  }
-
-  return {
-    total,
-    remaining,
-    folded,
-    state: progressState(total, remaining, folded),
-  };
-}
-
-/** Done lines are the viewed and the folded ones; the state names which. */
-function progressState(
-  total: ChangeCounts,
-  remaining: ChangeCounts,
-  folded: ChangeCounts,
-): CoverageProgress["state"] {
-  const size = total.additions + total.deletions,
-    unread = remaining.additions + remaining.deletions,
-    viewed = size - unread - folded.additions - folded.deletions;
-
-  if (size > 0 && unread === 0) return viewed > 0 ? "viewed" : "folded";
-
-  return viewed > 0 ? "partial" : "unread";
-}
-
-export function coverageSources(
-  file: CoverageFile,
-  coverage = file.viewed,
-): FileLineRange[] {
-  return (["base", "head"] as const).flatMap((side) =>
-    coverage[side].map(([start, end]) => ({
-      side,
-      file: side === "base" ? (file.previousPath ?? file.path) : file.path,
-      fromLine: start + 1,
-      toLine: end,
-    })),
-  );
-}

package/dist/agent-cli-x6-Eh1S-.js

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

package/dist/profile-BstQkIY0.js

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

package/src/review-api/file-lenses.ts

@@ -1,125 +0,0 @@
-import {
-  type FileLineRange,
-  type Lens,
-  type LensSource,
-  anchorSelection,
-} from "@dev.fast/whiteboard-document";
-import {
-  type CoverageFile,
-  coverageSources,
-  scopedCoverage,
-  sourcesForFile,
-  subtractIntervals,
-  unionIntervals,
-} from "@review/viewed-coverage.js";
-import { Minimatch } from "minimatch";
-
-/** Subtract authored coverage, independently of viewed state. */
-export function uncategorizedSources(
-  files: readonly CoverageFile[],
-  sources: readonly FileLineRange[],
-): FileLineRange[] {
-  const own = sourcesForFile(sources);
-
-  return files.flatMap((file) => {
-    const covered = scopedCoverage(file, own(file));
-
-    return coverageSources(file, {
-      base: subtractIntervals(file.changed.base, covered.base),
-      head: subtractIntervals(file.changed.head, covered.head),
-    });
-  });
-}
-
-/** Resolve all targets into one union; a whole-file range subsumes narrower ranges. */
-export function resolveFileLens(
-  lens: Lens,
-  files: readonly CoverageFile[],
-  fileSources: ReadonlyMap<string, FileLineRange[]>,
-  resolve: (source: LensSource) => FileLineRange[],
-) {
-  const targets = lens.targets;
-
-  const selected = targets.flatMap((target) => {
-    if (target.kind === "ranges")
-      return target.sources.flatMap((source) =>
-        resolve(anchorSelection(source)),
-      );
-    const matches = fileLensMatcher(target.patterns);
-
-    return files
-      .filter(matches)
-      .flatMap((file) => fileSources.get(file.path) ?? []);
-  });
-
-  const groups = new Map<string, FileLineRange[]>();
-
-  for (const source of selected) {
-    const key = JSON.stringify([source.side, source.file]);
-    const group = groups.get(key) ?? [];
-    group.push(source);
-    groups.set(key, group);
-  }
-
-  const sources = [...groups.values()].flatMap((group) =>
-    unionIntervals(
-      group.map((source) => [source.fromLine - 1, source.toLine]),
-    ).map(([start, end]) => ({
-      ...group[0],
-      fromLine: start + 1,
-      toLine: end,
-    })),
-  );
-
-  const fileCount = new Set(sources.map(changedPath(files))).size;
-
-  return {
-    sources,
-    fileCount,
-    wholeFiles: targets.every((target) => target.kind === "files"),
-  };
-}
-
-/** The changed file a source names on its side, or the source's own file. */
-export function changedPath(
-  files: readonly CoverageFile[],
-): (source: FileLineRange) => string {
-  const paths = new Map<string, string>();
-
-  for (const file of files)
-    for (const key of [
-      `base:${file.previousPath ?? file.path}`,
-      `head:${file.path}`,
-    ])
-      if (!paths.has(key)) paths.set(key, file.path);
-
-  return (source) => paths.get(`${source.side}:${source.file}`) ?? source.file;
-}
-
-/** Evaluate authored patterns against the changed-file list, never the filesystem. */
-export function fileLensMatcher(
-  patterns: readonly string[],
-): (file: { path: string; previousPath?: string }) => boolean {
-  const matchers = patterns.map((pattern) => {
-    const normalized = pattern.startsWith("./") ? pattern.slice(2) : pattern;
-
-    const glob = new Minimatch(normalized, {
-      nocase: process.platform === "darwin" || process.platform === "win32",
-      windowsPathsNoEscape: true,
-      nonegate: true,
-      nocomment: true,
-      optimizationLevel: 2,
-      platform: "linux",
-      nocaseMagicOnly: true,
-    });
-
-    return (path: string) => path === normalized || glob.match(path);
-  });
-
-  return (file) =>
-    matchers.some(
-      (matches) =>
-        matches(file.path) ||
-        (file.previousPath !== undefined && matches(file.previousPath)),
-    );
-}

package/src/unified-diff.ts

@@ -1,71 +0,0 @@
-export interface DiffHunk {
-  file: string;
-  oldStart: number;
-  oldLines: number;
-  newStart: number;
-  newLines: number;
-  lines: DiffHunkLine[];
-}
-
-export interface DiffHunkLine {
-  kind: "context" | "add" | "remove";
-  oldLine: number | null;
-  newLine: number | null;
-  text: string;
-}
-
-const HUNK_HEADER =
-  /^@@ -(?<oldStart>\d+)(?:,(?<oldLines>\d+))? \+(?<newStart>\d+)(?:,(?<newLines>\d+))? @@/;
-
-export function parseUnifiedPatch(
-  file: string,
-  patch: string | null | undefined,
-): DiffHunk[] {
-  if (!patch) return [];
-
-  const hunks: DiffHunk[] = [];
-  let current: DiffHunk | null = null;
-  let oldLine = 0;
-  let newLine = 0;
-
-  for (const row of patch.split(/\r?\n/)) {
-    const header = HUNK_HEADER.exec(row);
-
-    if (header?.groups) {
-      current = {
-        file,
-        oldStart: Number(header.groups.oldStart),
-        oldLines: Number(header.groups.oldLines ?? "1"),
-        newStart: Number(header.groups.newStart),
-        newLines: Number(header.groups.newLines ?? "1"),
-        lines: [],
-      };
-      oldLine = current.oldStart;
-      newLine = current.newStart;
-      hunks.push(current);
-      continue;
-    }
-
-    if (!current || row.startsWith("\")) continue;
-
-    const marker = row[0];
-    const text = row.slice(1);
-    let line: DiffHunkLine | null = null;
-
-    if (marker === " ") {
-      line = { kind: "context", oldLine, newLine, text };
-      oldLine += 1;
-      newLine += 1;
-    } else if (marker === "+") {
-      line = { kind: "add", oldLine: null, newLine, text };
-      newLine += 1;
-    } else if (marker === "-") {
-      line = { kind: "remove", oldLine, newLine: null, text };
-      oldLine += 1;
-    }
-
-    if (line) current.lines.push(line);
-  }
-
-  return hunks;
-}

package/dist/{s3-CsG2EWED.js → s3-d8RGPKwt.js}

@@ -1,7 +1,7 @@
 import { a as jsonNumber, c as jsonString, o as jsonObject, r as jsonArray, u as parseJsonText } from "./src-X9phtB2j.js";
-import { S as sessionMetaSchema, b as commitShaSchema, y as byCommitSchema } from "./src-CGP5ytbV.js";
 import { i as isS3MockMode, l as s3CacheIdentity, s as resolveS3Credentials, u as s3MockRoot } from "./s3-config-CjcLyFV5.js";
 import { t as errorMessage } from "./error-message-OtiDonty.js";
+import { S as sessionMetaSchema, b as commitShaSchema, y as byCommitSchema } from "./src-CGP5ytbV.js";
 import { existsSync, mkdirSync, readFileSync, readdirSync, rmSync, statSync, writeFileSync } from "node:fs";
 import path from "node:path";
 import { tmpdir } from "node:os";