@mulmobridge/client 0.1.1 → 0.1.3

package/dist/client.d.ts

@@ -1,5 +1,5 @@
 import { type Socket } from "socket.io-client";
-import { type Attachment } from "@mulmobridge/protocol";
+import { type Attachment, type BridgeOptions } from "@mulmobridge/protocol";
 export interface MessageAck {
     ok: boolean;
     reply?: string;
@@ -16,6 +16,14 @@ export interface BridgeClientOptions {
     transportId: string;
     /** Defaults to `$MULMOCLAUDE_API_URL` or `http://localhost:3001`. */
     apiUrl?: string;
+    /** Flat primitive bag forwarded to the host app's startChat
+     *  callback via the handshake (`BridgeOptions` from the
+     *  protocol). Values must be string / number / boolean — nested
+     *  objects are rejected server-side by the chat-service. If
+     *  omitted, the client auto-scrapes `<TRANSPORT>_BRIDGE_*` /
+     *  `BRIDGE_*` env vars (producing string values). Pass `{}`
+     *  explicitly to opt out of the scrape. */
+    options?: BridgeOptions;
 }
 export interface BridgeClient {
     /** Send a user turn to MulmoClaude, wait for the assistant reply. */

package/dist/client.js

@@ -12,8 +12,9 @@
 // See `docs/bridge-protocol.md` for the wire-level contract and a
 // minimal non-Node equivalent.
 import { io } from "socket.io-client";
-import { CHAT_SOCKET_EVENTS, CHAT_SOCKET_PATH, } from "@mulmobridge/protocol";
+import { CHAT_SOCKET_EVENTS, CHAT_SOCKET_PATH } from "@mulmobridge/protocol";
 import { readBridgeToken, TOKEN_FILE_PATH } from "./token.js";
+import { readBridgeEnvOptions } from "./options.js";
 // 6 min > the server's REPLY_TIMEOUT_MS (5 min) so the server's
 // timeout surfaces as a reply, not a client-side cancellation.
 const REPLY_TIMEOUT_MS = 6 * 60 * 1000;
@@ -38,9 +39,18 @@ export function requireBearerToken() {
 export function createBridgeClient(opts) {
     const apiUrl = opts.apiUrl ?? process.env.MULMOCLAUDE_API_URL ?? DEFAULT_API_URL;
     const token = requireBearerToken();
+    // `opts.options === undefined` → scrape env automatically.
+    // `opts.options === {}` → opt out of the scrape explicitly.
+    const options = opts.options ?? readBridgeEnvOptions(opts.transportId, process.env);
+    // Only include the `options` key in the handshake when there's
+    // something to send — keeps old servers unaware of the field from
+    // ever seeing an empty object on the wire.
+    const auth = { transportId: opts.transportId, token };
+    if (Object.keys(options).length > 0)
+        auth.options = options;
     const socket = io(apiUrl, {
         path: CHAT_SOCKET_PATH,
-        auth: { transportId: opts.transportId, token },
+        auth,
         transports: ["websocket"],
     });
     installDefaultLogging(socket);
@@ -71,9 +81,7 @@ function sendMessage(socket, externalChatId, text, attachments) {
     if (attachments && attachments.length > 0)
         payload.attachments = attachments;
     return new Promise((resolve) => {
-        socket
-            .timeout(REPLY_TIMEOUT_MS)
-            .emit(CHAT_SOCKET_EVENTS.message, payload, (err, ack) => {
+        socket.timeout(REPLY_TIMEOUT_MS).emit(CHAT_SOCKET_EVENTS.message, payload, (err, ack) => {
             if (err) {
                 resolve({ ok: false, error: `timeout: ${err.message}` });
                 return;

package/dist/index.d.ts

@@ -1,3 +1,5 @@
-export { createBridgeClient, requireBearerToken, type MessageAck, type PushEvent, type BridgeClientOptions, type BridgeClient, } from "./client.js";
+export { createBridgeClient, requireBearerToken, type MessageAck, type PushEvent, type BridgeClientOptions, type BridgeClient } from "./client.js";
 export { readBridgeToken, TOKEN_FILE_PATH } from "./token.js";
+export { readBridgeEnvOptions } from "./options.js";
+export { chunkText } from "./text.js";
 export { mimeFromExtension, isImageMime, isPdfMime, isSupportedAttachmentMime, isNativeAttachmentMime, parseDataUrl, buildDataUrl, type ParsedDataUrl, } from "./mime.js";

package/dist/index.js

@@ -1,4 +1,6 @@
 // @mulmobridge/client — shared socket.io client for all MulmoBridge bridges.
-export { createBridgeClient, requireBearerToken, } from "./client.js";
+export { createBridgeClient, requireBearerToken } from "./client.js";
 export { readBridgeToken, TOKEN_FILE_PATH } from "./token.js";
+export { readBridgeEnvOptions } from "./options.js";
+export { chunkText } from "./text.js";
 export { mimeFromExtension, isImageMime, isPdfMime, isSupportedAttachmentMime, isNativeAttachmentMime, parseDataUrl, buildDataUrl, } from "./mime.js";

package/dist/options.d.ts

@@ -0,0 +1,13 @@
+/**
+ * Read `<TRANSPORT>_BRIDGE_*` and `BRIDGE_*` env vars into a
+ * lowerCamelCase-keyed bag ready to hand to `createBridgeClient`.
+ *
+ * Precedence when the same key resolves from both forms:
+ * transport-specific wins over shared.
+ *
+ * Example:
+ *   SLACK_BRIDGE_DEFAULT_ROLE=slack
+ *   BRIDGE_DEFAULT_ROLE=general
+ *   → `{ defaultRole: "slack" }`
+ */
+export declare function readBridgeEnvOptions(transportId: string, env: Readonly<Record<string, string | undefined>>): Record<string, string>;

package/dist/options.js

@@ -0,0 +1,79 @@
+// Env-var scraper for the bridge options bag.
+//
+// Bridges don't want to hand-maintain a forward-list of env vars
+// that should travel to the host app. Instead we scrape a single
+// dedicated prefix pattern at `createBridgeClient()` time:
+//
+//   <TRANSPORT>_BRIDGE_<KEY>  — transport-specific, wins on clash
+//   BRIDGE_<KEY>              — shared default across every bridge
+//
+// Both forms strip the prefix and convert the `UPPER_SNAKE` tail to
+// `lowerCamel`. Empty string values are dropped so a stray
+// `FOO=""` doesn't shadow `BAR`'s match.
+//
+// The `_BRIDGE_` segment is deliberate: it lets the bridge keep its
+// own secrets (`SLACK_BOT_TOKEN`, `SLACK_APP_TOKEN`, …) naturally
+// outside the scrape — they have no `_BRIDGE_` segment so they're
+// never picked up, no reserved-list needed.
+// Convert UPPER_SNAKE_CASE → lowerCamelCase. Leading digits and
+// adjacent underscores degrade gracefully (adjacent underscores
+// collapse to a single word break; leading digits are allowed but
+// kept as-is after the first segment is lowercased).
+function snakeToLowerCamel(snake) {
+    const parts = snake
+        .toLowerCase()
+        .split("_")
+        .filter((segment) => segment.length > 0);
+    if (parts.length === 0)
+        return "";
+    const [head, ...rest] = parts;
+    return head + rest.map((part) => (part ? part[0].toUpperCase() + part.slice(1) : "")).join("");
+}
+// Strip the prefix, return null if the name doesn't match.
+function matchBridgePrefix(name, transportPrefix) {
+    if (name.startsWith(transportPrefix)) {
+        const tail = name.slice(transportPrefix.length);
+        return tail.length > 0 ? tail : null;
+    }
+    if (name.startsWith("BRIDGE_")) {
+        const tail = name.slice("BRIDGE_".length);
+        return tail.length > 0 ? tail : null;
+    }
+    return null;
+}
+/**
+ * Read `<TRANSPORT>_BRIDGE_*` and `BRIDGE_*` env vars into a
+ * lowerCamelCase-keyed bag ready to hand to `createBridgeClient`.
+ *
+ * Precedence when the same key resolves from both forms:
+ * transport-specific wins over shared.
+ *
+ * Example:
+ *   SLACK_BRIDGE_DEFAULT_ROLE=slack
+ *   BRIDGE_DEFAULT_ROLE=general
+ *   → `{ defaultRole: "slack" }`
+ */
+export function readBridgeEnvOptions(transportId, env) {
+    const transportPrefix = `${transportId.toUpperCase()}_BRIDGE_`;
+    const shared = {};
+    const specific = {};
+    for (const [name, value] of Object.entries(env)) {
+        if (typeof value !== "string" || value.length === 0)
+            continue;
+        const tail = matchBridgePrefix(name, transportPrefix);
+        if (tail === null)
+            continue;
+        const key = snakeToLowerCamel(tail);
+        if (!key)
+            continue;
+        if (name.startsWith(transportPrefix)) {
+            specific[key] = value;
+        }
+        else {
+            shared[key] = value;
+        }
+    }
+    // Transport-specific overrides shared on conflict — spread order
+    // (shared first, then specific) gives exactly that behaviour.
+    return { ...shared, ...specific };
+}

package/dist/text.d.ts

@@ -0,0 +1,5 @@
+/**
+ * Split text into chunks of at most `max` characters.
+ * Returns `["(empty reply)"]` when text is empty.
+ */
+export declare function chunkText(text: string, max: number): string[];

package/dist/text.js

@@ -0,0 +1,14 @@
+// Text utilities shared across bridge packages.
+/**
+ * Split text into chunks of at most `max` characters.
+ * Returns `["(empty reply)"]` when text is empty.
+ */
+export function chunkText(text, max) {
+    if (text.length === 0)
+        return ["(empty reply)"];
+    const chunks = [];
+    for (let i = 0; i < text.length; i += max) {
+        chunks.push(text.slice(i, i + max));
+    }
+    return chunks;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mulmobridge/client",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "description": "Socket.io client library for MulmoBridge — shared by all bridge implementations",
   "type": "module",
   "main": "./dist/index.js",
@@ -11,6 +11,24 @@
       "import": "./dist/index.js",
       "require": "./dist/index.js",
       "default": "./dist/index.js"
+    },
+    "./text": {
+      "types": "./dist/text.d.ts",
+      "import": "./dist/text.js",
+      "require": "./dist/text.js",
+      "default": "./dist/text.js"
+    },
+    "./mime": {
+      "types": "./dist/mime.d.ts",
+      "import": "./dist/mime.js",
+      "require": "./dist/mime.js",
+      "default": "./dist/mime.js"
+    },
+    "./token": {
+      "types": "./dist/token.d.ts",
+      "import": "./dist/token.js",
+      "require": "./dist/token.js",
+      "default": "./dist/token.js"
     }
   },
   "files": [