pty-manager 1.3.1 → 1.3.2

package/dist/index.mjs

@@ -309,6 +309,12 @@ var PTYSession = class _PTYSession extends 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() {
@@ -651,49 +657,16 @@ var PTYSession = class _PTYSession extends 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 }) => {
@@ -706,6 +679,56 @@ var PTYSession = class _PTYSession extends 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.