skiller 0.9.1 → 0.9.3

package/dist/cli/commands.js

@@ -16,6 +16,42 @@ function skillsArgsBuilder(y) {
         description: 'Arguments passed through to the local skills CLI',
     });
 }
+function migrateClaudePluginsArgsBuilder(y) {
+    return y
+        .option('project-root', {
+        type: 'string',
+        description: 'Project root directory',
+        default: process.cwd(),
+    })
+        .option('execute', {
+        type: 'boolean',
+        description: 'Actually install the resolved repos through the local skills CLI',
+        default: false,
+    });
+}
+function migrateRulesToSkillsArgsBuilder(y) {
+    return y
+        .option('project-root', {
+        type: 'string',
+        description: 'Project root directory',
+        default: process.cwd(),
+    })
+        .option('execute', {
+        type: 'boolean',
+        description: 'Actually replace selected local rules after detection',
+        default: false,
+    })
+        .option('yes', {
+        type: 'boolean',
+        description: 'Auto-replace only unambiguous exact matches without prompting',
+        default: false,
+    })
+        .positional('rules', {
+        type: 'string',
+        array: true,
+        description: 'Specific rule names or .mdc files to check (default: all .agents/rules/*.mdc)',
+    });
+}
 /**
  * Sets up and parses CLI commands.
  */
@@ -110,42 +146,13 @@ async function run() {
             default: false,
         });
     }, handlers_1.initHandler)
-        .command('migrate claude-plugins', 'Plan or execute a one-shot migration from legacy Claude plugins to skills installs', (y) => {
-        return y
-            .option('project-root', {
-            type: 'string',
-            description: 'Project root directory',
-            default: process.cwd(),
-        })
-            .option('execute', {
-            type: 'boolean',
-            description: 'Actually install the resolved repos through the local skills CLI',
-            default: false,
-        });
-    }, handlers_1.migrateClaudePluginsHandler)
-        .command('migrate rules-to-skills [rules..]', 'Detect local .agents/rules .mdc files that already exist on skills.sh and optionally replace them', (y) => {
+        .command('migrate', 'Migration utilities', (y) => {
         return y
-            .option('project-root', {
-            type: 'string',
-            description: 'Project root directory',
-            default: process.cwd(),
-        })
-            .option('execute', {
-            type: 'boolean',
-            description: 'Actually replace selected local rules after detection',
-            default: false,
-        })
-            .option('yes', {
-            type: 'boolean',
-            description: 'Auto-replace only unambiguous exact matches without prompting',
-            default: false,
-        })
-            .positional('rules', {
-            type: 'string',
-            array: true,
-            description: 'Specific rule names or .mdc files to check (default: all .agents/rules/*.mdc)',
-        });
-    }, handlers_1.migrateRulesToSkillsHandler)
+            .command('claude-plugins', 'Plan or execute a one-shot migration from legacy Claude plugins to skills installs', migrateClaudePluginsArgsBuilder, handlers_1.migrateClaudePluginsHandler)
+            .command('rules-to-skills [rules..]', 'Detect local .agents/rules .mdc files that already exist on skills.sh and optionally replace them', migrateRulesToSkillsArgsBuilder, handlers_1.migrateRulesToSkillsHandler)
+            .demandCommand(1, 'You need to specify a migrate subcommand')
+            .strict();
+    }, () => undefined)
         .command('revert', 'Revert skiller configurations from supported AI agents', (y) => {
         return y
             .option('project-root', {

package/dist/core/ClaudePluginMigration.js

@@ -37,8 +37,13 @@ exports.planClaudePluginSkillsMigration = planClaudePluginSkillsMigration;
 const fs = __importStar(require("fs/promises"));
 const os = __importStar(require("os"));
 const path = __importStar(require("path"));
+const child_process_1 = require("child_process");
+const util_1 = require("util");
 const project_paths_1 = require("./project-paths");
 const SkillsManifest_1 = require("./SkillsManifest");
+const SkillsUtils_1 = require("./SkillsUtils");
+const FrontmatterParser_1 = require("./FrontmatterParser");
+const execFileAsync = (0, util_1.promisify)(child_process_1.execFile);
 function getClaudeHomeDir() {
     return process.env.HOME || os.homedir();
 }
@@ -72,6 +77,68 @@ function normalizeInstallSource(source) {
     }
     return null;
 }
+function normalizeCloneSource(source) {
+    if (/^[a-z][a-z0-9+.-]*:\/\//i.test(source) || source.startsWith('git@')) {
+        return source;
+    }
+    if (/^[^/\s]+\/[^/\s]+$/.test(source)) {
+        return `https://github.com/${source}.git`;
+    }
+    return source;
+}
+function formatInspectionError(source, err) {
+    if (typeof err === 'object' && err !== null) {
+        const stderr = 'stderr' in err ? err.stderr : '';
+        const stdout = 'stdout' in err ? err.stdout : '';
+        const message = stderr?.trim() ||
+            stdout?.trim() ||
+            ('message' in err ? String(err.message) : '');
+        if (message.length > 0) {
+            return `Failed to inspect resolved source ${source}: ${message.split('\n')[0]}`;
+        }
+    }
+    return `Failed to inspect resolved source ${source}`;
+}
+async function inspectSkillsInstallSource(source) {
+    const tmpDir = await fs.mkdtemp(path.join(os.tmpdir(), 'skiller-plugin-source-'));
+    const repoDir = path.join(tmpDir, 'repo');
+    try {
+        await execFileAsync('git', [
+            'clone',
+            '--depth',
+            '1',
+            '--quiet',
+            normalizeCloneSource(source),
+            repoDir,
+        ]);
+        const { skills } = await (0, SkillsUtils_1.walkSkillsTree)(repoDir);
+        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 };
+                }
+            }
+            catch {
+                // Keep scanning. One malformed skill should not hide valid siblings.
+            }
+        }
+        return {
+            installable: false,
+            reason: `Resolved source ${source} has no valid SKILL.md files with name and description`,
+        };
+    }
+    catch (err) {
+        return {
+            installable: false,
+            reason: formatInspectionError(source, err),
+        };
+    }
+    finally {
+        await fs.rm(tmpDir, { recursive: true, force: true });
+    }
+}
 async function readEnabledPluginIds(projectRoot) {
     const settingsPath = path.join(projectRoot, project_paths_1.LEGACY_SKILLER_DIR, 'settings.json');
     const raw = await readJsonFile(settingsPath);
@@ -178,7 +245,8 @@ function sortPlan(installsBySource, unresolved) {
         unresolved: [...unresolved].sort((a, b) => a.pluginId.localeCompare(b.pluginId)),
     };
 }
-async function planClaudePluginSkillsMigration(projectRoot) {
+async function planClaudePluginSkillsMigration(projectRoot, options = {}) {
+    const inspectSource = options.inspectSource ?? inspectSkillsInstallSource;
     const pluginIds = new Set([
         ...(await readEnabledPluginIds(projectRoot)),
         ...(await readManifestPluginIds(projectRoot)),
@@ -189,6 +257,7 @@ async function planClaudePluginSkillsMigration(projectRoot) {
     const installsBySource = new Map();
     const unresolved = [];
     const pluginCatalogCache = new Map();
+    const sourceInspectionCache = new Map();
     for (const pluginId of [...pluginIds].sort((a, b) => a.localeCompare(b))) {
         const parts = parsePluginId(pluginId);
         if (!parts) {
@@ -215,6 +284,19 @@ async function planClaudePluginSkillsMigration(projectRoot) {
             });
             continue;
         }
+        let inspection = sourceInspectionCache.get(resolvedSource);
+        if (!inspection) {
+            inspection = await inspectSource(resolvedSource);
+            sourceInspectionCache.set(resolvedSource, inspection);
+        }
+        if (!inspection.installable) {
+            unresolved.push({
+                pluginId,
+                reason: inspection.reason ??
+                    `Resolved source ${resolvedSource} is not installable through skills`,
+            });
+            continue;
+        }
         const existing = installsBySource.get(resolvedSource);
         if (existing) {
             existing.pluginIds.add(pluginId);

package/package.json

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