@sema-agent/core 2.3.0 → 2.4.0

package/dist/agents/send-message-tool.d.ts

@@ -21,6 +21,8 @@ export interface SendMessageToolOptions {
     senderName?: string;
     roster?: import("./roster-store.js").RosterStore;
     siblingRetain?: SubagentRetainLedger;
+    parentTaskId?: string;
+    parentSessionId?: string;
     onBackgroundChildEvent?: (event: import("../core/types.js").BackgroundChildEvent) => void;
     agentStore?: import("../core/background-agent-store.js").BackgroundAgentStore;
     mailbox?: import("../core/mailbox-store.js").MailboxStore;
@@ -34,6 +36,8 @@ export interface SendMessageToolOptions {
         details?: unknown;
     }>;
 }
+export declare const SEND_MESSAGE_SUMMARY_MAX = 200;
+export declare function clipSendMessageSummary(raw: string): string;
 export declare function createSendMessageTool(opts: SendMessageToolOptions): import("../core/types.js").AgentTool<Type.TObject<{
     to: Type.TString;
     message: Type.TString;

package/dist/agents/send-message-tool.js

@@ -8,6 +8,12 @@ import { createSubagentResume } from "./subagent.js";
 export const SEND_MESSAGE_TOOL_NAME = "SendMessage";
 let uplinkSeqGlobal = Date.now();
 const UPLINK_RESULT_MAX = 8000;
+export const SEND_MESSAGE_SUMMARY_MAX = 200;
+export function clipSendMessageSummary(raw) {
+    return raw.length > SEND_MESSAGE_SUMMARY_MAX
+        ? `${raw.slice(0, SEND_MESSAGE_SUMMARY_MAX)} [summary truncated: ${raw.length} chars total]`
+        : raw;
+}
 const TEAMMATE_MESSAGE_TAG = "teammate-message";
 function frameTeammateMessage(args) {
     const summaryAttr = args.summary !== undefined ? ` summary="${escapeAttributeValue(args.summary)}"` : "";
@@ -84,6 +90,9 @@ export function createSendMessageTool(opts) {
                 };
             }
             const senderId = ctx.taskId ?? opts.owner;
+            const [parentTaskId, parentSessionId] = ctx.parentTaskId !== undefined ? [ctx.parentTaskId, ctx.parentSessionId] : [opts.parentTaskId, opts.parentSessionId];
+            const senderIsChild = parentTaskId !== undefined || opts.uplink !== undefined || opts.senderName !== undefined || opts.siblingRetain !== undefined;
+            const senderLabel = opts.senderName ?? (senderIsChild ? senderId ?? "main" : "main");
             if (senderId !== undefined && isObserverTaskId(senderId)) {
                 return { content: OBSERVER_SENDMESSAGE_SENDER_REFUSAL, details: { error: "observer_sender" }, isError: true };
             }
@@ -92,7 +101,7 @@ export function createSendMessageTool(opts) {
             }
             if (normalizeAgentName(to) === "main") {
                 if (opts.uplink && senderId !== undefined) {
-                    const uplinkSummaryRaw = typeof a.summary === "string" && a.summary.trim() !== "" ? a.summary.trim().slice(0, 200) : message.slice(0, 80);
+                    const uplinkSummaryRaw = typeof a.summary === "string" && a.summary.trim() !== "" ? clipSendMessageSummary(a.summary.trim()) : message.slice(0, 80);
                     const fromLabel = opts.senderName !== undefined ? opts.senderName : senderId;
                     try {
                         opts.uplink({
@@ -112,11 +121,7 @@ export function createSendMessageTool(opts) {
                         details: { type: "send-message", status: "uplinked", to: "main", seq: uplinkSeqGlobal },
                     };
                 }
-                const hasParent = ctx.parentTaskId !== undefined ||
-                    opts.uplink !== undefined ||
-                    opts.senderName !== undefined ||
-                    opts.siblingRetain !== undefined;
-                if (!hasParent) {
+                if (!senderIsChild) {
                     return {
                         content: `You are the main conversation — "main" addresses you. Send to a named agent instead.`,
                         details: { error: "main_is_self", to },
@@ -151,9 +156,9 @@ export function createSendMessageTool(opts) {
                 if (!row)
                     return undefined;
                 if (!canAccessAgentRecord(row, access)) {
-                    if (ctx.parentTaskId === undefined)
+                    if (parentTaskId === undefined)
                         return undefined;
-                    const parentView = { ...access, owner: ctx.parentTaskId, ...(ctx.parentSessionId !== undefined ? { sessionId: ctx.parentSessionId } : {}) };
+                    const parentView = { ...access, owner: parentTaskId, ...(parentSessionId !== undefined ? { sessionId: parentSessionId } : {}) };
                     if (!canAccessAgentRecord(row, parentView))
                         return undefined;
                 }
@@ -271,10 +276,10 @@ export function createSendMessageTool(opts) {
                         };
                     }
                     const clipped = message.length > UPLINK_RESULT_MAX ? `${message.slice(0, UPLINK_RESULT_MAX)}\n[message truncated: ${message.length} chars total]` : message;
-                    const t3Summary = typeof a.summary === "string" && a.summary.trim() !== "" ? a.summary.trim().slice(0, 200) : undefined;
-                    const t3From = opts.senderName ?? senderId;
+                    const t3Summary = typeof a.summary === "string" && a.summary.trim() !== "" ? clipSendMessageSummary(a.summary.trim()) : undefined;
+                    const t3From = senderLabel;
                     try {
-                        await opts.mailbox.append(scope, handle, { ...(t3From !== undefined ? { from: t3From } : {}), content: t3Summary !== undefined ? `[${t3Summary}] ${clipped}` : clipped, sentAt: now });
+                        await opts.mailbox.append(scope, handle, { from: t3From, content: t3Summary !== undefined ? `[${t3Summary}] ${clipped}` : clipped, sentAt: now });
                     }
                     catch (e) {
                         await rollback();
@@ -340,8 +345,8 @@ export function createSendMessageTool(opts) {
             };
             let idRow = opts.registry.getAccessibleTask(to, access);
             let resolvedAccess = access;
-            if (!idRow && ctx.parentTaskId !== undefined) {
-                const parentView = { ...access, owner: ctx.parentTaskId, ...(ctx.parentSessionId !== undefined ? { sessionId: ctx.parentSessionId } : {}) };
+            if (!idRow && parentTaskId !== undefined) {
+                const parentView = { ...access, owner: parentTaskId, ...(parentSessionId !== undefined ? { sessionId: parentSessionId } : {}) };
                 const siblingById = opts.registry.getAccessibleTask(to, parentView);
                 if (siblingById) {
                     idRow = siblingById;
@@ -356,8 +361,8 @@ export function createSendMessageTool(opts) {
             let target = idRow;
             if (!target) {
                 let byName = opts.registry.resolveBackgroundAgentByName(to, access);
-                if (byName.status === "not_found" && ctx.parentTaskId !== undefined) {
-                    const parentView = { ...access, owner: ctx.parentTaskId, ...(ctx.parentSessionId !== undefined ? { sessionId: ctx.parentSessionId } : {}) };
+                if (byName.status === "not_found" && parentTaskId !== undefined) {
+                    const parentView = { ...access, owner: parentTaskId, ...(parentSessionId !== undefined ? { sessionId: parentSessionId } : {}) };
                     const sibling = opts.registry.resolveBackgroundAgentByName(to, parentView);
                     if (sibling.status === "found" || sibling.status === "ambiguous") {
                         byName = sibling;
@@ -412,8 +417,8 @@ export function createSendMessageTool(opts) {
             }
             if (row.status === "running" || row.status === "pending") {
                 return await withTargetLane(targetLaneKey(access.scope, targetId), async () => {
-                    const s2Summary = typeof a.summary === "string" && a.summary.trim() !== "" ? a.summary.trim().slice(0, 200) : undefined;
-                    const fromLabel = opts.senderName ?? senderId ?? "main";
+                    const s2Summary = typeof a.summary === "string" && a.summary.trim() !== "" ? clipSendMessageSummary(a.summary.trim()) : undefined;
+                    const fromLabel = senderLabel;
                     const s2Clipped = message.length > UPLINK_RESULT_MAX
                         ? `${message.slice(0, UPLINK_RESULT_MAX)}\n[message truncated: ${message.length} chars total]`
                         : message;
@@ -444,11 +449,19 @@ export function createSendMessageTool(opts) {
                             isError: true,
                         };
                     }
-                    return {
-                        content: `Message not sent: ${who} just finished (delivery raced its completion). Send again to continue it from its transcript.`,
-                        details: { error: "settle_race", to },
-                        isError: true,
-                    };
+                    const nowRow = opts.registry.getAccessibleTask(targetId, resolvedAccess);
+                    const stillLive = nowRow !== undefined && (nowRow.status === "running" || nowRow.status === "pending");
+                    return stillLive
+                        ? {
+                            content: `Message not sent: ${who} is still starting up (it has not begun reading messages yet) — the message was not delivered. Send again shortly.`,
+                            details: { error: "settle_race", to },
+                            isError: true,
+                        }
+                        : {
+                            content: `Message not sent: ${who} just finished (delivery raced its completion). Send again to continue it from its transcript.`,
+                            details: { error: "settle_race", to },
+                            isError: true,
+                        };
                 });
             }
             if (row.status === "killed") {
@@ -510,8 +523,8 @@ export function createSendMessageTool(opts) {
                     ...(row.rootSessionId !== undefined ? { rowRootSessionId: row.rootSessionId } : {}),
                     ...(opts.notify ? { currentParentNotify: opts.notify } : {}),
                 });
-                const summary = typeof a.summary === "string" && a.summary.trim() !== "" ? a.summary.trim().slice(0, 200) : undefined;
-                const fromPrefix = ctx.parentTaskId !== undefined ? `(message from teammate "${opts.senderName ?? senderId ?? "unknown"}")\n` : "";
+                const summary = typeof a.summary === "string" && a.summary.trim() !== "" ? clipSendMessageSummary(a.summary.trim()) : undefined;
+                const fromPrefix = parentTaskId !== undefined ? `(message from teammate "${opts.senderName ?? senderId ?? "unknown"}")\n` : "";
                 try {
                     const safeSummary = summary !== undefined ? escapeEnvelopeTag(TEAMMATE_MESSAGE_TAG, summary) : undefined;
                     const safeMessage = escapeEnvelopeTag(TEAMMATE_MESSAGE_TAG, message);