baldart 3.7.0 → 3.7.1

package/CHANGELOG.md

@@ -5,6 +5,20 @@ All notable changes to BALDART will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [3.7.1] - 2026-05-22
+
+`baldart update` now actively migrates pre-v3.7.0 configs to the new multi-tool layout instead of silently leaving them on `[claude]` only.
+
+### Fixed
+
+- **`src/commands/update.js` legacy-config migration** — after the subtree pull, if `baldart.config.yml` exists but `tools.enabled` is missing/empty, the CLI now:
+  1. Detects the legacy state.
+  2. Surfaces the new capability and shows the autodetected tool list (Claude always, plus Codex if `~/.codex/` exists).
+  3. Asks: *"Backfill `tools.enabled: [claude, codex]` into baldart.config.yml?"*.
+  4. On accept: writes the array into the config AND re-runs `mergeSkills` for the new tools so `.agents/skills/` is populated immediately — not deferred to the next update.
+
+Without this, a pre-v3.7.0 install upgrading to v3.7.x would never get the Codex skill dir populated, because the per-tool dispatch falls back to `['claude']` when `tools.enabled` is absent.
+
 ## [3.7.0] - 2026-05-22
 
 BALDART is now AI-tool agnostic. Every framework skill is installed into both Claude Code (`.claude/skills/`) AND OpenAI Codex CLI (`.agents/skills/`) when both are enabled — same source, two symlinks, zero duplication. Codex skill format is structurally identical to Claude's (`SKILL.md` with `name` + `description` frontmatter + optional `scripts/`/`references/`/`assets/`), so the same file works for both runtimes.

package/VERSION

@@ -1 +1 @@
-3.7.0
+3.7.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "baldart",
-  "version": "3.7.0",
+  "version": "3.7.1",
   "description": "Claude Agent Framework - Reusable framework for coordinating AI agents and humans in software projects",
   "bin": {
     "baldart": "./bin/baldart.js"

package/src/commands/update.js

@@ -370,17 +370,47 @@ async function update(options = {}) {
           await configureCmd();
         }
       } else {
-        // Optional: detect schema drift by comparing template vs existing.
+        // Schema migration: backfill `tools.enabled` for configs created
+        // before v3.7.0 (the multi-tool release). Without this, a legacy
+        // install that has Codex on the machine would never get .agents/skills/
+        // populated, because update would keep defaulting to ['claude'].
         const yaml = require('js-yaml');
+        let needsRewrite = false;
+        let cur = {};
+        try { cur = yaml.load(fs.readFileSync(configPath, 'utf8')) || {}; } catch (_) { cur = null; }
+
+        if (cur && (!cur.tools || !Array.isArray(cur.tools.enabled) || cur.tools.enabled.length === 0)) {
+          const toolAdapters = require('../utils/tool-adapters');
+          const suggested = toolAdapters.defaultEnabled();
+          UI.newline();
+          UI.warning('Pre-v3.7.0 config detected — `tools.enabled` is missing.');
+          UI.info(`From v3.7.0, BALDART can install framework skills for multiple AI tools (Claude Code, OpenAI Codex CLI, …) from the same source.`);
+          UI.info(`Autodetected on this machine: ${suggested.join(', ')}`);
+          const ok = await UI.confirm(`Backfill \`tools.enabled: [${suggested.join(', ')}]\` into baldart.config.yml?`, true);
+          if (ok) {
+            cur.tools = cur.tools || {};
+            cur.tools.enabled = suggested;
+            fs.writeFileSync(configPath, yaml.dump(cur, { lineWidth: 100, noRefs: true }));
+            UI.success('Backfilled tools.enabled. Re-running the symlink reconcile to install per-tool skill dirs…');
+            // Re-run merge so the just-backfilled tools get their skill dirs populated now.
+            try { symlinks.mergeSkills({ tools: suggested }); }
+            catch (err) { UI.warning(`Skill merge for new tools failed: ${err.message}`); }
+            needsRewrite = true;
+          } else {
+            UI.info('Skipped. You can add it manually under `tools:` in baldart.config.yml or rerun `npx baldart configure`.');
+          }
+        }
+
+        // Detect schema drift on other keys (features.*, paths.*).
         const templatePath = path.join('.framework', 'framework', 'templates', 'baldart.config.template.yml');
         if (fs.existsSync(templatePath)) {
           try {
             const tpl = yaml.load(fs.readFileSync(templatePath, 'utf8')) || {};
-            const cur = yaml.load(fs.readFileSync(configPath, 'utf8')) || {};
+            const cur2 = needsRewrite ? cur : (yaml.load(fs.readFileSync(configPath, 'utf8')) || {});
             const missingFeatures = Object.keys(tpl.features || {})
-              .filter((k) => !(k in (cur.features || {})));
+              .filter((k) => !(k in (cur2.features || {})));
             const missingPaths = Object.keys(tpl.paths || {})
-              .filter((k) => !(k in (cur.paths || {})));
+              .filter((k) => !(k in (cur2.paths || {})));
             if (missingFeatures.length || missingPaths.length) {
               UI.newline();
               UI.warning(