chore: Update package version to 1.0.9 and implement support for copying skills for Cursor due to symlink limitations

Author: xuhongbin

.agents/skills/react-best-practices

@@ -0,0 +1 @@
+/Users/bigo/.emp-agent/skills/react-best-practices

.superset/config.json

@@ -0,0 +1,4 @@
+{
+  "setup": [],
+  "teardown": []
+}

packages/eskill/README.md

@@ -78,7 +78,7 @@ The CLI automatically detects and links to:
 - **ClawdBot** - `~/.openclaw/skills/` or `~/.clawdbot/skills/`
 - **Cline** - `~/.cline/skills/`
 - **Codex** - `~/.codex/skills/`
-- **Cursor** - `~/.cursor/skills/`
+- **Cursor** - `~/.cursor/skills/` (uses copy instead of symlink due to [Cursor bug](https://forum.cursor.com/t/cursor-doesnt-follow-symlinks-to-discover-skills/149693))
 - **Droid** - `~/.factory/skills/`
 - **Gemini** - `~/.gemini/skills/`
 - **GitHub Copilot** - `~/.copilot/skills/`
@@ -101,9 +101,9 @@ The CLI automatically detects and links to:
 ├── config.json
 └── cache/
 
-# Symlinks to AI agents
+# Symlinks (or copies for Cursor) to AI agents
 ~/.claude/skills/skill-name -> ~/.emp-agent/skills/skill-name
-~/.cursor/skills/skill-name -> ~/.emp-agent/skills/skill-name
+~/.cursor/skills/skill-name/   # Copy (Cursor doesn't follow symlinks)
 ~/.windsurf/skills/skill-name -> ~/.emp-agent/skills/skill-name
 ```
 
@@ -264,6 +264,22 @@ ls -la ~/.claude/skills/
 # Should show symlink arrows (->)
 ```
 
+### Cursor not recognizing skills
+
+Cursor has a [known bug](https://forum.cursor.com/t/cursor-doesnt-follow-symlinks-to-discover-skills/149693) where it does not follow symlinks to discover skills. eskill now **copies** (instead of symlinking) skills to `~/.cursor/skills/` for Cursor.
+
+If you installed skills before this fix, reinstall to get the copy:
+
+```bash
+# Reinstall for Cursor only (replaces symlink with copy)
+eskill install <skill-name> --force --agent cursor
+
+# Or reinstall for all agents
+eskill install <skill-name> --force
+```
+
+Then restart Cursor. Skills appear in **Cursor Settings → Rules → Agent Decides**.
+
 ## License
 
 MIT License - EMP Team

packages/eskill/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@empjs/skill",
-  "version": "1.0.7",
+  "version": "1.0.9",
   "description": "Unified CLI tool for managing AI agent skills across Claude Code, Cursor, Windsurf, and more",
   "type": "module",
   "bin": {

packages/eskill/src/commands/install.ts

@@ -318,22 +318,25 @@ export async function install(skillNameOrPath: string, options: InstallOptions =
 
   // Target path in shared directory
   const targetPath = getSharedSkillPath(skillName)
+  const sourceIsTarget = path.resolve(skillPath) === path.resolve(targetPath)
 
-  // Check if already exists
-  if (fs.existsSync(targetPath)) {
+  // Check if already exists (skip if source is the shared dir - e.g. reinstalling for Cursor only)
+  const alreadyExists = fs.existsSync(targetPath) && !sourceIsTarget
+  if (alreadyExists) {
     if (options.force) {
       logger.warn('Removing existing installation...')
       fs.rmSync(targetPath, {recursive: true, force: true})
     } else {
-      logger.error(`Skill already exists`)
-      logger.dim(`Location: ${shortenPath(targetPath)}`)
-      logger.dim(`Tip: Add --force to overwrite`)
-      process.exit(1)
+      logger.info(`Skill already exists, updating agent links...`)
     }
   }
 
-  // Copy or link to shared directory
-  if (options.link) {
+  // Copy or link to shared directory (skip if already exists without --force, or source is shared dir)
+  if (sourceIsTarget) {
+    logger.info(`Skill already in shared directory, updating agent links...`)
+  } else if (alreadyExists && !options.force) {
+    // Skip copy, just update agent links
+  } else if (options.link) {
     // Dev mode: create symlink
     fs.symlinkSync(skillPath, targetPath, 'dir')
     logger.success(`Linked to shared directory (dev mode)`)

packages/eskill/src/commands/list.ts

@@ -100,20 +100,25 @@ export function list(): void {
 
       console.log(chalk.green('📦') + ` ${chalk.bold(skill)}${versionDisplay ? ' ' + versionDisplay : ''}${devTag}`)
 
-      // Check which agents are linked
+      // Check which agents are linked (symlink) or copied (e.g. Cursor)
       const cwd = process.cwd()
       const linkedAgents: string[] = []
       for (const agent of AGENTS) {
         const skillsDirs = getAgentSkillsDirs(agent, cwd)
-        const hasSymlink = skillsDirs.some(dir => {
+        const hasRef = skillsDirs.some(dir => {
           const agentSkillPath = path.join(dir, skill)
-          if (fs.existsSync(agentSkillPath) && isSymlink(agentSkillPath)) {
+          if (!fs.existsSync(agentSkillPath)) return false
+          if (isSymlink(agentSkillPath)) {
             const target = readSymlink(agentSkillPath)
             return target === skillPath
           }
+          // Cursor uses copy instead of symlink
+          if (agent.useCopyInsteadOfSymlink) {
+            return fs.statSync(agentSkillPath).isDirectory()
+          }
           return false
         })
-        if (hasSymlink) {
+        if (hasRef) {
           linkedAgents.push(agent.displayName)
         }
       }

packages/eskill/src/commands/remove.ts

@@ -3,7 +3,7 @@ import path from 'node:path'
 import {AGENTS, getAgentSkillsDirs} from '../config/agents.js'
 import {logger} from '../utils/logger.js'
 import {detectInstalledAgents, extractSkillName, getAgentSkillPaths, getSharedSkillPath} from '../utils/paths.js'
-import {isSymlink, removeSymlink} from '../utils/symlink.js'
+import {removeSymlink} from '../utils/symlink.js'
 
 export interface RemoveOptions {
   agent?: string // specific agent or 'all'
@@ -77,24 +77,22 @@ export async function remove(skillName: string, options: RemoveOptions = {}): Pr
     process.exit(1)
   }
 
-  // Check if there are any remaining symlinks in other agents
-  const remainingSymlinks: string[] = []
+  // Check if there are any remaining links/copies in other agents
+  const remainingRefs: string[] = []
   for (const agent of AGENTS) {
     const agentSkillPaths = getAgentSkillPaths(agent.name, extractedName, cwd)
-    const hasSymlink = agentSkillPaths.some(path => {
-      return fs.existsSync(path) && isSymlink(path)
-    })
-    if (hasSymlink) {
-      remainingSymlinks.push(agent.displayName)
+    const hasRef = agentSkillPaths.some(p => fs.existsSync(p))
+    if (hasRef) {
+      remainingRefs.push(agent.displayName)
     }
   }
 
-  if (remainingSymlinks.length > 0) {
-    logger.warn(`\n⚠️  Warning: Found remaining symlinks in:`)
-    for (const agentName of remainingSymlinks) {
+  if (remainingRefs.length > 0) {
+    logger.warn(`\n⚠️  Warning: Found remaining references in:`)
+    for (const agentName of remainingRefs) {
       logger.info(`  - ${agentName}`)
     }
-    logger.info('\nYou may need to manually remove these symlinks')
+    logger.info('\nYou may need to manually remove these')
   } else {
     logger.success(`✅ Skill "${extractedName}" removed successfully!`)
   }

packages/eskill/src/config/agents.ts

@@ -8,6 +8,8 @@ export interface AgentConfig {
   skillsDir?: string // Single directory (backward compatibility)
   skillsDirs?: string[] | ((cwd?: string) => string[]) // Multiple directories or function to get directories
   enabled: boolean
+  /** Use copy instead of symlink (e.g. Cursor doesn't follow symlinks) */
+  useCopyInsteadOfSymlink?: boolean
 }
 
 const HOME = os.homedir()
@@ -92,6 +94,8 @@ export const AGENTS: AgentConfig[] = [
     displayName: 'Cursor',
     skillsDir: path.join(HOME, '.cursor', 'skills'),
     enabled: true,
+    /** Cursor does not follow symlinks to discover skills (known bug). Use copy instead. */
+    useCopyInsteadOfSymlink: true,
   },
   {
     name: 'droid',

packages/eskill/src/utils/symlink.ts

@@ -6,7 +6,26 @@ import {logger} from './logger.js'
 import {getAgentSkillPaths, getSharedSkillPath} from './paths.js'
 
 /**
- * Create symlink from shared directory to agent directory(ies)
+ * Recursively copy directory (excludes node_modules and hidden files)
+ */
+function copyDir(src: string, dest: string): void {
+  fs.mkdirSync(dest, {recursive: true})
+  const entries = fs.readdirSync(src, {withFileTypes: true})
+  for (const entry of entries) {
+    if (entry.name === 'node_modules' || entry.name.startsWith('.')) continue
+    const srcPath = path.join(src, entry.name)
+    const destPath = path.join(dest, entry.name)
+    if (entry.isDirectory()) {
+      copyDir(srcPath, destPath)
+    } else {
+      fs.copyFileSync(srcPath, destPath)
+    }
+  }
+}
+
+/**
+ * Create symlink (or copy for Cursor) from shared directory to agent directory(ies)
+ * Cursor does not follow symlinks to discover skills - use copy instead.
  */
 export function createSymlink(skillName: string, agent: AgentConfig, cwd?: string): boolean {
   const source = getSharedSkillPath(skillName)
@@ -17,10 +36,10 @@ export function createSymlink(skillName: string, agent: AgentConfig, cwd?: strin
   }
 
   const targets = getAgentSkillPaths(agent.name, skillName, cwd)
+  const useCopy = agent.useCopyInsteadOfSymlink === true
   let successCount = 0
 
   for (const target of targets) {
-    // Ensure target directory exists
     const targetDir = path.dirname(target)
     if (!fs.existsSync(targetDir)) {
       try {
@@ -37,8 +56,10 @@ export function createSymlink(skillName: string, agent: AgentConfig, cwd?: strin
         const stats = fs.lstatSync(target)
         if (stats.isSymbolicLink()) {
           fs.unlinkSync(target)
+        } else if (stats.isDirectory()) {
+          fs.rmSync(target, {recursive: true, force: true})
         } else {
-          logger.warn(`Target exists but is not a symlink, skipping: ${target}`)
+          logger.warn(`Target exists but is not a symlink/dir, skipping: ${target}`)
           continue
         }
       } catch (error: any) {
@@ -48,10 +69,14 @@ export function createSymlink(skillName: string, agent: AgentConfig, cwd?: strin
     }
 
     try {
-      fs.symlinkSync(source, target, 'dir')
+      if (useCopy) {
+        copyDir(source, target)
+      } else {
+        fs.symlinkSync(source, target, 'dir')
+      }
       successCount++
     } catch (error: any) {
-      logger.error(`Failed to create symlink at ${target}: ${error.message}`)
+      logger.error(`Failed to ${useCopy ? 'copy' : 'symlink'} at ${target}: ${error.message}`)
     }
   }
 
@@ -65,7 +90,8 @@ export function createSymlink(skillName: string, agent: AgentConfig, cwd?: strin
 }
 
 /**
- * Remove symlink from agent directory(ies)
+ * Remove symlink or copied directory from agent directory(ies)
+ * Handles both symlinks (most agents) and directories (Cursor uses copy)
  */
 export function removeSymlink(skillName: string, agent: AgentConfig, cwd?: string): boolean {
   const targets = getAgentSkillPaths(agent.name, skillName, cwd)
@@ -81,11 +107,14 @@ export function removeSymlink(skillName: string, agent: AgentConfig, cwd?: strin
       if (stats.isSymbolicLink()) {
         fs.unlinkSync(target)
         removedCount++
+      } else if (stats.isDirectory()) {
+        fs.rmSync(target, {recursive: true, force: true})
+        removedCount++
       } else {
-        logger.warn(`Not a symlink: ${target}`)
+        logger.warn(`Not a symlink or directory: ${target}`)
       }
     } catch (error: any) {
-      logger.error(`Failed to remove symlink at ${target}: ${error.message}`)
+      logger.error(`Failed to remove at ${target}: ${error.message}`)
     }
   }