hillclimb 0.8.2 → 0.8.4

package/dist/main.js

@@ -121,7 +121,8 @@ function detectRepoRoot() {
     const root = execSync("git rev-parse --show-toplevel", {
       encoding: "utf-8",
       timeout: 3e3,
-      stdio: ["pipe", "pipe", "pipe"]
+      stdio: ["pipe", "pipe", "pipe"],
+      windowsHide: true
     }).trim();
     if (!root) return null;
     return { root, name: path2.basename(root) };
@@ -984,7 +985,7 @@ function copilotUninstallMatching(settings, eventName, predicate) {
   }
   return true;
 }
-var OPENCODE_PLUGIN_VERSION = 7;
+var OPENCODE_PLUGIN_VERSION = 8;
 var OPENCODE_PLUGIN_MARKER = `// HILLCLIMB_OPENCODE_PLUGIN_VERSION=${OPENCODE_PLUGIN_VERSION}`;
 var OPENCODE_PLUGIN_CONTENT = `${OPENCODE_PLUGIN_MARKER}
 // Auto-installed by \`npx hillclimb\`. Do not edit manually \u2014 re-running
@@ -1069,6 +1070,7 @@ function spawnHillclimb(subcommand, payload) {
     const child = spawn("npx", ["hillclimb@latest"].concat(args), {
       detached: true,
       stdio: ["pipe", "ignore", "ignore"],
+      windowsHide: true,
     });
     child.on("error", () => {});
     if (child.stdin) {
@@ -1413,7 +1415,8 @@ function trackedHookFiles(repoRoot, files) {
   try {
     const output = execFileSync("git", ["ls-files", "--", ...rels], {
       cwd: repoRoot,
-      stdio: ["ignore", "pipe", "ignore"]
+      stdio: ["ignore", "pipe", "ignore"],
+      windowsHide: true
     }).toString("utf-8").trim();
     if (!output) return [];
     return output.split(/\r?\n/).map((rel) => path6.join(repoRoot, rel));
@@ -1617,7 +1620,11 @@ function sleep(ms) {
 function openBrowser(url) {
   const [cmd, args] = process.platform === "darwin" ? ["open", [url]] : process.platform === "win32" ? ["cmd", ["/c", "start", "", url]] : ["xdg-open", [url]];
   try {
-    const child = spawn(cmd, args, { detached: true, stdio: "ignore" });
+    const child = spawn(cmd, args, {
+      detached: true,
+      stdio: "ignore",
+      windowsHide: true
+    });
     child.on("error", () => {
     });
     child.unref();
@@ -2342,7 +2349,11 @@ var VOLATILE_KEYS = /* @__PURE__ */ new Set([
   "wall_time_ms"
 ]);
 function sessionsDir() {
-  return process.env.HILLCLIMB_CODEX_SESSIONS_DIR ?? path9.join(os3.homedir(), ".codex", "sessions");
+  if (process.env.HILLCLIMB_CODEX_SESSIONS_DIR) {
+    return process.env.HILLCLIMB_CODEX_SESSIONS_DIR;
+  }
+  const codexHome = process.env.CODEX_HOME || path9.join(os3.homedir(), ".codex");
+  return path9.join(codexHome, "sessions");
 }
 function asRecord(value) {
   return value !== null && typeof value === "object" && !Array.isArray(value) ? value : null;
@@ -2512,6 +2523,10 @@ async function findCodexRolloutPath(sessionId) {
   }
   return void 0;
 }
+async function isCodexInternalThread(payload, sessionId) {
+  if (!sessionId || payload.transcript_path) return false;
+  return await findCodexRolloutPath(sessionId) === void 0;
+}
 function commonPrefixLength(a, b) {
   const limit = Math.min(a.length, b.length);
   let count = 0;
@@ -11755,14 +11770,20 @@ function getSourceBaseDir(sourceName) {
   }
 }
 function archivePathFor(file) {
+  return path11.posix.join(file.sourceName, path11.basename(file.absolutePath));
+}
+function legacyArchivePathFor(file) {
   const baseDir = getSourceBaseDir(file.sourceName);
   const relativePath = file.absolutePath.startsWith(baseDir) ? path11.relative(baseDir, file.absolutePath) : path11.basename(file.absolutePath);
   return path11.join(file.sourceName, relativePath);
 }
 function addGroupToArchive(archive, group, selectedSources) {
+  const used = /* @__PURE__ */ new Set();
   for (const file of group.files) {
     if (!selectedSources.has(file.sourceName)) continue;
-    const archivePath = archivePathFor(file);
+    let archivePath = archivePathFor(file);
+    if (used.has(archivePath)) archivePath = legacyArchivePathFor(file);
+    used.add(archivePath);
     if (file.content) {
       archive.append(file.content, { name: archivePath });
     } else {
@@ -13058,7 +13079,7 @@ async function runUploadInner(payload) {
       "warn",
       `Skipping upload: missing required hook fields (session_id=${!!sessionId}, transcript_path=${!!payload.transcript_path}, cwd=${!!cwd})`
     );
-    return false;
+    return "skipped";
   }
   const match = await findProjectForCwd(cwd);
   if (!match) {
@@ -13066,13 +13087,13 @@ async function runUploadInner(payload) {
       "warn",
       `Skipping session ${sessionId}: no hillclimb config for cwd ${cwd}. Run \`npx hillclimb\` in the repo.`
     );
-    return false;
+    return "skipped";
   }
   const { repoRoot, config } = match;
   const sourceTool = await resolveSourceTool(payload, repoRoot);
   if (!sourceTool) {
     payload.tool = "unknown";
-    return false;
+    return "skipped";
   }
   payload.tool = sourceTool;
   appendLog(
@@ -13086,11 +13107,18 @@ async function runUploadInner(payload) {
     sessionId
   );
   if (!transcriptPath) {
+    if (sourceTool === "codex") {
+      appendLog(
+        "info",
+        `Skipping session ${sessionId}: Codex internal thread (no transcript_path and no rollout under ~/.codex/sessions); not a user session`
+      );
+      return "internal-thread";
+    }
     appendLog(
       "warn",
       `Skipping upload: missing required hook fields (session_id=true, transcript_path=false, cwd=true)`
     );
-    return false;
+    return "skipped";
   }
   const transcriptResolved = path15.resolve(transcriptPath);
   try {
@@ -13100,23 +13128,32 @@ async function runUploadInner(payload) {
         "warn",
         `Skipping session ${sessionId}: transcript_path is not a file: ${transcriptResolved}`
       );
-      return false;
+      return "skipped";
     }
   } catch (err) {
+    const detail = err instanceof Error ? err.message : String(err);
+    const missing = err?.code === "ENOENT";
+    if (missing && eventKind2 === "sessionEnd") {
+      appendLog(
+        "info",
+        `Skipping session ${sessionId}: session ended without any turns; transcript was never written (${transcriptResolved})`
+      );
+      return "skipped";
+    }
     appendLog(
       "warn",
-      `Skipping session ${sessionId}: transcript_path not readable (${transcriptResolved}): ${err instanceof Error ? err.message : String(err)}`
+      `Skipping session ${sessionId}: transcript_path not readable (${transcriptResolved}): ${detail}`
     );
-    return false;
+    return "skipped";
   }
   if (!await hasAssistantMessage(transcriptResolved)) {
     appendLog(
       "info",
       `Skipping session ${sessionId}: transcript contains no assistant messages (nothing to upload).`
     );
-    return false;
+    return "skipped";
   }
-  return await uploadSession({
+  const uploaded = await uploadSession({
     sessionId,
     transcriptPath: transcriptResolved,
     repoRoot,
@@ -13125,6 +13162,7 @@ async function runUploadInner(payload) {
     eventKind: eventKind2,
     recordedAt
   });
+  return uploaded ? "uploaded" : "skipped";
 }
 var AGENT_MAX_UPLOAD_BYTES = 1024 * 1024 * 1024;
 var CLI_VERSION = "0.8.0";
@@ -13344,6 +13382,10 @@ async function uploadSession(args) {
       sourceName: sourceTool,
       absolutePath: transcriptPath
     });
+    const legacyTranscriptArchivePath = legacyArchivePathFor({
+      sourceName: sourceTool,
+      absolutePath: transcriptPath
+    });
     const alreadySubmitted = prior?.submitted ?? false;
     const submitThisUpload = config.autoSubmit && !alreadySubmitted;
     if (mode === "meta-refresh" && prior?.contributionId && prior.epochMeta && cursor) {
@@ -13421,7 +13463,7 @@ async function uploadSession(args) {
           epochMeta2 = {
             transitionKind: "legacy-adopted",
             baseline: "legacy-latest-zip",
-            transcriptArchivePath,
+            transcriptArchivePath: legacyTranscriptArchivePath,
             rawByteOffset: baseCursor.rawByteOffset,
             rawPrefixSha256: baseCursor.rawPrefixSha256
           };
@@ -13437,7 +13479,7 @@ async function uploadSession(args) {
               epoch: epoch2,
               transitionKind: "legacy-adopted",
               baseline: "legacy-latest-zip",
-              transcriptArchivePath,
+              transcriptArchivePath: legacyTranscriptArchivePath,
               cursor: baseCursor,
               recordedAt,
               lineage: codexLineage
@@ -13828,6 +13870,7 @@ async function runUpload() {
     const child = spawn2(process.execPath, [entrypoint, "upload"], {
       detached: true,
       stdio: ["pipe", "ignore", "ignore"],
+      windowsHide: true,
       env: workerEnv
     });
     child.on("error", (err) => {
@@ -13877,16 +13920,16 @@ async function runUploadWorker() {
   const toolOverride = process.env[TOOL_ENV_FLAG];
   if (toolOverride) payload.tool = toolOverride;
   appendLog("info", `worker: payload summary: ${summarizePayload(payload)}`);
-  let completed = false;
+  let outcome = "skipped";
   try {
-    completed = await runUploadInner(payload);
+    outcome = await runUploadInner(payload);
   } catch (err) {
     appendLog(
       "error",
       `worker: unexpected error: ${err instanceof Error ? err.stack ?? err.message : String(err)}`
     );
   } finally {
-    if (completed) {
+    if (outcome !== "internal-thread") {
       await recordDebugLogCompletion({
         kind: "agent",
         tool: fallbackSourceTool(payload),
@@ -13920,7 +13963,10 @@ var EXEC_OPTS = {
   timeout: GIT_COMMAND_TIMEOUT_MS,
   // A rewrite diff for a text file capped at ≤100 MB must fit. This transient
   // allocation only grows as large as the actual command output.
-  maxBuffer: 256 * 1024 * 1024
+  maxBuffer: 256 * 1024 * 1024,
+  // The detached worker has no console on Windows, so without this every git
+  // child would allocate a fresh visible console window (one flash per call).
+  windowsHide: true
 };
 var MAX_ERROR_OUTPUT_CHARS = 2e3;
 var DEFAULT_SNAPSHOT_LIMITS = {
@@ -14062,7 +14108,8 @@ function readTreeBlobHead(repoRoot, sha) {
     const { stdout } = spawnSync("git", ["cat-file", "blob", sha], {
       cwd: repoRoot,
       timeout: GIT_COMMAND_TIMEOUT_MS,
-      maxBuffer: BINARY_SNIFF_BYTES
+      maxBuffer: BINARY_SNIFF_BYTES,
+      windowsHide: true
     });
     return stdout?.length ? stdout.subarray(0, BINARY_SNIFF_BYTES) : null;
   } catch {
@@ -14998,7 +15045,8 @@ function execGit(cwd, args) {
     cwd,
     stdio: ["pipe", "pipe", "pipe"],
     timeout: 3e4,
-    maxBuffer: 50 * 1024 * 1024
+    maxBuffer: 50 * 1024 * 1024,
+    windowsHide: true
   }).toString("utf-8").trim();
 }
 function canUploadFile(filename, buffer) {
@@ -16611,6 +16659,7 @@ async function runGitTraces() {
       {
         detached: true,
         stdio: ["pipe", "ignore", "ignore"],
+        windowsHide: true,
         env: workerEnv
       }
     );
@@ -16679,6 +16728,13 @@ async function runGitTracesWorker() {
     );
     return;
   }
+  if (tool === "codex" && await isCodexInternalThread(payload, resolveHookSessionId(payload))) {
+    appendLog(
+      "info",
+      `git-traces: skipping Codex internal thread ${resolveHookSessionId(payload)} on ${event} (no transcript_path and no rollout on disk; not a user session)`
+    );
+    return;
+  }
   try {
     await selfHealHook2(payload, tool);
     switch (eventKind2) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hillclimb",
-  "version": "0.8.2",
+  "version": "0.8.4",
   "description": "Extract and export AI coding tool logs grouped by repo",
   "license": "MIT",
   "type": "module",