joyskills-cli 0.2.8 → 0.2.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "joyskills-cli",
-  "version": "0.2.8",
+  "version": "0.2.10",
   "description": "Team-level skill governance compatible with open skill / Claude Skills",
   "main": "src/index.js",
   "bin": {

package/src/commands/install.js

@@ -206,29 +206,26 @@ async function tryInstallFromRegistryDir(skillName, registryDirPath, targetDir,
       return false;
     }
   } else {
-    // No registry.yaml: scan skill directories directly
-    const skillPath = path.join(registryDirPath, skillName);
-    if (fs.existsSync(path.join(skillPath, 'SKILL.md'))) {
-      const targetPath = path.join(targetDir, skillName);
-      copyRecursive(skillPath, targetPath);
-
-      const content = fs.readFileSync(path.join(skillPath, 'SKILL.md'), 'utf-8');
-      const versionMatch = content.match(/version:\s*(.+)/);
-      const version = versionMatch?.[1]?.trim() || '1.0.0';
+    // No registry.yaml: scan skill directories recursively
+    const skills = await findSkills(registryDirPath);
+    const skill = skills.find(s => s.name === skillName);
+    
+    if (!skill) return false;
+    
+    const targetPath = path.join(targetDir, skillName);
+    copyRecursive(skill.path, targetPath);
 
-      const lockfileManager = new LockfileManager(projectRoot);
-      await lockfileManager.load();
-      lockfileManager.updateSkill(skillName, {
-        version,
-        source: sourceLabel,
-        installedAt: new Date().toISOString()
-      });
-      await lockfileManager.save();
+    const lockfileManager = new LockfileManager(projectRoot);
+    await lockfileManager.load();
+    lockfileManager.updateSkill(skillName, {
+      version: skill.version || '1.0.0',
+      source: sourceLabel,
+      installedAt: new Date().toISOString()
+    });
+    await lockfileManager.save();
 
-      console.log(chalk.green(`✅ Installed ${skillName} v${version} from ${sourceLabel}`));
-      return true;
-    }
-    return false;
+    console.log(chalk.green(`✅ Installed ${skillName} v${skill.version || '1.0.0'} from ${sourceLabel}`));
+    return true;
   }
 }
 
@@ -380,6 +377,14 @@ async function installFromGitUrl(gitUrl, targetDir, options) {
     console.log(chalk.green('   ✓ Repository cloned'));
   } else {
     console.log(chalk.gray('   Using cached repository'));
+    // Try to update cache
+    try {
+      const git = simpleGit(cachePath);
+      await git.pull('origin', 'main', ['--depth', '1']);
+      console.log(chalk.gray('   ✓ Cache updated'));
+    } catch (e) {
+      // Ignore update errors, use cached version
+    }
   }
 
   const skills = await findSkills(cachePath);
@@ -442,6 +447,14 @@ async function installFromGitHub(owner, repo, skillPath, targetDir, options) {
     console.log(chalk.green('   ✓ Repository cloned'));
   } else {
     console.log(chalk.gray('   Using cached repository'));
+    // Try to update cache
+    try {
+      const git = simpleGit(cachePath);
+      await git.pull('origin', 'main', ['--depth', '1']);
+      console.log(chalk.gray('   ✓ Cache updated'));
+    } catch (e) {
+      // Ignore update errors, use cached version
+    }
   }
   
   // Find skills in the repo

package/src/commands/remove.js

@@ -23,7 +23,8 @@ export function removeCommand(program) {
   program
     .command('remove <skill>')
     .description('Remove a skill')
-    .action(async (skillName) => {
+    .option('-y, --yes', 'Skip confirmation prompt')
+    .action(async (skillName, options) => {
       const projectRoot = process.cwd();
       const lockfileManager = new LockfileManager(projectRoot);
 

package/src/commands/upgrade.js

@@ -269,9 +269,12 @@ async function upgradeFromGitHub(skill, skillsDir) {
   const git = simpleGit(cachePath);
   await git.pull();
   
-  // Find skill in cache
-  const skillSourcePath = path.join(cachePath, skill.name);
-  if (!fs.existsSync(skillSourcePath)) {
+  // Find skill in cache (support nested directories)
+  const { findSkills } = await import('../commands/install.js');
+  const skills = await findSkills(cachePath);
+  const skillInfo = skills.find(s => s.name === skill.name);
+  
+  if (!skillInfo) {
     throw new Error(`Skill not found in repository: ${skill.name}`);
   }
   
@@ -282,7 +285,7 @@ async function upgradeFromGitHub(skill, skillsDir) {
   }
   
   // Copy new version
-  copyRecursive(skillSourcePath, targetPath);
+  copyRecursive(skillInfo.path, targetPath);
 }
 
 async function upgradeFromRegistry(skill, skillsDir) {