pty-manager 1.6.4 → 1.6.6

package/README.md

@@ -217,7 +217,7 @@ interface SpawnConfig {
   timeout?: number;      // Session timeout in ms
   readySettleMs?: number; // Override adapter's ready settle delay
   stallTimeoutMs?: number; // Override manager stall timeout for this session
-  traceTaskCompletion?: boolean; // Verbose completion trace logs (auto-enabled for Claude)
+  traceTaskCompletion?: boolean; // Verbose completion trace logs (off by default)
 }
 ```
 
@@ -498,14 +498,13 @@ If the fast-path timer doesn't fire (e.g. the prompt indicator disappears from t
 - `debounce_reject_signal`
 - `transition_ready`
 
-By default, tracing is enabled for adapter type `claude`. You can override per session:
+Tracing is off by default. Enable per session:
 
 ```typescript
 const handle = await manager.spawn({
   name: 'agent',
   type: 'claude',
-  traceTaskCompletion: true, // force on
-  // traceTaskCompletion: false, // force off
+  traceTaskCompletion: true,
 });
 ```
 

package/dist/index.d.mts

@@ -41,7 +41,7 @@ interface SpawnConfig {
      *  Ms of output silence after detectReady match before emitting session_ready. */
     readySettleMs?: number;
     /** Enable verbose task-completion trace logs.
-     *  If unset, PTYSession enables this automatically for adapter type "claude". */
+     *  Disabled by default; set true to enable. */
     traceTaskCompletion?: boolean;
     /** Override or disable specific adapter auto-response rules for this session.
      *  Keys are regex source strings (from rule.pattern.source).
@@ -535,6 +535,12 @@ declare class PTYSession extends EventEmitter {
      * word boundaries — e.g. "Do\x1b[5Cyou" becomes "Do you", not "Doyou".
      */
     private stripAnsiForStall;
+    /**
+     * Less-aggressive ANSI stripping for classifier context.
+     * Preserves visible TUI symbols (e.g. ❯, ✻) and durations while removing
+     * escape/control sequences so the classifier keeps useful evidence.
+     */
+    private stripAnsiForClassifier;
     /**
      * Handle external stall classification result.
      * Called by the manager after onStallClassify resolves.
@@ -557,7 +563,7 @@ declare class PTYSession extends EventEmitter {
     private isTaskCompleteSignal;
     /**
      * Claude-oriented task completion traces for PTY debugging.
-     * Enabled by config.traceTaskCompletion, otherwise defaults to enabled for Claude.
+     * Disabled by default; enable via config.traceTaskCompletion.
      */
     private traceTaskCompletion;
     private shouldTraceTaskCompletion;

package/dist/index.d.ts

@@ -41,7 +41,7 @@ interface SpawnConfig {
      *  Ms of output silence after detectReady match before emitting session_ready. */
     readySettleMs?: number;
     /** Enable verbose task-completion trace logs.
-     *  If unset, PTYSession enables this automatically for adapter type "claude". */
+     *  Disabled by default; set true to enable. */
     traceTaskCompletion?: boolean;
     /** Override or disable specific adapter auto-response rules for this session.
      *  Keys are regex source strings (from rule.pattern.source).
@@ -535,6 +535,12 @@ declare class PTYSession extends EventEmitter {
      * word boundaries — e.g. "Do\x1b[5Cyou" becomes "Do you", not "Doyou".
      */
     private stripAnsiForStall;
+    /**
+     * Less-aggressive ANSI stripping for classifier context.
+     * Preserves visible TUI symbols (e.g. ❯, ✻) and durations while removing
+     * escape/control sequences so the classifier keeps useful evidence.
+     */
+    private stripAnsiForClassifier;
     /**
      * Handle external stall classification result.
      * Called by the manager after onStallClassify resolves.
@@ -557,7 +563,7 @@ declare class PTYSession extends EventEmitter {
     private isTaskCompleteSignal;
     /**
      * Claude-oriented task completion traces for PTY debugging.
-     * Enabled by config.traceTaskCompletion, otherwise defaults to enabled for Claude.
+     * Disabled by default; enable via config.traceTaskCompletion.
      */
     private traceTaskCompletion;
     private shouldTraceTaskCompletion;

package/dist/index.js

@@ -463,8 +463,10 @@ var PTYSession = class _PTYSession extends import_events.EventEmitter {
       return;
     }
     const stripped = this.stripAnsiForStall(this.outputBuffer).trim();
+    const visible = this.stripAnsiForClassifier(this.outputBuffer).trim();
     const tail = stripped.slice(-500);
-    const hash = this.simpleHash(tail);
+    const fallbackTail = visible.slice(-500);
+    const hash = this.simpleHash(tail || fallbackTail);
     if (hash === this._lastContentHash) {
       return;
     }
@@ -539,10 +541,16 @@ var PTYSession = class _PTYSession extends import_events.EventEmitter {
       return;
     }
     const recentRaw = this.outputBuffer.slice(-2e3);
-    const recentOutput = this.stripAnsiForStall(recentRaw);
+    const recentOutput = this.stripAnsiForClassifier(recentRaw).trim();
     const stallDurationMs = this._stallStartedAt ? Date.now() - this._stallStartedAt : this._stallTimeoutMs;
     this.logger.debug(
-      { sessionId: this.id, stallDurationMs, bufferTailLength: tail.length },
+      {
+        sessionId: this.id,
+        stallDurationMs,
+        bufferTailLength: tail.length,
+        recentOutputLength: recentOutput.length,
+        recentOutputHash: this.simpleHash(recentOutput.slice(-500))
+      },
       "Stall detected"
     );
     this.emit("stall_detected", recentOutput, stallDurationMs);
@@ -587,6 +595,23 @@ var PTYSession = class _PTYSession extends import_events.EventEmitter {
     result = result.replace(/ {2,}/g, " ");
     return result;
   }
+  /**
+   * Less-aggressive ANSI stripping for classifier context.
+   * Preserves visible TUI symbols (e.g. ❯, ✻) and durations while removing
+   * escape/control sequences so the classifier keeps useful evidence.
+   */
+  stripAnsiForClassifier(str) {
+    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, " ");
+    result = result.replace(/ {2,}/g, " ");
+    return result;
+  }
   /**
    * Handle external stall classification result.
    * Called by the manager after onStallClassify resolves.
@@ -710,7 +735,7 @@ var PTYSession = class _PTYSession extends import_events.EventEmitter {
   }
   /**
    * Claude-oriented task completion traces for PTY debugging.
-   * Enabled by config.traceTaskCompletion, otherwise defaults to enabled for Claude.
+   * Disabled by default; enable via config.traceTaskCompletion.
    */
   traceTaskCompletion(event, ctx = {}) {
     if (!this.shouldTraceTaskCompletion()) return;
@@ -739,10 +764,7 @@ var PTYSession = class _PTYSession extends import_events.EventEmitter {
     );
   }
   shouldTraceTaskCompletion() {
-    if (typeof this.config.traceTaskCompletion === "boolean") {
-      return this.config.traceTaskCompletion;
-    }
-    return this.adapter.adapterType === "claude";
+    return this.config.traceTaskCompletion === true;
   }
   /**
    * Cancel a pending task_complete timer (new output arrived that
@@ -925,7 +947,7 @@ var PTYSession = class _PTYSession extends import_events.EventEmitter {
       this.clearStallTimer();
       this.emit("exit", exitDetection.code || 0);
     }
-    if (this._status !== "starting" && this._status !== "authenticating") {
+    if (this._status === "ready") {
       this.tryParseOutput();
     }
   }