klyro 1.0.12 → 1.0.14

package/dist/cli/repl.js

@@ -2689,7 +2689,7 @@ 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 · ↑/↓ input history · PgUp/Dn scroll (KLYRO_MOUSE=1 adds wheel ±3 lines)',
+                        '  Ctrl+B/F page · Shift/Ctrl+↑/↓ line · ↑ history at live tail, scrolls while reading above · Ctrl+P/N history always · PgUp/Dn or Ctrl+U/D 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'),
                 });

package/dist/tui/app.js

@@ -1052,24 +1052,34 @@ export function App(props) {
             void props.onSlash({ kind: 'quit' });
             return;
         }
-        // Contextual ↑/↓ (§8.3): history always wins when entries exist —
-        // empty input + ↑ recalls the last prompt (standard REPL behavior),
-        // typing filters by prefix is unnecessary so plain recall applies;
-        // the viewport scrolls only when there is no history to show.
-        if (key.upArrow && !key.shift && !key.ctrl) {
+        // 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. Ctrl+P / Ctrl+N
+        // are escape-free aliases for history prev/next: single-byte codes that
+        // always browse, even on terminals that mangle arrow-key sequences.
+        // ↓ is otherwise unchanged: browse newer while browsing, else scroll
+        // toward the tail.
+        const wantHistPrev = (key.upArrow && !key.shift && !key.ctrl) || (key.ctrl && inputStr === 'p');
+        const wantHistNext = (key.downArrow && !key.shift && !key.ctrl) || (key.ctrl && inputStr === 'n');
+        if (wantHistPrev) {
+            if (!key.ctrl && 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) {
+            else if (!key.ctrl && isFullscreen && maxTop > 0) {
                 commands.lineUp();
-                return;
             }
+            return;
         }
-        if (key.downArrow && !key.shift && !key.ctrl) {
+        if (wantHistNext) {
             if (histIdx !== null) {
                 const next = histIdx + 1;
                 if (next >= history.length) {
@@ -1083,8 +1093,8 @@ export function App(props) {
                 setVimCursor(null);
                 return;
             }
-            if (input.trim() !== '')
-                return; // single line with text, nothing newer
+            if (input.trim() !== '' || key.ctrl)
+                return; // text present, or Ctrl+N: nothing newer
             if (isFullscreen && maxTop > 0) {
                 commands.lineDown();
                 return;

package/dist/tui/app.test.js

@@ -257,29 +257,69 @@ describe('App', () => {
         await new Promise((r) => setTimeout(r, 30));
         expect(lastFrame() ?? '').toMatch(/MSG-00-tag/);
     });
-    it('↑ on empty input recalls history newest-first; ↓ browses back (user report)', async () => {
+    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('Ctrl+P / Ctrl+N browse history without arrow keys', async () => {
         const onPrompt = vi.fn(async () => { });
         const { stdin, lastFrame } = render(_jsx(App, { initialModel: "m", maxSteps: 10, cwd: "/test", onPrompt: onPrompt, onSlash: async () => { } }));
-        stdin.write('hi');
+        stdin.write('zzfirst');
         await new Promise((r) => setTimeout(r, 20));
         stdin.write('\x0d');
         await new Promise((r) => setTimeout(r, 50));
-        stdin.write('second prompt');
+        stdin.write('zzsecond');
         await new Promise((r) => setTimeout(r, 20));
         stdin.write('\x0d');
         await new Promise((r) => setTimeout(r, 50));
         expect(onPrompt).toHaveBeenCalledTimes(2);
-        // Input is empty after each submit; plain ↑ must recall, not scroll.
-        stdin.write('\x1b[A');
+        // Ctrl+P recalls newest-first with zero escape sequences involved.
+        stdin.write('\x10');
         await new Promise((r) => setTimeout(r, 30));
-        expect(lastFrame() ?? '').toContain('second prompt');
-        stdin.write('\x1b[A');
+        stdin.write('!');
+        await new Promise((r) => setTimeout(r, 30));
+        expect(lastFrame() ?? '').toContain('zzsecond!');
+        // Ctrl+P twice walks older with no typing between, Ctrl+N walks newer.
+        stdin.write('\x10');
+        await new Promise((r) => setTimeout(r, 30));
+        stdin.write('\x10');
+        await new Promise((r) => setTimeout(r, 30));
+        stdin.write('\x0e');
         await new Promise((r) => setTimeout(r, 30));
-        expect(lastFrame() ?? '').toContain('hi');
-        // ↓ walks back toward newer entries.
-        stdin.write('\x1b[B');
+        stdin.write('#');
         await new Promise((r) => setTimeout(r, 30));
-        expect(lastFrame() ?? '').toContain('second prompt');
+        expect(lastFrame() ?? '').toContain('zzsecond#');
     });
     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 () => { } }));

package/package.json

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