gitdone-agent 0.6.10 → 0.6.12

package/index.js

@@ -9,10 +9,10 @@
 // roots anytime with --root (repeatable). The running agent reads its config
 // from ~/.gitdone-agent/config.json, so the autostart entry needs no args.
 
-import { execSync, spawn } from 'node:child_process'
+import { execSync, execFileSync, spawn } from 'node:child_process'
 import {
   existsSync, writeFileSync, readFileSync, unlinkSync,
-  mkdirSync, copyFileSync, appendFileSync, readdirSync, rmSync,
+  mkdirSync, copyFileSync, appendFileSync, readdirSync, rmSync, statSync,
 } from 'node:fs'
 import { resolve, join } from 'node:path'
 import { homedir, hostname, tmpdir } from 'node:os'
@@ -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.10'
+const AGENT_VERSION = '0.6.12'
 
 const AGENT_DIR = join(homedir(), '.gitdone-agent')
 const CONFIG_PATH = join(AGENT_DIR, 'config.json')
@@ -231,9 +231,20 @@ function runDoctor() {
 
 // ─── Git helpers ─────────────────────────────────────────────────────────────
 
+// Room for large command output. `git status --porcelain -uall` (gd-369) lists
+// every untracked file individually, so a repo with a big new/untracked tree can
+// blow past execSync's default 1 MB buffer — which would throw and silently
+// return '' (an EMPTY snapshot, worse than the collapsed view). 64 MB is plenty.
+const GIT_MAX_BUFFER = 64 * 1024 * 1024
+
+// Cap for synthesising an untracked file's full-content diff (gd-370) — above
+// this the file keeps the "no diff" placeholder so a huge/binary blob can't
+// bloat the snapshot payload.
+const UNTRACKED_DIFF_MAX_BYTES = 1024 * 1024
+
 function git(cmd, cwd) {
   try {
-    return execSync(cmd, { cwd, encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'] }).trim()
+    return execSync(cmd, { cwd, encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'], maxBuffer: GIT_MAX_BUFFER }).trim()
   } catch {
     return ''
   }
@@ -245,12 +256,28 @@ function git(cmd, cwd) {
 // (gd-276). Callers decode per-file via decodeDiffText().
 function gitRaw(cmd, cwd) {
   try {
-    return execSync(cmd, { cwd, encoding: 'buffer', stdio: ['pipe', 'pipe', 'pipe'] })
+    return execSync(cmd, { cwd, encoding: 'buffer', stdio: ['pipe', 'pipe', 'pipe'], maxBuffer: GIT_MAX_BUFFER })
   } catch {
     return Buffer.alloc(0)
   }
 }
 
+// Full-content diff of a single UNTRACKED file (against /dev/null), so the
+// console can show a new file's whole content as an all-added diff — like
+// GitHub Desktop (gd-370). Uses execFileSync (argv, no shell) so paths with
+// spaces / Cyrillic reach git intact, unlike a shell command string. `--no-index`
+// exits 1 when the file differs from empty (i.e. always) — that's expected, and
+// its stdout still holds the diff. Returns raw bytes for parseDiffByFile.
+function gitDiffUntracked(file, cwd) {
+  try {
+    return execFileSync('git', ['diff', '--no-index', '--', '/dev/null', file], {
+      cwd, encoding: 'buffer', stdio: ['pipe', 'pipe', 'pipe'], maxBuffer: GIT_MAX_BUFFER,
+    })
+  } catch (err) {
+    return err && err.stdout && err.stdout.length ? err.stdout : Buffer.alloc(0)
+  }
+}
+
 // Like git(), but surfaces failures (with stderr) instead of swallowing them —
 // used for push/pull where we need to detect auth rejection.
 function gitTry(cmd, cwd) {
@@ -461,7 +488,10 @@ function parseDiffByFile(diffBuf) {
 function getSnapshot(repoPath) {
   const branch = git('git branch --show-current', repoPath) || 'HEAD'
 
-  const statusLines = git('git status --porcelain', repoPath).split('\n').filter(Boolean)
+  // -uall lists every untracked file individually instead of collapsing a new
+  // directory into a single `dir/` entry — so the change list matches what
+  // GitHub Desktop shows, file-for-file (gd-369).
+  const statusLines = git('git status --porcelain -uall', repoPath).split('\n').filter(Boolean)
   const modified = []
   const staged = []
   const statuses = {}
@@ -485,6 +515,22 @@ function getSnapshot(repoPath) {
 
   const diffs = parseDiffByFile(gitRaw('git diff HEAD', repoPath))
 
+  // Untracked files (`??`) aren't in `git diff HEAD`, so they had no content to
+  // show ("Нов или untracked файл — няма diff"). Synthesise an all-added diff for
+  // 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 parsed = parseDiffByFile(gitDiffUntracked(file, repoPath))
+    const val = parsed[file] ?? Object.values(parsed)[0]
+    if (val) diffs[file] = val
+  }
+
   // Origin remote → lets the server auto-detect the GitHub repo for the History tab.
   const remoteUrl = git('git config --get remote.origin.url', repoPath) || null
 

package/package.json

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