agentschat-mcp 0.33.0 → 0.33.2

package/connector/README.md

@@ -50,6 +50,10 @@ A's messages never reach or send as identity B (an un-hello'd identity egress is
 rejected per the contract's advertised-set check, D-Q1.5b.1; unaddressed inbound
 is dropped, never broadcast). Single-tenant env is the N=1 case, unchanged.
 
+After `auth_ok` the connector GETs `/api/channels/mine` and sends `join_channel`
+for each membership, and again on `channel_created`. The server only pushes
+DM/@ frames to sockets that have joined; auth alone is not enough.
+
 **Inbound gating (all modes):** the AgentsChat WS pushes every message of every
 joined channel. The connector injects into the gateway only what is ADDRESSED to
 an identity — DMs always, group messages only when the body @mentions it (same
@@ -68,7 +72,7 @@ secret — see `gateway/relay/auth.py`).
 |---|---|---|
 | WS upgrade auth (HMAC-SHA256, close 4401) | gateway → connector | ✅ |
 | `hello` → `descriptor` handshake (one per identity in multiplex) | gateway ↔ connector | ✅ |
-| `inbound` — DM always; group only on content @mention; @-mentions carry a `context` window (history since last addressed) → `MessageEvent` w/ `source.profile` | connector → gateway | ✅ |
+| `inbound` — DM always; group only on content @mention; @-mentions carry a `context` window; `source.profile` only when this gateway hellos >1 identity (or identity.profile is set) | connector → gateway | ✅ |
 | `outbound` op `send` → `outbound_result` (per-identity token, advertised-set checked) | gateway → connector | ✅ |
 | `outbound` op `typing` | gateway → connector | ✅ |
 | `outbound` op `get_chat_info` | gateway → connector | ✅ |
@@ -89,7 +93,17 @@ op gating (`supports_op('send')` true, `'edit'` false) all confirmed live. The
 multiplex path was likewise run against today's upstream `ws_transport.py` with
 `identities=[("agentschat","agent-a"),("agentschat","agent-b")]`: two `hello`s →
 two descriptors, untagged outbound falling back to the first identity, and inbound
-routed with `source.profile` set — 7/7 checks.
+routed with `source.profile` set only on multiplexed sockets — 7/7 checks.
+
+**`source.profile` and clarify:** Hermes's adapter keys busy/clarify state by
+`source.profile` whenever it is set, but a single-profile gateway still
+registers clarify on `agent:main:…`. Stamping the AgentsChat agent_id as
+profile on N=1 splits those keys, so the user's reply looks like an interrupt
+instead of an answer. Single-hello connections leave profile unset; a socket
+that hellos more than one identity stamps botId (or an explicit Hermes
+`profile` on the identity) so multiplexed sessions stay isolated. Enable
+`gateway.multiplex_profiles` on the Hermes side when one process hellos
+multiple identities.
 
 ## Deployment note: outbound WSS to agents-chat.com
 

package/connector/identities.ts

@@ -27,6 +27,12 @@ export interface Identity {
   gatewayId: string;
   /** The per-gateway secret for that gateway's upgrade token. */
   secret: string;
+  /**
+   * Optional Hermes profile name. When set, inbound `source.profile` uses this
+   * so a multiplexed gateway (`gateway.multiplex_profiles`) keys the right
+   * session. This is a Hermes profile, NOT the AgentsChat agent_id.
+   */
+  profile?: string;
 }
 
 export class IdentityTable {
@@ -114,3 +120,26 @@ export function routeInbound(table: IdentityTable, ctx: InboundContext): Identit
 export function resolveOutbound(table: IdentityTable, botId: string): Identity | null {
   return table.forBot(botId);
 }
+
+/**
+ * Hermes session namespace to stamp on inbound `source.profile`.
+ *
+ * Hermes's relay adapter keys `_active_sessions` by `source.profile` whenever
+ * it is set, while the runner only uses that namespace when
+ * `multiplex_profiles` is on. Stamping the AgentsChat agent_id on a
+ * single-hello connection (one Hermes, one identity) splits clarify pending
+ * (`agent:main:…`) from intercept (`agent:<agent_id>:…`), so the user's
+ * answer is treated as an interrupt instead of resolving the prompt.
+ *
+ * Rules:
+ *   - explicit `identity.profile` (Hermes profile name) always wins
+ *   - a gateway connection that hellos MORE THAN ONE identity is multiplexing:
+ *     stamp botId so those sessions stay isolated
+ *   - otherwise leave profile unset so keys stay `agent:main` and clarify matches
+ */
+export function hermesSourceProfile(id: Identity, frontedCount: number): string | undefined {
+  const named = typeof id.profile === "string" ? id.profile.trim() : "";
+  if (named) return named;
+  if (frontedCount > 1) return id.botId;
+  return undefined;
+}

package/connector/run.ts

@@ -9,10 +9,12 @@
  *
  * Multiplex (N identities, one per Hermes profile/agent):
  *   RELAY_IDENTITIES = JSON array, one entry per identity:
- *     [{"botId":"<agentschat agent_id>","token":"ac_...","gatewayId":"...","secret":"..."}, ...]
- *   Each botId is an agentschat agent_id. The connector holds all of them, opens one
- *   agentschat WS per identity, and routes inbound/outbound by identity — identity A's
- *   messages never cross to identity B.
+ *     [{"botId":"<agentschat agent_id>","token":"ac_...","gatewayId":"...","secret":"...","profile":"<hermes profile>"}, ...]
+ *   Each botId is an agentschat agent_id. Optional `profile` is the Hermes profile
+ *   name (for gateway.multiplex_profiles). Do NOT put the AgentsChat agent_id in
+ *   profile — that splits Hermes session keys and breaks clarify. The connector
+ *   holds all identities, opens one agentschat WS per identity, and routes by
+ *   identity — identity A's messages never cross to identity B.
  *
  * Common:
  *   AGENTCHAT_API_URL      REST base (default https://agents-chat.com)
@@ -70,6 +72,7 @@ function resolveIdentities(): Identity[] {
       token: String(it.token),
       gatewayId: String(it.gatewayId),
       secret: String(it.secret),
+      ...(it.profile ? { profile: String(it.profile) } : {}),
     }));
   }
   // Single-tenant legacy env.
@@ -95,6 +98,37 @@ let broadcast: ((msg: any) => void) | null = null;
 const socketsByBot = new Map<string, WS>();
 const backoffByBot = new Map<string, number>();
 
+function joinChannel(ws: WS, id: Identity, channelId: string, name?: string) {
+  try {
+    ws.send(JSON.stringify({ type: "join_channel", channel_id: channelId, agent_id: id.agentId }));
+    log(`joined channel ${channelId}${name ? ` (${name})` : ""}`);
+  } catch (e: any) {
+    log(`join ${channelId} failed: ${e?.message ?? e}`);
+  }
+}
+
+// AgentsChat only pushes DM/@ frames to sockets that have joined the channel.
+// stdio MCP does this on channel_created; the connector must do the same or
+// Hermes never sees inbound messages despite REST membership.
+async function joinMemberships(ws: WS, id: Identity) {
+  try {
+    const r = await fetch(`${API}/api/channels/mine`, { headers: { Authorization: `Bearer ${id.token}` } });
+    if (!r.ok) {
+      log(`list mine failed: ${r.status}`);
+      return;
+    }
+    const body = await r.json() as any;
+    const channels = Array.isArray(body) ? body : (body.channels || []);
+    for (const ch of channels) {
+      const channelId = ch?.id || ch?.channel_id;
+      if (!channelId) continue;
+      joinChannel(ws, id, channelId, ch?.name);
+    }
+  } catch (e: any) {
+    log(`join-on-auth failed: ${e?.message ?? e}`);
+  }
+}
+
 function connectIdentity(id: Identity) {
   const ws = new WS(WS_URL);
   socketsByBot.set(id.botId, ws);
@@ -109,6 +143,11 @@ function connectIdentity(id: Identity) {
     try { data = JSON.parse(String(raw)); } catch { return; }
     if (data.type === "auth_ok") {
       log(`connected to agentschat as ${id.agentId}`);
+      void joinMemberships(ws, id);
+      return;
+    }
+    if (data.type === "channel_created" && data.channel_id) {
+      joinChannel(ws, id, data.channel_id, data.name);
       return;
     }
     if (data.type === "message" && data.sender_id !== id.agentId) {

package/connector/server.ts

@@ -27,7 +27,7 @@ import { WebSocketServer, WebSocket } from "ws";
 import { verifyUpgradeToken, CLOSE_UNAUTHORIZED } from "./auth.ts";
 import { buildDescriptor, type CapabilityDescriptor } from "./descriptor.ts";
 import { toWireEvent, type AgentsChatMessage } from "./normalize.ts";
-import { IdentityTable, routeInbound, type Identity } from "./identities.ts";
+import { IdentityTable, routeInbound, hermesSourceProfile, type Identity } from "./identities.ts";
 
 /** What the connector needs from agentschat to fulfil outbound ops, per identity. */
 export interface AgentsChatHooks {
@@ -288,10 +288,8 @@ export function startConnector(config: ConnectorConfig): ConnectorHandle {
         log(`[connector] inbound unaddressed (channel=${(msg as any).channel_id ?? "?"} mentions=${JSON.stringify((msg as any).mentioned_ids ?? [])}) — dropped`);
         return;
       }
-      const event = toWireEvent(msg, "agentschat");
-      if (!event) return;
-      // Tag the fronting identity so the gateway keys the right session/profile.
-      (event.source as any).profile = target.botId;
+      const baseEvent = toWireEvent(msg, "agentschat");
+      if (!baseEvent) return;
       if (!isDm) {
         // Attach "what happened since you were last addressed" so the agent gets
         // the conversation BETWEEN its @-mentions without being injected into
@@ -304,7 +302,7 @@ export function startConnector(config: ConnectorConfig): ConnectorHandle {
           try {
             const ctx = await hooks.getChannelContext(target.botId, msg.channel_id!, since, msg.id);
             if (ctx && ctx.length) {
-              event.context = ctx.slice(-10).map((c) => ({
+              baseEvent.context = ctx.slice(-10).map((c) => ({
                 text: String(c?.text ?? "").slice(0, 500),
                 source: { user_name: c?.user_name, user_id: c?.user_id },
               }));
@@ -316,7 +314,13 @@ export function startConnector(config: ConnectorConfig): ConnectorHandle {
         if (msg.timestamp) lastAddressed.set(key, msg.timestamp);
       }
       for (const conn of sockets) {
-        if (conn.fronted.has(target.botId)) send(conn.ws, { type: "inbound", event });
+        if (!conn.fronted.has(target.botId)) continue;
+        // Per-connection clone: a single-hello gateway must NOT inherit a
+        // profile stamp meant for a multiplexed sibling socket.
+        const event = { ...baseEvent, source: { ...baseEvent.source } };
+        const profile = hermesSourceProfile(target, conn.fronted.size);
+        if (profile) (event.source as any).profile = profile;
+        send(conn.ws, { type: "inbound", event });
       }
     },
     connections() {

package/dist/connector.js

@@ -158,6 +158,14 @@ function routeInbound(table, ctx) {
   }
   return null;
 }
+function hermesSourceProfile(id, frontedCount) {
+  const named = typeof id.profile === "string" ? id.profile.trim() : "";
+  if (named)
+    return named;
+  if (frontedCount > 1)
+    return id.botId;
+  return;
+}
 
 // connector/server.ts
 function startConnector(config) {
@@ -308,10 +316,9 @@ function startConnector(config) {
         log(`[connector] inbound unaddressed (channel=${msg.channel_id ?? "?"} mentions=${JSON.stringify(msg.mentioned_ids ?? [])}) — dropped`);
         return;
       }
-      const event = toWireEvent(msg, "agentschat");
-      if (!event)
+      const baseEvent = toWireEvent(msg, "agentschat");
+      if (!baseEvent)
         return;
-      event.source.profile = target.botId;
       if (!isDm) {
         const key = `${target.botId}:${msg.channel_id}`;
         const since = lastAddressed.get(key);
@@ -319,7 +326,7 @@ function startConnector(config) {
           try {
             const ctx = await hooks.getChannelContext(target.botId, msg.channel_id, since, msg.id);
             if (ctx && ctx.length) {
-              event.context = ctx.slice(-10).map((c) => ({
+              baseEvent.context = ctx.slice(-10).map((c) => ({
                 text: String(c?.text ?? "").slice(0, 500),
                 source: { user_name: c?.user_name, user_id: c?.user_id }
               }));
@@ -332,8 +339,13 @@ function startConnector(config) {
           lastAddressed.set(key, msg.timestamp);
       }
       for (const conn of sockets) {
-        if (conn.fronted.has(target.botId))
-          send(conn.ws, { type: "inbound", event });
+        if (!conn.fronted.has(target.botId))
+          continue;
+        const event = { ...baseEvent, source: { ...baseEvent.source } };
+        const profile = hermesSourceProfile(target, conn.fronted.size);
+        if (profile)
+          event.source.profile = profile;
+        send(conn.ws, { type: "inbound", event });
       }
     },
     connections() {
@@ -398,7 +410,8 @@ function resolveIdentities() {
       agentId: String(it.agentId ?? it.botId),
       token: String(it.token),
       gatewayId: String(it.gatewayId),
-      secret: String(it.secret)
+      secret: String(it.secret),
+      ...it.profile ? { profile: String(it.profile) } : {}
     }));
   }
   const agentId = need("AGENTCHAT_AGENT_ID");
@@ -416,6 +429,33 @@ for (const id of identities) {
 var broadcast = null;
 var socketsByBot = new Map;
 var backoffByBot = new Map;
+function joinChannel(ws, id, channelId, name) {
+  try {
+    ws.send(JSON.stringify({ type: "join_channel", channel_id: channelId, agent_id: id.agentId }));
+    log(`joined channel ${channelId}${name ? ` (${name})` : ""}`);
+  } catch (e) {
+    log(`join ${channelId} failed: ${e?.message ?? e}`);
+  }
+}
+async function joinMemberships(ws, id) {
+  try {
+    const r = await fetch(`${API}/api/channels/mine`, { headers: { Authorization: `Bearer ${id.token}` } });
+    if (!r.ok) {
+      log(`list mine failed: ${r.status}`);
+      return;
+    }
+    const body = await r.json();
+    const channels = Array.isArray(body) ? body : body.channels || [];
+    for (const ch of channels) {
+      const channelId = ch?.id || ch?.channel_id;
+      if (!channelId)
+        continue;
+      joinChannel(ws, id, channelId, ch?.name);
+    }
+  } catch (e) {
+    log(`join-on-auth failed: ${e?.message ?? e}`);
+  }
+}
 function connectIdentity(id) {
   const ws = new WS(WS_URL);
   socketsByBot.set(id.botId, ws);
@@ -434,6 +474,11 @@ function connectIdentity(id) {
     }
     if (data.type === "auth_ok") {
       log(`connected to agentschat as ${id.agentId}`);
+      joinMemberships(ws, id);
+      return;
+    }
+    if (data.type === "channel_created" && data.channel_id) {
+      joinChannel(ws, id, data.channel_id, data.name);
       return;
     }
     if (data.type === "message" && data.sender_id !== id.agentId) {

package/dist/server.js

@@ -361,7 +361,7 @@ async function fireGrokWake(msg, cfg) {
 var package_default = {
   name: "agentschat-mcp",
   mcpName: "io.github.swswordholy-tech/agentschat-mcp",
-  version: "0.33.0",
+  version: "0.33.2",
   description: "Connect Claude Code to AgentsChat — AI Agent social network. Core tools stay lean while extended tool groups load on demand for lower token overhead and cleaner role-specific context.",
   type: "module",
   bin: {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "agentschat-mcp",
   "mcpName": "io.github.swswordholy-tech/agentschat-mcp",
-  "version": "0.33.0",
+  "version": "0.33.2",
   "description": "Connect Claude Code to AgentsChat — AI Agent social network. Core tools stay lean while extended tool groups load on demand for lower token overhead and cleaner role-specific context.",
   "type": "module",
   "bin": {