@threadplane/langgraph 0.0.53 → 0.0.55

package/README.md

@@ -147,7 +147,7 @@ import { provideAgent, LangGraphThreadsAdapter, LANGGRAPH_THREADS_CONFIG } from
 
 export const appConfig: ApplicationConfig = {
   providers: [
-    provideAgent({ apiUrl: 'https://your-langgraph-platform.com' }),
+    provideAgent({ apiUrl: 'https://your-langgraph-platform.com', assistantId: 'my-agent' }),
     { provide: LANGGRAPH_THREADS_CONFIG, useValue: { apiUrl: 'https://your-langgraph-platform.com' } },
     LangGraphThreadsAdapter,
   ],

package/fesm2022/threadplane-langgraph.mjs

@@ -496,7 +496,7 @@ function isLgTraceEnabled() {
 function lgTrace(...args) {
     if (isLgTraceEnabled()) {
         // eslint-disable-next-line no-console
-        console.debug('[ngaf-chat-stream]', ...args);
+        console.debug('[tplane-chat-stream]', ...args);
     }
 }
 function captureAgentRuntimeTelemetry(sink, event, properties) {
@@ -540,6 +540,10 @@ function createStreamManagerBridge({ options, subjects, threadId$, destroy$ }) {
     /** True when the current abort was user-initiated (via stop()). Reset at the start of every new runStream(). */
     let userAbortRequested = false;
     const toolProgressMap = new Map();
+    // Message ids whose content is known-final (installed by a canonical
+    // replacement). Late streamed deltas for these ids are stale stragglers and
+    // are ignored — decided by identity, never by comparing text to text.
+    const canonicalMessageIds = new Set();
     const queuedRuns = [];
     let drainingQueue = false;
     const subagentManager = new SubagentTracker({
@@ -547,9 +551,9 @@ function createStreamManagerBridge({ options, subjects, threadId$, destroy$ }) {
         onSubagentChange: publishSubagents,
     });
     const telemetryProperties = { transport: 'langgraph', surface: 'agent' };
-    captureAgentRuntimeTelemetry(options.telemetry, 'ngaf:runtime_instance_created', telemetryProperties);
+    captureAgentRuntimeTelemetry(options.telemetry, 'tplane:runtime_instance_created', telemetryProperties);
     function captureRuntimeRequestTelemetry(requestType) {
-        captureAgentRuntimeTelemetry(options.telemetry, 'ngaf:runtime_request_created', {
+        captureAgentRuntimeTelemetry(options.telemetry, 'tplane:runtime_request_created', {
             ...telemetryProperties,
             requestType,
         });
@@ -579,6 +583,7 @@ function createStreamManagerBridge({ options, subjects, threadId$, destroy$ }) {
         toolProgressMap.clear();
         subagentManager.clear();
         reasoningTimingMap.clear();
+        canonicalMessageIds.clear();
     }
     function setThreadId(id, resetState) {
         if (resetState) {
@@ -734,7 +739,7 @@ function createStreamManagerBridge({ options, subjects, threadId$, destroy$ }) {
         abortController = new AbortController();
         const startedAt = Date.now();
         captureRuntimeRequestTelemetry('join_queued');
-        captureAgentRuntimeTelemetry(options.telemetry, 'ngaf:stream_started', telemetryProperties);
+        captureAgentRuntimeTelemetry(options.telemetry, 'tplane:stream_started', telemetryProperties);
         subjects.custom$.next([]);
         subjects.toolProgress$.next([]);
         toolProgressMap.clear();
@@ -754,7 +759,7 @@ function createStreamManagerBridge({ options, subjects, threadId$, destroy$ }) {
                 // post-process node mutations (RemoveMessage, id-match content
                 // replacement) reflected on the server are picked up client-side.
                 await refreshHistory(true);
-                captureAgentRuntimeTelemetry(options.telemetry, 'ngaf:stream_ended', {
+                captureAgentRuntimeTelemetry(options.telemetry, 'tplane:stream_ended', {
                     ...telemetryProperties,
                     durationMs: Date.now() - startedAt,
                 });
@@ -763,7 +768,7 @@ function createStreamManagerBridge({ options, subjects, threadId$, destroy$ }) {
         catch (err) {
             subjects.error$.next(toAgentError(err));
             subjects.status$.next(ResourceStatus.Error);
-            captureAgentRuntimeTelemetry(options.telemetry, 'ngaf:stream_errored', {
+            captureAgentRuntimeTelemetry(options.telemetry, 'tplane:stream_errored', {
                 ...telemetryProperties,
                 durationMs: Date.now() - startedAt,
                 errorClass: agentRuntimeTelemetryErrorClass(err),
@@ -776,12 +781,13 @@ function createStreamManagerBridge({ options, subjects, threadId$, destroy$ }) {
         userAbortRequested = false;
         const startedAt = Date.now();
         captureRuntimeRequestTelemetry(requestType);
-        captureAgentRuntimeTelemetry(options.telemetry, 'ngaf:stream_started', telemetryProperties);
+        captureAgentRuntimeTelemetry(options.telemetry, 'tplane:stream_started', telemetryProperties);
         subjects.status$.next(ResourceStatus.Loading);
         subjects.error$.next(undefined);
         subjects.custom$.next([]);
         subjects.toolProgress$.next([]);
         toolProgressMap.clear();
+        canonicalMessageIds.clear();
         lastPayload = payload;
         lastOptions = opts;
         // Tracks whether at least one stream event has been processed this run.
@@ -820,7 +826,7 @@ function createStreamManagerBridge({ options, subjects, threadId$, destroy$ }) {
                 // authoritative after run completion.
                 await refreshHistory(true);
                 await drainQueue();
-                captureAgentRuntimeTelemetry(options.telemetry, 'ngaf:stream_ended', {
+                captureAgentRuntimeTelemetry(options.telemetry, 'tplane:stream_ended', {
                     ...telemetryProperties,
                     durationMs: Date.now() - startedAt,
                 });
@@ -839,7 +845,7 @@ function createStreamManagerBridge({ options, subjects, threadId$, destroy$ }) {
                     : new AgentError({ kind: 'connection', message: AGENT_ERROR_MESSAGES.connection, retryable: true, cause: err });
                 subjects.error$.next(e);
                 subjects.status$.next(ResourceStatus.Error);
-                captureAgentRuntimeTelemetry(options.telemetry, 'ngaf:stream_errored', {
+                captureAgentRuntimeTelemetry(options.telemetry, 'tplane:stream_errored', {
                     ...telemetryProperties,
                     durationMs: Date.now() - startedAt,
                     errorClass: agentRuntimeTelemetryErrorClass(err),
@@ -848,7 +854,7 @@ function createStreamManagerBridge({ options, subjects, threadId$, destroy$ }) {
             else {
                 subjects.error$.next(toAgentError(err));
                 subjects.status$.next(ResourceStatus.Error);
-                captureAgentRuntimeTelemetry(options.telemetry, 'ngaf:stream_errored', {
+                captureAgentRuntimeTelemetry(options.telemetry, 'tplane:stream_errored', {
                     ...telemetryProperties,
                     durationMs: Date.now() - startedAt,
                     errorClass: agentRuntimeTelemetryErrorClass(err),
@@ -881,7 +887,8 @@ function createStreamManagerBridge({ options, subjects, threadId$, destroy$ }) {
             // Partial and message-tuple events are incremental. Merge them by id
             // so optimistic human messages and earlier tool messages are preserved.
             if (event.type === 'messages/partial' || event.messageMetadata) {
-                subjects.messages$.next(mergeMessages(subjects.messages$.value, normalized, reasoningTimingMap));
+                const mode = event.messageMetadata ? 'delta' : 'snapshot';
+                subjects.messages$.next(mergeMessages(subjects.messages$.value, normalized, reasoningTimingMap, mode, canonicalMessageIds));
                 if (isLgTraceEnabled()) {
                     const msgs = subjects.messages$.value;
                     const last = msgs[msgs.length - 1];
@@ -950,7 +957,7 @@ function createStreamManagerBridge({ options, subjects, threadId$, destroy$ }) {
                         // drop the partial AI (or even the optimistic human) and
                         // tear down their DOM mid-stream. Merge by id keeps both,
                         // updates content where ids match, preserves the rest.
-                        subjects.messages$.next(mergeMessages(subjects.messages$.value, remapped, reasoningTimingMap));
+                        subjects.messages$.next(mergeMessages(subjects.messages$.value, remapped, reasoningTimingMap, 'snapshot', canonicalMessageIds));
                         if (isLgTraceEnabled()) {
                             lgTrace('bridge.values-sync', {
                                 incomingLength: stateMessages.length,
@@ -1146,7 +1153,7 @@ function createStreamManagerBridge({ options, subjects, threadId$, destroy$ }) {
             abortController = new AbortController();
             const startedAt = Date.now();
             captureRuntimeRequestTelemetry('join');
-            captureAgentRuntimeTelemetry(options.telemetry, 'ngaf:stream_started', telemetryProperties);
+            captureAgentRuntimeTelemetry(options.telemetry, 'tplane:stream_started', telemetryProperties);
             subjects.custom$.next([]);
             subjects.toolProgress$.next([]);
             toolProgressMap.clear();
@@ -1160,7 +1167,7 @@ function createStreamManagerBridge({ options, subjects, threadId$, destroy$ }) {
                 }
                 subjects.status$.next(ResourceStatus.Resolved);
                 await refreshHistory();
-                captureAgentRuntimeTelemetry(options.telemetry, 'ngaf:stream_ended', {
+                captureAgentRuntimeTelemetry(options.telemetry, 'tplane:stream_ended', {
                     ...telemetryProperties,
                     durationMs: Date.now() - startedAt,
                 });
@@ -1168,7 +1175,7 @@ function createStreamManagerBridge({ options, subjects, threadId$, destroy$ }) {
             catch (err) {
                 subjects.error$.next(toAgentError(err));
                 subjects.status$.next(ResourceStatus.Error);
-                captureAgentRuntimeTelemetry(options.telemetry, 'ngaf:stream_errored', {
+                captureAgentRuntimeTelemetry(options.telemetry, 'tplane:stream_errored', {
                     ...telemetryProperties,
                     durationMs: Date.now() - startedAt,
                     errorClass: agentRuntimeTelemetryErrorClass(err),
@@ -1360,7 +1367,7 @@ function collapseAdjacentAi(messages) {
     }
     return out;
 }
-function mergeMessages(existing, incoming, reasoningTimingMap) {
+function mergeMessages(existing, incoming, reasoningTimingMap, mode = 'snapshot', canonicalMessageIds) {
     const merged = [...existing];
     for (const msg of incoming) {
         const rawIn = msg;
@@ -1400,6 +1407,13 @@ function mergeMessages(existing, incoming, reasoningTimingMap) {
             const existing = merged[idx];
             const existingId = existing['id'];
             const incomingRaw = msg;
+            const targetId = (existingId ?? incomingRaw['id']);
+            // Identity backstop: once a message's content is known-final, late
+            // streamed deltas for it are stale stragglers — ignore them outright.
+            if (mode === 'delta' && targetId && canonicalMessageIds?.has(targetId)
+                && !isFinalCanonicalReasoningContent(incomingRaw['content'])) {
+                continue;
+            }
             // Keep the *existing* id so downstream track-by-id sees stable identity.
             // For complex-content streaming (OpenAI gpt-5/o-series, Anthropic) the
             // SDK emits per-chunk *delta* arrays — not accumulated arrays — so a
@@ -1407,7 +1421,10 @@ function mergeMessages(existing, incoming, reasoningTimingMap) {
             // latest token. Accumulate text-bearing content across chunks here
             // and hand a string to consumers; downstream code already handles
             // string content uniformly.
-            const accumulatedContent = accumulateContent(existing.content, incomingRaw['content']);
+            const accumulatedContent = accumulateContent(existing.content, incomingRaw['content'], mode);
+            if (targetId && isFinalCanonicalReasoningContent(incomingRaw['content'])) {
+                canonicalMessageIds?.add(targetId);
+            }
             // Only accumulate reasoning when the incoming message explicitly carries
             // a `reasoning` field or complex-content array blocks with
             // type='reasoning'/'thinking'. Never use a plain string content value
@@ -1456,12 +1473,22 @@ function mergeMessages(existing, incoming, reasoningTimingMap) {
 }
 /**
  * Merge an incoming chunk's content into prior accumulated content for the
- * same message id.
+ * same message id. Behavior is governed by `mode`, which reflects the
+ * DECLARED kind of the source event rather than a guess from comparing text:
+ *
+ * - mode 'delta' (messages-tuple / `event.messageMetadata` truthy): the
+ *   payload is a genuine per-chunk delta. Append unconditionally — a
+ *   prefix-comparison "dedupe" here would silently drop legitimate tokens
+ *   that coincide with the message-so-far (e.g. every bare "|" while
+ *   streaming a markdown table). Staleness after the message goes canonical
+ *   is instead handled by identity in `mergeMessages` (canonicalMessageIds).
+ * - mode 'snapshot' (messages/partial, values-sync): the payload carries the
+ *   message-so-far, not a delta, so mutual prefix comparison picks the
+ *   longer state and ignores stale shorter snapshots.
  *
- * - string + string → concat (delta append)
- * - array + array  → concat extracted text from existing + incoming blocks
- * - array + string → use the string (server final-id swap)
- * - empty existing → use incoming as-is
+ * In both modes, a "final canonical" reasoning+text array (see
+ * `isFinalCanonicalReasoningContent`) always replaces whatever was
+ * accumulated — it's the authoritative final message, not another chunk.
  *
  * We deliberately collapse complex content arrays to a string at this layer.
  * The langgraph-sdk client does not accumulate complex-content arrays the
@@ -1493,7 +1520,7 @@ function isFinalCanonicalReasoningContent(content) {
     }
     return hasReasoning && hasText;
 }
-function accumulateContent(existing, incoming) {
+function accumulateContent(existing, incoming, mode = 'snapshot') {
     const existingText = extractText(existing);
     const incomingText = extractText(incoming);
     // Always return a string. We never want array content escaping the bridge:
@@ -1504,23 +1531,24 @@ function accumulateContent(existing, incoming) {
         return incomingText;
     if (incomingText.length === 0)
         return existingText;
-    // Incoming is a strict-superset of accumulated (final-id swap with full content).
+    // Final-canonical detection applies in both modes: the authoritative
+    // "reasoning + text" array replaces whatever was accumulated.
+    if (isFinalCanonicalReasoningContent(incoming))
+        return incomingText;
+    if (mode === 'delta') {
+        // Tuple chunks are declared deltas. Append unconditionally — any
+        // text-comparison "dedupe" here can silently drop legitimate tokens
+        // that coincide with the message prefix (e.g. every bare "|" in a
+        // markdown table). Staleness is handled by identity in mergeMessages.
+        return existingText + incomingText;
+    }
+    // Snapshot mode (messages/partial, values-sync): payloads carry the
+    // message-so-far, so mutual prefix comparison picks the longer state and
+    // ignores stale shorter snapshots.
     if (incomingText.startsWith(existingText))
         return incomingText;
-    // Existing already a strict-superset — chunk arrived after the canonical
-    // message merged in via values-sync. Keep what we have.
     if (existingText.startsWith(incomingText))
         return existingText;
-    // Final-canonical detection: when incoming is the "reasoning + text"
-    // array shape that ships the authoritative final message after a
-    // streaming run, replace the partial streamed accumulator with the
-    // canonical text instead of appending. Without this branch a small
-    // formatting difference between the streamed accumulator and the
-    // canonical text breaks the prefix checks above and visible content
-    // is duplicated (`existingText + incomingText`).
-    if (isFinalCanonicalReasoningContent(incoming))
-        return incomingText;
-    // Otherwise treat incoming as a delta and append.
     return existingText + incomingText;
 }
 function extractText(content) {