skiller 0.5.2 → 0.5.4

package/dist/core/GitignoreUtils.js

@@ -81,7 +81,11 @@ async function updateGitignore(projectRoot, paths) {
         return relative.replace(/\\/g, '/'); // Convert to POSIX format
     })
         .filter((p) => {
-        // Never include any path that resides inside a .claude directory (inputs, not outputs)
+        // Allow .claude/skills (generated output) but exclude other .claude/* paths (inputs)
+        if (p === '.claude/skills' || p === '/.claude/skills') {
+            return true;
+        }
+        // Never include any other path that resides inside a .claude directory (inputs, not outputs)
         return !p.includes('/.claude/') && !p.startsWith('.claude/');
     })
         .map((p) => {

package/dist/core/SkillsProcessor.js

@@ -66,40 +66,40 @@ async function discoverSkills(projectRoot, skillerDir) {
 }
 /**
  * Gets the paths that skills will generate, for gitignore purposes.
- * Returns empty array if skills directory doesn't exist.
+ * When generateFromRules is true, always includes .claude/skills even if it doesn't exist yet.
  */
 async function getSkillsGitignorePaths(projectRoot, options = {}) {
-    const claudeSkillsDir = path.join(projectRoot, constants_1.CLAUDE_SKILLS_PATH);
-    // Check if skills directory exists
-    let skillsExist = false;
-    try {
-        await fs.access(claudeSkillsDir);
-        skillsExist = true;
-    }
-    catch {
-        return [];
-    }
-    if (!skillsExist) {
-        return [];
-    }
     const paths = [];
     // Gitignore .claude/skills if:
-    // 1. generate_from_rules is explicitly true in config, OR
-    // 2. .claude/rules directory exists (skills are generated from rules)
+    // 1. generate_from_rules is explicitly true in config (always gitignore, even if dir doesn't exist)
+    // 2. OR .claude/rules directory exists (skills are generated from rules)
+    // 3. OR .claude/skills exists AND .claude/rules exists (legacy check)
     if (options.generateFromRules) {
-        // Config says skills are generated from rules
+        // Config says skills are generated from rules - always gitignore
         paths.push(path.join(projectRoot, constants_1.CLAUDE_SKILLS_PATH));
     }
     else {
-        // Check if .claude/rules exists (fallback for when config not passed)
-        const claudeRulesDir = path.join(projectRoot, '.claude', 'rules');
+        // Check if .claude/skills exists
+        const claudeSkillsDir = path.join(projectRoot, constants_1.CLAUDE_SKILLS_PATH);
+        let skillsExist = false;
         try {
-            await fs.access(claudeRulesDir);
-            // .claude/rules exists, so .claude/skills is generated
-            paths.push(path.join(projectRoot, constants_1.CLAUDE_SKILLS_PATH));
+            await fs.access(claudeSkillsDir);
+            skillsExist = true;
         }
         catch {
-            // .claude/rules doesn't exist, so .claude/skills is versioned (don't gitignore)
+            // Skills directory doesn't exist
+        }
+        if (skillsExist) {
+            // 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)

package/package.json

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