gitdone-agent 0.5.3 → 0.5.6

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.5.3'
+const AGENT_VERSION = '0.5.6'
 
 const AGENT_DIR = join(homedir(), '.gitdone-agent')
 const CONFIG_PATH = join(AGENT_DIR, 'config.json')
@@ -111,7 +111,9 @@ function getNodePath() {
 // native installer) so spawn works WITHOUT a shell — that lets us pass a
 // multi-line, non-ASCII prompt as a single argv element with no escaping. An
 // npm shim (claude.cmd) needs shell:true, where we collapse the prompt to one
-// line to survive cmd.exe parsing.
+// line to survive cmd.exe parsing. Returns `found:false` when nothing was
+// located, so callers can show a clear message instead of spawning bare
+// `claude` and letting cmd.exe emit "'claude' is not recognized…".
 function findClaude() {
   const tryCmd = (c) => {
     try {
@@ -120,10 +122,27 @@ function findClaude() {
     } catch { return [] }
   }
   const cands = [...tryCmd('where claude'), ...tryCmd('which claude')]
+
+  // The agent is auto-started at login, so its PATH is frozen at that moment —
+  // a `claude` installed (or a PATH entry added) afterwards is invisible to
+  // `where`/`which` above. Probe the well-known install locations directly so a
+  // freshly-installed CLI works without a Windows re-login.
+  const home = homedir()
+  const appData = process.env.APPDATA || join(home, 'AppData', 'Roaming')
+  for (const p of [
+    join(home, '.local', 'bin', 'claude.exe'), // Windows native installer
+    join(appData, 'npm', 'claude.cmd'),        // Windows npm global shim
+    join(home, '.local', 'bin', 'claude'),     // macOS/Linux native installer
+    '/usr/local/bin/claude',                   // Homebrew (Intel) / manual install
+    '/opt/homebrew/bin/claude',                // Homebrew (Apple silicon)
+  ]) {
+    if (existsSync(p) && !cands.includes(p)) cands.push(p)
+  }
+
   const exe = cands.find((p) => /\.exe$/i.test(p))
-  if (exe) return { path: exe, shell: false }
-  if (cands.length) return { path: cands[0], shell: true }
-  return { path: 'claude', shell: true }
+  if (exe) return { path: exe, shell: false, found: true }
+  if (cands.length) return { path: cands[0], shell: true, found: true }
+  return { path: 'claude', shell: true, found: false }
 }
 
 // Kill any previously-running agent processes so an update applies WITHOUT a
@@ -194,6 +213,8 @@ function runDoctor() {
   console.log(`  autostart vbs : ${getVbsPath()} ${existsSync(getVbsPath()) ? '(ok)' : '(not installed)'}`)
   console.log(`  config        : ${CONFIG_PATH} ${existsSync(CONFIG_PATH) ? '(ok)' : '(MISSING)'}`)
   console.log(`  log file      : ${LOG_PATH} ${existsSync(LOG_PATH) ? '(ok)' : '(none yet)'}`)
+  const claude = findClaude()
+  console.log(`  claude cli    : ${claude.found ? claude.path : 'NOT FOUND — инсталирай от claude.ai/code и рестартирай агента'}`)
   if (cfg) {
     console.log(`  machineId     : ${cfg.machineId}`)
     console.log(`  server        : ${cfg.url}`)
@@ -545,7 +566,14 @@ function runAiCommand(cfg, cmd, repoPath) {
     return
   }
 
-  const { path: claudePath, shell } = findClaude()
+  const { path: claudePath, shell, found } = findClaude()
+  if (!found) {
+    const msg = `Claude Code CLI не е намерен на този компютър (${cfg.hostname}). Инсталирай го от claude.ai/code и се увери, че „claude" е в PATH, после рестартирай агента.`
+    log(`✗ ai_run ${runId}: claude not found`)
+    postRunEvents(cfg, runId, [{ kind: 'SYSTEM', text: `✗ ${msg}` }], 'error', 'claude not found')
+    reportCommandResult(cfg, cmd.id, 'error', 'claude not found')
+    return
+  }
 
   let settingsPath
   try { settingsPath = ensureAiHookFiles() } catch (e) { log(`✗ ai hook setup failed: ${e.message}`) }
@@ -645,7 +673,14 @@ function runAiChat(cfg, cmd, repoPath) {
     return
   }
 
-  const { path: claudePath, shell } = findClaude()
+  const { path: claudePath, shell, found } = findClaude()
+  if (!found) {
+    const msg = `Claude Code CLI не е намерен на този компютър (${cfg.hostname}). Инсталирай го от claude.ai/code и се увери, че „claude" е в PATH, после рестартирай агента.`
+    log(`✗ ai_chat ${sessionId}: claude not found`)
+    postSessionEvents(cfg, sessionId, [{ role: 'SYSTEM', text: `✗ ${msg}` }], 'error')
+    reportCommandResult(cfg, cmd.id, 'error', 'claude not found')
+    return
+  }
 
   let settingsPath
   try { settingsPath = ensureAiSessionHookFiles() } catch (e) { log(`✗ ai session hook setup failed: ${e.message}`) }
@@ -726,6 +761,38 @@ function runAiChat(cfg, cmd, repoPath) {
   })
 }
 
+// deploy — run the project's deploy script (scripts/deploy.sh) in the repo.
+// Long-running (git pull + npm build + service restart), so we spawn it detached
+// from the poll loop and report the result on exit. The agent is its OWN process
+// (not part of any gitdone systemd service), so a `systemctl restart` inside the
+// script doesn't kill the deploy mid-flight — which a server-side spawn would.
+function runDeployCommand(cfg, cmd, repoPath) {
+  const script = 'scripts/deploy.sh'
+  if (!existsSync(join(repoPath, script))) {
+    reportCommandResult(cfg, cmd.id, 'error', `няма ${script} в repo-то — няма какво да деплойна`)
+    log(`✗ deploy @ ${repoPath}: липсва ${script}`)
+    return
+  }
+  log(`▸ deploy стартиран @ ${repoPath}`)
+  const child = spawn('bash', [script], { cwd: repoPath, stdio: ['ignore', 'pipe', 'pipe'] })
+  // Keep only the tail of the output — enough to explain a failure without
+  // shipping a whole build log back.
+  let tail = ''
+  const capture = (d) => { tail = (tail + d.toString()).slice(-2000) }
+  child.stdout.on('data', capture)
+  child.stderr.on('data', capture)
+  child.on('error', (err) => {
+    reportCommandResult(cfg, cmd.id, 'error', `процесна грешка: ${err.message}`)
+    log(`✗ deploy failed @ ${repoPath}: ${err.message}`)
+  })
+  child.on('close', (code) => {
+    const ok = code === 0
+    const detail = tail.trim() ? `\n${tail.trim()}` : ''
+    reportCommandResult(cfg, cmd.id, ok ? 'done' : 'error', `exit ${code}${detail}`.slice(0, 4000))
+    log(`${ok ? '✓' : '✗'} deploy @ ${repoPath} приключи (code ${code})`)
+  })
+}
+
 async function executeCommand(cfg, cmd, repoPath) {
   // ai_run is long-running + streaming — launch it in the background and return
   // so the snapshot loop keeps going. It reports its own result on exit.
@@ -738,6 +805,13 @@ async function executeCommand(cfg, cmd, repoPath) {
     runAiChat(cfg, cmd, repoPath)
     return
   }
+  // deploy — run the repo's deploy script. Long-running (pull + build + service
+  // restart), so it's launched in the background and reports its own result on
+  // exit, same as ai_run.
+  if (cmd.type === 'deploy') {
+    runDeployCommand(cfg, cmd, repoPath)
+    return
+  }
 
   let result = 'ok'
   let status = 'done'
@@ -762,6 +836,14 @@ async function executeCommand(cfg, cmd, repoPath) {
       result = pushOrPull(cmd.type, repoPath, auth)
     } else if (cmd.type === 'discard') {
       result = discardFile(repoPath, cmd.payload?.file)
+    } else {
+      // Unknown command type — almost always a newer server feature (e.g. the
+      // gd-255 `deploy` command) reaching an agent too old to handle it. Do NOT
+      // silently mark it done: an older agent used to fall through here and
+      // report status=done/result=ok, so a queued deploy looked "completed" while
+      // nothing ran. Surface it as an error so the failure is visible and the
+      // user knows to update the agent.
+      throw new Error(`непозната команда „${cmd.type}" — обнови gitdone-agent (npm i -g gitdone-agent@latest)`)
     }
     log(`✓ ${cmd.type} @ ${repoPath}: ${redact(result, auth?.token)}`)
   } catch (err) {

package/package.json

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