skiller 0.7.1 → 0.7.2

package/dist/core/SkillsProcessor.js

@@ -39,6 +39,7 @@ exports.discoverSkills = discoverSkills;
 exports.getSkillsGitignorePaths = getSkillsGitignorePaths;
 exports.propagateSkills = propagateSkills;
 exports.copySkillFoldersFromRules = copySkillFoldersFromRules;
+exports.copyMdcFilesFromRules = copyMdcFilesFromRules;
 const path = __importStar(require("path"));
 const fs = __importStar(require("fs/promises"));
 const yaml = __importStar(require("js-yaml"));
@@ -140,8 +141,15 @@ async function syncMdcToSkillMd(skillsDir, verbose, dryRun) {
         }
         try {
             if (siblingMdcContent !== null && skillMdContent === null) {
-                // Case 1: Sibling .mdc exists but no SKILL.md → generate SKILL.md with @reference
+                // Case 1: Sibling .mdc exists but no SKILL.md
                 const { frontmatter: mdcFrontmatter } = (0, FrontmatterParser_1.parseFrontmatter)(siblingMdcContent);
+                // Skip SKILL.md generation for .mdc files with alwaysApply: true
+                // These are Cursor-style rules, not Claude Code skills
+                if (mdcFrontmatter?.alwaysApply === true) {
+                    (0, constants_1.logVerboseInfo)(`Skipping SKILL.md generation for ${skillName} (alwaysApply rule)`, verbose, dryRun);
+                    continue;
+                }
+                // Generate SKILL.md with @reference (absolute path)
                 const skillFrontmatter = {
                     name: skillName,
                     description: mdcFrontmatter?.description || `Skill: ${skillName}`,
@@ -150,14 +158,14 @@ async function syncMdcToSkillMd(skillsDir, verbose, dryRun) {
 ${yaml.dump(skillFrontmatter, { lineWidth: -1, noRefs: true }).trim()}
 ---
 
-@./${skillName}.mdc
+@.claude/skills/${skillName}/${skillName}.mdc
 `;
                 if (dryRun) {
                     (0, constants_1.logVerboseInfo)(`DRY RUN: Would generate ${skillName}/SKILL.md with @reference`, verbose, dryRun);
                 }
                 else {
                     await fs.writeFile(skillMdPath, newSkillMd, 'utf8');
-                    (0, constants_1.logVerboseInfo)(`Generated ${skillName}/SKILL.md with @./${skillName}.mdc reference`, verbose, dryRun);
+                    (0, constants_1.logVerboseInfo)(`Generated ${skillName}/SKILL.md with @.claude/skills/${skillName}/${skillName}.mdc reference`, verbose, dryRun);
                 }
                 synced.push(skillName);
             }
@@ -167,22 +175,32 @@ ${yaml.dump(skillFrontmatter, { lineWidth: -1, noRefs: true }).trim()}
                 const refCheck = isReferenceBody(skillBody);
                 if (refCheck.isReference) {
                     // Case 2: SKILL.md is @reference → source file is truth
-                    if (refCheck.referencePath === `./${skillName}.mdc`) {
+                    // Check for both relative and absolute sibling reference patterns
+                    const isRelativeSiblingRef = refCheck.referencePath === `./${skillName}.mdc`;
+                    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
-                            if (mdcFrontmatter?.description &&
-                                mdcFrontmatter.description !== skillFrontmatter?.description) {
+                            // 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,
+                                    description: mdcFrontmatter?.description ||
+                                        skillFrontmatter?.description ||
+                                        `Skill: ${skillName}`,
                                 };
                                 const newSkillMd = `---
 ${yaml.dump(newFrontmatter, { lineWidth: -1, noRefs: true }).trim()}
 ---
 
-@./${skillName}.mdc
+@.claude/skills/${skillName}/${skillName}.mdc
 `;
                                 if (dryRun) {
                                     (0, constants_1.logVerboseInfo)(`DRY RUN: Would update ${skillName}/SKILL.md frontmatter from .mdc`, verbose, dryRun);
@@ -215,8 +233,7 @@ ${yaml.dump(newFrontmatter, { lineWidth: -1, noRefs: true }).trim()}
                             const { frontmatter: refFrontmatter, body: refBody } = (0, FrontmatterParser_1.parseFrontmatter)(referencedContent);
                             // Create sibling .mdc with the content
                             let mdcContent;
-                            if (refFrontmatter &&
-                                Object.keys(refFrontmatter).length > 0) {
+                            if (refFrontmatter && Object.keys(refFrontmatter).length > 0) {
                                 const mdcFrontmatterData = {};
                                 if (refFrontmatter.description) {
                                     mdcFrontmatterData.description = refFrontmatter.description;
@@ -242,7 +259,7 @@ ${refBody}
                             else {
                                 mdcContent = referencedContent;
                             }
-                            // Update SKILL.md to point to sibling .mdc
+                            // Update SKILL.md to point to sibling .mdc (absolute path)
                             const newFrontmatter = {
                                 name: skillFrontmatter?.name || skillName,
                                 description: refFrontmatter?.description ||
@@ -253,7 +270,7 @@ ${refBody}
 ${yaml.dump(newFrontmatter, { lineWidth: -1, noRefs: true }).trim()}
 ---
 
-@./${skillName}.mdc
+@.claude/skills/${skillName}/${skillName}.mdc
 `;
                             if (dryRun) {
                                 (0, constants_1.logVerboseInfo)(`DRY RUN: Would migrate ${skillName} from ${refCheck.referencePath} to sibling pattern`, verbose, dryRun);
@@ -290,7 +307,7 @@ ${skillBody}
                     else {
                         mdcContent = skillBody;
                     }
-                    // Update SKILL.md to @reference
+                    // Update SKILL.md to @reference (absolute path)
                     const newSkillFrontmatter = {
                         name: skillFrontmatter?.name || skillName,
                         description: skillFrontmatter?.description || `Skill: ${skillName}`,
@@ -299,7 +316,7 @@ ${skillBody}
 ${yaml.dump(newSkillFrontmatter, { lineWidth: -1, noRefs: true }).trim()}
 ---
 
-@./${skillName}.mdc
+@.claude/skills/${skillName}/${skillName}.mdc
 `;
                     if (dryRun) {
                         (0, constants_1.logVerboseInfo)(`DRY RUN: Would generate ${skillName}/${skillName}.mdc and update SKILL.md`, verbose, dryRun);
@@ -465,3 +482,48 @@ async function copySkillFoldersFromRules(skillerDir, verbose, dryRun) {
     }
     (0, constants_1.logVerboseInfo)(`Copied ${skillFolders.length} skill folder(s) from rules to skills`, verbose, dryRun);
 }
+/**
+ * Copies standalone .mdc files from .claude/rules to .claude/skills/name/name.mdc.
+ * These are rule files (not skill folders) that should be available in the skills directory.
+ * No SKILL.md is generated - these remain as .mdc files only.
+ */
+async function copyMdcFilesFromRules(skillerDir, verbose, dryRun) {
+    const rulesDir = path.join(skillerDir, 'rules');
+    const skillsDir = path.join(skillerDir, 'skills');
+    const copiedNames = [];
+    // Check if rules directory exists
+    try {
+        await fs.access(rulesDir);
+    }
+    catch {
+        return copiedNames;
+    }
+    const entries = await fs.readdir(rulesDir, { withFileTypes: true });
+    // Find .mdc files at rules root (not in subdirectories)
+    const mdcFiles = entries.filter((e) => e.isFile() && e.name.endsWith('.mdc'));
+    for (const mdcFile of mdcFiles) {
+        const skillName = path.basename(mdcFile.name, '.mdc');
+        const sourcePath = path.join(rulesDir, mdcFile.name);
+        const targetDir = path.join(skillsDir, skillName);
+        const targetPath = path.join(targetDir, mdcFile.name);
+        try {
+            if (dryRun) {
+                (0, constants_1.logVerboseInfo)(`DRY RUN: Would copy ${mdcFile.name} from rules to skills/${skillName}/${mdcFile.name}`, verbose, dryRun);
+            }
+            else {
+                await fs.mkdir(targetDir, { recursive: true });
+                const content = await fs.readFile(sourcePath, 'utf8');
+                await fs.writeFile(targetPath, content, 'utf8');
+                (0, constants_1.logVerboseInfo)(`Copied ${mdcFile.name} from rules to skills/${skillName}/${mdcFile.name}`, verbose, dryRun);
+            }
+            copiedNames.push(skillName);
+        }
+        catch (err) {
+            (0, constants_1.logWarn)(`Failed to copy ${mdcFile.name}: ${err.message}`, dryRun);
+        }
+    }
+    if (copiedNames.length > 0) {
+        (0, constants_1.logVerboseInfo)(`Copied ${copiedNames.length} .mdc file(s) from rules to skills`, verbose, dryRun);
+    }
+    return copiedNames;
+}

package/dist/lib.js

@@ -102,11 +102,12 @@ async function applyAllAgentConfigs(projectRoot, includedAgents, configPath, cli
         (0, constants_1.logVerbose)(`Selected ${selectedAgents.length} agents: ${selectedAgents.map((a) => a.getName()).join(', ')}`, verbose);
         // Propagate skills (or cleanup if disabled) - do this for each nested directory
         const skillsEnabledResolved = resolveSkillsEnabled(skillsEnabled, rootConfig.skills?.enabled);
-        const { propagateSkills, copySkillFoldersFromRules } = await Promise.resolve().then(() => __importStar(require('./core/SkillsProcessor')));
-        // Copy skill folders from rules to skills
+        const { propagateSkills, copySkillFoldersFromRules, copyMdcFilesFromRules, } = await Promise.resolve().then(() => __importStar(require('./core/SkillsProcessor')));
+        // Copy skill folders and .mdc files from rules to skills
         if (skillsEnabledResolved) {
             for (const configEntry of hierarchicalConfigs) {
                 await copySkillFoldersFromRules(configEntry.skillerDir, verbose, dryRun);
+                await copyMdcFilesFromRules(configEntry.skillerDir, verbose, dryRun);
             }
         }
         // Propagate skills for each nested .claude directory (or cleanup if disabled)
@@ -130,10 +131,11 @@ async function applyAllAgentConfigs(projectRoot, includedAgents, configPath, cli
         (0, constants_1.logVerbose)(`Selected ${selectedAgents.length} agents: ${selectedAgents.map((a) => a.getName()).join(', ')}`, verbose);
         // Propagate skills (or cleanup if disabled)
         const skillsEnabledResolved = resolveSkillsEnabled(skillsEnabled, singleConfig.config.skills?.enabled);
-        const { propagateSkills, copySkillFoldersFromRules } = await Promise.resolve().then(() => __importStar(require('./core/SkillsProcessor')));
-        // Copy skill folders from rules to skills
+        const { propagateSkills, copySkillFoldersFromRules, copyMdcFilesFromRules, } = await Promise.resolve().then(() => __importStar(require('./core/SkillsProcessor')));
+        // Copy skill folders and .mdc files from rules to skills
         if (skillsEnabledResolved) {
             await copySkillFoldersFromRules(singleConfig.skillerDir, verbose, dryRun);
+            await copyMdcFilesFromRules(singleConfig.skillerDir, verbose, dryRun);
         }
         // Always call propagateSkills - it handles cleanup when disabled
         await propagateSkills(projectRoot, selectedAgents, skillsEnabledResolved, verbose, dryRun, singleConfig.skillerDir);

package/package.json

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