itismyskillmarket 1.3.30 → 1.3.31

package/dist/index.js

@@ -476,6 +476,7 @@ import fs9 from "fs-extra";
 import path8 from "path";
 import { exec } from "child_process";
 import { promisify } from "util";
+import * as tar from "tar";
 
 // src/adapters/base.ts
 import fs3 from "fs-extra";
@@ -780,17 +781,18 @@ async function installSkill(skillId, version, options) {
     console.log("Downloading package...");
     await fs9.ensureDir(cacheDir);
     try {
-      await execAsync(`npm pack ${packageName}@${targetVersion} --pack-destination ${cacheDir}`);
-      const files = await fs9.readdir(cacheDir);
-      const tarball = files.find(
-        (f) => f.endsWith(".tgz") && f.includes(packageName.replace(/^@/, "").replace("/", "-"))
+      const { stdout } = await execAsync(
+        `npm pack ${packageName}@${targetVersion} --pack-destination "${cacheDir}"`
       );
-      if (tarball) {
-        await execAsync(`tar -xzf "${path8.join(cacheDir, tarball)}" -C "${cacheDir}"`);
-        await fs9.remove(path8.join(cacheDir, tarball));
-        const extractedDir = path8.join(cacheDir, "package");
-        const finalDir = targetDir;
-        await fs9.move(extractedDir, finalDir, { overwrite: true });
+      const tarballName = stdout.trim();
+      const tarballPath = path8.join(cacheDir, tarballName);
+      if (await fs9.pathExists(tarballPath)) {
+        await tar.extract({
+          file: tarballPath,
+          cwd: cacheDir
+        });
+        await fs9.remove(tarballPath);
+        await fs9.move(path8.join(cacheDir, "package"), targetDir, { overwrite: true });
       }
     } catch (err) {
       throw new Error(`Failed to download package: ${err}`);
@@ -953,28 +955,63 @@ async function updateSkill(skillId) {
   }
   console.log(`Checking updates for ${installed.length} skill(s)...
 `);
-  let hasUpdates = false;
-  for (const skill of installed) {
-    const pkgInfo = await fetchSkillPackage(skill.id);
-    if (pkgInfo) {
-      const latestVersion = pkgInfo["dist-tags"]?.latest;
-      if (latestVersion && latestVersion !== skill.version) {
-        console.log(`  ${skill.id}: ${skill.version} \u2192 ${latestVersion} [UPDATE]`);
-        hasUpdates = true;
+  const checkResults = await Promise.allSettled(
+    installed.map(async (skill) => {
+      const pkgInfo = await fetchSkillPackage(skill.id);
+      if (!pkgInfo) {
+        return { skill, latestVersion: null, error: "failed to fetch remote" };
+      }
+      const latestVersion = pkgInfo["dist-tags"]?.latest || null;
+      return { skill, latestVersion, error: null };
+    })
+  );
+  const toUpdate = [];
+  let upToDate = 0;
+  let fetchFailed = 0;
+  for (const result of checkResults) {
+    if (result.status === "rejected") {
+      fetchFailed++;
+      continue;
+    }
+    const { skill, latestVersion, error } = result.value;
+    if (error) {
+      console.log(`  ${skill.id}: ${skill.version} (${error})`);
+      fetchFailed++;
+    } else if (latestVersion && latestVersion !== skill.version) {
+      console.log(`  ${skill.id}: ${skill.version} \u2192 ${latestVersion} [UPDATE]`);
+      toUpdate.push({ skill, latestVersion });
+    } else {
+      console.log(`  ${skill.id}: ${skill.version} (up to date)`);
+      upToDate++;
+    }
+  }
+  if (toUpdate.length > 0) {
+    console.log(`
+Updating ${toUpdate.length} skill(s)...
+`);
+    const updateResults = await Promise.allSettled(
+      toUpdate.map(async ({ skill, latestVersion }) => {
         try {
           await installSkill(skill.id, latestVersion);
+          return { id: skill.id, success: true };
         } catch (err) {
-          console.error(`  Failed to update ${skill.id}:`, err);
+          return { id: skill.id, success: false, error: err };
         }
-      } else {
-        console.log(`  ${skill.id}: ${skill.version} (up to date)`);
+      })
+    );
+    for (const result of updateResults) {
+      if (result.status === "fulfilled" && result.value.success) {
+        console.log(`  \u2705 ${result.value.id} updated`);
+      } else if (result.status === "fulfilled" && !result.value.success) {
+        console.error(`  \u274C Failed to update ${result.value.id}:`, result.value.error);
       }
-    } else {
-      console.log(`  ${skill.id}: ${skill.version} (failed to fetch remote)`);
     }
   }
-  if (!hasUpdates) {
+  if (toUpdate.length === 0) {
     console.log("\nAll skills are up to date!");
+  } else {
+    console.log(`
+\u{1F4CA} Update summary: ${toUpdate.length} updated, ${upToDate} up-to-date, ${fetchFailed} failed`);
   }
 }
 
@@ -1432,10 +1469,12 @@ Installing to ${targetPlatforms.length} platform(s)...
 }
 
 // src/commands/publish.ts
-import { execSync } from "child_process";
+import { exec as exec2 } from "child_process";
 import { existsSync as existsSync2 } from "fs";
 import { join as join2 } from "path";
 import { fileURLToPath } from "url";
+import { promisify as promisify2 } from "util";
+var execAsync2 = promisify2(exec2);
 async function publishSkill(skillName, options) {
   const __dirname4 = fileURLToPath(new URL(".", import.meta.url));
   const projectRoot = join2(__dirname4, "..");
@@ -1447,10 +1486,7 @@ async function publishSkill(skillName, options) {
   if (!options?.skipInstall) {
     console.log("Running npm install...");
     try {
-      execSync("npm install", {
-        cwd: skillDir,
-        stdio: "inherit"
-      });
+      await execAsync2("npm install", { cwd: skillDir });
     } catch (err) {
       console.warn("Warning: npm install failed, continuing anyway...");
     }
@@ -1458,9 +1494,8 @@ async function publishSkill(skillName, options) {
   if (options?.version) {
     console.log(`Updating version to ${options.version}...`);
     try {
-      execSync(`npm version ${options.version} --no-git-tag-version`, {
-        cwd: skillDir,
-        stdio: "inherit"
+      await execAsync2(`npm version ${options.version} --no-git-tag-version`, {
+        cwd: skillDir
       });
     } catch (err) {
       throw new Error(`Failed to update version: ${err}`);
@@ -1468,10 +1503,7 @@ async function publishSkill(skillName, options) {
   }
   console.log("Publishing to npm...");
   try {
-    execSync("npm publish --access=public", {
-      cwd: skillDir,
-      stdio: "inherit"
-    });
+    await execAsync2("npm publish --access=public", { cwd: skillDir });
   } catch (err) {
     throw new Error(`Failed to publish: ${err}`);
   }
@@ -1931,8 +1963,15 @@ function npmExec(command) {
   try {
     return execSync2(command, { encoding: "utf-8", stdio: ["ignore", "pipe", "pipe"] }).trim();
   } catch (err) {
-    const msg = err.stderr?.toString() || err.message || "Unknown error";
-    throw new Error(`npm command failed: ${msg.trim()}`);
+    if (err && typeof err === "object" && "stderr" in err) {
+      const stderr = err.stderr;
+      const msg = Buffer.isBuffer(stderr) ? stderr.toString() : stderr;
+      throw new Error(`npm command failed: ${msg.trim()}`);
+    }
+    if (err instanceof Error) {
+      throw new Error(`npm command failed: ${err.message}`);
+    }
+    throw new Error("npm command failed with an unknown error");
   }
 }
 async function adminDeprecate(skillId, options = {}) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "itismyskillmarket",
-  "version": "1.3.30",
+  "version": "1.3.31",
   "description": "Cross-platform skill manager for AI coding tools",
   "type": "module",
   "bin": {
@@ -15,7 +15,8 @@
   "dependencies": {
     "adm-zip": "^0.5.17",
     "commander": "^12.0.0",
-    "fs-extra": "^11.2.0"
+    "fs-extra": "^11.2.0",
+    "tar": "^7.5.15"
   },
   "devDependencies": {
     "@types/fs-extra": "^11.0.4",