@thehammer/danx-session-telemetry 0.1.9 → 0.1.10

package/dist/usage-ledger.js

@@ -292,27 +292,31 @@ export function ledgerLineFromLogEntry(entry) {
         ? d.content
         : [];
     const hasMessageId = typeof d.messageId === "string" && d.messageId.length > 0;
+    const hasUsage = Boolean(d.usage && typeof d.usage === "object");
+    const toolUseIds = content
+        .filter((b) => b.type === "tool_use" && typeof b.id === "string")
+        .map((b) => b.id);
     // DX-3261 — `byteOffset` only matters as the synthetic key for a line with
     // no `message.id` (`syntheticMessageKey`); a missing/invalid one used to
     // silently default to `0`, collapsing EVERY no-id reply from this source
     // onto one key so only the largest single reply was ever credited. Fail
-    // loud rather than repeat that silent collapse. Harmless when a real
-    // `messageId` is present (byteOffset is never used to key that line), so
-    // only fail when the line actually needs it.
-    if (!hasMessageId && typeof d.byte_offset !== "number") {
-        throw new LedgerLineDataError(`ledgerLineFromLogEntry: missing/invalid byte_offset on a no-message-id line (source=${typeof d.subagent_id === "string" ? d.subagent_id : MAIN_SOURCE})`);
+    // loud rather than repeat that silent collapse — but ONLY when the line
+    // will actually reach that keying step: `UsageLedgerCounter.observe()`
+    // discards a no-id line with neither usage nor a tool-use id before ever
+    // building a key (a tool-call block with no reported `id`, an empty
+    // wrapper entry), so requiring a valid `byteOffset` on THOSE too would
+    // reject lines the counter was always going to ignore. Harmless when a
+    // real `messageId` is present (byteOffset is never used to key that line).
+    if (!hasMessageId && (hasUsage || toolUseIds.length > 0) && typeof d.byte_offset !== "number") {
+        throw new LedgerLineDataError(`ledgerLineFromLogEntry: missing/invalid byte_offset on a no-message-id line carrying usage or a tool-use id (source=${typeof d.subagent_id === "string" ? d.subagent_id : MAIN_SOURCE})`);
     }
     return {
         messageId: hasMessageId ? d.messageId : undefined,
         source: typeof d.subagent_id === "string" ? d.subagent_id : MAIN_SOURCE,
         byteOffset: typeof d.byte_offset === "number" ? d.byte_offset : 0,
         model: typeof d.model === "string" && d.model.length > 0 ? d.model : undefined,
-        usage: d.usage && typeof d.usage === "object"
-            ? d.usage
-            : undefined,
+        usage: hasUsage ? d.usage : undefined,
         timestamp: entry.timestamp,
-        toolUseIds: content
-            .filter((b) => b.type === "tool_use" && typeof b.id === "string")
-            .map((b) => b.id),
+        toolUseIds,
     };
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@thehammer/danx-session-telemetry",
-  "version": "0.1.9",
+  "version": "0.1.10",
   "description": "Shared Claude Code transcript collector (DX-2962) — incremental JSONL tailing for the danxbot worker and the danx-dashboard-mcp desktop bridge, so both run identical collector code.",
   "license": "MIT",
   "type": "module",