@lumi.ai/runner 0.1.0 → 0.3.0-dev.6

package/README.md

@@ -52,6 +52,7 @@ lumi-runner login --token <customToken> --api-key <key> --project <projectId>
 | `ship add [ids…]` / `remove` / `list` | Which Ships this machine serves (omit ids to pick from a list) |
 | `doctor` | **Preflight** — can this machine actually run a job? Non-zero exit if not |
 | `service install` / `uninstall` / `restart` / `status` | Run the daemon in the background, always |
+| `uninstall` | Remove the service, then tell you the `npm rm -g` step (`--purge` also clears config) |
 | `start` | Run the daemon in the foreground |
 | `status` | Ships, approval state, current job, queue depth, today's tokens |
 | `logs [-f]` | The daemon's own rotating log file |
@@ -69,7 +70,7 @@ 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, and the background service.
+reachability, the background service, and whether that service still points at a CLI that exists.
 
 ### Background service
 
@@ -104,6 +105,34 @@ from the transcript before upload. `lumi-runner doctor` tells you when a Ship is
 
 The only thing stored locally is your own session, in `~/.lumi-runner/config.json` (mode `0600`).
 
+## Uninstalling
+
+Remove the service **before** the package:
+
+```bash
+lumi-runner uninstall
+npm rm -g @lumi.ai/runner
+```
+
+Order matters, and getting it wrong is silent. The launchd plist / systemd unit records an
+*absolute path* to the CLI inside `node_modules` — it has to, since the OS needs something to exec.
+`npm rm -g` deletes that file but knows nothing about your service, so the service stays registered
+and keeps respawning a binary that is no longer there, roughly every ten seconds, for as long as
+the machine lives. npm cannot help here: since npm 7 it does not run `preuninstall`/`postuninstall`
+scripts at all, so no package can clean up after itself.
+
+Already removed the package? Reinstall it, uninstall properly, then remove it again:
+
+```bash
+npm i -g @lumi.ai/runner && lumi-runner uninstall && npm rm -g @lumi.ai/runner
+```
+
+`lumi-runner doctor` reports a machine that is already in this state, naming the missing path.
+
+Neither command touches `~/.lumi-runner`. That directory is this machine's **identity** — its
+runner id and your session — so uninstalling and reinstalling keeps the approvals your captains
+already granted. Add `--purge` to delete it too, and expect to be approved again from scratch.
+
 ## Upgrading from `crew-runner`
 
 This package was previously distributed inside the Lumi monorepo as `@lumi/crew-runner`, with a

package/dist/cli.js

@@ -502,7 +502,7 @@ function mcpUrl(config2) {
 }
 
 // src/version.ts
-var RUNNER_VERSION = true ? "0.1.0" : "0.0.0-dev";
+var RUNNER_VERSION = true ? "0.3.0-dev.6" : "0.0.0-dev";
 
 // src/auth.ts
 import os2 from "node:os";
@@ -2563,6 +2563,34 @@ ${envEntries}
 function escapeXml(value) {
   return value.replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/"/g, "&quot;");
 }
+function unescapeXml(value) {
+  return value.replace(/&lt;/g, "<").replace(/&gt;/g, ">").replace(/&quot;/g, '"').replace(/&amp;/g, "&");
+}
+function parsePlistCliPath(plist) {
+  const block = /<key>ProgramArguments<\/key>\s*<array>([\s\S]*?)<\/array>/.exec(plist);
+  if (!block) return void 0;
+  const args = [...block[1].matchAll(/<string>([\s\S]*?)<\/string>/g)].map((m) => unescapeXml(m[1]));
+  return args.length >= 2 ? args[1] : void 0;
+}
+function parseSystemdCliPath(unit) {
+  const line = /^ExecStart=(.*)$/m.exec(unit);
+  if (!line) return void 0;
+  const command = line[1].trim();
+  if (!command.endsWith(" start")) return void 0;
+  const parts = command.slice(0, -" start".length).split(" ");
+  return parts.length === 2 ? parts[1] : void 0;
+}
+function execFacts(unitPath, parse) {
+  let text;
+  try {
+    text = fs4.readFileSync(unitPath, "utf8");
+  } catch {
+    return {};
+  }
+  const execPath = parse(text);
+  if (!execPath || !path4.isAbsolute(execPath)) return {};
+  return { execPath, execMissing: !fs4.existsSync(execPath) };
+}
 function systemdUnit() {
   const env = serviceEnv();
   const envLines = Object.entries(env).map(([k, v]) => `Environment="${k}=${v}"`).join("\n");
@@ -2588,13 +2616,17 @@ 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 printed = run("launchctl", ["print", `gui/${uid()}/${SERVICE_LABEL}`]);
-    if (!printed.ok) return { state: "installed", detail: "LaunchAgent present but not loaded.", unitPath };
+    if (!printed.ok) {
+      return { state: "installed", detail: "LaunchAgent present but not loaded.", unitPath, ...exec };
+    }
     const running = /\bpid = \d+/.test(printed.out);
     return {
       state: running ? "running" : "installed",
       detail: running ? "LaunchAgent loaded and running." : "LaunchAgent loaded but not running.",
-      unitPath
+      unitPath,
+      ...exec
     };
   }
   if (process.platform === "linux") {
@@ -2604,7 +2636,8 @@ function serviceStatus() {
     return {
       state: active.out === "active" ? "running" : "installed",
       detail: `systemd user unit is ${active.out || "unknown"}.`,
-      unitPath
+      unitPath,
+      ...execFacts(unitPath, parseSystemdCliPath)
     };
   }
   if (process.platform === "win32") {
@@ -2758,19 +2791,32 @@ async function checkMcp(url) {
     );
   }
 }
-function checkService() {
-  const status = serviceStatus();
+function serviceCheckFrom(status) {
+  if (status.execMissing) {
+    return warn(
+      "service",
+      "Background service",
+      `Installed, but the unit points at a CLI that no longer exists (${status.execPath}). It is registered and retrying forever without ever starting.`,
+      "Run `lumi-runner uninstall` to remove it, or `lumi-runner service install` to repoint it at this install."
+    );
+  }
   if (status.state === "running") return ok("service", "Background service", status.detail);
   if (status.state === "installed") {
     return warn("service", "Background service", status.detail, "Run `lumi-runner service restart`.");
   }
   if (status.state === "unsupported") return warn("service", "Background service", status.detail);
-  return warn(
-    "service",
-    "Background service",
-    "Not installed \u2014 the daemon only runs while a terminal is open.",
-    "Run `lumi-runner service install` to keep this Ship online whenever the machine is."
-  );
+  if (status.state === "not-installed") {
+    return warn(
+      "service",
+      "Background service",
+      "Not installed \u2014 the daemon only runs while a terminal is open.",
+      "Run `lumi-runner service install` to keep this Ship online whenever the machine is."
+    );
+  }
+  return warn("service", "Background service", `Unrecognised service state: ${status.state}.`);
+}
+function checkService() {
+  return serviceCheckFrom(serviceStatus());
 }
 async function checkShips(session) {
   const { config: config2, fb, user } = session;
@@ -3142,6 +3188,48 @@ async function runServiceStatus() {
   }
   say.line(`  ${status.detail}`);
   if (status.unitPath) say.line(`  ${status.unitPath}`);
+  if (status.execMissing) {
+    say.warn(`The unit runs ${status.execPath}, which no longer exists. Run \`lumi-runner uninstall\`.`);
+  }
+  return 0;
+}
+
+// src/cli/commands/uninstall.ts
+import fs5 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 && fs5.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) {
+      fs5.rmSync(dir, { recursive: true, force: true });
+      purged = true;
+    }
+  }
+  if (isJson()) {
+    emitJson({
+      serviceRemoved: hadService,
+      unitPath: before.unitPath ?? null,
+      configDir: dir,
+      configRemoved: purged,
+      // The step this command cannot take for the user — see the note above.
+      remaining: "npm rm -g @lumi.ai/runner"
+    });
+    return 0;
+  }
+  if (hadService) say.success(`Service removed${before.unitPath ? ` (${before.unitPath})` : ""}.`);
+  else say.info("No background service was installed.");
+  if (purged) say.success(`Removed ${dir}.`);
+  else if (options.purge) say.info(`Left ${dir} alone.`);
+  else say.info(`Kept ${dir} \u2014 your session and this machine's runner id. Add --purge to remove it.`);
+  say.step("Now remove the package:");
+  say.line("    npm rm -g @lumi.ai/runner");
   return 0;
 }
 
@@ -3425,6 +3513,7 @@ service.command("install").description("Install and start the background service
 service.command("uninstall").description("Stop and remove the background service").action(action(runServiceUninstall));
 service.command("restart").description("Restart the background service").action(action(runServiceRestart));
 service.command("status").description("Is the background service installed and running?").action(action(runServiceStatus));
+program.command("uninstall").description("Remove the background service and print the final npm step").option("--purge", "also delete the config directory (loses this machine's runner id)").action(action(async (options) => runUninstall(options)));
 var config = program.command("config").description("Local per-machine preferences");
 config.command("list").description("Show current settings").action(action(runConfigList));
 config.command("set <key> <value>").description("Change a setting (notifications | keepAwake \u2192 on/off)").action(action(async (key, value) => runConfigSet(key, value)));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lumi.ai/runner",
-  "version": "0.1.0",
+  "version": "0.3.0-dev.6",
   "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.",