skiller 0.7.4 → 0.7.6

package/dist/core/FrontmatterParser.js

@@ -61,6 +61,7 @@ function parseFrontmatter(content) {
         // No frontmatter found
         return {
             frontmatter: null,
+            rawFrontmatter: null,
             body: content,
         };
     }
@@ -106,6 +107,7 @@ function parseFrontmatter(content) {
         // YAML parsing failed - treat as no frontmatter
         return {
             frontmatter: null,
+            rawFrontmatter: null,
             body: content,
         };
     }
@@ -120,6 +122,7 @@ function extractFrontmatter(parsed, body) {
     if (!parsed) {
         return {
             frontmatter: {},
+            rawFrontmatter: null,
             body: body.trim(),
         };
     }
@@ -146,6 +149,7 @@ function extractFrontmatter(parsed, body) {
     }
     return {
         frontmatter,
+        rawFrontmatter: parsed,
         body: body.trim(),
     };
 }

package/dist/core/SkillsProcessor.js

@@ -152,10 +152,15 @@ async function syncMdcToSkillMd(skillsDir, verbose, dryRun) {
                     continue;
                 }
                 // Generate SKILL.md with @reference (absolute path)
+                // Keep all frontmatter from .mdc except globs and alwaysApply
                 const skillFrontmatter = {
                     name: skillName,
-                    description: mdcFrontmatter?.description || `Skill: ${skillName}`,
+                    ...Object.fromEntries(Object.entries(mdcFrontmatter || {}).filter(([key]) => key !== 'globs' && key !== 'alwaysApply')),
                 };
+                // Ensure description has a default
+                if (!skillFrontmatter.description) {
+                    skillFrontmatter.description = `Skill: ${skillName}`;
+                }
                 const newSkillMd = `---
 ${yaml.dump(skillFrontmatter, { lineWidth: -1, noRefs: true }).trim()}
 ---
@@ -189,7 +194,7 @@ ${yaml.dump(skillFrontmatter, { lineWidth: -1, noRefs: true }).trim()}
                     }
                 }
                 // SKILL.md exists - check if it's a reference
-                const { frontmatter: skillFrontmatter, body: skillBody } = (0, FrontmatterParser_1.parseFrontmatter)(skillMdContent);
+                const { frontmatter: skillFrontmatter, rawFrontmatter: skillRawFrontmatter, body: skillBody, } = (0, FrontmatterParser_1.parseFrontmatter)(skillMdContent);
                 const refCheck = isReferenceBody(skillBody);
                 if (refCheck.isReference) {
                     // Case 2: SKILL.md is @reference → source file is truth
@@ -198,38 +203,25 @@ ${yaml.dump(skillFrontmatter, { lineWidth: -1, noRefs: true }).trim()}
                     const isAbsoluteSiblingRef = refCheck.referencePath ===
                         `.claude/skills/${skillName}/${skillName}.mdc`;
                     if (isRelativeSiblingRef || isAbsoluteSiblingRef) {
-                        // Sibling reference pattern - validate/update frontmatter if .mdc changed
-                        if (siblingMdcContent !== null) {
-                            const { frontmatter: mdcFrontmatter } = (0, FrontmatterParser_1.parseFrontmatter)(siblingMdcContent);
-                            // Update SKILL.md frontmatter if description changed in .mdc
-                            // Also migrate from relative to absolute path if needed
-                            const needsUpdate = (mdcFrontmatter?.description &&
-                                mdcFrontmatter.description !==
-                                    skillFrontmatter?.description) ||
-                                isRelativeSiblingRef; // Migrate old relative refs to absolute
-                            if (needsUpdate) {
-                                const newFrontmatter = {
-                                    name: skillFrontmatter?.name || skillName,
-                                    description: mdcFrontmatter?.description ||
-                                        skillFrontmatter?.description ||
-                                        `Skill: ${skillName}`,
-                                };
-                                const newSkillMd = `---
-${yaml.dump(newFrontmatter, { lineWidth: -1, noRefs: true }).trim()}
+                        // Sibling reference pattern - only migrate path if needed (don't touch frontmatter)
+                        if (isRelativeSiblingRef) {
+                            // Migrate old relative refs to absolute path, preserving existing frontmatter
+                            const newSkillMd = `---
+${yaml.dump(skillFrontmatter || { name: skillName }, { lineWidth: -1, noRefs: true }).trim()}
 ---
 
 @.claude/skills/${skillName}/${skillName}.mdc
 `;
-                                if (dryRun) {
-                                    (0, constants_1.logVerboseInfo)(`DRY RUN: Would update ${skillName}/SKILL.md frontmatter from .mdc`, verbose, dryRun);
-                                }
-                                else {
-                                    await fs.writeFile(skillMdPath, newSkillMd, 'utf8');
-                                    (0, constants_1.logVerboseInfo)(`Updated ${skillName}/SKILL.md frontmatter from .mdc`, verbose, dryRun);
-                                }
-                                synced.push(skillName);
+                            if (dryRun) {
+                                (0, constants_1.logVerboseInfo)(`DRY RUN: Would migrate ${skillName}/SKILL.md to absolute path`, verbose, dryRun);
                             }
+                            else {
+                                await fs.writeFile(skillMdPath, newSkillMd, 'utf8');
+                                (0, constants_1.logVerboseInfo)(`Migrated ${skillName}/SKILL.md to absolute path`, verbose, dryRun);
+                            }
+                            synced.push(skillName);
                         }
+                        // If already absolute path, nothing to do - SKILL.md is source of truth for frontmatter
                     }
                     else if (refCheck.referencePath) {
                         // Pre-0.7 pattern or other external reference - migrate to sibling pattern
@@ -304,10 +296,15 @@ ${yaml.dump(newFrontmatter, { lineWidth: -1, noRefs: true }).trim()}
                     // Generate .mdc from SKILL.md body (no frontmatter needed - description is in SKILL.md)
                     const mdcContent = skillBody;
                     // Update SKILL.md to @reference (absolute path)
-                    const newSkillFrontmatter = {
-                        name: skillFrontmatter?.name || skillName,
-                        description: skillFrontmatter?.description || `Skill: ${skillName}`,
-                    };
+                    // Preserve ALL existing frontmatter (use rawFrontmatter to keep custom fields like user-invocable)
+                    // Only add defaults for missing name/description
+                    const newSkillFrontmatter = skillRawFrontmatter ? { ...skillRawFrontmatter } : {};
+                    if (!newSkillFrontmatter.name) {
+                        newSkillFrontmatter.name = skillName;
+                    }
+                    if (!newSkillFrontmatter.description) {
+                        newSkillFrontmatter.description = `Skill: ${skillName}`;
+                    }
                     const newSkillMd = `---
 ${yaml.dump(newSkillFrontmatter, { lineWidth: -1, noRefs: true }).trim()}
 ---

package/package.json

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