@thehammer/danx-session-telemetry 0.1.1 → 0.1.3

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

@@ -89,6 +89,28 @@ export interface SessionLogWatcherOptions {
  * 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.
+ *
+ * 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
+ * `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),
+ * so the code fell into the catch and kept that malformed string. The old
+ * regex (`/[/._]/g`) never touched `\` or `:`, so every backslash and the
+ * 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
+ * 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
+ * leading dash, `:` and each `\` both become `-`, exactly like `/` and `.` do
+ * for a POSIX cwd. `isAbsolute` (platform-native — win32 on Windows, posix
+ * elsewhere) replaces the POSIX-only `startsWith("/")` check so an
+ * already-absolute Windows path is never re-prefixed, and the replacement
+ * regex now also covers `\` and `:` so no separator survives into the
+ * encoded name on either platform.
  */
 export declare function encodeClaudeProjectsCwd(cwd: string): string;
 /**

package/dist/session-log-watcher.js

@@ -23,7 +23,7 @@
 import { existsSync, realpathSync } from "node:fs";
 import { open, readFile, readdir, stat } from "node:fs/promises";
 import { homedir } from "node:os";
-import { basename, dirname, join } from "node:path";
+import { basename, dirname, isAbsolute, join } from "node:path";
 import { createLogger } from "./internal-logger.js";
 import { buildAssistantSummary, buildToolResultSummary, } from "./tool-summary.js";
 const log = createLogger("session-log-watcher");
@@ -68,9 +68,31 @@ export const DISPATCH_TAG_PREFIX = "<!-- danxbot-dispatch:";
  * 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.
+ *
+ * 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
+ * `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),
+ * so the code fell into the catch and kept that malformed string. The old
+ * regex (`/[/._]/g`) never touched `\` or `:`, so every backslash and the
+ * 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
+ * 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
+ * leading dash, `:` and each `\` both become `-`, exactly like `/` and `.` do
+ * for a POSIX cwd. `isAbsolute` (platform-native — win32 on Windows, posix
+ * elsewhere) replaces the POSIX-only `startsWith("/")` check so an
+ * already-absolute Windows path is never re-prefixed, and the replacement
+ * regex now also covers `\` and `:` so no separator survives into the
+ * encoded name on either platform.
  */
 export function encodeClaudeProjectsCwd(cwd) {
-    const normalized = cwd.startsWith("/") ? cwd : `/${cwd}`;
+    const normalized = isAbsolute(cwd) ? cwd : `/${cwd}`;
     let resolved;
     try {
         resolved = realpathSync(normalized);
@@ -80,12 +102,13 @@ export function encodeClaudeProjectsCwd(cwd) {
         resolved = normalized;
     }
     // Claude Code encodes its cwd into ~/.claude/projects/<dir> by replacing
-    // EVERY non-alphanumeric separator — `/`, `.`, AND `_` — with `-`. The `_`
-    // matters: the clean-room key (`cleanRoomKey`, DX-1511)
-    // joins its components with `__`, so Claude writes the dir as `--` on disk.
-    // Omitting `_` here made the watcher wait on a `__` path Claude never
-    // creates → "session file not found after 180s" on every clean-room dispatch.
-    return resolved.replace(/[/._]/g, "-");
+    // EVERY non-alphanumeric separator with `-`: `/`, `.`, and `_` on every
+    // platform, plus `\` and the drive-letter `:` on Windows. The `_` matters:
+    // the clean-room key (`cleanRoomKey`, DX-1511) joins its components with
+    // `__`, so Claude writes the dir as `--` on disk. Omitting `_` here made
+    // the watcher wait on a `__` path Claude never creates → "session file not
+    // found after 180s" on every clean-room dispatch.
+    return resolved.replace(/[/\\:._]/g, "-");
 }
 /**
  * Derives the Claude Code session directory from a working directory path.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@thehammer/danx-session-telemetry",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "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",
@@ -37,6 +37,6 @@
   "devDependencies": {
     "@types/node": "^22.0.0",
     "typescript": "^5.7.0",
-    "vitest": "^4.1.5"
+    "vitest": "^4.0.18"
   }
 }