@intellectronica/ruler 0.3.24 → 0.3.25

package/README.md

@@ -633,7 +633,7 @@ For agents that support MCP but don't have native skills support, Ruler automati
 
 1. Copies skills to `.skillz/` directory
 2. Configures a Skillz MCP server in the agent's configuration
-3. Uses `uvx` to launch the server with the absolute path to `.skillz`
+3. Uses `uvx` to launch the server with the project-relative path to `.skillz`
 
 Agents using native skills support (Claude Code, GitHub Copilot, Kilo Code, OpenAI Codex CLI, OpenCode, Pi Coding Agent, Goose, Amp, Mistral Vibe, Roo Code, Gemini CLI, and Cursor) **do not** use the Skillz MCP server and instead use their own native skills directories.
 
@@ -642,7 +642,7 @@ Example auto-generated MCP server configuration:
 ```toml
 [mcp_servers.skillz]
 command = "uvx"
-args = ["skillz@latest", "/absolute/path/to/project/.skillz"]
+args = ["skillz@latest", ".skillz"]
 ```
 
 ### `.gitignore` Integration

package/dist/core/SkillsProcessor.js

@@ -808,7 +808,7 @@ async function propagateSkillsForSkillz(projectRoot, options) {
     if (options.dryRun) {
         return [
             `Copy skills from ${constants_1.RULER_SKILLS_PATH} to ${constants_1.SKILLZ_DIR}`,
-            `Configure Skillz MCP server with absolute path to ${constants_1.SKILLZ_DIR}`,
+            `Configure Skillz MCP server with path to ${constants_1.SKILLZ_DIR}`,
         ];
     }
     // Use atomic replace: copy to temp, then rename
@@ -843,11 +843,11 @@ async function propagateSkillsForSkillz(projectRoot, options) {
  * Builds MCP config for Skillz server.
  */
 function buildSkillzMcpConfig(projectRoot) {
-    const skillzAbsPath = path.resolve(projectRoot, constants_1.SKILLZ_DIR);
+    void projectRoot;
     return {
         [constants_1.SKILLZ_MCP_SERVER_NAME]: {
             command: 'uvx',
-            args: ['skillz@latest', skillzAbsPath],
+            args: ['skillz@latest', constants_1.SKILLZ_DIR],
         },
     };
 }

package/dist/core/apply-engine.js

@@ -260,13 +260,12 @@ async function processSingleConfiguration(agents, configuration, projectRoot, ve
     return await applyConfigurationsToAgents(agents, configuration.concatenatedRules, configuration.rulerMcpJson, configuration.config, projectRoot, verbose, dryRun, cliMcpEnabled, cliMcpStrategy, backup, skillsEnabled);
 }
 /**
- * Adds Skillz MCP server to rulerMcpJson if skills exist and any agent needs it.
+ * Adds Skillz MCP server to rulerMcpJson if skills exist and this agent needs it.
  * Returns augmented MCP config or original if no changes needed.
  */
-async function addSkillzMcpServerIfNeeded(rulerMcpJson, projectRoot, agents, verbose) {
-    // Check if any agent supports MCP stdio but not native skills
-    const hasAgentNeedingSkillz = agents.some((agent) => agent.supportsMcpStdio?.() && !agent.supportsNativeSkills?.());
-    if (!hasAgentNeedingSkillz) {
+async function addSkillzMcpServerIfNeeded(rulerMcpJson, projectRoot, agent, verbose) {
+    // Check if this agent supports MCP stdio but not native skills
+    if (!agent.supportsMcpStdio?.() || agent.supportsNativeSkills?.()) {
         return rulerMcpJson;
     }
     // Check if .skillz directory exists
@@ -280,7 +279,7 @@ async function addSkillzMcpServerIfNeeded(rulerMcpJson, projectRoot, agents, ver
         // Initialize empty config if null
         const baseConfig = rulerMcpJson || { mcpServers: {} };
         const mcpServers = baseConfig.mcpServers || {};
-        (0, constants_1.logVerbose)('Adding Skillz MCP server to configuration for agents that need it', verbose);
+        (0, constants_1.logVerbose)(`Adding Skillz MCP server to configuration for ${agent.getName()}`, verbose);
         return {
             ...baseConfig,
             mcpServers: {
@@ -308,17 +307,13 @@ async function addSkillzMcpServerIfNeeded(rulerMcpJson, projectRoot, agents, ver
 async function applyConfigurationsToAgents(agents, concatenatedRules, rulerMcpJson, config, projectRoot, verbose, dryRun, cliMcpEnabled = true, cliMcpStrategy, backup = true, skillsEnabled = true) {
     const generatedPaths = [];
     let agentsMdWritten = false;
-    // Add Skillz MCP server to rulerMcpJson if skills are enabled
-    // This must happen before calling agent.applyRulerConfig() so that agents
-    // that handle MCP internally (e.g. Codex, Gemini) receive the Skillz server
-    let augmentedRulerMcpJson = rulerMcpJson;
-    if (skillsEnabled && !dryRun) {
-        augmentedRulerMcpJson = await addSkillzMcpServerIfNeeded(rulerMcpJson, projectRoot, agents, verbose);
-    }
     for (const agent of agents) {
         (0, constants_1.logInfo)(`Applying rules for ${agent.getName()}...`, dryRun);
         (0, constants_1.logVerbose)(`Processing agent: ${agent.getName()}`, verbose);
         const agentConfig = config.agentConfigs[agent.getIdentifier()];
+        const agentRulerMcpJson = skillsEnabled && !dryRun
+            ? await addSkillzMcpServerIfNeeded(rulerMcpJson, projectRoot, agent, verbose)
+            : rulerMcpJson;
         // Collect output paths for .gitignore
         const outputPaths = (0, agent_utils_1.getAgentOutputPaths)(agent, projectRoot, agentConfig);
         (0, constants_1.logVerbose)(`Agent ${agent.getName()} output paths: ${outputPaths.join(', ')}`, verbose);
@@ -344,7 +339,7 @@ async function applyConfigurationsToAgents(agents, concatenatedRules, rulerMcpJs
                 }
             }
             let finalAgentConfig = agentConfig;
-            if (agent.getIdentifier() === 'augmentcode' && augmentedRulerMcpJson) {
+            if (agent.getIdentifier() === 'augmentcode' && agentRulerMcpJson) {
                 const resolvedStrategy = cliMcpStrategy ??
                     agentConfig?.mcp?.strategy ??
                     config.mcp?.strategy ??
@@ -358,11 +353,11 @@ async function applyConfigurationsToAgents(agents, concatenatedRules, rulerMcpJs
                 };
             }
             if (!skipApplyForThisAgent) {
-                await agent.applyRulerConfig(concatenatedRules, projectRoot, augmentedRulerMcpJson, finalAgentConfig, backup);
+                await agent.applyRulerConfig(concatenatedRules, projectRoot, agentRulerMcpJson, finalAgentConfig, backup);
             }
         }
         // Handle MCP configuration
-        await handleMcpConfiguration(agent, agentConfig, config, augmentedRulerMcpJson, projectRoot, generatedPaths, verbose, dryRun, cliMcpEnabled, cliMcpStrategy, backup, skillsEnabled);
+        await handleMcpConfiguration(agent, agentConfig, config, agentRulerMcpJson, projectRoot, generatedPaths, verbose, dryRun, cliMcpEnabled, cliMcpStrategy, backup, skillsEnabled);
     }
     return generatedPaths;
 }

package/package.json

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