joyskills-cli 0.3.3 → 0.3.4

package/package.json

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

package/src/commands/install.js

@@ -14,6 +14,22 @@ import chalk from 'chalk';
 // Cache directory for cloned repos
 const CACHE_DIR = process.env.JOYSKILL_CACHE_DIR || path.join(os.homedir(), '.joyskill', 'cache');
 
+/**
+ * Get git commit hash for a directory
+ */
+async function getGitCommit(dirPath) {
+  try {
+    const git = simpleGit(dirPath);
+    const isRepo = await git.checkIsRepo();
+    if (!isRepo) return null;
+    
+    const log = await git.log({ maxCount: 1 });
+    return log.latest?.hash || null;
+  } catch {
+    return null;
+  }
+}
+
 export function installCommand(program) {
   program
     .command('install [skill]')
@@ -252,6 +268,9 @@ async function tryInstallFromRegistryDir(skillName, registryDirPath, targetDir,
       const installInfo = fs.lstatSync(targetPath);
       const installMethod = installInfo.isSymbolicLink() ? 'symlink' : 'copy';
 
+      // Get git commit for the skill source
+      const commitHash = await getGitCommit(sourcePath);
+      
       const lockfileManager = new LockfileManager(projectRoot);
       await lockfileManager.load();
       lockfileManager.updateSkill(skillName, {
@@ -259,6 +278,7 @@ async function tryInstallFromRegistryDir(skillName, registryDirPath, targetDir,
         source: sourceLabel,
         registry: registryManager.getRegistryInfo().registryId,
         installMethod,
+        commit: commitHash,
         installedAt: new Date().toISOString()
       });
       await lockfileManager.save();
@@ -279,11 +299,15 @@ async function tryInstallFromRegistryDir(skillName, registryDirPath, targetDir,
     const targetPath = path.join(targetDir, skillName);
     copyRecursive(skill.path, targetPath);
 
+    // Get git commit for the skill source
+    const commitHash = await getGitCommit(skill.path);
+
     const lockfileManager = new LockfileManager(projectRoot);
     await lockfileManager.load();
     lockfileManager.updateSkill(skillName, {
       version: skill.version || '1.0.0',
       source: sourceLabel,
+      commit: commitHash,
       installedAt: new Date().toISOString()
     });
     await lockfileManager.save();

package/src/commands/update.js

@@ -119,6 +119,22 @@ async function getAllUpdatableSkills(projectRoot) {
   return updatable;
 }
 
+/**
+ * 获取目录的 git commit hash
+ */
+async function getDirCommit(dirPath) {
+  try {
+    const git = simpleGit(dirPath);
+    const isRepo = await git.checkIsRepo();
+    if (!isRepo) return null;
+    
+    const log = await git.log({ maxCount: 1 });
+    return log.latest?.hash || null;
+  } catch {
+    return null;
+  }
+}
+
 /**
  * 更新 team:// 类型的 Skill
  */
@@ -126,7 +142,10 @@ async function updateTeamSkill(skill, projectRoot) {
   const lockfile = new LockfileManager(projectRoot);
   await lockfile.load();
   
-  const updateInfo = await checkTeamSkillUpdate(skill, skill.source);
+  const lockData = lockfile.getSkill(skill.name);
+  const localCommit = lockData?.commit;
+  
+  const updateInfo = await checkTeamSkillUpdate(skill, skill.source, localCommit);
   
   if (!updateInfo.hasUpdate) {
     throw new Error('No update available');
@@ -140,10 +159,14 @@ async function updateTeamSkill(skill, projectRoot) {
   // 复制新版本
   copyRecursive(updateInfo.registrySkillPath, skill.path);
   
+  // 获取新版本的 commit
+  const newCommit = await getDirCommit(updateInfo.registrySkillPath);
+  
   // 更新 lockfile
   lockfile.updateSkill(skill.name, {
     version: updateInfo.latestVersion,
     source: skill.source,
+    commit: newCommit,
     updatedAt: new Date().toISOString(),
   });
   await lockfile.save();
@@ -199,7 +222,8 @@ async function getSpecifiedSkills(projectRoot, skillNames) {
     
     if (teamSource) {
       // team:// 类型
-      const updateInfo = await checkTeamSkillUpdate(skill, teamSource);
+      const localCommit = lockData?.commit;
+      const updateInfo = await checkTeamSkillUpdate(skill, teamSource, localCommit);
       if (updateInfo.hasUpdate) {
         skills.push({
           name: skill.name,
@@ -238,7 +262,7 @@ async function getSpecifiedSkills(projectRoot, skillNames) {
 /**
  * 检查 team:// 类型的 Skill 是否有更新
  */
-async function checkTeamSkillUpdate(skill, source) {
+async function checkTeamSkillUpdate(skill, source, localCommit = null) {
   const registryName = source.replace('team:', '');
   const JOYSKILL_CONFIG_DIR = process.env.JOYSKILL_CONFIG_DIR || path.join(os.homedir(), '.joyskill');
   const configPath = path.join(JOYSKILL_CONFIG_DIR, 'config.json');
@@ -254,7 +278,7 @@ async function checkTeamSkillUpdate(skill, source) {
     return { hasUpdate: false, error: `Registry ${registryName} not found` };
   }
   
-  // 从 registry 查找最新版本
+  // 从 registry 查找 skill
   const { findSkills } = await import('./install.js');
   const registrySkills = await findSkills(reg.path);
   const registrySkill = registrySkills.find(s => s.name === skill.name);
@@ -263,11 +287,22 @@ async function checkTeamSkillUpdate(skill, source) {
     return { hasUpdate: false, error: 'Skill not found in registry' };
   }
   
-  const hasUpdate = registrySkill.version && registrySkill.version !== skill.version;
+  // 获取 registry 中 skill 的 commit
+  const remoteCommit = await getDirCommit(registrySkill.path);
+  
+  // 使用 commit 对比，如果没有 commit 则 fallback 到 version 对比
+  let hasUpdate = false;
+  if (localCommit && remoteCommit) {
+    hasUpdate = localCommit !== remoteCommit;
+  } else {
+    hasUpdate = registrySkill.version && registrySkill.version !== skill.version;
+  }
   
   return {
     hasUpdate,
     latestVersion: registrySkill.version,
+    localCommit,
+    remoteCommit,
     registryPath: reg.path,
     registrySkillPath: registrySkill.path,
   };
@@ -348,7 +383,8 @@ async function getUpdatableTeamSkills(projectRoot) {
   for (const skill of allSkills) {
     const lockData = lockfile.getSkill(skill.name);
     if (lockData?.source?.startsWith('team:')) {
-      const updateInfo = await checkTeamSkillUpdate(skill, lockData.source);
+      const localCommit = lockData?.commit;
+      const updateInfo = await checkTeamSkillUpdate(skill, lockData.source, localCommit);
       if (updateInfo.hasUpdate) {
         updatable.push({
           name: skill.name,

package/src/version-checker.js

@@ -80,8 +80,9 @@ export async function checkRemoteUpdate(skillPath, currentCommit) {
  * 检查单个 Skill
  */
 export async function checkSkill(skill, projectRoot) {
-  // 首先尝试从 lockfile 获取 source
+  // 首先尝试从 lockfile 获取 source 和 commit
   let teamSource = null;
+  let localCommit = null;
   
   if (projectRoot) {
     const lockfile = new LockfileManager(projectRoot);
@@ -89,6 +90,7 @@ export async function checkSkill(skill, projectRoot) {
     const lockData = lockfile.getSkill(skill.name);
     if (lockData?.source?.startsWith('team:')) {
       teamSource = lockData.source;
+      localCommit = lockData.commit;
     }
   }
   
@@ -97,9 +99,9 @@ export async function checkSkill(skill, projectRoot) {
     teamSource = await findSkillInRegistries(skill.name);
   }
   
-  // 如果是 team:// skill，对比 registry 版本
+  // 如果是 team:// skill，对比 registry commit
   if (teamSource) {
-    return await checkTeamSkill(skill, teamSource);
+    return await checkTeamSkill(skill, teamSource, localCommit);
   }
   
   // Git skill: 对比 commit
@@ -157,10 +159,26 @@ async function findSkillInRegistries(skillName) {
   return null;
 }
 
+/**
+ * 获取目录的 git commit hash
+ */
+async function getDirCommit(dirPath) {
+  try {
+    const git = simpleGit(dirPath);
+    const isRepo = await git.checkIsRepo();
+    if (!isRepo) return null;
+    
+    const log = await git.log({ maxCount: 1 });
+    return log.latest?.hash || null;
+  } catch {
+    return null;
+  }
+}
+
 /**
  * 检查 team:// skill 更新
  */
-async function checkTeamSkill(skill, source) {
+async function checkTeamSkill(skill, source, localCommit) {
   const registryName = source.replace('team:', '');
   const JOYSKILL_CONFIG_DIR = process.env.JOYSKILL_CONFIG_DIR || path.join(os.homedir(), '.joyskill');
   const configPath = path.join(JOYSKILL_CONFIG_DIR, 'config.json');
@@ -188,7 +206,7 @@ async function checkTeamSkill(skill, source) {
     };
   }
   
-  // 从 registry 查找最新版本
+  // 从 registry 查找 skill
   const { findSkills } = await import('./commands/install.js');
   const registrySkills = await findSkills(reg.path);
   const registrySkill = registrySkills.find(s => s.name === skill.name);
@@ -203,14 +221,24 @@ async function checkTeamSkill(skill, source) {
     };
   }
   
-  const hasUpdate = registrySkill.version && registrySkill.version !== skill.version;
+  // 获取 registry 中 skill 的最新 commit
+  const remoteCommit = await getDirCommit(registrySkill.path);
+  
+  // 如果没有 localCommit（旧版本安装的），使用 version 对比作为 fallback
+  let hasUpdate = false;
+  if (localCommit && remoteCommit) {
+    hasUpdate = localCommit !== remoteCommit;
+  } else {
+    hasUpdate = registrySkill.version && registrySkill.version !== skill.version;
+  }
   
   return {
     name: skill.name,
     hasUpdate,
     currentVersion: skill.version,
-    latestVersion: registrySkill.version,
-    commitsBehind: hasUpdate ? 1 : 0,  // 版本不同即视为有更新
+    localCommit: typeof localCommit === 'string' ? localCommit.slice(0, 7) : 'unknown',
+    remoteCommit: typeof remoteCommit === 'string' ? remoteCommit.slice(0, 7) : 'unknown',
+    commitsBehind: hasUpdate ? 1 : 0,
     source: `team:${registryName}`,
   };
 }