atom-agent 1.3.0 → 1.4.0

package/CHANGELOG.md

@@ -1,5 +1,27 @@
 # Changelog
 
+## 1.4.0 — 2026-09-11
+
+- Uncapped TUI diffs (`src/ui/diff.ts`, `src/ui/side-by-side.tsx`,
+  `src/ui/diff-view.tsx`, `src/ui/transcript.tsx`, `src/ui/modals.tsx`,
+  `src/ui/diff-panel.tsx`): the diff engine no longer truncates at 400
+  changed lines — `computeDiff` and `computeSideBySide` return the full
+  hunk/row list with `truncated: false`, and the transcript, approval
+  preview, and `/diff` panel render it whole (no more `… N more rows` or
+  `(diff truncated at 400 changed lines)` trailers). A 500-line write now
+  shows its complete before/after instead of a capped head. Smoothness is
+  preserved by the existing per-mount memoization (`SideBySideInner`,
+  `DiffViewInner`, `LineBody`, `TranscriptRow`), append-once `<Static>`
+  commits, and the engine's linear-time fallbacks (Myers prefix/suffix past
+  1000 lines, flat word runs past 200 tokens/line). Safety caps stay
+  enforced: binary detect and the 1MB file skip. `MAX_CHANGED_LINES`,
+  `TRANSCRIPT_DIFF_MAX_LINES`, `APPROVAL_DIFF_MAX_LINES`, and
+  `DIFF_PANEL_MAX_LINES` are retained as compatibility exports (the row
+  caps now read `Infinity`); `maxRows`/`maxLines` remain as opt-in windows
+  for callers that want a collapsed tail. Known tradeoff: large approvals
+  grow the permission modal, pushing the `y/a/t/n` options further down —
+  the file on disk was and remains the whole truth
+
 ## 1.3.0 — 2026-09-11
 
 - Session goals (`src/goal.ts`, `src/App.tsx`, `src/agent/loop.ts`,

package/dist/ui/diff-panel.js

@@ -4,10 +4,10 @@ import { computeDiff } from "./diff.js";
 import { SideBySideDiffView } from "./side-by-side.js";
 import { theme } from "./theme.js";
 import { LIST_WINDOW, windowedList } from "./tool-inspector.js";
-// Cap for the panel detail (hunk headers excluded): full-file writes can
-// be large — the head reviews, the trailer names the remainder, the file
-// on disk is the whole truth.
-export const DIFF_PANEL_MAX_LINES = 200;
+// Retained for compatibility (no longer applied — the panel renders the
+// full preview; smoothness comes from the per-mount memo + word fallbacks,
+// not from a row cap).
+export const DIFF_PANEL_MAX_LINES = Infinity;
 // Group committed session previews by file: latest preview per path
 // wins (a later edit supersedes the earlier view of the same file),
 // first-seen path order kept. Previews without a path are ungroupable
@@ -51,5 +51,5 @@ export function DiffPanel({ files, index, expanded }) {
                     return (_jsxs(Text, { color: hi ? theme.color.selection : undefined, children: [hi ? `${theme.symbol.select} ` : theme.spacing.rowIndent, "+", r.adds, " \u2212", r.dels, " ", r.path] }, r.path));
                 }), win.below > 0 ? (_jsxs(Text, { dimColor: true, children: [theme.symbol.moreBelow, " ", win.below, " more"] })) : null, _jsxs(Text, { dimColor: true, children: [theme.symbol.moreAbove, "/", theme.symbol.moreBelow, " move \u00B7 Enter expands \u00B7 Esc closes"] })] }));
     }
-    return (_jsxs(Box, { flexDirection: "column", children: [_jsxs(Text, { bold: true, children: [rec.path, _jsxs(Text, { dimColor: true, children: [" ", theme.symbol.separator, " +", rec.adds, " \u2212", rec.dels] })] }), _jsx(Text, { dimColor: true, children: theme.symbol.rule.repeat(32) }), _jsx(SideBySideDiffView, { oldText: rec.oldText, newText: rec.newText, lang: rec.lang, maxRows: DIFF_PANEL_MAX_LINES }), _jsx(Text, { dimColor: true, children: theme.symbol.rule.repeat(32) }), _jsxs(Text, { dimColor: true, children: [theme.symbol.moreAbove, "/", theme.symbol.moreBelow, " prev/next file \u00B7 Enter collapses \u00B7 Esc closes"] })] }));
+    return (_jsxs(Box, { flexDirection: "column", children: [_jsxs(Text, { bold: true, children: [rec.path, _jsxs(Text, { dimColor: true, children: [" ", theme.symbol.separator, " +", rec.adds, " \u2212", rec.dels] })] }), _jsx(Text, { dimColor: true, children: theme.symbol.rule.repeat(32) }), _jsx(SideBySideDiffView, { oldText: rec.oldText, newText: rec.newText, lang: rec.lang }), _jsx(Text, { dimColor: true, children: theme.symbol.rule.repeat(32) }), _jsxs(Text, { dimColor: true, children: [theme.symbol.moreAbove, "/", theme.symbol.moreBelow, " prev/next file \u00B7 Enter collapses \u00B7 Esc closes"] })] }));
 }

package/dist/ui/diff-view.js

@@ -6,8 +6,9 @@ import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 // changed-word backgrounds + syntax foregrounds (ui/highlight, zero-dep),
 // one <Text> per run (no per-character nodes), word diff + highlighting
 // memoized/cached so the 1s busy tick never recomputes. Long lines are
-// never truncated (Ink wraps; copy/paste stays intact) — only hunk COUNT
-// is capped via maxLines so the modal stays compact. All paint comes from
+// never truncated (Ink wraps; copy/paste stays intact); the full hunk list
+// renders by default (an explicit maxLines windows it when a caller passes
+// one). All paint comes from
 // ui/theme tokens (Ink supports color + backgroundColor on Text — verified
 // against the installed Ink 7 typings).
 import React from "react";

package/dist/ui/diff.js

@@ -4,6 +4,10 @@
 export const CHANGE_THRESHOLD = 0.4;
 // Mirrors Claude's DiffDetailView guards.
 export const MAX_FILE_BYTES = 1_000_000;
+// Retained for compatibility (no longer applied — the engine returns the
+// full hunk list; views render it whole). Safety caps that remain enforced:
+// MAX_FILE_BYTES, binary detect, MAX_MYERS_LINES fallback, MAX_WORD_TOKENS
+// fallback.
 export const MAX_CHANGED_LINES = 400;
 export const DIFF_CONTEXT = 3;
 // Worst-case guards for the O(ND) Myers pass + O(w1*w2) word pass.
@@ -183,7 +187,7 @@ export function computeDiff(oldText, newText) {
     const hunks = [];
     let adds = 0;
     let dels = 0;
-    let truncated = false;
+    const truncated = false;
     let hunkStart = Math.max(0, changeIdx[0] - DIFF_CONTEXT);
     let hunkEnd = Math.min(raw.length, changeIdx[0] + DIFF_CONTEXT + 1);
     const flush = (s, e) => {
@@ -242,7 +246,6 @@ export function computeDiff(oldText, newText) {
         }
         hunks.push({ oldStart, oldLines: oCount, newStart, newLines: nCount, lines });
     };
-    let changed = 0;
     for (let c = 1; c < changeIdx.length; c++) {
         const prev = changeIdx[c - 1];
         const cur = changeIdx[c];
@@ -251,32 +254,11 @@ export function computeDiff(oldText, newText) {
         }
         else {
             flush(hunkStart, hunkEnd);
-            changed = hunks.reduce((t, h) => t + h.lines.filter((l) => l.kind !== "context").length, 0);
-            if (changed >= MAX_CHANGED_LINES) {
-                truncated = true;
-                return { hunks, adds, dels, truncated, skipped: null, isNewFile: oldText === null };
-            }
             hunkStart = Math.max(0, cur - DIFF_CONTEXT);
             hunkEnd = Math.min(raw.length, cur + DIFF_CONTEXT + 1);
         }
     }
     flush(hunkStart, hunkEnd);
-    changed = hunks.reduce((t, h) => t + h.lines.filter((l) => l.kind !== "context").length, 0);
-    if (changed > MAX_CHANGED_LINES) {
-        // Trim trailing hunks past the budget (keep the head — the user
-        // reviews top-down; the notice names the remainder).
-        let kept = 0;
-        const out = [];
-        for (const h of hunks) {
-            const n = h.lines.filter((l) => l.kind !== "context").length;
-            if (kept + n > MAX_CHANGED_LINES)
-                break;
-            out.push(h);
-            kept += n;
-        }
-        truncated = true;
-        return { hunks: out, adds, dels, truncated, skipped: null, isNewFile: oldText === null };
-    }
     return { hunks, adds, dels, truncated, skipped: null, isNewFile: oldText === null };
 }
 export function computeSideBySide(oldText, newText) {
@@ -316,9 +298,7 @@ export function computeSideBySide(oldText, newText) {
     const rows = [];
     let adds = 0;
     let dels = 0;
-    let truncated = false;
-    const changedSoFar = () => dels + adds;
-    // Returns false when the change budget is exhausted (stop windowing).
+    const truncated = false;
     const flushSlice = (s, e) => {
         const slice = raw.slice(s, e);
         let k = 0;
@@ -376,12 +356,7 @@ export function computeSideBySide(oldText, newText) {
             }
             dels += delRun.length;
             adds += addRun.length;
-            if (changedSoFar() >= MAX_CHANGED_LINES) {
-                truncated = true;
-                return false;
-            }
         }
-        return true;
     };
     let hunkStart = Math.max(0, changeIdx[0] - DIFF_CONTEXT);
     let hunkEnd = Math.min(raw.length, changeIdx[0] + DIFF_CONTEXT + 1);
@@ -392,31 +367,11 @@ export function computeSideBySide(oldText, newText) {
             hunkEnd = Math.min(raw.length, cur + DIFF_CONTEXT + 1);
         }
         else {
-            if (!flushSlice(hunkStart, hunkEnd)) {
-                return { kind: "diff", rows, adds, dels, truncated, isNewFile: oldText === null };
-            }
+            flushSlice(hunkStart, hunkEnd);
             hunkStart = Math.max(0, cur - DIFF_CONTEXT);
             hunkEnd = Math.min(raw.length, cur + DIFF_CONTEXT + 1);
         }
     }
     flushSlice(hunkStart, hunkEnd);
-    if (dels + adds > MAX_CHANGED_LINES) {
-        // Single-hunk overflow: the slice already pushed past the budget —
-        // trim trailing rows past 400 changes (keep the head; the notice
-        // names the remainder). Mirrors computeDiff's trailing-hunk trim.
-        let kept = 0;
-        let cut = rows.length;
-        for (let i = 0; i < rows.length; i++) {
-            if (rows[i].kind === "change") {
-                kept += 1;
-                if (kept > MAX_CHANGED_LINES) {
-                    cut = i;
-                    break;
-                }
-            }
-        }
-        rows.length = cut;
-        truncated = true;
-    }
     return { kind: "diff", rows, adds, dels, truncated, isNewFile: oldText === null };
 }

package/dist/ui/modals.js

@@ -7,10 +7,10 @@ import React from "react";
 import { Box, Text } from "ink";
 import { SideBySideDiffView } from "./side-by-side.js";
 import { theme } from "./theme.js";
-// Max diff body lines inside the approval modal (hunk headers excluded;
-// the trailer names the remainder). Keeps the modal scannable while the
-// 1s busy tick repaints around it.
-export const APPROVAL_DIFF_MAX_LINES = 40;
+// Retained for compatibility (no longer applied — the approval preview
+// renders the full diff; smoothness comes from the per-mount memo + word
+// fallbacks, not from a row cap).
+export const APPROVAL_DIFF_MAX_LINES = Infinity;
 export const APPROVAL_OPTIONS = ["once", "always", "trustAll", "no"];
 // Command/file preview: the audit description minus its `⚙ name` prefix
 // (the tool name already headlines above). Falls back to the full text
@@ -39,7 +39,7 @@ export const ApprovalBox = React.memo(function ApprovalBox({ toolName, descripti
         { label: "[t]rust all write/edit/bash this session", option: "trustAll" },
         { label: "[n]o — deny this call", option: "no" },
     ];
-    return (_jsxs(Box, { flexDirection: "column", borderStyle: theme.border.style, borderColor: theme.border.approval, paddingX: theme.spacing.pickerPadX, children: [_jsxs(Text, { bold: true, color: theme.color.warning, children: [theme.symbol.warningMark, " Atom permission \u2014 allow this tool?"] }), _jsx(Text, { bold: true, children: approvalTitle(toolName) }), _jsx(Text, { color: theme.color.code, children: approvalPreview(toolName, description) }), diff ? _jsx(SideBySideDiffView, { oldText: diff.oldText, newText: diff.newText, lang: diff.lang, maxRows: APPROVAL_DIFF_MAX_LINES }) : null, rows.map((r, i) => (_jsxs(Text, { color: i === selected ? theme.color.selection : undefined, children: [i === selected ? `${theme.symbol.select} ` : theme.spacing.rowIndent, r.label] }, r.option))), _jsx(Text, { dimColor: true, children: "\u2191/\u2193 + Enter selects \u00B7 y/a/t/n shortcuts \u00B7 Esc denies" })] }));
+    return (_jsxs(Box, { flexDirection: "column", borderStyle: theme.border.style, borderColor: theme.border.approval, paddingX: theme.spacing.pickerPadX, children: [_jsxs(Text, { bold: true, color: theme.color.warning, children: [theme.symbol.warningMark, " Atom permission \u2014 allow this tool?"] }), _jsx(Text, { bold: true, children: approvalTitle(toolName) }), _jsx(Text, { color: theme.color.code, children: approvalPreview(toolName, description) }), diff ? _jsx(SideBySideDiffView, { oldText: diff.oldText, newText: diff.newText, lang: diff.lang }) : null, rows.map((r, i) => (_jsxs(Text, { color: i === selected ? theme.color.selection : undefined, children: [i === selected ? `${theme.symbol.select} ` : theme.spacing.rowIndent, r.label] }, r.option))), _jsx(Text, { dimColor: true, children: "\u2191/\u2193 + Enter selects \u00B7 y/a/t/n shortcuts \u00B7 Esc denies" })] }));
 });
 export const QuestionBox = React.memo(function QuestionBox({ question, options, allowCustom, askCustom, askSelIndex }) {
     questionRenderProbe.count += 1;

package/dist/ui/side-by-side.js

@@ -14,7 +14,8 @@ import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 //   (code-point safe); below NARROW_COLUMNS the view degrades to the
 //   stacked unified DiffView instead of destroying the layout.
 // - computed once per mount (useMemo, keyed on inputs + pane width) and
-//   capped (maxRows + trailer) — never recomputed per tick, never floods.
+//   rendered whole (an explicit maxRows windows it when a caller passes one)
+//   — never recomputed per tick, never floods via re-computation.
 // All paint comes from ui/theme tokens.
 import React from "react";
 import { Box, Text } from "ink";
@@ -24,10 +25,11 @@ import { DiffView, LineBody } from "./diff-view.js";
 import { theme } from "./theme.js";
 // Below this width two panes cannot breathe — stack unified instead.
 export const SBS_NARROW_COLUMNS = 70;
-// Committed-transcript cap (rows, context + change): the scrollback shows
-// the reviewable head; the file on disk is the whole truth. The engine
-// still truncates past 400 changed lines with its own notice.
-export const TRANSCRIPT_DIFF_MAX_LINES = 120;
+// Uncapped: views render the full row list (smooth via per-mount useMemo +
+// append-once Static + word-token/Myers fallbacks in the engine). maxRows
+// remains as an opt-in window for callers/tests that want a collapsed tail.
+// Retained for compatibility.
+export const TRANSCRIPT_DIFF_MAX_LINES = Infinity;
 function truncateTo(s, width) {
     const chars = [...s];
     if (chars.length <= width)

package/dist/ui/transcript.js

@@ -5,7 +5,7 @@ import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-run
 // All paint comes from ui/theme tokens — no literal colors or glyphs here.
 import React from "react";
 import { Box, Static, Text } from "ink";
-import { SideBySideDiffView, TRANSCRIPT_DIFF_MAX_LINES } from "./side-by-side.js";
+import { SideBySideDiffView } from "./side-by-side.js";
 import { ErrorCard, classifyToolError } from "./errors.js";
 import { MarkdownText, ToolLine } from "./markdown.js";
 import { theme } from "./theme.js";
@@ -73,9 +73,9 @@ export function renderTranscriptItem(item) {
             // write/edit label swallowed by pairing (success line immediately
             // followed by an error line) keeps its committed diff above the card.
             const labelDiff = item.label?.diff;
-            return (_jsxs(React.Fragment, { children: [item.label ? _jsx(ToolLine, { content: item.label.content, ms: item.label.ms }) : null, labelDiff && !item.label?.error ? (_jsx(SideBySideDiffView, { oldText: labelDiff.oldText, newText: labelDiff.newText, lang: labelDiff.lang, maxRows: TRANSCRIPT_DIFF_MAX_LINES })) : null, _jsx(ErrorCard, { classified: classified })] }, i));
+            return (_jsxs(React.Fragment, { children: [item.label ? _jsx(ToolLine, { content: item.label.content, ms: item.label.ms }) : null, labelDiff && !item.label?.error ? (_jsx(SideBySideDiffView, { oldText: labelDiff.oldText, newText: labelDiff.newText, lang: labelDiff.lang })) : null, _jsx(ErrorCard, { classified: classified })] }, i));
         }
-        return (_jsxs(React.Fragment, { children: [_jsx(ToolLine, { content: t.content, error: t.error, ms: t.ms }), t.diff && !t.error ? (_jsx(SideBySideDiffView, { oldText: t.diff.oldText, newText: t.diff.newText, lang: t.diff.lang, maxRows: TRANSCRIPT_DIFF_MAX_LINES })) : null] }, i));
+        return (_jsxs(React.Fragment, { children: [_jsx(ToolLine, { content: t.content, error: t.error, ms: t.ms }), t.diff && !t.error ? (_jsx(SideBySideDiffView, { oldText: t.diff.oldText, newText: t.diff.newText, lang: t.diff.lang })) : null] }, i));
     }
     return (_jsxs(Box, { flexDirection: "column", marginBottom: theme.spacing.turnGap, children: [_jsx(Text, { children: _jsxs(Text, { color: theme.color.assistant, bold: true, children: [theme.symbol.speakerAssistant, " "] }) }), _jsx(MarkdownText, { text: t.content })] }, i));
 }

package/documentation/extensions.md

@@ -146,7 +146,7 @@ Exact `ExtensionAPI` methods — nothing else exists:
 | `isProjectTrusted()` | Whether this load is trusted (degrade gracefully when false) |
 | `setStatusSegment(text)` | One status-bar slot per extension (upsert by owner) |
 | `setWidget(def)` | Panel widget (`placement: "panel"`, keyed by owner + id) |
-| `notify(message)` | Transient `(name) message` transcript line |
+| `notify(message)` | Transient `(name) message` transcript line (staged queue caps at 100, drop-oldest) |
 | `promptUser(question, options?, allowCustom?)` | Modal dialog; rejects headless, during activation, or while one is open |
 
 Stores behind the API: `src/tools/custom.ts`, `intercept.ts`, `overrides.ts`, `provider-hooks.ts`, `compaction-hooks.ts`, `src/extension-commands.ts`, `src/extension-ui.ts`, `src/project-trust.ts`.

package/examples/extensions/01-audit-gate.js

@@ -4,8 +4,8 @@
 // as the normal model-visible result (approval is skipped, the turn
 // continues); anything this hook does not block runs untouched.
 //
-// Gallery sample for documentation/extensions.md — that guide references
-// this file by name and never duplicates it. Covered by
+// Gallery sample for documentation/extensions.md — that guide excerpts
+// this file and references it by name. Covered by
 // tests/extension-gallery.test.ts, which loads this exact file through the
 // real loadExtensions path.
 module.exports = function auditGate(api) {

package/examples/extensions/02-notes-tool.js

@@ -5,8 +5,8 @@
 // `Error: invalid call: ...` result and the implementation never runs), and
 // dispatches through the shared loop.
 //
-// Gallery sample for documentation/extensions.md — that guide references
-// this file by name and never duplicates it. Covered by
+// Gallery sample for documentation/extensions.md — that guide excerpts
+// this file and references it by name. Covered by
 // tests/extension-gallery.test.ts, which loads this exact file through the
 // real loadExtensions path.
 const notes = [];

package/examples/extensions/03-custom-command.js

@@ -6,8 +6,8 @@
 // transcript) so the dialog flow is testable headless — tests drive the
 // command with a stub askUser and unit-test the helper directly.
 //
-// Gallery sample for documentation/extensions.md — that guide references
-// this file by name and never duplicates it. Covered by
+// Gallery sample for documentation/extensions.md — that guide excerpts
+// this file and references it by name. Covered by
 // tests/extension-gallery.test.ts, which loads this exact file through the
 // real loadExtensions path.
 function summarizeChoice(choice, extra) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "atom-agent",
-  "version": "1.3.0",
+  "version": "1.4.0",
   "description": "Agentic terminal coding assistant: multi-provider LLM loop with local file/shell/web tools in an Ink (React) TUI.",
   "type": "module",
   "license": "MIT",