@phenx-inc/ctlsurf 0.1.9 → 0.1.10

package/out/main/index.js

@@ -3625,14 +3625,15 @@ function requireXtermHeadless() {
 var xtermHeadlessExports = requireXtermHeadless();
 class ConversationBridge {
   wsClient = null;
-  flushTimer = null;
-  flushIntervalMs = 3e3;
   sessionActive = false;
   inputBuffer = "";
+  bytesAccumulated = 0;
   // Virtual terminal for processing escape sequences into rendered text
   terminal;
   lastSnapshot = "";
-  pendingWrites = 0;
+  // Flush after accumulating enough data or on a timer
+  FLUSH_BYTES = 2e3;
+  flushTimer = null;
   constructor() {
     this.terminal = new xtermHeadlessExports.Terminal({ cols: 120, rows: 50, scrollback: 1e4 });
   }
@@ -3642,9 +3643,11 @@ class ConversationBridge {
   startSession() {
     this.terminal.reset();
     this.lastSnapshot = "";
-    this.pendingWrites = 0;
+    this.bytesAccumulated = 0;
     this.inputBuffer = "";
     this.sessionActive = true;
+    if (this.flushTimer) clearInterval(this.flushTimer);
+    this.flushTimer = setInterval(() => this.flush(), 5e3);
     console.log("[bridge] Session started");
   }
   /**
@@ -3654,17 +3657,11 @@ class ConversationBridge {
    */
   feedOutput(data) {
     if (!this.sessionActive) return;
-    this.pendingWrites++;
-    this.terminal.write(data, () => {
-      this.pendingWrites--;
-      this.scheduleFlush();
-    });
-  }
-  scheduleFlush() {
-    if (this.flushTimer) {
-      clearTimeout(this.flushTimer);
+    this.terminal.write(data);
+    this.bytesAccumulated += data.length;
+    if (this.bytesAccumulated >= this.FLUSH_BYTES) {
+      this.flush();
     }
-    this.flushTimer = setTimeout(() => this.flush(), this.flushIntervalMs);
   }
   feedInput(data) {
     if (!this.sessionActive) return;
@@ -3685,10 +3682,6 @@ class ConversationBridge {
    * Only flushes when all pending writes have completed.
    */
   flush() {
-    if (this.pendingWrites > 0) {
-      this.scheduleFlush();
-      return;
-    }
     const buf = this.terminal.buffer.active;
     const totalLines = buf.baseY + this.terminal.rows;
     const allLines = [];
@@ -3707,6 +3700,7 @@ class ConversationBridge {
     }
     this.lastSnapshot = currentSnapshot;
     const cleaned = newContent.replace(/\n{3,}/g, "\n\n").trim();
+    this.bytesAccumulated = 0;
     if (cleaned.length === 0) return;
     this.sendEntry("terminal_output", cleaned);
   }
@@ -3718,15 +3712,11 @@ class ConversationBridge {
       content
     });
   }
-  async endSession() {
+  endSession() {
     if (!this.sessionActive) return;
-    if (this.pendingWrites > 0) {
-      await new Promise((resolve) => setTimeout(resolve, 500));
-    }
-    this.pendingWrites = 0;
     this.flush();
     if (this.flushTimer) {
-      clearTimeout(this.flushTimer);
+      clearInterval(this.flushTimer);
       this.flushTimer = null;
     }
     this.sessionActive = false;
@@ -7855,7 +7845,7 @@ class Orchestrator {
   // ─── PTY & Agent ────────────────────────────────
   async spawnAgent(agent, cwd) {
     if (this.ptyManager) {
-      await this.bridge.endSession();
+      this.bridge.endSession();
       this.ptyManager.kill();
     }
     this.currentAgent = agent;
@@ -7873,7 +7863,7 @@ class Orchestrator {
     const thisPtyManager = this.ptyManager;
     this.ptyManager.onExit(async (exitCode) => {
       this.events.onPtyExit(exitCode);
-      await this.bridge.endSession();
+      this.bridge.endSession();
       if (thisPtyManager === this.ptyManager && this.currentAgent && isCodingAgent(this.currentAgent)) {
         this.workerWs.disconnect();
       }
@@ -7896,7 +7886,7 @@ class Orchestrator {
     this.workerWs.sendTerminalResize(cols, rows);
   }
   async killAgent() {
-    await this.bridge.endSession();
+    this.bridge.endSession();
     this.ptyManager?.kill();
     this.ptyManager = null;
     if (this.currentAgent && isCodingAgent(this.currentAgent)) {
@@ -7945,7 +7935,7 @@ class Orchestrator {
   }
   // ─── Shutdown ───────────────────────────────────
   async shutdown() {
-    await this.bridge.endSession();
+    this.bridge.endSession();
     this.ptyManager?.kill();
     this.ptyManager = null;
     this.workerWs.disconnect();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@phenx-inc/ctlsurf",
-  "version": "0.1.9",
+  "version": "0.1.10",
   "description": "Agent-agnostic terminal and desktop app for ctlsurf — run Claude Code, Codex, or any coding agent with live session logging and remote control",
   "main": "out/main/index.js",
   "bin": {

package/src/main/bridge.ts

@@ -11,15 +11,17 @@ import { WorkerWsClient } from './workerWs'
  */
 export class ConversationBridge {
   private wsClient: WorkerWsClient | null = null
-  private flushTimer: ReturnType<typeof setTimeout> | null = null
-  private flushIntervalMs: number = 3000
   private sessionActive: boolean = false
   private inputBuffer: string = ''
+  private bytesAccumulated: number = 0
 
   // Virtual terminal for processing escape sequences into rendered text
   private terminal: Terminal
   private lastSnapshot: string = ''
-  private pendingWrites: number = 0
+
+  // Flush after accumulating enough data or on a timer
+  private readonly FLUSH_BYTES = 2000
+  private flushTimer: ReturnType<typeof setInterval> | null = null
 
   constructor() {
     this.terminal = new Terminal({ cols: 120, rows: 50, scrollback: 10000 })
@@ -32,9 +34,14 @@ export class ConversationBridge {
   startSession(): void {
     this.terminal.reset()
     this.lastSnapshot = ''
-    this.pendingWrites = 0
+    this.bytesAccumulated = 0
     this.inputBuffer = ''
     this.sessionActive = true
+
+    // Periodic flush via setInterval (setTimeout may not fire under raw-mode TUI)
+    if (this.flushTimer) clearInterval(this.flushTimer)
+    this.flushTimer = setInterval(() => this.flush(), 5000)
+
     console.log('[bridge] Session started')
   }
 
@@ -46,18 +53,13 @@ export class ConversationBridge {
   feedOutput(data: string): void {
     if (!this.sessionActive) return
 
-    this.pendingWrites++
-    this.terminal.write(data, () => {
-      this.pendingWrites--
-      this.scheduleFlush()
-    })
-  }
+    this.terminal.write(data)
+    this.bytesAccumulated += data.length
 
-  private scheduleFlush(): void {
-    if (this.flushTimer) {
-      clearTimeout(this.flushTimer)
+    // Flush when enough data accumulated
+    if (this.bytesAccumulated >= this.FLUSH_BYTES) {
+      this.flush()
     }
-    this.flushTimer = setTimeout(() => this.flush(), this.flushIntervalMs)
   }
 
   feedInput(data: string): void {
@@ -82,11 +84,6 @@ export class ConversationBridge {
    * Only flushes when all pending writes have completed.
    */
   private flush(): void {
-    if (this.pendingWrites > 0) {
-      this.scheduleFlush()
-      return
-    }
-
     const buf = this.terminal.buffer.active
     // Read all lines: scrollback (baseY) + visible area (rows)
     const totalLines = buf.baseY + this.terminal.rows
@@ -114,6 +111,9 @@ export class ConversationBridge {
     this.lastSnapshot = currentSnapshot
 
     const cleaned = newContent.replace(/\n{3,}/g, '\n\n').trim()
+
+    this.bytesAccumulated = 0
+
     if (cleaned.length === 0) return
 
     this.sendEntry('terminal_output', cleaned)
@@ -128,19 +128,14 @@ export class ConversationBridge {
     })
   }
 
-  async endSession(): Promise<void> {
+  endSession(): void {
     if (!this.sessionActive) return
 
-    // If writes are pending, give xterm a moment to process them
-    if (this.pendingWrites > 0) {
-      await new Promise<void>(resolve => setTimeout(resolve, 500))
-    }
-
-    this.pendingWrites = 0
+    // Give xterm a moment to process any remaining writes
     this.flush()
 
     if (this.flushTimer) {
-      clearTimeout(this.flushTimer)
+      clearInterval(this.flushTimer)
       this.flushTimer = null
     }
 

package/src/main/orchestrator.ts

@@ -284,7 +284,7 @@ export class Orchestrator {
 
   async spawnAgent(agent: AgentConfig, cwd: string): Promise<void> {
     if (this.ptyManager) {
-      await this.bridge.endSession()
+      this.bridge.endSession()
       this.ptyManager.kill()
     }
 
@@ -307,7 +307,7 @@ export class Orchestrator {
 
     this.ptyManager.onExit(async (exitCode: number) => {
       this.events.onPtyExit(exitCode)
-      await this.bridge.endSession()
+      this.bridge.endSession()
       if (thisPtyManager === this.ptyManager && this.currentAgent && isCodingAgent(this.currentAgent)) {
         this.workerWs.disconnect()
       }
@@ -335,7 +335,7 @@ export class Orchestrator {
   }
 
   async killAgent(): Promise<void> {
-    await this.bridge.endSession()
+    this.bridge.endSession()
     this.ptyManager?.kill()
     this.ptyManager = null
     if (this.currentAgent && isCodingAgent(this.currentAgent)) {
@@ -391,7 +391,7 @@ export class Orchestrator {
   // ─── Shutdown ───────────────────────────────────
 
   async shutdown(): Promise<void> {
-    await this.bridge.endSession()
+    this.bridge.endSession()
     this.ptyManager?.kill()
     this.ptyManager = null
     this.workerWs.disconnect()