@bli-cockpit/cli 0.2.67 → 0.2.69

package/dist/commands/setup-receipt.js

@@ -15,8 +15,13 @@
  *                        in, so the session file is the evidence — there is no
  *                        second thing to ask.
  *   device               that same session, still unexpired.
- *   collector.autostart  `autostartStatus`, which asks launchd or Task
- *                        Scheduler what it actually has registered.
+ *   collector.autostart  `readAutostartRegistration`, which asks launchd or
+ *                        Task Scheduler what it actually has registered. That
+ *                        reader is SHARED with doctor's `autostart-alive` row
+ *                        (BLI-3793) — this module used to ask the scheduler
+ *                        itself, without the machine's saved roots, and so
+ *                        printed `autostart_not_loaded` under a doctor row
+ *                        that said the same agent was running.
  *
  * The five AGENT-HOST words in the printed block are NOT computed here. They
  * are BLI-3729's `MemoryInstallReceipt`, already read back off
@@ -36,7 +41,7 @@
 import os from "node:os";
 import { SETUP_RECEIPT_SCHEMA_VERSION, setupReceiptGaps, setupReceiptLine, } from "@bli-cockpit/telemetry-core";
 import { readHeartbeatMemoryReceipt } from "./heartbeat.js";
-import { autostartStatus } from "../autostart.js";
+import { autostartSetupPiece, readAutostartRegistration, } from "./autostart-reading.js";
 import { readLocalSessionReference } from "../local-state.js";
 import { getCollectorRuntimePaths } from "../local-state-paths.js";
 import { readJsonFile, writeJsonFile } from "../local-state-files.js";
@@ -63,14 +68,19 @@ export async function buildSetupReceipt(io, options = {}) {
     });
     const autostart = await probes.autostart().catch((error) => {
         console.error(`${TAG} scheduler could not be read`, JSON.stringify({ reason: "autostart_probe_failed", error_name: errorName(error) }));
-        return { status: "unreadable" };
+        return {
+            state: "unreadable",
+            reason: "autostart_probe_failed",
+            message: "the operating-system scheduler could not be read this run",
+            roots: [],
+        };
     });
     const receipt = {
         schema_version: SETUP_RECEIPT_SCHEMA_VERSION,
         checked_at: checkedAt,
         browser: browserPiece(session),
         device: devicePiece(session),
-        collector: { autostart: autostartPiece(autostart) },
+        collector: { autostart: autostartSetupPiece(autostart) },
     };
     const gaps = setupReceiptGaps(receipt, memory);
     console.error(`${TAG} read back`, JSON.stringify({
@@ -97,19 +107,16 @@ function withDefaultProbes(io, homeDir, options) {
                 };
             }),
         autostart: options.probes?.autostart ??
-            (async () => {
-                // The scheduler can only be READ through a process runner; a machine
-                // whose io carries none is unknown, never "absent" (BLI-2541: never
-                // report a state you did not observe).
-                const exec = io.exec;
-                if (!exec)
-                    return { status: "unreadable", message: "runner_unavailable" };
-                const result = await autostartStatus({ homeDir, exec });
-                return {
-                    status: result.status,
-                    ...(result.message ? { message: result.message } : {}),
-                };
-            }),
+            // The one shared reader. It handles the no-process-runner host itself
+            // (`unreadable`, never "absent" — BLI-2541: never report a state you did
+            // not observe) and it hands the scheduler this machine's saved roots,
+            // which is what stopped this word from contradicting doctor's row
+            // (BLI-3793).
+            (async () => readAutostartRegistration({
+                exec: io.exec,
+                homeDir,
+                ...(options.dashboardUrl ? { dashboardUrl: options.dashboardUrl } : {}),
+            })),
     };
 }
 /**
@@ -139,32 +146,6 @@ function devicePiece(session) {
     }
     return { status: "missing", reason: `session_${session.state}` };
 }
-/**
- * The scheduler is `ok` only when the host says it is registered AND loaded.
- * `installed` on macOS means the plist is on disk; `loaded` means launchd has
- * it. A plist nobody loaded runs nothing, which is exactly the failure that
- * looks fine from the outside.
- */
-function autostartPiece(result) {
-    if (result.status === "loaded" || result.status === "installed") {
-        return { status: "ok", reason: `autostart_${result.status}` };
-    }
-    if (result.status === "not_loaded") {
-        return { status: "missing", reason: "autostart_not_loaded" };
-    }
-    if (result.status === "absent" || result.status === "uninstalled") {
-        return { status: "missing", reason: "autostart_absent" };
-    }
-    if (result.status === "unsupported") {
-        return { status: "skipped", reason: "platform_unsupported" };
-    }
-    return { status: "unknown", reason: normalizeReason(result.status) };
-}
-/** Reason labels are `[a-z0-9_:.-]` by schema; anything else becomes one word. */
-function normalizeReason(value) {
-    const cleaned = value.trim().toLowerCase().replace(/[^a-z0-9_:.-]+/gu, "_");
-    return cleaned.slice(0, 120) || "unlabelled";
-}
 function errorName(error) {
     return error instanceof Error ? error.name : typeof error;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bli-cockpit/cli",
-  "version": "0.2.67",
+  "version": "0.2.69",
   "private": false,
   "type": "module",
   "bin": {
@@ -28,7 +28,7 @@
   },
   "dependencies": {
     "@bli-cockpit/memory-mcp": "0.1.9",
-    "@bli-cockpit/mcp": "0.1.9",
+    "@bli-cockpit/mcp": "0.1.10",
     "@bli-cockpit/telemetry-core": "0.1.30"
   }
 }