pty-manager 1.3.1 → 1.3.3

package/README.md

@@ -445,7 +445,9 @@ await session.selectMenuOption(2);  // Sends Down, Down, Enter with 50ms delays
 
 ## Stall Detection & Task Completion
 
-Content-based stall detection monitors sessions for output that stops changing. The content hash normalizes ANSI escape codes, TUI spinner characters, and countdown/duration text (e.g. `8m 17s` → constant) so that live timers don't perpetually reset the stall timer. When a stall is detected, the session first tries the adapter's `detectTaskComplete()` fast-path. If the adapter recognizes the output as a completed task (e.g. duration summary + idle prompt), it transitions directly to `ready` and emits `task_complete` — skipping the expensive LLM stall classifier entirely.
+Content-based stall detection monitors sessions for output that stops changing. The content hash strips the full buffer first, then slices the last 500 characters of the normalized text — this ensures identical visual content always produces the same hash regardless of how many raw escape sequences surround it. The normalization strips ANSI escape codes, TUI spinner characters, and countdown/duration text (e.g. `8m 17s` → constant) so that live timers and TUI redraws don't perpetually reset the stall timer. All detection work (ready, blocking prompt, login, exit, stall) runs in a deferred `setImmediate()` tick so that node-pty's synchronous data delivery cannot starve the event loop — timers and I/O callbacks always get a chance to run between data bursts. The output buffer is capped at 100 KB to prevent unbounded growth during long tasks.
+
+When a stall is detected, the session first tries the adapter's `detectTaskComplete()` fast-path. If the adapter recognizes the output as a completed task (e.g. duration summary + idle prompt), it transitions directly to `ready` and emits `task_complete` — skipping the expensive LLM stall classifier entirely.
 
 If the adapter doesn't recognize the output, the session falls back to emitting `stall_detected` for external classification.
 

package/dist/index.d.mts

@@ -446,6 +446,8 @@ declare class PTYSession extends EventEmitter {
     private _lastContentHash;
     private _taskCompleteTimer;
     private static readonly TASK_COMPLETE_DEBOUNCE_MS;
+    private _processScheduled;
+    private static readonly MAX_OUTPUT_BUFFER;
     readonly id: string;
     readonly config: SpawnConfig;
     constructor(adapter: CLIAdapter, config: SpawnConfig, logger?: Logger, stallDetectionEnabled?: boolean, defaultStallTimeoutMs?: number);
@@ -533,6 +535,12 @@ declare class PTYSession extends EventEmitter {
      * Set up event handlers for the PTY
      */
     private setupEventHandlers;
+    /**
+     * Process the accumulated output buffer.
+     * Called via setImmediate() from the onData handler so that heavy regex
+     * work runs in its own event-loop tick, not inside node-pty's native callback.
+     */
+    private processOutputBuffer;
     /**
      * Detect blocking prompts and handle them with auto-responses or user notification.
      * Deduplicates emissions - won't re-emit the same blocking prompt repeatedly.

package/dist/index.d.ts

@@ -446,6 +446,8 @@ declare class PTYSession extends EventEmitter {
     private _lastContentHash;
     private _taskCompleteTimer;
     private static readonly TASK_COMPLETE_DEBOUNCE_MS;
+    private _processScheduled;
+    private static readonly MAX_OUTPUT_BUFFER;
     readonly id: string;
     readonly config: SpawnConfig;
     constructor(adapter: CLIAdapter, config: SpawnConfig, logger?: Logger, stallDetectionEnabled?: boolean, defaultStallTimeoutMs?: number);
@@ -533,6 +535,12 @@ declare class PTYSession extends EventEmitter {
      * Set up event handlers for the PTY
      */
     private setupEventHandlers;
+    /**
+     * Process the accumulated output buffer.
+     * Called via setImmediate() from the onData handler so that heavy regex
+     * work runs in its own event-loop tick, not inside node-pty's native callback.
+     */
+    private processOutputBuffer;
     /**
      * Detect blocking prompts and handle them with auto-responses or user notification.
      * Deduplicates emissions - won't re-emit the same blocking prompt repeatedly.

package/dist/index.js

@@ -347,6 +347,12 @@ var PTYSession = class _PTYSession extends import_events.EventEmitter {
   // Task completion detection (idle detection when busy)
   _taskCompleteTimer = null;
   static TASK_COMPLETE_DEBOUNCE_MS = 1500;
+  // Deferred output processing — prevents node-pty's synchronous data
+  // delivery from starving the event loop (timers, I/O callbacks, etc.)
+  _processScheduled = false;
+  // Output buffer cap — prevents unbounded growth during long tasks
+  static MAX_OUTPUT_BUFFER = 1e5;
+  // 100 KB
   id;
   config;
   get status() {
@@ -444,9 +450,9 @@ var PTYSession = class _PTYSession extends import_events.EventEmitter {
       this.clearStallTimer();
       return;
     }
-    const tail = this.outputBuffer.slice(-500);
-    const stripped = this.stripAnsiForStall(tail).trim();
-    const hash = this.simpleHash(stripped);
+    const stripped = this.stripAnsiForStall(this.outputBuffer).trim();
+    const tail = stripped.slice(-500);
+    const hash = this.simpleHash(tail);
     if (hash === this._lastContentHash) {
       return;
     }
@@ -689,49 +695,16 @@ var PTYSession = class _PTYSession extends import_events.EventEmitter {
     this.ptyProcess.onData((data) => {
       this._lastActivityAt = /* @__PURE__ */ new Date();
       this.outputBuffer += data;
-      if (this._status === "busy" || this._status === "authenticating") {
-        this.resetStallTimer();
+      if (this.outputBuffer.length > _PTYSession.MAX_OUTPUT_BUFFER) {
+        this.outputBuffer = this.outputBuffer.slice(-_PTYSession.MAX_OUTPUT_BUFFER);
       }
       this.emit("output", data);
-      if ((this._status === "starting" || this._status === "authenticating") && this.adapter.detectReady(this.outputBuffer)) {
-        this._status = "ready";
-        this._lastBlockingPromptHash = null;
-        this.outputBuffer = "";
-        this.clearStallTimer();
-        this.emit("ready");
-        this.logger.info({ sessionId: this.id }, "Session ready");
-        return;
-      }
-      if (this._status === "busy" && this.adapter.detectReady(this.outputBuffer)) {
-        this.scheduleTaskComplete();
-      } else {
-        this.cancelTaskComplete();
-      }
-      const blockingPrompt = this.detectAndHandleBlockingPrompt();
-      if (blockingPrompt) {
-        return;
-      }
-      if (this._status !== "ready" && this._status !== "busy") {
-        const loginDetection = this.adapter.detectLogin(this.outputBuffer);
-        if (loginDetection.required && this._status !== "authenticating") {
-          this._status = "authenticating";
-          this.clearStallTimer();
-          this.emit("login_required", loginDetection.instructions, loginDetection.url);
-          this.logger.warn(
-            { sessionId: this.id, loginType: loginDetection.type },
-            "Login required"
-          );
-          return;
-        }
-      }
-      const exitDetection = this.adapter.detectExit(this.outputBuffer);
-      if (exitDetection.exited) {
-        this._status = "stopped";
-        this.clearStallTimer();
-        this.emit("exit", exitDetection.code || 0);
-      }
-      if (this._status !== "starting" && this._status !== "authenticating") {
-        this.tryParseOutput();
+      if (!this._processScheduled) {
+        this._processScheduled = true;
+        setImmediate(() => {
+          this._processScheduled = false;
+          this.processOutputBuffer();
+        });
       }
     });
     this.ptyProcess.onExit(({ exitCode, signal }) => {
@@ -744,6 +717,56 @@ var PTYSession = class _PTYSession extends import_events.EventEmitter {
       this.emit("exit", exitCode);
     });
   }
+  /**
+   * Process the accumulated output buffer.
+   * Called via setImmediate() from the onData handler so that heavy regex
+   * work runs in its own event-loop tick, not inside node-pty's native callback.
+   */
+  processOutputBuffer() {
+    if (this._status === "busy" || this._status === "authenticating") {
+      this.resetStallTimer();
+    }
+    if ((this._status === "starting" || this._status === "authenticating") && this.adapter.detectReady(this.outputBuffer)) {
+      this._status = "ready";
+      this._lastBlockingPromptHash = null;
+      this.outputBuffer = "";
+      this.clearStallTimer();
+      this.emit("ready");
+      this.logger.info({ sessionId: this.id }, "Session ready");
+      return;
+    }
+    if (this._status === "busy" && this.adapter.detectReady(this.outputBuffer)) {
+      this.scheduleTaskComplete();
+    } else {
+      this.cancelTaskComplete();
+    }
+    const blockingPrompt = this.detectAndHandleBlockingPrompt();
+    if (blockingPrompt) {
+      return;
+    }
+    if (this._status !== "ready" && this._status !== "busy") {
+      const loginDetection = this.adapter.detectLogin(this.outputBuffer);
+      if (loginDetection.required && this._status !== "authenticating") {
+        this._status = "authenticating";
+        this.clearStallTimer();
+        this.emit("login_required", loginDetection.instructions, loginDetection.url);
+        this.logger.warn(
+          { sessionId: this.id, loginType: loginDetection.type },
+          "Login required"
+        );
+        return;
+      }
+    }
+    const exitDetection = this.adapter.detectExit(this.outputBuffer);
+    if (exitDetection.exited) {
+      this._status = "stopped";
+      this.clearStallTimer();
+      this.emit("exit", exitDetection.code || 0);
+    }
+    if (this._status !== "starting" && this._status !== "authenticating") {
+      this.tryParseOutput();
+    }
+  }
   /**
    * Detect blocking prompts and handle them with auto-responses or user notification.
    * Deduplicates emissions - won't re-emit the same blocking prompt repeatedly.