dsh-interconnect 0.11.15 → 0.11.16

package/lib/tool-interconnect/index.js

@@ -840,6 +840,7 @@ var LINK_HANDSHAKE_TIMEOUT_MS = 1e4;
 var LINK_CHANNEL = "/interconnect/link";
 var MAX_REQUEST_ID_CHARS = 256;
 var MAX_LISTED_SESSIONS = 100;
+var frameFitsLink = (frame) => Buffer.byteLength(JSON.stringify(frame), "utf8") <= MAX_LINK_FRAME_BYTES;
 var rowBytes = (row) => Buffer.byteLength(JSON.stringify(row), "utf8") + 1;
 var notificationSchema = Schema.union([
   Schema.object({ kind: Schema.const("agent/created").required(), sessionId: Schema.string().required() }),
@@ -1126,22 +1127,30 @@ var InterconnectService = class extends Service {
    * Send a `msg` frame over the live link to a peer and resolve with its
    * correlated `msg-result`. Undefined when the peer has no live link here
    * (not configured, or the link is down) — sends can only address configured+
-   * connected peers by design.
+   * connected peers by design — and a `message-too-large` result when the
+   * encoded frame would not fit the link.
    */
   async msgRequest(instanceId, kind, payload) {
     const state = this.linkStates.get(instanceId);
     if (state === void 0 || !state.writable()) return void 0;
     const reqId = `m${++this.reqIdCounter}-${randomUUID()}`;
-    const message = {
-      kind,
-      sessionId: payload.sessionId,
-      text: payload.text,
-      ...payload.sender === void 0 ? {} : { sender: payload.sender },
-      ...payload.delivery === void 0 ? {} : { delivery: payload.delivery },
-      ...payload.resume === void 0 ? {} : { resume: payload.resume }
+    const frame = {
+      type: "msg",
+      reqId,
+      message: {
+        kind,
+        sessionId: payload.sessionId,
+        text: payload.text,
+        ...payload.sender === void 0 ? {} : { sender: payload.sender },
+        ...payload.delivery === void 0 ? {} : { delivery: payload.delivery },
+        ...payload.resume === void 0 ? {} : { resume: payload.resume }
+      }
     };
+    if (!frameFitsLink(frame)) {
+      return { delivered: false, instance: instanceId, reason: "message-too-large" };
+    }
     return this.waitForResult(reqId, "msg", (failure) => {
-      const wrote = state.sendFrame({ type: "msg", reqId, message });
+      const wrote = state.sendFrame(frame);
       if (!wrote) failure(new Error(`interconnect: peer link to ${instanceId} closed while sending`));
     });
   }
@@ -1792,7 +1801,7 @@ function apply(ctx) {
       text: {
         type: "string",
         required: true,
-        description: "Message text delivered to the peer session."
+        description: 'Message text delivered to the peer session. At most 1 MiB once encoded: a larger message is refused with reason "message-too-large".'
       },
       delivery: {
         type: "string",
@@ -1823,6 +1832,8 @@ function apply(ctx) {
               return `not delivered: could not wake "${_args.sessionId}" on ${value.instance} (no persisted session under that id, or the wake itself failed)`;
             case "session-owned-by-subagent":
               return `not delivered: "${_args.sessionId}" is a subagent's session on ${value.instance} \u2014 its parent agent owns delivery, so reach it through that parent`;
+            case "message-too-large":
+              return "not delivered: the message is too large for the peer link (limit 1 MiB); shorten the text and try again";
             default:
               return `not delivered: no live session "${_args.sessionId}" on ${value.instance}` + (_args.resume === true ? " \u2014 use interconnect_list to see which sessions are live there" : " \u2014 use interconnect_list to see which sessions are live there, or set resume to wake this one");
           }
@@ -1973,6 +1984,9 @@ function apply(ctx) {
           if (value.reason === "no-sender-known") {
             return "not delivered: no sender identity is known to reply to";
           }
+          if (value.reason === "message-too-large") {
+            return "not delivered: the reply is too large for the peer link (limit 1 MiB); shorten the text and try again";
+          }
           if (value.reason === "session-owned-by-subagent") {
             return "not delivered: the recorded sender's session belongs to a subagent and its parent agent owns delivery";
           }