@yuanchuan/aivo 0.31.0 → 0.31.2

package/bin/aivo.js

@@ -1,14 +1,9 @@
 #!/usr/bin/env node
 
-const { spawn, spawnSync } = require("node:child_process");
+const { spawn } = require("node:child_process");
 const os = require("node:os");
-const { getInstalledBinaryPath, getPackageRoot } = require("../lib/paths");
+const { getInstalledBinaryPath } = require("../lib/paths");
 const { formatLaunchError } = require("../lib/launcher");
-const {
-  WINDOWS_NPM_UPDATE_COMMAND,
-  shouldDelegateWindowsNpmUpdate,
-  spawnWindowsNpmUpdate
-} = require("../lib/update");
 
 const args = process.argv.slice(2);
 
@@ -23,68 +18,16 @@ function forwardExit(child) {
   });
 }
 
-const useColor =
-  Boolean(process.stdout.isTTY) &&
-  !process.env.NO_COLOR &&
-  !args.includes("--no-color");
-const CYAN = useColor ? "\x1b[36m" : "";
-const GREEN = useColor ? "\x1b[32m" : "";
-const DIM = useColor ? "\x1b[2m" : "";
-const RESET = useColor ? "\x1b[0m" : "";
-
-function readInstalledVersion() {
-  try {
-    const result = spawnSync(getInstalledBinaryPath(), ["--version"], {
-      encoding: "utf8",
-      timeout: 5000
-    });
-    if (result.status === 0 && typeof result.stdout === "string") {
-      const trimmed = result.stdout.trim();
-      if (trimmed) {
-        const parts = trimmed.split(/\s+/);
-        return parts[parts.length - 1];
-      }
-    }
-  } catch {
-    // best-effort: fall through to null
-  }
-  return null;
-}
-
-function runWindowsNpmUpdate() {
-  console.log(`${CYAN}→${RESET} Updating via npm...`);
-  console.log(`  ${DIM}Running:${RESET} ${GREEN}${WINDOWS_NPM_UPDATE_COMMAND}${RESET}`);
-
-  const child = spawnWindowsNpmUpdate(spawn);
-  child.on("error", (error) => {
-    console.error(`Failed to launch npm.cmd for update: ${error.message}`);
-    process.exit(1);
-  });
-  child.on("exit", (code, signal) => {
-    if (signal) {
-      process.exitCode = 128 + (os.constants.signals[signal] || 1);
-      return;
-    }
-    if (code === 0) {
-      const version = readInstalledVersion();
-      const suffix = version ? ` to version ${version}` : "";
-      console.log(`${GREEN}✓${RESET} Updated${suffix}`);
-    }
-    process.exit(code ?? 1);
-  });
-}
-
-if (shouldDelegateWindowsNpmUpdate(args, { packageRoot: getPackageRoot() })) {
-  runWindowsNpmUpdate();
-} else {
-  const binaryPath = getInstalledBinaryPath();
-  const child = spawn(binaryPath, args, {
-    stdio: "inherit"
-  });
-
-  forwardExit(child);
-  child.on("error", (error) => {
-    console.error(formatLaunchError(error, binaryPath));
-    process.exit(1);
-  });
-}
+// Every subcommand — including `update` — runs through the native binary. `update`
+// self-updates in place (download + atomic rename), which works even on Windows
+// where `npm install -g` can't overwrite the in-use global package binary.
+const binaryPath = getInstalledBinaryPath();
+const child = spawn(binaryPath, args, {
+  stdio: "inherit"
+});
+
+forwardExit(child);
+child.on("error", (error) => {
+  console.error(formatLaunchError(error, binaryPath));
+  process.exit(1);
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yuanchuan/aivo",
-  "version": "0.31.0",
+  "version": "0.31.2",
   "description": "npm wrapper for the aivo CLI",
   "license": "MIT",
   "repository": {

package/lib/update.js

@@ -1,54 +0,0 @@
-"use strict";
-
-const WINDOWS_NPM_UPDATE_COMMAND = "npm.cmd install -g @yuanchuan/aivo@latest";
-
-function normalizePathForMatch(value) {
-  return String(value || "")
-    .replace(/\\/g, "/")
-    .replace(/^\/\/\?\//, "")
-    .toLowerCase();
-}
-
-function isNpmManagedPackageRoot(packageRoot) {
-  const normalized = normalizePathForMatch(packageRoot);
-  return normalized.includes("/node_modules/@yuanchuan/aivo");
-}
-
-function shouldDelegateWindowsNpmUpdate(argv, options = {}) {
-  const platform = options.platform || process.platform;
-  const packageRoot = options.packageRoot;
-
-  if (platform !== "win32") {
-    return false;
-  }
-
-  if (!isNpmManagedPackageRoot(packageRoot)) {
-    return false;
-  }
-
-  if (!Array.isArray(argv) || argv[0] !== "update") {
-    return false;
-  }
-
-  const rest = argv.slice(1);
-  if (rest.length === 0) {
-    return true;
-  }
-
-  return rest.every((arg) => arg === "--no-color");
-}
-
-function spawnWindowsNpmUpdate(spawnImpl, options = {}) {
-  const comspec = options.comspec || process.env.ComSpec || process.env.COMSPEC || "cmd.exe";
-  return spawnImpl(comspec, ["/d", "/s", "/c", WINDOWS_NPM_UPDATE_COMMAND], {
-    stdio: "inherit"
-  });
-}
-
-module.exports = {
-  WINDOWS_NPM_UPDATE_COMMAND,
-  isNpmManagedPackageRoot,
-  normalizePathForMatch,
-  shouldDelegateWindowsNpmUpdate,
-  spawnWindowsNpmUpdate
-};