@songsid/agend 2.1.8-beta.5 → 2.1.8-beta.7

package/dist/daemon.d.ts

@@ -517,6 +517,16 @@ export declare class Daemon extends EventEmitter {
      * would silently no-op exactly when we most need to stop feeding the CLI.
      */
     private pausePending;
+    /**
+     * #1044: pause on the idle edge only if the auth error is still on screen.
+     * Set when nothing could vouch for the hit — the backend has no token-free
+     * auth check (muse), or it could not answer. A real expired session prints
+     * its failure at the end of the turn it broke, so it is still at the bottom
+     * when that turn ends; a phrase the pattern caught mid-turn has scrolled on
+     * by then. Recovery cannot decide this: an expired session is back at its
+     * prompt after every failed turn, which is what "recovered" means.
+     */
+    private pauseReconfirmAuth;
     /**
      * An auth failure was detected and nothing has resolved it yet.
      *
@@ -935,7 +945,12 @@ export declare class Daemon extends EventEmitter {
      * failures: pause() alone no-ops while the CLI is busy/stuck, which is the
      * usual state when the error surfaces. Cleared by a successful pause or wake.
      */
-    requestPauseWhenIdle(): void;
+    requestPauseWhenIdle(opts?: {
+        reconfirmAuth?: boolean;
+    }): void;
+    /** The backend's own auth-error pattern, within the pane's last rows. */
+    private authErrorAtPaneBottom;
+    private static readonly AUTH_RECONFIRM_ROWS;
     pause(): Promise<void>;
     /** Respawn the CLI in the preserved window and block until its prompt is ready. */
     wake(timeoutMs?: number): Promise<void>;

package/dist/daemon.js

@@ -1065,6 +1065,16 @@ export class Daemon extends EventEmitter {
      * would silently no-op exactly when we most need to stop feeding the CLI.
      */
     pausePending = false;
+    /**
+     * #1044: pause on the idle edge only if the auth error is still on screen.
+     * Set when nothing could vouch for the hit — the backend has no token-free
+     * auth check (muse), or it could not answer. A real expired session prints
+     * its failure at the end of the turn it broke, so it is still at the bottom
+     * when that turn ends; a phrase the pattern caught mid-turn has scrolled on
+     * by then. Recovery cannot decide this: an expired session is back at its
+     * prompt after every failed turn, which is what "recovered" means.
+     */
+    pauseReconfirmAuth = false;
     /**
      * An auth failure was detected and nothing has resolved it yet.
      *
@@ -3190,11 +3200,28 @@ export class Daemon extends EventEmitter {
      * failures: pause() alone no-ops while the CLI is busy/stuck, which is the
      * usual state when the error surfaces. Cleared by a successful pause or wake.
      */
-    requestPauseWhenIdle() {
+    requestPauseWhenIdle(opts = {}) {
         if (this.pauseWakeState === "paused")
             return;
+        const reconfirm = opts.reconfirmAuth === true;
+        // A pause already pending unconditionally stays unconditional.
+        this.pauseReconfirmAuth = this.pausePending ? this.pauseReconfirmAuth && reconfirm : reconfirm;
         this.pausePending = true;
-    }
+        // A conditional request is the only pause path it has (#1044): decide on
+        // the current pane now instead of at the next state sweep, so a pane that
+        // is already idle is not left open to a delivery in the meantime.
+        if (reconfirm)
+            void this.captureAndEvaluateInstanceState("deferred_pause");
+    }
+    /** The backend's own auth-error pattern, within the pane's last rows. */
+    authErrorAtPaneBottom(pane) {
+        const rows = pane.replace(/\r/g, "").split("\n").filter(r => r.trim() !== "");
+        const bottom = rows.slice(-Daemon.AUTH_RECONFIRM_ROWS).join("\n");
+        return (this.backend?.getErrorPatterns?.() ?? [])
+            .filter(ep => ep.type === "auth_error")
+            .some(ep => new RegExp(ep.pattern.source, ep.pattern.flags.replace("g", "")).test(bottom));
+    }
+    static AUTH_RECONFIRM_ROWS = 15;
     async pause() {
         if (this.pauseWakeState === "paused")
             return;
@@ -3277,6 +3304,7 @@ export class Daemon extends EventEmitter {
         // An explicit wake (e.g. after the user re-logs in) cancels a deferred
         // auth pause — otherwise the instance would pause again the moment it idles.
         this.pausePending = false;
+        this.pauseReconfirmAuth = false;
         this.authFailureUnresolved = false;
         this.loginScreenReported = false;
         this.fatalStartupReported = false;
@@ -3404,6 +3432,16 @@ export class Daemon extends EventEmitter {
         // live, kept its warm slot, and re-raised a hang alert on every scan.
         if (this.pausePending && (snapshot.state === "idle" || snapshot.state === "stuck") && this.pasteQueueDepth === 0) {
             this.pausePending = false;
+            const reconfirm = this.pauseReconfirmAuth;
+            this.pauseReconfirmAuth = false;
+            // An unreadable pane keeps the pause: when in doubt, stop feeding a CLI
+            // that may be unable to answer (42512a46).
+            if (reconfirm && pane !== undefined && !this.authErrorAtPaneBottom(pane)) {
+                this.authFailureUnresolved = false;
+                this.logger.warn({ state: snapshot.state }, "Deferred auth pause dropped: the auth-error text is no longer at the bottom of the pane at the end of the turn, "
+                    + "and this backend has no auth check to confirm it (likely conversation text)");
+                return;
+            }
             this.pauseAllowStuck = snapshot.state === "stuck";
             this.emit("auto_pause_requested", { name: this.name, idleSince: snapshot.stateChangedAt });
             return;