@mulmobridge/chat-service 1.1.0 → 1.3.0

package/dist/index.d.ts

@@ -14,6 +14,13 @@ export interface ChatService {
      *  before `attachSocket`: the message is queued and flushes on
      *  the next bridge connection for that transport. */
     pushToBridge: PushFn;
+    /** Subscribe an IN-PROCESS bridge to this transport's pushes (#3080).
+     *  Returns the unsubscribe function. An in-process bridge is always live, so
+     *  its pushes are never queued. */
+    registerInProcessBridge(transportId: string, handler: (event: {
+        chatId: string;
+        message: string;
+    }) => void): () => void;
 }
 export declare function createChatService(deps: ChatServiceDeps): ChatService;
 export type { Attachment, ChatServiceDeps, StartChatFn, StartChatParams, OnSessionEventFn } from "./types.js";

package/dist/index.js

@@ -60,7 +60,42 @@ export function createChatService(deps) {
     // The queue is shared with the socket layer so any pushes
     // enqueued during the pre-attach window flush on first connect.
     let livePush = null;
+    // In-process bridges (#3080). Keyed by transportId, at most one each: two
+    // bridges claiming one transportId is what #3079 exists to stop, and the
+    // registry refuses the second rather than silently fanning a reply out twice.
+    const inProcessBridges = new Map();
+    const registerInProcessBridge = (transportId, handler) => {
+        if (inProcessBridges.has(transportId)) {
+            throw new Error(`an in-process bridge is already registered for transport "${transportId}"`);
+        }
+        inProcessBridges.set(transportId, handler);
+        return () => {
+            if (inProcessBridges.get(transportId) === handler)
+                inProcessBridges.delete(transportId);
+        };
+    };
     const pushToBridge = (transportId, chatId, message) => {
+        // An in-process bridge is in this very process, so it is live by
+        // construction — deliver and return rather than consulting the socket
+        // layer, whose miss path would queue a message that has been delivered.
+        const inProcess = inProcessBridges.get(transportId);
+        if (inProcess) {
+            try {
+                // Runs on WHATEVER stack called `pushToBridge` — a route handler, a
+                // scheduler tick. A socket bridge cannot reach us here because its
+                // handler runs in its own process; an in-process one can, so the
+                // isolation has to be written rather than assumed.
+                inProcess({ chatId, message });
+            }
+            catch (err) {
+                logger.error("chat-service", "in-process bridge push handler threw", {
+                    transportId,
+                    chatId,
+                    error: err instanceof Error ? err.message : String(err),
+                });
+            }
+            return;
+        }
         if (livePush) {
             livePush(transportId, chatId, message);
             return;
@@ -152,6 +187,7 @@ export function createChatService(deps) {
             livePush = handle.pushToBridge;
         },
         pushToBridge,
+        registerInProcessBridge,
     };
 }
 export { writeFileAtomic } from "./atomic-write.js";

package/dist/relay.js

@@ -6,10 +6,8 @@
 // point, session events, role lookup, logger) arrive through
 // `createRelay(deps)` so the module has no direct imports from the
 // host.
-import { EVENT_TYPES } from "@mulmobridge/protocol";
+import { EVENT_TYPES, resolveReplyTimeoutMs } from "@mulmobridge/protocol";
 import { createKeyedSerializer } from "./keyed-serializer.js";
-// ── Constants ────────────────────────────────────────────────
-const REPLY_TIMEOUT_MS = 5 * 60 * 1000;
 // ── Factory ──────────────────────────────────────────────────
 export function createRelay(deps) {
     const serialize = createKeyedSerializer();
@@ -96,7 +94,8 @@ async function processRelayMessage(deps, params) {
         };
     }
     try {
-        const reply = await collectAgentReply(onSessionEvent, chatState.sessionId, params.onChunk);
+        const replyTimeoutMs = resolveBridgeReplyTimeout(bridgeOptions, logger, transportId);
+        const reply = await collectAgentReply(onSessionEvent, chatState.sessionId, replyTimeoutMs, params.onChunk);
         await store.setChatState(transportId, {
             ...chatState,
             updatedAt: new Date().toISOString(),
@@ -139,15 +138,23 @@ export function resolveDefaultRole(bridgeOptions, getRole, fallbackRoleId, logge
     }
     return resolved.id;
 }
+// The bridge client reads the same option for its ack timer, so a bad value
+// is warned about on both ends and both fall back to the same default.
+function resolveBridgeReplyTimeout(bridgeOptions, logger, transportId) {
+    const { replyTimeoutMs, warning } = resolveReplyTimeoutMs(bridgeOptions?.replyTimeoutMs);
+    if (warning)
+        logger.warn("chat-service", "bridge reply timeout option ignored", { transportId, warning });
+    return replyTimeoutMs;
+}
 // Kept out of the factory closure so future packaging doesn't need
 // to re-capture anything; `onSessionEvent` arrives as a plain param.
-function collectAgentReply(onSessionEvent, chatSessionId, onChunk) {
+function collectAgentReply(onSessionEvent, chatSessionId, replyTimeoutMs, onChunk) {
     return new Promise((resolve) => {
         const textChunks = [];
         const timer = setTimeout(() => {
             unsubscribe();
             resolve(textChunks.join("") || "The request timed out before a reply was generated.");
-        }, REPLY_TIMEOUT_MS);
+        }, replyTimeoutMs);
         const unsubscribe = onSessionEvent(chatSessionId, (event) => {
             const type = event.type;
             if (type === EVENT_TYPES.text) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mulmobridge/chat-service",
-  "version": "1.1.0",
+  "version": "1.3.0",
   "description": "Server-side chat service for MulmoBridge — socket.io + REST bridge to Claude Code agents",
   "type": "module",
   "main": "./dist/index.js",
@@ -20,14 +20,14 @@
   "scripts": {
     "build": "tsc",
     "prepack": "yarn build",
-    "typecheck": "tsc --noEmit",
+    "typecheck": "tsc -p tsconfig.typecheck.json",
     "test": "tsx --test test/test_*.ts",
     "lint": "eslint src test"
   },
   "license": "MIT",
   "author": "Receptron Team",
   "dependencies": {
-    "@mulmobridge/protocol": "^1.0.1"
+    "@mulmobridge/protocol": "^1.1.0"
   },
   "peerDependencies": {
     "express": "^5.0.0",