klyro 1.0.11 → 1.0.13

package/dist/cli/repl.js

@@ -25,7 +25,7 @@ import { parseUnifiedDiff } from '../tui/diff-parser.js';
 import { parse } from './slash/parser.js';
 import { resolveProvider, providerHelp, lastProviderError } from '../providers.js';
 import { readVersion } from '../version.js';
-import { MouseFilter, MOUSE_ENABLE, MOUSE_DISABLE, PASTE_ENABLE, PASTE_DISABLE, PasteFilter, createReadWrapper } from '../tui/mouse.js';
+import { MouseFilter, MOUSE_ENABLE, MOUSE_DISABLE, PASTE_ENABLE, PASTE_DISABLE, PasteFilter, createReadWrapper, isMouseReportingEnabled } from '../tui/mouse.js';
 import { inferProviderFromBaseURL } from '../agent/registry.js';
 import { getDefaultSessionStore } from '../persistence/session.js';
 import { buildSystemPrompt, parseImageInput } from '../context/system-prompt.js';
@@ -372,13 +372,20 @@ export async function startRepl(opts = {}) {
     // ── Full-screen takeover like OpenCode — always when klyro in TTY (user explicitly wants it)
     // Scroll now works correctly via internal viewport, not native terminal scroll
     const isAltScreen = useTui && !!process.stdout.isTTY && process.env.KLYRO_NO_ALT !== '1';
+    // Mouse reporting is opt-in (KLYRO_MOUSE=1): when on, the terminal sends
+    // clicks/wheel to the app (wheel scrolls ±3 lines) but native text
+    // selection and right-click paste stop working. Default off so select to
+    // copy and right-click paste work out of the box; bracketed paste
+    // (keyboard paste) is unaffected and always enabled below.
+    const mouseReporting = isAltScreen && isMouseReportingEnabled();
     const enterAlt = () => {
         if (!isAltScreen)
             return;
         try {
             process.stdout.write('\x1b[?1049h\x1b[?25l'); // alt screen + hide cursor
             process.stdout.write('\x1b[H\x1b[2J'); // home + clear
-            process.stdout.write(MOUSE_ENABLE); // wheel events (SGR), see tui/mouse.ts
+            if (mouseReporting)
+                process.stdout.write(MOUSE_ENABLE); // wheel events (SGR), see tui/mouse.ts
         }
         catch { /* ignore */ }
     };
@@ -386,7 +393,8 @@ export async function startRepl(opts = {}) {
         if (!isAltScreen)
             return;
         try {
-            process.stdout.write(MOUSE_DISABLE);
+            if (mouseReporting)
+                process.stdout.write(MOUSE_DISABLE);
             process.stdout.write('\x1b[?25h\x1b[?1049l'); // show cursor + leave alt
         }
         catch { /* ignore */ }
@@ -2681,8 +2689,8 @@ export async function startRepl(opts = {}) {
                         '  Enter send · Shift+Enter newline · Tab complete slash · Esc drop queued / Esc×2 cancel run',
                         '  Ctrl+C cancel (1st) / quit (2nd) · Ctrl+O expand last tool group · Ctrl+G jump bottom',
                         '  PgUp/PgDn or Ctrl+U/Ctrl+D half-page · Ctrl+Home/End top/bottom · Home/End jump · Space jump to unread',
-                        '  Ctrl+B/F page · Shift/Ctrl+↑/↓ line · ↑/↓ history (with text) else scroll · wheel ±3 lines',
-                        '  Shift+drag selects · /vim toggles vim input mode · /keymap <note> saves a display note',
+                        '  Ctrl+B/F page · Shift/Ctrl+↑/↓ line · ↑ history at live tail, scrolls while reading above · PgUp/Dn scroll (KLYRO_MOUSE=1 adds wheel ±3 lines)',
+                        '  Text selection/copy and right-click paste work natively · Shift+Enter newline · /vim toggles vim input mode · /keymap <note> saves a display note',
                     ].join('\n'),
                 });
                 return;

package/dist/tui/app.js

@@ -1052,16 +1052,22 @@ export function App(props) {
             void props.onSlash({ kind: 'quit' });
             return;
         }
-        // Contextual ↑/↓ (§8.3): text in buffer (or browsing) → history;
-        // empty buffer → scroll viewport one line.
+        // Contextual ↑/↓ (§8.3): both behaviors, split by where you are.
+        // Scrolled up reading the transcript (not at the live tail, not already
+        // browsing) → plain ↑ keeps scrolling so chat scroll is never lost.
+        // At the live tail (or no scroll surface) → ↑ browses input history
+        // newest-first, so send + ↑ restores the last prompt. ↓ is unchanged:
+        // browse newer while browsing, else scroll toward the tail.
         if (key.upArrow && !key.shift && !key.ctrl) {
-            if (input.trim() !== '' || histIdx !== null) {
-                if (history.length > 0) {
-                    const next = histIdx === null ? history.length - 1 : Math.max(0, histIdx - 1);
-                    setHistIdx(next);
-                    setInput(history[next] ?? '');
-                    setVimCursor(null);
-                }
+            if (isFullscreen && maxTop > 0 && !atBottom && histIdx === null && input.trim() === '') {
+                commands.lineUp();
+                return;
+            }
+            if (history.length > 0) {
+                const next = histIdx === null ? history.length - 1 : Math.max(0, histIdx - 1);
+                setHistIdx(next);
+                setInput(history[next] ?? '');
+                setVimCursor(null);
                 return;
             }
             if (isFullscreen && maxTop > 0) {

package/dist/tui/app.test.js

@@ -247,6 +247,7 @@ describe('App', () => {
         expect(lastFrame() ?? '').toContain('first recallable prompt');
     });
     it('↑ on empty input scrolls one line instead of history', async () => {
+        // No prompts submitted yet → history is empty → viewport scrolls.
         const { stdin, lastFrame } = render(_jsx(App, { initialModel: "m", maxSteps: 10, cwd: "/test", onPrompt: async () => { }, onSlash: async () => { }, isFullscreen: true, initialTranscript: makeInitialTranscript(25) }));
         await new Promise((r) => setTimeout(r, 50));
         stdin.write(KEY_HOME);
@@ -256,6 +257,41 @@ describe('App', () => {
         await new Promise((r) => setTimeout(r, 30));
         expect(lastFrame() ?? '').toMatch(/MSG-00-tag/);
     });
+    it('↑ at live tail recalls history; scrolled up it scrolls (both kept)', async () => {
+        const onPrompt = vi.fn(async () => { });
+        const { stdin, lastFrame } = render(_jsx(App, { initialModel: "m", maxSteps: 10, cwd: "/test", onPrompt: onPrompt, onSlash: async () => { }, isFullscreen: true, initialTranscript: makeInitialTranscript(25) }));
+        await new Promise((r) => setTimeout(r, 50));
+        stdin.write('zzrecallme');
+        await new Promise((r) => setTimeout(r, 20));
+        stdin.write('\x0d');
+        await new Promise((r) => setTimeout(r, 50));
+        expect(onPrompt).toHaveBeenCalledWith('zzrecallme');
+        // At the live tail, empty input + ↑ recalls into the input line: typing
+        // '!' must extend the recalled text (only the input holds 'zzrecallme!').
+        stdin.write('\x1b[A');
+        await new Promise((r) => setTimeout(r, 30));
+        stdin.write('!');
+        await new Promise((r) => setTimeout(r, 30));
+        expect(lastFrame() ?? '').toContain('zzrecallme!');
+    });
+    it('↑ while scrolled up keeps scrolling and leaves input empty', async () => {
+        const { stdin, lastFrame } = render(_jsx(App, { initialModel: "m", maxSteps: 10, cwd: "/test", onPrompt: async () => { }, onSlash: async () => { }, isFullscreen: true, initialTranscript: makeInitialTranscript(25) }));
+        await new Promise((r) => setTimeout(r, 50));
+        stdin.write('zznoscroll');
+        await new Promise((r) => setTimeout(r, 20));
+        stdin.write('\x0d');
+        await new Promise((r) => setTimeout(r, 50));
+        stdin.write(KEY_HOME);
+        await new Promise((r) => setTimeout(r, 30));
+        // Scrolled up + empty input + ↑ → scroll (MSG-00 stays), input untouched:
+        // typing '!' must NOT extend the submitted prompt.
+        stdin.write('\x1b[A');
+        await new Promise((r) => setTimeout(r, 30));
+        expect(lastFrame() ?? '').toMatch(/MSG-00-tag/);
+        stdin.write('!');
+        await new Promise((r) => setTimeout(r, 30));
+        expect(lastFrame() ?? '').not.toContain('zznoscroll!');
+    });
     it('/c shows top-6 suggestions and Tab completes', async () => {
         const { stdin, lastFrame } = render(_jsx(App, { initialModel: "m", maxSteps: 10, cwd: "/test", onPrompt: async () => { }, onSlash: async () => { } }));
         stdin.write('/c');

package/dist/tui/mouse.d.ts

@@ -34,6 +34,16 @@ export declare class MouseFilter {
 }
 export declare const MOUSE_ENABLE = "\u001B[?1000h\u001B[?1006h";
 export declare const MOUSE_DISABLE = "\u001B[?1000l\u001B[?1006l";
+/**
+ * Whether the TUI may request terminal mouse reporting (wheel scrolling).
+ *
+ * Default OFF: with button reporting enabled the terminal routes
+ * selection clicks and right-click paste to the app (which swallows them),
+ * so native select-to-copy and right-click-paste break. Native selection
+ * works out of the box; set `KLYRO_MOUSE=1` to opt into wheel scrolling
+ * (Shift+drag still selects natively in most terminals).
+ */
+export declare function isMouseReportingEnabled(env?: Readonly<Record<string, string | undefined>>): boolean;
 export declare const PASTE_START = "\u001B[200~";
 export declare const PASTE_END = "\u001B[201~";
 export declare const PASTE_ENABLE = "\u001B[?2004h";

package/dist/tui/mouse.js

@@ -92,6 +92,18 @@ export class MouseFilter {
 }
 export const MOUSE_ENABLE = '\x1b[?1000h\x1b[?1006h'; // button events + SGR coords
 export const MOUSE_DISABLE = '\x1b[?1000l\x1b[?1006l';
+/**
+ * Whether the TUI may request terminal mouse reporting (wheel scrolling).
+ *
+ * Default OFF: with button reporting enabled the terminal routes
+ * selection clicks and right-click paste to the app (which swallows them),
+ * so native select-to-copy and right-click-paste break. Native selection
+ * works out of the box; set `KLYRO_MOUSE=1` to opt into wheel scrolling
+ * (Shift+drag still selects natively in most terminals).
+ */
+export function isMouseReportingEnabled(env = process.env) {
+    return env.KLYRO_MOUSE === '1';
+}
 export const PASTE_START = '\x1b[200~';
 export const PASTE_END = '\x1b[201~';
 export const PASTE_ENABLE = '\x1b[?2004h'; // bracketed paste: terminal wraps pastes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "klyro",
-  "version": "1.0.11",
+  "version": "1.0.13",
   "description": "Klyro — autonomous coding harness CLI that streams from any OpenAI-compatible or Anthropic LLM endpoint.",
   "type": "module",
   "main": "dist/index.js",