joyskills-cli 0.3.7 → 0.3.8

package/package.json

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

package/src/commands/install.js

@@ -597,7 +597,9 @@ async function installFromGitUrl(gitUrl, targetDir, options, subPath = '') {
   }
 
   for (const skill of selectedSkills) {
-    await installSkillFiles(skill, targetDir);
+    // For Git URL installs with subPath, use the last part of subPath as target name
+    const targetName = subPath ? subPath.split('/').pop() : skill.name;
+    await installSkillFiles(skill, targetDir, targetName);
   }
 
   // Update lockfile only for project-level installation (npm-like behavior)
@@ -605,7 +607,8 @@ async function installFromGitUrl(gitUrl, targetDir, options, subPath = '') {
     const lockfileManager = new LockfileManager(process.cwd());
     await lockfileManager.load();
     for (const skill of selectedSkills) {
-      lockfileManager.updateSkill(skill.name, {
+      const targetName = subPath ? subPath.split('/').pop() : skill.name;
+      lockfileManager.updateSkill(targetName, {
         version: skill.version || '1.0.0',
         source: 'git',
         repository: gitUrl,
@@ -691,7 +694,10 @@ async function installFromGitHub(owner, repo, skillPath, targetDir, options) {
   
   // Install selected skills
   for (const skill of selectedSkills) {
-    await installSkillFiles(skill, targetDir);
+    // For GitHub installs, use the last part of skillPath as target name
+    // e.g., "anthropics/skills/pdf" -> install as "pdf", not "skills/pdf"
+    const targetName = skillPath ? skillPath.split('/').pop() : skill.name;
+    await installSkillFiles(skill, targetDir, targetName);
   }
   
   // Update lockfile only for project-level installation (npm-like behavior)
@@ -699,7 +705,9 @@ async function installFromGitHub(owner, repo, skillPath, targetDir, options) {
     const lockfileManager = new LockfileManager(process.cwd());
     await lockfileManager.load();
     for (const skill of selectedSkills) {
-      lockfileManager.updateSkill(skill.name, {
+      // Use the same targetName for lockfile consistency
+      const targetName = skillPath ? skillPath.split('/').pop() : skill.name;
+      lockfileManager.updateSkill(targetName, {
         version: skill.version || '1.0.0',
         source: 'github',
         repository: `${owner}/${repo}`,
@@ -904,8 +912,8 @@ function formatSize(bytes) {
   return (bytes / (1024 * 1024)).toFixed(1) + 'MB';
 }
 
-async function installSkillFiles(skill, targetDir) {
-  const targetPath = path.join(targetDir, skill.name);
+async function installSkillFiles(skill, targetDir, targetName = null) {
+  const targetPath = path.join(targetDir, targetName || skill.name);
   
   if (fs.existsSync(targetPath)) {
     fs.rmSync(targetPath, { recursive: true, force: true });