chainlesschain 0.162.175 → 0.162.177

package/src/commands/update.js

@@ -1,11 +1,58 @@
 import chalk from "chalk";
 import semver from "semver";
-import { execSync } from "node:child_process";
 import { checkForUpdates } from "../lib/version-checker.js";
 import { downloadRelease } from "../lib/downloader.js";
 import { VERSION } from "../constants.js";
 import { askConfirm } from "../lib/prompts.js";
 import logger from "../lib/logger.js";
+import { executionBroker } from "../lib/process-execution-broker/index.js";
+import { resolveNpmInvocation } from "../lib/npm-invocation.js";
+
+export const _deps = {
+  platform: process.platform,
+  execPath: process.execPath,
+  cliEntryPath: process.argv[1] || null,
+  spawnSync: (...args) => executionBroker.spawnSync(...args),
+};
+
+export function updateProcessInvocations(
+  platform = _deps.platform,
+  execPath = _deps.execPath,
+  cliEntryPath = _deps.cliEntryPath,
+) {
+  return {
+    npm: resolveNpmInvocation({ platform, execPath }),
+    chainlesschain: cliEntryPath
+      ? { command: execPath, prefixArgs: [cliEntryPath] }
+      : {
+          command:
+            platform === "win32" ? "chainlesschain.cmd" : "chainlesschain",
+          prefixArgs: [],
+        },
+  };
+}
+
+function runUpdateProcess(command, args, origin) {
+  const result = _deps.spawnSync(command, args, {
+    encoding: "utf-8",
+    stdio: "pipe",
+    origin,
+    policy: "allow",
+    scope: "update",
+    shell: false,
+  });
+  if (result?.error) throw result.error;
+  if (result?.status != null && result.status !== 0) {
+    const error = new Error(
+      `Update process failed (exit ${result.status}): ${command}`,
+    );
+    error.status = result.status;
+    error.stdout = result.stdout;
+    error.stderr = result.stderr;
+    throw error;
+  }
+  return String(result?.stdout || "");
+}
 
 /**
  * A version that is safe to interpolate into `npm install -g chainlesschain@<v>`.
@@ -19,7 +66,7 @@ export function isInstallableVersion(v) {
   return typeof v === "string" && semver.valid(v) !== null;
 }
 
-async function selfUpdateCli(targetVersion) {
+export async function selfUpdateCli(targetVersion) {
   if (VERSION === targetVersion) {
     return true; // Already at the target version
   }
@@ -33,16 +80,24 @@ async function selfUpdateCli(targetVersion) {
 
   try {
     logger.info("Updating CLI package...");
-    execSync(`npm install -g chainlesschain@${targetVersion}`, {
-      encoding: "utf-8",
-      stdio: "pipe",
-    });
+    const invocations = updateProcessInvocations();
+    runUpdateProcess(
+      invocations.npm.command,
+      [
+        ...invocations.npm.prefixArgs,
+        "install",
+        "-g",
+        `chainlesschain@${targetVersion}`,
+      ],
+      "update:npm-install",
+    );
     // Verify the update actually took effect
     try {
-      const newVersion = execSync("chainlesschain --version", {
-        encoding: "utf-8",
-        stdio: "pipe",
-      }).trim();
+      const newVersion = runUpdateProcess(
+        invocations.chainlesschain.command,
+        [...invocations.chainlesschain.prefixArgs, "--version"],
+        "update:version-check",
+      ).trim();
       if (newVersion === targetVersion) {
         logger.success(`CLI updated to v${targetVersion}`);
         return true;