@nucleoicons/skills 0.1.5 → 0.1.6

package/dist/install.d.ts

@@ -1,5 +1,11 @@
 import { type SkillsReleaseData } from "./api.js";
 import { type InstallManifest, type PackageTarget } from "./config.js";
+/**
+ * Resolve a real npm binary. Prefer the one next to the running Node
+ * (`process.execPath`) so we don't depend on PATH / shell aliases
+ * (common cause of `spawn npm ENOENT` even when `npm -v` works in zsh).
+ */
+export declare function resolveNpmBin(): string;
 export declare function readInstallManifest(root?: string): Promise<InstallManifest | null>;
 export declare function cleanInstallRoot(root?: string): Promise<void>;
 /**

package/dist/install.js

@@ -1,12 +1,47 @@
 import { existsSync } from "node:fs";
 import { cp, mkdir, readdir, readFile, rm, stat, writeFile } from "node:fs/promises";
-import { join } from "node:path";
+import { dirname, join } from "node:path";
 import { execFile } from "node:child_process";
 import { promisify } from "node:util";
 import extract from "extract-zip";
 import { downloadUrlToFile, formatBytes } from "./api.js";
 import { installRoot, nucleoHomeDir } from "./config.js";
 const execFileAsync = promisify(execFile);
+/**
+ * Resolve a real npm binary. Prefer the one next to the running Node
+ * (`process.execPath`) so we don't depend on PATH / shell aliases
+ * (common cause of `spawn npm ENOENT` even when `npm -v` works in zsh).
+ */
+export function resolveNpmBin() {
+    const binDir = dirname(process.execPath);
+    const candidates = process.platform === "win32"
+        ? [join(binDir, "npm.cmd"), join(binDir, "npm.exe"), "npm.cmd", "npm"]
+        : [join(binDir, "npm"), "npm"];
+    for (const candidate of candidates) {
+        if (candidate === "npm" || candidate === "npm.cmd" || existsSync(candidate)) {
+            return candidate;
+        }
+    }
+    return process.platform === "win32" ? "npm.cmd" : "npm";
+}
+async function runNpm(args, cwd) {
+    const npmBin = resolveNpmBin();
+    try {
+        await execFileAsync(npmBin, args, {
+            cwd,
+            env: process.env,
+            // Windows .cmd shims need a shell; Unix absolute paths do not.
+            shell: process.platform === "win32",
+        });
+    }
+    catch (error) {
+        const detail = error instanceof Error ? error.message : String(error);
+        throw new Error(`${detail}\n` +
+            `Tried to run: ${npmBin} ${args.join(" ")}\n` +
+            `If npm is installed, ensure it sits next to node (${process.execPath}), or run:\n` +
+            `  cd ${cwd} && npm ${args.join(" ")}`);
+    }
+}
 export async function readInstallManifest(root = installRoot()) {
     const path = join(root, "manifest.json");
     if (!existsSync(path)) {
@@ -191,10 +226,7 @@ export async function buildMcp(root = installRoot()) {
     const hasDist = existsSync(distEntry);
     console.log("  npm install (mcp)…");
     try {
-        await execFileAsync("npm", ["install", "--omit=dev"], {
-            cwd: mcpDir,
-            env: process.env,
-        });
+        await runNpm(["install", "--omit=dev"], mcpDir);
     }
     catch (error) {
         const detail = error instanceof Error ? error.message : String(error);
@@ -203,10 +235,7 @@ export async function buildMcp(root = installRoot()) {
     if (!hasDist || existsSync(join(mcpDir, "src"))) {
         console.log("  npm run build (mcp)…");
         try {
-            await execFileAsync("npm", ["run", "build"], {
-                cwd: mcpDir,
-                env: process.env,
-            });
+            await runNpm(["run", "build"], mcpDir);
         }
         catch (error) {
             if (!existsSync(distEntry)) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nucleoicons/skills",
-  "version": "0.1.5",
+  "version": "0.1.6",
   "description": "CLI to install and update Nucleo Pro MCP + icon skills",
   "type": "module",
   "bin": {