@bli-cockpit/cli 0.2.20 → 0.2.22

package/dist/autostart.js

@@ -8,6 +8,21 @@ export const AUTOSTART_LABEL = "com.bli.cockpit.sync";
 export const WINDOWS_AUTOSTART_TASK_NAME = "BLI Cockpit Sync";
 const WINDOWS_AUTOSTART_SCRIPT_NAME = "autostart-sync.ps1";
 const WINDOWS_AUTOSTART_REGISTRATION_SCRIPT_NAME = "autostart-register.ps1";
+/**
+ * Windowless launcher for the scheduled sync (BLI-2677). Task Scheduler ran
+ * powershell.exe directly under an interactive token, and PowerShell is a
+ * console-subsystem binary, so every 15-minute tick flashed a conhost window
+ * over whatever the operator was doing \u2014 on every Windows machine in the
+ * fleet. wscript.exe is a GUI-subsystem host, so it never allocates a console,
+ * and it starts the PowerShell child with window style 0 (hidden). S4U was
+ * considered and rejected: it breaks on Microsoft-account logons and the task
+ * deliberately stays on InteractiveToken with least privilege.
+ */
+const WINDOWS_AUTOSTART_LAUNCHER_NAME = "autostart-sync.vbs";
+const WINDOWS_SYNC_LOG_NAME = "sync.log";
+/** With the window hidden, the log is the only place output goes; cap it so an
+ * always-on machine syncing every 15 minutes cannot grow it without bound. */
+const WINDOWS_SYNC_LOG_LIMIT_BYTES = 10 * 1024 * 1024;
 const UTF8_BOM = "\uFEFF";
 export const DEFAULT_AUTOSTART_INTERVAL_SECONDS = 15 * 60;
 /**
@@ -179,6 +194,7 @@ async function installWindowsTask(options) {
     const intervalMinutes = Math.max(1, Math.ceil(intervalSeconds / 60));
     const scriptPath = path.join(getCollectorRuntimePaths(homeDir).state_dir, WINDOWS_AUTOSTART_SCRIPT_NAME);
     const registrationPath = path.join(getCollectorRuntimePaths(homeDir).state_dir, WINDOWS_AUTOSTART_REGISTRATION_SCRIPT_NAME);
+    const launcherPath = windowsAutostartLauncherPath(homeDir);
     const nodeExecutable = path.win32.resolve(options.nodeExecutable ?? process.execPath);
     const cliEntryPoint = path.win32.resolve(options.cliEntryPoint ?? process.argv[1] ?? "");
     await mkdir(path.dirname(scriptPath), { recursive: true });
@@ -188,10 +204,18 @@ async function installWindowsTask(options) {
         dashboardUrl,
         nodeExecutable,
         cliEntryPoint,
+        syncLogPath: windowsSyncLogPath(homeDir),
     })}`, "utf8");
+    // No BOM on the launcher: wscript.exe reads .vbs files as ANSI and treats a
+    // UTF-8 BOM as an invalid character. The rendered content is ASCII-safe for
+    // every path this fleet has.
+    await writeFile(launcherPath, renderWindowsLauncherScript({
+        powershellPath: windowsPowerShellPath(),
+        scriptPath,
+    }), "utf8");
     await writeFile(registrationPath, `${UTF8_BOM}${renderWindowsRegistrationScript({
         taskName: WINDOWS_AUTOSTART_TASK_NAME,
-        scriptPath,
+        launcherPath,
         intervalMinutes,
     })}`, "utf8");
     const created = await options.exec("powershell.exe", [
@@ -240,10 +264,12 @@ async function installWindowsTask(options) {
 async function uninstallWindowsTask(options) {
     const scriptPath = windowsAutostartScriptPath(options.homeDir);
     const registrationPath = windowsAutostartRegistrationScriptPath(options.homeDir);
+    const launcherPath = windowsAutostartLauncherPath(options.homeDir);
     const current = await windowsTaskStatus(options);
     if (current.status === "absent") {
         await rm(scriptPath, { force: true });
         await rm(registrationPath, { force: true });
+        await rm(launcherPath, { force: true });
         return {
             ...current,
             plist_path: scriptPath,
@@ -259,6 +285,7 @@ async function uninstallWindowsTask(options) {
     if (removed.code === 0) {
         await rm(scriptPath, { force: true });
         await rm(registrationPath, { force: true });
+        await rm(launcherPath, { force: true });
     }
     return {
         status: removed.code === 0 ? "uninstalled" : "not_loaded",
@@ -275,6 +302,7 @@ async function uninstallWindowsTask(options) {
 }
 async function windowsTaskStatus(options) {
     const scriptPath = windowsAutostartScriptPath(options.homeDir);
+    const launcherPath = windowsAutostartLauncherPath(options.homeDir);
     const query = await options.exec("schtasks.exe", [
         "/Query",
         "/TN",
@@ -304,9 +332,9 @@ async function windowsTaskStatus(options) {
     const intervalSeconds = options.intervalSeconds ?? DEFAULT_AUTOSTART_INTERVAL_SECONDS;
     const intervalMinutes = Math.max(1, Math.ceil(intervalSeconds / 60));
     const registrationProblems = windowsTaskRegistrationProblems(query.stdout, {
-        scriptPath,
+        launcherPath,
         intervalMinutes,
-        powershellPath: windowsPowerShellPath(),
+        wscriptPath: windowsWScriptPath(),
     });
     if (disabled)
         registrationProblems.unshift("task is disabled");
@@ -321,12 +349,32 @@ async function windowsTaskStatus(options) {
             dashboardUrl: options.dashboardUrl ?? DEFAULT_DASHBOARD_URL,
             nodeExecutable: path.win32.resolve(options.nodeExecutable ?? process.execPath),
             cliEntryPoint: path.win32.resolve(options.cliEntryPoint ?? process.argv[1] ?? ""),
+            syncLogPath: windowsSyncLogPath(options.homeDir),
         })}`;
         const currentScript = await readFile(scriptPath, "utf8").catch(() => null);
         if (currentScript !== expectedScript) {
             registrationProblems.push("sync script does not match the current roots or Cockpit runtime");
         }
     }
+    // Same shape as the sync-script checks: existence always, content only when
+    // the caller supplied roots (a bare `autostart status` stays permissive by
+    // design — see BLI-2362). The launcher's content depends on nothing but the
+    // PowerShell path and the sync-script path, but gating keeps the two
+    // commands' disagreement surface identical to the script check above.
+    const launcherExists = await fileExists(launcherPath);
+    if (!launcherExists) {
+        registrationProblems.push("sync launcher is missing");
+    }
+    else if (options.repoRoots && options.repoRoots.length > 0) {
+        const expectedLauncher = renderWindowsLauncherScript({
+            powershellPath: windowsPowerShellPath(),
+            scriptPath,
+        });
+        const currentLauncher = await readFile(launcherPath, "utf8").catch(() => null);
+        if (currentLauncher !== expectedLauncher) {
+            registrationProblems.push("sync launcher does not match the current Cockpit runtime");
+        }
+    }
     // Both branches log. A line that only fires on failure cannot answer "did
     // background collection validate at all today?", which is the question that
     // would have caught a validator rejecting every healthy Windows task.
@@ -362,12 +410,23 @@ function windowsAutostartScriptPath(homeDir) {
 function windowsAutostartRegistrationScriptPath(homeDir) {
     return path.join(getCollectorRuntimePaths(homeDir ?? os.homedir()).state_dir, WINDOWS_AUTOSTART_REGISTRATION_SCRIPT_NAME);
 }
+function windowsAutostartLauncherPath(homeDir) {
+    return path.join(getCollectorRuntimePaths(homeDir ?? os.homedir()).state_dir, WINDOWS_AUTOSTART_LAUNCHER_NAME);
+}
+function windowsSyncLogPath(homeDir) {
+    return path.join(getCollectorRuntimePaths(homeDir ?? os.homedir()).state_dir, WINDOWS_SYNC_LOG_NAME);
+}
 function renderWindowsRegistrationScript(options) {
     return [
         "$ErrorActionPreference = 'Stop'",
         `$taskName = ${powershellLiteral(options.taskName)}`,
-        `$syncScriptPath = ${powershellLiteral(options.scriptPath)}`,
-        "$powershellPath = Join-Path $PSHOME 'powershell.exe'",
+        `$launcherPath = ${powershellLiteral(options.launcherPath)}`,
+        // wscript.exe is the windowless script host: the task runs the .vbs
+        // launcher, which starts PowerShell with its window hidden, so the
+        // 15-minute sync never flashes a terminal (BLI-2677). //B is batch mode —
+        // a script error becomes a nonzero exit instead of a modal dialog that
+        // would hang the scheduled run.
+        "$wscriptPath = Join-Path (Join-Path $env:SystemRoot 'System32') 'wscript.exe'",
         // Windows PowerShell 5.1 passes a native argument's embedded quotes to
         // CommandLineToArgvW unescaped, so a /TR value with bare quotes falls out
         // of the quoted region at the first space inside a quoted path:
@@ -376,8 +435,8 @@ function renderWindowsRegistrationScript(options) {
         // Escaping the embedded quotes as \" keeps the whole /TR value one
         // argument; Task Scheduler still normalizes what it stores (BLI-2541),
         // which the read-back validator already compares path-wise.
-        "$actionArgs = '-NoProfile -NonInteractive -ExecutionPolicy Bypass -File \\\"' + $syncScriptPath + '\\\"'",
-        "$taskCommand = '\\\"' + $powershellPath + '\\\" ' + $actionArgs",
+        "$actionArgs = '//B \\\"' + $launcherPath + '\\\"'",
+        "$taskCommand = '\\\"' + $wscriptPath + '\\\" ' + $actionArgs",
         `& schtasks.exe /Create /TN $taskName /TR $taskCommand /SC MINUTE /MO ${options.intervalMinutes} /IT /RL LIMITED /F | Out-Null`,
         "if ($LASTEXITCODE -ne 0) { throw \"schtasks /Create exited $LASTEXITCODE\" }",
         "$settings = New-ScheduledTaskSettingsSet -AllowStartIfOnBatteries -DontStopIfGoingOnBatteries -StartWhenAvailable -MultipleInstances IgnoreNew",
@@ -385,6 +444,30 @@ function renderWindowsRegistrationScript(options) {
         "",
     ].join("\r\n");
 }
+/**
+ * The .vbs launcher the scheduled task actually runs. WshShell.Run with window
+ * style 0 starts PowerShell with its console hidden — no flash, no focus
+ * steal — and waiting on the child propagates the sync exit code back to Task
+ * Scheduler as LastTaskResult. VBScript escapes a quote inside a string by
+ * doubling it; Windows paths cannot contain quotes, so the doubled quotes here
+ * are only the fixed delimiters around the two paths.
+ */
+function renderWindowsLauncherScript(options) {
+    const quotedPowershell = `""${options.powershellPath}""`;
+    const quotedScript = `""${options.scriptPath}""`;
+    return [
+        "' BLI Cockpit sync launcher (BLI-2677). wscript.exe never allocates a",
+        "' console and starts PowerShell hidden, so the 15-minute background sync",
+        "' does not flash a terminal over the operator. Sync output lands in",
+        "' sync.log next to this file instead of a visible window.",
+        "Option Explicit",
+        "Dim windowsShell, syncExitCode",
+        'Set windowsShell = CreateObject("WScript.Shell")',
+        `syncExitCode = windowsShell.Run("${quotedPowershell} -NoProfile -NonInteractive -ExecutionPolicy Bypass -File ${quotedScript}", 0, True)`,
+        "WScript.Quit syncExitCode",
+        "",
+    ].join("\r\n");
+}
 function renderWindowsSyncScript(options) {
     const dashboardArgs = options.dashboardUrl === DEFAULT_DASHBOARD_URL
         ? ""
@@ -392,18 +475,29 @@ function renderWindowsSyncScript(options) {
     const discoveryArgs = options.discoveryArgs.length > 0 ? ` ${options.discoveryArgs.join(" ")}` : "";
     const commands = options.workDirs.flatMap((root) => [
         "try {",
-        `  & $nodeExecutable $cliEntryPoint sync --workspace ${powershellLiteral(root)}${dashboardArgs}${discoveryArgs} --json`,
+        `  & $nodeExecutable $cliEntryPoint sync --workspace ${powershellLiteral(root)}${dashboardArgs}${discoveryArgs} --json 2>&1 | ForEach-Object { "$_" } | Add-Content -LiteralPath $syncLogPath -Encoding UTF8`,
         "  if ($LASTEXITCODE -ne 0) { $exitCode = $LASTEXITCODE }",
         "} catch {",
-        "  [Console]::Error.WriteLine($_.Exception.Message)",
+        "  Add-Content -LiteralPath $syncLogPath -Encoding UTF8 -Value $_.Exception.Message",
         "  $exitCode = 1",
         "}",
     ]);
     return [
-        "$ErrorActionPreference = 'Stop'",
+        // 'Continue', not 'Stop': the collector logs to stderr on success as well
+        // as failure (that is the repo's logging contract), and Windows PowerShell
+        // 5.1 under 'Stop' turns the first redirected stderr line of a native
+        // command into a terminating NativeCommandError — 'Stop' would kill every
+        // healthy run at its first log line. Failures are tracked by exit code.
+        "$ErrorActionPreference = 'Continue'",
         "$exitCode = 0",
         `$nodeExecutable = ${powershellLiteral(options.nodeExecutable)}`,
         `$cliEntryPoint = ${powershellLiteral(options.cliEntryPoint)}`,
+        // The task window is hidden (BLI-2677), so this log is the only place the
+        // sync's output exists; sync.err.log stays macOS-only because launchd does
+        // stream separation for free and PowerShell 5.1 does not.
+        `$syncLogPath = ${powershellLiteral(options.syncLogPath)}`,
+        `$syncLogFile = Get-Item -LiteralPath $syncLogPath -ErrorAction SilentlyContinue`,
+        `if ($syncLogFile -and $syncLogFile.Length -gt ${WINDOWS_SYNC_LOG_LIMIT_BYTES}) { Set-Content -LiteralPath $syncLogPath -Value '' -Encoding UTF8 }`,
         ...commands,
         "exit $exitCode",
         "",
@@ -462,7 +556,7 @@ function windowsTaskRegistrationProblems(taskXml, expected) {
     const actionArguments = exactExecBlock
         .match(/<Arguments>\s*([^<]*?)\s*<\/Arguments>/iu)?.[1]
         ?.trim() ?? "";
-    const expectedArgumentFlags = "-NoProfile -NonInteractive -ExecutionPolicy Bypass -File";
+    const expectedArgumentFlags = "//B";
     const requirements = [
         [
             /<DisallowStartIfOnBatteries>\s*false\s*<\/DisallowStartIfOnBatteries>/iu,
@@ -531,10 +625,10 @@ function windowsTaskRegistrationProblems(taskXml, expected) {
     if (!exactExecBlock) {
         problems.push("task must contain exactly one Exec action");
     }
-    if (!sameWindowsPath(command, expected.powershellPath)) {
-        problems.push("task action does not run the expected Windows PowerShell");
+    if (!sameWindowsPath(command, expected.wscriptPath)) {
+        problems.push("task action does not run the expected windowless script host");
     }
-    const actionArgumentProblem = windowsActionArgumentProblem(actionArguments, expectedArgumentFlags, expected.scriptPath);
+    const actionArgumentProblem = windowsActionArgumentProblem(actionArguments, expectedArgumentFlags, expected.launcherPath);
     if (actionArgumentProblem) {
         problems.push(actionArgumentProblem);
     }
@@ -547,30 +641,34 @@ function windowsPowerShellPath(env = process.env) {
     const windowsRoot = env.SystemRoot ?? env.WINDIR ?? "C:\\Windows";
     return path.win32.join(windowsRoot, "System32", "WindowsPowerShell", "v1.0", "powershell.exe");
 }
+function windowsWScriptPath(env = process.env) {
+    const windowsRoot = env.SystemRoot ?? env.WINDIR ?? "C:\\Windows";
+    return path.win32.join(windowsRoot, "System32", "wscript.exe");
+}
 /**
  * Task Scheduler stores a normalized form of the action it was given, not the
- * string we passed: `schtasks /Create /TR "<powershell> <args>"` is split into
+ * string we passed: `schtasks /Create /TR "<host> <args>"` is split into
  * <Command> plus <Arguments>, and the quotes around a space-free script path
  * are dropped. Comparing <Arguments> against the exact string we built
  * therefore fails on a task that is registered perfectly — observed on
  * DESKTOP-G2UO1GK (CLI 0.2.13, 2026-08-12), where the stored value differed
- * from the expected one only by those quotes.
+ * from the expected one only by those quotes (BLI-2541).
  *
- * Compare the flags exactly, because those are ours, and compare the script as
- * a path, because that is what Windows normalizes. Each failure returns its own
- * reason so a repair message says which half is wrong rather than "arguments
- * differ".
+ * Compare the flags exactly, because those are ours, and compare the launcher
+ * as a path, because that is what Windows normalizes. Each failure returns its
+ * own reason so a repair message says which half is wrong rather than
+ * "arguments differ".
  */
-function windowsActionArgumentProblem(actionArguments, expectedFlags, expectedScriptPath) {
+function windowsActionArgumentProblem(actionArguments, expectedFlags, expectedLauncherPath) {
     if (!actionArguments.startsWith(expectedFlags)) {
-        return "task action does not run PowerShell with the expected Cockpit sync flags";
+        return "task action does not run the launcher with the expected batch-mode flag";
     }
-    const scriptArgument = unquoteWindowsArgument(actionArguments.slice(expectedFlags.length));
-    if (!scriptArgument) {
-        return "task action names no sync script to run";
+    const launcherArgument = unquoteWindowsArgument(actionArguments.slice(expectedFlags.length));
+    if (!launcherArgument) {
+        return "task action names no sync launcher to run";
     }
-    if (!sameWindowsPath(scriptArgument, expectedScriptPath)) {
-        return "task action runs a script other than the Cockpit sync script";
+    if (!sameWindowsPath(launcherArgument, expectedLauncherPath)) {
+        return "task action runs a script other than the Cockpit sync launcher";
     }
     return null;
 }

package/dist/commands/local.js

@@ -21,6 +21,7 @@ import { resolveDiscoveryLimits, saveDiscoveryLimits, } from "../discovery-limit
 import { runAttributedWorktreeSync, matchesLiveSyncWorktree, } from "./session-sync.js";
 import { COLLECTION_ROOT_REQUIRED, missingCollectionRootMessage, normalizeRootsDetailed, resolveOnboardingRoots, rootRejectionExplanation, } from "../onboarding-roots.js";
 import { rawEvidenceGcSummary, runRawEvidenceLocalGc, } from "../raw-evidence-gc.js";
+import { envWithNodeRuntimeOnPath, runScheduledSelfUpdate, } from "../scheduled-self-update.js";
 import { enqueueInstallEventEntry, readPendingInstallEventEntries, recordInstallEventAttemptFailure, removeInstallEventEntry, } from "../spool/install-event-outbox.js";
 import { createCapturedExecRunner, createInteractiveExecRunner, } from "../process-runner.js";
 import { normalizeCollectionRoots } from "../root-normalization.js";
@@ -250,6 +251,9 @@ function localSubcommandHelp(command) {
                 "Newly discovered repos get a general ambient work context automatically.",
                 "Discovery scans 3 folder levels and up to 50 repos by default; tune with",
                 "--max-depth and --max-repos.",
+                "Also self-updates the CLI from npm latest once per day, strictly after",
+                "collection finishes; set COCKPIT_DISABLE_AUTO_UPDATE=1 to freeze the",
+                "installed version during incident triage.",
             ],
         ],
         [
@@ -2149,6 +2153,9 @@ async function runSync(command, io) {
             json: command.json,
             io,
         });
+        // BLI-2601: self-update runs only after collection's own outcome above is
+        // already decided and reported, win or lose. See the function doc.
+        await runScheduledSelfUpdateAfterSync(command, io, dashboardUrl);
         return result.exitCode;
     }
     catch (error) {
@@ -2167,9 +2174,107 @@ async function runSync(command, io) {
             json: command.json,
             io,
         });
+        await runScheduledSelfUpdateAfterSync(command, io, dashboardUrl);
         throw error;
     }
 }
+/**
+ * BLI-2601: the fleet keeps itself current on npm `latest` without anyone
+ * re-running `npm i -g @bli-cockpit/cli` by hand after day 0. This always
+ * runs AFTER `runSync` has already decided and reported collection's own
+ * outcome above — a stuck or failing self-update can never block or delay
+ * collection, and a collection failure never blocks the chance to
+ * self-update. Every error path here is swallowed on purpose: a failure is
+ * reported as its own named `update` receipt, never surfaced as a `sync`
+ * failure or thrown from this function.
+ */
+async function runScheduledSelfUpdateAfterSync(command, io, dashboardUrl) {
+    let event;
+    try {
+        event = await runScheduledSelfUpdateForSync(command, io);
+    }
+    catch (error) {
+        // The throttle/probe/install machinery below is defensive already; this
+        // is the last-resort net so an update crash truly cannot touch the sync
+        // result above.
+        event = {
+            step: "update",
+            status: "fail",
+            error_code: "self_update_threw",
+            error_detail: redactedSyncErrorDetail(error),
+        };
+    }
+    if (!event)
+        return;
+    await reportInstallEventsBestEffort({
+        homeDir: command.homeDir,
+        dashboardUrl,
+        command: "update",
+        events: [event],
+        json: command.json,
+        io,
+    });
+}
+async function runScheduledSelfUpdateForSync(command, io) {
+    const rawExec = io.exec;
+    if (!rawExec) {
+        // Only the real production `defaultIo()` supplies a process runner. A
+        // caller that omitted one gets a silent no-op rather than this reaching
+        // for a real npm binary it was never given — never observed in
+        // production, where `defaultIo()` always sets `exec`.
+        return null;
+    }
+    // Every spawn in the scheduled path carries the running node's bin dir on
+    // PATH — see envWithNodeRuntimeOnPath. Interactive doctor never needed
+    // this; the scheduler's stripped environment does.
+    const spawnEnv = envWithNodeRuntimeOnPath(io.env ?? process.env);
+    const exec = (cmd, args, options) => rawExec(cmd, args, { ...options, env: options?.env ?? spawnEnv });
+    const scheduledIo = { ...io, exec };
+    const paths = getCollectorRuntimePaths(command.homeDir);
+    const result = await runScheduledSelfUpdate(paths, {
+        exec,
+        currentVersion: LOCAL_COLLECTOR_VERSION,
+        install: (tag) => attemptScheduledSelfUpdateInstall(scheduledIo, tag),
+    }, { env: io.env });
+    return scheduledSelfUpdateInstallEvent(result);
+}
+async function attemptScheduledSelfUpdateInstall(io, tag) {
+    try {
+        // Reuses the exact npm-install machinery `cockpit doctor`'s
+        // `fixCliLatest` uses (see doctor.ts:243-280) so there is one place that
+        // knows how to invoke `npm i -g` and classify EACCES. Unlike doctor,
+        // this call never re-execs — see runScheduledSelfUpdate's doc comment.
+        await runSelfUpdate(io, { json: true, tag });
+        return { ok: true };
+    }
+    catch (error) {
+        if (!(error instanceof SelfUpdateError))
+            throw error;
+        return { ok: false, eacces: error.eacces };
+    }
+}
+function scheduledSelfUpdateInstallEvent(result) {
+    // The steady-state "already checked today" case is a pure no-op; reporting
+    // it would post a receipt on ~95 of every 96 sync ticks for no new
+    // information. Only a real attempt (ok, fail, or an explicit disable)
+    // produces a receipt.
+    if (result.reason === "throttled_recent_attempt")
+        return null;
+    if (result.status === "ok")
+        return { step: "update", status: "ok" };
+    const detail = [
+        result.target_version ? `target ${result.target_version}` : null,
+        result.installed_version ? `installed ${result.installed_version}` : null,
+    ]
+        .filter((part) => Boolean(part))
+        .join("; ");
+    return {
+        step: "update",
+        status: result.status,
+        error_code: result.reason,
+        ...(detail ? { error_detail: detail } : {}),
+    };
+}
 async function runSyncWithHealthReceipt(command, io) {
     const backfillLock = await inspectBackfillLock(getCollectorRuntimePaths(command.homeDir));
     if (backfillLock.held) {

package/dist/commands/public-root.js

@@ -15,7 +15,7 @@ export async function runCockpitCli(argv, io) {
   }
 
   if (command === "--version" || command === "-V" || command === "version") {
-    writeLine(io?.stdout ?? process.stdout, "0.2.20");
+    writeLine(io?.stdout ?? process.stdout, "0.2.22");
     return 0;
   }
 

package/dist/scheduled-self-update.js

@@ -0,0 +1,136 @@
+import fs from "node:fs/promises";
+import path from "node:path";
+const SELF_UPDATE_MIN_INTERVAL_MS = 24 * 60 * 60 * 1000;
+export const SELF_UPDATE_THROTTLE_MARKER = ".last-self-update-check";
+/**
+ * launchd starts the scheduled tick with a bare PATH (`/usr/bin:/bin:...`) —
+ * the plist hardcodes absolute node/CLI paths for exactly that reason — so a
+ * bare `npm` spawn that works in every interactive shell resolves to nothing
+ * inside the tick, and every scheduled update would fail `npm_install_failed`
+ * on the whole Mac fleet while interactive `cockpit doctor` kept working. The
+ * npm shim ships beside the node binary in the standard layouts on both host
+ * families, so the running node's own directory is the one PATH entry that is
+ * always right.
+ */
+export function envWithNodeRuntimeOnPath(env, nodeExecutable = process.execPath) {
+    const nodeBinDir = path.dirname(nodeExecutable);
+    const currentPath = env["PATH"] ?? "";
+    if (currentPath.split(path.delimiter).includes(nodeBinDir))
+        return env;
+    return {
+        ...env,
+        PATH: [nodeBinDir, currentPath].filter(Boolean).join(path.delimiter),
+    };
+}
+/**
+ * BLI-2601. Runs at most once per day from inside `cockpit sync`'s
+ * post-collection tail, so the fleet converges on npm `latest` without
+ * anyone re-running `npm i -g @bli-cockpit/cli` by hand after day 0.
+ *
+ * Deliberately does not re-exec: this process already has the OLD code
+ * loaded in memory, so nothing in-process could prove a re-exec actually
+ * picked up the new build. The next scheduled tick launches a fresh
+ * `cockpit` process from disk and picks up whatever npm actually installed —
+ * that is the verification, not a re-exec here.
+ */
+export async function runScheduledSelfUpdate(paths, deps, options = {}) {
+    const env = options.env ?? process.env;
+    const now = options.now ?? new Date();
+    const tag = options.tag ?? "latest";
+    const marker = path.join(paths.state_dir, SELF_UPDATE_THROTTLE_MARKER);
+    const lastCheck = await fs.stat(marker).catch(() => null);
+    if (lastCheck && now.getTime() - lastCheck.mtimeMs < SELF_UPDATE_MIN_INTERVAL_MS) {
+        // The steady-state case: already checked today, nothing to do. Not
+        // reported as an "attempt" by the caller — this fires on ~95 of every 96
+        // ticks and would otherwise be pure noise.
+        return { status: "skipped", reason: "throttled_recent_attempt" };
+    }
+    // The marker is written for the attempt, not the outcome — same idiom as
+    // raw-evidence GC (raw-evidence-gc.ts). A machine stuck on a permissions
+    // error must not spend every 15-min tick re-hitting the npm registry.
+    await fs.mkdir(paths.state_dir, { recursive: true }).catch(() => undefined);
+    await fs.writeFile(marker, now.toISOString()).catch(() => undefined);
+    // Incident triage freeze: an operator can pin a machine's installed
+    // version while debugging without touching the scheduler itself. Mirrors
+    // COCKPIT_DISABLE_GC. Checked after the throttle write on purpose, so a
+    // disabled machine reports itself at most once per day too, instead of on
+    // every tick.
+    if (env["COCKPIT_DISABLE_AUTO_UPDATE"] === "1") {
+        return { status: "skipped", reason: "disabled_by_env" };
+    }
+    const targetVersion = await latestCliVersionFromNpm(deps.exec, tag);
+    if (targetVersion && targetVersion === deps.currentVersion) {
+        return { status: "ok", reason: "already_latest", target_version: targetVersion };
+    }
+    const installed = await deps.install(tag);
+    if (!installed.ok) {
+        return {
+            status: "fail",
+            reason: installed.eacces ? "eacces_needs_chown" : "npm_install_failed",
+            ...(targetVersion ? { target_version: targetVersion } : {}),
+        };
+    }
+    // Verify against what the platform returns, not the exit code npm handed
+    // back (BLI-2541 lesson: a green exit code is not proof). Read the
+    // globally installed package's version back off disk via `npm ls -g
+    // --json` rather than trusting that install succeeding means the version
+    // this process expected is what is actually there.
+    const installedVersion = await probeInstalledCliVersion(deps.exec);
+    if (!installedVersion || (targetVersion && installedVersion !== targetVersion)) {
+        return {
+            status: "fail",
+            reason: "stale_after_self_update",
+            ...(targetVersion ? { target_version: targetVersion } : {}),
+            ...(installedVersion ? { installed_version: installedVersion } : {}),
+        };
+    }
+    return {
+        status: "ok",
+        reason: "updated",
+        target_version: targetVersion ?? installedVersion,
+        installed_version: installedVersion,
+    };
+}
+async function latestCliVersionFromNpm(exec, tag) {
+    const result = await exec("npm", [
+        "view",
+        `@bli-cockpit/cli@${tag}`,
+        "version",
+        "--json",
+    ]);
+    if (result.code !== 0)
+        return null;
+    return parseNpmVersionField(result.stdout);
+}
+async function probeInstalledCliVersion(exec) {
+    // `npm ls` can exit non-zero on unrelated extraneous/peer-dependency
+    // warnings even when the requested package resolved cleanly, so the JSON
+    // body is parsed regardless of exit code — the printed tree is the ground
+    // truth here, not the exit code.
+    const result = await exec("npm", ["ls", "-g", "@bli-cockpit/cli", "--json"]);
+    try {
+        const parsed = JSON.parse(result.stdout);
+        const version = parsed.dependencies?.["@bli-cockpit/cli"]?.version;
+        return typeof version === "string" && version.trim() ? version.trim() : null;
+    }
+    catch {
+        return null;
+    }
+}
+// Small, intentional duplication of doctor.ts's `parseNpmVersion`: importing
+// it here would create a cycle (doctor.ts already imports types from
+// commands/local.ts, which would need to import this file to wire the sync
+// integration). The parsing itself is a few lines and has no behavior this
+// module doesn't already own.
+function parseNpmVersionField(stdout) {
+    const trimmed = stdout.trim();
+    if (!trimmed)
+        return null;
+    try {
+        const parsed = JSON.parse(trimmed);
+        return typeof parsed === "string" && parsed.trim() ? parsed.trim() : null;
+    }
+    catch {
+        return trimmed.replace(/^"|"$/gu, "") || null;
+    }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bli-cockpit/cli",
-  "version": "0.2.20",
+  "version": "0.2.22",
   "private": false,
   "type": "module",
   "bin": {