pty-manager 1.6.2 → 1.6.4

package/dist/pty-worker.js

@@ -331,6 +331,8 @@ var PTYSession = class _PTYSession extends import_events.EventEmitter {
   _stallBackoffMs = 0;
   // Initialized in constructor from _stallTimeoutMs
   static MAX_STALL_BACKOFF_MS = 3e4;
+  _stallEmissionCount = 0;
+  static MAX_STALL_EMISSIONS = 5;
   // Task completion detection (idle detection when busy)
   _taskCompleteTimer = null;
   _taskCompletePending = false;
@@ -448,6 +450,7 @@ var PTYSession = class _PTYSession extends import_events.EventEmitter {
       return;
     }
     this._lastContentHash = hash;
+    this._stallEmissionCount = 0;
     if (this._stallTimer) {
       clearTimeout(this._stallTimer);
       this._stallTimer = null;
@@ -470,6 +473,7 @@ var PTYSession = class _PTYSession extends import_events.EventEmitter {
     this._stallStartedAt = null;
     this._lastContentHash = null;
     this._stallBackoffMs = this._stallTimeoutMs;
+    this._stallEmissionCount = 0;
   }
   /**
    * Called when the stall timer fires (no output for stallTimeoutMs).
@@ -506,6 +510,15 @@ var PTYSession = class _PTYSession extends import_events.EventEmitter {
       );
       return;
     }
+    this._stallEmissionCount++;
+    if (this._stallEmissionCount > _PTYSession.MAX_STALL_EMISSIONS) {
+      this.logger.warn(
+        { sessionId: this.id, count: this._stallEmissionCount },
+        "Max stall emissions reached \u2014 suspending stall detection for this task"
+      );
+      this.clearStallTimer();
+      return;
+    }
     const recentRaw = this.outputBuffer.slice(-2e3);
     const recentOutput = this.stripAnsiForStall(recentRaw);
     const stallDurationMs = this._stallStartedAt ? Date.now() - this._stallStartedAt : this._stallTimeoutMs;
@@ -545,6 +558,8 @@ var PTYSession = class _PTYSession extends import_events.EventEmitter {
     let result = str.replace(/\x1b\[\d*[CDABGdEF]/g, " ");
     result = result.replace(/\x1b\[\d*(?:;\d+)?[Hf]/g, " ");
     result = result.replace(/\x1b\[\d*[JK]/g, " ");
+    result = result.replace(/\x1b\](?:[^\x07\x1b]|\x1b[^\\])*(?:\x07|\x1b\\)/g, "");
+    result = result.replace(/\x1bP(?:[^\x1b]|\x1b[^\\])*\x1b\\/g, "");
     result = result.replace(/\x1B(?:[@-Z\\-_]|\[[0-?]*[ -/]*[@-~])/g, "");
     result = result.replace(/[\x00-\x08\x0b-\x1f\x7f]/g, "");
     result = result.replace(/\xa0/g, " ");
@@ -628,10 +643,15 @@ var PTYSession = class _PTYSession extends import_events.EventEmitter {
    * being a no-op when already scheduled. This allows TUI agents that
    * continue rendering decorative output (status bar, update notices) after
    * the prompt to eventually settle, rather than having the timer cancelled
-   * by every new data chunk. The callback re-verifies detectReady() before
-   * transitioning, so stale triggers are safe.
+   * by every new data chunk. The callback re-verifies the task-complete
+   * signal before transitioning, so stale triggers are safe.
    */
   scheduleTaskComplete() {
+    const wasPending = this._taskCompletePending;
+    this.traceTaskCompletion("debounce_schedule", {
+      wasPending,
+      debounceMs: _PTYSession.TASK_COMPLETE_DEBOUNCE_MS
+    });
     if (this._taskCompleteTimer) {
       clearTimeout(this._taskCompleteTimer);
     }
@@ -639,17 +659,72 @@ var PTYSession = class _PTYSession extends import_events.EventEmitter {
     this._taskCompleteTimer = setTimeout(() => {
       this._taskCompleteTimer = null;
       this._taskCompletePending = false;
-      if (this._status !== "busy") return;
-      if (!this.adapter.detectReady(this.outputBuffer)) return;
+      const signal = this.isTaskCompleteSignal(this.outputBuffer);
+      this.traceTaskCompletion("debounce_fire", { signal });
+      if (this._status !== "busy") {
+        this.traceTaskCompletion("debounce_reject_status", { signal });
+        return;
+      }
+      if (!signal) {
+        this.traceTaskCompletion("debounce_reject_signal", { signal });
+        return;
+      }
       this._status = "ready";
       this._lastBlockingPromptHash = null;
       this.outputBuffer = "";
       this.clearStallTimer();
       this.emit("status_changed", "ready");
       this.emit("task_complete");
+      this.traceTaskCompletion("transition_ready", { signal: true });
       this.logger.info({ sessionId: this.id }, "Task complete \u2014 agent returned to idle prompt");
     }, _PTYSession.TASK_COMPLETE_DEBOUNCE_MS);
   }
+  /**
+   * Adapter-level task completion check with compatibility fallback.
+   * Prefer detectTaskComplete() because detectReady() may be broad for TUIs.
+   */
+  isTaskCompleteSignal(output) {
+    if (this.adapter.detectTaskComplete) {
+      return this.adapter.detectTaskComplete(output);
+    }
+    return this.adapter.detectReady(output);
+  }
+  /**
+   * Claude-oriented task completion traces for PTY debugging.
+   * Enabled by config.traceTaskCompletion, otherwise defaults to enabled for Claude.
+   */
+  traceTaskCompletion(event, ctx = {}) {
+    if (!this.shouldTraceTaskCompletion()) return;
+    const output = this.outputBuffer;
+    const detectTaskComplete = this.adapter.detectTaskComplete ? this.adapter.detectTaskComplete(output) : void 0;
+    const detectReady = this.adapter.detectReady(output);
+    const detectLoading = this.adapter.detectLoading ? this.adapter.detectLoading(output) : void 0;
+    const normalizedTail = this.stripAnsiForStall(output.slice(-280));
+    this.logger.debug(
+      {
+        sessionId: this.id,
+        adapterType: this.adapter.adapterType,
+        event,
+        status: this._status,
+        taskCompletePending: this._taskCompletePending,
+        signal: ctx.signal,
+        wasPending: ctx.wasPending,
+        debounceMs: ctx.debounceMs,
+        detectTaskComplete,
+        detectReady,
+        detectLoading,
+        tailHash: this.simpleHash(normalizedTail),
+        tailSnippet: normalizedTail.slice(-140)
+      },
+      "Task completion trace"
+    );
+  }
+  shouldTraceTaskCompletion() {
+    if (typeof this.config.traceTaskCompletion === "boolean") {
+      return this.config.traceTaskCompletion;
+    }
+    return this.adapter.adapterType === "claude";
+  }
   /**
    * Cancel a pending task_complete timer (new output arrived that
    * doesn't match the idle prompt, so the agent is still working).
@@ -802,7 +877,9 @@ var PTYSession = class _PTYSession extends import_events.EventEmitter {
       return;
     }
     if (this._status === "busy") {
-      if (this._taskCompletePending || this.adapter.detectReady(this.outputBuffer)) {
+      const signal = this.isTaskCompleteSignal(this.outputBuffer);
+      if (this._taskCompletePending || signal) {
+        this.traceTaskCompletion("busy_signal", { signal });
         this.scheduleTaskComplete();
       }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pty-manager",
-  "version": "1.6.2",
+  "version": "1.6.4",
   "description": "PTY session manager with lifecycle management, pluggable adapters, and blocking prompt detection",
   "main": "dist/index.js",
   "module": "dist/index.mjs",