skiller 0.9.2 → 0.9.4

package/dist/cli/handlers.js

@@ -323,12 +323,15 @@ async function migrateClaudePluginsHandler(argv) {
             console.log('[skiller] Run again with --execute to install the resolved repos through skills.');
             return;
         }
-        if (plan.unresolved.length > 0) {
-            throw new Error(`Cannot execute migration until all plugins resolve:\n${plan.unresolved.map((entry) => `- ${entry.pluginId}: ${entry.reason}`).join('\n')}`);
+        if (plan.installs.length === 0 && plan.unresolved.length > 0) {
+            throw new Error(`Cannot execute migration because no installable plugin repos were found:\n${plan.unresolved.map((entry) => `- ${entry.pluginId}: ${entry.reason}`).join('\n')}`);
         }
         for (const install of plan.installs) {
             await (0, skills_cli_1.runSkillsCli)(projectRoot, buildClaudePluginMigrationArgs(install.source));
         }
+        if (plan.unresolved.length > 0) {
+            console.log(`[skiller] Skipped unresolved plugins:\n${plan.unresolved.map((entry) => `- ${entry.pluginId}: ${entry.reason}`).join('\n')}`);
+        }
         console.log('[skiller] Claude plugin repo migration completed. Remove the plugin entries from .claude/settings.json, then rerun skiller apply.');
     }
     catch (err) {

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.2",
+  "version": "0.9.4",
   "description": "Skiller — apply the same rules to all coding agents",
   "main": "dist/lib.js",
   "publishConfig": {