@vincemakes/kiso-tui-cells 0.38.0 → 0.39.0

package/dist/components.d.ts

@@ -306,8 +306,18 @@ export declare function diffBody(diff: import("./diff.js").DiffLine[] | null, W:
 export declare const CAP_TASK_LIVE = 6;
 /** W20 — the settled block's duration, the `2h 14m` form (the task
  *  narrative's long-horizon idiom): minutes+seconds under an hour,
- *  hours+minutes past it. */
+ *  hours+minutes past it. Unchanged. */
 export declare function formatDuration(totalSeconds: number): string;
+/** The LIVE elapsed label — every place a duration is shown while it is
+ *  still running, and on the card that settles from it, so a card and the
+ *  status row can never disagree.
+ *
+ *  It replaced the hand-written second counts, which is how the status
+ *  row came to read "working 637s": ten minutes as a four-figure number,
+ *  with no branch anywhere that said otherwise. Past an hour it keeps
+ *  seconds, because a clock the user is watching tick should not stop
+ *  ticking. */
+export declare function elapsedLabel(totalSeconds: number): string;
 /** The status container's row: the status text (+ the tail) with the
  *  right-aligned "/ commands · ↑ history" hint in the idle state —
  *  the hint CUT FIRST when the width is short (the #16g rule); when

package/dist/components.js

@@ -823,7 +823,7 @@ class ToolExecution {
             // rewrites it in place: `3s · esc stops` → `exit 0 · 90 lines ·
             // 3.2s`.
             const gestures = c.name === "shell" ? " · esc stops · alt+⏎ redirects" : "";
-            const status = pickTier([`${elapsed}s${gestures}`, `${elapsed}s`], Math.max(1, W - visibleWidth(noteIndent())));
+            const status = pickTier([`${elapsedLabel(elapsed)}${gestures}`, elapsedLabel(elapsed)], Math.max(1, W - visibleWidth(noteIndent())));
             const live = toolBlockBody(c, W, ctx);
             if (live.length > 0)
                 return slabBlock(gutterCut(gutter, `${verbCol} ${liveTarget(c)}`, W)[0], live, status, W);
@@ -846,7 +846,7 @@ class ToolExecution {
             // way so the COMMAND keeps something to say, and the elapsed —
             // pin 4's core — never does.
             const MIN_TARGET = 10; // the gutter, the verb column, a character of command
-            const oneRow = pickTier([`${elapsed}s${gestures}`, `${elapsed}s`], Math.max(1, W - MIN_TARGET - 3));
+            const oneRow = pickTier([`${elapsedLabel(elapsed)}${gestures}`, elapsedLabel(elapsed)], Math.max(1, W - MIN_TARGET - 3));
             const room = Math.max(4, W - visibleWidth(oneRow) - 3);
             const only = gutterCut(gutter, `${verbCol} ${liveTarget(c)}`, room)[0];
             return slabBlock(`${only}${p.dim} · ${oneRow}${p.reset}`, [], null, W);
@@ -854,7 +854,10 @@ class ToolExecution {
         // W2: ◦ replaces → for QUEUED — · is the separator inside every
         // metadata group; a queued marker that is also the separator
         // glyph reads as noise
-        return gutterCut(`${p.dim}◦${p.reset} `, `${verbCol} ${liveTarget(c)}`, W);
+        // The `◦` carried "not started yet" ALONE, and a marker is not a
+        // word: on a screen where running and queued rows sit together, a
+        // reader has to already know the glyph. The suffix says it.
+        return gutterCut(`${p.dim}◦${p.reset} `, `${verbCol} ${liveTarget(c)}${p.dim} · queued${p.reset}`, W);
     }
 }
 // ---- TUI2-R1 (A): the self-naming expand affordance ----
@@ -1780,15 +1783,42 @@ class Banner {
  *  done-collapse. Every live row CUTS at W (never folds) — the block's
  *  height is its row count. */
 export const CAP_TASK_LIVE = 6;
-/** W20 — the settled block's duration, the `2h 14m` form (the task
- *  narrative's long-horizon idiom): minutes+seconds under an hour,
- *  hours+minutes past it. */
-export function formatDuration(totalSeconds) {
+/** The two duration idioms, from ONE implementation.
+ *
+ *  They agree in every branch but the hour: W20's settled task block says
+ *  `2h 14m` (a long-horizon narrative does not care about seconds), and a
+ *  LIVE elapsed label says `1h 2m 3s` (a running clock does). A near-copy
+ *  differing in one branch is the drift that a shared helper exists to
+ *  prevent, so the branch is a parameter.
+ *
+ *  Negative is clamped: a clock skew is not a negative duration. */
+function duration(totalSeconds, hoursKeepSeconds) {
     const s = Math.max(0, Math.round(totalSeconds));
     if (s < 60)
         return `${s}s`;
     const m = Math.floor(s / 60);
-    return m < 60 ? `${m}m ${s % 60}s` : `${Math.floor(m / 60)}h ${m % 60}m`;
+    if (m < 60)
+        return `${m}m ${s % 60}s`;
+    const h = Math.floor(m / 60);
+    return hoursKeepSeconds ? `${h}h ${m % 60}m ${s % 60}s` : `${h}h ${m % 60}m`;
+}
+/** W20 — the settled block's duration, the `2h 14m` form (the task
+ *  narrative's long-horizon idiom): minutes+seconds under an hour,
+ *  hours+minutes past it. Unchanged. */
+export function formatDuration(totalSeconds) {
+    return duration(totalSeconds, false);
+}
+/** The LIVE elapsed label — every place a duration is shown while it is
+ *  still running, and on the card that settles from it, so a card and the
+ *  status row can never disagree.
+ *
+ *  It replaced the hand-written second counts, which is how the status
+ *  row came to read "working 637s": ten minutes as a four-figure number,
+ *  with no branch anywhere that said otherwise. Past an hour it keeps
+ *  seconds, because a clock the user is watching tick should not stop
+ *  ticking. */
+export function elapsedLabel(totalSeconds) {
+    return duration(totalSeconds, true);
 }
 /**
  * W20 — the task checklist as STATE: ONE live block that redraws in

package/dist/index.d.ts

@@ -6,7 +6,7 @@
  * package); the cli never imports it directly. Experimental — no
  * API-stability promise yet.
  */
-export { SPINNER, foldLine, foldWords, visibleWidth, bodySpacing, Container, cellComponent, foldTerms, CAP_TASK_LIVE, formatDuration, statusLine, boxTop, boxBottom, terminalPipe, type FrameCtx, type RenderLine, type Component, type BodyCell, } from "./components.js";
+export { SPINNER, foldLine, foldWords, visibleWidth, bodySpacing, Container, cellComponent, foldTerms, CAP_TASK_LIVE, elapsedLabel, formatDuration, statusLine, boxTop, boxBottom, terminalPipe, type FrameCtx, type RenderLine, type Component, type BodyCell, } from "./components.js";
 export { editFileDiff, truncateDiff, writeFileDiff, type DiffLine, type DiffResult } from "./diff.js";
 export { pendingQueueRows } from "./components.js";
 export { charWidth, displayWidth, leadWidth, widthOf } from "./width.js";

package/dist/index.js

@@ -6,7 +6,7 @@
  * package); the cli never imports it directly. Experimental — no
  * API-stability promise yet.
  */
-export { SPINNER, foldLine, foldWords, visibleWidth, bodySpacing, Container, cellComponent, foldTerms, CAP_TASK_LIVE, formatDuration, statusLine, boxTop, boxBottom, terminalPipe, } from "./components.js";
+export { SPINNER, foldLine, foldWords, visibleWidth, bodySpacing, Container, cellComponent, foldTerms, CAP_TASK_LIVE, elapsedLabel, formatDuration, statusLine, boxTop, boxBottom, terminalPipe, } from "./components.js";
 export { editFileDiff, truncateDiff, writeFileDiff } from "./diff.js";
 // W22 (the v8 input round): the pending-queue chips — the SAME
 // UserMessage chip with the □ gutter, pre-rendered above the input

package/dist/md.js

@@ -713,35 +713,36 @@ function tableRows(b, W) {
     if (cols === null)
         return recordRows(t, W);
     const p = palette();
-    // R2: no rails. The drawn width is two columns of inset plus the
-    // columns and their two-space gutters — a table is bounded by the
-    // blank lines above and below it, exactly as every other block on the
-    // screen is, and it was the last box left on a screen that has decided
-    // not to have boxes. Alignment does the work the rails were doing, and
-    // a copied table is closer to markdown without them.
+    // DECLARED REVERSAL (2026-09-17), TABLES ONLY. R2 removed the rails at
+    // the nineteen-screen review and MD-1.3 put ONE rule under the header;
+    // both are superseded here. Every other R2 hairline rule stands, and
+    // the record form keeps NO borders — it is the fallback, not a table.
+    //
+    // The rails are hairlines in the R2 hairline colour, and the styling
+    // goes on AFTER the measure, exactly as it does for a cell: a colour
+    // can never move a column.
+    const rule = (l, m, r) => `${p.dim}${l}${cols.map((w) => "\u2500".repeat(w + 2)).join(m)}${r}${p.reset}`;
+    const V = `${p.dim}\u2502${p.reset}`;
     const row = (cells, bold) => {
         const boxes = cells.map((c, i) => cellBox(c, cols[i], t.align[i], bold));
         const rows = [];
         for (let k = 0; k < Math.max(...boxes.map((x) => x.length)); k += 1) {
             // a short box pays its blanks so the columns to its right do not
             // move: a cell is a BOX, and the row is as tall as its tallest.
-            rows.push(`  ${boxes.map((x, i) => x[k] ?? " ".repeat(cols[i])).join("  ")}`.replace(/\s+$/, ""));
+            // Trailing space is NOT stripped any more — it sits inside the
+            // closing rail, and stripping it would pull the rail left.
+            rows.push(`${V}${boxes.map((x, i) => ` ${x[k] ?? " ".repeat(cols[i])} `).join(V)}${V}`);
         }
         return rows;
     };
-    // MD-1.3 / R2 AMENDMENT 1 (owner ruling, 2026-09-11) — ONE rule under
-    // the header row, at the grid's own width. R2 removed the RAILS: the
-    // four-sided box that BOUNDS a table. This bounds nothing; it SEPARATES
-    // the header from the body, which is the one job the round's governing
-    // distinction gives a rule — a rule separates, a gutter scopes, a rail
-    // bounds. Rails stay out.
-    //
-    // What it buys is not decoration: without it a six-row table's header
-    // was carried by SGR bold ALONE, so in a pipe, under NO_COLOR, or on a
-    // terminal with weak bold, seven identical rows arrived with nothing
-    // saying which one names the columns.
-    const ruleW = cols.reduce((n, w) => n + w, 0) + Math.max(0, cols.length - 1) * 2;
-    return [...row(t.header, true), `  ${p.dim}${"\u2500".repeat(ruleW)}${p.reset}`, ...t.rows.flatMap((r) => row(r, false))];
+    const body = t.rows.flatMap((r, i) => (i === 0 ? row(r, false) : [rule("\u251c", "\u253c", "\u2524"), ...row(r, false)]));
+    return [
+        rule("\u250c", "\u252c", "\u2510"),
+        ...row(t.header, true),
+        rule("\u251c", "\u253c", "\u2524"),
+        ...body,
+        rule("\u2514", "\u2534", "\u2518"),
+    ];
 }
 /** A cell's column count: what a human sees, styling removed. */
 function cellWidth(cell) {
@@ -753,13 +754,22 @@ function cellWidth(cell) {
  *  report's sample stopped reading as a table — and it is the one number
  *  that decides when the record form is still the better answer. */
 const CELL_FLOOR = 8;
-/** The drawn width of a grid with these columns, by the measure the R2
- *  table has always used: the two-column inset plus every column AND its
- *  two-space gutter. Conservative by one gutter (the last column has
- *  none), which is where the table's right margin comes from — kept as
- *  it was, because the record threshold has always been stated in it. */
+/** The drawn width of a grid with these columns.
+ *
+ *  DECLARED REVERSAL (2026-09-17, tables only): the R2 measure was the
+ *  two-column inset plus every column and its two-space gutter — the
+ *  shape of a table with NO RAILS. With rails it is `sum + 3n + 1`: one
+ *  rail between every pair of columns and one at each edge (n+1), and one
+ *  space of padding inside every rail (2n).
+ *
+ *  This function's old comment said the record threshold "has always been
+ *  stated in it", and that is exactly why changing it IS the re-cut: the
+ *  grid costs n−1 columns more than the railless form — 2 for a
+ *  three-column table, 6 for a seven-column one — and the record fallback
+ *  arrives that much earlier. Nothing else moves: the greedy shrink, the
+ *  CELL_FLOOR of 8 and the fallback criterion are untouched. */
 function gridWidth(cols) {
-    return cols.reduce((n, w) => n + w + 2, 2);
+    return cols.reduce((n, w) => n + w, 0) + 3 * cols.length + 1;
 }
 /**
  * MD-1.1 — take one column off the WIDEST column until the grid fits.

package/dist/render.js

@@ -220,8 +220,14 @@ function toolSummaryDetail(name, input, result) {
             const exit = exitCodeOf(result);
             return `${command} (exit ${exit})`;
         }
-        case "list_dir":
-            return String(input.path ?? "(root)");
+        case "list_dir": {
+            // "(root)" already answered an ABSENT path. A model that sends
+            // "." explicitly is making the same request and was getting the
+            // dot on the row. `./` likewise. Anything else — including `..`
+            // and a dotfile directory like `.github` — is a real path.
+            const dir = String(input.path ?? ".");
+            return dir === "." || dir === "./" ? "(root)" : dir;
+        }
         case "search_text":
             return searchSubject(input);
         default:
@@ -261,8 +267,14 @@ function toolTargetRaw(name, input) {
             return String(input.path ?? "?");
         case "shell":
             return String(input.command ?? "?");
-        case "list_dir":
-            return String(input.path ?? "(root)");
+        case "list_dir": {
+            // "(root)" already answered an ABSENT path. A model that sends
+            // "." explicitly is making the same request and was getting the
+            // dot on the row. `./` likewise. Anything else — including `..`
+            // and a dotfile directory like `.github` — is a real path.
+            const dir = String(input.path ?? ".");
+            return dir === "." || dir === "./" ? "(root)" : dir;
+        }
         case "search_text":
             return searchSubject(input);
         default:

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vincemakes/kiso-tui-cells",
-	"version": "0.38.0",
+	"version": "0.39.0",
 	"description": "kiso tui-cells \u2014 the components cell renderer (components, diff, width, the render slice). Zero runtime dependencies: input is data, output is bytes.",
 	"type": "module",
 	"license": "MIT",