itismyskillmarket 1.3.30 → 1.3.32

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 = {}) {
@@ -2561,11 +2600,17 @@ API_ROUTES.POST["/api/upload"] = async (req, res, _url) => {
       jsonResponse(res, 400, { error: "ZIP archive is empty" });
       return;
     }
-    const pkgEntry = entries.find((e) => e.entryName === "package.json" || e.entryName.endsWith("/package.json"));
+    const normalizeEntryName = (name) => name.replace(/\\/g, "/").replace(/^\.\//, "");
+    const pkgEntry = entries.find((e) => {
+      const normalized = normalizeEntryName(e.entryName);
+      return normalized === "package.json" || normalized.endsWith("/package.json");
+    });
     let skillName = "";
     let pkgInfo = {};
+    let pkgEntryRelativePath = "";
     if (pkgEntry) {
       try {
+        pkgEntryRelativePath = normalizeEntryName(pkgEntry.entryName);
         pkgInfo = JSON.parse(pkgEntry.getData().toString("utf-8"));
         skillName = pkgInfo.skillmarket?.id || pkgInfo.name?.replace(/^@[^/]+\//, "") || "";
       } catch {
@@ -2587,7 +2632,13 @@ API_ROUTES.POST["/api/upload"] = async (req, res, _url) => {
     mkdirSync(skillDir, { recursive: true });
     zip.extractAllTo(skillDir, true);
     const skillMdExists = existsSync3(join3(skillDir, "SKILL.md")) || entries.some((e) => e.entryName.endsWith("SKILL.md"));
-    const pkgJsonPath = join3(skillDir, "package.json");
+    let pkgJsonPath = join3(skillDir, "package.json");
+    if (pkgEntryRelativePath) {
+      const extractedPkgPath = join3(skillDir, pkgEntryRelativePath);
+      if (existsSync3(extractedPkgPath)) {
+        pkgJsonPath = extractedPkgPath;
+      }
+    }
     if (existsSync3(pkgJsonPath)) {
       try {
         pkgInfo = JSON.parse(readFileSync2(pkgJsonPath, "utf-8"));

package/gui/app.js

@@ -220,6 +220,10 @@ const translations = {
     'upload.id': 'ID',
     'upload.version': 'Version',
     'upload.description': 'Description',
+    'upload.validation': 'Validation',
+    'upload.stats': 'Stats',
+    'upload.done': '✅ Done',
+    'upload.noSkillUploaded': 'No skill uploaded. Please upload first.',
     'upload.platforms': 'Platforms',
     'upload.fileCount': 'Files',
     'upload.hasPackageJson': 'package.json',
@@ -444,6 +448,10 @@ const translations = {
     'upload.id': 'ID',
     'upload.version': '版本',
     'upload.description': '描述',
+    'upload.validation': '验证',
+    'upload.stats': '统计',
+    'upload.done': '✅ 完成',
+    'upload.noSkillUploaded': '未上传技能，请先上传。',
     'upload.platforms': '支持平台',
     'upload.fileCount': '文件数',
     'upload.hasPackageJson': 'package.json',
@@ -1149,7 +1157,7 @@ function goBackFromPlatformDetail() {
 
 async function loadHelp() {
   const container = document.getElementById('help-content');
-  container.innerHTML = '<div class="loading">Loading...</div>';
+  container.innerHTML = `<div class="loading">${t('loading.generic')}</div>`;
 
   try {
     const response = await fetch('/api/config');
@@ -2056,7 +2064,7 @@ async function handleUpload(file, skillNameOverride) {
     }
 
     progressFill.style.width = '100%';
-    progressText.textContent = '✅ Done';
+    progressText.textContent = t('upload.done');
 
     uploadState.data = result;
     uploadState.skillName = skillNameOverride || result.skillName;
@@ -2097,7 +2105,7 @@ function renderUploadPreview(data) {
     </div>
 
     <div class="upload-preview-section">
-      <h3>Validation</h3>
+      <h3>${t('upload.validation')}</h3>
       <div>
         <span class="upload-preview-badge ${data.hasPackageJson ? 'success' : 'warning'}">
           📦 ${t('upload.hasPackageJson')}: ${data.hasPackageJson ? t('upload.yes') : t('upload.no')}
@@ -2109,7 +2117,7 @@ function renderUploadPreview(data) {
     </div>
 
     <div class="upload-preview-section" style="border-bottom:none;margin-bottom:0;padding-bottom:0;">
-      <h3>Stats</h3>
+      <h3>${t('upload.stats')}</h3>
       <div class="upload-preview-stats">
         <div class="upload-preview-stat">
           <div class="upload-preview-stat-value">${data.fileCount}</div>
@@ -2127,7 +2135,7 @@ function renderUploadPreview(data) {
 /** 执行上传后的操作 */
 async function executeUploadAction(action) {
   if (!uploadState.data || !uploadState.skillName) {
-    showToast('No skill uploaded. Please upload first.', 'error');
+    showToast(t('upload.noSkillUploaded'), 'error');
     return;
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "itismyskillmarket",
-  "version": "1.3.30",
+  "version": "1.3.32",
   "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",