gitdone-agent 0.6.12 → 0.6.13

package/index.js

@@ -27,7 +27,7 @@ import { randomUUID } from 'node:crypto'
 // Reported to the server on every sync so the web UI can flag outdated agents.
 // Keep in lockstep with packages/agent/package.json "version" AND
 // src/lib/agentVersion.ts LATEST_AGENT_VERSION.
-const AGENT_VERSION = '0.6.12'
+const AGENT_VERSION = '0.6.13'
 
 const AGENT_DIR = join(homedir(), '.gitdone-agent')
 const CONFIG_PATH = join(AGENT_DIR, 'config.json')
@@ -242,6 +242,17 @@ const GIT_MAX_BUFFER = 64 * 1024 * 1024
 // bloat the snapshot payload.
 const UNTRACKED_DIFF_MAX_BYTES = 1024 * 1024
 
+// Synthesising a per-file diff spawns a synchronous `git diff --no-index` per
+// untracked file. Individually cheap, but a repo with a huge untracked/scratch
+// tree (e.g. extracted game assets — thousands of files) would spawn thousands
+// of blocking git processes and freeze the single-threaded agent for MINUTES on
+// every snapshot, starving the 30s sync heartbeat so the machine flips offline.
+// Bound the synthesis two ways — a file-count cap and a wall-clock budget — so
+// no single repo can ever monopolise the event loop. Anything past either keeps
+// the "no diff" placeholder, identical to the size-cap path above.
+const UNTRACKED_DIFF_MAX_FILES = 200
+const UNTRACKED_DIFF_TIME_BUDGET_MS = 3000
+
 function git(cmd, cwd) {
   try {
     return execSync(cmd, { cwd, encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'], maxBuffer: GIT_MAX_BUFFER }).trim()
@@ -520,12 +531,20 @@ function getSnapshot(repoPath) {
   // each from /dev/null so the console shows the whole new file — like GitHub
   // Desktop (gd-370). Skip anything over the size cap so a stray big/binary blob
   // can't bloat the snapshot; those keep the "no diff" placeholder.
-  for (const [file, xy] of Object.entries(statuses)) {
-    if (xy !== '??' || diffs[file]) continue
-    try {
-      const st = statSync(join(repoPath, file))
-      if (!st.isFile() || st.size > UNTRACKED_DIFF_MAX_BYTES) continue
-    } catch { continue }
+  const untracked = Object.keys(statuses).filter((f) => statuses[f] === '??' && !diffs[f])
+  const diffDeadline = Date.now() + UNTRACKED_DIFF_TIME_BUDGET_MS
+  let synthesised = 0
+  for (const file of untracked) {
+    // Stop before either bound so a scratch tree can't starve the heartbeat; the
+    // rest keep the "no diff" placeholder. Log once so it's visible why.
+    if (synthesised >= UNTRACKED_DIFF_MAX_FILES || Date.now() > diffDeadline) {
+      log(`⚠ untracked diff cap reached @ ${repoPath} (${synthesised}/${untracked.length} synthesised) — rest shown without preview`)
+      break
+    }
+    let st
+    try { st = statSync(join(repoPath, file)) } catch { continue }
+    if (!st.isFile() || st.size > UNTRACKED_DIFF_MAX_BYTES) continue
+    synthesised++
     const parsed = parseDiffByFile(gitDiffUntracked(file, repoPath))
     const val = parsed[file] ?? Object.values(parsed)[0]
     if (val) diffs[file] = val

package/package.json

@@ -1,6 +1,6 @@
 {
     "name":  "gitdone-agent",
-    "version":  "0.6.12",
+    "version":  "0.6.13",
     "description":  "Local git agent for gitdone — watches a local repo and sends snapshots to gitdone.eu",
     "type":  "module",
     "bin":  {