@songsid/agend 2.1.8-beta.6 → 2.1.8-beta.8

package/dist/cross-instance-envelope.d.ts

@@ -2,5 +2,12 @@
 export declare const MAX_ASSEMBLED_CROSS_INSTANCE_MESSAGE_BYTES = 16000;
 /** Render metadata carried in a cross-instance handoff. */
 export declare function renderCrossInstanceHandoffMetadata(meta: Record<string, string>): string;
+/**
+ * A task_summary that merely echoes the body is noise in the injected block
+ * (#1037): senders auto-derive it from the first 200 chars when none is
+ * given. It is redundant iff the body already opens with it — an explicit
+ * summary carrying anything beyond the body is still shown.
+ */
+export declare function isRedundantTaskSummary(content: string, taskSummary: string | undefined): boolean;
 /** Exact cross-instance block pasted by the receiving daemon. */
 export declare function formatCrossInstanceInboundMessage(content: string, meta: Record<string, string>): string;

package/dist/cross-instance-envelope.js

@@ -16,12 +16,30 @@ export function renderCrossInstanceHandoffMetadata(meta) {
     add("attachment_file_id", meta.attachment_file_id);
     return rows.length ? `\n(${rows.join(" | ")})` : "";
 }
+/**
+ * A task_summary that merely echoes the body is noise in the injected block
+ * (#1037): senders auto-derive it from the first 200 chars when none is
+ * given. It is redundant iff the body already opens with it — an explicit
+ * summary carrying anything beyond the body is still shown.
+ */
+export function isRedundantTaskSummary(content, taskSummary) {
+    const summary = (taskSummary ?? "").trim();
+    if (!summary)
+        return false;
+    const body = (content ?? "").trim();
+    return body.length > 0 && body.startsWith(summary);
+}
 /** Exact cross-instance block pasted by the receiving daemon. */
 export function formatCrossInstanceInboundMessage(content, meta) {
     const fromInstance = meta.from_instance || "unknown";
     const fromLabel = meta.from_display ? `${meta.from_display} (${fromInstance})` : fromInstance;
     let formatted = `[from:${fromLabel}] ${content}`;
-    formatted += renderCrossInstanceHandoffMetadata(meta);
+    // Dedup only the injected block — logs and Telegram labels keep the raw
+    // task_summary exactly as the sender provided it.
+    const renderMeta = isRedundantTaskSummary(content, meta.task_summary)
+        ? { ...meta, task_summary: "" }
+        : meta;
+    formatted += renderCrossInstanceHandoffMetadata(renderMeta);
     formatted += meta.requires_reply === "true"
         ? "\n(A reply IS required: use report_result with the correlation_id above — or send_to_instance. Not direct text.)"
         : "\n(If you need to reply, use send_to_instance tool, NOT direct text. If there is nothing to add, you may stay silent.)";

package/dist/cross-instance-envelope.js.map

@@ -1 +1 @@
-{"version":3,"file":"cross-instance-envelope.js","sourceRoot":"","sources":["../src/cross-instance-envelope.ts"],"names":[],"mappings":"AAAA,4FAA4F;AAC5F,MAAM,CAAC,MAAM,0CAA0C,GAAG,MAAM,CAAC;AAEjE,2DAA2D;AAC3D,MAAM,UAAU,kCAAkC,CAAC,IAA4B;IAC7E,MAAM,IAAI,GAAa,EAAE,CAAC;IAC1B,MAAM,GAAG,GAAG,CAAC,KAAa,EAAE,KAAyB,EAAE,EAAE;QACvD,IAAI,KAAK,IAAI,KAAK,CAAC,IAAI,EAAE;YAAE,IAAI,CAAC,IAAI,CAAC,GAAG,KAAK,KAAK,KAAK,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;IACpE,CAAC,CAAC;IACF,GAAG,CAAC,YAAY,EAAE,IAAI,CAAC,UAAU,CAAC,CAAC;IACnC,GAAG,CAAC,gBAAgB,EAAE,IAAI,CAAC,cAAc,CAAC,CAAC;IAC3C,GAAG,CAAC,cAAc,EAAE,IAAI,CAAC,YAAY,CAAC,CAAC;IACvC,GAAG,CAAC,cAAc,EAAE,IAAI,CAAC,YAAY,CAAC,CAAC;IACvC,GAAG,CAAC,mBAAmB,EAAE,IAAI,CAAC,iBAAiB,CAAC,CAAC;IACjD,GAAG,CAAC,QAAQ,EAAE,IAAI,CAAC,MAAM,CAAC,CAAC;IAC3B,GAAG,CAAC,oBAAoB,EAAE,IAAI,CAAC,kBAAkB,CAAC,CAAC;IACnD,OAAO,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC;AACtD,CAAC;AAED,iEAAiE;AACjE,MAAM,UAAU,iCAAiC,CAAC,OAAe,EAAE,IAA4B;IAC7F,MAAM,YAAY,GAAG,IAAI,CAAC,aAAa,IAAI,SAAS,CAAC;IACrD,MAAM,SAAS,GAAG,IAAI,CAAC,YAAY,CAAC,CAAC,CAAC,GAAG,IAAI,CAAC,YAAY,KAAK,YAAY,GAAG,CAAC,CAAC,CAAC,YAAY,CAAC;IAC9F,IAAI,SAAS,GAAG,SAAS,SAAS,KAAK,OAAO,EAAE,CAAC;IACjD,SAAS,IAAI,kCAAkC,CAAC,IAAI,CAAC,CAAC;IACtD,SAAS,IAAI,IAAI,CAAC,cAAc,KAAK,MAAM;QACzC,CAAC,CAAC,kHAAkH;QACpH,CAAC,CAAC,wHAAwH,CAAC;IAC7H,OAAO,SAAS,CAAC;AACnB,CAAC"}
+{"version":3,"file":"cross-instance-envelope.js","sourceRoot":"","sources":["../src/cross-instance-envelope.ts"],"names":[],"mappings":"AAAA,4FAA4F;AAC5F,MAAM,CAAC,MAAM,0CAA0C,GAAG,MAAM,CAAC;AAEjE,2DAA2D;AAC3D,MAAM,UAAU,kCAAkC,CAAC,IAA4B;IAC7E,MAAM,IAAI,GAAa,EAAE,CAAC;IAC1B,MAAM,GAAG,GAAG,CAAC,KAAa,EAAE,KAAyB,EAAE,EAAE;QACvD,IAAI,KAAK,IAAI,KAAK,CAAC,IAAI,EAAE;YAAE,IAAI,CAAC,IAAI,CAAC,GAAG,KAAK,KAAK,KAAK,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;IACpE,CAAC,CAAC;IACF,GAAG,CAAC,YAAY,EAAE,IAAI,CAAC,UAAU,CAAC,CAAC;IACnC,GAAG,CAAC,gBAAgB,EAAE,IAAI,CAAC,cAAc,CAAC,CAAC;IAC3C,GAAG,CAAC,cAAc,EAAE,IAAI,CAAC,YAAY,CAAC,CAAC;IACvC,GAAG,CAAC,cAAc,EAAE,IAAI,CAAC,YAAY,CAAC,CAAC;IACvC,GAAG,CAAC,mBAAmB,EAAE,IAAI,CAAC,iBAAiB,CAAC,CAAC;IACjD,GAAG,CAAC,QAAQ,EAAE,IAAI,CAAC,MAAM,CAAC,CAAC;IAC3B,GAAG,CAAC,oBAAoB,EAAE,IAAI,CAAC,kBAAkB,CAAC,CAAC;IACnD,OAAO,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC;AACtD,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,sBAAsB,CAAC,OAAe,EAAE,WAA+B;IACrF,MAAM,OAAO,GAAG,CAAC,WAAW,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC;IAC3C,IAAI,CAAC,OAAO;QAAE,OAAO,KAAK,CAAC;IAC3B,MAAM,IAAI,GAAG,CAAC,OAAO,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC;IACpC,OAAO,IAAI,CAAC,MAAM,GAAG,CAAC,IAAI,IAAI,CAAC,UAAU,CAAC,OAAO,CAAC,CAAC;AACrD,CAAC;AAED,iEAAiE;AACjE,MAAM,UAAU,iCAAiC,CAAC,OAAe,EAAE,IAA4B;IAC7F,MAAM,YAAY,GAAG,IAAI,CAAC,aAAa,IAAI,SAAS,CAAC;IACrD,MAAM,SAAS,GAAG,IAAI,CAAC,YAAY,CAAC,CAAC,CAAC,GAAG,IAAI,CAAC,YAAY,KAAK,YAAY,GAAG,CAAC,CAAC,CAAC,YAAY,CAAC;IAC9F,IAAI,SAAS,GAAG,SAAS,SAAS,KAAK,OAAO,EAAE,CAAC;IACjD,wEAAwE;IACxE,kDAAkD;IAClD,MAAM,UAAU,GAAG,sBAAsB,CAAC,OAAO,EAAE,IAAI,CAAC,YAAY,CAAC;QACnE,CAAC,CAAC,EAAE,GAAG,IAAI,EAAE,YAAY,EAAE,EAAE,EAAE;QAC/B,CAAC,CAAC,IAAI,CAAC;IACT,SAAS,IAAI,kCAAkC,CAAC,UAAU,CAAC,CAAC;IAC5D,SAAS,IAAI,IAAI,CAAC,cAAc,KAAK,MAAM;QACzC,CAAC,CAAC,kHAAkH;QACpH,CAAC,CAAC,wHAAwH,CAAC;IAC7H,OAAO,SAAS,CAAC;AACnB,CAAC"}

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;