@tablation/crew 0.36.0 → 0.37.0

package/dist/cli.js

@@ -10616,7 +10616,7 @@ function planEventsSweep(stateDir, retentionDays, now = Date.now()) {
 }
 
 // src/cli.ts
-import { readFileSync as readFileSync15, existsSync as existsSync22, mkdirSync as mkdirSync12, writeFileSync as writeFileSync12, unlinkSync as unlinkSync5, copyFileSync as copyFileSync2, renameSync as renameSync2, statSync as statSync9 } from "fs";
+import { readFileSync as readFileSync16, existsSync as existsSync23, mkdirSync as mkdirSync13, writeFileSync as writeFileSync13, unlinkSync as unlinkSync5, copyFileSync as copyFileSync2, renameSync as renameSync2, statSync as statSync9 } from "fs";
 import { createInterface } from "readline/promises";
 import { dirname as dirOf, resolve as resolvePath } from "path";
 
@@ -11354,7 +11354,7 @@ function syncPassengerSyncDaemons(plans, cfg, stateDir, deps = {}) {
 }
 
 // src/daemon.ts
-import { readdirSync as readdirSync6, statSync as statSync8 } from "fs";
+import { existsSync as existsSync22, mkdirSync as mkdirSync12, readFileSync as readFileSync15, readdirSync as readdirSync6, rmSync as rmSync8, statSync as statSync8, writeFileSync as writeFileSync12 } from "fs";
 import { join as join21 } from "path";
 var DEFAULT_FLOOR_MS = 5e3;
 var DEFAULT_CEILING_MS = INTERVAL_SECONDS * 1e3;
@@ -11401,9 +11401,38 @@ function latestMtime(path) {
 function computeTreeSignature(paths) {
   return paths.map((p) => String(latestMtime(p))).join(":");
 }
-function makeStaleChecker(paths) {
+function makeStaleChecker(paths, opts = {}) {
   const initial = computeTreeSignature(paths);
-  return () => computeTreeSignature(paths) !== initial;
+  return () => computeTreeSignature(paths) !== initial || opts.reloadSentinel !== void 0 && existsSync22(opts.reloadSentinel);
+}
+function reloadSentinelFor(stateDir) {
+  return join21(stateDir, "reload-requested");
+}
+function daemonStampFor(stateDir) {
+  return join21(stateDir, "daemon.started");
+}
+function noteDaemonStarted(stateDir, now = Date.now()) {
+  mkdirSync12(stateDir, { recursive: true });
+  rmSync8(reloadSentinelFor(stateDir), { force: true });
+  writeFileSync12(daemonStampFor(stateDir), `${JSON.stringify({ pid: process.pid, startedAt: now })}
+`);
+}
+function readDaemonStarted(stateDir) {
+  try {
+    const v = JSON.parse(readFileSync15(daemonStampFor(stateDir), "utf8")).startedAt;
+    return typeof v === "number" ? v : null;
+  } catch {
+    return null;
+  }
+}
+function configChangedAfter(files, startedAt) {
+  for (const f of files) {
+    try {
+      if (statSync8(f).mtimeMs > startedAt) return f;
+    } catch {
+    }
+  }
+  return null;
 }
 var SPAWN_PARK_AFTER = 3;
 var SpawnBreaker = class {
@@ -11804,6 +11833,7 @@ function usage() {
   crew watch [route]             live view of what the crew is doing
   crew status [route] [--json]   paused/running state; --json for a machine reader
   crew projects [--route R] [--json]   the Projects rows each route can read (id, name, prefix, work mode)
+  crew reload                    make the running daemon re-read crew.yaml (--now: restart it immediately)
   crew doctor [route] [--fix]    preflight; --fix enables clean routes and offers crew install
   crew ports [route]             which checkout owns which ports, and what is up
   crew reap [route]              kill orphaned servers, drop worktrees for closed tickets
@@ -11858,7 +11888,12 @@ var value = (name) => {
 var positional = argv.filter((a, i) => !a.startsWith("--") && !argv[i - 1]?.startsWith("--"));
 var command = positional[0];
 if (!command) usage();
-var STATE_WRITING_COMMANDS = /* @__PURE__ */ new Set(["connect", "disconnect", "install", "uninstall", "release", "merge", "deploy"]);
+function daemonConfigPaths(c) {
+  const files = [c.configFile];
+  for (const r of c.routes) if (r.apiKeyFile && !files.includes(r.apiKeyFile)) files.push(r.apiKeyFile);
+  return files;
+}
+var STATE_WRITING_COMMANDS = /* @__PURE__ */ new Set(["connect", "disconnect", "install", "uninstall", "reload", "release", "merge", "deploy"]);
 var STATE_WRITING_SUBCOMMANDS = {
   agents: /* @__PURE__ */ new Set(["sync"]),
   skills: /* @__PURE__ */ new Set(["sync"])
@@ -11968,7 +12003,7 @@ if (command === "passenger-sync-daemon") {
   const releaseFleetWide = ["merge", "deploy", "release"].includes(command) && flag("fleet");
   const DAEMON_SUBCOMMANDS = /* @__PURE__ */ new Set(["start", "stop", "status", "restart"]);
   const named = command === "daemon" && DAEMON_SUBCOMMANDS.has(positional[1] ?? "") ? positional[2] : positional[1];
-  const fleetWide = command === "inbox" || command === "connect" || command === "disconnect" || command === "agents" || command === "skills" || command === "logbook" || command === "repos" || command === "projects" || releaseFleetWide || command === "pause" || command === "resume" || FLEET_CAPABLE.has(command) && !named && cfg.routes.length > 1;
+  const fleetWide = command === "inbox" || command === "connect" || command === "disconnect" || command === "agents" || command === "skills" || command === "logbook" || command === "repos" || command === "projects" || releaseFleetWide || command === "pause" || command === "resume" || command === "reload" || FLEET_CAPABLE.has(command) && !named && cfg.routes.length > 1;
   let route;
   try {
     route = fleetWide ? cfg.routes[0] : findRoute(cfg, named);
@@ -12175,7 +12210,7 @@ if (command === "passenger-sync-daemon") {
     if (host === "linux" && hasSystemd) {
       const label = labelFor(CREW_HOME, "run");
       const staleTimer = join22(homedir7(), ".config", "systemd", "user", `${label}.timer`);
-      if (existsSync22(staleTimer)) {
+      if (existsSync23(staleTimer)) {
         process.stdout.write(
           `removing stale run timer from before the persistent-service cutover: ${staleTimer}${dryRun ? " (dry run)" : ""}
 `
@@ -12201,7 +12236,7 @@ if (command === "passenger-sync-daemon") {
     const tracker = new Tracker(c, cfg.ship);
     problems.push(...validateContract(tracker.contract).map((p) => `contract: ${p}`));
     for (const r of await resolvedRepos(c)) {
-      if (!existsSync22(r.dir)) problems.push(`repo ${r.name}: ${r.dir} is missing on disk`);
+      if (!existsSync23(r.dir)) problems.push(`repo ${r.name}: ${r.dir} is missing on disk`);
       if (!satisfies(host, r.config.platform)) problems.push(`repo ${r.name}: requires ${r.config.platform}, this host is ${host}`);
       problems.push(...validateEffective(r.config).map((p) => `repo ${r.name}: ${p}`));
     }
@@ -12219,7 +12254,7 @@ if (command === "passenger-sync-daemon") {
   }
   async function doctorFix(o) {
     const routes = await Promise.all(routesInScope(cfg.routes, named).map((c) => gatherHealth({ route: c.route, enabled: c.enabled, hasRepos: reposOf(c).length > 0 }, () => routeProblems(c))));
-    const unitExists = (p) => existsSync22(p);
+    const unitExists = (p) => existsSync23(p);
     const crontab = o.host === "linux" ? spawnSync5("crontab", ["-l"], { encoding: "utf8" }).stdout ?? "" : "";
     const schedulerAbsent = o.host !== "windows" ? !schedulerInstalled(planInstall(cfg.ship, CREW_HOME, o.host, detectSystemd(), "run"), unitExists, crontab) : true;
     const plan = planFix({ routes, schedulerAbsent, host: o.host });
@@ -12236,7 +12271,7 @@ if (command === "passenger-sync-daemon") {
       for (const a of plan.actions) {
         if (a.kind === "enable") {
           if (prompter && !await prompter.confirm(`Route ${a.route} passes every check \u2014 set enabled: true in ${cfg.configFile}?`, false)) continue;
-          writeFileSync12(cfg.configFile, planEnable(readFileSync15(cfg.configFile, "utf8"), a.route));
+          writeFileSync13(cfg.configFile, planEnable(readFileSync16(cfg.configFile, "utf8"), a.route));
           enabledSound.add(a.route);
           changed2.push(`enabled route ${a.route} in ${cfg.configFile}`);
         } else {
@@ -13141,6 +13176,9 @@ ${changed2.map((c) => `  - ${c}
       }
       requireArmed("run the daemon loop");
       const watchPaths = isCompiledBinary(import.meta.url) ? [process.execPath] : [join22(CREW_HOME, "src"), join22(CREW_HOME, "dist"), join22(CREW_HOME, "bin")];
+      const configPaths = daemonConfigPaths(cfg);
+      watchPaths.push(...configPaths);
+      noteDaemonStarted(cfg.ship.stateDir);
       const spawnBreaker = new SpawnBreaker();
       const stopReason = await runDaemonLoop({
         runPass: () => runOnePass({
@@ -13158,14 +13196,16 @@ ${changed2.map((c) => `  - ${c}
 `)
           })
         }),
-        checkStale: makeStaleChecker(watchPaths),
+        checkStale: makeStaleChecker(watchPaths, { reloadSentinel: reloadSentinelFor(cfg.ship.stateDir) }),
         onPass: (result) => {
           if (result.started.length) emit.emit(`daemon pass started: ${result.started.join(", ")}`);
           for (const s of result.skipped) emit.emit(`daemon pass skipped ${s.role} \u2014 ${s.reason}`);
         }
       });
       if (stopReason === "stale") {
-        emit.emit("crew daemon: installed tree changed on disk \u2014 exiting cleanly so the OS-level restart relaunches with the update");
+        emit.emit(
+          "crew daemon: installed tree, crew.yaml or a reload request changed on disk \u2014 exiting cleanly so the OS-level restart relaunches with the update"
+        );
         process.exit(0);
       }
       break;
@@ -13289,11 +13329,11 @@ ${changed2.map((c) => `  - ${c}
 `);
       if (!dryRun) {
         const resolvedPath = resolvedPathFor(cfg.ship.stateDir, target.route);
-        const raw = existsSync22(resolvedPath) ? JSON.parse(readFileSync15(resolvedPath, "utf8")) : {};
+        const raw = existsSync23(resolvedPath) ? JSON.parse(readFileSync16(resolvedPath, "utf8")) : {};
         raw.agentPersonas = personaResult.agentPersonas;
         raw.agentSkills = skillResult.agentSkills;
-        mkdirSync12(dirOf(resolvedPath), { recursive: true });
-        writeFileSync12(resolvedPath, `${JSON.stringify(raw, null, 2)}
+        mkdirSync13(dirOf(resolvedPath), { recursive: true });
+        writeFileSync13(resolvedPath, `${JSON.stringify(raw, null, 2)}
 `);
       }
       if (personaResult.outcomes.some((o) => o.action === "diverged") || skillResult.outcomes.some((o) => o.action === "diverged")) {
@@ -13338,10 +13378,10 @@ ${changed2.map((c) => `  - ${c}
 `);
       if (!dryRun) {
         const resolvedPath = resolvedPathFor(cfg.ship.stateDir, target.route);
-        const raw = existsSync22(resolvedPath) ? JSON.parse(readFileSync15(resolvedPath, "utf8")) : {};
+        const raw = existsSync23(resolvedPath) ? JSON.parse(readFileSync16(resolvedPath, "utf8")) : {};
         raw.agentSkills = result.agentSkills;
-        mkdirSync12(dirOf(resolvedPath), { recursive: true });
-        writeFileSync12(resolvedPath, `${JSON.stringify(raw, null, 2)}
+        mkdirSync13(dirOf(resolvedPath), { recursive: true });
+        writeFileSync13(resolvedPath, `${JSON.stringify(raw, null, 2)}
 `);
       }
       if (result.outcomes.some((o) => o.action === "diverged")) process.exitCode = 1;
@@ -13354,14 +13394,14 @@ ${changed2.map((c) => `  - ${c}
         process.exit(2);
       };
       if (sub !== "add" && sub !== "list") usage2();
-      const configText = readFileSync15(cfg.configFile, "utf8");
+      const configText = readFileSync16(cfg.configFile, "utf8");
       let routeName = positional[2];
       let addPath = ".";
       let addName;
       if (sub === "list" && !routeName) usage2();
       if (sub === "add") {
         try {
-          const a = parseAddArgs(positional.slice(2), routeNamesOf(configText), (p) => existsSync22(resolvePath(p.replace(/^~(?=\/|$)/, homedir7()))));
+          const a = parseAddArgs(positional.slice(2), routeNamesOf(configText), (p) => existsSync23(resolvePath(p.replace(/^~(?=\/|$)/, homedir7()))));
           routeName = a.route;
           addPath = a.path;
           addName = a.name;
@@ -13376,9 +13416,9 @@ ${changed2.map((c) => `  - ${c}
       }
       const resolvedFile = resolvedPathFor(cfg.ship.stateDir, routeName);
       let resolvedRemotes;
-      if (existsSync22(resolvedFile)) {
+      if (existsSync23(resolvedFile)) {
         try {
-          resolvedRemotes = JSON.parse(readFileSync15(resolvedFile, "utf8"));
+          resolvedRemotes = JSON.parse(readFileSync16(resolvedFile, "utf8"));
         } catch {
         }
       }
@@ -13393,14 +13433,14 @@ ${changed2.map((c) => `  - ${c}
           for (const r of repos) {
             const origin2 = originOf(r.dir);
             const row = matchRepoName(origin2, resolvedRemotes);
-            const note = !existsSync22(r.dir) ? "missing on disk" : row ? `Repos row: ${row}` : `no Repos row matches ${normalizeRemote(origin2) ?? "this checkout (no origin remote)"}`;
+            const note = !existsSync23(r.dir) ? "missing on disk" : row ? `Repos row: ${row}` : `no Repos row matches ${normalizeRemote(origin2) ?? "this checkout (no origin remote)"}`;
             process.stdout.write(`${r.name}	${r.dir}	${note}
 `);
           }
           break;
         }
         const target = resolvePath(addPath.replace(/^~(?=\/|$)/, homedir7()));
-        const top = existsSync22(target) ? gitOk(target, ["rev-parse", "--show-toplevel"]) : null;
+        const top = existsSync23(target) ? gitOk(target, ["rev-parse", "--show-toplevel"]) : null;
         if (!top) throw new ReposError(`${target} is not a git checkout`);
         const origin = originOf(top);
         const plan = planRepoAdd({
@@ -13449,15 +13489,15 @@ ${changed2.map((c) => `  - ${c}
           const created = await rowTracker.createRepoRow(rowPayload);
           process.stdout.write(`created Repos row ${String(rowPayload.name)} (${String(rowPayload.remote)}) ${created.id}
 `);
-          if (existsSync22(resolvedFile)) {
-            const raw = JSON.parse(readFileSync15(resolvedFile, "utf8"));
+          if (existsSync23(resolvedFile)) {
+            const raw = JSON.parse(readFileSync16(resolvedFile, "utf8"));
             raw.repoNames = { ...raw.repoNames, [created.id]: String(rowPayload.name) };
             raw.repoRemotes = { ...raw.repoRemotes, [created.id]: String(rowPayload.remote) };
-            writeFileSync12(resolvedFile, `${JSON.stringify(raw, null, 2)}
+            writeFileSync13(resolvedFile, `${JSON.stringify(raw, null, 2)}
 `);
           }
         }
-        writeFileSync12(cfg.configFile, plan.text);
+        writeFileSync13(cfg.configFile, plan.text);
         process.stdout.write(`added ${top} to ${routeName} as ${plan.mode === "dir" ? "dir" : `repos.${plan.name}`} in ${cfg.configFile}
 `);
       } catch (e) {
@@ -13696,10 +13736,10 @@ Signed in as ${result.identity.email} \u2014 minted an API key in workspace "${r
           operator = await pickOperator(found.holds);
         }
         let previous;
-        const hadPreviousFile = existsSync22(resolvedPath);
+        const hadPreviousFile = existsSync23(resolvedPath);
         if (hadPreviousFile) {
           try {
-            previous = JSON.parse(readFileSync15(resolvedPath, "utf8"));
+            previous = JSON.parse(readFileSync16(resolvedPath, "utf8"));
           } catch {
           }
         }
@@ -13760,10 +13800,10 @@ Signed in as ${result.identity.email} \u2014 minted an API key in workspace "${r
           process.stderr.write(`(dry run) would write resolved ids to ${resolvedPath}
 `);
         } else {
-          mkdirSync12(dirname12(resolvedPath), { recursive: true });
+          mkdirSync13(dirname12(resolvedPath), { recursive: true });
           if (hadPreviousFile) copyFileSync2(resolvedPath, `${resolvedPath}.bak`);
           const tmpPath = `${resolvedPath}.tmp${process.pid}`;
-          writeFileSync12(tmpPath, `${JSON.stringify(resolved, null, 2)}
+          writeFileSync13(tmpPath, `${JSON.stringify(resolved, null, 2)}
 `);
           renameSync2(tmpPath, resolvedPath);
           process.stderr.write(`Resolved ids written to ${resolvedPath}
@@ -13777,8 +13817,8 @@ Signed in as ${result.identity.email} \u2014 minted an API key in workspace "${r
             process.stderr.write(`(dry run) would write the minted API key to ${apiKeyFile}
 `);
           } else {
-            mkdirSync12(dirname12(apiKeyFile), { recursive: true });
-            writeFileSync12(apiKeyFile, `${apiKeyVar}=${apiKey}
+            mkdirSync13(dirname12(apiKeyFile), { recursive: true });
+            writeFileSync13(apiKeyFile, `${apiKeyVar}=${apiKey}
 `, { mode: 384 });
             process.stderr.write(`API key written to ${apiKeyFile}
 `);
@@ -14285,6 +14325,38 @@ Signed in as ${result.identity.email} \u2014 minted an API key in workspace "${r
       if (!await installScheduler({ replace: flag("replace") })) process.exit(2);
       break;
     }
+    case "reload": {
+      const sentinel = reloadSentinelFor(cfg.ship.stateDir);
+      if (dryRun) {
+        process.stdout.write(`would write ${sentinel}
+`);
+        break;
+      }
+      const host = hostPlatform();
+      const control = host === "windows" ? null : planDaemonControl(CREW_HOME, host, detectSystemd());
+      const unitStatus = control?.statusCommand ? spawnSync5(control.statusCommand[0], control.statusCommand.slice(1), { cwd: CREW_HOME, encoding: "utf8" }) : null;
+      const unitRunning = unitStatus !== null && !unitStatus.error && unitStatus.status === 0;
+      if (!unitRunning) {
+        process.stdout.write("crew reload: no daemon unit is running \u2014 nothing to reload (the next run reads crew.yaml fresh)\n");
+        break;
+      }
+      mkdirSync13(cfg.ship.stateDir, { recursive: true });
+      writeFileSync13(sentinel, `${(/* @__PURE__ */ new Date()).toISOString()}
+`);
+      process.stdout.write("crew reload: requested \u2014 the daemon finishes its in-flight pass, then relaunches with the new config\n");
+      if (flag("now") && control?.stopCommand && control.startCommand) {
+        for (const cmd of [control.stopCommand, control.startCommand]) {
+          const res = spawnSync5(cmd[0], cmd.slice(1), { cwd: CREW_HOME, encoding: "utf8" });
+          if (cmd === control.startCommand && (res.error || res.status !== 0)) {
+            process.stderr.write(`crew reload: ${cmd.join(" ")} exited ${res.status ?? "error"}
+`);
+            process.exitCode = 1;
+          }
+        }
+        process.stdout.write("crew reload: restarted the daemon unit now\n");
+      }
+      break;
+    }
     case "uninstall": {
       const host = hostPlatform();
       if (host === "windows") {
@@ -14320,7 +14392,7 @@ Signed in as ${result.identity.email} \u2014 minted an API key in workspace "${r
     }
     case "log": {
       try {
-        const text = readFileSync15(cfg.ship.logFile, "utf8").split("\n");
+        const text = readFileSync16(cfg.ship.logFile, "utf8").split("\n");
         process.stdout.write(`${text.slice(-(Number(value("tail")) || 40)).join("\n")}
 `);
       } catch {
@@ -14530,13 +14602,23 @@ routes:            ${cfg.routes.map((c) => c.route).join(", ")}
       );
       try {
         const evFile = eventFileFor(cfg.ship.stateDir);
-        const evSize = existsSync22(evFile) ? statSync9(evFile).size : 0;
+        const evSize = existsSync23(evFile) ? statSync9(evFile).size : 0;
         process.stdout.write(
           evSize > cfg.ship.eventsRotateBytes * 4 ? `events log:        WARN ${evFile} is ${Math.round(evSize / 1048576)} MB (> 4x eventsRotateBytes); run \`crew reap\` to rotate it
 ` : "events log:        ok\n"
         );
       } catch {
       }
+      const daemonStartedAt = readDaemonStarted(cfg.ship.stateDir);
+      if (daemonStartedAt !== null) {
+        const changed2 = configChangedAfter(daemonConfigPaths(cfg), daemonStartedAt);
+        if (changed2) {
+          process.stdout.write(
+            `daemon config:     WARN ${changed2} changed at ${new Date(statSync9(changed2).mtimeMs).toISOString()} but the daemon started at ${new Date(daemonStartedAt).toISOString()} \u2014 run \`crew reload\`
+`
+          );
+        }
+      }
       const tracker = new Tracker(route, cfg.ship);
       const contractProblems = validateContract(tracker.contract);
       process.stdout.write(

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tablation/crew",
-  "version": "0.36.0",
+  "version": "0.37.0",
   "description": "A standing team of headless agents that picks work off a Tablation board, does it on a machine you control, and reports back.",
   "type": "module",
   "license": "MIT",