joyskills-cli 0.2.1 → 0.2.3

package/package.json

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

package/src/commands/install.js

@@ -67,42 +67,94 @@ function detectSkillType(skillName) {
 }
 
 /**
- * 安装公开 skill（使用 openskills）
+ * 安装公开 skill（自己实现，不依赖 openskills）
  */
 async function installPublicSkill(skillName, options, localManager, lockfileManager) {
   console.log(`📦 Installing public skill: ${skillName}`);
-  console.log(`💡 Using openskills install...\n`);
 
+  const registryManager = require('../registry');
+  const tmpDir = `/tmp/joyskills-${Date.now()}`;
+  
   try {
-    // 检查 openskills 是否可用
-    try {
-      execSync('npx openskills --version', { stdio: 'pipe' });
-    } catch (error) {
-      throw new Error('openskills not found. Please install: npm install -g openskills');
+    // 1. 解析 skill 名称
+    // anthropics/skills/pdf -> owner=anthropics, repo=skills, 相对路径=skills/pdf
+    // skillforge/code-review -> owner=skillforge, repo=code-review, 相对路径=code-review
+    const parts = skillName.split('/');
+    if (parts.length < 2) {
+      throw new Error(`Invalid skill name: ${skillName}. Expected format: owner/repo or owner/repo/path`);
     }
 
-    // 调用 openskills install
-    const cmd = `npx openskills install ${skillName}`;
-    try {
-      execSync(cmd, { stdio: 'inherit' });
-    } catch (error) {
-      throw new Error(`openskills install failed: Command failed: ${cmd}`);
+    const owner = parts[0];
+    const repo = parts[1];
+    
+    // 相对路径：如果有子路径，用 repo/subpath；否则用 repo
+    const relativePath = parts.length > 2 
+      ? `${repo}/${parts.slice(2).join('/')}`  // anthropics/skills/pdf -> skills/pdf
+      : repo;  // skillforge/code-review -> code-review
+
+    // 2. clone 仓库
+    const repoUrl = `https://github.com/${owner}/${repo}`;
+    console.log(`🔄 Cloning from: ${repoUrl}`);
+    
+    execSync(`git clone --depth 1 ${repoUrl} ${tmpDir}`, { 
+      stdio: 'pipe',
+      encoding: 'utf-8'
+    });
+
+    // 3. 寻找 SKILL.md
+    const skillSourceDir = path.join(tmpDir, relativePath);
+    const skillMdPath = path.join(skillSourceDir, 'SKILL.md');
+    
+    if (!fs.existsSync(skillMdPath)) {
+      throw new Error(`SKILL.md not found at ${relativePath}. Please check the skill path.`);
     }
 
-    // 更新 lockfile
+    // 4. 确定安装目标目录（只用最后一级目录名）
+    const skillDirName = parts[parts.length - 1];  // anthropics/skills/pdf -> pdf
+    const targetDir = options.global 
+      ? localManager.getGlobalSkillPath(skillDirName)
+      : localManager.getProjectSkillPath(skillDirName);
+
+    const location = options.global ? 'global (~/.claude/skills)' : 'project (./.claude/skills)';
+    console.log(`📍 Location: ${location}`);
+
+    // 5. 复制文件
+    if (fs.existsSync(targetDir)) {
+      fs.rmSync(targetDir, { recursive: true, force: true });
+    }
+    fs.mkdirSync(targetDir, { recursive: true });
+
+    // 复制所有文件
+    const files = fs.readdirSync(skillSourceDir);
+    for (const file of files) {
+      const srcPath = path.join(skillSourceDir, file);
+      const destPath = path.join(targetDir, file);
+      
+      if (fs.statSync(srcPath).isDirectory()) {
+        fs.cpSync(srcPath, destPath, { recursive: true });
+      } else {
+        fs.copyFileSync(srcPath, destPath);
+      }
+    }
+
+    // 6. 更新 lockfile
     lockfileManager.updateSkill(skillName, {
       version: options.version || 'latest',
       source: 'public',
       installedAt: new Date().toISOString()
     });
-
     await lockfileManager.save();
 
-    console.log(`\n✅ Successfully installed public skill: ${skillName}`);
+    console.log(`✅ Successfully installed ${skillName}`);
     console.log(`💡 Run 'joySkills sync' to update AGENTS.md`);
 
   } catch (error) {
-    throw new Error(`openskills install failed: ${error.message}`);
+    throw new Error(`Failed to install ${skillName}: ${error.message}`);
+  } finally {
+    // 清理临时目录
+    if (fs.existsSync(tmpDir)) {
+      fs.rmSync(tmpDir, { recursive: true, force: true });
+    }
   }
 }