baldart 3.8.1 → 3.8.2

package/CHANGELOG.md

@@ -5,6 +5,16 @@ 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.8.2] - 2026-05-22
+
+`baldart` (smart doctor) no longer double-confirms safe actions. Previously, running `baldart` with a remote-ahead state would ask *"Run: Pull N commit(s)?"* and then the `update` command would ask *"Proceed with update?"* immediately after — two prompts for the same intent.
+
+### Fixed
+
+- **`src/commands/doctor.js` action runner** — every action now carries an `autoOk` flag. Safe & idempotent actions (`update`, `configure*`, `migrate`, `register-edit-gate`) run without the outer doctor confirmation (their own internal prompts take over). Actions that change external state or need explicit version classification (`add`, `push`) keep the outer confirmation so the user signals intent explicitly.
+
+Result: `baldart` in a repo with "remote ahead" now goes straight into the update flow (with its own pre-flight stash + diff preview + proceed prompt), instead of double-asking.
+
 ## [3.8.1] - 2026-05-22
 
 Documentation: promote the "one command for everything" install path in the README and surface the global-install recommendation in CLAUDE.md.

package/VERSION

@@ -1 +1 @@
-3.8.1
+3.8.2

package/package.json

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

package/src/commands/doctor.js

@@ -228,11 +228,19 @@ function planActions(state) {
     return actions;
   }
 
+  // `autoOk` policy:
+  //   - true  → safe & idempotent action that has its own internal prompts
+  //             (update, configure, migrate, hook register). The outer doctor
+  //             prompt would be a redundant double-confirmation, so we skip it.
+  //   - false → state-changing action that warrants explicit intent at the
+  //             doctor level (add = first install, push = writes to remote
+  //             and triggers a version-classification dialog).
   if (!state.frameworkPresent) {
     actions.push({
       key: 'add',
       label: 'Install BALDART framework',
       why: 'No .framework/ directory found in this repo.',
+      autoOk: false,
       run: async () => {
         const add = require('./add');
         await add(REPO_DEFAULT, { branch: 'main' });
@@ -246,6 +254,7 @@ function planActions(state) {
       key: 'migrate',
       label: 'Migrate legacy v2.0.x layout',
       why: '.claude/skills/ is a bulk symlink — v2.0.x layout. v2.1.1+ uses per-item merge.',
+      autoOk: true,
       run: () => require('./migrate')(),
     });
     return actions;
@@ -256,6 +265,7 @@ function planActions(state) {
       key: 'configure-malformed',
       label: 'Rewrite baldart.config.yml (current file is malformed)',
       why: 'Existing baldart.config.yml is not valid YAML — re-running configure will rebuild it.',
+      autoOk: true,
       run: () => require('./configure')(),
     });
     return actions;
@@ -266,6 +276,7 @@ function planActions(state) {
       key: 'configure',
       label: 'Generate baldart.config.yml',
       why: 'Skills require a config to resolve project-specific paths/identity/stack. Without it they will prompt on every invocation.',
+      autoOk: true,
       run: () => require('./configure')(),
     });
     return actions;
@@ -276,6 +287,7 @@ function planActions(state) {
       key: 'configure-drift',
       label: `Refresh baldart.config.yml (${state.configSchemaMissing.length} new key(s) from framework template)`,
       why: 'Framework added config keys this consumer doesn\'t yet declare: ' + state.configSchemaMissing.join(', '),
+      autoOk: true,
       run: () => require('./configure')(),
     });
     // Drift is non-blocking — keep checking for other actions after.
@@ -286,6 +298,7 @@ function planActions(state) {
       key: 'register-edit-gate',
       label: 'Register framework-edit-gate hook (.claude/settings.json)',
       why: 'Prevents project-specific tokens (Neo-Brutalism, merchant, …) from being written to .framework/ accidentally. Standard in v3.3.0+; missing on installs predating that version.',
+      autoOk: true,
       run: async () => {
         const res = require('../utils/hooks').register();
         UI.success(`Hook registered: ${res.status} (${res.path})`);
@@ -301,6 +314,7 @@ function planActions(state) {
       key: 'update',
       label: `Pull ${state.remote.behind} commit(s) from upstream`,
       why: 'Remote framework is ahead of your local copy. Updating first keeps your contributions on a fresh base.',
+      autoOk: true, // update has its own internal "Proceed?" prompt + pre-flight stash
       run: () => require('./update')(),
     });
   }
@@ -312,6 +326,7 @@ function planActions(state) {
       why: state.workingTreeDirty
         ? `${state.workingTreeDirty} uncommitted file(s) in .framework/ — commit them through \`baldart push\`.`
         : `${state.commitsAhead} commit(s) on .framework/ not yet pushed.`,
+      autoOk: false, // push writes to remote and asks for version classification — keep explicit intent
       run: () => require('./push')(),
     });
   }
@@ -472,7 +487,16 @@ async function doctor(opts = {}) {
   for (const action of actions) {
     let proceed = auto;
     if (!auto) {
-      proceed = await UI.confirm(`Run: ${action.label}?`, true);
+      // Safe & idempotent actions (`autoOk: true`) run without an outer
+      // confirmation — they have their own internal prompts. Actions that
+      // change external state or require explicit version classification
+      // (add, push) still ask here so the user signals intent.
+      if (action.autoOk) {
+        UI.info(`→ Running: ${action.label}`);
+        proceed = true;
+      } else {
+        proceed = await UI.confirm(`Run: ${action.label}?`, true);
+      }
     }
     if (!proceed) {
       UI.info(`Skipped: ${action.label}`);