koneck 2.128.71 → 2.128.73

package/dist/ink-chat.js

@@ -1669,10 +1669,10 @@ function renderCodeLine(text, kind, width, lang) {
  * through the list, Alt+PgUp/PgDn and the wheel scroll the file, Alt+T opens the fold.
  */
 export function DiffPanel({ files, width, height, activity, activityRows = 0, scroll = 0, selected = 0, showGenerated = false }) {
-    // A rule down the left instead of a box round it: two rows more for the diff, and the panel
-    // reads as a pane beside the conversation, as in the panel it was modelled on.
+    // Both side borders and the left padding occupy three columns. Reserve a row for the
+    // bottom border so the last visible diff line and the scroll count agree.
     const inner = Math.max(20, width - 3);
-    const room = Math.max(4, height - activityRows);
+    const room = Math.max(4, height - activityRows - 1);
     const listLen = listLength(files, showGenerated);
     const listRows = Math.max(1, Math.min(listLen, 8, Math.max(2, Math.floor(room * 0.3))));
     const view = fileListView(files, { selected, rows: listRows, showGenerated });
@@ -1694,7 +1694,7 @@ export function DiffPanel({ files, width, height, activity, activityRows = 0, sc
      * element per row is not. So the rows are described in full — the scroll needs the count — and
      * drawn only for the window.
      */
-    const fixedRows = 2 + list.length + hints.length + (d ? 4 : 0) + (activity ? activityRows : 0);
+    const fixedRows = 3 + list.length + hints.length + (d ? 4 : 0) + (activity ? activityRows : 0);
     const bodyHeight = Math.max(1, height - fixedRows);
     const numbers = d ? d.hunks.flatMap(h => h.lines.map(l => l.after ?? l.before ?? 0)) : [];
     const gutter = Math.max(3, String(Math.max(0, ...numbers)).length);
@@ -1719,7 +1719,7 @@ export function DiffPanel({ files, width, height, activity, activityRows = 0, sc
     const scrolled = maxScroll > 0
         ? `${from > 0 ? `↑${from}` : ''}${from < maxScroll ? ` ↓${maxScroll - from}` : ''}`.trim()
         : '';
-    return (_jsxs(Box, { flexDirection: "column", width: width, height: height, borderStyle: "single", borderColor: DIM, borderTop: false, borderRight: false, borderBottom: false, paddingLeft: 1, flexShrink: 0, overflowY: "hidden", children: [_jsxs(Box, { justifyContent: "space-between", width: inner, children: [_jsxs(Box, { children: [_jsx(Text, { color: INK, bold: true, children: files.length === 0 ? 'no changes yet' : `${files.length} file${files.length === 1 ? '' : 's'} changed` }), view.added > 0 && _jsxs(Text, { color: DIFF_ADD, children: [" +", view.added] }), view.removed > 0 && _jsxs(Text, { color: DIFF_DEL, children: [" -", view.removed] })] }), _jsx(Text, { color: DIM, children: "/diff off" })] }), activity && _jsxs(_Fragment, { children: [_jsx(Text, { color: DIM, children: '─'.repeat(inner) }), _jsx(Box, { flexDirection: "column", height: Math.max(1, activityRows - 1), overflowY: "hidden", children: activity })] }), _jsx(Text, { children: " " }), list, hints, d && _jsxs(_Fragment, { children: [_jsx(Text, { children: " " }), _jsx(Text, { color: DIM, children: '─'.repeat(inner) }), _jsxs(Box, { justifyContent: "space-between", width: inner, children: [_jsx(Text, { color: INK, bold: true, wrap: "truncate-start", children: d.path }), _jsxs(Text, { color: DIM, children: [d.created && d.removed === 0 ? ' new' : d.deleted ? ' deleted' : '', scrolled ? ` ${scrolled}` : ''] })] }), _jsx(Text, { color: DIM, children: '─'.repeat(inner) })] }), _jsx(Box, { flexDirection: "column", flexGrow: 1, flexShrink: 1, overflowY: "hidden", children: body })] }));
+    return (_jsxs(Box, { flexDirection: "column", width: width, height: height, borderStyle: "single", borderColor: DIM, borderTop: false, paddingLeft: 1, flexShrink: 0, overflowY: "hidden", children: [_jsxs(Box, { justifyContent: "space-between", width: inner, children: [_jsxs(Box, { children: [_jsx(Text, { color: INK, bold: true, children: files.length === 0 ? 'no changes yet' : `${files.length} file${files.length === 1 ? '' : 's'} changed` }), view.added > 0 && _jsxs(Text, { color: DIFF_ADD, children: [" +", view.added] }), view.removed > 0 && _jsxs(Text, { color: DIFF_DEL, children: [" -", view.removed] })] }), _jsx(Text, { color: DIM, children: "/diff off" })] }), activity && _jsxs(_Fragment, { children: [_jsx(Text, { color: DIM, children: '─'.repeat(inner) }), _jsx(Box, { flexDirection: "column", height: Math.max(1, activityRows - 1), overflowY: "hidden", children: activity })] }), _jsx(Text, { children: " " }), list, hints, d && _jsxs(_Fragment, { children: [_jsx(Text, { children: " " }), _jsx(Text, { color: DIM, children: '─'.repeat(inner) }), _jsxs(Box, { justifyContent: "space-between", width: inner, children: [_jsx(Text, { color: INK, bold: true, wrap: "truncate-start", children: d.path }), _jsxs(Text, { color: DIM, children: [d.created && d.removed === 0 ? ' new' : d.deleted ? ' deleted' : '', scrolled ? ` ${scrolled}` : ''] })] }), _jsx(Text, { color: DIM, children: '─'.repeat(inner) })] }), _jsx(Box, { flexDirection: "column", flexGrow: 1, flexShrink: 1, overflowY: "hidden", children: body })] }));
 }
 /**
  * Exported for one reason: so a test can mount it.
@@ -2539,6 +2539,16 @@ export function App({ config: initialConfig, clearFrame, synchronizedFrames = fa
      * been chosen either way (/mouse, or `mouse` in the config), which is then what happens.
      */
     const [sidePanelShown, setSidePanelShown] = useState(false);
+    /*
+     * The bell, for when KONECK needs the person: a long turn finished, or a question is waiting.
+     * A terminal in another tab or window marks it — Windows Terminal flashes the tab and the
+     * taskbar — so a finished hour-long run is noticed without watching for it. `bell: false` stops it.
+     */
+    const ring = useCallback(() => {
+        if (cfg.bell === false || !process.stdout.isTTY)
+            return;
+        process.stdout.write('\u0007');
+    }, [cfg.bell]);
     const mouseChosenRef = useRef(cfg.mouse !== undefined);
     const wheelTracked = mouse || (sidePanelShown && !mouseChosenRef.current);
     /*
@@ -2931,6 +2941,10 @@ export function App({ config: initialConfig, clearFrame, synchronizedFrames = fa
      * all — only in the browser, and there as plain text.
      */
     const [userQ, setUserQ] = useState(null);
+    // A question waiting on the person rings once, as it appears: see `ring`.
+    const waitingOnPerson = !!ask || !!planAsk || !!userQ;
+    useEffect(() => { if (waitingOnPerson)
+        ring(); }, [waitingOnPerson, ring]);
     /** The picker's rows as last drawn, for a click to be matched against. */
     const pickerTargetsRef = useRef([]);
     const [updatePane, setUpdatePane] = useState(null);
@@ -7035,6 +7049,9 @@ export function App({ config: initialConfig, clearFrame, synchronizedFrames = fa
                     // whichever of those three happened last, which for anyone who closes the terminal
                     // normally was an old snapshot — the reported "always resumes the old one".
                     void doSave().catch(() => { });
+                    // A turn worth walking away from is worth being called back for.
+                    if (elapsed >= 30_000)
+                        ring();
                     // A run that stopped short says so, rather than letting the footer imply it finished.
                     if (incompleteRef.current) {
                         addSystem(incompleteRef.current);
@@ -7303,7 +7320,7 @@ export function App({ config: initialConfig, clearFrame, synchronizedFrames = fa
     // content, so the right edge stepped in and out; and a box inside a box left
     // markdown tables needlessly narrow. A marker plus the content reads better and
     // cannot misalign, which is how Claude Code presents its own answers.
-    _jsxs(Box, { flexDirection: "column", marginTop: row.continued ? 0 : 1, children: [_jsxs(Box, { gap: 1, children: [_jsx(Text, { color: CYAN, bold: true, children: row.continued ? '  ' : G.brand }), _jsx(Box, { flexDirection: "column", children: _jsx(Markdown, { text: row.text ?? '', width: barWidth - 3 }) })] }), row.tokens != null && (_jsxs(Text, { color: MUTED, children: [_jsx(Text, { color: GREEN, children: "* " }), row.done ? `${row.done} in ${fmtElapsed(row.elapsed ?? 0)}` : fmtElapsed(row.elapsed ?? 0), '  |  ', row.tokens, " tokens", '  |  ', cfg.provider, " ", modelShort] }))] }, index)));
+    _jsxs(Box, { flexDirection: "column", marginTop: row.continued ? 0 : 1, children: [_jsxs(Box, { gap: 1, children: [_jsx(Text, { color: CYAN, bold: true, children: row.continued ? '  ' : G.brand }), _jsx(Box, { flexDirection: "column", children: _jsx(Markdown, { text: row.text ?? '', width: barWidth - 3 }) })] }), row.tokens != null && (_jsxs(Text, { color: MUTED, children: [_jsxs(Text, { color: GREEN, bold: true, children: [G.ok, " Done"] }), ` in ${fmtElapsed(row.elapsed ?? 0)}`, '  |  ', row.tokens, " tokens", '  |  ', cfg.provider, " ", modelShort] }))] }, index)));
     /**
      * A row that has stopped changing, and so never needs rendering twice.
      *
@@ -7808,7 +7825,7 @@ export function App({ config: initialConfig, clearFrame, synchronizedFrames = fa
                             })(), keyPrompt && (_jsxs(Box, { flexDirection: "column", borderStyle: "round", borderColor: AMBER, paddingX: 1, marginTop: 1, width: barWidth, children: [_jsxs(Text, { color: AMBER, bold: true, children: ["API key for ", keyPrompt.provider] }), _jsx(Text, { color: MUTED, children: "Paste it and press enter, or ctrl+v to read the clipboard. Held in memory for this session only; esc to cancel." }), keyPrompt.note && _jsx(Text, { color: CYAN, children: keyPrompt.note }), _jsxs(Box, { marginTop: 1, children: [_jsxs(Text, { color: MUTED, children: [keyPrompt.env, ": "] }), _jsx(Text, { color: INK, children: '*'.repeat(Math.min(keyPrompt.value.length, 48)) }), _jsx(Cursor, {}), keyPrompt.value.length > 0 && (_jsxs(Text, { color: DIM, children: ["  ", keyPrompt.value.length, " characters"] }))] })] })), themePick && (() => {
                                 const w = Math.min(barWidth, 84);
                                 const labelWidth = Math.max(...THEMES.map(t => t.label.length));
-                                return (_jsxs(Box, { marginTop: 1, flexDirection: "column", borderStyle: "round", borderColor: CYAN, paddingX: 1, width: w, children: [_jsx(Text, { color: INK, bold: true, children: "Theme" }), _jsx(Text, { color: MUTED, children: "Sets KONECK's colours, not your terminal's background." }), _jsx(Text, { children: " " }), THEMES.map((t, i) => {
+                                return (_jsxs(Box, { marginTop: 1, flexShrink: 0, flexDirection: "column", borderStyle: "round", borderColor: CYAN, paddingX: 1, width: w, children: [_jsx(Text, { color: INK, bold: true, children: "Theme" }), _jsx(Text, { color: MUTED, children: "Sets KONECK's colours, not your terminal's background." }), _jsx(Text, { children: " " }), THEMES.map((t, i) => {
                                             const on = i === themePick.at;
                                             return (_jsxs(Text, { wrap: "truncate", ...(on ? { backgroundColor: CYAN, color: ON_ACCENT } : { color: INK }), children: [on ? ' › ' : '   ', _jsx(Text, { bold: true, children: i + 1 }), '  ', t.label.padEnd(labelWidth), _jsx(Text, { ...(on ? {} : { color: DIM }), children: '   ' + t.note }), t.name === themePick.was ? '  (current)' : '', ' '] }, t.name));
                                         }), _jsx(Text, { children: " " }), _jsx(ThemePreview, { theme: THEMES[themePick.at].name, width: w - 4 }), _jsxs(Text, { color: DIM, children: ["\u2191\u2193 or 1\u2013", THEMES.length, " to try \u00B7 enter to keep \u00B7 esc to go back"] })] }));
@@ -7820,7 +7837,15 @@ export function App({ config: initialConfig, clearFrame, synchronizedFrames = fa
                                     '',
                                     'Press a number, or up/down and enter, or click one — or type your own answer. Esc hides this.',
                                 ];
-                                return (_jsx(Box, { marginTop: 1, children: _jsx(Panel, { width: Math.min(barWidth, 84), color: AMBER, title: "The model is asking", children: lines }) }));
+                                /*
+                                 * Never shrunk: the transcript above gives way instead.
+                                 *
+                                 * Reported with a screenshot: on the alternate screen, after a long reply, only the last
+                                 * line of this panel showed — "minor bug fixes" in an amber box — and the question and the
+                                 * other options were clipped away, so it looked like a stray message rather than a
+                                 * question. The transcript is the box meant to give ground; this one must not.
+                                 */
+                                return (_jsx(Box, { marginTop: 1, flexShrink: 0, children: _jsx(Panel, { width: Math.min(barWidth, 84), color: AMBER, title: "The model is asking", children: lines }) }));
                             })(), planAsk && (() => {
                                 /*
                                  * The question, straight after the plan rather than instead of it.
@@ -7837,7 +7862,7 @@ export function App({ config: initialConfig, clearFrame, synchronizedFrames = fa
                                     'Up/down and enter, or press the number. Esc leaves it in plan mode.',
                                     ...(planAsk.file ? [`Saved to \`${planAsk.file}\``] : ['This plan could not be written to disk.']),
                                 ];
-                                return (_jsx(Box, { marginTop: 1, children: _jsx(Panel, { width: Math.min(barWidth, 84), color: CYAN, title: "Plan ready", children: lines }) }));
+                                return (_jsx(Box, { marginTop: 1, flexShrink: 0, children: _jsx(Panel, { width: Math.min(barWidth, 84), color: CYAN, title: "Plan ready", children: lines }) }));
                             })(), ask && (() => {
                                 let detail = '';
                                 let isCommand = false;
@@ -7955,9 +7980,6 @@ export function App({ config: initialConfig, clearFrame, synchronizedFrames = fa
                 const notices = [];
                 if (agentState !== 'ready')
                     notices.push({ text: statusLabel.toLowerCase(), color: statusColor });
-                const planNote = plan ? planSummary(plan) : '';
-                if (planNote)
-                    notices.push({ text: `plan ${planNote}`, color: CYAN });
                 if (effort !== 'medium')
                     notices.push({ text: `effort ${effort}`, color: AMBER });
                 if (autoUpdated)
@@ -7970,6 +7992,15 @@ export function App({ config: initialConfig, clearFrame, synchronizedFrames = fa
                     notices.push({ text: `${pasteCount} paste${pasteCount === 1 ? '' : 's'} held, sent in full`,
                         color: AMBER });
                 }
+                // The plan last, given only the room the rest leaves: its step is the one notice of no
+                // fixed length, and the screen edge used to cut it mid-word ("…critical journeys; r").
+                if (plan) {
+                    const used = displayWidth(ml.title) + 2 + displayWidth(ml.hint)
+                        + notices.reduce((w, n) => w + 5 + displayWidth(n.text), 0) + 5 + displayWidth('plan ');
+                    const planNote = planSummary(plan, Math.max(0, uiWidth - 9 - used));
+                    if (planNote)
+                        notices.push({ text: `plan ${planNote}`, color: CYAN });
+                }
                 return (_jsx(Box, { marginTop: 1, paddingX: 2, width: uiWidth - 5, flexShrink: 0, children: _jsxs(Text, { wrap: "truncate", children: [_jsx(Text, { color: TONE[ml.tone], bold: true, children: ml.title }), _jsxs(Text, { color: MUTED, children: ['  ', ml.hint] }), notices.map((n, i) => (_jsxs(Text, { children: [_jsx(Text, { color: DIM, children: '  ·  ' }), _jsx(Text, { color: n.color, children: n.text })] }, i)))] }, `m${notices.length}`) }));
             })(), _jsx(Box, { borderStyle: "single", borderLeft: false, borderRight: false, borderColor: DIM, paddingX: 2, width: uiWidth - 5, flexShrink: 0, children: _jsxs(Text, { wrap: "wrap", children: [_jsxs(Text, { color: busy ? DIM : AMBER, bold: true, children: [G.caret, " "] }), (() => {
                             const sel = selectionRange(selAnchor, caret, draft.length);