koneck 2.128.27 → 2.128.29

package/dist/ink-chat.js

@@ -8,7 +8,7 @@ import { pacedInterval, startingInterval, floorFor, shouldDrawFrame } from './fr
 import { estimateCost, formatCost, rateText } from './pricing.js';
 import { generateSessionId, saveSession, listSessions, loadSession, relativeAge, renameSession, forkSession, archiveSession, findSession, } from './session.js';
 import { loadMemory } from './memory.js';
-import { loadKoneckConfig, saveKoneckConfig, setConfigKey, configKeyDescriptions, CONFIG_FILE, CONFIG_SCHEMA, stepValue, displayValue } from './config-store.js';
+import { loadKoneckConfig, saveKoneckConfig, setConfigKey, configKeyDescriptions, CONFIG_FILE, CONFIG_SCHEMA, stepValue, displayValue, settingsWindow } from './config-store.js';
 import { PROVIDERS, resolveProvider, getApiKey, addUserProvider, removeUserProvider, userProvidersPath, keyEnvFor, BUILT_INS as BUILT_IN_PROVIDERS, servesLocally, servesOwnWeights, endpointForProvider } from './providers.js';
 import { normaliseEndpoint, withEndpoint, savedEndpoints, shortSource, resolveModelChoice, withModel } from './endpoints.js';
 import { loadProjectConfig, loadGlobalConfig, saveGlobalConfig, withoutStoredApiKey } from './config.js';
@@ -715,6 +715,14 @@ const PANEL_DIFF_LINES = 400;
 const USAGE_TABS = ['settings', 'session', 'context', 'models', 'limits'];
 const MIN_REPLY_WIDTH = 24; // below this a table cannot render legibly anyway
 const PICKER_MAX_ROWS = 16;
+/**
+ * Rows kept for everything around a picker in ordinary scrollback.
+ *
+ * The composer with its border, the status bar, the footer, the picker's own border and hint
+ * row, and the blanks between them. Found by measurement rather than by adding those up: at
+ * 80x24, nine and twelve both still wiped the terminal and fourteen did not.
+ */
+const DEFAULT_PICKER_RESERVE = 14;
 /**
  * Number of choices to keep visible in a palette.
  *
@@ -723,13 +731,24 @@ const PICKER_MAX_ROWS = 16;
  * the composer and the rest remains available through search and paging.
  */
 export function pickerRowLimit(kind, itemCount, terminalRows, reservedRows) {
-    // In ordinary scrollback this is not a fixed overlay: it can extend into the terminal's normal
-    // scroll region, so shrinking a command picker just because a pane has ten rows is needless
-    // friction. Only alternate-screen callers pass a reservation, because that is a fixed frame
-    // and overflow there makes Ink clear/repaint the entire screen.
-    const terminalBudget = reservedRows === undefined
-        ? PICKER_MAX_ROWS
-        : Math.max(1, (terminalRows && terminalRows > 0 ? terminalRows : 24) - reservedRows);
+    /*
+     * The window bounds this on every screen, and it used not to.
+     *
+     * The reasoning here was that ordinary scrollback is not a fixed frame, so a picker may run as
+     * long as it likes and let the terminal scroll — and that only the alternate screen needed a
+     * reservation, because overflow there makes Ink clear and repaint everything.
+     *
+     * Ink does not make that distinction. It compares the height of what it is about to draw
+     * against `stdout.rows` and wipes the terminal when it does not fit, whichever screen is in
+     * use, because it cannot erase lines that have scrolled away. The picker is part of the
+     * repainting region in both. Measured at 80x24: typing "/config" wiped the screen three times
+     * before the command had even been chosen, so every slash command flashed the whole terminal.
+     *
+     * The reservation differs — the alternate screen has a transcript above the picker to leave
+     * room for — but there is always one.
+     */
+    const reserved = reservedRows ?? DEFAULT_PICKER_RESERVE;
+    const terminalBudget = Math.max(1, (terminalRows && terminalRows > 0 ? terminalRows : 24) - reserved);
     // Keep kind in the signature: it documents that this is the common policy for *every* picker
     // and lets a future picker opt into a different policy without duplicating this calculation.
     void kind;
@@ -2389,6 +2408,14 @@ export function App({ config: initialConfig, clearFrame, synchronizedFrames = fa
     }, [effort]);
     const [showAnalytics, setShowAnalytics] = useState(false);
     const [updatePane, setUpdatePane] = useState(null);
+    /**
+     * The version installed in the background, if one was; see auto-update.ts.
+     *
+     * Only ever says "installed", never "updated": the code this session is running was read into
+     * memory when it started, and replacing the files on disk does not change it. The restart is
+     * the part that matters and so it is the part the line ends on.
+     */
+    const [autoUpdated, setAutoUpdated] = useState(null);
     const [updating, setUpdating] = useState('idle');
     const [updateLog, setUpdateLog] = useState('');
     // Settable, because resuming has to adopt the restored session's id. Without that, every save
@@ -2804,6 +2831,35 @@ export function App({ config: initialConfig, clearFrame, synchronizedFrames = fa
         timer = setTimeout(tick, interval);
         return () => clearTimeout(timer);
     }, [busy, animate, ask]);
+    /*
+     * Check for a new version and install it, quietly, in the background.
+     *
+     * Deliberately late and deliberately silent. It is network and a subprocess, neither of which
+     * should compete with the first turn, and there is nothing to report unless it succeeds — a
+     * registry that cannot be reached or an npm that cannot write to its global prefix is not the
+     * user's problem to hear about mid-task. See auto-update.ts for what it refuses to do.
+     */
+    useEffect(() => {
+        if (cfg.autoUpdate === false)
+            return;
+        let live = true;
+        const timer = setTimeout(() => {
+            void (async () => {
+                try {
+                    const { autoUpdate } = await import('./auto-update.js');
+                    const outcome = await autoUpdate({
+                        currentVersion: VERSION,
+                        modulePath: fileURLToPath(import.meta.url),
+                        enabled: cfg.autoUpdate !== false,
+                    });
+                    if (live && outcome.state === 'installed')
+                        setAutoUpdated(outcome.version);
+                }
+                catch { /* an update that cannot happen is not worth interrupting anyone over */ }
+            })();
+        }, 4_000);
+        return () => { live = false; clearTimeout(timer); };
+    }, [cfg.autoUpdate]);
     // A queued prompt runs the moment the agent is free again. Keyed on `busy` rather than done
     // inside the turn's own exit paths, so every way a turn can end — success, failure, cancel —
     // drains the queue the same way.
@@ -4784,6 +4840,12 @@ export function App({ config: initialConfig, clearFrame, synchronizedFrames = fa
                         setCfg(c => ({ ...c, autoCopy: ['true', '1', 'yes'].includes(val.toLowerCase()) }));
                     if (key === 'animate')
                         setAnimate(['true', '1', 'yes'].includes(val.toLowerCase()));
+                    // Applied to the live session, not only written to the file. Every other switch here
+                    // takes effect at once, and one that quietly waited for a restart was a setting you
+                    // could not tell you had changed.
+                    if (key === 'autoUpdate') {
+                        setCfg(c => ({ ...c, autoUpdate: ['true', '1', 'yes'].includes(val.toLowerCase()) }));
+                    }
                     if (key === 'mouse') {
                         setMouse(['true', '1', 'yes'].includes(val.toLowerCase()));
                     }
@@ -5397,12 +5459,12 @@ export function App({ config: initialConfig, clearFrame, synchronizedFrames = fa
                 return;
             }
             if (key.pageDown) {
-                const jump = pickerRowLimit(picker.kind, list.length, ...(altScreen ? [termRows, 13] : []));
+                const jump = pickerRowLimit(picker.kind, list.length, termRows, altScreen ? 13 : undefined);
                 setPickIndex(i => Math.min(i + jump, Math.max(0, list.length - 1)));
                 return;
             }
             if (key.pageUp) {
-                const jump = pickerRowLimit(picker.kind, list.length, ...(altScreen ? [termRows, 13] : []));
+                const jump = pickerRowLimit(picker.kind, list.length, termRows, altScreen ? 13 : undefined);
                 setPickIndex(i => Math.max(0, i - jump));
                 return;
             }
@@ -6360,13 +6422,40 @@ export function App({ config: initialConfig, clearFrame, synchronizedFrames = fa
                                 const meter = (fraction, label) => [`${bar(fraction, BAR_W)} ${Math.round(fraction * 100)}%`, `${label}`, ''];
                                 const lines = [tabs, '---'];
                                 if (usagePane === 'settings') {
-                                    lines.push('**Settings**  — written to disk and applied now', '');
+                                    const compact = termRows < 30;
+                                    const chrome = compact ? 21 : 21;
+                                    if (!compact)
+                                        lines.push('**Settings**  — written to disk and applied now', '');
                                     // What is actually in force, not just what this file happens to hold. provider and
                                     // baseURL commonly come from ~/.koneckrc, and showing those as "(not set)" made a
                                     // configured session look unconfigured.
                                     const endpoint = endpointFor(cfg.provider);
                                     const labelW = Math.max(...CONFIG_SCHEMA.map(f => f.label.length));
+                                    /*
+                                     * Only as many rows as there is room for.
+                                     *
+                                     * Everything else in this pane costs rows too — the tabs, the heading, the two footer
+                                     * lines, the panel's own border — and below it sit the composer and the status bar. Drawn
+                                     * in full on a twenty-four-row terminal the list ran off the top, leaving what looked
+                                     * like a stunted four-row panel. It also made the repainting region as tall as the
+                                     * window, which is what makes Ink wipe the terminal on every render.
+                                     */
+                                    /*
+                                     * How many rows there is room for, and what gets dropped when there is not.
+                                     *
+                                     * Measured, not estimated. At 80x24 the pane came to twenty-five lines with the list
+                                     * already cut to its minimum of three, and twenty-five lines in a twenty-four-row
+                                     * window is what makes Ink wipe the whole terminal on every render. Shrinking the list
+                                     * further could not have fixed that — the fixed parts were the problem — so on a short
+                                     * terminal the pane drops its own furniture first: the heading, the blank beneath it,
+                                     * and the line naming the config file. They are the three that say least.
+                                     */
+                                    const { start, end } = settingsWindow(CONFIG_SCHEMA.length, settingRow, Math.max(3, termRows - chrome));
+                                    if (start > 0)
+                                        lines.push(`  ${G.caret} ${start} more above`);
                                     for (const [i, f] of CONFIG_SCHEMA.entries()) {
+                                        if (i < start || i >= end)
+                                            continue;
                                         const selected = i === settingRow;
                                         const beingTyped = editing?.key === String(f.key);
                                         // An endpoint always has a source worth naming, so it never reads as unset or as
@@ -6393,11 +6482,15 @@ export function App({ config: initialConfig, clearFrame, synchronizedFrames = fa
                                                         : `   enter to edit${f.clearable ? ', x clears' : ''}`;
                                         lines.push(`${mark} ${f.label.padEnd(labelW)}  ${selected ? '**' + shown + '**' : shown}${hint}`);
                                     }
+                                    if (end < CONFIG_SCHEMA.length) {
+                                        lines.push(`  ${G.caret} ${CONFIG_SCHEMA.length - end} more below`);
+                                    }
                                     lines.push('');
                                     lines.push(editing
                                         ? 'Typing. **enter** saves, **ctrl+u** empties the field, **esc** abandons.'
                                         : 'Up/down choose · enter change · x clear · left/right tabs');
-                                    lines.push(`Saved to ${CONFIG_FILE}`);
+                                    if (!compact)
+                                        lines.push(`Saved to ${CONFIG_FILE}`);
                                 }
                                 if (usagePane === 'session') {
                                     const wall = Date.now() - sessionStartRef.current;
@@ -6579,7 +6672,7 @@ export function App({ config: initialConfig, clearFrame, synchronizedFrames = fa
                                 // footer. They consume six rows and the picker itself has six non-choice rows (margin,
                                 // border, title/search and pager), so reserve thirteen in total. Letting 16 choices into
                                 // a 24-row frame made Ink reflow the live region and visibly blink on every keypress.
-                                const pickerRows = pickerRowLimit(picker.kind, list.length, ...(altScreen ? [termRows, 13] : []));
+                                const pickerRows = pickerRowLimit(picker.kind, list.length, termRows, altScreen ? 13 : undefined);
                                 // Keep the highlighted row inside the visible window as the selection moves.
                                 const start = Math.max(0, Math.min(pickIndex - Math.floor(pickerRows / 2), list.length - pickerRows));
                                 const shown = list.slice(Math.max(0, start), Math.max(0, start) + pickerRows);
@@ -6671,17 +6764,17 @@ export function App({ config: initialConfig, clearFrame, synchronizedFrames = fa
                                 return (_jsx(Box, { marginTop: 1, children: _jsx(Panel, { width: panelWidth(lines, barWidth, 46), color: AMBER, title: `Permission — ${modeSpec(mode).label} mode`, children: lines }) }));
                             })()] }), sidePanelOpen && (_jsx(Box, { flexDirection: "column", width: diffPanelWidth, flexShrink: 0, children: diffPanelOpen && _jsx(DiffPanel, { files: liveDiffs, width: diffPanelWidth, height: Math.max(6, frameRows - 6), activityRows: activityPanelOpen ? 4 : 0, activity: activityPanelOpen ? _jsxs(_Fragment, { children: [_jsxs(Text, { color: CYAN, bold: true, children: ["Activity \u00B7 ", fmtElapsed(busyStart.current ? Date.now() - busyStart.current : 0)] }), _jsx(Text, { color: AMBER, wrap: "truncate", children: ask ? 'Waiting for your approval' : liveToolRef.current
                                             ? 'Running ' + liveToolRef.current.name : thinkingRef.current.chars > 0
-                                            ? 'Waiting for model · reasoning received' : 'Waiting for model' }), _jsx(Text, { color: MUTED, wrap: "truncate", children: liveToolRef.current?.detail || plan?.steps.find(s => s.state !== 'done')?.text || 'esc to interrupt' })] }) : undefined }) }))] }), _jsxs(Box, { marginTop: 1, borderStyle: "round", borderColor: busy ? DIM : CYAN, paddingX: 1, width: uiWidth - 5, flexShrink: 0, children: [_jsxs(Text, { color: CYAN, bold: true, children: [G.caret, " "] }), (() => {
-                        const sel = selectionRange(selAnchor, caret, draft.length);
-                        if (sel) {
-                            return (_jsxs(_Fragment, { children: [_jsx(Text, { color: INK, children: draft.slice(0, sel[0]) }), _jsx(Text, { backgroundColor: CYAN, color: "#10222A", children: draft.slice(sel[0], sel[1]) }), _jsx(Text, { color: INK, children: draft.slice(sel[1]) })] }));
-                        }
-                        return (_jsxs(_Fragment, { children: [_jsx(Text, { color: INK, children: draft.slice(0, caret) }), busy
-                                    ? _jsx(Text, { color: INK, children: draft.slice(caret) })
-                                    : caret < draft.length
-                                        ? _jsxs(_Fragment, { children: [_jsx(Text, { backgroundColor: CYAN, color: "#10222A", children: draft[caret] }), _jsx(Text, { color: INK, children: draft.slice(caret + 1) })] })
-                                        : _jsx(Text, { color: CYAN, children: "\u2588" })] }));
-                    })()] }), copyNotice && _jsx(Text, { color: CYAN, children: copyNotice }), _jsx(Box, { marginTop: 1, paddingX: 1, width: uiWidth - 4, flexShrink: 0, children: _jsxs(Text, { wrap: "truncate", backgroundColor: "#24343B", children: [repaint % 2 === 1 ? '\u200b' : '', _jsx(Text, { color: MUTED, children: "TAB " }), _jsx(Text, { color: INK, children: "Analytics" }), _jsx(Text, { color: MUTED, children: " / " }), _jsx(Text, { color: CYAN, children: "/HELP" }), _jsx(Text, { color: INK, children: " Commands" }), _jsx(Text, { color: MUTED, children: " / SHIFT+TAB " }), _jsx(Text, { color: INK, children: "Mode" }), _jsx(Text, { color: MUTED, children: " / " }), _jsx(Text, { color: statusColor, children: statusLabel.toLowerCase() }), plan && planSummary(plan) && (_jsxs(_Fragment, { children: [_jsx(Text, { color: MUTED, children: " / PLAN: " }), _jsx(Text, { color: CYAN, children: planSummary(plan) })] })), _jsx(Text, { color: MUTED, children: " / MODE: " }), _jsx(Text, { color: mode === 'auto' ? INK : AMBER, children: modeSpec(mode).label }), _jsx(Text, { color: MUTED, children: " / EFFORT: " }), _jsx(Text, { color: effort === 'medium' ? INK : AMBER, children: effort }), mouse && (_jsxs(_Fragment, { children: [_jsx(Text, { color: MUTED, children: " / " }), _jsx(Text, { color: AMBER, children: "WHEEL: on \u2014 shift to select \u00B7 /mouse off" })] })), _jsx(Text, { color: MUTED, children: " / MODEL: " }), _jsx(Text, { color: INK, children: modelShort }), _jsx(Text, { color: MUTED, children: " / v" }), _jsx(Text, { color: updatePane?.behind ? AMBER : INK, children: VERSION }), updatePane?.behind && _jsx(Text, { color: AMBER, children: " \u2191" }), pasteCount > 0 && (_jsxs(_Fragment, { children: [_jsx(Text, { color: MUTED, children: " / " }), _jsxs(Text, { color: AMBER, children: [pasteCount, " paste", pasteCount === 1 ? '' : 's', " held, sent in full"] })] })), ' '.repeat(uiWidth)] }) })] }));
+                                            ? 'Waiting for model · reasoning received' : 'Waiting for model' }), _jsx(Text, { color: MUTED, wrap: "truncate", children: liveToolRef.current?.detail || plan?.steps.find(s => s.state !== 'done')?.text || 'esc to interrupt' })] }) : undefined }) }))] }), _jsx(Box, { marginTop: 1, borderStyle: "round", borderColor: busy ? DIM : CYAN, paddingX: 1, width: uiWidth - 5, flexShrink: 0, children: _jsxs(Text, { wrap: "wrap", children: [_jsxs(Text, { color: CYAN, bold: true, children: [G.caret, " "] }), (() => {
+                            const sel = selectionRange(selAnchor, caret, draft.length);
+                            if (sel) {
+                                return (_jsxs(_Fragment, { children: [_jsx(Text, { color: INK, children: draft.slice(0, sel[0]) }), _jsx(Text, { backgroundColor: CYAN, color: "#10222A", children: draft.slice(sel[0], sel[1]) }), _jsx(Text, { color: INK, children: draft.slice(sel[1]) })] }));
+                            }
+                            return (_jsxs(_Fragment, { children: [_jsx(Text, { color: INK, children: draft.slice(0, caret) }), busy
+                                        ? _jsx(Text, { color: INK, children: draft.slice(caret) })
+                                        : caret < draft.length
+                                            ? _jsxs(_Fragment, { children: [_jsx(Text, { backgroundColor: CYAN, color: "#10222A", children: draft[caret] }), _jsx(Text, { color: INK, children: draft.slice(caret + 1) })] })
+                                            : _jsx(Text, { color: CYAN, children: "\u2588" })] }));
+                        })()] }) }), copyNotice && _jsx(Text, { color: CYAN, children: copyNotice }), _jsx(Box, { marginTop: 1, paddingX: 1, width: uiWidth - 4, flexShrink: 0, children: _jsxs(Text, { wrap: "truncate", backgroundColor: "#24343B", children: [repaint % 2 === 1 ? '\u200b' : '', _jsx(Text, { color: MUTED, children: "TAB " }), _jsx(Text, { color: INK, children: "Analytics" }), _jsx(Text, { color: MUTED, children: " / " }), _jsx(Text, { color: CYAN, children: "/HELP" }), _jsx(Text, { color: INK, children: " Commands" }), _jsx(Text, { color: MUTED, children: " / SHIFT+TAB " }), _jsx(Text, { color: INK, children: "Mode" }), _jsx(Text, { color: MUTED, children: " / " }), _jsx(Text, { color: statusColor, children: statusLabel.toLowerCase() }), plan && planSummary(plan) && (_jsxs(_Fragment, { children: [_jsx(Text, { color: MUTED, children: " / PLAN: " }), _jsx(Text, { color: CYAN, children: planSummary(plan) })] })), _jsx(Text, { color: MUTED, children: " / MODE: " }), _jsx(Text, { color: mode === 'auto' ? INK : AMBER, children: modeSpec(mode).label }), _jsx(Text, { color: MUTED, children: " / EFFORT: " }), _jsx(Text, { color: effort === 'medium' ? INK : AMBER, children: effort }), autoUpdated && (_jsxs(_Fragment, { children: [_jsx(Text, { color: MUTED, children: " / " }), _jsxs(Text, { color: GREEN, children: [G.ok, " Update installed \u00B7 restart to use it"] })] })), mouse && (_jsxs(_Fragment, { children: [_jsx(Text, { color: MUTED, children: " / " }), _jsx(Text, { color: AMBER, children: "WHEEL: on \u2014 shift to select \u00B7 /mouse off" })] })), _jsx(Text, { color: MUTED, children: " / MODEL: " }), _jsx(Text, { color: INK, children: modelShort }), _jsx(Text, { color: MUTED, children: " / v" }), _jsx(Text, { color: updatePane?.behind ? AMBER : INK, children: VERSION }), updatePane?.behind && _jsx(Text, { color: AMBER, children: " \u2191" }), pasteCount > 0 && (_jsxs(_Fragment, { children: [_jsx(Text, { color: MUTED, children: " / " }), _jsxs(Text, { color: AMBER, children: [pasteCount, " paste", pasteCount === 1 ? '' : 's', " held, sent in full"] })] })), ' '.repeat(uiWidth)] }) })] }));
 }
 export async function runInkChatMode(config) {
     // Nothing is written before the first frame. Filling the screen to push the prompt to the foot