codeam-cli 2.23.34 → 2.23.36

package/CHANGELOG.md

@@ -4,6 +4,28 @@ All notable changes to `codeam-cli` are documented here.
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/), and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [2.23.35] — 2026-05-31
+
+### Changed
+
+- **cli:** Move bracketed-paste into Claude strategy + agent-leak hook
+
+### Fixed
+
+- **cli:** Wrap multi-line prompts in bracketed-paste so \r submits
+
+## [2.23.34] — 2026-05-31
+
+### Fixed
+
+- **cli:** Wrap multi-line prompts in bracketed-paste so \r submits
+
+## [2.23.33] — 2026-05-31
+
+### Performance
+
+- **cli:** Coalesce file-watcher emissions in a 250 ms window
+
 ## [2.23.32] — 2026-05-31
 
 ### Added

package/dist/index.js

@@ -441,7 +441,7 @@ var import_qrcode_terminal = __toESM(require("qrcode-terminal"));
 // package.json
 var package_default = {
   name: "codeam-cli",
-  version: "2.23.34",
+  version: "2.23.36",
   description: "Workflow-continuity bridge for AI coding agents. Wrap Claude Code or Codex in a PTY and supervise, approve, and redirect the session from any device \u2014 async. The terminal companion for CodeAgent Mobile.",
   type: "commonjs",
   main: "dist/index.js",
@@ -5774,7 +5774,7 @@ function readAnonId() {
 }
 function superProperties() {
   return {
-    cliVersion: true ? "2.23.34" : "0.0.0-dev",
+    cliVersion: true ? "2.23.36" : "0.0.0-dev",
     nodeVersion: process.version,
     platform: process.platform,
     arch: process.arch,
@@ -6307,21 +6307,13 @@ var AgentService = class _AgentService {
   /**
    * Write one prompt to the PTY and submit it.
    *
-   * For multi-line text (or any text > 1 line), Claude Code's TUI
-   * triggers bracketed-paste mode and treats the write as a paste:
-   * a `[Pasted text #N]` marker lands in the input field, and any
-   * `\r` we send while the paste boundary is still open is swallowed
-   * as part of the paste content — never reaching the submit
-   * handler. The result is the prompt sitting in the input forever
-   * (the user reported stacking `[Pasted text #N]` markers with no
-   * agent reply).
-   *
-   * Fix: explicitly bracket the paste ourselves (`ESC[200~ <text>
-   * ESC[201~`) so the end marker closes the paste deterministically.
-   * A short delay later we send `\r` — now OUTSIDE the bracket — so
-   * Claude's input handler treats it as Submit, not paste content.
-   * Single-line text doesn't trigger paste mode, so we keep the
-   * legacy `text + \r` path for that case (cheaper, no markers).
+   * Per-agent input wire formatting (e.g. Claude Code's bracketed-
+   * paste markers) lives in the runtime strategy's
+   * `prepareInputWrites(text)`. This method only knows the generic
+   * contract: emit N writes in order, wait `submitDelayMs`, then
+   * send `\r` to submit. The default (when the strategy doesn't
+   * override) is one `s.write(text)` followed by `\r` after 50 ms,
+   * which works for plain REPL agents (Aider).
    *
    * Marks the agent busy so subsequent `sendCommand` calls queue
    * instead of stacking pastes.
@@ -6331,14 +6323,12 @@ var AgentService = class _AgentService {
     const s = this.strategy;
     this.agentBusy = true;
     log.trace("agent", `submit text=${text.length}B (queued=${this.pendingInputs.length})`);
-    const isMultiline = text.includes("\n");
-    if (isMultiline) {
-      s.write(`\x1B[200~${text}\x1B[201~`);
-      setTimeout(() => s.write("\r"), 80);
-    } else {
-      s.write(text);
-      setTimeout(() => s.write("\r"), 50);
-    }
+    const prep = this.runtime.prepareInputWrites?.(text) ?? {
+      writes: [text],
+      submitDelayMs: 50
+    };
+    for (const w3 of prep.writes) s.write(w3);
+    setTimeout(() => s.write("\r"), prep.submitDelayMs);
   }
   drainPending() {
     if (!this.strategy || this.pendingInputs.length === 0) return;
@@ -9831,6 +9821,31 @@ var ClaudeRuntimeStrategy = class {
   constructor(os26) {
     this.os = os26;
   }
+  /**
+   * Claude Code's react-ink TUI enables bracketed-paste mode at
+   * boot (`ESC[?2004h`). When a multi-line write arrives, Claude
+   * opens a paste boundary that stays open until it sees the
+   * matching `ESC[201~` end marker. A naïve `text + \r` lands the
+   * `\r` INSIDE the open bracket as paste CONTENT, and the prompt
+   * sits in the input forever (the user's mobile COMMENT flow was
+   * stacking `[Pasted text #N]` markers with no submit).
+   *
+   * Fix: wrap multi-line prompts in the bracket markers ourselves
+   * so the end marker closes the paste deterministically. The
+   * caller emits `\r` 80 ms later — now OUTSIDE the bracket — and
+   * Claude's input handler treats it as a normal Submit. Single-
+   * line prompts don't trigger paste mode so we keep the bare
+   * `text + \r` path for that case.
+   */
+  prepareInputWrites(text) {
+    if (text.includes("\n")) {
+      return {
+        writes: [`\x1B[200~${text}\x1B[201~`],
+        submitDelayMs: 80
+      };
+    }
+    return { writes: [text], submitDelayMs: 50 };
+  }
   async prepareLaunch() {
     const sessionId = (0, import_node_crypto4.randomUUID)();
     const sessionArgs = ["--session-id", sessionId];
@@ -11732,6 +11747,21 @@ var PtyBuffer = class {
     this.raw = "";
     this.lastPushAt = 0;
   }
+  /**
+   * Re-arm the terminal-input detector without flipping `active`.
+   * Called by the orchestrator when a pending terminal-turn signal
+   * doesn't pan out — the JSONL polling timed out without finding a
+   * new user message, so whatever produced the printable byte (most
+   * commonly Claude's ghost-text completion painted ~300-500 ms after
+   * the previous turn settled) was NOT a real user keystroke. We
+   * need detection to fire again on the next legitimate keystroke,
+   * otherwise the human's eventual CLI prompt is lost and only the
+   * agent's reply lands on mobile (the duplicate-response /
+   * missing-user-bubble class of bug the user reported).
+   */
+  resetTerminalInputGate() {
+    this.terminalInputPending = false;
+  }
   /**
    * Ingest a raw PTY frame. Always accumulates so cold-startup
    * frames aren't lost. Returns whether the buffer was active at
@@ -11916,6 +11946,22 @@ var OutputService = class _OutputService {
     }
     await this.send({ type: "new_turn", done: false }, { critical: true });
   }
+  /**
+   * Re-arm the terminal-turn detection gate without opening a bogus
+   * turn. Called by the orchestrator when `waitForNewUserMessage`
+   * times out — Claude's ghost-text completion (painted between
+   * turns) trips the printable-byte detector before the user has
+   * actually typed anything. Calling `startTerminalTurn` here would
+   * emit `clear` + `new_turn` to mobile and then `beginTurn` would
+   * activate the PTY buffer; the tick poll would render the still-
+   * visible previous response as fresh `text` chunks → a duplicate
+   * agent bubble on mobile. Resetting the gates instead means the
+   * next legitimate keystroke re-fires detection cleanly.
+   */
+  resetTerminalTurnGate() {
+    this.terminalTurnPending = false;
+    this.pty.resetTerminalInputGate();
+  }
   /**
    * Begin a turn after a `resume_session` request. Includes the
    * `resumedSessionId` so the client wipes its history and
@@ -12403,7 +12449,7 @@ var HistoryService = class _HistoryService {
    * Poll the JSONL until a new user message appears after previousCount entries.
    * Returns the text of the new user message, or null if not found within timeoutMs.
    */
-  async waitForNewUserMessage(previousCount, timeoutMs = 4e3) {
+  async waitForNewUserMessage(previousCount, timeoutMs = 6e4) {
     const deadline = Date.now() + timeoutMs;
     while (Date.now() < deadline) {
       if (!this.currentConversationId) return null;
@@ -16348,7 +16394,13 @@ async function start(requestedAgent) {
     },
     () => {
       const prevCount = historySvc.getCurrentMessageCount();
-      historySvc.waitForNewUserMessage(prevCount).then((userText) => outputSvc.startTerminalTurn(userText ?? void 0)).catch(() => outputSvc.startTerminalTurn(void 0));
+      historySvc.waitForNewUserMessage(prevCount).then((userText) => {
+        if (userText) {
+          void outputSvc.startTerminalTurn(userText);
+        } else {
+          outputSvc.resetTerminalTurnGate();
+        }
+      }).catch(() => outputSvc.resetTerminalTurnGate());
     },
     session.pluginAuthToken,
     runtime
@@ -19192,7 +19244,7 @@ function checkChokidar() {
 }
 async function doctor(args2 = []) {
   const json = args2.includes("--json");
-  const cliVersion = true ? "2.23.34" : "0.0.0-dev";
+  const cliVersion = true ? "2.23.36" : "0.0.0-dev";
   const apiBase = resolveApiBaseUrl();
   const diagnosticId = (0, import_node_crypto6.randomUUID)();
   log.info("doctor", `run id=${diagnosticId} cli=${cliVersion}`);
@@ -19391,7 +19443,7 @@ async function completion(args2) {
 // src/commands/version.ts
 var import_picocolors13 = __toESM(require("picocolors"));
 function version2() {
-  const v = true ? "2.23.34" : "unknown";
+  const v = true ? "2.23.36" : "unknown";
   console.log(`${import_picocolors13.default.bold("codeam-cli")} ${import_picocolors13.default.cyan(v)}`);
 }
 
@@ -19619,7 +19671,7 @@ function checkForUpdates() {
   if (process.env.CODEAM_DISABLE_UPDATE_CHECK === "1") return;
   if (process.env.CI) return;
   if (!process.stdout.isTTY) return;
-  const current = true ? "2.23.34" : null;
+  const current = true ? "2.23.36" : null;
   if (!current) return;
   const cache = readCache();
   const fresh = cache && Date.now() - cache.fetchedAt < TTL_MS;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codeam-cli",
-  "version": "2.23.34",
+  "version": "2.23.36",
   "description": "Workflow-continuity bridge for AI coding agents. Wrap Claude Code or Codex in a PTY and supervise, approve, and redirect the session from any device — async. The terminal companion for CodeAgent Mobile.",
   "type": "commonjs",
   "main": "dist/index.js",