@intellectronica/ruler 0.3.29 → 0.3.30

package/dist/agents/CodexCliAgent.js

@@ -102,9 +102,13 @@ class CodexCliAgent {
             // Add the ruler servers
             for (const [serverName, serverConfig] of Object.entries(rulerServers)) {
                 // Create a properly formatted MCP server entry
-                const mcpServer = {
-                    command: serverConfig.command,
-                };
+                const mcpServer = {};
+                if (serverConfig.command) {
+                    mcpServer.command = serverConfig.command;
+                }
+                if (serverConfig.url) {
+                    mcpServer.url = serverConfig.url;
+                }
                 if (serverConfig.args) {
                     mcpServer.args = serverConfig.args;
                 }
@@ -133,11 +137,14 @@ class CodexCliAgent {
             config: path.join(projectRoot, '.codex', 'config.toml'),
         };
     }
+    getMcpServerKey() {
+        return 'mcp_servers';
+    }
     supportsMcpStdio() {
         return true;
     }
     supportsMcpRemote() {
-        return false; // Codex CLI only supports STDIO based on PR description
+        return true;
     }
     supportsNativeSkills() {
         return true;

package/dist/core/apply-engine.js

@@ -41,6 +41,7 @@ exports.applyConfigurationsToAgents = applyConfigurationsToAgents;
 exports.updateGitignore = updateGitignore;
 const path = __importStar(require("path"));
 const fs_1 = require("fs");
+const toml_1 = require("@iarna/toml");
 const FileSystemUtils = __importStar(require("./FileSystemUtils"));
 const RuleProcessor_1 = require("./RuleProcessor");
 const ConfigLoader_1 = require("./ConfigLoader");
@@ -544,8 +545,28 @@ async function applyStandardMcpConfiguration(agent, filteredMcpJson, dest, agent
         else if (agent.getIdentifier() === 'factory') {
             mcpToMerge = transformMcpForFactoryDroid(filteredMcpJson);
         }
-        const existing = await (0, mcp_1.readNativeMcp)(dest);
-        const merged = (0, merge_1.mergeMcp)(existing, mcpToMerge, strategy, serverKey);
+        const CODEX_AGENT_ID = 'codex';
+        const isCodexToml = agent.getIdentifier() === CODEX_AGENT_ID && dest.endsWith('.toml');
+        let existing = await (0, mcp_1.readNativeMcp)(dest);
+        if (isCodexToml) {
+            try {
+                const tomlContent = await fs_1.promises.readFile(dest, 'utf8');
+                existing = (0, toml_1.parse)(tomlContent);
+            }
+            catch (error) {
+                (0, constants_1.logVerbose)(`Failed to read Codex MCP TOML at ${dest}: ${error.message}`, verbose);
+                // ignore missing or invalid TOML, fall back to previously read value
+            }
+        }
+        let merged = (0, merge_1.mergeMcp)(existing, mcpToMerge, strategy, serverKey);
+        if (isCodexToml) {
+            const { [serverKey]: servers, ...rest } = merged;
+            merged = {
+                ...rest,
+                // Codex CLI expects MCP servers under mcp_servers in config.toml.
+                mcp_servers: servers ?? {},
+            };
+        }
         // Firebase Studio (IDX) expects no "type" fields in .idx/mcp.json server entries.
         // Sanitize merged config by stripping 'type' from each server when targeting Firebase.
         const sanitizeForFirebase = (obj) => {
@@ -593,14 +614,23 @@ async function applyStandardMcpConfiguration(agent, filteredMcpJson, dest, agent
         let toWrite = sanitizeForFirebase(merged);
         toWrite = sanitizeForGemini(toWrite);
         // Only backup and write if content would actually change (idempotent)
-        const currentContent = JSON.stringify(existing, null, 2);
-        const newContent = JSON.stringify(toWrite, null, 2);
+        const currentContent = isCodexToml
+            ? (0, toml_1.stringify)(existing)
+            : JSON.stringify(existing, null, 2);
+        const newContent = isCodexToml
+            ? (0, toml_1.stringify)(toWrite)
+            : JSON.stringify(toWrite, null, 2);
         if (currentContent !== newContent) {
             if (backup) {
                 const { backupFile } = await Promise.resolve().then(() => __importStar(require('../core/FileSystemUtils')));
                 await backupFile(dest);
             }
-            await (0, mcp_1.writeNativeMcp)(dest, toWrite);
+            if (isCodexToml) {
+                await FileSystemUtils.writeGeneratedFile(dest, (0, toml_1.stringify)(toWrite));
+            }
+            else {
+                await (0, mcp_1.writeNativeMcp)(dest, toWrite);
+            }
         }
         else {
             (0, constants_1.logVerbose)(`MCP config for ${agent.getName()} is already up to date - skipping backup and write`, verbose);

package/dist/paths/mcp.js

@@ -59,7 +59,7 @@ async function getNativeMcpPath(adapterName, projectRoot) {
             candidates.push(path.join(projectRoot, '.mcp.json'));
             break;
         case 'OpenAI Codex CLI':
-            candidates.push(path.join(projectRoot, '.codex', 'config.json'));
+            candidates.push(path.join(projectRoot, '.codex', 'config.toml'));
             break;
         case 'Aider':
             candidates.push(path.join(projectRoot, '.mcp.json'));

package/package.json

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