koneck 2.128.72 → 2.128.74

package/dist/ink-chat.js

@@ -10,7 +10,8 @@ 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 as loadConfigAt, saveKoneckConfig as saveConfigAt, setConfigKey as setConfigKeyAt, 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 { deleteProvider, deletable, describeDeleted } from './provider-delete.js';
+import { PROVIDERS, resolveProvider, getApiKey, addUserProvider, 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';
 import { rememberKey, forgetAllKeys } from './credentials.js';
@@ -347,6 +348,11 @@ export function wrapPreserving(text, width) {
  * file, which every earlier grep had failed to match for that reason.
  */
 export const ADD_PROVIDER = '\u0000add-provider';
+/** The provider picker's delete row, and the rows of the two pickers it leads to. */
+export const DELETE_PROVIDER = '\u0000delete-provider';
+const DELETE_ONE = '\u0000delete:';
+const DELETE_CONFIRMED = '\u0000delete-confirmed:';
+const DELETE_KEPT = '\u0000delete-kept';
 /**
  * Checks one field and says what is wrong with it, or nothing.
  *
@@ -751,7 +757,7 @@ const COMMANDS = [
     { cmd: '/key', desc: 'Enter an API key for the current provider' },
     { cmd: '/logout', desc: 'Forget Koneck-stored API credentials; keep preferences' },
     { cmd: '/model', desc: 'Show or set the model by name' },
-    { cmd: '/provider', desc: 'Show or switch provider' },
+    { cmd: '/provider', desc: 'Show, switch, add or delete a provider' },
     { cmd: '/status', desc: 'Provider, model, mode, tokens, cost, workspace' },
     { cmd: '/diff', desc: 'Open the live diff panel; /diff off closes it' },
     { cmd: '/permissions', desc: 'Show or set approval posture: on or off' },
@@ -2539,6 +2545,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 +2947,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);
@@ -4097,6 +4117,11 @@ export function App({ config: initialConfig, clearFrame, synchronizedFrames = fa
         return [
             { value: ADD_PROVIDER, label: '+ add a provider',
                 desc: 'an OpenAI-compatible endpoint KONECK does not ship with' },
+            // Only when there is something to delete, so the row never leads to an empty list.
+            ...(Object.keys(PROVIDERS).some(n => deletable(n))
+                ? [{ value: DELETE_PROVIDER, label: '- delete one',
+                        desc: 'delete a provider you added, or the key you stored for one' }]
+                : []),
             ...Object.values(PROVIDERS).map(p => ({
                 value: p.name,
                 label: p.name,
@@ -4105,6 +4130,46 @@ export function App({ config: initialConfig, clearFrame, synchronizedFrames = fa
             })),
         ];
     }
+    /** What can be deleted, as picker rows: yours first, then built-ins with something stored. */
+    function deleteProviderItems() {
+        return Object.values(PROVIDERS).filter(p => deletable(p.name))
+            .sort((a, b) => Number(!!b.userDefined) - Number(!!a.userDefined))
+            .map(p => ({
+            value: DELETE_ONE + p.name, label: p.name,
+            desc: p.userDefined ? `yours - ${p.baseURL}` : 'built in - forgets the key and settings you stored',
+            current: p.name === cfg.provider,
+        }));
+    }
+    /**
+     * Deletes a provider and says what went. The provider in use loses its key in this session too,
+     * and when it is gone altogether the picker opens to choose another — the next message would
+     * otherwise fail against a provider that no longer exists.
+     */
+    async function deleteProviderNamed(name) {
+        const wasCurrent = name === cfg.provider;
+        const done = await deleteProvider(name);
+        if (!done) {
+            addSystem(PROVIDERS[name]
+                ? `Nothing is stored for "${name}" — it ships with KONECK, with no key or settings of yours to delete.`
+                : `No provider called "${name}".`);
+            return;
+        }
+        (await import('./model-catalog.js')).forgetCatalog(name);
+        if (!wasCurrent) {
+            addSystem(describeDeleted(done));
+            return;
+        }
+        if (PROVIDERS[name]) {
+            const cleared = { ...cfg, apiKey: undefined };
+            cfgRef.current = cleared;
+            setCfg(cleared);
+            await resetSession(cleared);
+            addSystem(describeDeleted(done) + ' This session was using it, and has been reset.');
+            return;
+        }
+        addSystem(describeDeleted(done) + ' This session was using it — choose another below.');
+        openPicker('provider', providerItems());
+    }
     /** Opens the declare-a-provider prompt, optionally with fields already known. */
     function startAddProvider(seed = {}) {
         setAddProv({ step: 'name', name: '', url: '', model: '', value: '', ...seed });
@@ -4322,6 +4387,28 @@ export function App({ config: initialConfig, clearFrame, synchronizedFrames = fa
             startAddProvider();
             return;
         }
+        // Deleting: choose which, then say yes. The key cannot be got back from KONECK once it is gone.
+        if (kind === 'provider' && item.value === DELETE_PROVIDER) {
+            openPicker('provider', deleteProviderItems());
+            return;
+        }
+        if (kind === 'provider' && item.value.startsWith(DELETE_ONE)) {
+            const name = item.value.slice(DELETE_ONE.length);
+            openPicker('provider', [
+                { value: DELETE_CONFIRMED + name, label: `yes, delete ${name}`,
+                    desc: PROVIDERS[name]?.userDefined ? 'its declaration, stored key and saved settings' : 'its stored key and saved settings' },
+                { value: DELETE_KEPT, label: 'no, keep it', desc: '' },
+            ]);
+            return;
+        }
+        if (kind === 'provider' && item.value.startsWith(DELETE_CONFIRMED)) {
+            await deleteProviderNamed(item.value.slice(DELETE_CONFIRMED.length));
+            return;
+        }
+        if (kind === 'provider' && item.value === DELETE_KEPT) {
+            addSystem('Kept.');
+            return;
+        }
         if (kind === 'provider') {
             const def = PROVIDERS[item.value];
             // The endpoint saved for the provider being switched to, not its built-in address. Passing
@@ -4792,25 +4879,19 @@ export function App({ config: initialConfig, clearFrame, synchronizedFrames = fa
                     });
                     return;
                 }
-                if (verb?.toLowerCase() === 'remove' || verb?.toLowerCase() === 'forget') {
+                if (['remove', 'delete', 'forget'].includes(verb?.toLowerCase() ?? '')) {
                     const target = (restArgs[0] ?? '').toLowerCase();
                     if (!target) {
-                        const mine = Object.values(PROVIDERS).filter(p => p.userDefined).map(p => p.name);
-                        addSystem(mine.length
-                            ? `Usage: /provider remove <name>\nYours: ${mine.join(', ')}`
-                            : 'You have not declared any providers, so there is nothing to remove.');
-                        return;
-                    }
-                    const wasShadowing = BUILT_IN_PROVIDERS[target] !== undefined;
-                    if (!removeUserProvider(target)) {
-                        addSystem(PROVIDERS[target]
-                            ? `"${target}" ships with KONECK — it was not declared by you, so there is nothing to remove.`
-                            : `No declared provider called "${target}".`);
+                        const items = deleteProviderItems();
+                        if (!items.length) {
+                            addSystem('Nothing to delete: you have added no providers and stored no keys.');
+                            return;
+                        }
+                        openPicker('provider', items);
                         return;
                     }
-                    addSystem(wasShadowing
-                        ? `Removed your "${target}". The built-in one is back, pointing at ${BUILT_IN_PROVIDERS[target].baseURL}.`
-                        : `Removed "${target}".`);
+                    // Typed out in full, the name is the confirmation; the picker asks, because a row is one key away.
+                    await deleteProviderNamed(target);
                     return;
                 }
                 if (!arg) {
@@ -4830,7 +4911,7 @@ export function App({ config: initialConfig, clearFrame, synchronizedFrames = fa
                         + '\nSwitch:  /provider <name>, or pick one below\n'
                         + 'Add:     /provider add  — asks for a name, an endpoint and a default model\n'
                         + '         /provider add <name> <url> [model]  if you have them to hand\n'
-                        + 'Remove:  /provider remove <name>');
+                        + 'Delete:  /provider delete [name]  — one you added, or the key you stored for one');
                     openPicker('provider', providerItems());
                     return;
                 }
@@ -7035,6 +7116,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 +7387,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 +7892,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 +7904,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 +7929,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 +8047,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 +8059,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);