@npeercy/skills 0.1.8 → 0.1.9

package/lib/skills.js

@@ -77,10 +77,9 @@ async function resolveSkillId(input, config) {
 }
 
 function linkNameForSkill(skillId) {
-  if (skillId.startsWith('__builtin__/local/')) {
-    return `builtin--${splitSkillId(skillId).name}`;
-  }
-  return symlinkName(skillId);
+  // Use bare skill name so directory matches name: field in SKILL.md.
+  // Pi expects these to match; mismatches show as "[Skill conflicts]".
+  return splitSkillId(skillId).name;
 }
 
 async function promptForToken(serverUrl) {

package/lib/startup.js

@@ -1,4 +1,6 @@
-import { loadConfig, saveConfig } from './config.js';
+import { existsSync, readdirSync, lstatSync, readlinkSync, unlinkSync, symlinkSync, mkdirSync } from 'fs';
+import { join, resolve, dirname } from 'path';
+import { loadConfig, saveConfig, loadState, saveState, detectAgents } from './config.js';
 import { ensureBuiltinsInstalled, getLocalDiagnostics } from './skills.js';
 
 function cmpSemver(a, b) {
@@ -52,10 +54,66 @@ async function runUpdateCheck(currentVersion, { silent = false } = {}) {
   }
 }
 
+function migrateOldSymlinks() {
+  // Migrate from old naming (org--user--name, builtin--name) to bare skill name.
+  // Pi expects directory name == name: field in SKILL.md.
+  const st = loadState();
+  const agents = detectAgents();
+  let migrated = 0;
+
+  for (const [id, rec] of Object.entries(st.installed)) {
+    const parts = id.split('/');
+    const bareName = parts[parts.length - 1]; // the skill name
+
+    // Figure out old link name
+    let oldLinkName;
+    if (id.startsWith('__builtin__/local/')) {
+      oldLinkName = `builtin--${bareName}`;
+    } else {
+      oldLinkName = id.replace(/\//g, '--');
+    }
+
+    // If old == new, nothing to do
+    if (oldLinkName === bareName) continue;
+
+    for (const agentName of (rec.agents || [])) {
+      const agentPath = agents[agentName];
+      if (!agentPath || !existsSync(agentPath)) continue;
+
+      const oldLink = join(agentPath, oldLinkName);
+      const newLink = join(agentPath, bareName);
+
+      // Remove old symlink if it exists
+      try {
+        if (lstatSync(oldLink).isSymbolicLink()) {
+          unlinkSync(oldLink);
+        }
+      } catch { /* doesn't exist */ }
+
+      // Create new symlink if not already there
+      if (!existsSync(newLink) && rec.path && existsSync(rec.path)) {
+        try {
+          mkdirSync(agentPath, { recursive: true });
+          symlinkSync(rec.path, newLink);
+          migrated++;
+        } catch { /* best effort */ }
+      }
+    }
+  }
+
+  return migrated;
+}
+
 async function runAutoHeal(command, { jsonMode = false } = {}) {
   if (process.env.SKILLS_NO_STARTUP_HEAL === '1') return;
   if (['init'].includes(command)) return;
 
+  // Migrate old-style symlinks (org--user--name → bare name)
+  const migrated = migrateOldSymlinks();
+  if (!jsonMode && migrated > 0) {
+    console.log(`✓ Startup heal: migrated ${migrated} symlink(s) to match Pi naming.`);
+  }
+
   const result = await ensureBuiltinsInstalled({ quiet: true });
   if (!jsonMode && result.changed) {
     console.log(`✓ Startup heal: installed ${result.installed.length} missing built-in skill(s).`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@npeercy/skills",
-  "version": "0.1.8",
+  "version": "0.1.9",
   "description": "CLI-first skill marketplace for coding agents",
   "type": "module",
   "bin": {