@threadplane/langgraph 0.0.58 → 0.0.59

package/fesm2022/threadplane-langgraph.mjs

@@ -278,6 +278,7 @@ class SubagentTracker {
             this.subagents.set(id, {
                 id,
                 generation: existing?.generation ?? createSubagentGeneration(),
+                kind: 'tool',
                 status: existing?.status ?? 'pending',
                 toolCall: {
                     id,
@@ -328,14 +329,14 @@ class SubagentTracker {
             return toolCallId;
         };
         for (const [toolCallId, subagent] of this.subagents) {
-            if (mapped.has(toolCallId))
+            if (subagent.kind !== 'tool' || mapped.has(toolCallId))
                 continue;
             if (subagent.toolCall.args['description'] === description) {
                 return establish(toolCallId);
             }
         }
         for (const [toolCallId, subagent] of this.subagents) {
-            if (mapped.has(toolCallId))
+            if (subagent.kind !== 'tool' || mapped.has(toolCallId))
                 continue;
             const subagentDescription = subagent.toolCall.args['description'];
             if (typeof subagentDescription !== 'string' || !subagentDescription)
@@ -344,7 +345,11 @@ class SubagentTracker {
                 return establish(toolCallId);
             }
         }
+        // Last-resort fallback — tool children only. A subgraph child is keyed by
+        // its own namespace and must never absorb an unrelated child's events.
         for (const [toolCallId, subagent] of this.subagents) {
+            if (subagent.kind !== 'tool')
+                continue;
             if (!mapped.has(toolCallId) && (subagent.status === 'pending' || subagent.status === 'running')) {
                 return establish(toolCallId);
             }
@@ -372,6 +377,46 @@ class SubagentTracker {
         });
         this.onSubagentChange?.();
     }
+    /**
+     * Register a plain-subgraph child stream on its first namespaced event.
+     *
+     * Unlike tool children — announced ahead of time by the parent's tool call —
+     * a compiled child added as a plain node has no announcement: its existence
+     * is learned from the first event carrying its namespace. It starts
+     * 'running' because by the time we see an event, it is.
+     */
+    ensureSubgraphStream(key, name) {
+        if (this.subagents.has(key))
+            return;
+        this.subagents.set(key, {
+            id: key,
+            generation: createSubagentGeneration(),
+            kind: 'subgraph',
+            status: 'running',
+            toolCall: { id: key, name, args: {} },
+            values: {},
+            messages: [],
+        });
+        this.onSubagentChange?.();
+    }
+    /**
+     * Settle still-running subgraph children when the run reaches a terminal
+     * outcome. Tool children settle through their tool result
+     * (`processToolMessage`); subgraph children have no result message, so the
+     * run's own settle is their completion signal. Paused/interrupted runs must
+     * NOT call this — a child can resume with the thread.
+     */
+    settleRunningSubgraphs(outcome) {
+        let changed = false;
+        for (const [key, subagent] of this.subagents) {
+            if (subagent.kind !== 'subgraph' || subagent.status !== 'running')
+                continue;
+            this.subagents.set(key, { ...subagent, status: outcome });
+            changed = true;
+        }
+        if (changed)
+            this.onSubagentChange?.();
+    }
     updateSubagentValues(namespaceId, values) {
         const toolCallId = this.resolveToolCallId(namespaceId);
         const subagent = this.subagents.get(toolCallId);
@@ -437,12 +482,40 @@ class SubagentTracker {
         return this.namespaceToToolCallId.get(namespaceId) ?? namespaceId;
     }
 }
-function isSubagentNamespace(namespace) {
+/**
+ * True when a stream event belongs to a child graph rather than the parent —
+ * i.e. it carries any namespace at all. This is the single classification
+ * question; which child owns the event is a separate (attribution) question.
+ *
+ * Kept consistent with the terminal-evidence guard, which has always refused
+ * ANY namespaced event as proof the parent run finished.
+ */
+function isChildNamespace(namespace) {
     if (!namespace)
         return false;
     if (typeof namespace === 'string')
-        return namespace.includes('tools:');
-    return namespace.some(segment => segment.startsWith('tools:'));
+        return namespace.length > 0;
+    return namespace.length > 0;
+}
+/**
+ * Derive a child stream's identity from an event namespace.
+ *
+ * `tools:<id>` segments identify a tool-dispatched child by its tool-call id.
+ * Any other segment (e.g. `research:<uuid>` from a compiled graph added with
+ * `add_node`) identifies a plain subgraph child: the full segment is the key
+ * (unique per invocation) and the part before the first ':' is the node name.
+ */
+function childStreamRefFromNamespace(namespace) {
+    for (const segment of namespace) {
+        if (segment.startsWith('tools:')) {
+            return { key: segment.slice(6), name: '', kind: 'tool' };
+        }
+    }
+    const first = namespace[0];
+    if (!first)
+        return undefined;
+    const colon = first.indexOf(':');
+    return { key: first, name: colon > 0 ? first.slice(0, colon) : first, kind: 'subgraph' };
 }
 function extractToolCallIdFromNamespace(namespace) {
     if (!namespace)
@@ -649,6 +722,13 @@ function createStreamManagerBridge({ options, subjects, threadId$, destroy$ }) {
                 continue;
             finalizeMessage(attempt, id, outcome);
         }
+        // Subgraph children have no tool result to settle them; the run's own
+        // terminal outcome is their completion signal. Paused/interrupted runs
+        // are excluded — a child can resume with the thread.
+        if (outcome === 'success' || outcome === 'error' || outcome === 'aborted') {
+            subagentManager.settleRunningSubgraphs(outcome === 'success' ? 'complete' : 'error');
+            publishSubagents();
+        }
     }
     function finishOutcome(attempt) {
         return attempt.terminalOutcome
@@ -1118,17 +1198,23 @@ function createStreamManagerBridge({ options, subjects, threadId$, destroy$ }) {
             const normalized = options.toMessage
                 ? msgs.map(options.toMessage)
                 : msgs;
-            if (isSubagentNamespace(namespace)) {
-                const namespaceId = namespace ? extractToolCallIdFromNamespace(namespace) : undefined;
-                if (namespaceId) {
+            // Any namespaced message event is child content. It feeds the child's
+            // stream and never merges into the parent transcript — the parent
+            // transcript is what the parent graph says. Shared-state children still
+            // surface at settle through the authoritative top-level `values` sync.
+            if (isChildNamespace(namespace)) {
+                const child = namespace ? childStreamRefFromNamespace(namespace) : undefined;
+                if (child) {
+                    if (child.kind === 'subgraph') {
+                        subagentManager.ensureSubgraphStream(child.key, child.name);
+                    }
                     for (const msg of normalized) {
-                        subagentManager.addMessageToSubagent(namespaceId, msg);
+                        subagentManager.addMessageToSubagent(child.key, msg);
                     }
                     publishSubagents();
                 }
-                if (options.filterSubagentMessages) {
-                    return;
-                }
+                storeMessageMetadata(normalized, event);
+                return;
             }
             // Partial and message-tuple events are incremental. Merge them by id
             // so optimistic human messages and earlier tool messages are preserved.
@@ -1143,7 +1229,7 @@ function createStreamManagerBridge({ options, subjects, threadId$, destroy$ }) {
                 const merged = mergeMessages(subjects.messages$.value, normalized, reasoningTimingMap, mode, canonicalMessageIds, affectedMessageIds, activeAttempt?.currentAssistantMessageId !== undefined
                     && activeAttempt.currentStepHasTerminalEvidence !== true);
                 subjects.messages$.next(merged);
-                if (!isSubagentNamespace(namespace)) {
+                {
                     trackAssistantMessages(merged.filter(message => {
                         const id = message['id'];
                         return typeof id === 'string' && affectedMessageIds.has(id);
@@ -1166,7 +1252,7 @@ function createStreamManagerBridge({ options, subjects, threadId$, destroy$ }) {
                 const affectedMessageIds = new Set();
                 const preserved = preserveIds(subjects.messages$.value, normalized, affectedMessageIds);
                 subjects.messages$.next(preserved);
-                if (!isSubagentNamespace(namespace)) {
+                {
                     trackAssistantMessages(preserved.filter(message => {
                         const id = message['id'];
                         return typeof id === 'string' && affectedMessageIds.has(id);
@@ -1185,8 +1271,11 @@ function createStreamManagerBridge({ options, subjects, threadId$, destroy$ }) {
         switch (baseType) {
             case 'values': {
                 const vals = extractEventData(event);
-                if (isSubagentNamespace(namespace) && isRecord$1(vals)) {
-                    updateSubagentValues(namespace, vals);
+                if (isChildNamespace(namespace)) {
+                    // A child's state must not clobber the parent's `values$` — route it
+                    // to the child stream and stop.
+                    if (isRecord$1(vals))
+                        updateSubagentValues(namespace, vals);
                     break;
                 }
                 if ((namespace?.length ?? 0) === 0) {
@@ -1252,7 +1341,9 @@ function createStreamManagerBridge({ options, subjects, threadId$, destroy$ }) {
             }
             case 'updates': {
                 const upd = extractEventData(event);
-                if (isSubagentNamespace(namespace)) {
+                if (isChildNamespace(namespace)) {
+                    // A child's updates must not spread-merge into the parent's
+                    // `values$` — they only mark the child stream running.
                     markSubagentRunning(namespace);
                     break;
                 }
@@ -1315,24 +1406,34 @@ function createStreamManagerBridge({ options, subjects, threadId$, destroy$ }) {
         publishSubagents();
     }
     function updateSubagentValues(namespace, values) {
-        const namespaceId = namespace ? extractToolCallIdFromNamespace(namespace) : undefined;
-        if (!namespaceId)
+        const child = namespace ? childStreamRefFromNamespace(namespace) : undefined;
+        if (!child)
             return;
-        const messages = values['messages'];
-        if (Array.isArray(messages) && messages.length > 0) {
-            const first = messages[0];
-            if (isRecord$1(first) && (first['type'] === 'human' || first['type'] === 'user') && typeof first['content'] === 'string') {
-                subagentManager.matchSubgraphToSubagent(namespaceId, first['content']);
+        if (child.kind === 'tool') {
+            // Attribution ladder applies to tool children only: their namespace id
+            // may need mapping onto a registered tool call.
+            const messages = values['messages'];
+            if (Array.isArray(messages) && messages.length > 0) {
+                const first = messages[0];
+                if (isRecord$1(first) && (first['type'] === 'human' || first['type'] === 'user') && typeof first['content'] === 'string') {
+                    subagentManager.matchSubgraphToSubagent(child.key, first['content']);
+                }
             }
         }
-        subagentManager.updateSubagentValues(namespaceId, values);
+        else {
+            subagentManager.ensureSubgraphStream(child.key, child.name);
+        }
+        subagentManager.updateSubagentValues(child.key, values);
         publishSubagents();
     }
     function markSubagentRunning(namespace) {
-        const namespaceId = namespace ? extractToolCallIdFromNamespace(namespace) : undefined;
-        if (!namespaceId)
+        const child = namespace ? childStreamRefFromNamespace(namespace) : undefined;
+        if (!child)
             return;
-        subagentManager.markRunningFromNamespace(namespaceId, namespace);
+        if (child.kind === 'subgraph') {
+            subagentManager.ensureSubgraphStream(child.key, child.name);
+        }
+        subagentManager.markRunningFromNamespace(child.key, namespace);
         publishSubagents();
     }
     function publishSubagents() {
@@ -2130,9 +2231,13 @@ function toSubagentRefs(subagents) {
     subagents.forEach((subagent, key) => {
         refs.set(key, {
             toolCallId: subagent.id,
+            // Tool children are named by their `subagent_type` arg; subgraph
+            // children by their node name (stored as the synthetic toolCall name).
             name: typeof subagent.toolCall.args['subagent_type'] === 'string'
                 ? subagent.toolCall.args['subagent_type']
-                : undefined,
+                : subagent.kind === 'subgraph'
+                    ? subagent.toolCall.name
+                    : undefined,
             status: signal(subagent.status),
             values: signal(subagent.values),
             messages: signal(subagent.messages),
@@ -3424,7 +3529,6 @@ function agentFactory() {
         ...(config.transport !== undefined ? { transport: config.transport } : {}),
         ...(config.clientOptions !== undefined ? { clientOptions: config.clientOptions } : {}),
         ...(config.telemetry !== undefined ? { telemetry: config.telemetry } : {}),
-        ...(config.filterSubagentMessages !== undefined ? { filterSubagentMessages: config.filterSubagentMessages } : {}),
         ...(config.subagentToolNames !== undefined ? { subagentToolNames: config.subagentToolNames } : {}),
         ...(config.transcriptNodeNames !== undefined ? { transcriptNodeNames: config.transcriptNodeNames } : {}),
     });