switchroom 0.12.24 → 0.12.26

package/dist/cli/switchroom.js

@@ -47247,8 +47247,8 @@ var {
 } = import__.default;
 
 // src/build-info.ts
-var VERSION = "0.12.24";
-var COMMIT_SHA = "7ab1329";
+var VERSION = "0.12.26";
+var COMMIT_SHA = "17486c7";
 
 // src/cli/agent.ts
 init_source();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "switchroom",
-  "version": "0.12.24",
+  "version": "0.12.26",
   "description": "Run Claude Code 24/7 on your Claude Pro/Max subscription over Telegram. Open-source alternative to OpenClaw and NanoClaw — no API keys.",
   "type": "module",
   "bin": {

package/telegram-plugin/dist/gateway/gateway.js

@@ -42961,10 +42961,12 @@ function createIpcServer(options) {
   const loggedLegacyUpdatePlaceholder = new Set;
   function removeClient(client3) {
     clients.delete(client3);
-    if (client3.agentName)
+    if (client3.agentName && agentIndex.get(client3.agentName) === client3) {
       agentIndex.delete(client3.agentName);
-    if (client3.topicId != null)
+    }
+    if (client3.topicId != null && topicIndex.get(client3.topicId) === client3) {
       topicIndex.delete(client3.topicId);
+    }
     loggedLegacyUpdatePlaceholder.delete(client3.id);
     onClientDisconnected(client3);
     log(`client disconnected: ${client3.id} (agent=${client3.agentName})`);
@@ -43070,6 +43072,13 @@ function createIpcServer(options) {
       agentIndex.delete(client3.agentName);
     if (client3.topicId != null)
       topicIndex.delete(client3.topicId);
+    const existingClient = agentIndex.get(msg.agentName);
+    if (existingClient && existingClient !== client3) {
+      log(`register: closing prior client for agent=${msg.agentName} ` + `(prior_id=${existingClient.id} new_id=${client3.id}) \u2014 bridge reconnect race`);
+      try {
+        existingClient.close();
+      } catch {}
+    }
     client3.agentName = msg.agentName;
     client3.topicId = msg.topicId ?? null;
     agentIndex.set(msg.agentName, client3);
@@ -47357,11 +47366,11 @@ function sweepStaleTurnActiveMarker(stateDir, opts) {
 }
 
 // ../src/build-info.ts
-var VERSION = "0.12.24";
-var COMMIT_SHA = "7ab1329";
-var COMMIT_DATE = "2026-05-20T14:48:10+10:00";
-var LATEST_PR = 1582;
-var COMMITS_AHEAD_OF_TAG = 3;
+var VERSION = "0.12.26";
+var COMMIT_SHA = "17486c7";
+var COMMIT_DATE = "2026-05-20T05:54:04Z";
+var LATEST_PR = 1586;
+var COMMITS_AHEAD_OF_TAG = 2;
 
 // gateway/boot-version.ts
 function formatRelativeAgo(iso) {
@@ -49234,7 +49243,9 @@ var ipcServer = createIpcServer({
   onClientRegistered(client3) {
     process.stderr.write(`telegram gateway: bridge registered \u2014 agent=${client3.agentName}
 `);
-    shadowEmit({ kind: "bridgeUp", at: Date.now() });
+    if (client3.agentName != null) {
+      shadowEmit({ kind: "bridgeUp", at: Date.now() });
+    }
     client3.send({ type: "status", status: "agent_connected" });
     if (client3.agentName != null) {
       const pending = pendingInboundBuffer.drain(client3.agentName);
@@ -49328,7 +49339,9 @@ var ipcServer = createIpcServer({
   onClientDisconnected(client3) {
     process.stderr.write(`telegram gateway: bridge disconnected \u2014 agent=${client3.agentName}
 `);
-    shadowEmit({ kind: "bridgeDown", at: Date.now() });
+    if (client3.agentName != null) {
+      shadowEmit({ kind: "bridgeDown", at: Date.now() });
+    }
     flushOnAgentDisconnect({
       agentName: client3.agentName,
       activeStatusReactions,

package/telegram-plugin/gateway/gateway.ts

@@ -3196,8 +3196,17 @@ const ipcServer: IpcServer = createIpcServer({
 
   onClientRegistered(client: IpcClient) {
     process.stderr.write(`telegram gateway: bridge registered — agent=${client.agentName}\n`)
-    // Phase 2b shadow: bridge up.
-    shadowEmit({ kind: 'bridgeUp', at: Date.now() })
+    // Phase 2b shadow: ONLY emit bridgeUp for the REAL bridge sidecar
+    // (with an agent name). Anonymous IPC clients (recall.py, mcp
+    // handshakes, etc.) connect briefly without a name and would
+    // false-positive a bridgeUp/bridgeDown cycle that doesn't reflect
+    // the real bridge state. This bug — discovered post-v0.12.24 — was
+    // causing the shadow state to read `bridge_dead` even when the
+    // real bridge was healthy, because every recall.py connect+disconnect
+    // would flip the state.
+    if (client.agentName != null) {
+      shadowEmit({ kind: 'bridgeUp', at: Date.now() })
+    }
     client.send({ type: 'status', status: 'agent_connected' })
 
     // #1150: drain any synthetic inbounds queued for this agent while
@@ -3323,8 +3332,12 @@ const ipcServer: IpcServer = createIpcServer({
 
   onClientDisconnected(client: IpcClient) {
     process.stderr.write(`telegram gateway: bridge disconnected — agent=${client.agentName}\n`)
-    // Phase 2b shadow: bridge down.
-    shadowEmit({ kind: 'bridgeDown', at: Date.now() })
+    // Phase 2b shadow: ONLY emit bridgeDown for the REAL bridge sidecar
+    // (matching the bridgeUp gate above). Anonymous IPC clients
+    // disconnect frequently — those are not bridge flaps.
+    if (client.agentName != null) {
+      shadowEmit({ kind: 'bridgeDown', at: Date.now() })
+    }
 
     // Scope the flush to clients that actually registered as an agent.
     // Anonymous one-shot connections (e.g. recall.py's legacy

package/telegram-plugin/gateway/ipc-server.ts

@@ -266,8 +266,24 @@ export function createIpcServer(options: IpcServerOptions): IpcServer {
 
   function removeClient(client: IpcClient & { _socket: ReturnType<typeof Bun.listen> extends infer S ? any : never }) {
     clients.delete(client);
-    if (client.agentName) agentIndex.delete(client.agentName);
-    if (client.topicId != null) topicIndex.delete(client.topicId);
+    // CRITICAL race fix (2026-05-20): only delete from agentIndex /
+    // topicIndex if the index still points to THIS client. A bridge
+    // that reconnects fast can have its NEW client overwrite
+    // agentIndex[name] (via handleRegister's replace-not-reject)
+    // BEFORE the OLD client's close+removeClient runs. Blindly
+    // deleting agentIndex[name] would remove the LIVE replacement
+    // client by accident → sendToAgent returns false → all subsequent
+    // inbound buffered until the bridge happens to reconnect in an
+    // ordering that works out. User-visible symptom was the chronic
+    // bridge-flap pattern (clerk + gymbro unresponsive 2026-05-20)
+    // where the gateway log showed "bridge registered" but messages
+    // were still getting buffered as if no bridge existed.
+    if (client.agentName && agentIndex.get(client.agentName) === client) {
+      agentIndex.delete(client.agentName);
+    }
+    if (client.topicId != null && topicIndex.get(client.topicId) === client) {
+      topicIndex.delete(client.topicId);
+    }
     loggedLegacyUpdatePlaceholder.delete(client.id);
     onClientDisconnected(client);
     log(`client disconnected: ${client.id} (agent=${client.agentName})`);
@@ -407,6 +423,28 @@ export function createIpcServer(options: IpcServerOptions): IpcServer {
     if (client.agentName) agentIndex.delete(client.agentName);
     if (client.topicId != null) topicIndex.delete(client.topicId);
 
+    // 2026-05-20 race fix: if a PRIOR client is registered as this
+    // agent name (a stale/zombie connection that hasn't been evicted
+    // yet), explicitly close it before installing this new client.
+    // Without this, the prior client remains in `clients` set, its
+    // heartbeat watchdog still ticks, and its eventual close+removeClient
+    // can confuse routing. The removeClient identity-check fix above
+    // means the index won't be wrongly deleted, but two concurrent
+    // clients claiming the same agent name is still a routing hazard
+    // — close the zombie cleanly here.
+    const existingClient = agentIndex.get(msg.agentName);
+    if (existingClient && existingClient !== client) {
+      log(
+        `register: closing prior client for agent=${msg.agentName} ` +
+        `(prior_id=${existingClient.id} new_id=${client.id}) — bridge reconnect race`,
+      );
+      try {
+        (existingClient as IpcClientImpl).close();
+      } catch {
+        /* nothing to do */
+      }
+    }
+
     client.agentName = msg.agentName;
     client.topicId = msg.topicId ?? null;