@intellectronica/ruler 0.3.31 → 0.3.33

package/dist/core/FileSystemUtils.js

@@ -101,7 +101,20 @@ async function readMarkdownFiles(rulerDir) {
         const entries = await fs_1.promises.readdir(dir, { withFileTypes: true });
         for (const entry of entries) {
             const fullPath = path.join(dir, entry.name);
-            if (entry.isDirectory()) {
+            // Resolve symlinks to determine actual type
+            let isDir = entry.isDirectory();
+            let isFile = entry.isFile();
+            if (entry.isSymbolicLink()) {
+                try {
+                    const stat = await fs_1.promises.stat(fullPath);
+                    isDir = stat.isDirectory();
+                    isFile = stat.isFile();
+                }
+                catch {
+                    continue; // skip broken symlinks
+                }
+            }
+            if (isDir) {
                 // Skip .ruler/skills; skills are propagated separately and should not be concatenated
                 const relativeFromRoot = path.relative(rulerDir, fullPath);
                 const isSkillsDir = relativeFromRoot === constants_1.SKILLS_DIR ||
@@ -111,7 +124,7 @@ async function readMarkdownFiles(rulerDir) {
                 }
                 await walk(fullPath);
             }
-            else if (entry.isFile() && entry.name.endsWith('.md')) {
+            else if (isFile && entry.name.endsWith('.md')) {
                 const content = await fs_1.promises.readFile(fullPath, 'utf8');
                 mdFiles.push({ path: fullPath, content });
             }

package/dist/core/SkillsProcessor.js

@@ -72,7 +72,7 @@ async function discoverSkills(projectRoot) {
  * 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, agents) {
     const skillsDir = path.join(projectRoot, constants_1.RULER_SKILLS_PATH);
     // Check if skills directory exists
     try {
@@ -83,19 +83,21 @@ async function getSkillsGitignorePaths(projectRoot) {
     }
     // Import here to avoid circular dependency
     const { CLAUDE_SKILLS_PATH, CODEX_SKILLS_PATH, OPENCODE_SKILLS_PATH, PI_SKILLS_PATH, GOOSE_SKILLS_PATH, VIBE_SKILLS_PATH, ROO_SKILLS_PATH, GEMINI_SKILLS_PATH, CURSOR_SKILLS_PATH, FACTORY_SKILLS_PATH, ANTIGRAVITY_SKILLS_PATH, } = await Promise.resolve().then(() => __importStar(require('../constants')));
-    return [
-        path.join(projectRoot, CLAUDE_SKILLS_PATH),
-        path.join(projectRoot, CODEX_SKILLS_PATH),
-        path.join(projectRoot, OPENCODE_SKILLS_PATH),
-        path.join(projectRoot, PI_SKILLS_PATH),
-        path.join(projectRoot, GOOSE_SKILLS_PATH),
-        path.join(projectRoot, VIBE_SKILLS_PATH),
-        path.join(projectRoot, ROO_SKILLS_PATH),
-        path.join(projectRoot, GEMINI_SKILLS_PATH),
-        path.join(projectRoot, CURSOR_SKILLS_PATH),
-        path.join(projectRoot, FACTORY_SKILLS_PATH),
-        path.join(projectRoot, ANTIGRAVITY_SKILLS_PATH),
-    ];
+    const selectedTargets = getSelectedSkillTargets(agents);
+    const targetPaths = {
+        claude: CLAUDE_SKILLS_PATH,
+        codex: CODEX_SKILLS_PATH,
+        opencode: OPENCODE_SKILLS_PATH,
+        pi: PI_SKILLS_PATH,
+        goose: GOOSE_SKILLS_PATH,
+        vibe: VIBE_SKILLS_PATH,
+        roo: ROO_SKILLS_PATH,
+        gemini: GEMINI_SKILLS_PATH,
+        cursor: CURSOR_SKILLS_PATH,
+        factory: FACTORY_SKILLS_PATH,
+        antigravity: ANTIGRAVITY_SKILLS_PATH,
+    };
+    return Array.from(selectedTargets).map((target) => path.join(projectRoot, targetPaths[target]));
 }
 /**
  * Module-level state to track if experimental warning has been shown.
@@ -115,6 +117,31 @@ function warnOnceExperimental(verbose, dryRun) {
     hasWarnedExperimental = true;
     (0, constants_1.logWarn)('Skills support is experimental and behavior may change in future releases.', dryRun);
 }
+const SKILL_TARGET_TO_IDENTIFIERS = new Map([
+    ['claude', ['claude', 'copilot', 'kilocode']],
+    ['codex', ['codex']],
+    ['opencode', ['opencode']],
+    ['pi', ['pi']],
+    ['goose', ['goose', 'amp']],
+    ['vibe', ['mistral']],
+    ['roo', ['roo']],
+    ['gemini', ['gemini-cli']],
+    ['cursor', ['cursor']],
+    ['factory', ['factory']],
+    ['antigravity', ['antigravity']],
+]);
+function getSelectedSkillTargets(agents) {
+    const selectedIdentifiers = new Set(agents
+        .filter((agent) => agent.supportsNativeSkills?.())
+        .map((agent) => agent.getIdentifier()));
+    const targets = new Set();
+    for (const [target, identifiers] of SKILL_TARGET_TO_IDENTIFIERS) {
+        if (identifiers.some((id) => selectedIdentifiers.has(id))) {
+            targets.add(target);
+        }
+    }
+    return targets;
+}
 /**
  * Cleans up skills directories when skills are disabled.
  * This ensures that stale skills from previous runs don't persist when skills are turned off.
@@ -330,28 +357,53 @@ async function propagateSkills(projectRoot, agents, skillsEnabled, verbose, dryR
     }
     // Warn about experimental features
     warnOnceExperimental(verbose, dryRun);
+    const selectedTargets = getSelectedSkillTargets(agents);
+    if (selectedTargets.size === 0) {
+        (0, constants_1.logVerboseInfo)('No selected agents require skills propagation, skipping skills propagation', verbose, dryRun);
+        return;
+    }
     // Copy to Claude skills directory if needed
-    if (hasNativeSkillsAgent) {
-        (0, constants_1.logVerboseInfo)(`Copying skills to ${constants_1.CLAUDE_SKILLS_PATH} for Claude Code, GitHub Copilot, and Kilo Code`, verbose, dryRun);
+    if (selectedTargets.has('claude')) {
+        (0, constants_1.logVerboseInfo)(`Copying skills to ${constants_1.CLAUDE_SKILLS_PATH} for Claude Code, GitHub Copilot, and KiloCode`, verbose, dryRun);
         await propagateSkillsForClaude(projectRoot, { dryRun });
+    }
+    if (selectedTargets.has('codex')) {
         (0, constants_1.logVerboseInfo)(`Copying skills to ${constants_1.CODEX_SKILLS_PATH} for OpenAI Codex CLI`, verbose, dryRun);
         await propagateSkillsForCodex(projectRoot, { dryRun });
+    }
+    if (selectedTargets.has('opencode')) {
         (0, constants_1.logVerboseInfo)(`Copying skills to ${constants_1.OPENCODE_SKILLS_PATH} for OpenCode`, verbose, dryRun);
         await propagateSkillsForOpenCode(projectRoot, { dryRun });
+    }
+    if (selectedTargets.has('pi')) {
         (0, constants_1.logVerboseInfo)(`Copying skills to ${constants_1.PI_SKILLS_PATH} for Pi Coding Agent`, verbose, dryRun);
         await propagateSkillsForPi(projectRoot, { dryRun });
+    }
+    if (selectedTargets.has('goose')) {
         (0, constants_1.logVerboseInfo)(`Copying skills to ${constants_1.GOOSE_SKILLS_PATH} for Goose and Amp`, verbose, dryRun);
         await propagateSkillsForGoose(projectRoot, { dryRun });
+    }
+    if (selectedTargets.has('vibe')) {
         (0, constants_1.logVerboseInfo)(`Copying skills to ${constants_1.VIBE_SKILLS_PATH} for Mistral Vibe`, verbose, dryRun);
         await propagateSkillsForVibe(projectRoot, { dryRun });
+    }
+    if (selectedTargets.has('roo')) {
         (0, constants_1.logVerboseInfo)(`Copying skills to ${constants_1.ROO_SKILLS_PATH} for Roo Code`, verbose, dryRun);
         await propagateSkillsForRoo(projectRoot, { dryRun });
+    }
+    if (selectedTargets.has('gemini')) {
         (0, constants_1.logVerboseInfo)(`Copying skills to ${constants_1.GEMINI_SKILLS_PATH} for Gemini CLI`, verbose, dryRun);
         await propagateSkillsForGemini(projectRoot, { dryRun });
+    }
+    if (selectedTargets.has('cursor')) {
         (0, constants_1.logVerboseInfo)(`Copying skills to ${constants_1.CURSOR_SKILLS_PATH} for Cursor`, verbose, dryRun);
         await propagateSkillsForCursor(projectRoot, { dryRun });
+    }
+    if (selectedTargets.has('factory')) {
         (0, constants_1.logVerboseInfo)(`Copying skills to ${constants_1.FACTORY_SKILLS_PATH} for Factory Droid`, verbose, dryRun);
         await propagateSkillsForFactory(projectRoot, { dryRun });
+    }
+    if (selectedTargets.has('antigravity')) {
         (0, constants_1.logVerboseInfo)(`Copying skills to ${constants_1.ANTIGRAVITY_SKILLS_PATH} for Antigravity`, verbose, dryRun);
         await propagateSkillsForAntigravity(projectRoot, { dryRun });
     }

package/dist/lib.js

@@ -125,7 +125,7 @@ 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 skillsPaths = await getSkillsGitignorePaths(projectRoot, selectedAgents);
         allGeneratedPaths = [...generatedPaths, ...skillsPaths];
     }
     await (0, apply_engine_1.updateGitignore)(projectRoot, allGeneratedPaths, loadedConfig, cliGitignoreEnabled, dryRun);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@intellectronica/ruler",
-  "version": "0.3.31",
+  "version": "0.3.33",
   "description": "Ruler — apply the same rules to all coding agents",
   "main": "dist/lib.js",
   "scripts": {