@indigoai-us/hq-cli 5.59.0 → 5.61.0

package/src/utils/version-check.ts

@@ -8,7 +8,9 @@ import { CLI_VERSION } from "../cli-version.js";
 const PACKAGE_NAME = "@indigoai-us/hq-cli";
 const REGISTRY_URL = `https://registry.npmjs.org/${encodeURIComponent(PACKAGE_NAME)}/latest`;
 const CACHE_TTL_MS = 24 * 60 * 60 * 1000;
+const CACHE_TTL_JITTER_MS = 60 * 60 * 1000;
 const FETCH_TIMEOUT_MS = 3_000;
+const REFRESH_LOCK_STALE_MS = 10 * 60 * 1000;
 
 interface CacheEntry {
   latest: string;
@@ -19,6 +21,10 @@ function cachePath(): string {
   return path.join(os.homedir(), ".hq", "version-check.json");
 }
 
+function lockPath(): string {
+  return path.join(os.homedir(), ".hq", "version-check.lock");
+}
+
 function isOptedOut(): boolean {
   return process.env.HQ_NO_UPDATE_CHECK === "1";
 }
@@ -49,6 +55,58 @@ function writeCache(entry: CacheEntry): void {
   }
 }
 
+function freshEnough(entry: CacheEntry, now = Date.now()): boolean {
+  const jitter = Math.floor(Math.random() * CACHE_TTL_JITTER_MS);
+  return now - entry.fetchedAt <= CACHE_TTL_MS - jitter;
+}
+
+function isKnownNoninteractiveStatusProbe(argv: readonly string[] = process.argv): boolean {
+  const args = argv.slice(2);
+  const positional = args.filter((arg) => !arg.startsWith("-"));
+  const json = args.includes("--json") || !process.stdout.isTTY;
+  if (!json) return false;
+  if (positional[0] === "mcp" && positional[1] === "status") return true;
+  if (positional[0] === "packs" && (positional[1] === "list" || positional[1] === "ls")) return true;
+  if (
+    positional[0] === "packages" &&
+    positional[1] === "packs" &&
+    (positional[2] === "list" || positional[2] === "ls")
+  ) {
+    return true;
+  }
+  if ((positional[0] === "sources" || positional[0] === "signals") && positional[1] === "list") {
+    return true;
+  }
+  return false;
+}
+
+function acquireRefreshLock(now = Date.now()): (() => void) | null {
+  const dir = lockPath();
+  try {
+    fs.mkdirSync(path.dirname(dir), { recursive: true });
+    fs.mkdirSync(dir);
+    fs.writeFileSync(path.join(dir, "owner"), `${process.pid}\n${now}\n`);
+    return () => {
+      try {
+        fs.rmSync(dir, { recursive: true, force: true });
+      } catch {
+        // best-effort lock cleanup
+      }
+    };
+  } catch {
+    try {
+      const stat = fs.statSync(dir);
+      if (now - stat.mtimeMs > REFRESH_LOCK_STALE_MS) {
+        fs.rmSync(dir, { recursive: true, force: true });
+        return acquireRefreshLock(now);
+      }
+    } catch {
+      // ignore lock inspection failures
+    }
+    return null;
+  }
+}
+
 export function maybeWarnNewVersion(): void {
   if (isOptedOut()) return;
   const entry = readCache();
@@ -68,7 +126,14 @@ export function maybeWarnNewVersion(): void {
 
 export async function refreshVersionCache(): Promise<void> {
   if (isOptedOut()) return;
+  if (isKnownNoninteractiveStatusProbe()) return;
+  const existing = readCache();
+  if (existing && freshEnough(existing)) return;
+  const releaseLock = acquireRefreshLock();
+  if (!releaseLock) return;
   try {
+    const lockedExisting = readCache();
+    if (lockedExisting && freshEnough(lockedExisting)) return;
     const res = await fetch(REGISTRY_URL, {
       headers: { Accept: "application/json" },
       signal: AbortSignal.timeout(FETCH_TIMEOUT_MS),
@@ -79,5 +144,12 @@ export async function refreshVersionCache(): Promise<void> {
     writeCache({ latest: body.version, fetchedAt: Date.now() });
   } catch {
     // best-effort; offline / registry down / timeout — silent
+  } finally {
+    releaseLock();
   }
 }
+
+export const __test__ = {
+  CACHE_TTL_MS,
+  isKnownNoninteractiveStatusProbe,
+};