skiller 0.9.5 → 0.9.6

package/dist/cli/handlers.js

@@ -71,6 +71,15 @@ async function executeSkillsWrapper(projectRoot, args) {
 function buildClaudePluginMigrationArgs(source) {
     return ['add', source, '--agent', 'universal', '--skill', '*', '-y'];
 }
+function resolveRegistryMatchSource(match) {
+    if (match.source)
+        return match.source;
+    const parts = match.slug.split('/').filter(Boolean);
+    if (parts.length >= 2) {
+        return `${parts[0]}/${parts[1]}`;
+    }
+    return match.slug;
+}
 function formatInstalls(count) {
     if (!count || count <= 0)
         return '0 installs';
@@ -168,6 +177,34 @@ async function cleanupLegacyClaudePluginState(projectRoot, pluginIds) {
         await fs.writeFile(manifestPath, JSON.stringify(manifest, null, 2) + '\n');
     }
 }
+async function cleanupMigratedPluginAuxiliaryRules(projectRoot, sources) {
+    const candidateRuleNames = await (0, ClaudePluginMigration_1.listClaudePluginAuxiliaryRuleNames)(sources);
+    if (candidateRuleNames.length === 0)
+        return [];
+    const sourceSet = new Set(sources);
+    const plan = await (0, RulesToSkillsMigration_1.planRulesToSkillsMigration)(projectRoot, candidateRuleNames);
+    const removable = new Set(plan.unmatched);
+    for (const candidate of plan.candidates) {
+        const exactMatchSources = candidate.matches.map(resolveRegistryMatchSource);
+        if (exactMatchSources.length === 0 ||
+            exactMatchSources.every((source) => sourceSet.has(source))) {
+            removable.add(candidate.ruleName);
+        }
+    }
+    const removed = [...removable].sort((a, b) => a.localeCompare(b));
+    for (const ruleName of removed) {
+        await (0, RulesToSkillsMigration_1.removeLocalRuleReplacementState)(projectRoot, ruleName, false);
+        await fs.rm(path.join(projectRoot, '.agents', 'skills', ruleName), {
+            force: true,
+            recursive: true,
+        });
+        await fs.rm(path.join(projectRoot, '.claude', 'skills', ruleName), {
+            force: true,
+            recursive: true,
+        });
+    }
+    return removed;
+}
 async function promptLine(message) {
     const rl = readline.createInterface({
         input: process.stdin,
@@ -415,6 +452,10 @@ async function migrateClaudePluginsHandler(argv) {
             ...plan.installs.flatMap((install) => install.pluginIds),
             ...plan.unresolved.map((entry) => entry.pluginId),
         ]);
+        const removedAuxiliaryRules = await cleanupMigratedPluginAuxiliaryRules(projectRoot, plan.installs.map((install) => install.source));
+        if (removedAuxiliaryRules.length > 0) {
+            console.log(`[skiller] Removed stale plugin-derived local rules:\n${removedAuxiliaryRules.map((name) => `- ${name}`).join('\n')}`);
+        }
         if (plan.unresolved.length > 0) {
             console.log(`[skiller] Skipped unresolved plugins:\n${plan.unresolved.map((entry) => `- ${entry.pluginId}: ${entry.reason}`).join('\n')}`);
         }

package/dist/core/ClaudePluginMigration.js

@@ -33,6 +33,7 @@ var __importStar = (this && this.__importStar) || (function () {
     };
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.listClaudePluginAuxiliaryRuleNames = listClaudePluginAuxiliaryRuleNames;
 exports.planClaudePluginSkillsMigration = planClaudePluginSkillsMigration;
 const fs = __importStar(require("fs/promises"));
 const os = __importStar(require("os"));
@@ -86,6 +87,20 @@ function normalizeCloneSource(source) {
     }
     return source;
 }
+function normalizeSkillNameForDir(value) {
+    return value.replace(/:/g, '-').trim();
+}
+function extractAuxiliarySkillNames(skillMd) {
+    const names = new Set();
+    const pattern = /\b[a-z0-9-]+:[a-z0-9-]+:([a-z0-9-]+)\b/g;
+    for (const match of skillMd.toLowerCase().matchAll(pattern)) {
+        const candidate = match[1]?.trim();
+        if (!candidate || !/[a-z]/.test(candidate))
+            continue;
+        names.add(candidate);
+    }
+    return [...names];
+}
 function formatInspectionError(source, err) {
     if (typeof err === 'object' && err !== null) {
         const stderr = 'stderr' in err ? err.stderr : '';
@@ -112,18 +127,30 @@ async function inspectSkillsInstallSource(source) {
             repoDir,
         ]);
         const { skills } = await (0, SkillsUtils_1.walkSkillsTree)(repoDir);
+        const publishedSkillNames = new Set();
+        const auxiliarySkillNames = new Set();
         for (const skill of skills) {
             try {
                 const skillMd = await fs.readFile(path.join(skill.path, 'SKILL.md'), 'utf8');
                 const { frontmatter } = (0, FrontmatterParser_1.parseFrontmatter)(skillMd);
                 if (frontmatter?.name && frontmatter.description) {
-                    return { installable: true };
+                    publishedSkillNames.add(normalizeSkillNameForDir(String(frontmatter.name)));
+                    for (const auxiliaryName of extractAuxiliarySkillNames(skillMd)) {
+                        auxiliarySkillNames.add(auxiliaryName);
+                    }
                 }
             }
             catch {
                 // Keep scanning. One malformed skill should not hide valid siblings.
             }
         }
+        if (publishedSkillNames.size > 0) {
+            return {
+                installable: true,
+                auxiliarySkillNames: [...auxiliarySkillNames].sort((a, b) => a.localeCompare(b)),
+                publishedSkillNames: [...publishedSkillNames].sort((a, b) => a.localeCompare(b)),
+            };
+        }
         return {
             installable: false,
             reason: `Resolved source ${source} has no valid SKILL.md files with name and description`,
@@ -139,6 +166,23 @@ async function inspectSkillsInstallSource(source) {
         await fs.rm(tmpDir, { recursive: true, force: true });
     }
 }
+async function listClaudePluginAuxiliaryRuleNames(sources, options = {}) {
+    const inspectSource = options.inspectSource ?? inspectSkillsInstallSource;
+    const auxiliarySkillNames = new Set();
+    for (const source of [...new Set(sources)].sort((a, b) => a.localeCompare(b))) {
+        const inspection = await inspectSource(source);
+        if (!inspection.installable)
+            continue;
+        const published = new Set((inspection.publishedSkillNames ?? []).map(normalizeSkillNameForDir));
+        for (const auxiliaryName of inspection.auxiliarySkillNames ?? []) {
+            const normalized = normalizeSkillNameForDir(auxiliaryName);
+            if (!normalized || published.has(normalized))
+                continue;
+            auxiliarySkillNames.add(normalized);
+        }
+    }
+    return [...auxiliarySkillNames].sort((a, b) => a.localeCompare(b));
+}
 async function readEnabledPluginIds(projectRoot) {
     const settingsPath = path.join(projectRoot, project_paths_1.LEGACY_SKILLER_DIR, 'settings.json');
     const raw = await readJsonFile(settingsPath);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skiller",
-  "version": "0.9.5",
+  "version": "0.9.6",
   "description": "Skiller — apply the same rules to all coding agents",
   "main": "dist/lib.js",
   "publishConfig": {