codeam-cli 1.0.8 → 1.1.0

package/dist/index.js

@@ -110,7 +110,7 @@ var import_picocolors = __toESM(require("picocolors"));
 // package.json
 var package_default = {
   name: "codeam-cli",
-  version: "1.0.8",
+  version: "1.1.0",
   description: "Remote control Claude Code from your mobile device",
   main: "dist/index.js",
   bin: {
@@ -510,13 +510,70 @@ var CommandRelayService = class {
 // src/services/claude.service.ts
 var import_child_process = require("child_process");
 var fs2 = __toESM(require("fs"));
+var os3 = __toESM(require("os"));
 var path2 = __toESM(require("path"));
+var PYTHON_PTY_HELPER = `import os,pty,sys,select,signal,struct,fcntl,termios,errno
+m,s=pty.openpty()
+try:
+    cols=int(os.environ.get('COLUMNS','220'))
+    rows=int(os.environ.get('LINES','50'))
+    fcntl.ioctl(s,termios.TIOCSWINSZ,struct.pack('HHHH',rows,cols,0,0))
+except Exception:pass
+pid=os.fork()
+if pid==0:
+    os.close(m)
+    os.setsid()
+    try:fcntl.ioctl(s,termios.TIOCSCTTY,0)
+    except Exception:pass
+    for fd in[0,1,2]:os.dup2(s,fd)
+    if s>2:os.close(s)
+    os.execvp(sys.argv[1],sys.argv[1:])
+    sys.exit(127)
+os.close(s)
+done=[False]
+def onchld(n,f):
+    try:os.waitpid(pid,os.WNOHANG)
+    except Exception:pass
+    done[0]=True
+def onwinch(n,f):
+    try:
+        sz=os.get_terminal_size(2)
+        fcntl.ioctl(m,termios.TIOCSWINSZ,struct.pack('HHHH',sz.lines,sz.columns,0,0))
+    except Exception:pass
+signal.signal(signal.SIGCHLD,onchld)
+signal.signal(signal.SIGWINCH,onwinch)
+i=sys.stdin.fileno()
+o=sys.stdout.fileno()
+while not done[0]:
+    try:r,_,_=select.select([i,m],[],[],0.1)
+    except OSError as e:
+        if e.errno==errno.EINTR:continue
+        break
+    if i in r:
+        try:
+            d=os.read(i,4096)
+            if d:os.write(m,d)
+            else:break
+        except OSError:break
+    if m in r:
+        try:
+            d=os.read(m,4096)
+            if d:os.write(o,d)
+        except OSError:done[0]=True
+try:os.kill(pid,signal.SIGTERM)
+except Exception:pass
+try:
+    _,st=os.waitpid(pid,0)
+    sys.exit((st>>8)&0xFF)
+except Exception:sys.exit(0)
+`;
 var ClaudeService = class {
   constructor(opts) {
     this.opts = opts;
   }
   opts;
   proc = null;
+  helperPath = null;
   spawn() {
     if (this.proc) {
       this.cleanup();
@@ -537,20 +594,24 @@ var ClaudeService = class {
     }
   }
   /**
-   * macOS / Linux: wrap Claude in `script` to give it a real PTY.
-   *
-   * Why: Claude Code (React Ink) checks stdin.isTTY. When stdin is a pipe it
-   * shows a "no stdin data in 3s" warning and then stops reading stdin entirely.
-   * `script` creates a PTY for the child so Claude sees stdin as a terminal,
-   * while the parent (`codeam`) still controls script's stdin/stdout as pipes —
-   * enabling mobile command injection and output capture.
+   * macOS / Linux: use a Python PTY helper to give Claude a real TTY.
+   * Falls back to direct spawn if python3 is not available.
    */
   spawnWithPty(claudeCmd) {
+    const python = findInPath("python3") ?? findInPath("python");
+    if (!python) {
+      console.error(
+        "  \xB7 python3 not found; mobile command injection may be limited.\n"
+      );
+      this.spawnDirect(claudeCmd);
+      return;
+    }
     const shell = process.env.SHELL || "/bin/sh";
     const cols = process.stdout.columns || 220;
     const rows = process.stdout.rows || 50;
-    const scriptArgs = buildScriptArgs(shell, claudeCmd);
-    this.proc = (0, import_child_process.spawn)("script", scriptArgs, {
+    this.helperPath = path2.join(os3.tmpdir(), "codeam-pty-helper.py");
+    fs2.writeFileSync(this.helperPath, PYTHON_PTY_HELPER, { mode: 420 });
+    this.proc = (0, import_child_process.spawn)(python, [this.helperPath, shell, "-c", `exec ${claudeCmd}`], {
       stdio: ["pipe", "pipe", "inherit"],
       cwd: this.opts.cwd,
       env: {
@@ -578,14 +639,14 @@ var ClaudeService = class {
     process.stdin.on("data", this.stdinHandler);
     process.on("SIGWINCH", this.handleResize);
     this.proc.on("exit", (code) => {
+      this.removeTempFile();
       this.cleanup();
       this.opts.onExit(code ?? 0);
     });
   }
   /**
-   * Windows fallback: direct spawn without PTY.
-   * Mobile command injection is limited (Claude may ignore pipe stdin after 3 s),
-   * but the terminal session still works for manual use.
+   * Windows (or Python-unavailable) fallback: direct spawn without PTY.
+   * Mobile command injection is limited on Windows.
    */
   spawnDirect(claudeCmd) {
     this.proc = (0, import_child_process.spawn)(claudeCmd, [], {
@@ -623,6 +684,7 @@ var ClaudeService = class {
   }
   kill() {
     this.proc?.kill();
+    this.removeTempFile();
     this.cleanup();
   }
   stdinHandler = (chunk) => {
@@ -646,15 +708,16 @@ var ClaudeService = class {
       }
     }
   }
-};
-function buildScriptArgs(shell, claudeCmd) {
-  const innerCmd = `exec ${claudeCmd}`;
-  if (process.platform === "darwin") {
-    return ["-q", "/dev/null", shell, "-c", innerCmd];
+  removeTempFile() {
+    if (this.helperPath) {
+      try {
+        fs2.unlinkSync(this.helperPath);
+      } catch {
+      }
+      this.helperPath = null;
+    }
   }
-  const shellCmd = `${shell} -c ${JSON.stringify(innerCmd)}`;
-  return ["-q", "-e", "-c", shellCmd, "/dev/null"];
-}
+};
 function findInPath(name) {
   const dirs = (process.env.PATH ?? "").split(path2.delimiter);
   for (const dir of dirs) {
@@ -682,68 +745,87 @@ var OutputService = class _OutputService {
   }
   sessionId;
   pluginId;
+  // Buffer that accumulates PTY text (ANSI stripped, growing during response)
   buffer = "";
-  flushTimer = null;
-  streamTimer = null;
+  lastSentBuffer = "";
+  stableCount = 0;
+  pollTimer = null;
+  startTime = 0;
   active = false;
-  static STREAM_INTERVAL_MS = 1e3;
-  static DONE_DEBOUNCE_MS = 2500;
+  // After this many stable polls (POLL_MS each) with content, mark done
+  static POLL_MS = 1e3;
+  static STABLE_THRESHOLD = 3;
+  // Give up after 2 minutes (absolute) or 30 seconds with no content at all
+  static MAX_MS = 12e4;
+  static EMPTY_TIMEOUT_MS = 3e4;
   /**
    * Call before sending a command from mobile.
-   * Clears previous output and pushes a new_turn event so the mobile app
-   * shows the typing indicator.
+   * Clears previous output, sends new_turn event, starts polling.
    */
-  async newTurn() {
-    this.stopTimers();
+  newTurn() {
+    this.stopPoll();
     this.buffer = "";
+    this.lastSentBuffer = "";
+    this.stableCount = 0;
     this.active = true;
-    await this.postChunk({ clear: true });
-    await this.postChunk({ type: "new_turn", content: "", done: false });
-    this.startStreamTimer();
+    this.startTime = Date.now();
+    this.postChunk({ clear: true }).then(() => this.postChunk({ type: "new_turn", content: "", done: false })).catch(() => {
+    });
+    this.pollTimer = setInterval(() => this.checkStability(), _OutputService.POLL_MS);
   }
   /** Feed raw terminal output from Claude (called on every stdout chunk). */
   push(raw) {
     if (!this.active) return;
     const text = stripAnsi(raw).replace(/[ \t]+\n/g, "\n").replace(/\n{3,}/g, "\n\n");
-    if (!text.trim()) return;
-    this.buffer += text;
-    this.resetDoneDebounce();
+    if (text) this.buffer += text;
   }
   dispose() {
-    this.stopTimers();
+    this.stopPoll();
     this.active = false;
   }
-  startStreamTimer() {
-    this.streamTimer = setInterval(() => {
-      const text = this.buffer;
-      if (text.trim()) {
-        this.postChunk({ type: "text", content: text, done: false }).catch(() => {
-        });
+  /**
+   * Stability check — runs every POLL_MS while active.
+   *
+   * Only finalises when we have content that has stopped changing.
+   * This avoids premature done signals while Claude is still thinking
+   * (which would set active=false and silently drop the real response).
+   */
+  checkStability() {
+    if (!this.active) return;
+    if (Date.now() - this.startTime > _OutputService.MAX_MS) {
+      this.finalize();
+      return;
+    }
+    const current = this.buffer;
+    if (!current.trim()) {
+      if (Date.now() - this.startTime > _OutputService.EMPTY_TIMEOUT_MS) {
+        this.finalize();
       }
-    }, _OutputService.STREAM_INTERVAL_MS);
-  }
-  resetDoneDebounce() {
-    if (this.flushTimer) clearTimeout(this.flushTimer);
-    this.flushTimer = setTimeout(() => {
-      this.flushTimer = null;
-      this.stopTimers();
-      const text = this.buffer;
-      this.buffer = "";
-      this.active = false;
-      if (text.trim()) {
-        this.postChunk({ type: "text", content: text, done: true }).catch(() => {
-        });
+      return;
+    }
+    if (current === this.lastSentBuffer) {
+      this.stableCount++;
+      if (this.stableCount >= _OutputService.STABLE_THRESHOLD) {
+        this.finalize();
       }
-    }, _OutputService.DONE_DEBOUNCE_MS);
-  }
-  stopTimers() {
-    if (this.flushTimer) {
-      clearTimeout(this.flushTimer);
-      this.flushTimer = null;
+    } else {
+      this.stableCount = 0;
+      this.lastSentBuffer = current;
+      this.postChunk({ type: "text", content: current, done: false }).catch(() => {
+      });
     }
-    if (this.streamTimer) {
-      clearInterval(this.streamTimer);
-      this.streamTimer = null;
+  }
+  finalize() {
+    const text = this.buffer;
+    this.stopPoll();
+    this.active = false;
+    this.postChunk({ type: "text", content: text, done: true }).catch(() => {
+    });
+  }
+  stopPoll() {
+    if (this.pollTimer) {
+      clearInterval(this.pollTimer);
+      this.pollTimer = null;
     }
   }
   postChunk(body) {
@@ -799,8 +881,7 @@ async function start() {
   const ws = new WebSocketService(session.id, pluginId);
   const outputSvc = new OutputService(session.id, pluginId);
   function sendPrompt(prompt) {
-    outputSvc.newTurn().catch(() => {
-    });
+    outputSvc.newTurn();
     claude.sendCommand(prompt);
   }
   const relay = new CommandRelayService(pluginId, (cmd) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codeam-cli",
-  "version": "1.0.8",
+  "version": "1.1.0",
   "description": "Remote control Claude Code from your mobile device",
   "main": "dist/index.js",
   "bin": {