pi-diff-review 0.1.12 → 0.1.14

package/src/review-navigation.ts

@@ -0,0 +1,94 @@
+import type { DiffRenderMode, SelectionBounds } from "./types.ts";
+
+export type JumpBoundaryResult = {
+  changed: boolean;
+  clearedSelection: boolean;
+};
+
+export class ReviewNavigationState {
+  selected: number;
+  scrollTop = 0;
+  selectionAnchor?: number;
+  diffRenderMode: DiffRenderMode = "unified";
+
+  constructor(
+    private readonly lineCount: number,
+    initialSelected = 0,
+  ) {
+    this.selected = this.clampLineIndex(initialSelected);
+  }
+
+  move(delta: number): boolean {
+    const next = this.clampLineIndex(this.selected + delta);
+    if (next === this.selected) return false;
+    this.selected = next;
+    return true;
+  }
+
+  jumpToBoundary(boundary: "start" | "end"): JumpBoundaryResult {
+    const next = boundary === "start" ? 0 : Math.max(0, this.lineCount - 1);
+    const hadSelection = this.selectionAnchor != null;
+    const changed = next !== this.selected || hadSelection;
+    this.selected = next;
+    if (hadSelection) this.selectionAnchor = undefined;
+    return { changed, clearedSelection: hadSelection };
+  }
+
+  extendSelection(delta: number): boolean {
+    if (this.selectionAnchor == null) {
+      this.selectionAnchor = this.selected;
+    }
+    const next = this.clampLineIndex(this.selected + delta);
+    if (next === this.selected) return false;
+    this.selected = next;
+    return true;
+  }
+
+  clearSelection(): boolean {
+    if (this.selectionAnchor == null) return false;
+    this.selectionAnchor = undefined;
+    return true;
+  }
+
+  hasSelection(): boolean {
+    return (
+      this.selectionAnchor != null && this.selectionAnchor !== this.selected
+    );
+  }
+
+  getSelectionBounds(): SelectionBounds | undefined {
+    if (this.selectionAnchor == null) return undefined;
+    return {
+      start: Math.min(this.selectionAnchor, this.selected),
+      end: Math.max(this.selectionAnchor, this.selected),
+    };
+  }
+
+  toggleDiffRenderMode(): DiffRenderMode {
+    this.diffRenderMode =
+      this.diffRenderMode === "unified" ? "split" : "unified";
+    this.scrollTop = 0;
+    return this.diffRenderMode;
+  }
+
+  ensureScroll(
+    viewportHeight: number,
+    selectedDisplayRow: number,
+    displayRowCount: number,
+  ): void {
+    if (selectedDisplayRow < this.scrollTop) {
+      this.scrollTop = selectedDisplayRow;
+    }
+    if (selectedDisplayRow >= this.scrollTop + viewportHeight) {
+      this.scrollTop = selectedDisplayRow - viewportHeight + 1;
+    }
+    this.scrollTop = Math.max(
+      0,
+      Math.min(this.scrollTop, Math.max(0, displayRowCount - viewportHeight)),
+    );
+  }
+
+  private clampLineIndex(index: number): number {
+    return Math.max(0, Math.min(Math.max(0, this.lineCount - 1), index));
+  }
+}

package/src/split-diff.ts

@@ -0,0 +1,61 @@
+import type { ReviewLine, SplitDiffCell, SplitDiffRow } from "./types.ts";
+
+export type SplitDiffRowsResult = {
+  rows: SplitDiffRow[];
+  rowByLineIndex: number[];
+};
+
+export function buildSplitDiffRows(lines: ReviewLine[]): SplitDiffRowsResult {
+  const rows: SplitDiffRow[] = [];
+  const rowByLineIndex: number[] = [];
+  let index = 0;
+
+  const pushRow = (row: SplitDiffRow) => {
+    const displayRow = rows.length;
+    rows.push(row);
+    if (row.kind === "full") {
+      rowByLineIndex[row.cell.index] = displayRow;
+    } else {
+      if (row.left) rowByLineIndex[row.left.index] = displayRow;
+      if (row.right) rowByLineIndex[row.right.index] = displayRow;
+    }
+  };
+
+  while (index < lines.length) {
+    const line = lines[index]!;
+
+    if (line.kind === "remove" || line.kind === "add") {
+      const removals: SplitDiffCell[] = [];
+      const additions: SplitDiffCell[] = [];
+
+      while (lines[index]?.kind === "remove") {
+        removals.push({ line: lines[index]!, index });
+        index++;
+      }
+      while (lines[index]?.kind === "add") {
+        additions.push({ line: lines[index]!, index });
+        index++;
+      }
+
+      const count = Math.max(removals.length, additions.length);
+      for (let offset = 0; offset < count; offset++) {
+        pushRow({
+          kind: "split",
+          left: removals[offset],
+          right: additions[offset],
+        });
+      }
+      continue;
+    }
+
+    if (line.kind === "context") {
+      const cell = { line, index };
+      pushRow({ kind: "split", left: cell, right: cell });
+    } else {
+      pushRow({ kind: "full", cell: { line, index } });
+    }
+    index++;
+  }
+
+  return { rows, rowByLineIndex };
+}

package/src/types.ts

@@ -42,7 +42,6 @@ export type DiffSource = {
   args: string[];
 };
 
-export type ReviewLayout = "side-by-side" | "stacked";
 export type DiffRenderMode = "unified" | "split";
 
 export type SplitDiffCell = {
@@ -60,5 +59,3 @@ export type ReviewTui = {
 };
 
 export type ReviewTheme = Theme;
-
-export type RightPaneMode = "comments" | "explanation";