@intellectronica/ruler 0.3.4 → 0.3.5

package/dist/agents/CodexCliAgent.js

@@ -44,26 +44,26 @@ const constants_1 = require("../constants");
 /**
  * OpenAI Codex CLI agent adapter.
  */
-class CodexCliAgent extends AgentsMdAgent_1.AgentsMdAgent {
+class CodexCliAgent {
+    constructor() {
+        this.agentsMdAgent = new AgentsMdAgent_1.AgentsMdAgent();
+    }
     getIdentifier() {
         return 'codex';
     }
     getName() {
         return 'OpenAI Codex CLI';
     }
-    async applyRulerConfig(concatenatedRules, projectRoot, rulerMcpJson, agentConfig) {
-        // First perform idempotent AGENTS.md write via base class (instructions file).
-        await super.applyRulerConfig(concatenatedRules, projectRoot, null, {
+    async applyRulerConfig(concatenatedRules, projectRoot, rulerMcpJson, agentConfig, backup = true) {
+        // First perform idempotent AGENTS.md write via composed AgentsMdAgent
+        await this.agentsMdAgent.applyRulerConfig(concatenatedRules, projectRoot, null, {
             // Preserve explicit outputPath precedence semantics if provided.
             outputPath: agentConfig?.outputPath ||
                 agentConfig?.outputPathInstructions ||
                 undefined,
-        });
-        // Resolve config path helper (mirrors previous logic)
-        const defaults = {
-            instructions: path.join(projectRoot, constants_1.DEFAULT_RULES_FILENAME),
-            config: path.join(projectRoot, '.codex', 'config.toml'),
-        };
+        }, backup);
+        // Use proper path resolution from getDefaultOutputPath and agentConfig
+        const defaults = this.getDefaultOutputPath(projectRoot);
         const mcpEnabled = agentConfig?.mcp?.enabled ?? true;
         if (mcpEnabled && rulerMcpJson) {
             // Apply MCP server filtering and transformation
@@ -73,7 +73,7 @@ class CodexCliAgent extends AgentsMdAgent_1.AgentsMdAgent {
                 return; // No compatible servers found
             }
             const filteredRulerMcpJson = filteredMcpConfig;
-            // Determine the config file path
+            // Determine the config file path using proper precedence
             const configPath = agentConfig?.outputPathConfig ?? defaults.config;
             // Ensure the parent directory exists
             await fs_1.promises.mkdir(path.dirname(configPath), { recursive: true });
@@ -177,6 +177,12 @@ class CodexCliAgent extends AgentsMdAgent_1.AgentsMdAgent {
             await (0, FileSystemUtils_1.writeGeneratedFile)(configPath, tomlContent);
         }
     }
+    getDefaultOutputPath(projectRoot) {
+        return {
+            instructions: path.join(projectRoot, constants_1.DEFAULT_RULES_FILENAME),
+            config: path.join(projectRoot, '.codex', 'config.toml'),
+        };
+    }
     supportsMcpStdio() {
         return true;
     }

package/dist/core/GitignoreUtils.js

@@ -57,8 +57,9 @@ async function updateGitignore(projectRoot, paths) {
             throw err;
         }
     }
-    // Convert paths to relative POSIX format
-    const relativePaths = paths.map((p) => {
+    // Convert paths to repo-relative POSIX format with leading /
+    const relativePaths = paths
+        .map((p) => {
         let relative;
         if (path.isAbsolute(p)) {
             relative = path.relative(projectRoot, p);
@@ -78,6 +79,14 @@ async function updateGitignore(projectRoot, paths) {
             }
         }
         return relative.replace(/\\/g, '/'); // Convert to POSIX format
+    })
+        .filter((p) => {
+        // Never include any path that resides inside a .ruler directory (inputs, not outputs)
+        return !p.includes('/.ruler/') && !p.startsWith('.ruler/');
+    })
+        .map((p) => {
+        // Always write full repository-relative paths (prefix with leading /)
+        return p.startsWith('/') ? p : `/${p}`;
     });
     // Get all existing paths from .gitignore (excluding Ruler block)
     const existingPaths = getExistingPathsExcludingRulerBlock(existingContent);

package/dist/core/apply-engine.js

@@ -402,7 +402,7 @@ async function applyStandardMcpConfiguration(agent, filteredMcpJson, dest, agent
  * @param cliGitignoreEnabled CLI gitignore setting
  * @param dryRun Whether to perform a dry run
  */
-async function updateGitignore(projectRoot, generatedPaths, config, cliGitignoreEnabled, dryRun, backup = true) {
+async function updateGitignore(projectRoot, generatedPaths, config, cliGitignoreEnabled, dryRun) {
     // Configuration precedence: CLI > TOML > Default (enabled)
     let gitignoreEnabled;
     if (cliGitignoreEnabled !== undefined) {
@@ -416,10 +416,8 @@ async function updateGitignore(projectRoot, generatedPaths, config, cliGitignore
     }
     if (gitignoreEnabled && generatedPaths.length > 0) {
         const uniquePaths = [...new Set(generatedPaths)];
-        // Add wildcard pattern for backup files only if backup is enabled
-        if (backup) {
-            uniquePaths.push('*.bak');
-        }
+        // Note: Individual backup patterns are added per-file in the collection phase
+        // No need to add a broad *.bak pattern here
         if (uniquePaths.length > 0) {
             const prefix = (0, constants_1.actionPrefix)(dryRun);
             if (dryRun) {

package/dist/lib.js

@@ -53,7 +53,7 @@ async function applyAllAgentConfigs(projectRoot, includedAgents, configPath, cli
         (0, constants_1.logVerbose)(`Selected ${selectedAgents.length} agents: ${selectedAgents.map((a) => a.getName()).join(', ')}`, verbose);
         generatedPaths = await (0, apply_engine_1.processSingleConfiguration)(selectedAgents, singleConfig, projectRoot, verbose, dryRun, cliMcpEnabled, cliMcpStrategy, backup);
     }
-    await (0, apply_engine_1.updateGitignore)(projectRoot, generatedPaths, loadedConfig, cliGitignoreEnabled, dryRun, backup);
+    await (0, apply_engine_1.updateGitignore)(projectRoot, generatedPaths, loadedConfig, cliGitignoreEnabled, dryRun);
 }
 /**
  * Normalizes per-agent config keys to agent identifiers for consistent lookup.

package/package.json

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