omniharness-cli 0.1.86 → 0.1.87

package/dist/ui/keys.js

@@ -102,10 +102,22 @@ export function parseRawKey(chunk) {
     return null;
 }
 /**
- * True when `value` is a CSI-u encoded key or protocol response (kitty key
- * events, push/pop echoes, query answers), never plain text to insert.
+ * True when `value` is a CSI-u encoded key, a protocol response (kitty key
+ * events, push/pop echoes, query answers), or a DECRQM mode-report reply
+ * (`CSI ? Ps ; Pm $ y`, e.g. the answer to the synchronized-output query) —
+ * never plain text to insert.
+ *
+ * `value` here is Ink's own parsed form, which strips the leading ESC before
+ * handing the rest to `useInput`. A terminal's reply to a query this project
+ * sends (synchronized output, in particular) arrives on the same stdin the
+ * editor reads from, and if nothing recognises it in this stripped form it is
+ * treated as pasted text and typed into whatever the user was editing — the
+ * escape sequence itself, landing in the input box. Some terminals are slow
+ * to answer that specific query, so this has to keep matching however late
+ * the reply arrives, not just during a short probe window at startup.
  */
 export function isEncodedKey(value) {
-    return /^\[\d+(?:;\d+)*u$|^\[[><?]\d+(?:;\d+)*u$/.test(value);
+    return /^\[\d+(?:;\d+)*u$|^\[[><?]\d+(?:;\d+)*u$/.test(value)
+        || /^\[\?\d+;\d\$y$/.test(value);
 }
 //# sourceMappingURL=keys.js.map

package/dist/ui/terminalInterface.js

@@ -334,22 +334,41 @@ export function TerminalInterface({ engine }) {
         stdout.on('resize', onResize);
         stdout.write(KITTY_PUSH);
         let kittyTimer;
-        // One probe pass: kitty keyboard protocol + synchronized output (DECSET 2026).
-        const onProbe = (chunk) => {
-            const text = chunk.toString();
-            if (isSyncOutputReply(text) && syncRestoreRef.current === null) {
-                syncRestoreRef.current = wrapSynchronizedOutput(stdout);
-            }
-            if (!isKittyQueryResponse(text))
+        // Two independent probes, not one shared listener. They used to be a
+        // single handler torn down together after 300ms, on the assumption both
+        // replies would have arrived by then. Some terminals answer the
+        // synchronized-output query well after that — observed landing right
+        // around a resize, though the terminal's own reasons for the delay are
+        // opaque from here — and a listener already removed cannot catch a late
+        // reply. Two things followed from that: synchronized output silently
+        // never turned on for the rest of the session, and the reply itself, with
+        // nothing left to claim it, reached Ink's own useInput and got typed into
+        // the input box as literal text (the isEncodedKey guard now recognises
+        // and drops that shape regardless, but the actual fix is to keep
+        // listening for it rather than lean on that alone).
+        const onSyncProbe = (chunk) => {
+            if (syncRestoreRef.current !== null)
+                return;
+            if (!isSyncOutputReply(chunk.toString()))
+                return;
+            syncRestoreRef.current = wrapSynchronizedOutput(stdout);
+            stdin?.off('data', onSyncProbe);
+        };
+        // The kitty check is a UI decision — whether Ctrl+letter and a distinct
+        // Shift+Tab are available — that has to resolve one way or the other, so
+        // it keeps a bounded timeout; "no answer in 300ms" is itself the answer.
+        const onKittyProbe = (chunk) => {
+            if (!isKittyQueryResponse(chunk.toString()))
                 return;
             if (kittyTimer)
                 clearTimeout(kittyTimer);
-            stdin?.off('data', onProbe);
+            stdin?.off('data', onKittyProbe);
             setKitty(true);
         };
         if (stdin) {
-            stdin.on('data', onProbe);
-            kittyTimer = setTimeout(() => { setKitty(false); stdin.off('data', onProbe); }, 300);
+            stdin.on('data', onSyncProbe);
+            stdin.on('data', onKittyProbe);
+            kittyTimer = setTimeout(() => { setKitty(false); stdin.off('data', onKittyProbe); }, 300);
             stdout.write(SYNC_QUERY);
             stdout.write(KITTY_QUERY);
         }
@@ -432,7 +451,8 @@ export function TerminalInterface({ engine }) {
         return () => {
             if (kittyTimer)
                 clearTimeout(kittyTimer);
-            stdin?.off('data', onProbe);
+            stdin?.off('data', onSyncProbe);
+            stdin?.off('data', onKittyProbe);
             stdout.off('resize', onResize); // resizeDebounce cancels the coalesced timer on .off itself
             const pendingApproval = approvalResolve.current;
             approvalResolve.current = null;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "omniharness-cli",
-  "version": "0.1.86",
+  "version": "0.1.87",
   "description": "OmniHarness — local-first agent orchestration harness for OmniRoute.",
   "license": "MIT",
   "type": "module",