skiller 0.5.6 → 0.6.1

package/dist/core/ConfigLoader.js

@@ -78,6 +78,8 @@ const skillerConfigSchema = zod_1.z.object({
     skills: zod_1.z
         .object({
         enabled: zod_1.z.boolean().optional(),
+        generate_from_rules: zod_1.z.boolean().optional(),
+        prune: zod_1.z.boolean().optional(),
     })
         .optional(),
     rules: zod_1.z
@@ -234,6 +236,9 @@ async function loadConfig(options) {
     if (typeof rawSkillsSection.generate_from_rules === 'boolean') {
         skillsConfig.generate_from_rules = rawSkillsSection.generate_from_rules;
     }
+    if (typeof rawSkillsSection.prune === 'boolean') {
+        skillsConfig.prune = rawSkillsSection.prune;
+    }
     const rawRulesSection = raw.rules && typeof raw.rules === 'object' && !Array.isArray(raw.rules)
         ? raw.rules
         : {};

package/dist/core/SkillsProcessor.js

@@ -41,6 +41,7 @@ exports.propagateSkillsForSkillz = propagateSkillsForSkillz;
 exports.buildSkillzMcpConfig = buildSkillzMcpConfig;
 const path = __importStar(require("path"));
 const fs = __importStar(require("fs/promises"));
+const readline = __importStar(require("readline"));
 const constants_1 = require("../constants");
 const SkillsUtils_1 = require("./SkillsUtils");
 const FrontmatterParser_1 = require("./FrontmatterParser");
@@ -153,11 +154,106 @@ async function findMdcFiles(dir) {
     }
     return mdcFiles;
 }
+/**
+ * Prompts the user via readline to confirm deletion of orphaned skills.
+ * Returns true if user confirms, false otherwise.
+ */
+async function promptForPrune(orphanedSkills) {
+    // If not running in a TTY (e.g., CI/CD), skip prompting
+    if (!process.stdin.isTTY) {
+        (0, constants_1.logWarn)(`Found ${orphanedSkills.length} orphaned skill(s) but not in interactive mode. Set prune = true/false in skiller.toml to handle automatically.`, false);
+        return false;
+    }
+    const rl = readline.createInterface({
+        input: process.stdin,
+        output: process.stdout,
+    });
+    return new Promise((resolve) => {
+        console.log(`\nFound ${orphanedSkills.length} orphaned skill(s) not generated from rules:`);
+        orphanedSkills.forEach((skill) => console.log(`  - ${skill}`));
+        console.log('');
+        rl.question('Delete these orphaned skills? [y/N]: ', (answer) => {
+            rl.close();
+            const confirmed = answer.toLowerCase() === 'y' || answer.toLowerCase() === 'yes';
+            if (confirmed) {
+                console.log('');
+            }
+            else {
+                console.log("\nTip: Add 'prune = true' or 'prune = false' to [skills] in skiller.toml to avoid this prompt.\n");
+            }
+            resolve(confirmed);
+        });
+    });
+}
+/**
+ * Prunes orphaned skills (skills that exist in .claude/skills but are not generated from any .mdc file).
+ * @param skillsDir Path to the skills directory
+ * @param generatedSkillNames Set of skill names that were generated from .mdc files
+ * @param prune Prune setting: true=auto-delete, false=keep, undefined=prompt
+ * @param verbose Whether to log verbose output
+ * @param dryRun Whether to perform a dry run
+ */
+async function pruneOrphanedSkills(skillsDir, generatedSkillNames, prune, verbose, dryRun) {
+    // Check if skills directory exists
+    try {
+        await fs.access(skillsDir);
+    }
+    catch {
+        // No skills directory, nothing to prune
+        return;
+    }
+    // Get all existing skill directories
+    const entries = await fs.readdir(skillsDir, { withFileTypes: true });
+    const existingSkillDirs = entries
+        .filter((entry) => entry.isDirectory())
+        .map((entry) => entry.name);
+    // Find orphaned skills (exist but not in generatedSkillNames)
+    const orphanedSkills = existingSkillDirs.filter((name) => !generatedSkillNames.has(name));
+    if (orphanedSkills.length === 0) {
+        (0, constants_1.logVerboseInfo)('No orphaned skills found', verbose, dryRun);
+        return;
+    }
+    // Determine whether to delete based on prune setting
+    let shouldDelete = false;
+    if (prune === true) {
+        // Auto-delete
+        shouldDelete = true;
+        (0, constants_1.logVerboseInfo)(`Auto-pruning ${orphanedSkills.length} orphaned skill(s) (prune = true)`, verbose, dryRun);
+    }
+    else if (prune === false) {
+        // Keep orphans
+        (0, constants_1.logVerboseInfo)(`Keeping ${orphanedSkills.length} orphaned skill(s) (prune = false)`, verbose, dryRun);
+        return;
+    }
+    else {
+        // prune is undefined - prompt user
+        shouldDelete = await promptForPrune(orphanedSkills);
+        if (!shouldDelete) {
+            return;
+        }
+    }
+    // Delete orphaned skills
+    for (const skillName of orphanedSkills) {
+        const skillPath = path.join(skillsDir, skillName);
+        if (dryRun) {
+            (0, constants_1.logVerboseInfo)(`DRY RUN: Would remove orphaned skill ${skillName}`, verbose, dryRun);
+        }
+        else {
+            await fs.rm(skillPath, { recursive: true, force: true });
+            (0, constants_1.logVerboseInfo)(`Removed orphaned skill ${skillName}`, verbose, dryRun);
+        }
+    }
+}
 /**
  * Generates skills from .mdc rule files with frontmatter.
  * Creates skill files in the skills directory with @filename references to the original .mdc files.
+ * @param projectRoot Root directory of the project
+ * @param skillerDir Path to the skiller directory (.claude)
+ * @param verbose Whether to log verbose output
+ * @param dryRun Whether to perform a dry run
+ * @param prune Prune setting: true=auto-delete orphans, false=keep orphans, undefined=prompt
  */
-async function generateSkillsFromRules(projectRoot, skillerDir, verbose, dryRun) {
+async function generateSkillsFromRules(projectRoot, skillerDir, verbose, dryRun, prune) {
     // Determine skills directory based on skillerDir
     const skillsDir = path.join(skillerDir, 'skills');
     // Find all .mdc files in the skiller directory
@@ -168,6 +264,7 @@ async function generateSkillsFromRules(projectRoot, skillerDir, verbose, dryRun)
     }
     let generatedCount = 0;
     const skillsToRemove = [];
+    const generatedSkillNames = new Set();
     for (const mdcFile of mdcFiles) {
         // Read file content
         const content = await fs.readFile(mdcFile, 'utf8');
@@ -249,6 +346,7 @@ ${fileReference}
             }
         }
         generatedCount++;
+        generatedSkillNames.add(fileName);
     }
     if (generatedCount > 0) {
         (0, constants_1.logVerboseInfo)(`Generated ${generatedCount} skill(s) from .mdc files`, verbose, dryRun);
@@ -277,6 +375,8 @@ ${fileReference}
             }
         }
     }
+    // Prune orphaned skills if prune setting is configured
+    await pruneOrphanedSkills(skillsDir, generatedSkillNames, prune, verbose, dryRun);
 }
 /**
  * Cleans up skills directories (.claude/skills and .skillz) when skills are disabled.

package/dist/core/UnifiedConfigLoader.js

@@ -41,6 +41,19 @@ const FileSystemUtils = __importStar(require("./FileSystemUtils"));
 const FileSystemUtils_1 = require("./FileSystemUtils");
 const hash_1 = require("./hash");
 const RuleProcessor_1 = require("./RuleProcessor");
+/**
+ * Expand environment variables in a string.
+ * Supports ${VAR} syntax, replacing with process.env[VAR] or empty string if not found.
+ */
+function expandEnvVars(value) {
+    return value.replace(/\$\{([^}]+)\}/g, (_, key) => process.env[key] || '');
+}
+/**
+ * Expand environment variables in all values of a Record<string, string>.
+ */
+function expandEnvRecord(record) {
+    return Object.fromEntries(Object.entries(record).map(([k, v]) => [k, expandEnvVars(v)]));
+}
 async function loadUnifiedConfig(options) {
     // Resolve the effective .claude directory (local or global), mirroring the main loader behavior
     const resolvedSkillerDir = (await FileSystemUtils.findSkillerDir(options.projectRoot, true)) ||
@@ -99,6 +112,9 @@ async function loadUnifiedConfig(options) {
             if (typeof skillsObj.generate_from_rules === 'boolean') {
                 skillsConfig.generate_from_rules = skillsObj.generate_from_rules;
             }
+            if (typeof skillsObj.prune === 'boolean') {
+                skillsConfig.prune = skillsObj.prune;
+            }
         }
     }
     // Parse rules configuration
@@ -227,9 +243,10 @@ async function loadUnifiedConfig(options) {
                 if (Array.isArray(serverDef.args)) {
                     server.args = serverDef.args.map(String);
                 }
-                // Parse env
+                // Parse env with ${VAR} expansion
                 if (serverDef.env && typeof serverDef.env === 'object') {
-                    server.env = Object.fromEntries(Object.entries(serverDef.env).filter(([, v]) => typeof v === 'string'));
+                    const rawEnv = Object.fromEntries(Object.entries(serverDef.env).filter(([, v]) => typeof v === 'string'));
+                    server.env = expandEnvRecord(rawEnv);
                 }
                 // Parse URL and headers
                 if (typeof serverDef.url === 'string') {
@@ -350,7 +367,8 @@ async function loadUnifiedConfig(options) {
                     if (Array.isArray(def.args))
                         server.args = def.args.map(String);
                     if (def.env && typeof def.env === 'object') {
-                        server.env = Object.fromEntries(Object.entries(def.env).filter(([, v]) => typeof v === 'string'));
+                        const rawEnv = Object.fromEntries(Object.entries(def.env).filter(([, v]) => typeof v === 'string'));
+                        server.env = expandEnvRecord(rawEnv);
                     }
                     if (typeof def.url === 'string')
                         server.url = def.url;

package/dist/lib.js

@@ -107,10 +107,11 @@ async function applyAllAgentConfigs(projectRoot, includedAgents, configPath, cli
         if (skillsEnabledResolved) {
             const generateFromRules = rootConfig.skills?.generate_from_rules ?? false;
             if (generateFromRules) {
+                const pruneConfig = rootConfig.skills?.prune;
                 for (const configEntry of hierarchicalConfigs) {
                     const nestedRoot = path.dirname(configEntry.skillerDir);
                     (0, constants_1.logVerbose)(`Generating skills from .mdc files for nested directory: ${nestedRoot}`, verbose);
-                    await generateSkillsFromRules(nestedRoot, configEntry.skillerDir, verbose, dryRun);
+                    await generateSkillsFromRules(nestedRoot, configEntry.skillerDir, verbose, dryRun, pruneConfig);
                 }
             }
         }
@@ -141,7 +142,8 @@ async function applyAllAgentConfigs(projectRoot, includedAgents, configPath, cli
             const generateFromRules = singleConfig.config.skills?.generate_from_rules ?? false;
             if (generateFromRules) {
                 (0, constants_1.logVerbose)('Generating skills from .mdc files', verbose);
-                await generateSkillsFromRules(projectRoot, singleConfig.skillerDir, verbose, dryRun);
+                const pruneConfig = singleConfig.config.skills?.prune;
+                await generateSkillsFromRules(projectRoot, singleConfig.skillerDir, verbose, dryRun, pruneConfig);
             }
         }
         // Always call propagateSkills - it handles cleanup when disabled

package/package.json

@@ -1,12 +1,13 @@
 {
 	"name": "skiller",
-	"version": "0.5.6",
+	"version": "0.6.1",
 	"description": "Skiller — apply the same rules to all coding agents",
 	"main": "dist/lib.js",
 	"publishConfig": {
 		"access": "public"
 	},
 	"scripts": {
+		"postinstall": "[ -n \"$CI\" ] || npx skiller@latest apply",
 		"lint": "eslint \"src/**/*.{ts,tsx}\"",
 		"lint:fix": "eslint \"src/**/*.{ts,tsx}\" --fix",
 		"format": "prettier --write \"src/**/*.{ts,tsx,json,md}\"",
@@ -15,7 +16,7 @@
 		"test:coverage": "jest --coverage",
 		"test:integration": "jest tests/e2e/skiller.integration.test.ts --verbose",
 		"build": "tsc",
-		"release": "pnpm build && pnpm changeset version"
+		"release": "pnpm build && pnpm changeset publish"
 	},
 	"repository": {
 		"type": "git",
@@ -50,7 +51,8 @@
 		"skiller": "dist/cli/index.js"
 	},
 	"devDependencies": {
-		"@changesets/cli": "2.27.11",
+		"@changesets/changelog-github": "^0.5.2",
+		"@changesets/cli": "2.29.8",
 		"@eslint/js": "^9.39.1",
 		"@types/iarna__toml": "^2.0.5",
 		"@types/jest": "^29.5.14",