skiller 0.5.0 → 0.5.1

package/dist/core/SkillsProcessor.js

@@ -68,7 +68,7 @@ async function discoverSkills(projectRoot, skillerDir) {
  * Gets the paths that skills will generate, for gitignore purposes.
  * Returns empty array if skills directory doesn't exist.
  */
-async function getSkillsGitignorePaths(projectRoot) {
+async function getSkillsGitignorePaths(projectRoot, options = {}) {
     const claudeSkillsDir = path.join(projectRoot, constants_1.CLAUDE_SKILLS_PATH);
     // Check if skills directory exists
     let skillsExist = false;
@@ -83,16 +83,24 @@ async function getSkillsGitignorePaths(projectRoot) {
         return [];
     }
     const paths = [];
-    // When using .claude/skills, check if it's generated from .claude/rules
-    // If .claude/rules exists, then .claude/skills is generated and should be gitignored
-    const claudeRulesDir = path.join(projectRoot, '.claude', 'rules');
-    try {
-        await fs.access(claudeRulesDir);
-        // .claude/rules exists, so .claude/skills is generated
+    // Gitignore .claude/skills if:
+    // 1. generate_from_rules is explicitly true in config, OR
+    // 2. .claude/rules directory exists (skills are generated from rules)
+    if (options.generateFromRules) {
+        // Config says skills are generated from rules
         paths.push(path.join(projectRoot, constants_1.CLAUDE_SKILLS_PATH));
     }
-    catch {
-        // .claude/rules doesn't exist, so .claude/skills is versioned (don't gitignore)
+    else {
+        // Check if .claude/rules exists (fallback for when config not passed)
+        const claudeRulesDir = path.join(projectRoot, '.claude', 'rules');
+        try {
+            await fs.access(claudeRulesDir);
+            // .claude/rules exists, so .claude/skills is generated
+            paths.push(path.join(projectRoot, constants_1.CLAUDE_SKILLS_PATH));
+        }
+        catch {
+            // .claude/rules doesn't exist, so .claude/skills is versioned (don't gitignore)
+        }
     }
     // Always gitignore .skillz (for MCP agents)
     paths.push(path.join(projectRoot, constants_1.SKILLZ_DIR));

package/dist/core/apply-engine.js

@@ -408,10 +408,16 @@ async function handleMcpConfiguration(agent, agentConfig, config, skillerMcpJson
             typeof filteredMcpJson.mcpServers === 'object') {
             const mcpServers = { ...filteredMcpJson.mcpServers };
             delete mcpServers[SKILLZ_MCP_SERVER_NAME];
-            filteredMcpJson = {
-                ...filteredMcpJson,
-                mcpServers,
-            };
+            // If no servers remain after removal, set to null to skip file generation
+            if (Object.keys(mcpServers).length === 0) {
+                filteredMcpJson = null;
+            }
+            else {
+                filteredMcpJson = {
+                    ...filteredMcpJson,
+                    mcpServers,
+                };
+            }
             (0, constants_1.logVerboseInfo)(`Removed Skillz MCP server for ${agent.getName()} (has native skills support)`, verbose, dryRun);
         }
     }
@@ -448,12 +454,19 @@ async function handleMcpConfiguration(agent, agentConfig, config, skillerMcpJson
             // No .skillz directory, skip adding Skillz server
         }
     }
-    if (!filteredMcpJson) {
+    // Skip if no MCP config or if mcpServers is empty
+    const hasServers = filteredMcpJson &&
+        filteredMcpJson.mcpServers &&
+        typeof filteredMcpJson.mcpServers === 'object' &&
+        Object.keys(filteredMcpJson.mcpServers).length >
+            0;
+    if (!hasServers) {
         (0, constants_1.logVerbose)(`No compatible MCP servers found for ${agent.getName()} - skipping MCP configuration`, verbose);
         return;
     }
     await updateGitignoreForMcpFile(dest, projectRoot, generatedPaths, backup);
-    await applyMcpConfiguration(agent, filteredMcpJson, dest, agentConfig, config, projectRoot, cliMcpStrategy, dryRun, verbose, backup);
+    await applyMcpConfiguration(agent, filteredMcpJson, // Safe: hasServers check above ensures this is non-null
+    dest, agentConfig, config, projectRoot, cliMcpStrategy, dryRun, verbose, backup);
 }
 async function updateGitignoreForMcpFile(dest, projectRoot, generatedPaths, backup = true) {
     if (dest.startsWith(projectRoot)) {

package/dist/lib.js

@@ -156,7 +156,10 @@ async function applyAllAgentConfigs(projectRoot, includedAgents, configPath, cli
     if (skillsEnabledForGitignore) {
         // Skills enabled by default or explicitly
         const { getSkillsGitignorePaths } = await Promise.resolve().then(() => __importStar(require('./core/SkillsProcessor')));
-        const skillsPaths = await getSkillsGitignorePaths(projectRoot);
+        const generateFromRules = loadedConfig.skills?.generate_from_rules ?? false;
+        const skillsPaths = await getSkillsGitignorePaths(projectRoot, {
+            generateFromRules,
+        });
         allGeneratedPaths = [...generatedPaths, ...skillsPaths];
     }
     await (0, apply_engine_1.updateGitignore)(projectRoot, allGeneratedPaths, loadedConfig, cliGitignoreEnabled, dryRun);

package/package.json

@@ -1,12 +1,13 @@
 {
 	"name": "skiller",
-	"version": "0.5.0",
+	"version": "0.5.1",
 	"description": "Skiller — apply the same rules to all coding agents",
 	"main": "dist/lib.js",
 	"publishConfig": {
 		"access": "public"
 	},
 	"scripts": {
+		"postinstall": "npx skiller@latest apply",
 		"lint": "eslint \"src/**/*.{ts,tsx}\"",
 		"lint:fix": "eslint \"src/**/*.{ts,tsx}\" --fix",
 		"format": "prettier --write \"src/**/*.{ts,tsx,json,md}\"",