@thehammer/danx-session-telemetry 0.1.9 → 0.1.11

package/dist/session-log-watcher.d.ts

@@ -90,20 +90,21 @@ export interface SessionLogWatcherOptions {
  * The `.`/`_` encoding matters: a hidden-dir segment (leading `.`) or a
  * clean-room key's `__` delimiter that the encoder does NOT collapse produces
  * a directory Claude Code never writes to — silently breaking
- * SessionLogWatcher attachment for every host-mode dispatch. The
- * separator-encoding regression tests below pin this.
+ * SessionLogWatcher attachment. The separator-encoding regression tests
+ * below pin this.
  *
- * Host mode (claude 2.1.104+, GitHub #51488 — symlinks under
- * `~/.claude/projects/` are no longer followed) lets claude write its
- * transcript to this exact realpath-encoded dir as a REAL directory; the
- * watcher reads the same path, so they converge with no symlink. Docker
- * binds `<repo>/claude-projects/` at `/home/danxbot/.claude/projects/` and
- * relies on the same encoding for the bound subdir name. Keep the encoding
- * logic centralized here so both runtimes stay in lockstep.
+ * Claude 2.1.104+ (GitHub #51488 — symlinks under `~/.claude/projects/` are
+ * no longer followed) writes its transcript to this exact realpath-encoded
+ * dir as a REAL directory; the watcher reads the same path, so they converge
+ * with no symlink. A danxbot worker container binds `<repo>/claude-projects/`
+ * at `/home/danxbot/.claude/projects/` and relies on the same encoding for
+ * the bound subdir name; an operator's own interactive session on its
+ * machine uses the native dir. Keep the encoding logic centralized here so
+ * every caller stays in lockstep.
  *
- * DX-3069 — Windows host-mode dispatches (this dev machine included, see
- * `.claude/rules/agent-dispatch.md` "Host" runtime) pass an ALREADY-ABSOLUTE
- * Windows cwd (`C:\Users\...`), which never starts with `/`. The prior
+ * DX-3069 — a Windows caller (an operator session on a Windows machine)
+ * passes an ALREADY-ABSOLUTE Windows cwd (`C:\Users\...`), which never
+ * starts with `/`. The prior
  * `cwd.startsWith("/") ? cwd : "/" + cwd` unconditionally prepended `/`,
  * producing a malformed path (`/C:\Users\...`) that `realpathSync` cannot
  * resolve (ENOENT — Windows reads the leading `/C:` as its own root segment),
@@ -112,7 +113,7 @@ export interface SessionLogWatcherOptions {
  * drive-letter colon survived verbatim into the "encoded" dir name, and
  * `join(homedir(), ".claude", "projects", encoded)` then produced a NESTED,
  * DOUBLED path — `C:\Users\...\.claude\projects\-C:\Users\...` — a directory
- * Claude Code never writes to, so every host-mode watcher on Windows silently
+ * Claude Code never writes to, so every watcher on Windows silently
  * never found its own transcript. Verified empirically against a real Claude
  * Code install on this machine: `C:\Users\newms\projects\danxbot` encodes on
  * disk to `C--Users-newms-projects-danxbot` (`ls ~/.claude/projects`) — no

package/dist/session-log-watcher.js

@@ -58,20 +58,21 @@ export const DISPATCH_TAG_PREFIX = "<!-- danxbot-dispatch:";
  * The `.`/`_` encoding matters: a hidden-dir segment (leading `.`) or a
  * clean-room key's `__` delimiter that the encoder does NOT collapse produces
  * a directory Claude Code never writes to — silently breaking
- * SessionLogWatcher attachment for every host-mode dispatch. The
- * separator-encoding regression tests below pin this.
+ * SessionLogWatcher attachment. The separator-encoding regression tests
+ * below pin this.
  *
- * Host mode (claude 2.1.104+, GitHub #51488 — symlinks under
- * `~/.claude/projects/` are no longer followed) lets claude write its
- * transcript to this exact realpath-encoded dir as a REAL directory; the
- * watcher reads the same path, so they converge with no symlink. Docker
- * binds `<repo>/claude-projects/` at `/home/danxbot/.claude/projects/` and
- * relies on the same encoding for the bound subdir name. Keep the encoding
- * logic centralized here so both runtimes stay in lockstep.
+ * Claude 2.1.104+ (GitHub #51488 — symlinks under `~/.claude/projects/` are
+ * no longer followed) writes its transcript to this exact realpath-encoded
+ * dir as a REAL directory; the watcher reads the same path, so they converge
+ * with no symlink. A danxbot worker container binds `<repo>/claude-projects/`
+ * at `/home/danxbot/.claude/projects/` and relies on the same encoding for
+ * the bound subdir name; an operator's own interactive session on its
+ * machine uses the native dir. Keep the encoding logic centralized here so
+ * every caller stays in lockstep.
  *
- * DX-3069 — Windows host-mode dispatches (this dev machine included, see
- * `.claude/rules/agent-dispatch.md` "Host" runtime) pass an ALREADY-ABSOLUTE
- * Windows cwd (`C:\Users\...`), which never starts with `/`. The prior
+ * DX-3069 — a Windows caller (an operator session on a Windows machine)
+ * passes an ALREADY-ABSOLUTE Windows cwd (`C:\Users\...`), which never
+ * starts with `/`. The prior
  * `cwd.startsWith("/") ? cwd : "/" + cwd` unconditionally prepended `/`,
  * producing a malformed path (`/C:\Users\...`) that `realpathSync` cannot
  * resolve (ENOENT — Windows reads the leading `/C:` as its own root segment),
@@ -80,7 +81,7 @@ export const DISPATCH_TAG_PREFIX = "<!-- danxbot-dispatch:";
  * drive-letter colon survived verbatim into the "encoded" dir name, and
  * `join(homedir(), ".claude", "projects", encoded)` then produced a NESTED,
  * DOUBLED path — `C:\Users\...\.claude\projects\-C:\Users\...` — a directory
- * Claude Code never writes to, so every host-mode watcher on Windows silently
+ * Claude Code never writes to, so every watcher on Windows silently
  * never found its own transcript. Verified empirically against a real Claude
  * Code install on this machine: `C:\Users\newms\projects\danxbot` encodes on
  * disk to `C--Users-newms-projects-danxbot` (`ls ~/.claude/projects`) — no
@@ -239,8 +240,8 @@ export async function findSessionFileByDispatchId(dir, dispatchId) {
             catch (err) {
                 // DX-1201 — a single unreadable SIBLING file must NOT abort the
                 // whole scan. Dispatched agents share a busy session dir (e.g.
-                // host-mode issue-worker dispatches all writing under
-                // `~/.claude/projects/-home-newms-web-danxbot/`), so any one of the
+                // every issue-worker dispatch of one repo writing under the same
+                // encoded `~/.claude/projects/<cwd>/` dir), so any one of the
                 // co-located multi-MB files can momentarily fail an `open`/`stat`/
                 // `read`: ENOENT (sibling rotated/deleted mid-scan), ESTALE (a
                 // worktree torn down under the agent), EACCES (perms race), EMFILE

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.11",
   "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",