@officexapp/vidfarm-devcli 0.21.55 → 0.21.56

package/dist/src/devcli/doctor.js

@@ -7,7 +7,7 @@
 // else prints as ⚠/✗ but does not fail the command, so agents can run doctor
 // unconditionally at session start.
 import { spawnSync } from "node:child_process";
-import { existsSync, readdirSync } from "node:fs";
+import { existsSync, readdirSync, readFileSync } from "node:fs";
 import { createRequire } from "node:module";
 import net from "node:net";
 import { homedir } from "node:os";
@@ -143,6 +143,28 @@ export async function runDoctorCommand(argv) {
     const killOrphans = Boolean(parsed.values["kill-orphans"]);
     const checks = [];
     const add = (name, level, detail) => checks.push({ name, level, detail });
+    // 0. Are the two halves current, and are they the SAME version? The skill and
+    // the devcli ship on one semver, and a gap between them is the usual cause of
+    // "that command doesn't exist" — so it belongs in the health check, not only
+    // in the update runbook. Never fails the doctor: offline is not a defect, and
+    // a pending update is information, not breakage.
+    try {
+        const { installedDevcliVersion, installedSkillPath, readSkillVersion, compareSemver, readUpdateState, hoursSinceLastCheck } = await import("./update-check.js");
+        const devcli = installedDevcliVersion();
+        const skillPath = installedSkillPath();
+        const skill = skillPath ? readSkillVersion(readFileSync(skillPath, "utf8")) : null;
+        const since = hoursSinceLastCheck(readUpdateState());
+        const staleNote = since === null ? " · never checked for updates" : since >= 24 ? ` · last update check ${Math.round(since / 24)}d ago` : "";
+        if (devcli && skill && compareSemver(devcli, skill) !== 0) {
+            add("versions", "warn", `devcli ${devcli} vs skill ${skill} — MISMATCHED. They ship together; run: vidfarm update-check`);
+        }
+        else {
+            add("versions", since !== null && since < 24 ? "ok" : "warn", `devcli ${devcli ?? "?"} · skill ${skill ?? "not installed"}${staleNote}${staleNote ? " — vidfarm update-check" : ""}`);
+        }
+    }
+    catch {
+        // A version probe must never take the doctor down with it.
+    }
     // 1. Node version.
     const nodeMajor = Number(process.versions.node.split(".")[0]);
     add("node", nodeMajor >= 22 ? "ok" : "fail", `v${process.versions.node}${nodeMajor >= 22 ? "" : " — vidfarm-devcli needs Node >= 22"}`);