joyskills-cli 0.3.1 → 0.3.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "joyskills-cli",
-  "version": "0.3.1",
+  "version": "0.3.2",
   "description": "JoySkills CLI v2.0 - Multi-agent skill management with JoyCode native support",
   "main": "src/index.js",
   "bin": {

package/src/commands/team.js

@@ -50,11 +50,12 @@ export function teamCommand(program) {
         const registryYamlPath = path.join(registryCachePath, 'registry.yaml');
         const hasRegistryYaml = fs.existsSync(registryYamlPath);
 
-        // Count skill directories (dirs with SKILL.md)
-        const entries = fs.readdirSync(registryCachePath, { withFileTypes: true });
-        const skillDirs = entries.filter(e => e.isDirectory() && fs.existsSync(path.join(registryCachePath, e.name, 'SKILL.md')));
+        // Count skills using findSkills (supports nested directories)
+        const { findSkills } = await import('./install.js');
+        const foundSkills = await findSkills(registryCachePath);
+        const skillCount = foundSkills.length;
 
-        if (!hasRegistryYaml && skillDirs.length === 0) {
+        if (!hasRegistryYaml && skillCount === 0) {
           throw new Error('Invalid registry: no registry.yaml and no skill directories found');
         }
 
@@ -66,14 +67,14 @@ export function teamCommand(program) {
         }
 
         let registryId = name;
-        let skillCount = skillDirs.length;
+        let finalSkillCount = skillCount;
 
         if (hasRegistryYaml) {
           const registryManager = new RegistryManager(registryCachePath);
           await registryManager.load();
           const info = registryManager.getRegistryInfo();
           registryId = info.registryId;
-          skillCount = registryManager.getAllSkills().length;
+          finalSkillCount = registryManager.getAllSkills().length;
         }
 
         config.registries[name] = {
@@ -88,7 +89,7 @@ export function teamCommand(program) {
 
         console.log(chalk.green(`✅ Successfully added registry: ${name}`));
         console.log(chalk.gray(`   Registry ID: ${registryId}`));
-        console.log(chalk.gray(`   Skills: ${skillCount}${hasRegistryYaml ? '' : ' (no registry.yaml, direct scan)'}`));
+        console.log(chalk.gray(`   Skills: ${finalSkillCount}${hasRegistryYaml ? '' : ' (no registry.yaml, direct scan)'}`));
         
       } catch (error) {
         console.error(chalk.red(`❌ Failed to add registry: ${error.message}`));

package/src/commands/update.js

@@ -276,14 +276,24 @@ async function checkGitSkillUpdate(skill) {
     return { hasUpdate: false, error: 'Not a git repository' };
   }
   
+  // 获取当前 commit
+  const currentLog = await git.log({ maxCount: 1 });
+  const currentCommit = currentLog.latest?.hash;
+  
+  // Fetch 远程最新
   await git.fetch(['--depth', '1']);
-  const log = await git.log({ maxCount: 1 });
-  const latestCommit = log.latest;
   
-  // 简化：如果有 fetch 成功，认为可能有更新
+  // 获取远程最新 commit
+  const remoteLog = await git.log({ maxCount: 1 });
+  const remoteCommit = remoteLog.latest?.hash;
+  
+  // 比较 commit
+  const hasUpdate = remoteCommit !== currentCommit;
+  
   return {
-    hasUpdate: true,
-    latestVersion: latestCommit?.hash?.slice(0, 7) || 'latest',
+    hasUpdate,
+    latestVersion: remoteCommit?.slice(0, 7) || 'latest',
+    currentVersion: currentCommit?.slice(0, 7),
   };
 }