@stigmer/ink 3.1.1 → 3.1.3

package/file-review.js

@@ -0,0 +1,113 @@
+import { FileChangeKind, FileDecisionAction, } from "@stigmer/protos/ai/stigmer/agentic/agentexecution/v1/enum_pb";
+import { deriveEffectiveVerdicts, fileReviewability } from "@stigmer/react";
+/**
+ * Single-letter badge for a file change's kind, matching the web review card:
+ * `A` added, `M` modified, `D` deleted, `R` renamed. A binary change reads as
+ * `M` — "binary" is a review-completeness property (from `FileContent.is_binary`
+ * via `fileReviewability`), not a distinct edit kind.
+ */
+export function kindLetter(kind) {
+    switch (kind) {
+        case FileChangeKind.ADD:
+            return "A";
+        case FileChangeKind.DELETE:
+            return "D";
+        case FileChangeKind.RENAME:
+            return "R";
+        default:
+            return "M";
+    }
+}
+/**
+ * The one rename-aware path-display rule for the terminal, over the raw
+ * before/after/kind triple shared by every file-change shape (reviewable
+ * `CapturedFileChange` and slim `FileChangeProgressEntry` alike). Kept private
+ * and fed by the typed adapters below so the two surfaces can never grow
+ * divergent rename formatting.
+ */
+function displayChangePath(pathBefore, pathAfter, kind) {
+    if (kind === FileChangeKind.RENAME && pathBefore && pathAfter) {
+        return `${pathBefore} → ${pathAfter}`;
+    }
+    return pathAfter || pathBefore;
+}
+/**
+ * The path to show for a reviewable change: the post-edit path, or the pre-edit
+ * path for a deletion. A rename shows `before → after` so both sides are visible.
+ */
+export function changeDisplayPath(change) {
+    return displayChangePath(change.pathBefore, change.pathAfter, change.kind);
+}
+/**
+ * The path to show for a mid-run progress entry (DD-32), using the same
+ * rename-aware rule as {@link changeDisplayPath}. The slim entry carries no
+ * bodies or digests, only the same before/after/kind triple, so the terminal
+ * renders its path identically to the reviewable surfaces.
+ */
+export function progressEntryDisplayPath(entry) {
+    return displayChangePath(entry.pathBefore, entry.pathAfter, entry.kind);
+}
+/**
+ * The honest, one-line reason a change cannot be kept as-is, or `null` when it
+ * is fully reviewable. Mirrors the web card's per-file block-reason copy so the
+ * terminal reviewer sees the same explanation.
+ */
+export function blockReasonNote(change) {
+    const reviewability = fileReviewability(change);
+    switch (reviewability.kind) {
+        case "reviewable":
+            return null;
+        case "binary":
+            return "binary — no text diff; keep as-is or discard";
+        case "unavailable":
+            switch (reviewability.reason) {
+                case "secret":
+                    return "contents withheld (looks like a secret) — discard only";
+                case "size":
+                    return "diff too large to display — discard only";
+                default:
+                    return "diff unavailable — discard only";
+            }
+    }
+}
+/**
+ * Folds a settled change set's decisions into kept / discarded / not-reviewed
+ * counts, using the same `deriveEffectiveVerdicts` fold the web uses so a
+ * "Keep all" set correctly reads every file as kept.
+ */
+export function settledCounts(set) {
+    const verdicts = deriveEffectiveVerdicts(set);
+    let kept = 0;
+    let discarded = 0;
+    for (const change of set.changes) {
+        const verdict = verdicts.get(change.id);
+        if (verdict === FileDecisionAction.APPROVE)
+            kept++;
+        else if (verdict === FileDecisionAction.REJECT)
+            discarded++;
+    }
+    return {
+        kept,
+        discarded,
+        notReviewed: set.changes.length - kept - discarded,
+    };
+}
+/**
+ * A one-line summary of a settled change set's outcome, e.g. `2 kept · 1
+ * discarded`, or `3 files changed` when nothing was individually decided.
+ */
+export function settledSummary(set) {
+    const { kept, discarded, notReviewed } = settledCounts(set);
+    const parts = [];
+    if (kept)
+        parts.push(`${kept} kept`);
+    if (discarded)
+        parts.push(`${discarded} discarded`);
+    if (notReviewed)
+        parts.push(`${notReviewed} not reviewed`);
+    if (parts.length > 0)
+        return parts.join(" · ");
+    const n = set.changes.length;
+    return `${n} file${n === 1 ? "" : "s"} changed`;
+}
+//# sourceMappingURL=file-review.js.map

package/file-review.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"file-review.js","sourceRoot":"","sources":["../src/file-review.ts"],"names":[],"mappings":"AAKA,OAAO,EACL,cAAc,EACd,kBAAkB,GACnB,MAAM,8DAA8D,CAAC;AACtE,OAAO,EAAE,uBAAuB,EAAE,iBAAiB,EAAE,MAAM,gBAAgB,CAAC;AAE5E;;;;;GAKG;AACH,MAAM,UAAU,UAAU,CAAC,IAAoB;IAC7C,QAAQ,IAAI,EAAE,CAAC;QACb,KAAK,cAAc,CAAC,GAAG;YACrB,OAAO,GAAG,CAAC;QACb,KAAK,cAAc,CAAC,MAAM;YACxB,OAAO,GAAG,CAAC;QACb,KAAK,cAAc,CAAC,MAAM;YACxB,OAAO,GAAG,CAAC;QACb;YACE,OAAO,GAAG,CAAC;IACf,CAAC;AACH,CAAC;AAED;;;;;;GAMG;AACH,SAAS,iBAAiB,CACxB,UAAkB,EAClB,SAAiB,EACjB,IAAoB;IAEpB,IAAI,IAAI,KAAK,cAAc,CAAC,MAAM,IAAI,UAAU,IAAI,SAAS,EAAE,CAAC;QAC9D,OAAO,GAAG,UAAU,MAAM,SAAS,EAAE,CAAC;IACxC,CAAC;IACD,OAAO,SAAS,IAAI,UAAU,CAAC;AACjC,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,iBAAiB,CAAC,MAA0B;IAC1D,OAAO,iBAAiB,CAAC,MAAM,CAAC,UAAU,EAAE,MAAM,CAAC,SAAS,EAAE,MAAM,CAAC,IAAI,CAAC,CAAC;AAC7E,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,wBAAwB,CAAC,KAA8B;IACrE,OAAO,iBAAiB,CAAC,KAAK,CAAC,UAAU,EAAE,KAAK,CAAC,SAAS,EAAE,KAAK,CAAC,IAAI,CAAC,CAAC;AAC1E,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,eAAe,CAAC,MAA0B;IACxD,MAAM,aAAa,GAAG,iBAAiB,CAAC,MAAM,CAAC,CAAC;IAChD,QAAQ,aAAa,CAAC,IAAI,EAAE,CAAC;QAC3B,KAAK,YAAY;YACf,OAAO,IAAI,CAAC;QACd,KAAK,QAAQ;YACX,OAAO,8CAA8C,CAAC;QACxD,KAAK,aAAa;YAChB,QAAQ,aAAa,CAAC,MAAM,EAAE,CAAC;gBAC7B,KAAK,QAAQ;oBACX,OAAO,wDAAwD,CAAC;gBAClE,KAAK,MAAM;oBACT,OAAO,0CAA0C,CAAC;gBACpD;oBACE,OAAO,iCAAiC,CAAC;YAC7C,CAAC;IACL,CAAC;AACH,CAAC;AAYD;;;;GAIG;AACH,MAAM,UAAU,aAAa,CAAC,GAAkB;IAC9C,MAAM,QAAQ,GAAG,uBAAuB,CAAC,GAAG,CAAC,CAAC;IAC9C,IAAI,IAAI,GAAG,CAAC,CAAC;IACb,IAAI,SAAS,GAAG,CAAC,CAAC;IAClB,KAAK,MAAM,MAAM,IAAI,GAAG,CAAC,OAAO,EAAE,CAAC;QACjC,MAAM,OAAO,GAAG,QAAQ,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;QACxC,IAAI,OAAO,KAAK,kBAAkB,CAAC,OAAO;YAAE,IAAI,EAAE,CAAC;aAC9C,IAAI,OAAO,KAAK,kBAAkB,CAAC,MAAM;YAAE,SAAS,EAAE,CAAC;IAC9D,CAAC;IACD,OAAO;QACL,IAAI;QACJ,SAAS;QACT,WAAW,EAAE,GAAG,CAAC,OAAO,CAAC,MAAM,GAAG,IAAI,GAAG,SAAS;KACnD,CAAC;AACJ,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,cAAc,CAAC,GAAkB;IAC/C,MAAM,EAAE,IAAI,EAAE,SAAS,EAAE,WAAW,EAAE,GAAG,aAAa,CAAC,GAAG,CAAC,CAAC;IAC5D,MAAM,KAAK,GAAa,EAAE,CAAC;IAC3B,IAAI,IAAI;QAAE,KAAK,CAAC,IAAI,CAAC,GAAG,IAAI,OAAO,CAAC,CAAC;IACrC,IAAI,SAAS;QAAE,KAAK,CAAC,IAAI,CAAC,GAAG,SAAS,YAAY,CAAC,CAAC;IACpD,IAAI,WAAW;QAAE,KAAK,CAAC,IAAI,CAAC,GAAG,WAAW,eAAe,CAAC,CAAC;IAC3D,IAAI,KAAK,CAAC,MAAM,GAAG,CAAC;QAAE,OAAO,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IAC/C,MAAM,CAAC,GAAG,GAAG,CAAC,OAAO,CAAC,MAAM,CAAC;IAC7B,OAAO,GAAG,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,UAAU,CAAC;AAClD,CAAC"}

package/index.d.ts

@@ -8,6 +8,10 @@ export { ToolCallGroup, type ToolCallGroupProps } from "./components/ToolCallGro
 export { SubAgentBlock, type SubAgentBlockProps } from "./components/SubAgentBlock.js";
 export { TodoList, type TodoListProps } from "./components/TodoList.js";
 export { ApprovalPrompt, type ApprovalPromptProps } from "./components/ApprovalPrompt.js";
+export { FileReviewPrompt, type FileReviewPromptProps } from "./components/FileReviewPrompt.js";
+export { FileReviewRecord, type FileReviewRecordProps } from "./components/FileReviewRecord.js";
+export { FileChangeProgressBar, type FileChangeProgressBarProps, } from "./components/FileChangeProgressBar.js";
+export { kindLetter, changeDisplayPath, progressEntryDisplayPath, blockReasonNote, settledCounts, settledSummary, type SettledCounts, } from "./file-review.js";
 export { ExecutionProgress, type ExecutionProgressProps } from "./components/ExecutionProgress.js";
 export { FollowUpInput, type FollowUpInputProps } from "./components/FollowUpInput.js";
 export { UsageWidget, type UsageWidgetProps } from "./components/UsageWidget.js";

package/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,kBAAkB,EAAE,KAAK,uBAAuB,EAAE,MAAM,eAAe,CAAC;AAGjF,OAAO,EACL,gBAAgB,EAChB,mBAAmB,EACnB,KAAK,gBAAgB,GACtB,MAAM,mBAAmB,CAAC;AAG3B,OAAO,EAAE,cAAc,EAAE,MAAM,eAAe,CAAC;AAG/C,OAAO,EAAE,YAAY,EAAE,KAAK,iBAAiB,EAAE,MAAM,8BAA8B,CAAC;AACpF,OAAO,EAAE,aAAa,EAAE,KAAK,kBAAkB,EAAE,MAAM,+BAA+B,CAAC;AACvF,OAAO,EAAE,YAAY,EAAE,KAAK,iBAAiB,EAAE,MAAM,8BAA8B,CAAC;AACpF,OAAO,EAAE,aAAa,EAAE,KAAK,kBAAkB,EAAE,MAAM,+BAA+B,CAAC;AACvF,OAAO,EAAE,aAAa,EAAE,KAAK,kBAAkB,EAAE,MAAM,+BAA+B,CAAC;AACvF,OAAO,EAAE,QAAQ,EAAE,KAAK,aAAa,EAAE,MAAM,0BAA0B,CAAC;AACxE,OAAO,EAAE,cAAc,EAAE,KAAK,mBAAmB,EAAE,MAAM,gCAAgC,CAAC;AAC1F,OAAO,EAAE,iBAAiB,EAAE,KAAK,sBAAsB,EAAE,MAAM,mCAAmC,CAAC;AACnG,OAAO,EAAE,aAAa,EAAE,KAAK,kBAAkB,EAAE,MAAM,+BAA+B,CAAC;AACvF,OAAO,EAAE,WAAW,EAAE,KAAK,gBAAgB,EAAE,MAAM,6BAA6B,CAAC;AACjF,OAAO,EAAE,YAAY,EAAE,KAAK,iBAAiB,EAAE,MAAM,8BAA8B,CAAC;AAGpF,OAAO,EAAE,cAAc,EAAE,KAAK,mBAAmB,EAAE,KAAK,UAAU,EAAE,MAAM,gCAAgC,CAAC;AAC3G,OAAO,EAAE,WAAW,EAAE,KAAK,gBAAgB,EAAE,MAAM,6BAA6B,CAAC;AACjF,OAAO,EAAE,aAAa,EAAE,KAAK,kBAAkB,EAAE,MAAM,+BAA+B,CAAC;AAGvF,OAAO,EAAE,WAAW,EAAE,KAAK,gBAAgB,EAAE,KAAK,eAAe,EAAE,MAAM,sBAAsB,CAAC;AAChG,OAAO,EAAE,UAAU,EAAE,KAAK,eAAe,EAAE,MAAM,qBAAqB,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,kBAAkB,EAAE,KAAK,uBAAuB,EAAE,MAAM,eAAe,CAAC;AAGjF,OAAO,EACL,gBAAgB,EAChB,mBAAmB,EACnB,KAAK,gBAAgB,GACtB,MAAM,mBAAmB,CAAC;AAG3B,OAAO,EAAE,cAAc,EAAE,MAAM,eAAe,CAAC;AAG/C,OAAO,EAAE,YAAY,EAAE,KAAK,iBAAiB,EAAE,MAAM,8BAA8B,CAAC;AACpF,OAAO,EAAE,aAAa,EAAE,KAAK,kBAAkB,EAAE,MAAM,+BAA+B,CAAC;AACvF,OAAO,EAAE,YAAY,EAAE,KAAK,iBAAiB,EAAE,MAAM,8BAA8B,CAAC;AACpF,OAAO,EAAE,aAAa,EAAE,KAAK,kBAAkB,EAAE,MAAM,+BAA+B,CAAC;AACvF,OAAO,EAAE,aAAa,EAAE,KAAK,kBAAkB,EAAE,MAAM,+BAA+B,CAAC;AACvF,OAAO,EAAE,QAAQ,EAAE,KAAK,aAAa,EAAE,MAAM,0BAA0B,CAAC;AACxE,OAAO,EAAE,cAAc,EAAE,KAAK,mBAAmB,EAAE,MAAM,gCAAgC,CAAC;AAC1F,OAAO,EAAE,gBAAgB,EAAE,KAAK,qBAAqB,EAAE,MAAM,kCAAkC,CAAC;AAChG,OAAO,EAAE,gBAAgB,EAAE,KAAK,qBAAqB,EAAE,MAAM,kCAAkC,CAAC;AAChG,OAAO,EACL,qBAAqB,EACrB,KAAK,0BAA0B,GAChC,MAAM,uCAAuC,CAAC;AAG/C,OAAO,EACL,UAAU,EACV,iBAAiB,EACjB,wBAAwB,EACxB,eAAe,EACf,aAAa,EACb,cAAc,EACd,KAAK,aAAa,GACnB,MAAM,kBAAkB,CAAC;AAC1B,OAAO,EAAE,iBAAiB,EAAE,KAAK,sBAAsB,EAAE,MAAM,mCAAmC,CAAC;AACnG,OAAO,EAAE,aAAa,EAAE,KAAK,kBAAkB,EAAE,MAAM,+BAA+B,CAAC;AACvF,OAAO,EAAE,WAAW,EAAE,KAAK,gBAAgB,EAAE,MAAM,6BAA6B,CAAC;AACjF,OAAO,EAAE,YAAY,EAAE,KAAK,iBAAiB,EAAE,MAAM,8BAA8B,CAAC;AAGpF,OAAO,EAAE,cAAc,EAAE,KAAK,mBAAmB,EAAE,KAAK,UAAU,EAAE,MAAM,gCAAgC,CAAC;AAC3G,OAAO,EAAE,WAAW,EAAE,KAAK,gBAAgB,EAAE,MAAM,6BAA6B,CAAC;AACjF,OAAO,EAAE,aAAa,EAAE,KAAK,kBAAkB,EAAE,MAAM,+BAA+B,CAAC;AAGvF,OAAO,EAAE,WAAW,EAAE,KAAK,gBAAgB,EAAE,KAAK,eAAe,EAAE,MAAM,sBAAsB,CAAC;AAChG,OAAO,EAAE,UAAU,EAAE,KAAK,eAAe,EAAE,MAAM,qBAAqB,CAAC"}

package/index.js

@@ -12,6 +12,11 @@ export { ToolCallGroup } from "./components/ToolCallGroup.js";
 export { SubAgentBlock } from "./components/SubAgentBlock.js";
 export { TodoList } from "./components/TodoList.js";
 export { ApprovalPrompt } from "./components/ApprovalPrompt.js";
+export { FileReviewPrompt } from "./components/FileReviewPrompt.js";
+export { FileReviewRecord } from "./components/FileReviewRecord.js";
+export { FileChangeProgressBar, } from "./components/FileChangeProgressBar.js";
+// File-review pure helpers (kind letters, paths, block reasons, settled counts)
+export { kindLetter, changeDisplayPath, progressEntryDisplayPath, blockReasonNote, settledCounts, settledSummary, } from "./file-review.js";
 export { ExecutionProgress } from "./components/ExecutionProgress.js";
 export { FollowUpInput } from "./components/FollowUpInput.js";
 export { UsageWidget } from "./components/UsageWidget.js";

package/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,WAAW;AACX,OAAO,EAAE,kBAAkB,EAAgC,MAAM,eAAe,CAAC;AAEjF,iDAAiD;AACjD,OAAO,EACL,gBAAgB,EAChB,mBAAmB,GAEpB,MAAM,mBAAmB,CAAC;AAE3B,WAAW;AACX,OAAO,EAAE,cAAc,EAAE,MAAM,eAAe,CAAC;AAE/C,aAAa;AACb,OAAO,EAAE,YAAY,EAA0B,MAAM,8BAA8B,CAAC;AACpF,OAAO,EAAE,aAAa,EAA2B,MAAM,+BAA+B,CAAC;AACvF,OAAO,EAAE,YAAY,EAA0B,MAAM,8BAA8B,CAAC;AACpF,OAAO,EAAE,aAAa,EAA2B,MAAM,+BAA+B,CAAC;AACvF,OAAO,EAAE,aAAa,EAA2B,MAAM,+BAA+B,CAAC;AACvF,OAAO,EAAE,QAAQ,EAAsB,MAAM,0BAA0B,CAAC;AACxE,OAAO,EAAE,cAAc,EAA4B,MAAM,gCAAgC,CAAC;AAC1F,OAAO,EAAE,iBAAiB,EAA+B,MAAM,mCAAmC,CAAC;AACnG,OAAO,EAAE,aAAa,EAA2B,MAAM,+BAA+B,CAAC;AACvF,OAAO,EAAE,WAAW,EAAyB,MAAM,6BAA6B,CAAC;AACjF,OAAO,EAAE,YAAY,EAA0B,MAAM,8BAA8B,CAAC;AAEpF,0CAA0C;AAC1C,OAAO,EAAE,cAAc,EAA6C,MAAM,gCAAgC,CAAC;AAC3G,OAAO,EAAE,WAAW,EAAyB,MAAM,6BAA6B,CAAC;AACjF,OAAO,EAAE,aAAa,EAA2B,MAAM,+BAA+B,CAAC;AAEvF,iBAAiB;AACjB,OAAO,EAAE,WAAW,EAA+C,MAAM,sBAAsB,CAAC;AAChG,OAAO,EAAE,UAAU,EAAwB,MAAM,qBAAqB,CAAC"}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,WAAW;AACX,OAAO,EAAE,kBAAkB,EAAgC,MAAM,eAAe,CAAC;AAEjF,iDAAiD;AACjD,OAAO,EACL,gBAAgB,EAChB,mBAAmB,GAEpB,MAAM,mBAAmB,CAAC;AAE3B,WAAW;AACX,OAAO,EAAE,cAAc,EAAE,MAAM,eAAe,CAAC;AAE/C,aAAa;AACb,OAAO,EAAE,YAAY,EAA0B,MAAM,8BAA8B,CAAC;AACpF,OAAO,EAAE,aAAa,EAA2B,MAAM,+BAA+B,CAAC;AACvF,OAAO,EAAE,YAAY,EAA0B,MAAM,8BAA8B,CAAC;AACpF,OAAO,EAAE,aAAa,EAA2B,MAAM,+BAA+B,CAAC;AACvF,OAAO,EAAE,aAAa,EAA2B,MAAM,+BAA+B,CAAC;AACvF,OAAO,EAAE,QAAQ,EAAsB,MAAM,0BAA0B,CAAC;AACxE,OAAO,EAAE,cAAc,EAA4B,MAAM,gCAAgC,CAAC;AAC1F,OAAO,EAAE,gBAAgB,EAA8B,MAAM,kCAAkC,CAAC;AAChG,OAAO,EAAE,gBAAgB,EAA8B,MAAM,kCAAkC,CAAC;AAChG,OAAO,EACL,qBAAqB,GAEtB,MAAM,uCAAuC,CAAC;AAE/C,gFAAgF;AAChF,OAAO,EACL,UAAU,EACV,iBAAiB,EACjB,wBAAwB,EACxB,eAAe,EACf,aAAa,EACb,cAAc,GAEf,MAAM,kBAAkB,CAAC;AAC1B,OAAO,EAAE,iBAAiB,EAA+B,MAAM,mCAAmC,CAAC;AACnG,OAAO,EAAE,aAAa,EAA2B,MAAM,+BAA+B,CAAC;AACvF,OAAO,EAAE,WAAW,EAAyB,MAAM,6BAA6B,CAAC;AACjF,OAAO,EAAE,YAAY,EAA0B,MAAM,8BAA8B,CAAC;AAEpF,0CAA0C;AAC1C,OAAO,EAAE,cAAc,EAA6C,MAAM,gCAAgC,CAAC;AAC3G,OAAO,EAAE,WAAW,EAAyB,MAAM,6BAA6B,CAAC;AACjF,OAAO,EAAE,aAAa,EAA2B,MAAM,+BAA+B,CAAC;AAEvF,iBAAiB;AACjB,OAAO,EAAE,WAAW,EAA+C,MAAM,sBAAsB,CAAC;AAChG,OAAO,EAAE,UAAU,EAAwB,MAAM,qBAAqB,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stigmer/ink",
-  "version": "3.1.1",
+  "version": "3.1.3",
   "description": "Ink (React for terminals) components for rendering Stigmer agent sessions in the terminal",
   "license": "Apache-2.0",
   "type": "module",
@@ -35,7 +35,7 @@
   },
   "dependencies": {
     "@connectrpc/connect-node": "^2.1.1",
-    "@stigmer/react": "3.1.1",
+    "@stigmer/react": "3.1.3",
     "ink-spinner": "^5.0.0",
     "ink-text-input": "^6.0.0",
     "marked": "^15.0.7",
@@ -43,8 +43,8 @@
   },
   "peerDependencies": {
     "@bufbuild/protobuf": "^2.0.0",
-    "@stigmer/protos": "3.1.1",
-    "@stigmer/sdk": "3.1.1",
+    "@stigmer/protos": "3.1.3",
+    "@stigmer/sdk": "3.1.3",
     "ink": "^7.0.0",
     "react": ">=19.0.0"
   }

package/src/__tests__/file-change-progress.test.tsx

@@ -0,0 +1,295 @@
+import React from "react";
+import { describe, it, expect } from "vitest";
+import { render } from "ink-testing-library";
+import { create } from "@bufbuild/protobuf";
+import {
+  FileChangeProgressSchema,
+  FileChangeProgressEntrySchema,
+  CapturedFileChangeSchema,
+} from "@stigmer/protos/ai/stigmer/agentic/agentexecution/v1/filereview_pb";
+import type {
+  FileChangeProgress,
+  FileChangeProgressEntry,
+} from "@stigmer/protos/ai/stigmer/agentic/agentexecution/v1/filereview_pb";
+import { FileChangeKind } from "@stigmer/protos/ai/stigmer/agentic/agentexecution/v1/enum_pb";
+import { FileChangeProgressBar } from "../components/FileChangeProgressBar.js";
+import { changeDisplayPath, progressEntryDisplayPath } from "../file-review.js";
+
+// --- fixtures -------------------------------------------------------------
+
+interface EntryOpts {
+  readonly pathBefore?: string;
+  readonly pathAfter?: string;
+  readonly kind?: FileChangeKind;
+  readonly linesAdded?: number;
+  readonly linesRemoved?: number;
+}
+
+function makeEntry(opts: EntryOpts): FileChangeProgressEntry {
+  return create(FileChangeProgressEntrySchema, {
+    pathBefore: opts.pathBefore ?? "",
+    pathAfter: opts.pathAfter ?? "",
+    kind: opts.kind ?? FileChangeKind.MODIFY,
+    linesAdded: opts.linesAdded ?? 0,
+    linesRemoved: opts.linesRemoved ?? 0,
+  });
+}
+
+interface ProgressOpts {
+  readonly filesChanged?: number;
+  readonly linesAdded?: number;
+  readonly linesRemoved?: number;
+  readonly entries?: FileChangeProgressEntry[];
+}
+
+function makeProgress(opts: ProgressOpts): FileChangeProgress {
+  const entries = opts.entries ?? [];
+  return create(FileChangeProgressSchema, {
+    changeSetId: "cs-1",
+    filesChanged: opts.filesChanged ?? entries.length,
+    linesAdded: opts.linesAdded ?? 0,
+    linesRemoved: opts.linesRemoved ?? 0,
+    entries,
+    capturedAt: "2026-07-05T00:00:00Z",
+  });
+}
+
+// --- FileChangeProgressBar: visibility ------------------------------------
+
+describe("FileChangeProgressBar — visibility", () => {
+  it("renders nothing when progress is undefined", () => {
+    const { lastFrame } = render(<FileChangeProgressBar progress={undefined} />);
+    expect((lastFrame() ?? "").trim()).toBe("");
+  });
+
+  it("renders nothing when no files have changed yet", () => {
+    const { lastFrame } = render(
+      <FileChangeProgressBar progress={makeProgress({ filesChanged: 0 })} />,
+    );
+    expect((lastFrame() ?? "").trim()).toBe("");
+  });
+});
+
+// --- FileChangeProgressBar: compact summary -------------------------------
+
+describe("FileChangeProgressBar — compact summary", () => {
+  it("uses the singular noun for a single file", () => {
+    const { lastFrame } = render(
+      <FileChangeProgressBar
+        progress={makeProgress({
+          filesChanged: 1,
+          entries: [makeEntry({ pathAfter: "one.ts" })],
+        })}
+      />,
+    );
+    expect(lastFrame() ?? "").toContain("1 file changing");
+  });
+
+  it("uses the plural noun for multiple files", () => {
+    const { lastFrame } = render(
+      <FileChangeProgressBar
+        progress={makeProgress({
+          filesChanged: 3,
+          entries: [makeEntry({ pathAfter: "one.ts" })],
+        })}
+      />,
+    );
+    expect(lastFrame() ?? "").toContain("3 files changing");
+  });
+
+  it("shows the aggregate +N -M when counts exist", () => {
+    const { lastFrame } = render(
+      <FileChangeProgressBar
+        progress={makeProgress({
+          filesChanged: 2,
+          linesAdded: 12,
+          linesRemoved: 4,
+          entries: [makeEntry({ pathAfter: "one.ts" })],
+        })}
+      />,
+    );
+    const out = lastFrame() ?? "";
+    expect(out).toContain("+12");
+    expect(out).toContain("-4");
+  });
+
+  it("hides the aggregate stat when both counts are zero (never +0 -0)", () => {
+    const { lastFrame } = render(
+      <FileChangeProgressBar
+        progress={makeProgress({
+          filesChanged: 1,
+          linesAdded: 0,
+          linesRemoved: 0,
+          entries: [makeEntry({ pathAfter: "one.ts" })],
+        })}
+      />,
+    );
+    // No "+" anywhere means no stat rendered; the count copy carries none.
+    expect(lastFrame() ?? "").not.toContain("+");
+  });
+});
+
+// --- FileChangeProgressBar: expansion -------------------------------------
+
+describe("FileChangeProgressBar — expansion", () => {
+  const progress = makeProgress({
+    filesChanged: 2,
+    linesAdded: 8,
+    linesRemoved: 1,
+    entries: [
+      makeEntry({ pathAfter: "src/app.ts", linesAdded: 5, linesRemoved: 1 }),
+      makeEntry({ pathAfter: "src/util.ts", linesAdded: 3, linesRemoved: 0 }),
+    ],
+  });
+
+  it("hides the per-file list when collapsed (default)", () => {
+    const { lastFrame } = render(<FileChangeProgressBar progress={progress} />);
+    const out = lastFrame() ?? "";
+    expect(out).toContain("2 files changing");
+    expect(out).not.toContain("src/app.ts");
+    expect(out).not.toContain("src/util.ts");
+  });
+
+  it("shows the per-file list with kind letter, path, and stats when expanded", () => {
+    const { lastFrame } = render(
+      <FileChangeProgressBar progress={progress} expanded />,
+    );
+    const out = lastFrame() ?? "";
+    expect(out).toContain("src/app.ts");
+    expect(out).toContain("src/util.ts");
+    expect(out).toContain("M"); // MODIFY kind letter
+    expect(out).toContain("+5");
+    expect(out).toContain("+3");
+  });
+
+  it("shows '… and K more' when the entry list is capped below the true total", () => {
+    const capped = makeProgress({
+      filesChanged: 5,
+      entries: [
+        makeEntry({ pathAfter: "a.ts" }),
+        makeEntry({ pathAfter: "b.ts" }),
+      ],
+    });
+    const { lastFrame } = render(
+      <FileChangeProgressBar progress={capped} expanded />,
+    );
+    expect(lastFrame() ?? "").toContain("and 3 more");
+  });
+
+  it("renders a rename entry as 'before → after'", () => {
+    const renamed = makeProgress({
+      filesChanged: 1,
+      entries: [
+        makeEntry({
+          pathBefore: "old/name.ts",
+          pathAfter: "new/name.ts",
+          kind: FileChangeKind.RENAME,
+        }),
+      ],
+    });
+    const { lastFrame } = render(
+      <FileChangeProgressBar progress={renamed} expanded />,
+    );
+    expect(lastFrame() ?? "").toContain("old/name.ts → new/name.ts");
+  });
+
+  it("hides per-file stats for a withheld entry (zero counts)", () => {
+    const secretish = makeProgress({
+      filesChanged: 1,
+      entries: [makeEntry({ pathAfter: ".env", linesAdded: 0, linesRemoved: 0 })],
+    });
+    const { lastFrame } = render(
+      <FileChangeProgressBar progress={secretish} expanded />,
+    );
+    const out = lastFrame() ?? "";
+    expect(out).toContain(".env");
+    expect(out).not.toContain("+");
+  });
+});
+
+// --- path helpers (direct unit coverage) ----------------------------------
+
+interface ChangeOpts {
+  readonly pathBefore?: string;
+  readonly pathAfter?: string;
+  readonly kind: FileChangeKind;
+}
+
+function makeChange(opts: ChangeOpts) {
+  return create(CapturedFileChangeSchema, {
+    id: "c-1",
+    pathBefore: opts.pathBefore ?? "",
+    pathAfter: opts.pathAfter ?? "",
+    kind: opts.kind,
+  });
+}
+
+describe("changeDisplayPath", () => {
+  it("shows the after-path for an add", () => {
+    expect(
+      changeDisplayPath(makeChange({ pathAfter: "new.ts", kind: FileChangeKind.ADD })),
+    ).toBe("new.ts");
+  });
+
+  it("shows the after-path for a modify", () => {
+    expect(
+      changeDisplayPath(
+        makeChange({ pathAfter: "app.ts", kind: FileChangeKind.MODIFY }),
+      ),
+    ).toBe("app.ts");
+  });
+
+  it("falls back to the before-path for a delete", () => {
+    expect(
+      changeDisplayPath(
+        makeChange({ pathBefore: "gone.ts", kind: FileChangeKind.DELETE }),
+      ),
+    ).toBe("gone.ts");
+  });
+
+  it("shows 'before → after' for a rename", () => {
+    expect(
+      changeDisplayPath(
+        makeChange({
+          pathBefore: "old.ts",
+          pathAfter: "new.ts",
+          kind: FileChangeKind.RENAME,
+        }),
+      ),
+    ).toBe("old.ts → new.ts");
+  });
+});
+
+describe("progressEntryDisplayPath", () => {
+  it("shows the after-path for an add", () => {
+    expect(
+      progressEntryDisplayPath(makeEntry({ pathAfter: "new.ts", kind: FileChangeKind.ADD })),
+    ).toBe("new.ts");
+  });
+
+  it("shows the after-path for a modify", () => {
+    expect(
+      progressEntryDisplayPath(makeEntry({ pathAfter: "app.ts" })),
+    ).toBe("app.ts");
+  });
+
+  it("falls back to the before-path for a delete", () => {
+    expect(
+      progressEntryDisplayPath(
+        makeEntry({ pathBefore: "gone.ts", kind: FileChangeKind.DELETE }),
+      ),
+    ).toBe("gone.ts");
+  });
+
+  it("shows 'before → after' for a rename — identical to changeDisplayPath", () => {
+    expect(
+      progressEntryDisplayPath(
+        makeEntry({
+          pathBefore: "old.ts",
+          pathAfter: "new.ts",
+          kind: FileChangeKind.RENAME,
+        }),
+      ),
+    ).toBe("old.ts → new.ts");
+  });
+});