@kybernesis/buzz 0.9.4 → 0.9.5

package/dist/turn.d.ts

@@ -64,6 +64,21 @@ export declare function composeMessage(text: string, attachments: MediaRef[], fe
 /** Continue a channel's existing conversation, replacing it only when it is explicitly stale. */
 export declare function answerTurn(client: Client, sessions: SessionStore, channel: string, message: TurnMessage, community: string, log?: (message: string) => void, agentSilenceTimeoutMs?: number): Promise<TurnOutcome>;
 /** What the room is told when eve refuses the turn itself. */
+/**
+ * The two answers from eve that mean the stored session cannot carry another
+ * turn: an unknown session id (404), and a session that ended (409 with
+ * `session_not_active`, "The session is no longer active."). A session ends
+ * when one of its turns fails hard, for instance a model call that 404s at
+ * the gateway; the channel must then start a fresh one.
+ *
+ * Only these replace the mapping. A 400 about the message, a timeout, or a
+ * transport failure says nothing about whether the session is alive, and
+ * discarding it on those is how conversations were lost (KYB-502). The 409
+ * was missed when the timeout work narrowed the rule to 404 alone, and a
+ * channel then failed every message with "The session is no longer active."
+ * until someone edited the store by hand.
+ */
+export declare function isSessionGone(error: unknown): boolean;
 export declare function rejectedTurnReply(error: unknown): string | null;
 /** Give the originating channel one accurate recovery message for bridge-owned silence. */
 export declare function agentSilenceReply(error: unknown): string | null;

package/dist/turn.js

@@ -268,7 +268,7 @@ export async function answerTurn(client, sessions, channel, message, community,
         catch (error) {
             // Transport uncertainty, request rejection, and our own timeout do not
             // prove the durable server conversation is stale. Preserve its mapping.
-            if (!(error instanceof ClientError) || error.status !== 404)
+            if (!isSessionGone(error))
                 throw error;
             log(`session for ${channel.slice(0, 8)} could not continue (${error.message}); starting a new one`);
             sessions.delete(community, channel);
@@ -305,6 +305,28 @@ export async function answerTurn(client, sessions, channel, message, community,
     }
 }
 /** What the room is told when eve refuses the turn itself. */
+/**
+ * The two answers from eve that mean the stored session cannot carry another
+ * turn: an unknown session id (404), and a session that ended (409 with
+ * `session_not_active`, "The session is no longer active."). A session ends
+ * when one of its turns fails hard, for instance a model call that 404s at
+ * the gateway; the channel must then start a fresh one.
+ *
+ * Only these replace the mapping. A 400 about the message, a timeout, or a
+ * transport failure says nothing about whether the session is alive, and
+ * discarding it on those is how conversations were lost (KYB-502). The 409
+ * was missed when the timeout work narrowed the rule to 404 alone, and a
+ * channel then failed every message with "The session is no longer active."
+ * until someone edited the store by hand.
+ */
+export function isSessionGone(error) {
+    if (!(error instanceof ClientError))
+        return false;
+    if (error.status === 404)
+        return true;
+    return (error.status === 409 &&
+        (error.code === "session_not_active" || /no longer active/i.test(String(error.message ?? ""))));
+}
 export function rejectedTurnReply(error) {
     if (!(error instanceof ClientError) || error.status !== 400)
         return null;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kybernesis/buzz",
-  "version": "0.9.4",
+  "version": "0.9.5",
   "description": "Put an eve agent in a Buzz workspace as a member: per-speaker verified identity resolved through the control plane, self-service identity linking, presence, typing and seen signals.",
   "license": "Apache-2.0",
   "type": "module",