@indigoai-us/hq-cli 5.345.135 → 5.345.137

package/dist/lib/doctor/fix/apply.js

@@ -46,7 +46,7 @@ import { removeStrayGateEntries } from "../stray-gate-entries.js";
 import { deriveRemediation } from "./remediation.js";
 import { botDir } from "../../bot/paths.js";
 import { readBotConfig } from "../../bot/config.js";
-import { buildBotDaemonPaths, rewriteBotDaemonNodePath } from "../../bot/daemon.js";
+import { buildBotDaemonPaths, installedDaemonHqBinary, repairBotDaemonIfStale, rewriteBotDaemonNodePath, } from "../../bot/daemon.js";
 import { classifyRefreshSyncOutcome, } from "./refresh-sync-outcome.js";
 import { readRefreshSyncProgress, writeRefreshSyncCursor, writeRefreshSyncLastRun, } from "./refresh-sync-progress.js";
 import { clientHealthShowManualOnlyEnabled } from "../client-health-reason-codes-flag.js";
@@ -130,6 +130,7 @@ export async function applyFixes(options) {
     const collect = options.runChecks ?? defaultRunChecks;
     const refreshSync = options.refreshSync ?? defaultRefreshSync;
     const rewriteBotNodePath = options.rewriteBotNodePath ?? defaultRewriteBotNodePath;
+    const repairBotHqBinary = options.repairBotHqBinary ?? defaultRepairBotHqBinary;
     // 2. Collect findings, then plan each newly unlocked allowlisted repair.
     //    Identity is deliberately the check id + target pair: an unsuccessful
     //    repair must not be applied repeatedly in one invocation.
@@ -154,7 +155,7 @@ export async function applyFixes(options) {
             const rem = deriveRemediation(result);
             if (!rem?.autoFixable)
                 continue;
-            const plan = planFix(hqRoot, result, rem, refreshSync, rewriteBotNodePath);
+            const plan = planFix(hqRoot, result, rem, refreshSync, rewriteBotNodePath, repairBotHqBinary);
             if (!plan)
                 continue;
             const key = JSON.stringify([plan.checkId, plan.target]);
@@ -421,10 +422,12 @@ export function uncommittedHookConfigChanges(hqRoot) {
 }
 // --- Fix planning --------------------------------------------------------------
 /** Build the concrete plan for one auto-fixable finding, or null if unplannable. */
-function planFix(hqRoot, result, rem, refreshSync, rewriteBotNodePath = defaultRewriteBotNodePath) {
+function planFix(hqRoot, result, rem, refreshSync, rewriteBotNodePath = defaultRewriteBotNodePath, repairBotHqBinary = defaultRepairBotHqBinary) {
     switch (rem.fixClass) {
         case "bot-node-path":
             return planBotNodePath(result, rem, rewriteBotNodePath);
+        case "bot-hq-binary":
+            return planBotHqBinary(result, rem, repairBotHqBinary);
         case "executable-bit":
             return planExecutableBit(hqRoot, result, rem);
         case "gate-profile":
@@ -467,6 +470,63 @@ export function defaultRewriteBotNodePath(name) {
         return null;
     return rewriteBotDaemonNodePath(buildBotDaemonPaths({ name, botDir: dir, hqRoot: config.hqRoot, hqBinary: "hq" }), { platform: "darwin" });
 }
+/** Reinstall one bot LaunchAgent whose hq binary is missing. */
+function planBotHqBinary(result, rem, repair) {
+    const name = rem.fixTarget;
+    if (!name)
+        return null;
+    return {
+        checkId: result.checkId,
+        fixClass: "bot-hq-binary",
+        target: name,
+        relpath: "",
+        preview: `  ~/Library/LaunchAgents/ai.getindigo.hq-bot.${name}.plist\n` +
+            `    ~ hq path -> the installed hq (same as hq bot daemon install ${name})\n` +
+            `      the agent is reloaded; its old hq path is missing, so it was not running`,
+        summary: `reinstall ${name} startup agent against the installed hq`,
+        apply: () => {
+            repair(name);
+        },
+        postStatus: (results) => results.find((entry) => entry.checkId === result.checkId)?.status ?? null,
+    };
+}
+/** Real launchctl for the `bot-hq-binary` reload; a failure is a non-ok result. */
+const doctorLaunchctl = (args) => {
+    const r = spawnSync("launchctl", args, { encoding: "utf8", stdio: ["ignore", "pipe", "pipe"] });
+    return { ok: r.status === 0, stdout: String(r.stdout ?? ""), stderr: String(r.stderr ?? r.error?.message ?? "") };
+};
+/**
+ * An hq path a bot may be pointed at unattended: an installed package entry
+ * or its stable shim, never a worktree or one-off build that happens to be
+ * running `hq doctor --fix`.
+ */
+export function isInstalledHqBinary(hqBinary) {
+    const normalized = hqBinary.replace(/\\/g, "/");
+    if (/\/node_modules\/@indigoai-us\/hq-cli\//i.test(normalized))
+        return true;
+    return /\/bin\/hq(\.cmd)?$/i.test(normalized);
+}
+/** Default `bot-hq-binary` repair against the real bot state and LaunchAgents. */
+export function defaultRepairBotHqBinary(name) {
+    if (process.platform !== "darwin")
+        return null;
+    const dir = botDir(name);
+    const config = readBotConfig(dir);
+    if (!config)
+        return null;
+    const paths = buildBotDaemonPaths({ name, botDir: dir, hqRoot: config.hqRoot });
+    if (!isInstalledHqBinary(paths.hqBinary) || !fs.existsSync(paths.hqBinary))
+        return null;
+    let from;
+    try {
+        from = installedDaemonHqBinary(paths);
+    }
+    catch {
+        from = null;
+    }
+    const repaired = repairBotDaemonIfStale(paths, { launchctl: doctorLaunchctl });
+    return repaired ? { from, to: paths.hqBinary } : null;
+}
 /** Pull one company or the personal vault to refresh a stale journal. */
 function planRefreshSync(hqRoot, result, rem, refreshSync) {
     const slug = rem.fixTarget;

package/dist/lib/doctor/fix/remediation.d.ts

@@ -40,7 +40,7 @@
  */
 import type { CheckResult } from "../types.js";
 /** The allowlisted safe repair classes `--fix` is permitted to apply. */
-export type FixClass = "executable-bit" | "gate-profile" | "remove-stray-gate-entries" | "refresh-sync" | "bot-node-path";
+export type FixClass = "executable-bit" | "gate-profile" | "remove-stray-gate-entries" | "refresh-sync" | "bot-node-path" | "bot-hq-binary";
 /**
  * A finding's structured remediation. `autoFixable`, `action`, and `command`
  * are the `--json` surface (AC1); `fixTarget`/`fixClass` are the internal handle

package/dist/lib/doctor/fix/remediation.js

@@ -119,6 +119,22 @@ export function deriveRemediation(result) {
             command: remediation ?? `hq bot start ${target}`,
         };
     }
+    // bot-hq-binary — a bot LaunchAgent whose hq path no longer exists (an
+    // update moved or removed it), so launchd cannot start the bot. The repair
+    // is what `hq bot daemon install <name>` does: rewrite the agent against
+    // the installed hq and reload it. A job whose hq is missing is not running a
+    // turn, so the reload cuts nothing short (feedback_c441dd9a).
+    const botBinary = /^bots\.(.+)\.daemon-binary$/.exec(checkId);
+    if (botBinary && result.status === "FAIL") {
+        const name = botBinary[1];
+        return {
+            autoFixable: true,
+            fixClass: "bot-hq-binary",
+            fixTarget: name,
+            action: `Reinstall the ${name} startup agent against the installed hq binary and reload it.`,
+            command: remediation ?? `hq bot daemon install ${name}`,
+        };
+    }
     // Available hq-core upgrade. WARN is intentional in `hq doctor` and is
     // not a broken sync; --fix has no mechanical repair for it.
     if (checkId === "sync.update.core") {

package/dist/lib/doctor/shared.d.ts

@@ -1,6 +1,16 @@
 /** Collect every `command` string from a `{hooks:{Event:[{hooks:[…]}]}}` doc. */
 export declare function collectCommands(parsed: unknown): string[];
-export declare function fileState(target: string): {
+/**
+ * Whether a file mode counts as executable for a hook check.
+ *
+ * Windows has no POSIX execute bit: Node reports 0o666 for every regular file
+ * that is not a .exe/.cmd/.bat/.com, and the hook runtimes start HQ's shell
+ * hooks through bash, which never consults it. A mode check there can only
+ * produce false FAILs (feedback_a60339e8, 27 of them on one install), so
+ * win32 treats every present file as executable.
+ */
+export declare function modeIsExecutable(mode: number, platform?: NodeJS.Platform): boolean;
+export declare function fileState(target: string, platform?: NodeJS.Platform): {
     present: boolean;
     executable: boolean;
 };

package/dist/lib/doctor/shared.js

@@ -23,10 +23,24 @@ export function collectCommands(parsed) {
     }
     return commands;
 }
-export function fileState(target) {
+/**
+ * Whether a file mode counts as executable for a hook check.
+ *
+ * Windows has no POSIX execute bit: Node reports 0o666 for every regular file
+ * that is not a .exe/.cmd/.bat/.com, and the hook runtimes start HQ's shell
+ * hooks through bash, which never consults it. A mode check there can only
+ * produce false FAILs (feedback_a60339e8, 27 of them on one install), so
+ * win32 treats every present file as executable.
+ */
+export function modeIsExecutable(mode, platform = process.platform) {
+    if (platform === "win32")
+        return true;
+    return (mode & 0o111) !== 0;
+}
+export function fileState(target, platform = process.platform) {
     try {
         const stat = fs.statSync(target);
-        return { present: stat.isFile(), executable: (stat.mode & 0o111) !== 0 };
+        return { present: stat.isFile(), executable: modeIsExecutable(stat.mode, platform) };
     }
     catch {
         return { present: false, executable: false };