@vincemakes/kiso-tui 0.24.0 → 0.24.2

package/dist/compositor.js

@@ -52,7 +52,7 @@ import { MOUSE_OFF } from "./editor.js";
 import { atPanelRows, bandHeader } from "./at-picker.js";
 // TUI2-R2 ②: the session picker's rows — the band's third occupant.
 import { sessionPickerRows } from "./session-picker.js";
-import { CAP_PREVIEW, Container, ROLLUP_NOUN, MOTION_FRAMES, MdStream, bodySpacing, boxBottom, boxTop, cellComponent, foldCountsObjects, foldTerms, focusToken, foldLine, gutterCut, cutLine, isExploreTool, moreRunningRow, pendingQueueRows, slotPad, slotTail, statusLine, visibleWidth, breathFrame, } from "./components.js";
+import { CAP_PREVIEW, thinkingRow, expandedCard, Container, ROLLUP_NOUN, MOTION_FRAMES, MdStream, bodySpacing, boxBottom, boxTop, cellComponent, foldCountsObjects, foldTerms, focusToken, foldLine, gutterCut, cutLine, isExploreTool, moreRunningRow, pendingQueueRows, slotPad, slotTail, statusLine, visibleWidth, breathFrame, } from "./components.js";
 import { bannerLines, escapeTerminal, foldResult, foldThinking, palette, renderTerminalGap, renderToolSummary, toolTarget } from "./render.js";
 import { displayVerb, keysSheetRows } from "./strings.js";
 // R5 — the transcript viewer's PURE projection. The compositor supplies
@@ -1258,18 +1258,19 @@ export class Body {
         }
         const turnsBack = this.#cells.slice(idx + 1).filter((c) => c.kind === "user").length;
         const p = palette();
-        const header = `${p.bold}✦${p.reset} expanded · ${escapeTerminal(`${displayVerb(cell.name)} ${toolTarget(cell.name, input)}`)} · ${turnsBack} ${turnsBack === 1 ? "turn" : "turns"} back`;
+        // 0.24.2 ③ — the expansion is a CARD, and `✦` is not its mark: that
+        // glyph is the turn recap's, and one symbol with two meanings is
+        // §4.1. The head row names the call, which is the tie to it that
+        // the mark's sentence used to be.
+        const verb = displayVerb(cell.name);
+        const elapsed = cell.startedAt !== null && cell.doneAt !== null ? ((cell.doneAt - cell.startedAt) / 1000).toFixed(1) : "?";
+        const n = cell.resultText === "" ? 0 : cell.resultText.split("\n").length;
         return {
             kind: "appended",
-            lines: [
-                header,
-                // TUI2-R2pre ④: the SECTION HEADERS say the act; the payloads
-                // below them (inputFull, resultText) are RAW and byte-identical.
-                `--- ${displayVerb(cell.name)} input ---`,
-                cell.inputFull,
-                `--- ${displayVerb(cell.name)} output${cell.isError ? " (error)" : ""} ---`,
-                cell.resultText,
-            ],
+            lines: expandedCard(verb, toolTarget(cell.name, input), `expanded · ${turnsBack} ${turnsBack === 1 ? "turn" : "turns"} back`, 
+            // TUI2-R2pre ④: the SECTION HEADERS say the act; the payloads
+            // below them (inputFull, resultText) are RAW and byte-identical.
+            [`--- ${verb} input ---`, cell.inputFull, `--- ${verb} output${cell.isError ? " (error)" : ""} ---`, cell.resultText], [cell.isError ? "failed" : "exit 0", `${n} line${n === 1 ? "" : "s"}`, `${elapsed}s`].filter((x) => x !== "").join(" · "), this.#opts.width()),
         };
     }
     // ---- the Dock façade (the CLI's chrome API — same shape as the old dock) ----
@@ -1821,8 +1822,33 @@ export class Body {
             out.push(...this.#space(i, prev, rows));
             prev = rows;
         }
+        // 0.24.2 ② — THE PLACEHOLDER. A turn in flight whose live region
+        // has drawn nothing leaves the composer under a blank stretch,
+        // with only the status row saying anything is happening; the owner
+        // read that as the product having stopped.
+        //
+        // So one row says it: `thinking…`, dim italic at column 2, no
+        // glyph — the SAME column and the same font a thinking paragraph
+        // takes, so whatever arrives replaces it in place and the eye sees
+        // a word change rather than a jump.
+        //
+        // It is NOT a cell. It never commits, never reaches the
+        // scrollback, and neither `/last` nor the pipe has heard of it —
+        // which is the whole of why a row that is a guess about the future
+        // is allowed at all: a row that never becomes history cannot make
+        // history wrong.
+        if (out.length === 0 && this.#thinkingGap())
+            return [...bodySpacing(this.#committed > 0 ? this.#lineCache[this.#committed - 1] : null, [thinkingRow()])];
         return out;
     }
+    /** 0.24.2 ② — is this the gap the placeholder is for? A turn has
+     *  begun and not ended, and nothing else is drawing a live row (the
+     *  caller has already found the projection empty, which is what "no
+     *  card is running" reduces to once every cell renders itself). */
+    #thinkingGap() {
+        const turn = this.#turns[this.#turns.length - 1];
+        return turn !== undefined && !turn.ended;
+    }
     /** The live region's scalar — the unit tests assert the cap directly
      *  (the e2e gate pins the screen consequence). W11: the formula's
      *  blanks are join artifacts — the count includes them (they are real
@@ -3119,6 +3145,8 @@ export class Body {
     }
     /** Invariant ①: every emitted line fits the width — a violation is a
      *  CRASH with the diagnostic, never a silent truncate. */
+    /** DC-48 — the field cut is announced ONCE per session. */
+    #cutOnce = false;
     #checked(line, W) {
         // Invariant ①b (R3f): a ROW IS ONE PHYSICAL ROW.
         //
@@ -3148,7 +3176,26 @@ export class Body {
         }
         const w = visibleWidth(line);
         if (w > W) {
-            throw new Error(`kiso-tui invariant ① violated: a line of visible width ${w} > ${W} was about to be emitted — ${JSON.stringify(line.slice(0, 80))}`);
+            // DECLARED REVERSAL of "the crash is the contract, not a
+            // symptom" — owner-lane ruling 2026-09-04, after the SECOND
+            // instance of this class in two days: DC-45 in a gate, DC-48 in
+            // the owner's hands. Under test the crash is exactly right and
+            // keeps every tooth; in the field it costs a human the
+            // composer's contents and the whole session, to save them a row
+            // that is one column too wide.
+            //
+            // So: THROW UNDER TEST, CUT IN THE FIELD, and say so. The
+            // notice is once per session — a defect worth reporting is
+            // worth saying once, and a row-by-row complaint on a resize
+            // storm would be its own defect.
+            const why = `kiso-tui invariant ① violated: a line of visible width ${w} > ${W} was about to be emitted — ${JSON.stringify(line.slice(0, 80))}`;
+            if (process.env.KISO_INVARIANTS === "throw")
+                throw new Error(why);
+            if (!this.#cutOnce) {
+                this.#cutOnce = true;
+                queueMicrotask(() => this.notice("kiso-tui: a row was cut to width (invariant ①) · please report"));
+            }
+            return cutLine(line, W);
         }
         return line;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vincemakes/kiso-tui",
-	"version": "0.24.0",
+	"version": "0.24.2",
 	"description": "kiso tui — the pure terminal layer (cell renderer, dock, raw editor, diff, palette). Zero runtime dependencies: input is data, output is bytes.",
 	"type": "module",
 	"license": "MIT",
@@ -35,6 +35,6 @@
 	},
 	"homepage": "https://github.com/vincemakes/kiso/tree/main/packages/tui#readme",
 	"dependencies": {
-		"@vincemakes/kiso-tui-cells": "0.24.0"
+		"@vincemakes/kiso-tui-cells": "0.24.2"
 	}
 }