@ouro.bot/cli 0.1.0-alpha.261 → 0.1.0-alpha.263

package/changelog.json

@@ -1,6 +1,18 @@
 {
   "_note": "This changelog is maintained as part of the PR/version-bump workflow. Agent-curated, not auto-generated. Agents read this file directly via read_file to understand what changed between versions.",
   "versions": [
+    {
+      "version": "0.1.0-alpha.263",
+      "changes": [
+        "fix(tui): arrow keys char-by-char (Ink 3.2 key.meta bug), Option+Arrow word-jump via CSI sequences."
+      ]
+    },
+    {
+      "version": "0.1.0-alpha.262",
+      "changes": [
+        "test(daemon): inject explicit `vi.mock(\"...heart/daemon/socket-client\", ...)` blocks into all 40 grandfathered test files in `TESTAGENT_NO_MOCK_ALLOWLIST`. The runtime guard in `socket-client.ts` already prevents real socket leaks under vitest, but the explicit mocks let those tests assert call counts cleanly and shrink the contract-test allowlist to zero. Both the testagent and the bundle-write allowlists in `test-isolation.contract.test.ts` are now empty Sets \u2014 any new offender fails the build."
+      ]
+    },
     {
       "version": "0.1.0-alpha.261",
       "changes": [

package/dist/senses/cli/input-keys.js

@@ -119,6 +119,12 @@ function classifyEscapeSequence(inputChar) {
     // End: \x1b[F or \x1b[4~
     if (seq === "F" || seq === "4~")
         return "end";
+    // Option+Left: \x1b[1;3D (CSI modified arrow with Alt modifier)
+    if (seq === "1;3D")
+        return "word-left";
+    // Option+Right: \x1b[1;3C
+    if (seq === "1;3C")
+        return "word-right";
     // PageUp: \x1b[5~, PageDown: \x1b[6~
     if (seq === "5~" || seq === "6~")
         return "ignore";

package/dist/senses/cli/ouro-tui.js

@@ -324,39 +324,20 @@ function InputArea({ onSubmit, onCtrlC, history, queuedInputs, onPopQueue, agent
             historyIdx.current = -1;
             return;
         }
-        // Left/right arrow: move cursor
+        // Left/right arrow: move cursor (char by char, token-aware)
+        // NOTE: Ink 3.2 bug — key.meta is ALWAYS true for arrow keys because all
+        // arrows start with \x1b and Ink sets meta=true for any ESC-prefixed input.
+        // Word-jump is handled via Meta+B/F (emacs bindings) instead.
         if (key.leftArrow) {
-            if (key.meta) {
-                // Option+Left: jump to previous word boundary
-                const before = inputRef.current.slice(0, cursorRef.current);
-                const match = before.match(/(?:^|\s)\S+\s*$/);
-                const newPos = match ? cursorRef.current - match[0].length + (match[0].startsWith(" ") ? 1 : 0) : 0;
-                cursorRef.current = Math.max(0, newPos);
-                setCursorPos(cursorRef.current);
-            }
-            else {
-                // Token-aware: hop over image ref chips
-                const chipStart = (0, image_ref_navigation_1.imageRefEndingAt)(inputRef.current, cursorRef.current);
-                cursorRef.current = chipStart !== undefined ? chipStart : Math.max(0, cursorRef.current - 1);
-                setCursorPos(cursorRef.current);
-            }
+            const chipStart = (0, image_ref_navigation_1.imageRefEndingAt)(inputRef.current, cursorRef.current);
+            cursorRef.current = chipStart !== undefined ? chipStart : Math.max(0, cursorRef.current - 1);
+            setCursorPos(cursorRef.current);
             return;
         }
         if (key.rightArrow) {
-            if (key.meta) {
-                // Option+Right: jump to next word boundary
-                const after = inputRef.current.slice(cursorRef.current);
-                const match = after.match(/^\s*\S+/);
-                const newPos = match ? cursorRef.current + match[0].length : inputRef.current.length;
-                cursorRef.current = Math.min(inputRef.current.length, newPos);
-                setCursorPos(cursorRef.current);
-            }
-            else {
-                // Token-aware: hop over image ref chips
-                const chipEnd = (0, image_ref_navigation_1.imageRefStartingAt)(inputRef.current, cursorRef.current);
-                cursorRef.current = chipEnd !== undefined ? chipEnd : Math.min(inputRef.current.length, cursorRef.current + 1);
-                setCursorPos(cursorRef.current);
-            }
+            const chipEnd = (0, image_ref_navigation_1.imageRefStartingAt)(inputRef.current, cursorRef.current);
+            cursorRef.current = chipEnd !== undefined ? chipEnd : Math.min(inputRef.current.length, cursorRef.current + 1);
+            setCursorPos(cursorRef.current);
             return;
         }
         // Up/Down: queue pop takes priority over history
@@ -542,22 +523,38 @@ function InputArea({ onSubmit, onCtrlC, history, queuedInputs, onPopQueue, agent
         // ─── Non-kill/non-yank keystroke resets ────────────────────────
         killRing.resetAccumulation();
         killRing.resetYankState();
+        // Detect raw escape sequences — Ink 3.2 strips \x1b prefix and sets key.meta,
+        // so we re-prepend it for classification when meta is set and inputChar starts with [
+        const escInput = key.meta && inputChar.startsWith("[") ? "\x1b" + inputChar : inputChar;
+        const escClass = (0, input_keys_1.classifyEscapeSequence)(escInput);
+        if (escClass === "home") {
+            cursorRef.current = (0, input_keys_1.handleHome)(inputRef.current, cursorRef.current);
+            setCursorPos(cursorRef.current);
+            return;
+        }
+        if (escClass === "end") {
+            cursorRef.current = (0, input_keys_1.handleEnd)(inputRef.current, cursorRef.current);
+            setCursorPos(cursorRef.current);
+            return;
+        }
+        if (escClass === "word-left") {
+            const before = inputRef.current.slice(0, cursorRef.current);
+            const match = before.match(/(?:^|\s)\S+\s*$/);
+            cursorRef.current = match ? Math.max(0, cursorRef.current - match[0].length + (match[0].startsWith(" ") ? 1 : 0)) : 0;
+            setCursorPos(cursorRef.current);
+            return;
+        }
+        if (escClass === "word-right") {
+            const after = inputRef.current.slice(cursorRef.current);
+            const match = after.match(/^\s*\S+/);
+            cursorRef.current = match ? Math.min(inputRef.current.length, cursorRef.current + match[0].length) : inputRef.current.length;
+            setCursorPos(cursorRef.current);
+            return;
+        }
+        if (escClass === "ignore")
+            return;
         // Regular character: insert at cursor position
         if (!key.ctrl && !key.meta && inputChar) {
-            // Detect raw escape sequences before inserting text
-            const escClass = (0, input_keys_1.classifyEscapeSequence)(inputChar);
-            if (escClass === "home") {
-                cursorRef.current = (0, input_keys_1.handleHome)(inputRef.current, cursorRef.current);
-                setCursorPos(cursorRef.current);
-                return;
-            }
-            if (escClass === "end") {
-                cursorRef.current = (0, input_keys_1.handleEnd)(inputRef.current, cursorRef.current);
-                setCursorPos(cursorRef.current);
-                return;
-            }
-            if (escClass === "ignore")
-                return; // PageUp/PageDown/mouse wheel fallback
             const before = inputRef.current.slice(0, cursorRef.current);
             const after = inputRef.current.slice(cursorRef.current);
             updateInput(before + inputChar + after, cursorRef.current + 1);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ouro.bot/cli",
-  "version": "0.1.0-alpha.261",
+  "version": "0.1.0-alpha.263",
   "main": "dist/heart/daemon/ouro-entry.js",
   "bin": {
     "cli": "dist/heart/daemon/ouro-bot-entry.js",