@indigoai-us/hq-cli 5.101.7 → 5.103.0

package/dist/utils/version-gate.js

@@ -167,6 +167,51 @@ export function resolveRunningManager() {
 export function resolveRunningPrefix() {
     return resolveRunningInstall().prefix;
 }
+/**
+ * Derive `PNPM_HOME` from a pnpm-managed install's own path. pnpm resolves its
+ * global bin directory from `PNPM_HOME` (or an explicit `global-bin-dir`), and
+ * `PNPM_HOME` is the path segment immediately preceding pnpm's global root, e.g.
+ *
+ *   /home/u/.local/share/pnpm/global/5/…  ->  PNPM_HOME=/home/u/.local/share/pnpm
+ *
+ * Returns null for non-pnpm installs or a `packageRoot` with no `/global/`
+ * segment (nothing to derive from). Reading it from the running install is more
+ * accurate than trusting the inherited environment: it names the exact copy
+ * that is actually on PATH.
+ */
+export function derivePnpmHome(install) {
+    if (install.manager !== "pnpm" || !install.packageRoot)
+        return null;
+    const normalized = install.packageRoot.replace(/\\/g, "/");
+    const marker = normalized.indexOf("/global/");
+    if (marker === -1)
+        return null;
+    const home = normalized.slice(0, marker);
+    return home || null;
+}
+/**
+ * Environment for the pnpm self-update spawn. `pnpm add -g` aborts with
+ * `ERR_PNPM_NO_GLOBAL_BIN_DIR` (exit 1) when neither `PNPM_HOME` nor a
+ * `global-bin-dir` setting is present — the exact failure seen when `hq`
+ * self-updates from a minimal-environment parent (a systemd `--user` service,
+ * cron, or any non-login shell that never sourced the profile exporting
+ * `PNPM_HOME`). When the running install is pnpm-managed and the parent lacks
+ * `PNPM_HOME`, inject one derived from the install's own path so the update can
+ * find the global bin dir it is about to rewrite.
+ *
+ * A no-op for npm/bun installs, and for any environment that already sets
+ * `PNPM_HOME` (never overridden — the caller's value wins). Returns `undefined`
+ * when nothing needs injecting, so callers can spawn with the inherited
+ * environment unchanged (and pass no `env` at all).
+ */
+export function pnpmUpdateEnv(install, base = process.env) {
+    if (install.manager !== "pnpm" || base.PNPM_HOME)
+        return undefined;
+    const home = derivePnpmHome(install);
+    if (!home)
+        return undefined;
+    return { ...base, PNPM_HOME: home };
+}
 export function buildPrefixedInstallArgv(prefix) {
     return ["install", "-g", "--prefix", prefix, LATEST_PACKAGE_SPEC];
 }
@@ -307,12 +352,13 @@ async function fetchVersionDecision() {
 // public surface (and the `__test__` block below) stable for existing callers
 // and tests.
 export { buildSpawnPlan, quoteForWindowsShell };
-export function runUpdateCommand(cmd, args) {
+export function runUpdateCommand(cmd, args, env) {
     try {
         const plan = buildSpawnPlan(cmd, args);
         const result = spawnSync(plan.cmd, plan.args, {
             stdio: "inherit",
             shell: plan.shell,
+            ...(env ? { env } : {}),
         });
         // spawnSync reports a missing executable via `error`, not a throw.
         if (result.error) {
@@ -335,8 +381,11 @@ export function runUpdateCommand(cmd, args) {
         };
     }
 }
-function performUpdateCommand(cmd, args, runner = runUpdateCommand) {
-    return runner(cmd, args);
+function performUpdateCommand(cmd, args, runner = runUpdateCommand, env) {
+    // Forward env only when there is one to forward, so the common path keeps the
+    // two-argument runner call it has always made (and existing runner spies see
+    // no phantom `undefined` third argument).
+    return env === undefined ? runner(cmd, args) : runner(cmd, args, env);
 }
 function performUpdate(command, runner = runUpdateCommand) {
     const parts = command.split(/\s+/).filter(Boolean);
@@ -442,11 +491,17 @@ function enforceUpdateRequired(decision, deps = {}) {
         primaryCmd = parts[0] ?? "";
         primaryArgs = parts.slice(1);
     }
+    // A pnpm global install needs PNPM_HOME to locate its global bin dir; a
+    // minimal-environment parent (systemd, cron, non-login shell) lacks it and
+    // `pnpm add -g` would abort with ERR_PNPM_NO_GLOBAL_BIN_DIR. Derive one from
+    // the running install so the update is not environment-dependent. No-op for
+    // npm/bun and when PNPM_HOME is already set.
+    const updateEnv = pnpmUpdateEnv(install);
     let result;
     if (isManagedOutsideNpm) {
         console.error(chalk.dim(`  Detected a ${install.manager}-managed global install; updating with ${install.manager}`));
         console.error(chalk.dim(`  Running: ${primaryCmd} ${primaryArgs.join(" ")}`));
-        result = performUpdateCommand(primaryCmd, primaryArgs, runner);
+        result = performUpdateCommand(primaryCmd, primaryArgs, runner, updateEnv);
     }
     else if (prefix) {
         console.error(chalk.dim(`  Installing into npm prefix: ${prefix}`));
@@ -558,8 +613,10 @@ export const __test__ = {
     buildPrefixedInstallArgv,
     buildSpawnPlan,
     cleanStalePartialInstall,
+    derivePnpmHome,
     enforceUpdateRequired,
     isBunManagedPackageDir,
+    pnpmUpdateEnv,
     isPnpmManagedPackageDir,
     npmPrefixFromPackageDir,
     nudgeUpdateRecommended,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@indigoai-us/hq-cli",
-  "version": "5.101.7",
+  "version": "5.103.0",
   "description": "HQ by Indigo management CLI — modules and cloud sync",
   "main": "dist/index.js",
   "bin": {