@amaster.ai/dsh-langfuse 0.1.35 → 0.1.37

package/README.md

@@ -7,7 +7,7 @@ Langfuse observability for [DeepSeek Harness (dsh)](https://github.com/deepseek-
 - one **generation** per LLM call (`llm/stream` waterfall), plus a nested `llm-request` span carrying the verbatim loop-built request
 - one **tool observation** per tool call (`tools/pre-execute` through `tools/result`)
 - one **span** per `hook/invoked` / `hook/result` pair, with hook point, decision, exit code, and duration (stderr summary follows `captureContent`)
-- one **trace** per session turn (`session/event`) — in the v5 SDK the trace IS its root span, ended (and thereby exported) at `turn/end`
+- one **trace** per session turn with observed activity (`session/event`) — the root is created on the first message or observed operation, retains the original turn start time, and ends at `turn/end`; completed turns without activity export nothing, while errors and cancellation remain visible
 - subagent child sessions nested under the parent's tree (`session/created` header link + `subagent/start` / `subagent/end`)
 - structured reasoning in generation output, optional image display, inherited user/tags, and final turn severity
 - optional W3C parent context supplied per session, without adopting the host's ambient OTEL context

package/lib/index.js

@@ -13,7 +13,7 @@
  *   shared identity).
  * - `hook/invoked` / `hook/result` session records → one **span** per hook run.
  * - `session/event` emit    → one Langfuse **trace** per agent turn
- *   (`turn/start` opens a root span — in the v5 SDK the root observation IS
+ *   (`turn/start` records the start; actual activity opens the root — in v5 it IS
  *   the trace — the turn's first `user/message` sets the input, `turn/end`
  *   stamps the end reason and ends the root, which is what exports it).
  * - Native A2A inbox tasks own a trace spanning admission through task termination;
@@ -590,8 +590,13 @@ export function apply(ctx, config) {
         }
     });
     function openTrace(sessionId, metadata, startTime) {
-        const context = sessionId ? traceContexts.get(sessionId) : undefined;
-        if (sessionId)
+        const pending = sessionId ? pendingTurns.get(sessionId) : undefined;
+        const context = pending
+            ? pending.context
+            : sessionId
+                ? traceContexts.get(sessionId)
+                : undefined;
+        if (sessionId && !pending)
             traceContexts.delete(sessionId);
         return reporter.openTrace({ name: config.traceName, sessionId, metadata, context, startTime });
     }
@@ -1120,17 +1125,20 @@ export function apply(ctx, config) {
                 if (stale)
                     endTurn(stale);
                 inboxInputs.delete(sessionId);
-                if (nativeInboxes.has(sessionId)) {
-                    sessions.delete(sessionId);
-                    pendingTurns.set(sessionId, { turn: event.data.turn, startTime: new Date(event.time) });
-                    break;
-                }
-                const trace = openTrace(sessionId, { turn: event.data.turn });
-                if (trace)
-                    sessions.set(sessionId, { trace, hasInput: false, turn: event.data.turn });
+                sessions.delete(sessionId);
+                pendingTurns.set(sessionId, {
+                    turn: event.data.turn,
+                    startTime: new Date(event.time),
+                    context: nativeInboxes.has(sessionId) ? undefined : traceContexts.get(sessionId),
+                });
+                // Ordinary turns consume request context at start, even if they remain empty.
+                if (!nativeInboxes.has(sessionId))
+                    traceContexts.delete(sessionId);
                 break;
             }
             case 'user/message': {
+                if (pendingTurns.has(sessionId))
+                    traceFor(sessionId);
                 const state = sessions.get(sessionId);
                 if (state && !state.hasInput && config.captureContent) {
                     setMessageInput(state.trace, event.data.content);
@@ -1139,6 +1147,8 @@ export function apply(ctx, config) {
                 break;
             }
             case 'assistant/message': {
+                if (pendingTurns.has(sessionId))
+                    traceFor(sessionId);
                 const state = sessions.get(sessionId);
                 if (state && config.captureContent) {
                     state.lastAssistantText = messageText(event.data.message) ?? state.lastAssistantText;
@@ -1146,8 +1156,10 @@ export function apply(ctx, config) {
                 break;
             }
             case 'turn/end': {
-                if (pendingTurns.has(sessionId))
+                if (pendingTurns.has(sessionId) &&
+                    (nativeInboxes.has(sessionId) || event.data.reason.kind !== 'completed'))
                     traceFor(sessionId);
+                pendingTurns.delete(sessionId);
                 const state = sessions.get(sessionId);
                 if (state) {
                     sessions.delete(sessionId);

package/lib/types/index.d.ts

@@ -13,7 +13,7 @@
  *   shared identity).
  * - `hook/invoked` / `hook/result` session records → one **span** per hook run.
  * - `session/event` emit    → one Langfuse **trace** per agent turn
- *   (`turn/start` opens a root span — in the v5 SDK the root observation IS
+ *   (`turn/start` records the start; actual activity opens the root — in v5 it IS
  *   the trace — the turn's first `user/message` sets the input, `turn/end`
  *   stamps the end reason and ends the root, which is what exports it).
  * - Native A2A inbox tasks own a trace spanning admission through task termination;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@amaster.ai/dsh-langfuse",
-  "version": "0.1.35",
+  "version": "0.1.37",
   "description": "Langfuse observability plugin for DeepSeek Harness: one generation per LLM call with a nested llm-request span, one span per tool call, one trace per session turn, subagent child sessions nested under the parent's tree.",
   "type": "module",
   "main": "lib/index.js",