@cspeach/cli 1.1.51 → 1.1.53

package/dist/ui/widgets/ask-card.js

@@ -22,7 +22,7 @@ import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
  * the text input is not focused) · Esc cancels; on the free-text row Esc
  * first returns to the options.
  */
-import { useMemo, useRef, useState } from 'react';
+import { useEffect, useLayoutEffect, useMemo, useRef, useState } from 'react';
 import { Box, Text, useInput } from 'ink';
 import TextInput from '../text-input.js';
 import { useReclaimRawStdin } from './reclaim-raw-stdin.js';
@@ -34,14 +34,31 @@ import { sanitizeControlChars } from '../../renderer/sanitize.js';
 import { wrapRows } from '../../renderer/wrap-rows.js';
 import { answerDisplayText } from '../ask-answer-text.js';
 import { CardFrame } from './confirm-card.js';
+import { emitTrace } from '../../renderer/trace.js';
 /** Option rows visible at once — the rest scroll. */
 export const VISIBLE_OPTION_ROWS = 5;
 /**
  * Single-key cards (the Yes/No agent confirm) drop keys received in the first
  * GUARD_MS after mount, so a key typed ahead while output streamed cannot
  * answer the card.
+ *
+ * 1.1.53 Task 20 (F-P) — every card, keys or not, also ignores Enter, digit
+ * picks and alias keys for GUARD_MS after each screen appears (the card, each
+ * next question of a form, the Review step), so the Enter of a "pick, then
+ * Enter" habit cannot answer the card that comes next. Typing into the
+ * typed-answer row is never dropped.
  */
 const GUARD_MS = 150;
+/** Task 20 — the per-screen pick window; GUARD_MS outside tests. */
+let pickGuardMs = GUARD_MS;
+/**
+ * Test seam only: interaction tests that drive keys faster than a person can
+ * type set 0; undefined restores GUARD_MS. The window itself is pinned by
+ * ask-card-stray-enter.test.tsx. The y/n mount guard is not affected.
+ */
+export function setAskPickGuardMsForTest(ms) {
+    pickGuardMs = ms ?? GUARD_MS;
+}
 /**
  * Card text is model- or tool-supplied. Tabs and newlines are flattened by
  * truncateToWidth; every OTHER control character is neutralised by the
@@ -56,7 +73,12 @@ const sanitize = sanitizeControlChars;
 function fit(text, cells) {
     return truncateToWidth(sanitize(text), cells);
 }
-/** Round 4 item 8b — the question wraps up to this many rows, then is cut. */
+/**
+ * Round 4 item 8b — the question's base rows. 1.1.53 Task 14: no longer the
+ * cap. With a row budget (App) the current question grows past it to as many
+ * rows as it needs while the card still fits (askCardLayout); without one it
+ * shows whole. Only a question that still does not fit is cut.
+ */
 export const QUESTION_MAX_ROWS = 3;
 /** Context wraps too; it gets at least one row per line and at most this many in all (or its line count, if larger). */
 export const CONTEXT_MAX_ROWS = 6;
@@ -70,6 +92,8 @@ const DESCRIPTION_INDENT = '     ';
 export const CONTEXT_FOLDED_ROWS = 3;
 /** …and then the question to this many. */
 export const QUESTION_FOLDED_ROWS = 2;
+/** 1.1.53 Task 14 — the dim last row of a cut question; the whole question is printed in the scrollback above. */
+export const FULL_QUESTION_NOTE = 'The full question is printed above.';
 /** The question rows (at most `maxRows`), with the multi-select suffix kept whole on the last row. */
 function questionRows(question, suffix, cells, maxRows = QUESTION_MAX_ROWS) {
     if (!suffix)
@@ -87,6 +111,58 @@ function questionRows(question, suffix, cells, maxRows = QUESTION_MAX_ROWS) {
     }
     return rows;
 }
+/**
+ * 1.1.53 Task 14 — the question as plain wrapped text for the scrollback:
+ * each line of it wrapped at `cells`, nothing cut. Printed above the card
+ * when the card has to cut the question, so the person can read it whole.
+ */
+export function fullQuestionText(question, cells) {
+    return question.split(/\r?\n/).map((line) => wrapRows(line, cells, Number.MAX_SAFE_INTEGER).join('\n')).join('\n');
+}
+/** The multi-select suffix, when the width has room for it (kept whole on the question's last row). */
+function questionSuffix(q, cells) {
+    return q.multiSelect && cells - MULTI_SUFFIX.length >= 1 ? MULTI_SUFFIX : '';
+}
+/** Rows the whole question needs (suffix included). */
+function questionNeed(q, cells) {
+    // The layout asks this many times per render (folds, growth, estimate): memoised.
+    const key = `${cells}|${q.multiSelect ? 1 : 0}|${q.question}`;
+    let need = needCache.get(key);
+    if (need === undefined) {
+        need = questionRows(q.question, questionSuffix(q, cells), cells, Number.MAX_SAFE_INTEGER).length;
+        if (needCache.size >= 64)
+            needCache.clear();
+        needCache.set(key, need);
+    }
+    return need;
+}
+const needCache = new Map();
+/** Fix round 2 (I-1) — a cut question gets the pointer row only from this many rows; below, the old cut rows. */
+const POINTER_MIN_ROWS = 3;
+/**
+ * 1.1.53 Task 14 — the question as drawn in at most `cap` rows. It fits: the
+ * same rows as before. It does not: with `cap` of POINTER_MIN_ROWS or more it
+ * is cut to `cap - 1` rows and the last row is the FULL_QUESTION_NOTE pointer;
+ * with fewer (the short-screen fold) it is cut to `cap` rows exactly as
+ * before, no pointer. Either way the block is min(need, cap) rows tall, so
+ * the height math is today's.
+ */
+function questionBlock(q, cells, cap) {
+    // Memoised like questionNeed: the fold and growth loops ask for many caps.
+    const key = `${cells}|${q.multiSelect ? 1 : 0}|${cap}|${q.question}`;
+    const hit = blockCache.get(key);
+    if (hit)
+        return hit;
+    const suffix = questionSuffix(q, cells);
+    const block = questionNeed(q, cells) <= cap
+        ? { lines: questionRows(q.question, suffix, cells, cap), cut: false }
+        : { lines: questionRows(q.question, suffix, cells, cap >= POINTER_MIN_ROWS ? cap - 1 : cap), cut: true };
+    if (blockCache.size >= 256)
+        blockCache.clear();
+    blockCache.set(key, block);
+    return block;
+}
+const blockCache = new Map();
 /**
  * Context rows: every line keeps at least one row; wrapping fills the rest of
  * the budget. Folded (`maxRows` below CONTEXT_MAX_ROWS, Task 4) the context
@@ -108,6 +184,8 @@ function contextRows(context, cells, maxRows = CONTEXT_MAX_ROWS) {
     });
     return out;
 }
+/** The rows question `i` may take at `layout`. */
+const questionCap = (layout, i) => layout.questionRowsFor?.[i] ?? layout.questionRows;
 /**
  * Lay out one option at `inner` cells. The label wraps onto LABEL_MAX_ROWS
  * under its own text; the (recommended) / (default) suffix stays whole on
@@ -151,6 +229,17 @@ export function optionRows(o, idx, q, inner, layout) {
 }
 /** Rows one option takes on the card. */
 const optionHeight = (r) => r.label.length + r.description.length;
+/**
+ * 1.1.53 R16 — the digit 1-9 a key chunk stands for, or 0. Remote desktops,
+ * VDI, ConPTY and key drivers can deliver a digit and Enter as one chunk
+ * ("2\r"): it counts as the digit alone, and the newline goes with it (never
+ * on to a following card). Any other multi-byte chunk ("22", an arrow plus a
+ * digit) is no digit.
+ */
+function digitKey(input) {
+    const m = /^([1-9])[\r\n]*$/.exec(input);
+    return m ? Number(m[1]) : 0;
+}
 export function toCardQuestions(request) {
     if ('questions' in request) {
         return request.questions.map((q) => {
@@ -212,29 +301,31 @@ const suffixText = (r) => (r.rec ? REC : '') + (r.def ? DEFAULT_SUFFIX : '');
 const askInner = (width) => Math.max(1, width - 4);
 /** The frame's top and bottom edges. */
 const FRAME_ROWS = 2;
+/** The rows of one question's screen when its question may take `cap` rows. */
+function screenRows(q, chipRows, inner, layout, cap) {
+    const ctx = contextRows(q.context, inner, layout.contextRows).length;
+    const block = questionBlock(q, inner, cap);
+    const qRows = block.lines.length + (block.cut && cap >= POINTER_MIN_ROWS ? 1 : 0); // + the pointer row
+    let body = 2; // margin + the text input row
+    if (!q.freeText) {
+        // Every wrapped label and description row counts; with more options
+        // than the window shows, the tallest window of VISIBLE_OPTION_ROWS.
+        const heights = q.options.map((o, i) => optionHeight(optionRows(o, i, q, inner, layout)));
+        const span = Math.min(heights.length, VISIBLE_OPTION_ROWS);
+        let tallest = 0;
+        for (let start = 0; start + span <= heights.length; start++) {
+            tallest = Math.max(tallest, heights.slice(start, start + span).reduce((x, y) => x + y, 0));
+        }
+        const arrows = q.options.length > VISIBLE_OPTION_ROWS ? 2 : 0;
+        body = 1 + tallest + arrows + (q.otherRow ? 1 : 0);
+    }
+    return chipRows + qRows + ctx + body + 1 + FRAME_ROWS; // question + body + hint + frame
+}
 /** The card's rows at `layout` — worst case over every question screen, plus the Review screen. */
 function askRowsFor(qs, inner, layout) {
     const chipRows = qs.length > 1 ? 2 : 0; // tab row + its blank line
     let worst = 0;
-    for (const q of qs) {
-        const ctx = contextRows(q.context, inner, layout.contextRows).length;
-        const suffix = q.multiSelect && inner - MULTI_SUFFIX.length >= 1 ? MULTI_SUFFIX : '';
-        const qRows = questionRows(q.question, suffix, inner, layout.questionRows).length;
-        let body = 2; // margin + the text input row
-        if (!q.freeText) {
-            // Every wrapped label and description row counts; with more options
-            // than the window shows, the tallest window of VISIBLE_OPTION_ROWS.
-            const heights = q.options.map((o, i) => optionHeight(optionRows(o, i, q, inner, layout)));
-            const span = Math.min(heights.length, VISIBLE_OPTION_ROWS);
-            let tallest = 0;
-            for (let start = 0; start + span <= heights.length; start++) {
-                tallest = Math.max(tallest, heights.slice(start, start + span).reduce((x, y) => x + y, 0));
-            }
-            const arrows = q.options.length > VISIBLE_OPTION_ROWS ? 2 : 0;
-            body = 1 + tallest + arrows + (q.otherRow ? 1 : 0);
-        }
-        worst = Math.max(worst, chipRows + qRows + ctx + body + 1 + FRAME_ROWS); // question + body + hint + frame
-    }
+    qs.forEach((q, i) => { worst = Math.max(worst, screenRows(q, chipRows, inner, layout, questionCap(layout, i))); });
     if (qs.length > 1) {
         // title + one row per answer + margin + choices + hint + frame
         worst = Math.max(worst, chipRows + 1 + qs.length + 1 + REVIEW_CHOICES.length + 1 + FRAME_ROWS);
@@ -256,10 +347,12 @@ export function askCardLayout(request, showDescriptions, width = uiWidth(), maxR
         contextRows: CONTEXT_MAX_ROWS,
         questionRows: QUESTION_MAX_ROWS,
     };
-    if (maxRows === undefined)
-        return layout;
     const qs = toCardQuestions(request);
     const inner = askInner(width);
+    // 1.1.53 Task 14 — no budget: every question shows whole.
+    if (maxRows === undefined) {
+        return { ...layout, questionRowsFor: qs.map((q) => Math.max(layout.questionRows, questionNeed(q, inner))) };
+    }
     const folds = [
         (l) => { l.labelRows = 1; },
         (l) => { l.descriptionRows = Math.min(l.descriptionRows, 1); },
@@ -272,6 +365,21 @@ export function askCardLayout(request, showDescriptions, width = uiWidth(), maxR
             break;
         fold(layout);
     }
+    // 1.1.53 Task 14 — then each question grows to the rows it needs while its
+    // screen still fits maxRows. What still does not fit stays cut (the AskCard
+    // prints it whole above). The question block is min(need, cap) rows tall
+    // (questionBlock), so the rest of the screen is measured once at the base
+    // and the cap follows directly (fix round 2, I-2 — no per-row re-wrap).
+    const chipRows = qs.length > 1 ? 2 : 0;
+    const base = layout.questionRows;
+    layout.questionRowsFor = qs.map((q) => {
+        const need = questionNeed(q, inner);
+        if (need <= base)
+            return base;
+        const rest = screenRows(q, chipRows, inner, layout, base) - base; // everything but the question
+        const room = maxRows - rest;
+        return need <= room ? need : Math.max(base, room);
+    });
     return layout;
 }
 /**
@@ -320,6 +428,9 @@ export function AskCard({ request, width, showDescriptions, maxRows, onResolveV1
     // while output streamed must not answer the card.
     const mountGuard = questions.some((qq) => qq.keys !== undefined);
     const mountedAt = useRef(Date.now());
+    // Task 20 — when the current screen appeared (reset on every focus change).
+    const screenAt = useRef(Date.now());
+    const pickGuarded = () => Date.now() - screenAt.current < pickGuardMs;
     const done = useRef(false);
     // Focus slots: 0..n-1 = questions; n = the Review step (2+ questions,
     // reachable only once every question has an answer).
@@ -329,6 +440,16 @@ export function AskCard({ request, width, showDescriptions, maxRows, onResolveV1
     const [drafts, setDrafts] = useState(() => questions.map(() => ''));
     const [answers, setAnswers] = useState(() => questions.map(() => null));
     const [reviewCursor, setReviewCursor] = useState(0);
+    const firstScreen = useRef(true);
+    useLayoutEffect(() => {
+        if (firstScreen.current) {
+            firstScreen.current = false;
+            return;
+        }
+        screenAt.current = Date.now();
+    }, [focus]);
+    // 1.1.53 Task 14 — questions already printed whole to the scrollback.
+    const printedFull = useRef(new Set());
     const isReview = n > 1 && focus === n;
     const q = isReview ? undefined : questions[focus];
     const cursor = q ? cursors[focus] : 0;
@@ -421,6 +542,8 @@ export function AskCard({ request, width, showDescriptions, maxRows, onResolveV1
             answerAndAdvance(focus, opt.value);
     };
     const onTextSubmit = (v) => {
+        if (pickGuarded())
+            return; // Task 20 — a stray Enter does not submit the row
         const trimmed = v.trim();
         if (trimmed.length === 0) {
             if (!isForm)
@@ -483,11 +606,15 @@ export function AskCard({ request, width, showDescriptions, maxRows, onResolveV1
                 setReviewCursor(1);
                 return;
             }
-            if (input === '1' || (key.return && reviewCursor === 0)) {
+            // R16 — "1\r" / "2\n" in one chunk act like the bare digit.
+            const reviewDigit = digitKey(input);
+            if (pickGuarded() && (reviewDigit > 0 || key.return))
+                return; // Task 20
+            if (reviewDigit === 1 || (key.return && reviewCursor === 0)) {
                 submit(answers);
                 return;
             }
-            if (input === '2' || (key.return && reviewCursor === 1)) {
+            if (reviewDigit === 2 || (key.return && reviewCursor === 1)) {
                 cancel();
                 return;
             }
@@ -510,11 +637,14 @@ export function AskCard({ request, width, showDescriptions, maxRows, onResolveV1
             return;
         // Ctrl+Y / Meta+Y are not "y": only a bare key answers.
         const alias = key.ctrl || key.meta ? undefined : q.keys?.[input.toLowerCase()];
+        // Task 20 — Enter, digit picks and alias keys wait out the screen's window.
+        if (pickGuarded() && (alias !== undefined || digitKey(input) > 0 || key.return))
+            return;
         if (alias !== undefined && q.options.some((o) => o.value === alias)) {
             answerAndAdvance(focus, alias);
             return;
         }
-        const digit = /^[1-9]$/.test(input) ? Number(input) - 1 : -1;
+        const digit = digitKey(input) - 1;
         if (digit >= 0) {
             if (digit >= rowCount)
                 return;
@@ -545,6 +675,19 @@ export function AskCard({ request, width, showDescriptions, maxRows, onResolveV1
     const frameHeader = isForm ? request.frameHeader : undefined;
     // The same fold as estimateAskCardRows, so the drawn card is the estimated card.
     const layout = askCardLayout(request, showDescriptions, width, maxRows);
+    // 1.1.53 Task 14 — a question the card has to cut is printed whole, once,
+    // to the scrollback just above the card (where answered questions land),
+    // when it becomes the current question. A question that fits prints
+    // nothing. Classic / headless have no trace emitter: emitTrace is a no-op.
+    const block = q ? questionBlock(q, inner, questionCap(layout, focus)) : undefined;
+    const questionCut = block?.cut === true;
+    const showPointer = questionCut && questionCap(layout, focus) >= POINTER_MIN_ROWS;
+    useEffect(() => {
+        if (!q || !questionCut || printedFull.current.has(focus))
+            return;
+        printedFull.current.add(focus);
+        emitTrace(fullQuestionText(q.question, inner));
+    }, [q, questionCut, focus, inner]);
     const chipRow = n > 1 ? (_jsx(Box, { marginBottom: 1, children: _jsx(Text, { wrap: "truncate-end", children: fitSegments([
                 ...questions.map((qq, i) => ({
                     text: `${i > 0 ? '  ' : ''}${answers[i] !== null ? g.answered : g.pending} ${fit(qq.header, inner)}`,
@@ -574,8 +717,7 @@ export function AskCard({ request, width, showDescriptions, maxRows, onResolveV1
     const hintKind = q.freeText ? 'text' : onOther ? 'other' : q.multiSelect ? 'multi' : 'pick';
     // The multi-select suffix is the card's only multi cue: it stays whole and
     // the question is cut instead (the same rule as "(recommended)" below).
-    const suffix = q.multiSelect && inner - MULTI_SUFFIX.length >= 1 ? MULTI_SUFFIX : '';
-    const questionLines = questionRows(q.question, suffix, inner, layout.questionRows);
+    const questionLines = block.lines;
     const hint = askCardHint(hintKind, {
         multiQuestion: n > 1,
         count: rowCount,
@@ -583,7 +725,7 @@ export function AskCard({ request, width, showDescriptions, maxRows, onResolveV1
     });
     // Task 4 — the same frame as ConfirmCard: QUESTION tag (plus the optional
     // header) on the top edge, the hint right under the options.
-    return (_jsxs(CardFrame, { tag: "QUESTION", header: frameHeader, width: width, children: [chipRow, questionLines.map((line, i) => (_jsx(Text, { bold: true, wrap: "truncate-end", children: line }, `q-${i}`))), contextLines.map((line, i) => (_jsx(Text, { ...dim, wrap: "truncate-end", children: line }, `ctx-${i}`))), q.freeText ? (_jsxs(Box, { marginTop: 1, children: [_jsx(Text, { ...roleProps('sap'), children: `${g.cursor} ` }), _jsx(TextInput, { value: drafts[focus], onChange: setDraft, onSubmit: onTextSubmit, maxDisplayWidth: inputWidth })] })) : (_jsxs(Box, { flexDirection: "column", marginTop: 1, children: [winStart > 0 && _jsx(Text, { ...dim, wrap: "truncate-end", children: `  ↑ ${winStart} more` }), visible.map((o, vi) => {
+    return (_jsxs(CardFrame, { tag: "QUESTION", header: frameHeader, width: width, children: [chipRow, questionLines.map((line, i) => (_jsx(Text, { bold: true, wrap: "truncate-end", children: line }, `q-${i}`))), showPointer ? _jsx(Text, { ...dim, wrap: "truncate-end", children: fit(FULL_QUESTION_NOTE, inner) }) : null, contextLines.map((line, i) => (_jsx(Text, { ...dim, wrap: "truncate-end", children: line }, `ctx-${i}`))), q.freeText ? (_jsxs(Box, { marginTop: 1, children: [_jsx(Text, { ...roleProps('sap'), children: `${g.cursor} ` }), _jsx(TextInput, { value: drafts[focus], onChange: setDraft, onSubmit: onTextSubmit, maxDisplayWidth: inputWidth })] })) : (_jsxs(Box, { flexDirection: "column", marginTop: 1, children: [winStart > 0 && _jsx(Text, { ...dim, wrap: "truncate-end", children: `  ↑ ${winStart} more` }), visible.map((o, vi) => {
                         const idx = winStart + vi;
                         const cur = !onOther && idx === cursor;
                         const tick = q.multiSelect ? `${ticks[focus].has(o.value) ? g.tickOn : g.tickOff} ` : '';

package/dist/ui/widgets/bar-chart.js

@@ -1,5 +1,6 @@
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { Box, Text } from 'ink';
+import { cardBorderStyle } from '../ascii-chars.js';
 import { roleProps } from '../role-props.js';
 // The model names a colour; the bar takes the matching theme role, so it follows
 // NO_COLOR and the light palette. White / unknown → the terminal's own text colour.
@@ -13,7 +14,7 @@ function barProps(c) {
 export function BarChart({ data }) {
     const max = Math.max(...data.map((d) => d.value), 1);
     const barWidth = 30;
-    return (_jsxs(Box, { flexDirection: "column", borderStyle: "round", borderColor: roleProps('frame').color, paddingX: 1, children: [_jsx(Text, { ...roleProps('sap', { bold: true }), children: "Distribution" }), data.map((item) => {
+    return (_jsxs(Box, { flexDirection: "column", borderStyle: cardBorderStyle(), borderColor: roleProps('frame').color, paddingX: 1, children: [_jsx(Text, { ...roleProps('sap', { bold: true }), children: "Distribution" }), data.map((item) => {
                 const filled = Math.round((item.value / max) * barWidth);
                 const bar = '\u2588'.repeat(filled) + '\u2591'.repeat(barWidth - filled);
                 return (_jsxs(Box, { children: [_jsxs(Text, { children: [item.label.padEnd(28), " "] }), _jsx(Text, { ...barProps(item.color), children: bar }), _jsxs(Text, { children: [" ", item.value] })] }, item.label));

package/dist/ui/widgets/confirm-card.js

@@ -192,7 +192,7 @@ export function CardFrame({ tag, header, footnote, width, children }) {
     const bottom = edgeSegments([box.bl, box.br], box.h, footnote, width);
     return (_jsxs(Box, { flexDirection: "column", flexShrink: 0, width: width, children: [_jsxs(Text, { wrap: "truncate-end", children: [_jsx(Text, { ...frame, children: top.lead }), _jsx(Text, { ...roleProps(TAG_ROLE[tag], { bold: true }), children: tagShown }), headerShown ? _jsx(Text, { ...roleProps('dim'), children: headerShown }) : null, _jsx(Text, { ...frame, children: top.tail })] }), _jsx(Box, { flexDirection: "column", width: width, paddingX: 1, borderStyle: inkBorder(box), borderTop: false, borderBottom: false, borderColor: frame.color, children: children }), _jsxs(Text, { wrap: "truncate-end", children: [_jsx(Text, { ...frame, children: bottom.lead }), _jsx(Text, { ...roleProps('dim'), children: bottom.label }), _jsx(Text, { ...frame, children: bottom.tail })] })] }));
 }
-export function ConfirmCard({ request, width, maxRows, onResolve, onInterruptTurn }) {
+export function ConfirmCard({ request, width, maxRows, onResolve, onInterruptTurn, onEscape }) {
     // Re-assert raw stdin on mount: an earlier inquirer prompt can leave Ink's
     // raw-mode bookkeeping stale, and the card would then get no keys.
     useReclaimRawStdin();
@@ -222,6 +222,7 @@ export function ConfirmCard({ request, width, maxRows, onResolve, onInterruptTur
         if (Date.now() - mountedAt.current < CONFIRM_INPUT_GUARD_MS)
             return;
         if (key.escape) {
+            onEscape?.();
             finish({ value: null, cancelled: true });
             return;
         }

package/dist/ui/widgets/dep-graph.js

@@ -1,8 +1,9 @@
 import { jsxs as _jsxs, jsx as _jsx } from "react/jsx-runtime";
 import { Box, Text } from 'ink';
+import { cardBorderStyle } from '../ascii-chars.js';
 import { roleProps } from '../role-props.js';
 export function DepGraph({ data }) {
-    return (_jsxs(Box, { flexDirection: "column", borderStyle: "round", borderColor: roleProps('frame').color, paddingX: 1, children: [_jsxs(Text, { ...roleProps('sap', { bold: true }), children: ["Dependencies \u2014 ", data.root] }), data.children.map((c, i) => {
+    return (_jsxs(Box, { flexDirection: "column", borderStyle: cardBorderStyle(), borderColor: roleProps('frame').color, paddingX: 1, children: [_jsxs(Text, { ...roleProps('sap', { bold: true }), children: ["Dependencies \u2014 ", data.root] }), data.children.map((c, i) => {
                 const isLast = i === data.children.length - 1;
                 const prefix = isLast ? '└─' : '├─';
                 return (_jsxs(Box, { children: [_jsxs(Text, { ...roleProps('dim'), children: [prefix, " "] }), _jsx(Text, { ...roleProps('warn'), children: c.name }), _jsxs(Text, { ...roleProps('dim'), children: [" (", c.type, ") "] }), _jsx(Text, { dimColor: true, children: c.relation })] }, c.name));

package/dist/ui/widgets/diff-viewer.js

@@ -1,9 +1,10 @@
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { Box, Text } from 'ink';
+import { cardBorderStyle } from '../ascii-chars.js';
 import { roleProps } from '../role-props.js';
 export function DiffViewer({ data }) {
     const lines = data.unified.split('\n');
-    return (_jsxs(Box, { flexDirection: "column", borderStyle: "round", borderColor: roleProps('frame').color, paddingX: 1, children: [_jsx(Text, { ...roleProps('dim', { bold: true }), children: "Diff" }), lines.map((line, idx) => {
+    return (_jsxs(Box, { flexDirection: "column", borderStyle: cardBorderStyle(), borderColor: roleProps('frame').color, paddingX: 1, children: [_jsx(Text, { ...roleProps('dim', { bold: true }), children: "Diff" }), lines.map((line, idx) => {
                 const role = line.startsWith('+') ? 'ok' :
                     line.startsWith('-') ? 'bad' :
                         line.startsWith('@@') ? 'sap' : 'dim';

package/dist/ui/widgets/question-card.js

@@ -1,5 +1,6 @@
 import { jsxs as _jsxs, jsx as _jsx } from "react/jsx-runtime";
 import { Box, Text } from 'ink';
+import { cardBorderStyle } from '../ascii-chars.js';
 import { roleProps } from '../role-props.js';
 /**
  * Ink-mode rendering of cspeach-widget:question. Uses a double-border box
@@ -8,5 +9,5 @@ import { roleProps } from '../role-props.js';
  * and types their answer as the next REPL prompt.
  */
 export function QuestionCard({ data }) {
-    return (_jsxs(Box, { borderStyle: "double", borderColor: roleProps('sap').color, flexDirection: "column", padding: 1, marginTop: 1, children: [_jsxs(Text, { bold: true, children: ["Question: ", data.id] }), _jsx(Box, { marginTop: 1, children: _jsx(Text, { children: data.question }) }), data.context ? (_jsx(Box, { marginTop: 1, children: _jsxs(Text, { dimColor: true, children: ["Context: ", data.context] }) })) : null, (data.kind === 'choice' || data.kind === 'multi') && data.choices ? (_jsxs(Box, { marginTop: 1, flexDirection: "column", children: [_jsx(Text, { bold: true, children: data.kind === 'choice' ? 'Pick one:' : 'Pick any (comma-separate):' }), data.choices.map((c, i) => (_jsxs(Text, { children: ["[", i + 1, "] ", c.label] }, c.value)))] })) : null] }));
+    return (_jsxs(Box, { borderStyle: cardBorderStyle('double'), borderColor: roleProps('sap').color, flexDirection: "column", padding: 1, marginTop: 1, children: [_jsxs(Text, { bold: true, children: ["Question: ", data.id] }), _jsx(Box, { marginTop: 1, children: _jsx(Text, { children: data.question }) }), data.context ? (_jsx(Box, { marginTop: 1, children: _jsxs(Text, { dimColor: true, children: ["Context: ", data.context] }) })) : null, (data.kind === 'choice' || data.kind === 'multi') && data.choices ? (_jsxs(Box, { marginTop: 1, flexDirection: "column", children: [_jsx(Text, { bold: true, children: data.kind === 'choice' ? 'Pick one:' : 'Pick any (comma-separate):' }), data.choices.map((c, i) => (_jsxs(Text, { children: ["[", i + 1, "] ", c.label] }, c.value)))] })) : null] }));
 }

package/dist/ui/widgets/shortcuts-sheet.js

@@ -1,7 +1,7 @@
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { Box, Text, useInput } from 'ink';
 import { roleProps } from '../role-props.js';
-import { shouldUseUnicodeBoxChars } from '../../renderer/tty.js';
+import { cardBorderStyle } from '../ascii-chars.js';
 import { truncateToWidth } from '../../renderer/ui-width.js';
 /** Spec §8 table, in order. */
 export const SHORTCUT_ROWS = [
@@ -21,5 +21,5 @@ export function ShortcutsSheet({ width, onClose }) {
     useInput((input, key) => { if (key.escape || (key.ctrl && input === 'c'))
         onClose(); });
     const inner = Math.max(8, width - 4);
-    return (_jsxs(Box, { flexDirection: "column", borderStyle: shouldUseUnicodeBoxChars() ? 'round' : 'classic', borderColor: roleProps('frame').color, paddingX: 1, width: width, children: [_jsx(Text, { bold: true, children: "Shortcuts" }), SHORTCUT_ROWS.map(([key, action]) => (_jsxs(Text, { wrap: "truncate-end", children: [_jsx(Text, { ...roleProps('sap'), children: key.padEnd(KEY_COL) }), truncateToWidth(action, Math.max(1, inner - KEY_COL))] }, key))), _jsx(Text, { ...roleProps('dim'), wrap: "truncate-end", children: truncateToWidth(SHORTCUTS_NOTE, inner) }), _jsx(Text, { ...roleProps('dim'), children: "Esc close" })] }));
+    return (_jsxs(Box, { flexDirection: "column", borderStyle: cardBorderStyle(), borderColor: roleProps('frame').color, paddingX: 1, width: width, children: [_jsx(Text, { bold: true, children: "Shortcuts" }), SHORTCUT_ROWS.map(([key, action]) => (_jsxs(Text, { wrap: "truncate-end", children: [_jsx(Text, { ...roleProps('sap'), children: key.padEnd(KEY_COL) }), truncateToWidth(action, Math.max(1, inner - KEY_COL))] }, key))), _jsx(Text, { ...roleProps('dim'), wrap: "truncate-end", children: truncateToWidth(SHORTCUTS_NOTE, inner) }), _jsx(Text, { ...roleProps('dim'), children: "Esc close" })] }));
 }

package/dist/ui/widgets/stack-frames.js

@@ -1,6 +1,7 @@
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { Box, Text } from 'ink';
+import { cardBorderStyle } from '../ascii-chars.js';
 import { roleProps } from '../role-props.js';
 export function StackFrames({ data }) {
-    return (_jsxs(Box, { flexDirection: "column", borderStyle: "round", borderColor: roleProps('bad').color, paddingX: 1, children: [_jsx(Text, { ...roleProps('bad', { bold: true }), children: "Stack Frames" }), data.map((f) => (_jsxs(Box, { flexDirection: "column", marginTop: f.frame === 0 ? 0 : 1, children: [_jsx(Box, { children: _jsxs(Text, { ...roleProps(f.type === 'error' ? 'bad' : 'dim'), children: ['#' + String(f.frame).padStart(2, '0'), " ", f.program, ":", f.line] }) }), _jsx(Box, { marginLeft: 5, children: _jsx(Text, { ...(f.type === 'error' ? {} : roleProps('dim')), dimColor: f.type === 'context', children: f.code }) })] }, f.frame)))] }));
+    return (_jsxs(Box, { flexDirection: "column", borderStyle: cardBorderStyle(), borderColor: roleProps('bad').color, paddingX: 1, children: [_jsx(Text, { ...roleProps('bad', { bold: true }), children: "Stack Frames" }), data.map((f) => (_jsxs(Box, { flexDirection: "column", marginTop: f.frame === 0 ? 0 : 1, children: [_jsx(Box, { children: _jsxs(Text, { ...roleProps(f.type === 'error' ? 'bad' : 'dim'), children: ['#' + String(f.frame).padStart(2, '0'), " ", f.program, ":", f.line] }) }), _jsx(Box, { marginLeft: 5, children: _jsx(Text, { ...(f.type === 'error' ? {} : roleProps('dim')), dimColor: f.type === 'context', children: f.code }) })] }, f.frame)))] }));
 }

package/dist/util/connect-retry.js

@@ -0,0 +1,65 @@
+/**
+ * 2026-10-23 read retry — one more try, after a short pause, when a GET read
+ * could not connect: a connect timeout, a DNS error, or a reset before any
+ * response. A brief DNS stall on a laptop (seen in the eval, likely Docker
+ * changing adapters) clears on the second try.
+ *
+ * GET only. Never wrap /v1/messages or any POST/write with this: a retried
+ * write could run twice. It does not touch the global dispatcher, agents or
+ * DNS order, so NODE_USE_ENV_PROXY keeps working on corporate VDIs.
+ *
+ * A copy of this rule lives in cspeach-proxy/src/skills/manifest.ts
+ * (fetchManifestResponse) — separate package, no shared code. Change both.
+ */
+/** The error codes worth one more try. Read from `.code` only, never the message. */
+const RETRYABLE_CODES = new Set(['UND_ERR_CONNECT_TIMEOUT', 'ENOTFOUND', 'EAI_AGAIN', 'ECONNRESET']);
+export const CONNECT_RETRY_DELAY_MS = 500;
+/** True when `e` (or its cause chain — undici wraps it as TypeError('fetch failed', { cause })) carries a retryable code. */
+export function isConnectRetryable(e) {
+    let cur = e;
+    for (let depth = 0; depth < 4 && cur && typeof cur === 'object'; depth++) {
+        const code = cur.code;
+        if (typeof code === 'string' && RETRYABLE_CODES.has(code))
+            return true;
+        cur = cur.cause;
+    }
+    return false;
+}
+const defaultSleep = (ms) => new Promise((resolve) => setTimeout(resolve, ms));
+/**
+ * fetch with one retry on a connect failure, for GET only. `makeInit` is
+ * called once per attempt so each try gets a fresh timeout signal; when that
+ * per-try timer fires (TimeoutError) it counts as a connect failure. A caller
+ * abort never does. An HTTP
+ * error status is a response, returned as is. After the one retry fails, the
+ * ORIGINAL error is thrown, so callers' messages stay the same.
+ */
+export async function fetchWithConnectRetry(url, makeInit, opts = {}) {
+    const doFetch = opts.fetchImpl ?? fetch;
+    const first = makeInit();
+    try {
+        return await doFetch(url, first);
+    }
+    catch (e) {
+        const method = (first.method ?? 'GET').toUpperCase();
+        if (method !== 'GET')
+            throw e;
+        const reasonName = first.signal?.aborted ? first.signal.reason?.name : undefined;
+        // Our own per-try timer fired (a stalled connect/DNS, in effect a connect
+        // timeout; it races undici's own 10 s connect timeout). makeInit hands the
+        // retry a fresh timer.
+        const ownTimerFired = reasonName === 'TimeoutError' && e?.name === 'TimeoutError';
+        if (!isConnectRetryable(e) && !ownTimerFired)
+            throw e;
+        // The caller gave up (user cancel / session end): never a second try.
+        if (first.signal?.aborted && reasonName !== 'TimeoutError')
+            throw e;
+        await (opts.sleep ?? defaultSleep)(CONNECT_RETRY_DELAY_MS);
+        try {
+            return await doFetch(url, makeInit());
+        }
+        catch {
+            throw e;
+        }
+    }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cspeach/cli",
-  "version": "1.1.51",
+  "version": "1.1.53",
   "description": "CSPeach — AI-assisted ABAP development CLI. Direct SAP access, signed skills, safety gates.",
   "keywords": [
     "abap",
@@ -54,7 +54,7 @@
     "@anthropic-ai/sdk": "^0.90.0",
     "@azure/msal-node": "6.0.1",
     "@cspeach/register-core": "0.4.2",
-    "@cspeach/sap-client": "0.1.30",
+    "@cspeach/sap-client": "0.1.31",
     "@iarna/toml": "^2.2.5",
     "@inkjs/ui": "^2.0.0",
     "@inquirer/prompts": "^7.0.0",