koneck 2.25.61 → 2.25.63

package/dist/ink-chat.js

@@ -2,13 +2,13 @@ import { jsx as _jsx, Fragment as _Fragment, jsxs as _jsxs } from "react/jsx-run
 import React, { useState, useEffect, useRef } from 'react';
 import { Box, Static, Text, render, useApp, useInput, useStdin } from 'ink';
 import { execa } from 'execa';
-import { createAgentSession, buildClient, resolveAgentConcurrency, toolsUnsupportedNotice } from './engine.js';
+import { createAgentSession, buildClient, resolveAgentConcurrency, toolsUnsupportedNotice, isLocalRuntime } from './engine.js';
 import { estimateCost, formatCost } 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 { PROVIDERS, resolveProvider, getApiKey, addUserProvider, userProvidersPath } from './providers.js';
-import { resolveEndpoint, normaliseEndpoint, withEndpoint, savedEndpoints } from './endpoints.js';
+import { resolveEndpoint, normaliseEndpoint, withEndpoint, savedEndpoints, shortSource } from './endpoints.js';
 import { loadProjectConfig, loadGlobalConfig } from './config.js';
 import { listCheckpoints, revertCheckpoint, snapshotTree, filesTouchedBy } from './checkpoint.js';
 import { fileDiff, diffSummary } from './diff-view.js';
@@ -501,6 +501,7 @@ const COMMANDS = [
     { cmd: '/models', desc: 'Browse and switch models for the connected provider' },
     { cmd: '/connect', desc: 'Connect a provider, or declare a new one with a URL' },
     { cmd: '/endpoint', desc: "Show or change this provider's URL, e.g. a remote Ollama" },
+    { cmd: '/redraw', desc: 'Repaint the terminal — same as ctrl+L' },
     { cmd: '/key', desc: 'Enter an API key for the current provider' },
     { cmd: '/model', desc: 'Show or set the model by name' },
     { cmd: '/provider', desc: 'Show or switch provider' },
@@ -733,6 +734,7 @@ Slash commands:
   /connect [provider]   Connect a provider (picker when no name given)
   /connect <name> <url> Declare a provider KONECK does not ship with, e.g.
                         /connect openrouter https://openrouter.ai/api/v1
+  /redraw               Repaint the terminal (ctrl+L) if a stale frame is left on screen
   /endpoint             Show the endpoint in force and where it came from
   /endpoint <url>       Point this provider somewhere else, e.g.
                         /endpoint http://192.168.1.50:11434/v1
@@ -1012,6 +1014,18 @@ function App({ config: initialConfig }) {
             toolsWarnedRef.current = model;
             addSystem(toolsUnsupportedNotice(model));
         },
+        onLooping: (copies) => {
+            // The loop itself is the symptom. On a local model the cause is almost always the window:
+            // Ollama serves 4096 tokens unless told otherwise, whatever the model was trained for, and
+            // a conversation that no longer fits is exactly what makes a model start repeating.
+            const local = isLocalRuntime(cfg.provider);
+            addSystem(`${cfg.model} repeated the same text ${copies} times, so the reply was cut short.` +
+                (local
+                    ? `\n\nThis is usually the context window rather than the model. Ollama serves 4096 ` +
+                        `tokens by default however large the model is — set ` +
+                        `\`OLLAMA_CONTEXT_LENGTH=32768\` and restart it, then \`/context\` to confirm.`
+                    : ''));
+        },
         onThinking: (text, totalChars) => {
             thinkingRef.current = { chars: totalChars, at: Date.now() };
             // The text was discarded before this, so the model's reasoning was never visible — only a
@@ -1075,6 +1089,8 @@ function App({ config: initialConfig }) {
      * reading something further up is not yanked away by the next tool call.
      */
     const [scrollBack, setScrollBack] = useState(0);
+    /** Bumped to make Ink write a frame whose content has not changed. See redrawScreen. */
+    const [repaint, setRepaint] = useState(0);
     const [ask, setAsk] = useState(null);
     useEffect(() => {
         if (!process.stdout.isTTY)
@@ -1415,6 +1431,30 @@ function App({ config: initialConfig }) {
         if (newCfg)
             setCfg(newCfg);
     }
+    /**
+     * Repaint the terminal from scratch.
+     *
+     * Reported as KONECK looking "split into two parts, two message input box": a previous frame
+     * left on screen above the live one, with the transcript written in between. Ink erases the
+     * lines it believes it wrote, and any line the terminal wrapped occupies two rows while
+     * counting as one — so the erase falls short and the old frame is stranded. It cannot be
+     * unstranded from inside the frame, because Ink is not aware it is there.
+     *
+     * Ctrl+L is what everybody presses when a terminal looks wrong, and KONECK had no binding for
+     * it at all. It wipes the screen and its scrollback; the conversation is untouched, since it
+     * lives in the session rather than in the terminal's memory of it.
+     */
+    function redrawScreen() {
+        if (!process.stdout.isTTY)
+            return;
+        // Clear the screen, clear the scrollback with it — a stranded frame that has scrolled out of
+        // view is still in the buffer, and leaving it there means it reappears on the next scroll —
+        // then home the cursor so Ink's next frame starts from a known position.
+        process.stdout.write('\u001b[2J\u001b[3J\u001b[H');
+        // Ink writes a frame only when its content differs from the last one, and the content has
+        // not changed — the screen it was drawn on has. Bumping this forces the write.
+        setRepaint(n => n + 1);
+    }
     async function doSave() {
         const s = await activeSession.p.catch(() => null);
         const firstUser = s?.messages.find((m) => m.role === 'user');
@@ -1977,6 +2017,10 @@ function App({ config: initialConfig }) {
                     `Session reset. /endpoint <url> changes the address.`);
                 return;
             }
+            case '/redraw': {
+                redrawScreen();
+                return;
+            }
             case '/url':
             case '/baseurl':
             case '/endpoint': {
@@ -2673,6 +2717,12 @@ function App({ config: initialConfig }) {
             void doSave().catch(() => { }).then(() => exit());
             return;
         }
+        // Ctrl+L repaints, as it does everywhere else. The conversation is not touched — only the
+        // terminal's picture of it, which is the thing that was wrong.
+        if (key.ctrl && input === 'l') {
+            redrawScreen();
+            return;
+        }
         // ── Credential entry owns the keyboard, and echoes nothing ───────────────
         // Scrolling the transcript, which on the alternate screen is the application's job: the
         // terminal's own buffer is not there to fall back on.
@@ -2879,6 +2929,10 @@ function App({ config: initialConfig }) {
                     }
                     return;
                 }
+                if (key.ctrl && input === 'u') {
+                    setEditing(e => (e ? { ...e, value: '' } : e));
+                    return;
+                }
                 if (key.backspace || key.delete) {
                     setEditing(e => (e ? { ...e, value: e.value.slice(0, -1) } : e));
                     return;
@@ -3295,7 +3349,16 @@ function App({ config: initialConfig }) {
     // window empty. A cap is still available through `maxWidth` in config for anyone who
     // prefers a narrow measure on a very wide screen, but nothing is capped by default.
     const uiWidth = computeUiWidth(termCols, configMaxWidth);
-    const barWidth = uiWidth - 4; // uiWidth minus the container's paddingX={2} on both sides
+    // uiWidth minus the container's paddingX={2} on both sides, and then one column of slack.
+    //
+    // Without the slack a full-width box draws a line exactly as wide as the terminal. Ink counts
+    // the lines it wrote in order to erase them next frame, and a line that the terminal wrapped
+    // occupies two rows while counting as one — so the erase falls short and the previous frame is
+    // stranded on screen. That is how a screen ends up showing two composers and two status bars.
+    // One character of a different width than Ink assumed is enough to cause it, and East Asian
+    // ambiguous glyphs (·, ↑↓, ⌕, –, all of which the UI uses) are rendered two cells wide by some
+    // terminals and one by others. A spare column costs nothing and removes the whole class.
+    const barWidth = uiWidth - 5;
     // Space a rendered reply actually has: the bar, minus the response box border (2), its
     // paddingX={1} (2), and the "ꓘK " prefix plus its gap (3). Getting this even one column
     // too wide makes Ink reflow table rows and shatter their borders.
@@ -3449,7 +3512,7 @@ function App({ config: initialConfig }) {
                             : displayValue(value);
                         const shown = beingTyped
                             ? `${editing.value}_`
-                            : isEndpoint ? `${raw}  (${endpoint.source})`
+                            : isEndpoint ? `${raw}  (${shortSource(endpoint.source)})`
                                 : raw + (here || value === undefined ? '' : ' (inherited)');
                         // The marker is what makes the selection legible without colour, which a panel line
                         // drawn as one string cannot carry per-segment.
@@ -3458,12 +3521,12 @@ function App({ config: initialConfig }) {
                             : f.type.kind === 'boolean' ? '   enter toggles'
                                 : f.type.kind === 'choice' ? `   enter cycles: ${f.type.choices.join(' / ')}`
                                     : f.type.kind === 'number' ? `   type a number, enter to save${f.clearable ? ', x clears' : ''}`
-                                        : `   enter to type a value${f.clearable ? ', x clears' : ''}`;
+                                        : `   enter to edit${f.clearable ? ', x clears' : ''}`;
                         lines.push(`${mark} ${f.label.padEnd(labelW)}  ${selected ? '**' + shown + '**' : shown}${hint}`);
                     }
                     lines.push('');
                     lines.push(editing
-                        ? 'Typing. **enter** saves, **esc** abandons.'
+                        ? '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}`);
                 }
@@ -3673,7 +3736,7 @@ function App({ config: initialConfig }) {
                         ? _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" })] }), _jsx(Box, { marginTop: 1, paddingX: 1, children: _jsxs(Text, { wrap: "truncate", backgroundColor: "#24343B", children: [_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 }), _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"] })] }))] }) })] }));
+                            : _jsx(Text, { color: CYAN, children: "\u2588" })] }), _jsx(Box, { marginTop: 1, paddingX: 1, 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 }), _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"] })] }))] }) })] }));
 }
 export async function runInkChatMode(config) {
     // Nothing is written before the first frame. Filling the screen to push the prompt to the foot