@lumi.ai/runner 0.6.2 → 0.6.4

package/README.md

@@ -84,7 +84,9 @@ Run this first when something isn't working. Everything it checks used to be dis
 running job*, surfacing as a failed task on your board minutes later: a missing `claude` binary,
 unsaved Ship credentials, a machine no captain approved. It checks Node ≥ 20, your stored session,
 per-Ship approval and credentials, the engine binary, `git`/`gh` when an agent uses GitHub, server
-reachability, the background service, and whether that service still points at a CLI that exists.
+reachability, the background service, whether that service still points at a CLI that exists, and
+whether the **service's own `PATH`** can still reach the binaries your jobs spawn — which is a
+different question from whether your shell can (see below).
 
 ### Background service
 
@@ -99,6 +101,21 @@ reachability, the background service, and whether that service still points at a
 Your `PATH` is captured into the unit at install time. launchd and systemd start processes with a
 minimal environment, so without that `claude`, `git` and `gh` would not be found.
 
+**It is captured once, and a restart does not refresh it.** `service restart` re-runs the unit
+exactly as written, so a daemon keeps the `PATH` of whatever shell first installed it. Install a
+tool somewhere new afterwards — `~/.local/bin`, a Homebrew prefix, an nvm switch — and the daemon
+cannot see it, while every check you can run by hand (`which claude`, `lumi-runner doctor`) is
+answered by your *current* shell and looks fine. The symptom is a job that fails minutes later
+with `spawn claude ENOENT`.
+
+Two things close that gap: `doctor` reports it as **Service PATH**, and `lumi-runner setup` now
+reinstalls the service (rather than merely restarting it) when your environment has drifted from
+the installed unit. To fix it directly, from a shell where the tool works:
+
+```bash
+lumi-runner service install    # rewrites the unit with your current PATH
+```
+
 ## How many jobs at once
 
 One, until you say otherwise — the same behaviour this daemon has always had.
@@ -187,8 +204,11 @@ always as a warning — being out of date never fails the preflight.
 - **Idle sleep is inhibited** (`caffeinate` / `systemd-inhibit`, best-effort on Windows), so a
   laptop doesn't suspend mid-session. It cannot veto you choosing Shut Down — no background process
   gets that veto on macOS, and it shouldn't.
-- **A desktop notification** fires on job start, finish and terminal failure, and when the daemon is
-  stopped with work in flight. Silence it with `lumi-runner config set notifications off`.
+- **Desktop notifications are off by default.** They fire on job start, finish, terminal failure and
+  on a daemon stopped with work in flight — which on a busy machine is an interruption every few
+  minutes carrying nothing the task board and the Daemons live log do not already show. Turn them on
+  with `lumi-runner config set notifications on`. (Sleep inhibition is a separate setting and stays
+  on: "don't close the lid" is `keepAwake`, not a notification.)
 - **SIGTERM releases the job.** The daemon aborts every running session, hands each job back to the
   queue with its retry budget **unspent**, and only then writes itself offline. Stopping the daemon
   never costs you an attempt.
@@ -254,7 +274,7 @@ claiming from one queue.
 |---|---|
 | `LUMI_RUNNER_HOME` | Config + log directory (default `~/.lumi-runner`) |
 | `CREW_CLAUDE_BIN` | Path to the Claude binary (default: `claude` from `PATH`) |
-| `CREW_NO_NOTIFY` | Disable desktop notifications |
+| `CREW_NO_NOTIFY` | Force desktop notifications off, whatever the config says (they are off by default) |
 | `CREW_NO_POWER` | Disable sleep inhibition |
 | `LUMI_RUNNER_REGISTRY` | npm registry to check for updates (default `https://registry.npmjs.org`) |
 | `LUMI_RUNNER_CHANNEL` | Force the update channel: `latest` or `dev` |

package/dist/cli.js

@@ -230,6 +230,8 @@ function jobTarget(job) {
     return { kind: "chat", chatId: job.chatId };
   return null;
 }
+var MAX_RUN_REPORT_CHARS = 6e3;
+var MAX_RUN_REPORT_SUMMARY_CHARS = 300;
 
 // ../shared/dist/jobProgress.js
 var MAX_JOB_STEPS = 10;
@@ -251,9 +253,9 @@ function str(input, key) {
   const value = input[key];
   return typeof value === "string" && value.trim() ? value : void 0;
 }
-function basename(path8) {
-  const parts = path8.split(/[\\/]/).filter(Boolean);
-  return parts[parts.length - 1] ?? path8;
+function basename(path9) {
+  const parts = path9.split(/[\\/]/).filter(Boolean);
+  return parts[parts.length - 1] ?? path9;
 }
 function hostOf(url) {
   try {
@@ -341,8 +343,8 @@ function builtinDetail(tool, input) {
     case "Write":
     case "Edit":
     case "MultiEdit": {
-      const path8 = str(input, "file_path");
-      return path8 ? basename(path8) : void 0;
+      const path9 = str(input, "file_path");
+      return path9 ? basename(path9) : void 0;
     }
     case "Glob":
     case "Grep":
@@ -759,12 +761,20 @@ function forgetShip(config2, shipId) {
 function allowsLocalMcp(config2, shipId) {
   return Array.isArray(config2.allowLocalMcp) && config2.allowLocalMcp.includes(shipId);
 }
+var TOGGLE_DEFAULTS = {
+  notifications: false,
+  keepAwake: true,
+  autoUpdate: true
+};
+function notificationsEnabled(config2) {
+  return config2?.notifications ?? TOGGLE_DEFAULTS.notifications;
+}
 function mcpUrl(config2) {
   return process.env.CREW_MCP_URL || config2.mcpUrl || `https://us-central1-${config2.projectId}.cloudfunctions.net/workspaceMcp`;
 }
 
 // src/version.ts
-var RUNNER_VERSION = true ? "0.6.2" : "0.0.0-dev";
+var RUNNER_VERSION = true ? "0.6.4" : "0.0.0-dev";
 
 // src/auth.ts
 import { signInWithCustomToken } from "firebase/auth";
@@ -1030,7 +1040,7 @@ var WINDOWS_SCRIPT = [
 ].join(" ");
 function enabled() {
   if (process.env.CREW_NO_NOTIFY === "1") return false;
-  return loadConfig()?.notifications !== false;
+  return notificationsEnabled(loadConfig());
 }
 function notify(title, body) {
   if (!enabled()) return;
@@ -1188,6 +1198,13 @@ async function loadSettledBlockers(shipRef, task) {
     return [];
   }
 }
+function reportSummaryLine(r) {
+  if (r.summary?.trim()) return r.summary.trim();
+  const body = r.report?.trim();
+  if (!body) return "No report.";
+  const firstLine2 = body.split("\n").find((l) => l.trim())?.trim() ?? body;
+  return firstLine2.length <= MAX_RUN_REPORT_SUMMARY_CHARS ? firstLine2 : `${firstLine2.slice(0, MAX_RUN_REPORT_SUMMARY_CHARS - 1)}\u2026`;
+}
 async function loadJobContext(db, shipId, job) {
   const shipRef = doc(db, COLLECTIONS.ships, shipId);
   const taskRef = doc(shipRef, COLLECTIONS.tasks, job.taskId);
@@ -1268,7 +1285,13 @@ async function loadJobContext(db, shipId, job) {
     activity: activityDocs.map((d) => ({ id: d.id, ...d.data() })).reverse(),
     // read newest-first, rendered chronologically
     activityTruncated: activitySnap.docs.length > MAX_ACTIVITY_IN_PROMPT,
-    previousReports: jobsSnap.docs.filter((d) => d.id !== job.id).slice(0, MAX_PREVIOUS_JOBS_READ).map((d) => d.data().report).filter((r) => !!r).reverse()
+    // §15.41. No longer filtered down to jobs that HAVE a report: the catalog carries a line per
+    // previous run, and "that run left no report" is itself worth knowing — it is the difference
+    // between a task nobody has worked and one where three runs died without saying why.
+    previousReports: jobsSnap.docs.filter((d) => d.id !== job.id).slice(0, MAX_PREVIOUS_JOBS_READ).map((d) => {
+      const data = d.data();
+      return { jobId: d.id, summary: data.reportSummary ?? null, report: data.report ?? null };
+    }).reverse()
     // chronological
   };
 }
@@ -1293,7 +1316,11 @@ function standingRules(ship2, statuses, task, agent) {
     // Amended with the pointer clause, which is the only thing keeping the three tiers from
     // duplicating each other: without it a diligent agent writes the same fact into its report,
     // its notebook and the org brain, and the report grows into a transcript of the other two.
-    "End every session by calling run_report: what you did, what remains, what the next run must know. Do not repeat what you already put in your memory or in Ship knowledge \u2014 point at it instead (the entry id, or the slug)."
+    // §15.41 added the summary clause. It is second, not an afterthought at the end, because that
+    // one string is what every LATER run on this task reads instead of this report — and what a
+    // human sees in their notification. An agent that treats it as a label writes "Run report",
+    // and four runs later the catalog says nothing at all.
+    "End every session by calling run_report: what you did, what remains, what the next run must know. Give it a summary too \u2014 one or two sentences that stand in for the whole report in later runs, so write the sentence you would want the next run to read first. Do not repeat what you already put in your memory or in Ship knowledge \u2014 point at it instead (the entry id, or the slug)."
   ].join("\n- ");
 }
 function formatTaskDate(millis, now = Date.now()) {
@@ -1333,26 +1360,17 @@ function buildPrompt(ctx, reason, mcpServers = [], tokenRepos, githubUnavailable
   const parts = [];
   const statuses = shipTaskStatuses(ctx.ship);
   const playbook = usableWorkflow(ctx.workflow);
+  let preamble = null;
   if (reason === "activity") {
-    parts.push(
-      "# Why this session started\n\nSomeone posted on your task after your last run. Read the newest activity below" + (playbook ? ", then follow your playbook below." : ", respond via task_comment, do any follow-up work it asks for, and set the task status honestly if it needs to change.")
-    );
+    preamble = "# Why this session started\n\nSomeone posted on your task after your last run. Read the newest activity below" + (playbook ? ", then follow your playbook above." : ", respond via task_comment, do any follow-up work it asks for, and set the task status honestly if it needs to change.");
   } else if (reason === "watch") {
-    parts.push(
-      "# Why this session started\n\nOne of your playbooks watches the board, and this task matched it. Nobody assigned it to you and nobody else is working it right now \u2014 you are picking it up.\n\n**Leave it in a status that says what happened.** The board is how the watch decides what still needs attention, so a task you leave in the same status will come back to you on the next scan" + (playbook ? ", which your playbook below is written for." : ".")
-    );
+    preamble = "# Why this session started\n\nOne of your playbooks watches the board, and this task matched it. Nobody assigned it to you and nobody else is working it right now \u2014 you are picking it up.\n\n**Leave it in a status that says what happened.** The board is how the watch decides what still needs attention, so a task you leave in the same status will come back to you on the next scan" + (playbook ? ", which your playbook above is written for." : ".");
   } else if (reason === "resume_after_approval") {
-    parts.push(
-      "# Why this session started\n\nYou asked a captain for permission and stopped. They have now answered \u2014 their decision is the newest entry in the task activity below. Read it first.\n\n**If they approved it, do that thing now** \u2014 the permission is granted for this task and may be single-use, so do not ask again for the same thing. **If they refused, do not retry and do not look for a way around it**: say what you will do instead, or hand the task back with task_assign."
-    );
+    preamble = "# Why this session started\n\nYou asked a captain for permission and stopped. They have now answered \u2014 their decision is the newest entry in the task activity below. Read it first.\n\n**If they approved it, do that thing now** \u2014 the permission is granted for this task and may be single-use, so do not ask again for the same thing. **If they refused, do not retry and do not look for a way around it**: say what you will do instead, or hand the task back with task_assign.";
   } else if (reason === "unblocked") {
-    parts.push(
-      '# Why this session started\n\nThis task was blocked and is not any more: everything it was waiting on is now done. Nobody has just assigned it to you \u2014 you have had it all along, and the work has become startable.\n\n**Read "What you were waiting on" below before you do anything else.** It carries what those tasks produced, which is the input this work was held up for; starting without it means redoing or contradicting somebody else' + (playbook ? ", then follow your playbook below." : "'s work.")
-    );
+    preamble = '# Why this session started\n\nThis task was blocked and is not any more: everything it was waiting on is now done. Nobody has just assigned it to you \u2014 you have had it all along, and the work has become startable.\n\n**Read "What you were waiting on" below before you do anything else.** It carries what those tasks produced, which is the input this work was held up for; starting without it means redoing or contradicting somebody else' + (playbook ? ", then follow your playbook above." : "'s work.");
   } else if (reason === "schedule") {
-    parts.push(
-      "# Why this session started\n\nThis is a scheduled run: one of your own playbooks created this task on its cron and assigned it to you. It is routine work, not a request from a person, so nobody is waiting on a reply" + (playbook ? " \u2014 the playbook below is the work, and the task's description is only a record of why it exists." : ".")
-    );
+    preamble = "# Why this session started\n\nThis is a scheduled run: one of your own playbooks created this task on its cron and assigned it to you. It is routine work, not a request from a person, so nobody is waiting on a reply" + (playbook ? " \u2014 the playbook above is the work, and the task's description is only a record of why it exists." : ".");
   }
   parts.push(`# Who you are
 
@@ -1377,6 +1395,7 @@ This is the playbook for this kind of work \u2014 follow it. Your contract and t
 ${playbook.instructions}`
     );
   }
+  if (preamble) parts.push(preamble);
   const t = ctx.task;
   const now = Date.now();
   const statusLabel = statusById(statuses, t.status)?.label ?? t.status;
@@ -1428,12 +1447,28 @@ ${p.description || "(no description)"}`
 ${note2}${feed}`);
   }
   if (ctx.previousReports.length > 0) {
-    const reports = ctx.previousReports.map((r, i) => `## Run ${i + 1}
+    const runs = ctx.previousReports;
+    const latest = runs[runs.length - 1];
+    const older = runs.slice(0, -1);
+    const sections = [];
+    if (older.length > 0) {
+      const lines = older.map((r) => `- (\`${r.jobId}\`) ${reportSummaryLine(r)}`).join("\n");
+      sections.push(
+        `## Earlier runs
+
+Oldest first, one line each. To read one in full, call \`task_get\` with this task's id and \`reportJobId\` set to the id in brackets.
 
-${r}`).join("\n\n");
+${lines}`
+      );
+    }
+    sections.push(
+      `## Most recent run (\`${latest.jobId}\`)
+
+${latest.report ?? "That run left no report."}`
+    );
     parts.push(`# Reports from previous runs on this task
 
-${reports}`);
+${sections.join("\n\n")}`);
   }
   const githubRepos = tokenRepos ?? ctx.agent.tools.github.repos;
   if (ctx.agent.tools.github.enabled && githubUnavailable) {
@@ -1509,7 +1544,10 @@ async function loadChatContext(db, shipId, job) {
           limit2(MAX_CHAT_REPORTS + 1)
         )
       );
-      previousReports = jobsSnap.docs.filter((d) => d.id !== job.id).slice(0, MAX_CHAT_REPORTS).map((d) => d.data().report).filter((r) => !!r).reverse();
+      previousReports = jobsSnap.docs.filter((d) => d.id !== job.id).slice(0, MAX_CHAT_REPORTS).map((d) => {
+        const data = d.data();
+        return { jobId: d.id, summary: data.reportSummary ?? null, report: data.report ?? null };
+      }).reverse();
     } catch {
       previousReports = [];
     }
@@ -1545,7 +1583,7 @@ function chatStandingRules(ship2, agent) {
     scopeRule,
     registerRule,
     routingRule,
-    "End the session by calling run_report: what was discussed and anything the next run must know. Do not repeat what you already put in your memory or in Ship knowledge \u2014 point at it instead."
+    "End the session by calling run_report: what was discussed and anything the next run must know. Give it a summary too \u2014 one or two sentences that stand in for the whole report once this conversation is long enough to scroll out of the window. Do not repeat what you already put in your memory or in Ship knowledge \u2014 point at it instead."
   ].join("\n- ");
 }
 function buildChatPrompt(ctx, mcpServers = [], tokenRepos, githubUnavailable) {
@@ -1568,15 +1606,22 @@ ${ctx.agent.contract}`);
 
 - ${chatStandingRules(ctx.ship, ctx.agent)}`);
   if (ctx.previousReports.length > 0) {
-    const reports = ctx.previousReports.map((r, i) => `## Earlier run ${i + 1}
+    const runs = ctx.previousReports;
+    const latest = runs[runs.length - 1];
+    const older = runs.slice(0, -1);
+    const sections = [];
+    if (older.length > 0) {
+      sections.push(older.map((r) => `- ${reportSummaryLine(r)}`).join("\n"));
+    }
+    sections.push(`## Most recent run
 
-${r}`).join("\n\n");
+${latest.report ?? "That run left no report."}`);
     parts.push(
       `# Earlier in this chat
 
 The conversation is longer than the window below. These are your own notes from previous runs on this chat, oldest first.
 
-${reports}`
+${sections.join("\n\n")}`
     );
   }
   const title = ctx.chat.title?.trim();
@@ -1890,7 +1935,7 @@ async function claudeHealthCheck() {
     const done = (health) => {
       if (settled) return;
       settled = true;
-      resolve(health);
+      resolve({ ...health, binary: bin });
     };
     let stdout = "";
     const child = spawn3(bin, ["--version"], { stdio: ["ignore", "pipe", "ignore"] });
@@ -2428,15 +2473,28 @@ function redactTranscript(transcript, knownSecrets) {
   return out;
 }
 async function uploadTranscript(storage, shipId, jobId, redacted) {
-  const path8 = `crew/${shipId}/transcripts/${jobId}.jsonl`;
-  await uploadBytes(storageRef(storage, path8), new TextEncoder().encode(redacted), {
+  const path9 = `crew/${shipId}/transcripts/${jobId}.jsonl`;
+  await uploadBytes(storageRef(storage, path9), new TextEncoder().encode(redacted), {
     contentType: "application/x-ndjson"
   });
-  return path8;
+  return path9;
 }
 function utcDay(millis) {
   return new Date(millis).toISOString().slice(0, 10);
 }
+function backstopReportContent(resultText) {
+  const text = resultText.trim();
+  if (!text) {
+    const none = "This run ended without writing a report, and left no final text to fall back on.";
+    return { report: none, summary: none };
+  }
+  const marker = "\n\n\u2026(truncated)";
+  const report4 = text.length <= MAX_RUN_REPORT_CHARS ? text : `${text.slice(0, MAX_RUN_REPORT_CHARS - marker.length)}${marker}`;
+  const firstLine2 = text.split("\n").find((l) => l.trim())?.trim() ?? text;
+  const ellipsis = "\u2026";
+  const summary = firstLine2.length <= MAX_RUN_REPORT_SUMMARY_CHARS ? firstLine2 : `${firstLine2.slice(0, MAX_RUN_REPORT_SUMMARY_CHARS - ellipsis.length)}${ellipsis}`;
+  return { report: report4, summary };
+}
 async function finalizeJob(db, shipId, job, input) {
   const shipRef = doc6(db, COLLECTIONS.ships, shipId);
   const jobRef = doc6(shipRef, COLLECTIONS.jobs, job.id);
@@ -2444,6 +2502,7 @@ async function finalizeJob(db, shipId, job, input) {
   const usageRef = doc6(shipRef, COLLECTIONS.usageDaily, utcDay(now));
   await runTransaction(db, async (tx) => {
     const usageSnap = await tx.get(usageRef);
+    const jobSnap = await tx.get(jobRef);
     const totals = { ...EMPTY_USAGE_TOTALS, ...usageSnap.data()?.totals ?? {} };
     const byAgent = usageSnap.data()?.byAgent ?? {};
     const agentAgg = {
@@ -2451,7 +2510,10 @@ async function finalizeJob(db, shipId, job, input) {
       ...byAgent[job.agentId] ?? {}
     };
     const u = input.usage;
+    const wroteReport = !!jobSnap.data()?.report;
+    const backstop = !wroteReport && input.resultText !== void 0 ? backstopReportContent(input.resultText) : null;
     tx.update(jobRef, {
+      ...backstop ? { report: backstop.report, reportSummary: backstop.summary } : {},
       status: input.status,
       endedAt: now,
       usage: u,
@@ -2771,6 +2833,10 @@ function serviceEnv() {
   }
   return env;
 }
+function serviceEnvDrift(current, installed) {
+  if (!installed) return [];
+  return Object.keys(current).filter((name) => current[name] !== installed[name]).sort();
+}
 function removeLegacyService() {
   const removed = [];
   if (process.platform === "darwin") {
@@ -2869,6 +2935,24 @@ function parsePlistCliPath(plist) {
   const args = [...block[1].matchAll(/<string>([\s\S]*?)<\/string>/g)].map((m) => unescapeXml(m[1]));
   return args.length >= 2 ? args[1] : void 0;
 }
+function parsePlistEnv(plist) {
+  const block = /<key>EnvironmentVariables<\/key>\s*<dict>([\s\S]*?)<\/dict>/.exec(plist);
+  if (!block) return void 0;
+  const env = {};
+  for (const m of block[1].matchAll(/<key>([\s\S]*?)<\/key>\s*<string>([\s\S]*?)<\/string>/g)) {
+    env[unescapeXml(m[1])] = unescapeXml(m[2]);
+  }
+  return env;
+}
+function parseSystemdEnv(unit) {
+  const env = {};
+  let found = false;
+  for (const m of unit.matchAll(/^Environment="([^="]+)=([^"]*)"\s*$/gm)) {
+    env[m[1]] = m[2];
+    found = true;
+  }
+  return found ? env : void 0;
+}
 function parseSystemdCliPath(unit) {
   const line = /^ExecStart=(.*)$/m.exec(unit);
   if (!line) return void 0;
@@ -2877,16 +2961,17 @@ function parseSystemdCliPath(unit) {
   const parts = command.slice(0, -" start".length).split(" ");
   return parts.length === 2 ? parts[1] : void 0;
 }
-function execFacts(unitPath, parse) {
+function execFacts(unitPath, parse, parseEnv) {
   let text;
   try {
     text = fs4.readFileSync(unitPath, "utf8");
   } catch {
     return {};
   }
+  const unitEnv = parseEnv(text);
   const execPath = parse(text);
-  if (!execPath || !path4.isAbsolute(execPath)) return {};
-  return { execPath, execMissing: !fs4.existsSync(execPath) };
+  if (!execPath || !path4.isAbsolute(execPath)) return { ...unitEnv ? { unitEnv } : {} };
+  return { execPath, execMissing: !fs4.existsSync(execPath), ...unitEnv ? { unitEnv } : {} };
 }
 function systemdUnit() {
   const env = serviceEnv();
@@ -2913,7 +2998,7 @@ function serviceStatus() {
   if (process.platform === "darwin") {
     const unitPath = launchAgentPath();
     if (!fs4.existsSync(unitPath)) return { state: "not-installed", detail: "No LaunchAgent installed." };
-    const exec = execFacts(unitPath, parsePlistCliPath);
+    const exec = execFacts(unitPath, parsePlistCliPath, parsePlistEnv);
     const printed = run("launchctl", ["print", `gui/${uid()}/${SERVICE_LABEL}`]);
     if (!printed.ok) {
       return { state: "installed", detail: "LaunchAgent present but not loaded.", unitPath, ...exec };
@@ -2934,7 +3019,7 @@ function serviceStatus() {
       state: active.out === "active" ? "running" : "installed",
       detail: `systemd user unit is ${active.out || "unknown"}.`,
       unitPath,
-      ...execFacts(unitPath, parseSystemdCliPath)
+      ...execFacts(unitPath, parseSystemdCliPath, parseSystemdEnv)
     };
   }
   if (process.platform === "win32") {
@@ -4005,6 +4090,10 @@ async function startDaemon() {
           status: "stopped",
           usage,
           transcriptPath,
+          // §15.41. A stopped run is the case that needs the backstop MOST: it was cut off
+          // mid-thought, so it almost certainly never reached `run_report` — and whatever it had
+          // got to is what the next run on this task would otherwise have to rediscover.
+          resultText,
           mcpServers: extraMcpServers.map((s) => s.key)
         });
         const by = slot.stop.by;
@@ -4046,6 +4135,9 @@ async function startDaemon() {
           status: "done",
           usage,
           transcriptPath,
+          // §15.41. Only used when the session never called `run_report` — the ordinary path is
+          // that it did, and a real report always wins inside the transaction.
+          resultText,
           mcpServers: extraMcpServers.map((s) => s.key)
         });
         log2(`Job ${job.id} done (${usage.inputTokens}in/${usage.outputTokens}out tokens).`);
@@ -4099,6 +4191,10 @@ async function startDaemon() {
           usage,
           transcriptPath,
           error: failure,
+          // §15.41. A failed run still did work, and the retry — or the next run after the retry
+          // is spent — starts from the context pack alone. `error` is the tail for a human; this
+          // is the continuity for the next session, and they are read by different readers.
+          resultText,
           mcpServers: extraMcpServers.map((s) => s.key)
         });
         if (target.kind === "chat") {
@@ -4437,7 +4533,7 @@ function glyph(level) {
 
 // src/cli/commands/config.ts
 var TOGGLES = {
-  notifications: "Desktop notifications when a job starts, finishes or fails",
+  notifications: "Desktop notifications when a job starts, finishes or fails (off by default)",
   keepAwake: "Keep this machine awake while a job is running",
   autoUpdate: "Install new versions of the runner by itself and restart (PRD \xA715.37)"
 };
@@ -4487,7 +4583,7 @@ function parseParallel(value) {
 async function runConfigList() {
   const config2 = requireConfig();
   const values = Object.fromEntries(
-    Object.keys(TOGGLES).map((key) => [key, config2[key] !== false])
+    Object.keys(TOGGLES).map((key) => [key, config2[key] ?? TOGGLE_DEFAULTS[key]])
   );
   const machine = machineCap(config2);
   const ships = config2.ships.map((shipId) => ({
@@ -4646,6 +4742,8 @@ function setParallel(config2, value, ship2) {
 
 // src/cli/commands/doctor.ts
 import { spawnSync as spawnSync3 } from "node:child_process";
+import fs7 from "node:fs";
+import path8 from "node:path";
 import { collection as collection6, doc as doc8, getDoc as getDoc7, getDocs as getDocs5 } from "firebase/firestore";
 
 // src/cli/session.ts
@@ -4717,8 +4815,36 @@ function serviceCheckFrom(status) {
   }
   return warn("service", "Background service", `Unrecognised service state: ${status.state}.`);
 }
-function checkService() {
-  return serviceCheckFrom(serviceStatus());
+function serviceBinaryCheckFrom(input) {
+  const { status, binaries, resolves } = input;
+  if (status.state === "not-installed" || status.state === "unsupported") return null;
+  const pathValue = status.unitEnv?.PATH;
+  if (pathValue === void 0) return null;
+  const relative = [...new Set(binaries)].filter((b) => !path8.isAbsolute(b));
+  if (relative.length === 0) return null;
+  const missing = relative.filter((b) => !resolves(b, pathValue));
+  const id = "service:path";
+  const label = "Service PATH";
+  if (missing.length === 0) {
+    return ok(id, label, `The daemon can reach ${relative.join(", ")}.`);
+  }
+  return warn(
+    id,
+    label,
+    `The daemon's PATH is missing ${missing.join(", ")} \u2014 it was captured when the service was installed, and jobs will fail with \`spawn ${missing[0]} ENOENT\` even though this terminal finds it.`,
+    "Run `lumi-runner service install` from a shell where these work, to rewrite the unit with the current PATH."
+  );
+}
+function resolvesOnPath(binary, pathValue) {
+  for (const dir of pathValue.split(path8.delimiter)) {
+    if (!dir) continue;
+    try {
+      fs7.accessSync(path8.join(dir, binary), fs7.constants.X_OK);
+      return true;
+    } catch {
+    }
+  }
+  return false;
 }
 function versionCheckFrom(input) {
   const { current, latest, channel, autoUpdate } = input;
@@ -4886,14 +5012,17 @@ async function runDoctor() {
     for (const id of shipResults.engines) engines.add(id);
   }
   const needsGithub = shipResults.needsGithub;
+  const jobBinaries = [];
   for (const engineId of engines) {
     const health = await getDriver(engineId).healthCheck();
+    if (health.binary) jobBinaries.push(health.binary);
     checks.push(
       health.ok ? ok(`engine:${engineId}`, `Engine "${engineId}"`, health.detail) : fail(`engine:${engineId}`, `Engine "${engineId}"`, health.detail, health.fix)
     );
   }
   if (needsGithub) {
     for (const binary of ["git", "gh"]) {
+      jobBinaries.push(binary);
       checks.push(
         onPath(binary) ? ok(`bin:${binary}`, `\`${binary}\``, "On PATH.") : fail(
           `bin:${binary}`,
@@ -4905,7 +5034,14 @@ async function runDoctor() {
     }
   }
   checks.push(await checkMcp(mcpUrl(config2)));
-  checks.push(checkService());
+  const service2 = serviceStatus();
+  checks.push(serviceCheckFrom(service2));
+  const servicePath = serviceBinaryCheckFrom({
+    status: service2,
+    binaries: jobBinaries,
+    resolves: resolvesOnPath
+  });
+  if (servicePath) checks.push(servicePath);
   checks.push(await checkVersion(config2));
   progress.stop("Checks complete.");
   return report2(checks);
@@ -5161,6 +5297,17 @@ async function runServiceUninstall() {
   say.success("Service removed. The daemon will not start on its own any more.");
   return 0;
 }
+async function runServiceRepair(reason) {
+  const result = installService();
+  restartService();
+  if (isJson()) {
+    emitJson({ repaired: true, reason, unitPath: result.unitPath, notes: result.notes, ...serviceStatus() });
+    return 0;
+  }
+  say.success(`Service reinstalled (${reason}): ${result.unitPath}`);
+  for (const note2 of result.notes) say.warn(note2);
+  return 0;
+}
 async function runServiceRestart() {
   restartService();
   if (isJson()) {
@@ -5185,20 +5332,20 @@ async function runServiceStatus() {
 }
 
 // src/cli/commands/uninstall.ts
-import fs7 from "node:fs";
+import fs8 from "node:fs";
 async function runUninstall(options) {
   const before = serviceStatus();
   const dir = configDir();
   const hadService = before.state !== "not-installed" && before.state !== "unsupported";
   if (hadService) uninstallService();
   let purged = false;
-  if (options.purge && fs7.existsSync(dir)) {
+  if (options.purge && fs8.existsSync(dir)) {
     const confirmed = await promptConfirm({
       message: `Delete ${dir}? This machine loses its identity \u2014 a captain has to approve it again after reinstalling.`,
       initialValue: false
     });
     if (confirmed) {
-      fs7.rmSync(dir, { recursive: true, force: true });
+      fs8.rmSync(dir, { recursive: true, force: true });
       purged = true;
     }
   }
@@ -5335,7 +5482,8 @@ async function runSetup(options) {
   }
   say.step("Checking this machine\u2026");
   const doctorExit = await runDoctor();
-  const service2 = serviceStatus().state;
+  const status = serviceStatus();
+  const service2 = status.state;
   if (service2 === "not-installed") {
     const install = await promptConfirm({
       message: "Start the daemon automatically whenever this machine is on?",
@@ -5345,8 +5493,14 @@ async function runSetup(options) {
     if (install) await runServiceInstall();
     else say.info("Skipped. Run `lumi-runner start` manually, or `lumi-runner service install` later.");
   } else if (service2 === "running" || service2 === "installed") {
-    say.step("Restarting the daemon so it picks up this configuration\u2026");
-    await runServiceRestart();
+    const drift = serviceEnvDrift(serviceEnv(), status.unitEnv);
+    if (drift.length > 0) {
+      say.step(`Reinstalling the service \u2014 ${drift.join(", ")} changed since it was installed\u2026`);
+      await runServiceRepair(`${drift.join(", ")} changed`);
+    } else {
+      say.step("Restarting the daemon so it picks up this configuration\u2026");
+      await runServiceRestart();
+    }
   }
   say.outro(
     doctorExit === 0 ? "Ready. This machine will claim jobs for its Ships." : "Setup finished, but some checks failed \u2014 fix those and re-run `lumi-runner doctor`."

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lumi.ai/runner",
-  "version": "0.6.2",
+  "version": "0.6.4",
   "type": "module",
   "description": "Lumi Crew runner daemon — claims jobs from your Ships and executes them as headless Claude sessions on your own machine.",
   "//name": "The ONLY package in this monorepo published to the public registry, so it is the one that does not follow the internal @lumi/crew-* convention: `@lumi` is not a scope we own, `@lumi.ai` is (the npm org). The workspace DIRECTORY stays packages/crew/runner — renaming the package is not renaming the folder.",