@yemi33/minions 0.1.2049 → 0.1.2050

package/dashboard/js/modal.js

@@ -7,6 +7,10 @@ function closeModal() {
   clearModalBackStack();
   // Hide Q&A section (only shown for document modals)
   document.getElementById('modal-qa').style.display = 'none';
+  // Remove settings-body marker so the next modal opens with the default
+  // monospace/pre-wrap modal-body styling rather than inheriting the Segoe-UI
+  // tabbed-settings overrides (W-mpmwxkcn000646cc).
+  document.getElementById('modal-body').classList.remove('settings-body');
   // Remove settings buttons if present
   const settingsBtn = document.getElementById('modal-settings-save');
   if (settingsBtn) settingsBtn.remove();

package/dashboard/js/refresh.js

@@ -89,6 +89,11 @@ const RENDER_VERSIONS = {
   pipelines: 1,
   pinned: 1,
   kbPayload: 2,
+  // settings is a modal-driven section, not a tick-driven render — but the
+  // cache-bust entry exists so future tick-driven code (or feature-flag-aware
+  // renderers) can call `_changed('settings', …)` and bump on visual revamps.
+  // Bumped to 2 by W-mpmwxkcn000646cc (left-rail tabbed Settings layout).
+  settings: 2,
 };
 const _sectionCache = {};
 const _lastValueByKey = {};

package/dashboard/js/settings.js

@@ -25,6 +25,9 @@ async function openSettings() {
   document.getElementById('modal-title').textContent = 'Settings';
   document.getElementById('modal-body').innerHTML = '<p style="color:var(--muted)">Loading...</p>';
   document.getElementById('modal').classList.add('open');
+  // Settings needs more horizontal room for the left rail + content split.
+  const modalEl = document.querySelector('#modal .modal');
+  if (modalEl) modalEl.classList.add('modal-wide');
 
   _settingsData = null;
   let data;
@@ -70,141 +73,24 @@ async function openSettings() {
     '</tr>';
   }).join('');
 
-  const html = '<div style="padding:8px 0;max-height:70vh;overflow-y:auto">' +
-
-    '<h3 style="font-size:13px;color:var(--blue);margin-bottom:8px">Engine</h3>' +
-    '<div style="display:grid;grid-template-columns:1fr 1fr;gap:8px;margin-bottom:16px">' +
-      settingsField('Tick Interval', 'set-tickInterval', e.tickInterval || 60000, 'ms', 'How often the engine runs discovery + dispatch') +
-      settingsField('Max Concurrent Agents', 'set-maxConcurrent', e.maxConcurrent || 5, '', 'Max agents working simultaneously') +
-      settingsField('Consolidation Threshold', 'set-inboxConsolidateThreshold', e.inboxConsolidateThreshold || 5, 'notes', 'Inbox notes before auto-consolidation') +
-      settingsField('Agent Timeout', 'set-agentTimeout', e.agentTimeout || 18000000, 'ms', 'Kill agent after this duration') +
-      settingsField('Max Turns', 'set-maxTurns', e.maxTurns || 100, '', 'Claude CLI --max-turns per agent') +
-      settingsField('Heartbeat Timeout', 'set-heartbeatTimeout', e.heartbeatTimeout || 300000, 'ms', 'No output = dead after this') +
-      settingsField('Worktree Create Timeout', 'set-worktreeCreateTimeout', e.worktreeCreateTimeout || 300000, 'ms', 'Timeout for git worktree add (increase for large repos/Windows)') +
-      settingsField('Worktree Create Retries', 'set-worktreeCreateRetries', e.worktreeCreateRetries || 1, '', 'Retry count for transient worktree add failures (0-3)') +
-      settingsField('Worktree Root', 'set-worktreeRoot', e.worktreeRoot || '../worktrees', '', 'Relative or absolute path for git worktrees; on Windows prefer a short path like C:\\wt') +
-      settingsField('Idle Alert', 'set-idleAlertMinutes', e.idleAlertMinutes || 15, 'min', 'Alert after agent idle this long') +
-      settingsField('Shutdown Timeout', 'set-shutdownTimeout', e.shutdownTimeout || 300000, 'ms', 'Max wait for agents during graceful shutdown') +
-      settingsField('Restart Grace Period', 'set-restartGracePeriod', e.restartGracePeriod || 1200000, 'ms', 'Grace period before orphan detection on restart') +
-      settingsField('Meeting Round Timeout', 'set-meetingRoundTimeout', e.meetingRoundTimeout || 900000, 'ms', 'Auto-advance meeting round after this') +
-      settingsField('Operator login (used in branch names)', 'set-operatorLogin', e.operatorLogin || '', '', 'Override the human operator login used in user/<loginname>/<wi-id>-<slug> branches. Empty = auto-resolve via gh / git email / OS username (currently resolves to: ' + (e._resolvedOperatorLogin || 'unknown') + ')') +
-      settingsField('Status WorkItems Retention', 'set-statusWorkItemsRetentionDays', e.statusWorkItemsRetentionDays ?? 7, 'days', 'Trim done/failed/cancelled work items older than N days from the /api/status workItems slice (active items are always shipped). Cuts SPA payload from ~3MB to <500KB. Set to 0 to disable trimming (full list shipped, restoring legacy behavior).') +
-      settingsField('Status Meetings Retention', 'set-statusMeetingsRetentionDays', e.statusMeetingsRetentionDays ?? 7, 'days', 'Trim completed/archived meetings older than N days from the /api/status meetings slice (active meetings are always shipped). Cuts SPA payload from ~4.3MB to <500KB. Detail modal still fetches full transcripts via /api/meetings/:id. Set to 0 to disable trimming (full list shipped, restoring legacy behavior).') +
-    '</div>' +
-    '<h3 style="font-size:13px;color:var(--blue);margin-bottom:8px">Automation</h3>' +
-    '<div style="display:flex;flex-direction:column;gap:6px;margin-bottom:16px">' +
-      settingsToggle('Auto-approve Plans', 'set-autoApprovePlans', !!e.autoApprovePlans, 'PRDs are approved automatically without human review') +
-      settingsToggle('Eval Loop', 'set-evalLoop', e.evalLoop !== false, 'Auto-review implementations and iterate fix cycles until pass') +
-      settingsToggle('Auto-decompose', 'set-autoDecompose', e.autoDecompose !== false, 'Large implement items are auto-split into sub-tasks') +
-      settingsToggle('Allow Temp Agents', 'set-allowTempAgents', !!e.allowTempAgents, 'Spawn ephemeral agents when all permanent agents are busy') +
-      settingsToggle('Auto-archive Plans', 'set-autoArchive', !!e.autoArchive, 'Automatically archive plans after verify completes (off = manual archive via dashboard)') +
-      settingsToggle('Auto-consolidate Memory', 'set-autoConsolidateMemory', !!e.autoConsolidateMemory, 'Periodically spawn the KB sweep (dedup + compress + normalize knowledge/) from the engine tick on a 4h cadence. Inbox→notes consolidation already runs every tick (gated by the Consolidation Threshold above); this toggle controls only the KB sweep that was previously dashboard-button-only.') +
-      settingsToggle('Auto-complete PRs', 'set-autoCompletePrs', !!e.autoCompletePrs, 'Auto-merge PRs when builds pass and review is approved (opt-in)') +
-      settingsToggle('CC Worker Pool', 'set-ccUseWorkerPool', (e.ccUseWorkerPool === undefined ? ((e.ccCli || e.defaultCli) === 'copilot') : !!e.ccUseWorkerPool), 'Route Command Center / doc-chat through a persistent copilot --acp worker per tab instead of spawning a fresh CLI per turn. Copilot-only (Agent Client Protocol transport); Claude does not implement ACP, so this toggle has no effect when CC runtime is Claude. Default ON for copilot (cold-spawn ~20s on Windows); forced OFF for non-copilot CC runtimes regardless of this toggle.') +
-    '</div>' +
-
-    '<h3 style="font-size:13px;color:var(--blue);margin-bottom:8px">PR Polling &amp; Dispatch Gates</h3>' +
-    '<div style="border:1px solid var(--border);border-radius:6px;padding:10px 12px;margin-bottom:16px">' +
-      '<div style="display:flex;flex-direction:column;gap:6px;margin-bottom:10px">' +
-        settingsToggle('ADO Polling', 'set-adoPollEnabled', e.adoPollEnabled !== false, 'Keeps ADO PR build results, votes, and comments fresh each tick; ADO PR dispatch gates are inert when this is off') +
-        settingsToggle('GitHub Polling', 'set-ghPollEnabled', e.ghPollEnabled !== false, 'Keeps GitHub PR build results, votes, and comments fresh each tick; GitHub PR dispatch gates are inert when this is off') +
-      '</div>' +
-      '<div style="margin-top:10px;padding-top:10px;border-top:1px solid var(--border);display:flex;flex-direction:column;gap:4px">' +
-        settingsToggle('Auto-apply review vote to PR', 'set-autoApplyReviewVote', !!e.autoApplyReviewVote, 'When ON, Minions review verdicts (APPROVE / REQUEST_CHANGES) automatically flip the platform vote on ADO/GitHub. When OFF (default), verdicts are informational only and the human casts the final vote.') +
-        settingsToggle('Auto-fix Builds', 'set-autoFixBuilds', e.autoFixBuilds !== false, 'Shared dispatch gate: auto-fix agent when a PR build fails; also requires that PR provider polling is enabled') +
-        settingsToggle('Auto-fix Conflicts', 'set-autoFixConflicts', e.autoFixConflicts !== false, 'Shared dispatch gate: auto-fix agent when a PR merge conflict is detected; also requires that PR provider polling is enabled') +
-        settingsToggle('Auto-review PRs', 'set-autoReviewPrs', e.autoReviewPrs !== false, 'Shared dispatch gate: review agent for newly opened agent PRs; also requires that PR provider polling is enabled') +
-        settingsToggle('Auto-re-review PRs', 'set-autoReReviewPrs', e.autoReReviewPrs !== false, 'Shared dispatch gate: review agent after a fix push is awaiting re-review; also requires that PR provider polling is enabled') +
-        settingsToggle('Auto-fix Review Feedback', 'set-autoFixReviewFeedback', e.autoFixReviewFeedback !== false, 'Shared dispatch gate: fix agent for minions changes-requested verdicts; also requires that PR provider polling is enabled') +
-        settingsToggle('Auto-fix Human Comments', 'set-autoFixHumanComments', e.autoFixHumanComments !== false, 'Shared dispatch gate: fix agent for actionable human PR comments; also requires that PR provider polling is enabled') +
-      '</div>' +
-      '<div style="display:grid;grid-template-columns:1fr 1fr;gap:8px;margin-top:10px">' +
-        settingsField('PR Status Poll Frequency', 'set-prPollStatusEvery', e.prPollStatusEvery ?? 12, 'ticks', 'Poll PR build/review/merge status every N ticks for both ADO and GitHub (~12 min at default tick rate)') +
-        settingsField('PR Comments Poll Frequency', 'set-prPollCommentsEvery', e.prPollCommentsEvery ?? 12, 'ticks', 'Poll PR human comments every N ticks for both ADO and GitHub (~12 min at default tick rate)') +
-      '</div>' +
-    '</div>' +
-
-    '<h3 style="font-size:13px;color:var(--blue);margin-bottom:8px">Limits &amp; Thresholds</h3>' +
-    '<div style="display:grid;grid-template-columns:1fr 1fr;gap:8px;margin-bottom:16px">' +
-      settingsField('Eval Max Cost', 'set-evalMaxCost', e.evalMaxCost === null || e.evalMaxCost === undefined ? '' : e.evalMaxCost, '$', 'USD ceiling per work item across all eval iterations (blank = no limit)') +
-      settingsField('Agent Busy Reassign', 'set-agentBusyReassignMs', e.agentBusyReassignMs || 600000, 'ms', 'Reassign work to another agent after it waits this long on a busy agent') +
-      settingsField('Max Retries Per Agent', 'set-maxRetriesPerAgent', e.maxRetriesPerAgent ?? 2, '', 'After the same agent fails the same work item this many times, the next retry reassigns to a different eligible agent (consults routing.md + availability). Falls back to the same agent only when no alternate is available. Counted separately from total maxRetries (which still caps overall retries).') +
-      settingsField('Version Check Interval', 'set-versionCheckInterval', e.versionCheckInterval || 3600000, 'ms', 'How often to check npm for updates (default: 1 hour)') +
-      settingsField('Ignored Comment Authors', 'set-ignoredCommentAuthors', (e.ignoredCommentAuthors || []).join(', '), '', 'Comma-separated usernames — comments auto-closed, never trigger fixes') +
-    '</div>' +
-
-    '<h3 style="font-size:13px;color:var(--blue);margin-bottom:8px">Projects</h3>' +
-    '<div style="display:flex;flex-direction:column;gap:12px;margin-bottom:16px">' +
-    (data.projects || []).map(function(p) {
-      // Cross-reference live /api/status to pull in the auto-detected
-      // remoteDefaultBranch (read-only) — settings already has localPath +
-      // mainBranch from /api/settings, but origin/HEAD is probe-derived and
-      // lives in /api/status only. W-mpg3whgp000d09ec / #2732.
-      var liveProj = null;
-      try {
-        if (window._lastStatus && Array.isArray(window._lastStatus.projects)) {
-          liveProj = window._lastStatus.projects.find(function(x) { return x.name === p.name; }) || null;
-        }
-      } catch (e) { liveProj = null; }
-      var remoteDefault = (liveProj && liveProj.remoteDefaultBranch) || '';
-      var mismatch = !!(liveProj && liveProj.branchMismatch);
-      var localBranch = (liveProj && liveProj.gitBranch) || '';
-      var driftNote = mismatch
-        ? '<div style="font-size:10px;color:var(--yellow);margin-top:4px">⚠ Configured main (<code>' + escHtml(p.mainBranch || '') + '</code>) differs from origin/HEAD (<code>' + escHtml(remoteDefault) + '</code>) — config is likely stale.</div>'
-        : '';
-      var pathRow = p.localPath
-        ? '<div style="font-size:10px;color:var(--muted);margin-bottom:6px;font-family:ui-monospace,SFMono-Regular,Menlo,monospace">' + escHtml(p.localPath) + '</div>'
-        : '';
-      var branchGrid = '<div style="display:grid;grid-template-columns:1fr 1fr;gap:8px;margin-bottom:6px">' +
-        settingsField('Configured main branch', 'set-mainBranch-' + p.name, p.mainBranch || '', '', 'Used by branch-naming + dependency-merge to identify mainline. Empty = auto-detect from origin/HEAD.') +
-        '<div>' +
-          '<label style="font-size:10px;color:var(--muted);display:block;margin-bottom:2px">Remote default <span style="opacity:0.6">(read-only)</span></label>' +
-          '<input id="set-remoteDefaultBranch-' + p.name + '" value="' + escHtml(remoteDefault || '(unset)') + '" readonly style="width:100%;padding:4px 6px;background:var(--surface2);border:1px solid var(--border);border-radius:4px;color:var(--muted);font-size:12px;font-family:ui-monospace,SFMono-Regular,Menlo,monospace">' +
-          '<div style="font-size:9px;color:var(--muted);margin-top:1px">Parsed from <code>git symbolic-ref refs/remotes/origin/HEAD</code>' + (localBranch ? '. Local HEAD: <code>' + escHtml(localBranch) + '</code>' : '') + '.</div>' +
-        '</div>' +
-      '</div>';
-      return '<div data-settings-project="' + escHtml(p.name) + '" style="border:1px solid var(--border);border-radius:6px;padding:10px 12px">' +
-        '<div style="display:flex;align-items:center;justify-content:space-between;margin-bottom:8px">' +
-          '<div style="font-size:12px;font-weight:600">' + escHtml(p.name) + '</div>' +
-          '<button onclick="MinionsSettings.removeProject(\'' + escHtml(p.name) + '\')" style="font-size:9px;padding:2px 8px;background:transparent;color:var(--red);border:1px solid var(--red);border-radius:3px;cursor:pointer">Remove</button>' +
-        '</div>' +
-        pathRow +
-        branchGrid +
-        driftNote +
-        '<div style="display:flex;flex-direction:column;gap:6px;margin-top:8px">' +
-        settingsToggle('Discover from PRs', 'set-ws-prs-' + p.name, p.workSources.pullRequests.enabled, 'Discovery gate: scan repo for open PRs and surface them as review tasks. Independent of ADO/GitHub polling — does not affect already-tracked PRs.') +
-        settingsToggle('Discover from Work Items', 'set-ws-wi-' + p.name, p.workSources.workItems.enabled, 'Auto-discover work from ADO/GitHub work items') +
-        '</div></div>';
-    }).join('') +
-    '</div>' +
-
-    '<h3 style="font-size:13px;color:var(--blue);margin-bottom:8px">Max Turns by Task Type</h3>' +
-    '<div style="font-size:10px;color:var(--muted);margin-bottom:6px">How many tool-use turns each task type gets before forced stop. Blank = built-in default.</div>' +
-    '<div style="display:grid;grid-template-columns:1fr 1fr 1fr;gap:8px;margin-bottom:16px">' +
-      settingsField('Explore', 'set-mt-explore', (e.maxTurnsByType || {}).explore || '', '', 'Default: 30') +
-      settingsField('Ask', 'set-mt-ask', (e.maxTurnsByType || {}).ask || '', '', 'Default: 20') +
-      settingsField('Review', 'set-mt-review', (e.maxTurnsByType || {}).review || '', '', 'Default: 30') +
-      settingsField('Implement', 'set-mt-implement', (e.maxTurnsByType || {}).implement || '', '', 'Default: 75') +
-      settingsField('Fix', 'set-mt-fix', (e.maxTurnsByType || {}).fix || '', '', 'Default: 75') +
-      settingsField('Test', 'set-mt-test', (e.maxTurnsByType || {}).test || '', '', 'Default: 50') +
-      settingsField('Verify', 'set-mt-verify', (e.maxTurnsByType || {}).verify || '', '', 'Default: 100') +
-      settingsField('Plan', 'set-mt-plan', (e.maxTurnsByType || {}).plan || '', '', 'Default: 30') +
-      settingsField('Decompose', 'set-mt-decompose', (e.maxTurnsByType || {}).decompose || '', '', 'Default: 15') +
-    '</div>' +
-
-    // ── Runtime (P-7a5c1f8e) — unified fleet runtime + CC overrides + advanced ──
-    '<h3 style="font-size:13px;color:var(--blue);margin-bottom:8px">Runtime</h3>' +
+  // ── Section bodies — every original control is grouped under one of these
+  // tabs. Top of the rail is Runtime + Auto-fix (Caleb's #1 feedback was that
+  // these were buried under generic Engine headers). The disclosure formerly
+  // wrapping the 8 advanced runtime toggles was split into Copilot Tuning and
+  // Claude Tuning tabs so each runtime's knobs live behind its own rail entry.
+  const paneRuntime =
+    '<h3>Runtime &amp; Models</h3>' +
+    '<div class="settings-pane-sub">Single source of truth for which CLI runtime + model the fleet spawns. Per-agent overrides live in the Agents table below.</div>' +
     '<div id="set-runtime-section" style="border:1px solid var(--border);border-radius:6px;padding:10px 12px;margin-bottom:16px">' +
-      '<div style="font-size:10px;color:var(--muted);margin-bottom:8px">Single source of truth for which CLI runtime + model the fleet spawns. Per-agent overrides live in the Agents table below.</div>' +
       '<div style="display:grid;grid-template-columns:1fr 2fr;gap:8px;margin-bottom:8px">' +
-        '<div>' +
+        '<div data-search="default cli fleet runtime">' +
           '<label style="font-size:10px;color:var(--muted);display:block;margin-bottom:2px">Default CLI</label>' +
           '<select id="set-defaultCli" style="width:100%;padding:4px 6px;background:var(--surface);border:1px solid var(--border);border-radius:4px;color:var(--text);font-size:12px">' +
             '<option value="">Loading…</option>' +
           '</select>' +
           '<div style="font-size:9px;color:var(--muted);margin-top:1px">Fleet-wide runtime — registered adapters from <code>/api/runtimes</code></div>' +
         '</div>' +
-        '<div>' +
+        '<div data-search="default model fleet">' +
           '<label style="font-size:10px;color:var(--muted);display:block;margin-bottom:2px">Default Model</label>' +
           '<div id="set-defaultModel-wrap"><input id="set-defaultModel" value="' + escHtml(e.defaultModel || '') + '" placeholder="Default (CLI chooses)" style="width:100%;padding:4px 6px;background:var(--surface);border:1px solid var(--border);border-radius:4px;color:var(--text);font-size:12px"></div>' +
           '<div style="font-size:9px;color:var(--muted);margin-top:1px">Empty = let the runtime pick its own default</div>' +
@@ -216,14 +102,14 @@ async function openSettings() {
           '<span style="font-size:9px;color:var(--muted)">(Command Center + doc-chat use the fleet defaults unless overridden)</span>' +
         '</summary>' +
         '<div style="display:grid;grid-template-columns:1fr 2fr 1fr;gap:8px;margin-top:8px">' +
-          '<div>' +
+          '<div data-search="cc cli command center runtime">' +
             '<label style="font-size:10px;color:var(--muted);display:block;margin-bottom:2px">CC CLI</label>' +
             '<select id="set-ccCli" style="width:100%;padding:4px 6px;background:var(--surface);border:1px solid var(--border);border-radius:4px;color:var(--text);font-size:12px">' +
               '<option value="">Loading…</option>' +
             '</select>' +
             '<div style="font-size:9px;color:var(--muted);margin-top:1px">Empty = inherit Default CLI</div>' +
           '</div>' +
-          '<div>' +
+          '<div data-search="cc model command center">' +
             '<label style="font-size:10px;color:var(--muted);display:block;margin-bottom:2px">CC Model</label>' +
             // Wrap the input in a dedicated container so loadModelsForRuntime
             // (which does `wrap.innerHTML = …` on the input's parent) only
@@ -233,7 +119,7 @@ async function openSettings() {
             '<div id="set-ccModel-wrap"><input id="set-ccModel" value="' + escHtml(e.ccModel || '') + '" placeholder="(inherits Default Model)" style="width:100%;padding:4px 6px;background:var(--surface);border:1px solid var(--border);border-radius:4px;color:var(--text);font-size:12px"></div>' +
             '<div style="font-size:9px;color:var(--muted);margin-top:1px">Empty = inherit Default Model</div>' +
           '</div>' +
-          '<div>' +
+          '<div data-search="cc effort reasoning">' +
             '<label style="font-size:10px;color:var(--muted);display:block;margin-bottom:2px">Effort</label>' +
             '<select id="set-ccEffort" style="width:100%;padding:4px 6px;background:var(--surface);border:1px solid var(--border);border-radius:4px;color:var(--text);font-size:12px">' +
               '<option value=""' + (!e.ccEffort ? ' selected' : '') + '>Default</option>' +
@@ -245,63 +131,193 @@ async function openSettings() {
           '</div>' +
         '</div>' +
       '</details>' +
-      // Advanced runtime settings — collapsed by default
-      '<details id="set-runtime-advanced-details" style="margin-top:8px;border-top:1px solid var(--border);padding-top:8px">' +
-        '<summary style="cursor:pointer;font-size:11px;color:var(--text);user-select:none">Advanced runtime settings ' +
-          '<span style="font-size:9px;color:var(--muted)">(per-runtime feature flags)</span>' +
-        '</summary>' +
-        '<div style="display:flex;flex-direction:column;gap:6px;margin-top:8px">' +
-          settingsToggle('Claude bare mode', 'set-claudeBareMode', !!e.claudeBareMode, '--bare suppresses CLAUDE.md auto-discovery; pair with explicit ccSystemPrompt or context will be lost') +
-        '</div>' +
-        '<div style="display:grid;grid-template-columns:1fr 1fr 1fr;gap:8px;margin-top:8px">' +
-          settingsField('Claude fallback model', 'set-claudeFallbackModel', e.claudeFallbackModel || '', 'e.g. sonnet', 'Passed via Claude --fallback-model. The Claude CLI swaps to this only on rate-limit (429). Used together with engine.copilotFallbackModel for the engine-level MODEL_UNAVAILABLE (overloaded/503) retry — see CLAUDE.md.') +
-          settingsField('Copilot fallback model', 'set-copilotFallbackModel', e.copilotFallbackModel || '', 'e.g. gpt-5.4', 'Copilot has no --fallback-model flag. On a MODEL_UNAVAILABLE (overloaded/503) retry, the engine OVERRIDES --model with this value (Copilot only).') +
-          settingsField('Max budget (USD)', 'set-maxBudgetUsd', e.maxBudgetUsd != null ? String(e.maxBudgetUsd) : '', '', 'Fleet ceiling for --max-budget-usd. 0 is a valid cap (read-only / dry-run). Empty = no cap. Claude only.') +
-        '</div>' +
-        '<div style="display:flex;flex-direction:column;gap:6px;margin-top:8px">' +
-          // Tooltip on copilotDisableBuiltinMcps MUST warn about the split-brain risk
-          settingsToggle('Copilot: disable built-in MCPs', 'set-copilotDisableBuiltinMcps', e.copilotDisableBuiltinMcps !== false,
-            '⚠ When OFF, Copilot agents can autonomously create PRs/labels/comments via the github-mcp-server, bypassing pull-requests.json tracking — Minions and Copilot end up with split views of the same PR. Keep ON unless you understand the risk.') +
-          settingsToggle('Copilot: suppress AGENTS.md', 'set-copilotSuppressAgentsMd', e.copilotSuppressAgentsMd !== false, '--no-custom-instructions: stops AGENTS.md auto-load from fighting Minions playbook prompts') +
-          settingsToggle('Copilot: reasoning summaries', 'set-copilotReasoningSummaries', !!e.copilotReasoningSummaries, '--enable-reasoning-summaries (Anthropic-family models only)') +
-          settingsToggle('Disable model discovery', 'set-disableModelDiscovery', !!e.disableModelDiscovery, 'Skip /api/runtimes/<name>/models REST calls fleet-wide. Settings UI falls back to free-text.') +
-        '</div>' +
-        '<div style="display:grid;grid-template-columns:1fr 3fr;gap:8px;margin-top:8px">' +
-          '<div>' +
-            '<label style="font-size:10px;color:var(--muted);display:block;margin-bottom:2px">Copilot stream</label>' +
-            '<select id="set-copilotStreamMode" style="width:100%;padding:4px 6px;background:var(--surface);border:1px solid var(--border);border-radius:4px;color:var(--text);font-size:12px">' +
-              '<option value="on"' + ((e.copilotStreamMode || 'on') === 'on' ? ' selected' : '') + '>on (incremental)</option>' +
-              '<option value="off"' + (e.copilotStreamMode === 'off' ? ' selected' : '') + '>off (batched)</option>' +
-            '</select>' +
-          '</div>' +
-        '</div>' +
-      '</details>' +
-    '</div>' +
-
-    '<h3 style="font-size:13px;color:var(--blue);margin-bottom:8px">Claude CLI</h3>' +
-    '<div style="display:grid;grid-template-columns:1fr;gap:8px;margin-bottom:8px">' +
-      settingsField('Allowed Tools', 'set-allowedTools', c.allowedTools || '', '', 'Claude allow-list passed through for compatibility; runtime bypass flags are adapter-owned.') +
-    '</div>' +
-    '<div style="font-size:10px;color:var(--muted);margin-bottom:16px;padding:6px 8px;border:1px solid var(--border);border-radius:4px;background:var(--surface-subtle, rgba(130,160,210,0.08))">' +
-      'Permission bypass is runtime-owned: Claude agents use <code>--dangerously-skip-permissions</code>; Copilot agents use <code>--autopilot --allow-all --no-ask-user</code>. There is no dashboard permission-mode setting.' +
     '</div>' +
-
-    '<h3 style="font-size:13px;color:var(--blue);margin-bottom:8px">Agents</h3>' +
-    '<div style="font-size:10px;color:var(--muted);margin-bottom:6px">CLI / Model placeholders show the fleet default each agent will inherit. Pick a value to pin per-agent; clear to re-inherit.</div>' +
+    '<h4>Agents</h4>' +
+    '<div style="font-size:10px;color:var(--muted);margin-bottom:6px">CLI / Model placeholders show the fleet default each agent will inherit. Pick a value to pin per-agent; clear to re-inherit. Per-agent monthly budget overrides the fleet ceiling.</div>' +
     '<table style="width:100%;border-collapse:collapse;margin-bottom:16px;font-size:11px">' +
       '<tr style="text-align:left;color:var(--muted)"><th style="padding:4px">Agent</th><th style="padding:4px">Role</th><th style="padding:4px">Skills</th><th style="padding:4px">CLI</th><th style="padding:4px">Model</th><th style="padding:4px">Budget $/mo</th></tr>' +
       agentRows +
-    '</table>' +
+    '</table>';
+
+  const paneAutoFix =
+    '<h3>Auto-fix &amp; Review Loop</h3>' +
+    '<div class="settings-pane-sub">Dispatch gates that decide when an agent is auto-spawned in response to PR build failures, merge conflicts, review verdicts, and human comments. All require the matching provider polling toggle (Polling tab).</div>' +
+    '<div class="settings-stack" style="margin-bottom:16px">' +
+      settingsToggle('Auto-fix Builds', 'set-autoFixBuilds', e.autoFixBuilds !== false, 'Shared dispatch gate: auto-fix agent when a PR build fails; also requires that PR provider polling is enabled') +
+      settingsToggle('Auto-fix Conflicts', 'set-autoFixConflicts', e.autoFixConflicts !== false, 'Shared dispatch gate: auto-fix agent when a PR merge conflict is detected; also requires that PR provider polling is enabled') +
+      settingsToggle('Auto-review PRs', 'set-autoReviewPrs', e.autoReviewPrs !== false, 'Shared dispatch gate: review agent for newly opened agent PRs; also requires that PR provider polling is enabled') +
+      settingsToggle('Auto-re-review PRs', 'set-autoReReviewPrs', e.autoReReviewPrs !== false, 'Shared dispatch gate: review agent after a fix push is awaiting re-review; also requires that PR provider polling is enabled') +
+      settingsToggle('Auto-fix Review Feedback', 'set-autoFixReviewFeedback', e.autoFixReviewFeedback !== false, 'Shared dispatch gate: fix agent for minions changes-requested verdicts; also requires that PR provider polling is enabled') +
+      settingsToggle('Auto-fix Human Comments', 'set-autoFixHumanComments', e.autoFixHumanComments !== false, 'Shared dispatch gate: fix agent for actionable human PR comments; also requires that PR provider polling is enabled') +
+      settingsToggle('Auto-apply review vote to PR', 'set-autoApplyReviewVote', !!e.autoApplyReviewVote, 'When ON, Minions review verdicts (APPROVE / REQUEST_CHANGES) automatically flip the platform vote on ADO/GitHub. When OFF (default), verdicts are informational only and the human casts the final vote.') +
+      settingsToggle('Eval Loop', 'set-evalLoop', e.evalLoop !== false, 'Auto-review implementations and iterate fix cycles until pass') +
+      settingsToggle('Auto-decompose', 'set-autoDecompose', e.autoDecompose !== false, 'Large implement items are auto-split into sub-tasks') +
+      settingsToggle('Auto-complete PRs', 'set-autoCompletePrs', !!e.autoCompletePrs, 'Auto-merge PRs when builds pass and review is approved (opt-in)') +
+      settingsToggle('Auto-approve Plans', 'set-autoApprovePlans', !!e.autoApprovePlans, 'PRDs are approved automatically without human review') +
+      settingsToggle('Auto-archive Plans', 'set-autoArchive', !!e.autoArchive, 'Automatically archive plans after verify completes (off = manual archive via dashboard)') +
+      settingsToggle('Auto-consolidate Memory', 'set-autoConsolidateMemory', !!e.autoConsolidateMemory, 'Periodically spawn the KB sweep (dedup + compress + normalize knowledge/) from the engine tick on a 4h cadence. Inbox→notes consolidation already runs every tick (gated by the Consolidation Threshold above); this toggle controls only the KB sweep that was previously dashboard-button-only.') +
+    '</div>' +
+    '<div class="settings-grid-2">' +
+      settingsField('Eval Max Cost', 'set-evalMaxCost', e.evalMaxCost === null || e.evalMaxCost === undefined ? '' : e.evalMaxCost, '$', 'USD ceiling per work item across all eval iterations (blank = no limit)') +
+    '</div>';
+
+  const projectsHtml = (data.projects || []).map(function(p) {
+    // Cross-reference live /api/status to pull in the auto-detected
+    // remoteDefaultBranch (read-only) — settings already has localPath +
+    // mainBranch from /api/settings, but origin/HEAD is probe-derived and
+    // lives in /api/status only. W-mpg3whgp000d09ec / #2732.
+    var liveProj = null;
+    try {
+      if (window._lastStatus && Array.isArray(window._lastStatus.projects)) {
+        liveProj = window._lastStatus.projects.find(function(x) { return x.name === p.name; }) || null;
+      }
+    } catch (e) { liveProj = null; }
+    var remoteDefault = (liveProj && liveProj.remoteDefaultBranch) || '';
+    var mismatch = !!(liveProj && liveProj.branchMismatch);
+    var localBranch = (liveProj && liveProj.gitBranch) || '';
+    var driftNote = mismatch
+      ? '<div style="font-size:10px;color:var(--yellow);margin-top:4px">⚠ Configured main (<code>' + escHtml(p.mainBranch || '') + '</code>) differs from origin/HEAD (<code>' + escHtml(remoteDefault) + '</code>) — config is likely stale.</div>'
+      : '';
+    var pathRow = p.localPath
+      ? '<div style="font-size:10px;color:var(--muted);margin-bottom:6px;font-family:ui-monospace,SFMono-Regular,Menlo,monospace">' + escHtml(p.localPath) + '</div>'
+      : '';
+    var branchGrid = '<div style="display:grid;grid-template-columns:1fr 1fr;gap:8px;margin-bottom:6px">' +
+      settingsField('Configured main branch', 'set-mainBranch-' + p.name, p.mainBranch || '', '', 'Used by branch-naming + dependency-merge to identify mainline. Empty = auto-detect from origin/HEAD.') +
+      '<div>' +
+        '<label style="font-size:10px;color:var(--muted);display:block;margin-bottom:2px">Remote default <span style="opacity:0.6">(read-only)</span></label>' +
+        '<input id="set-remoteDefaultBranch-' + p.name + '" value="' + escHtml(remoteDefault || '(unset)') + '" readonly style="width:100%;padding:4px 6px;background:var(--surface2);border:1px solid var(--border);border-radius:4px;color:var(--muted);font-size:12px;font-family:ui-monospace,SFMono-Regular,Menlo,monospace">' +
+        '<div style="font-size:9px;color:var(--muted);margin-top:1px">Parsed from <code>git symbolic-ref refs/remotes/origin/HEAD</code>' + (localBranch ? '. Local HEAD: <code>' + escHtml(localBranch) + '</code>' : '') + '.</div>' +
+      '</div>' +
+    '</div>';
+    return '<div data-settings-project="' + escHtml(p.name) + '" data-search="project ' + escHtml(p.name.toLowerCase()) + '" style="border:1px solid var(--border);border-radius:6px;padding:10px 12px;margin-bottom:12px">' +
+      '<div style="display:flex;align-items:center;justify-content:space-between;margin-bottom:8px">' +
+        '<div style="font-size:12px;font-weight:600">' + escHtml(p.name) + '</div>' +
+        '<button onclick="MinionsSettings.removeProject(\'' + escHtml(p.name) + '\')" style="font-size:9px;padding:2px 8px;background:transparent;color:var(--red);border:1px solid var(--red);border-radius:3px;cursor:pointer">Remove</button>' +
+      '</div>' +
+      pathRow +
+      branchGrid +
+      driftNote +
+      '<div style="display:flex;flex-direction:column;gap:6px;margin-top:8px">' +
+      settingsToggle('Discover from PRs', 'set-ws-prs-' + p.name, p.workSources.pullRequests.enabled, 'Discovery gate: scan repo for open PRs and surface them as review tasks. Independent of ADO/GitHub polling — does not affect already-tracked PRs.') +
+      settingsToggle('Discover from Work Items', 'set-ws-wi-' + p.name, p.workSources.workItems.enabled, 'Auto-discover work from ADO/GitHub work items') +
+      '</div></div>';
+  }).join('');
 
-    '<h3 style="font-size:13px;color:var(--blue);margin-bottom:8px">Routing Table</h3>' +
-    '<textarea id="set-routing" rows="12" style="width:100%;padding:8px;background:var(--surface);border:1px solid var(--border);border-radius:4px;color:var(--text);font-family:monospace;font-size:11px;resize:vertical">' + escHtml(data.routing || '') + '</textarea>' +
+  const paneProjects =
+    '<h3>Projects</h3>' +
+    '<div class="settings-pane-sub">Repositories linked to this Minions install. Each project controls its own discovery sources + mainline branch. Add new projects via the Projects bar at the top of the dashboard or <code>minions add &lt;path&gt;</code>.</div>' +
+    (projectsHtml || '<div class="settings-pane-empty">No projects linked yet.</div>');
+
+  const panePolling =
+    '<h3>Polling</h3>' +
+    '<div class="settings-pane-sub">Cadence for fetching PR build status, votes, and comments from the platforms. Disabling a provider here turns the matching Auto-fix gates into no-ops.</div>' +
+    '<div class="settings-stack" style="margin-bottom:12px">' +
+      settingsToggle('ADO Polling', 'set-adoPollEnabled', e.adoPollEnabled !== false, 'Keeps ADO PR build results, votes, and comments fresh each tick; ADO PR dispatch gates are inert when this is off') +
+      settingsToggle('GitHub Polling', 'set-ghPollEnabled', e.ghPollEnabled !== false, 'Keeps GitHub PR build results, votes, and comments fresh each tick; GitHub PR dispatch gates are inert when this is off') +
+    '</div>' +
+    '<div class="settings-grid-2">' +
+      settingsField('PR Status Poll Frequency', 'set-prPollStatusEvery', e.prPollStatusEvery ?? 12, 'ticks', 'Poll PR build/review/merge status every N ticks for both ADO and GitHub (~12 min at default tick rate)') +
+      settingsField('PR Comments Poll Frequency', 'set-prPollCommentsEvery', e.prPollCommentsEvery ?? 12, 'ticks', 'Poll PR human comments every N ticks for both ADO and GitHub (~12 min at default tick rate)') +
+    '</div>';
+
+  const paneConcurrency =
+    '<h3>Concurrency &amp; Timeouts</h3>' +
+    '<div class="settings-pane-sub">How many agents run at once and how long they get before being killed for silence or runaway duration.</div>' +
+    '<div class="settings-grid-2">' +
+      settingsField('Max Concurrent Agents', 'set-maxConcurrent', e.maxConcurrent || 5, '', 'Max agents working simultaneously') +
+      settingsField('Tick Interval', 'set-tickInterval', e.tickInterval || 60000, 'ms', 'How often the engine runs discovery + dispatch') +
+      settingsField('Agent Timeout', 'set-agentTimeout', e.agentTimeout || 18000000, 'ms', 'Kill agent after this duration') +
+      settingsField('Heartbeat Timeout', 'set-heartbeatTimeout', e.heartbeatTimeout || 300000, 'ms', 'No output = dead after this') +
+      settingsField('Idle Alert', 'set-idleAlertMinutes', e.idleAlertMinutes || 15, 'min', 'Alert after agent idle this long') +
+      settingsField('Agent Busy Reassign', 'set-agentBusyReassignMs', e.agentBusyReassignMs || 600000, 'ms', 'Reassign work to another agent after it waits this long on a busy agent') +
+      settingsField('Max Retries Per Agent', 'set-maxRetriesPerAgent', e.maxRetriesPerAgent ?? 2, '', 'After the same agent fails the same work item this many times, the next retry reassigns to a different eligible agent (consults routing.md + availability). Falls back to the same agent only when no alternate is available. Counted separately from total maxRetries (which still caps overall retries).') +
+      settingsField('Restart Grace Period', 'set-restartGracePeriod', e.restartGracePeriod || 1200000, 'ms', 'Grace period before orphan detection on restart') +
+      settingsField('Shutdown Timeout', 'set-shutdownTimeout', e.shutdownTimeout || 300000, 'ms', 'Max wait for agents during graceful shutdown') +
+      settingsField('Meeting Round Timeout', 'set-meetingRoundTimeout', e.meetingRoundTimeout || 900000, 'ms', 'Auto-advance meeting round after this') +
+    '</div>';
 
-    // Toggles persist immediately via POST /api/features/toggle — no Save needed.
-    '<details id="settings-features-details" style="margin-top:16px;border-top:1px solid var(--border);padding-top:12px">' +
-      '<summary style="cursor:pointer;font-size:13px;color:var(--blue);user-select:none">Show experimental flags ' +
+  const paneWorktree =
+    '<h3>Worker Pool &amp; Worktrees</h3>' +
+    '<div class="settings-pane-sub">Persistent CC worker pool and git worktree provisioning. Tune these only if you see worktree-create timeouts or slow CC cold-spawns.</div>' +
+    '<div class="settings-stack" style="margin-bottom:12px">' +
+      settingsToggle('CC Worker Pool', 'set-ccUseWorkerPool', (e.ccUseWorkerPool === undefined ? ((e.ccCli || e.defaultCli) === 'copilot') : !!e.ccUseWorkerPool), 'Route Command Center / doc-chat through a persistent copilot --acp worker per tab instead of spawning a fresh CLI per turn. Copilot-only (Agent Client Protocol transport); Claude does not implement ACP, so this toggle has no effect when CC runtime is Claude. Default ON for copilot (cold-spawn ~20s on Windows); forced OFF for non-copilot CC runtimes regardless of this toggle.') +
+    '</div>' +
+    '<div class="settings-grid-2">' +
+      settingsField('Worktree Create Timeout', 'set-worktreeCreateTimeout', e.worktreeCreateTimeout || 300000, 'ms', 'Timeout for git worktree add (increase for large repos/Windows)') +
+      settingsField('Worktree Create Retries', 'set-worktreeCreateRetries', e.worktreeCreateRetries || 1, '', 'Retry count for transient worktree add failures (0-3)') +
+      settingsField('Worktree Root', 'set-worktreeRoot', e.worktreeRoot || '../worktrees', '', 'Relative or absolute path for git worktrees; on Windows prefer a short path like C:\\wt') +
+    '</div>';
+
+  const paneCopilot =
+    '<h3>Copilot Tuning</h3>' +
+    '<div class="settings-pane-sub">Knobs that only apply when the runtime is Copilot. Includes the split-brain warning for the built-in MCPs toggle.</div>' +
+    '<div class="settings-stack" style="margin-bottom:12px">' +
+      // Tooltip on copilotDisableBuiltinMcps MUST warn about the split-brain risk
+      settingsToggle('Copilot: disable built-in MCPs', 'set-copilotDisableBuiltinMcps', e.copilotDisableBuiltinMcps !== false,
+        '⚠ When OFF, Copilot agents can autonomously create PRs/labels/comments via the github-mcp-server, bypassing pull-requests.json tracking — Minions and Copilot end up with split views of the same PR. Keep ON unless you understand the risk.') +
+      settingsToggle('Copilot: suppress AGENTS.md', 'set-copilotSuppressAgentsMd', e.copilotSuppressAgentsMd !== false, '--no-custom-instructions: stops AGENTS.md auto-load from fighting Minions playbook prompts') +
+      settingsToggle('Copilot: reasoning summaries', 'set-copilotReasoningSummaries', !!e.copilotReasoningSummaries, '--enable-reasoning-summaries (Anthropic-family models only)') +
+    '</div>' +
+    '<div class="settings-grid-2">' +
+      '<div data-search="copilot stream mode incremental batched">' +
+        '<label style="font-size:10px;color:var(--muted);display:block;margin-bottom:2px">Copilot stream</label>' +
+        '<select id="set-copilotStreamMode" style="width:100%;padding:4px 6px;background:var(--surface);border:1px solid var(--border);border-radius:4px;color:var(--text);font-size:12px">' +
+          '<option value="on"' + ((e.copilotStreamMode || 'on') === 'on' ? ' selected' : '') + '>on (incremental)</option>' +
+          '<option value="off"' + (e.copilotStreamMode === 'off' ? ' selected' : '') + '>off (batched)</option>' +
+        '</select>' +
+      '</div>' +
+      settingsField('Copilot fallback model', 'set-copilotFallbackModel', e.copilotFallbackModel || '', 'e.g. gpt-5.4', 'Copilot has no --fallback-model flag. On a MODEL_UNAVAILABLE (overloaded/503) retry, the engine OVERRIDES --model with this value (Copilot only).') +
+    '</div>';
+
+  const paneClaude =
+    '<h3>Claude Tuning</h3>' +
+    '<div class="settings-pane-sub">Knobs that only apply when the runtime is Claude. Permission bypass is runtime-owned and not configurable here.</div>' +
+    '<div class="settings-stack" style="margin-bottom:12px">' +
+      settingsToggle('Claude bare mode', 'set-claudeBareMode', !!e.claudeBareMode, '--bare suppresses CLAUDE.md auto-discovery; pair with explicit ccSystemPrompt or context will be lost') +
+    '</div>' +
+    '<div class="settings-grid-2">' +
+      settingsField('Claude fallback model', 'set-claudeFallbackModel', e.claudeFallbackModel || '', 'e.g. sonnet', 'Passed via Claude --fallback-model. The Claude CLI swaps to this only on rate-limit (429). Used together with engine.copilotFallbackModel for the engine-level MODEL_UNAVAILABLE (overloaded/503) retry — see CLAUDE.md.') +
+    '</div>' +
+    '<div style="display:grid;grid-template-columns:1fr;gap:8px;margin-top:12px">' +
+      settingsField('Allowed Tools', 'set-allowedTools', c.allowedTools || '', '', 'Claude allow-list passed through for compatibility; runtime bypass flags are adapter-owned.') +
+    '</div>' +
+    '<div data-search="permission bypass dangerously-skip-permissions autopilot allow-all" style="font-size:10px;color:var(--muted);margin-top:12px;padding:6px 8px;border:1px solid var(--border);border-radius:4px;background:var(--surface-subtle, rgba(130,160,210,0.08))">' +
+      'Permission bypass is runtime-owned: Claude agents use <code>--dangerously-skip-permissions</code>; Copilot agents use <code>--autopilot --allow-all --no-ask-user</code>. There is no dashboard permission-mode setting.' +
+    '</div>';
+
+  const paneBudget =
+    '<h3>Budget</h3>' +
+    '<div class="settings-pane-sub">Fleet-wide spend ceiling. Per-agent monthly caps live in the Agents table on the Runtime &amp; Models tab.</div>' +
+    '<div class="settings-grid-2">' +
+      settingsField('Max budget (USD)', 'set-maxBudgetUsd', e.maxBudgetUsd != null ? String(e.maxBudgetUsd) : '', '', 'Fleet ceiling for --max-budget-usd. 0 is a valid cap (read-only / dry-run). Empty = no cap. Claude only.') +
+    '</div>';
+
+  const paneMaxTurns =
+    '<h3>Max Turns by Task Type</h3>' +
+    '<div class="settings-pane-sub">How many tool-use turns each task type gets before forced stop. Blank = built-in default. The fleet-wide Max Turns acts as a fallback for any task type not listed.</div>' +
+    '<div class="settings-grid-2" style="margin-bottom:12px">' +
+      settingsField('Max Turns (fleet default)', 'set-maxTurns', e.maxTurns || 100, '', 'Claude CLI --max-turns per agent') +
+    '</div>' +
+    '<div class="settings-grid-3">' +
+      settingsField('Explore', 'set-mt-explore', (e.maxTurnsByType || {}).explore || '', '', 'Default: 30') +
+      settingsField('Ask', 'set-mt-ask', (e.maxTurnsByType || {}).ask || '', '', 'Default: 20') +
+      settingsField('Review', 'set-mt-review', (e.maxTurnsByType || {}).review || '', '', 'Default: 30') +
+      settingsField('Implement', 'set-mt-implement', (e.maxTurnsByType || {}).implement || '', '', 'Default: 75') +
+      settingsField('Fix', 'set-mt-fix', (e.maxTurnsByType || {}).fix || '', '', 'Default: 75') +
+      settingsField('Test', 'set-mt-test', (e.maxTurnsByType || {}).test || '', '', 'Default: 50') +
+      settingsField('Verify', 'set-mt-verify', (e.maxTurnsByType || {}).verify || '', '', 'Default: 100') +
+      settingsField('Plan', 'set-mt-plan', (e.maxTurnsByType || {}).plan || '', '', 'Default: 30') +
+      settingsField('Decompose', 'set-mt-decompose', (e.maxTurnsByType || {}).decompose || '', '', 'Default: 15') +
+    '</div>';
+
+  // Toggles persist immediately via POST /api/features/toggle — no Save needed.
+  const paneFeatures =
+    '<h3>Feature Flags</h3>' +
+    '<div class="settings-pane-sub">In-progress UX or behavior gates. Toggles persist immediately. Registry: <code>engine/features.js</code>. Env override: <code>MINIONS_FEATURE_&lt;NAME&gt;=1</code>.</div>' +
+    '<details id="settings-features-details" open style="border:1px solid var(--border);border-radius:6px;padding:12px">' +
+      '<summary style="cursor:pointer;font-size:12px;color:var(--text);user-select:none;margin-bottom:8px">Show experimental flags ' +
         '<span style="font-size:10px;color:var(--muted)">(' + featuresList.length + ' registered)</span>' +
       '</summary>' +
-      '<div style="font-size:10px;color:var(--muted);margin:8px 0 10px">In-progress UX or behavior gates. Toggles persist immediately. Registry: <code>engine/features.js</code>. Env override: <code>MINIONS_FEATURE_&lt;NAME&gt;=1</code>.</div>' +
       (featuresList.length === 0
         ? '<div style="font-size:11px;color:var(--muted);padding:12px;border:1px dashed var(--border);border-radius:4px;text-align:center">No experimental features registered. Add entries to <code>engine/features.js</code> to gate new work.</div>'
         : '<div style="display:flex;flex-direction:column;gap:8px">' +
@@ -314,7 +330,8 @@ async function openSettings() {
             if (f.addedIn) meta.push('added in ' + escHtml(f.addedIn));
             if (f.expires) meta.push('expires ' + escHtml(f.expires));
             meta.push(f.default ? 'default: on' : 'default: off');
-            return '<label data-feature-id="' + escHtml(f.id) + '" style="display:flex;align-items:flex-start;gap:8px;padding:8px;border:1px solid var(--border);border-radius:4px;cursor:pointer">' +
+            const searchText = (f.id + ' ' + (f.description || '')).toLowerCase().replace(/"/g, '');
+            return '<label data-feature-id="' + escHtml(f.id) + '" data-search="' + escHtml(searchText) + '" style="display:flex;align-items:flex-start;gap:8px;padding:8px;border:1px solid var(--border);border-radius:4px;cursor:pointer">' +
               '<input type="checkbox" data-feature-toggle="' + escHtml(f.id) + '"' + checked + ' style="margin-top:3px;cursor:pointer">' +
               '<div style="flex:1">' +
                 '<div style="font-size:11px;font-weight:600;color:var(--text)">' + escHtml(f.id) + expiredBadge + '</div>' +
@@ -324,9 +341,85 @@ async function openSettings() {
             '</label>';
           }).join('') +
         '</div>') +
-    '</details>' +
+    '</details>';
+
+  const paneAdvanced =
+    '<h3>Advanced</h3>' +
+    '<div class="settings-pane-sub">Routing table + low-level engine knobs that most operators never touch. Change with care.</div>' +
+    '<h4>Routing Table</h4>' +
+    '<div data-search="routing table playbook agent assignment" style="margin-bottom:16px">' +
+      '<textarea id="set-routing" rows="10" style="width:100%;padding:8px;background:var(--surface);border:1px solid var(--border);border-radius:4px;color:var(--text);font-family:monospace;font-size:11px;resize:vertical">' + escHtml(data.routing || '') + '</textarea>' +
+    '</div>' +
+    '<h4>Inbox &amp; Status Retention</h4>' +
+    '<div class="settings-grid-2">' +
+      settingsField('Consolidation Threshold', 'set-inboxConsolidateThreshold', e.inboxConsolidateThreshold || 5, 'notes', 'Inbox notes before auto-consolidation') +
+      settingsField('Status WorkItems Retention', 'set-statusWorkItemsRetentionDays', e.statusWorkItemsRetentionDays ?? 7, 'days', 'Trim done/failed/cancelled work items older than N days from the /api/status workItems slice (active items are always shipped). Cuts SPA payload from ~3MB to <500KB. Set to 0 to disable trimming (full list shipped, restoring legacy behavior).') +
+      settingsField('Status Meetings Retention', 'set-statusMeetingsRetentionDays', e.statusMeetingsRetentionDays ?? 7, 'days', 'Trim completed/archived meetings older than N days from the /api/status meetings slice (active meetings are always shipped). Cuts SPA payload from ~4.3MB to <500KB. Detail modal still fetches full transcripts via /api/meetings/:id. Set to 0 to disable trimming (full list shipped, restoring legacy behavior).') +
+      settingsField('Version Check Interval', 'set-versionCheckInterval', e.versionCheckInterval || 3600000, 'ms', 'How often to check npm for updates (default: 1 hour)') +
+    '</div>' +
+    '<h4>Operator &amp; Comments</h4>' +
+    '<div class="settings-grid-2">' +
+      settingsField('Operator login (used in branch names)', 'set-operatorLogin', e.operatorLogin || '', '', 'Override the human operator login used in user/<loginname>/<wi-id>-<slug> branches. Empty = auto-resolve via gh / git email / OS username (currently resolves to: ' + (e._resolvedOperatorLogin || 'unknown') + ')') +
+      settingsField('Ignored Comment Authors', 'set-ignoredCommentAuthors', (e.ignoredCommentAuthors || []).join(', '), '', 'Comma-separated usernames — comments auto-closed, never trigger fixes') +
+    '</div>' +
+    '<h4>Experimental Behavior</h4>' +
+    '<div class="settings-stack">' +
+      settingsToggle('Allow Temp Agents', 'set-allowTempAgents', !!e.allowTempAgents, 'Spawn ephemeral agents when all permanent agents are busy') +
+      settingsToggle('Disable model discovery', 'set-disableModelDiscovery', !!e.disableModelDiscovery, 'Skip /api/runtimes/<name>/models REST calls fleet-wide. Settings UI falls back to free-text.') +
+    '</div>';
+
+  // Section registry — order is intentional (Runtime + Auto-fix surface first
+  // per Caleb's feedback). Each entry maps a rail-button id → pane HTML.
+  const sections = [
+    { id: 'runtime', label: 'Runtime & Models', featured: true, html: paneRuntime },
+    { id: 'autofix', label: 'Auto-fix & Review Loop', featured: true, html: paneAutoFix },
+    { id: 'projects', label: 'Projects', html: paneProjects },
+    { id: 'polling', label: 'Polling', html: panePolling },
+    { id: 'concurrency', label: 'Concurrency & Timeouts', html: paneConcurrency },
+    { id: 'worktree', label: 'Worker Pool & Worktrees', html: paneWorktree },
+    { id: 'copilot', label: 'Copilot Tuning', html: paneCopilot },
+    { id: 'claude', label: 'Claude Tuning', html: paneClaude },
+    { id: 'budget', label: 'Budget', html: paneBudget },
+    { id: 'maxturns', label: 'Max Turns', html: paneMaxTurns },
+    { id: 'features', label: 'Feature Flags', html: paneFeatures },
+    { id: 'advanced', label: 'Advanced', html: paneAdvanced },
+  ];
+
+  // localStorage-persisted active tab. Falls back to the first section when no
+  // saved value or the saved value points at a section we no longer render.
+  const SAVED_TAB_KEY = 'minions.settings.activeTab';
+  let activeTab = sections[0].id;
+  try {
+    const saved = localStorage.getItem(SAVED_TAB_KEY);
+    if (saved && sections.some(s => s.id === saved)) activeTab = saved;
+  } catch { /* localStorage may be blocked in some embeds */ }
+
+  const railHtml = sections.map(s =>
+    '<button class="settings-rail-btn' + (s.featured ? ' featured' : '') + (s.id === activeTab ? ' active' : '') + '" data-settings-tab="' + s.id + '" type="button">' +
+      '<span class="rail-label">' + escHtml(s.label) + '</span>' +
+      '<span class="match-count" style="display:none">0</span>' +
+    '</button>'
+  ).join('');
 
-  '</div>';
+  const panesHtml = sections.map(s =>
+    '<div class="settings-pane' + (s.id === activeTab ? ' active' : '') + '" data-settings-pane="' + s.id + '">' +
+      s.html +
+    '</div>'
+  ).join('');
+
+  const html =
+    '<div class="settings-layout">' +
+      '<aside class="settings-rail">' +
+        '<div class="settings-search-wrap">' +
+          '<input id="settings-search" class="settings-search" type="text" placeholder="Search settings…" autocomplete="off">' +
+        '</div>' +
+        '<nav class="settings-rail-nav" id="settings-rail-nav">' + railHtml + '</nav>' +
+      '</aside>' +
+      '<div class="settings-content" id="settings-content">' +
+        '<div id="settings-status" style="font-size:11px;min-height:16px;margin-bottom:8px"></div>' +
+        panesHtml +
+      '</div>' +
+    '</div>';
 
   document.getElementById('modal-title').textContent = 'Settings';
 
@@ -351,12 +444,34 @@ async function openSettings() {
     saveBtn.onclick = saveSettings;
     actions.insertBefore(saveBtn, actions.lastElementChild);
   }
+  const body = document.getElementById('modal-body');
+  body.classList.add('settings-body');
   // eslint-disable-next-line no-unsanitized/property -- reason: structural HTML is a string literal; all user data wrapped in escHtml() (fields: engine, claude, agents, projects, routing, features)
-  document.getElementById('modal-body').innerHTML = '<div id="settings-status" style="font-size:11px;min-height:16px;margin-bottom:4px"></div>' + html;
-  document.getElementById('modal-body').style.fontFamily = '';
-  document.getElementById('modal-body').style.whiteSpace = '';
+  body.innerHTML = html;
+  body.style.fontFamily = '';
+  body.style.whiteSpace = '';
   document.getElementById('modal').classList.add('open');
 
+  // ── Wire up rail buttons (tab switching + localStorage persist) ─────────
+  body.querySelectorAll('[data-settings-tab]').forEach(function(btn) {
+    btn.addEventListener('click', function() {
+      const id = btn.getAttribute('data-settings-tab');
+      _selectSettingsTab(id);
+      try { localStorage.setItem(SAVED_TAB_KEY, id); } catch { /* ignore */ }
+    });
+  });
+
+  // ── Cross-tab search box ───────────────────────────────────────────────
+  // Filters every [data-search] row across every pane. Empty query restores
+  // the active-tab view; non-empty query shows ALL panes with matches and
+  // hides unmatched rows so operators can see results without tab-hopping.
+  const searchInput = document.getElementById('settings-search');
+  if (searchInput) {
+    searchInput.addEventListener('input', function() {
+      _applySettingsSearch(searchInput.value || '');
+    });
+  }
+
   // ── Runtime fleet wiring (P-7a5c1f8e) ──────────────────────────────────────
   // 1. Load registered runtimes into the defaultCli + ccCli dropdowns.
   // 2. Load models for the selected defaultCli into the defaultModel input.
@@ -573,7 +688,11 @@ async function loadModelsForAgent(agentId, runtimeName, currentValue) {
 }
 
 function settingsToggle(label, id, checked, hint) {
-  return '<div style="display:flex;align-items:center;gap:8px;padding:4px 0">' +
+  // data-search powers the cross-tab search box — lowercase label + hint so
+  // the input filter can substring-match without dragging the visible text
+  // through .toLowerCase() on every keystroke.
+  const searchKey = (String(label || '') + ' ' + String(hint || '')).toLowerCase().replace(/"/g, '');
+  return '<div class="settings-row" data-search="' + escHtml(searchKey) + '" style="display:flex;align-items:center;gap:8px;padding:4px 0">' +
     '<input type="checkbox" id="' + id + '"' + (checked ? ' checked' : '') + ' style="accent-color:var(--blue);width:16px;height:16px;cursor:pointer">' +
     '<label for="' + id + '" style="font-size:12px;color:var(--text);cursor:pointer">' + escHtml(label) + '</label>' +
     (hint ? '<span style="font-size:9px;color:var(--muted)">' + escHtml(hint) + '</span>' : '') +
@@ -581,13 +700,83 @@ function settingsToggle(label, id, checked, hint) {
 }
 
 function settingsField(label, id, value, unit, hint) {
-  return '<div>' +
+  const searchKey = (String(label || '') + ' ' + String(hint || '') + ' ' + String(unit || '')).toLowerCase().replace(/"/g, '');
+  return '<div class="settings-row" data-search="' + escHtml(searchKey) + '">' +
     '<label style="font-size:10px;color:var(--muted);display:block;margin-bottom:2px">' + escHtml(label) + (unit ? ' <span style="opacity:0.6">(' + escHtml(unit) + ')</span>' : '') + '</label>' +
     '<input id="' + id + '" value="' + escHtml(String(value)) + '" style="width:100%;padding:4px 6px;background:var(--surface);border:1px solid var(--border);border-radius:4px;color:var(--text);font-size:12px">' +
     (hint ? '<div style="font-size:9px;color:var(--muted);margin-top:1px">' + escHtml(hint) + '</div>' : '') +
   '</div>';
 }
 
+// ── Settings tab + search helpers ─────────────────────────────────────────
+// Extracted from openSettings so the tab switch / search-filter logic can be
+// unit-tested without rebuilding the whole modal. Both operate on whatever
+// DOM is currently mounted under #modal-body — no module state required.
+function _selectSettingsTab(id) {
+  const body = document.getElementById('modal-body');
+  if (!body) return;
+  body.querySelectorAll('[data-settings-tab]').forEach(function(btn) {
+    btn.classList.toggle('active', btn.getAttribute('data-settings-tab') === id);
+  });
+  body.querySelectorAll('[data-settings-pane]').forEach(function(pane) {
+    pane.classList.toggle('active', pane.getAttribute('data-settings-pane') === id);
+  });
+}
+
+function _applySettingsSearch(rawQuery) {
+  const body = document.getElementById('modal-body');
+  if (!body) return;
+  const q = String(rawQuery || '').trim().toLowerCase();
+  const searching = q.length > 0;
+  body.classList.toggle('searching', searching);
+
+  // Reset every row + every rail badge before re-applying.
+  body.querySelectorAll('[data-search]').forEach(function(row) {
+    row.classList.remove('no-match', 'match-highlight');
+  });
+  const panes = body.querySelectorAll('[data-settings-pane]');
+  const railBtns = body.querySelectorAll('[data-settings-tab]');
+
+  if (!searching) {
+    // Restore single-active-pane mode.
+    panes.forEach(function(p) { p.classList.remove('no-match'); });
+    railBtns.forEach(function(btn) {
+      btn.classList.remove('no-match');
+      const badge = btn.querySelector('.match-count');
+      if (badge) badge.style.display = 'none';
+    });
+    return;
+  }
+
+  // Per-pane match counts drive both the rail badge and the pane visibility.
+  // Folding row.textContent into the haystack means operators can search by
+  // any visible label/hint string, not just the curated data-search tokens.
+  panes.forEach(function(pane) {
+    const paneId = pane.getAttribute('data-settings-pane');
+    let matches = 0;
+    pane.querySelectorAll('[data-search]').forEach(function(row) {
+      const haystack = (row.getAttribute('data-search') || '').toLowerCase();
+      const txt = (row.textContent || '').toLowerCase();
+      if (haystack.includes(q) || txt.includes(q)) {
+        row.classList.add('match-highlight');
+        matches++;
+      } else {
+        row.classList.add('no-match');
+      }
+    });
+    pane.classList.toggle('no-match', matches === 0);
+    const btn = body.querySelector('[data-settings-tab="' + paneId + '"]');
+    if (btn) {
+      btn.classList.toggle('no-match', matches === 0);
+      const badge = btn.querySelector('.match-count');
+      if (badge) {
+        badge.textContent = String(matches);
+        badge.style.display = matches > 0 ? 'inline-block' : 'none';
+      }
+    }
+  });
+}
+
 async function saveSettings() {
   const status = document.getElementById('settings-status');
   const saveBtn = document.getElementById('modal-settings-save');
@@ -827,4 +1016,4 @@ async function removeProject(name) {
   }
 }
 
-window.MinionsSettings = { openSettings, saveSettings, addProject, removeProject, resetSettingsToDefaults };
+window.MinionsSettings = { openSettings, saveSettings, addProject, removeProject, resetSettingsToDefaults, _selectSettingsTab, _applySettingsSearch };

package/dashboard/styles.css

@@ -710,6 +710,41 @@
 
   .modal-body { padding: var(--space-7) var(--space-8); overflow-y: auto; overflow-x: auto; white-space: pre-wrap; font-size: var(--text-md); line-height: 1.7; color: var(--muted); font-family: Consolas, monospace; }
 
+  /* ── Settings: left-rail tabbed layout (W-mpmwxkcn000646cc) ─────────────
+     Replaces the prior flat scroll of every engine.* knob with a Slack/VS-Code-style
+     vertical nav + per-tab pane. Search input filters control rows across all tabs
+     by data-search attribute. .modal.modal-wide is added by openSettings() so the
+     rail + content fit comfortably side-by-side. */
+  .modal-body.settings-body { padding: 0; white-space: normal; font-size: var(--text-md); line-height: 1.45; color: var(--text); font-family: 'Segoe UI', system-ui, sans-serif; }
+  .settings-layout { display: flex; min-height: 480px; max-height: calc(80vh - 64px); }
+  .settings-rail { width: 220px; min-width: 220px; background: var(--surface2); border-right: 1px solid var(--border); display: flex; flex-direction: column; overflow: hidden; }
+  .settings-search-wrap { padding: var(--space-5) var(--space-5) var(--space-4); border-bottom: 1px solid var(--border); }
+  .settings-search { width: 100%; padding: var(--space-3) var(--space-4); background: var(--surface); border: 1px solid var(--border); border-radius: var(--radius-sm); color: var(--text); font-size: var(--text-md); font-family: inherit; }
+  .settings-search:focus { outline: none; border-color: var(--blue); }
+  .settings-rail-nav { flex: 1; overflow-y: auto; padding: var(--space-3) 0; }
+  .settings-rail-btn { display: flex; align-items: center; justify-content: space-between; width: 100%; padding: var(--space-3) var(--space-6); background: transparent; border: none; border-left: 3px solid transparent; color: var(--muted); font-size: var(--text-md); font-family: inherit; cursor: pointer; text-align: left; transition: all var(--transition-fast); }
+  .settings-rail-btn:hover { color: var(--text); background: var(--surface); }
+  .settings-rail-btn.active { color: var(--blue); border-left-color: var(--blue); background: var(--surface); font-weight: 600; }
+  .settings-rail-btn.featured { color: var(--text); }
+  .settings-rail-btn.featured.active { color: var(--blue); }
+  .settings-rail-btn.no-match { opacity: 0.3; }
+  .settings-rail-btn .match-count { font-size: var(--text-xs); color: var(--blue); background: var(--bg); border-radius: var(--radius-xl); padding: 1px 6px; min-width: 18px; text-align: center; }
+  .settings-content { flex: 1; overflow-y: auto; padding: var(--space-7) var(--space-8); min-width: 0; }
+  .settings-pane { display: none; }
+  .settings-pane.active { display: block; }
+  .settings-pane h3 { font-size: var(--text-xl); color: var(--blue); margin-bottom: var(--space-3); font-weight: 600; }
+  .settings-pane .settings-pane-sub { font-size: var(--text-base); color: var(--muted); margin-bottom: var(--space-7); }
+  .settings-pane h4 { font-size: var(--text-lg); color: var(--text); margin: var(--space-7) 0 var(--space-4); font-weight: 600; }
+  .settings-grid-2 { display: grid; grid-template-columns: 1fr 1fr; gap: var(--space-4); }
+  .settings-grid-3 { display: grid; grid-template-columns: 1fr 1fr 1fr; gap: var(--space-4); }
+  .settings-stack { display: flex; flex-direction: column; gap: var(--space-3); }
+  .settings-row { padding: 2px 0; }
+  .settings-row.no-match { display: none; }
+  .settings-row.match-highlight { background: rgba(88,166,255,0.08); border-left: 2px solid var(--blue); padding-left: var(--space-3); border-radius: var(--radius-sm); }
+  .settings-body.searching .settings-pane { display: block; }
+  .settings-body.searching .settings-pane.no-match { display: none; }
+  .settings-pane-empty { font-size: var(--text-base); color: var(--muted); padding: var(--space-6) 0; font-style: italic; }
+
   /* Responsive: tablet / narrow window */
   @media (max-width: 1100px) {
     .layout { grid-template-columns: 1fr; }

package/docs/deprecated.json

@@ -1,19 +1,19 @@
 [
   {
     "id": "config-poll-key-migration",
-    "location": "engine/queries.js:123-162",
+    "location": "engine/queries.js:126-163",
     "function": "migrateDeprecatedConfigPollKeysOnce",
     "reason": "One-time boot migration that renames legacy engine config keys (adoPollStatusEvery → prPollStatusEvery, adoPollCommentsEvery → prPollCommentsEvery). Keys were renamed on 2026-04-16; this shim ensures any persisted config.json on a long-lived host gets transparently rewritten on next boot.",
     "targetRemovalDate": "2026-08-09",
-    "notes": "Safe to delete on or after 2026-08-09 (3 months after the rename) once we are confident every engine has rebooted at least once and the closure-flagged migration (_configPollKeyMigrationChecked at line 123) is no longer reachable on any persisted config. Removal is one function + the call site in getConfig() at engine/queries.js:165."
+    "notes": "Safe to delete on or after 2026-08-09 (3 months after the rename) once we are confident every engine has rebooted at least once and the closure-flagged migration (_configPollKeyMigrationChecked at line 124) is no longer reachable on any persisted config. Removal is one function + the call site in getConfig() at engine/queries.js:166."
   },
   {
     "id": "legacy-done-aliases",
-    "location": "engine/cleanup.js:970-972",
+    "location": "engine/cleanup.js:1052-1054",
     "constants": ["LEGACY_DONE_ALIASES", "LEGACY_NEEDS_REVIEW_STATUS"],
     "reason": "Read-side tolerance: cleanup sweep auto-migrates four obsolete work-item / PRD status strings ('in-pr', 'implemented', 'complete', 'needs-human-review') to the canonical 'done' / 'failed' values. The aliases are no longer written anywhere in the engine; the constants exist only to repair stale on-disk values from old engine versions.",
     "targetRemovalDate": null,
-    "notes": "Keep indefinitely until telemetry / a sweep log shows zero migrations performed for 30 consecutive days across all known projects (work-items.json + prd/*.json). At that point the constants and both _migrateLegacyItem branches in engine/cleanup.js (definitions at :970-972; usage at :973-1000 for work items and :1051-1057 for PRD missing_features) can be deleted. Total cost on disk today: 4 strings."
+    "notes": "Keep indefinitely until telemetry / a sweep log shows zero migrations performed for 30 consecutive days across all known projects (work-items.json + prd/*.json). At that point the constants and both _migrateLegacyItem branches in engine/cleanup.js (definitions at :1052-1054; usage at :1055-1071 for work items and :1156-1162 for PRD missing_features) can be deleted. Total cost on disk today: 4 strings."
   },
   {
     "id": "completion-fallback-parsers",
@@ -27,6 +27,51 @@
     "enforcingTest": "test/unit/completion-fallback-telemetry.test.js:217-234",
     "enforcingSweepWindowTest": "test/unit/completion-fallback-sweep-window.test.js",
     "notes": "Do NOT set removedAt until telemetry confirms zero usage across the sweepWindowDays from sweepStartDate. The follow-up code-removal PR (dropping parseStructuredCompletion at engine/lifecycle.js:2856, parseCompletionFieldSummary at :3054, and the gated fallback at :3852-3877) is dispatched separately once the window is observed clean."
+  },
+  {
+    "id": "config-claude-binary-override",
+    "description": "Legacy `config.claude.binary` runtime-resolution override. Older `minions init` versions persisted a `config.claude.binary` field that pointed the Claude runtime at a specific binary path. The runtime adapter still honors this override on every Claude spawn; the engine emits a `deprecated-config-claude` warning at config-load time but does NOT delete the override, so the override branch in claude.js is load-bearing for any install that still carries a non-default value.",
+    "code": [
+      { "file": "engine/runtimes/claude.js", "lines": "82-93", "note": "resolveBinary() respects config.claude.binary on every Claude spawn (probes npm package dir or direct binary path)" },
+      { "file": "engine/shared.js", "lines": "2311-2326", "note": "warnings.push({ id: 'deprecated-config-claude' }) — surface-only; never deletes the override" },
+      { "file": "engine/shared.js", "lines": "2770-2774", "note": "DEFAULT_CLAUDE.binary baseline that the warning + prune logic compares against" }
+    ],
+    "removalGate": "Telemetry: the `deprecated-config-claude` warning emitted at engine/shared.js:2321-2324 must report zero hits across all known engines for >=30 consecutive days, AND a sweep of every persisted config.json must show no `config.claude.binary` value that diverges from DEFAULT_CLAUDE.binary. Only then is the override branch in resolveBinary() (engine/runtimes/claude.js:82-93) removable, along with the `_deprecatedConfigClaudeFields` membership for `binary` and the warning emitter at engine/shared.js:2311-2326.",
+    "targetRemovalDate": null,
+    "notes": "Do NOT set targetRemovalDate — removal must be signal-gated, not calendar-gated. This entry is paired with `prune-default-claude-config`: the prune strips DEFAULT-matching values but intentionally preserves user overrides, which is precisely why the override branch in claude.js stays reachable. Removing the override before the prune entry's gate clears would silently break installs that still rely on a custom binary path."
+  },
+  {
+    "id": "legacy-cc-model-migration",
+    "description": "applyLegacyCcModelMigration: in-memory shim that promotes legacy `engine.ccModel` to `engine.defaultModel` when defaultModel is unset, so single-model installs keep working after the runtime fleet refactor (P-3b8e5f1d). No on-disk rewrite — the persisted config.json continues to carry the legacy `ccModel` field. Called unconditionally on every engine boot from cli.start().",
+    "code": [
+      { "file": "engine/shared.js", "lines": "2236", "note": "applyLegacyCcModelMigration definition (function signature + once-per-process flag via _resetLegacyCcModelMigrationFlag)" },
+      { "file": "engine/cli.js", "lines": "471-472", "note": "Boot call site inside start(); wrapped in try/catch so a migration failure cannot block startup" },
+      { "file": "CLAUDE.md", "lines": "316", "note": "Architectural documentation calling out the in-memory promotion contract" },
+      { "file": "docs/slim-ux/concepts.md", "lines": "671", "note": "Surface-level concepts doc cross-reference" },
+      { "file": "test/unit.test.js", "lines": "19402", "note": "Source-inspection test pinning the CLAUDE.md description against the function name" },
+      { "file": "test/unit/runtime-fleet-helpers.test.js", "lines": "209-254", "note": "Behavioural unit tests (promotion, no-op when defaultModel set, no-op when ccModel unset, empty-string handling, once-only logging, null-safety)" },
+      { "file": "test/unit/runtime-fleet-helpers.test.js", "lines": "500-505", "note": "Source-inspection test pinning the cli.js boot call site" }
+    ],
+    "removalGate": "Telemetry: the once-per-boot deprecation log line emitted by applyLegacyCcModelMigration (via the injected logger at engine/shared.js:2236) must show zero promotion events across all known engines for >=30 consecutive days, AND a sweep of every persisted config.json must confirm no `engine.ccModel` field remains. Once both conditions hold, removal deletes the function + _resetLegacyCcModelMigrationFlag export at engine/shared.js:4977, the boot call at engine/cli.js:471-472, the CLAUDE.md:316 paragraph and docs/slim-ux/concepts.md:671 reference, and the tests at runtime-fleet-helpers.test.js:209-254 + :500-505 + unit.test.js:19402.",
+    "targetRemovalDate": null,
+    "notes": "Do NOT set targetRemovalDate — gating is signal-based. The function is silent on no-op (returns false without logging), so the meaningful telemetry signal is the absence of the promotion log line over the sweep window, NOT the absence of function invocations (cli.js calls it every boot regardless)."
+  },
+  {
+    "id": "prune-default-claude-config",
+    "description": "pruneDefaultClaudeConfig: active sanitizer that strips generated `config.claude.{binary,outputFormat,allowedTools,permissionMode}` defaults from persisted config.json so the `deprecated-config-claude` warning stops tripping on stale defaults left by older `minions init` versions. Sub-cluster of `config-claude-binary-override` — the prune deliberately preserves non-default user overrides (binary/allowedTools), which is what keeps the override branch in engine/runtimes/claude.js load-bearing.",
+    "code": [
+      { "file": "engine/shared.js", "lines": "2776-2809", "note": "pruneDefaultClaudeConfig definition: preserves non-default binary/allowedTools, always strips permissionMode + outputFormat" },
+      { "file": "engine/shared.js", "lines": "4989", "note": "Module export entry" },
+      { "file": "dashboard.js", "lines": "196", "note": "Called when loading config for the dashboard UI" },
+      { "file": "dashboard.js", "lines": "8753", "note": "Called during first config save handler" },
+      { "file": "dashboard.js", "lines": "8983", "note": "Called during second config save path" },
+      { "file": "dashboard.js", "lines": "9102", "note": "Called during third config save path" },
+      { "file": "minions.js", "lines": "385", "note": "Called during CLI init/update flow" },
+      { "file": "test/unit.test.js", "lines": "2153-2196", "note": "Behavioural unit tests (default strip, override preservation, outputFormat unconditional strip) + dashboard call-site source pin" },
+      { "file": "test/unit/runtime-fleet-helpers.test.js", "lines": "546", "note": "Source-inspection test pinning the dashboard handler call site" }
+    ],
+    "removalGate": "Telemetry: pruneDefaultClaudeConfig must return false (no mutation) for every call across all known engines for >=30 consecutive days (add an `_engine.pruneDefaultClaudeConfigStrips` counter if needed to observe this), AND the parent `config-claude-binary-override` entry must have already cleared its own gate. The dependency is strict: removing the prune while users still rely on the override branch would surface the `deprecated-config-claude` warning on every stale generated default. Once both conditions hold, removal is the function definition (engine/shared.js:2776-2809), the export at :4989, all 5 call sites (dashboard.js:196, :8753, :8983, :9102; minions.js:385), and the tests at unit.test.js:2153-2196 + runtime-fleet-helpers.test.js:546.",
+    "targetRemovalDate": null,
+    "notes": "Do NOT set targetRemovalDate — gating is signal-based AND ordered. This entry MUST NOT be removed before `config-claude-binary-override` clears its gate, otherwise installs with stale defaults will flood the deprecation channel until their next config save. The 5 call sites form a complete coverage net: load (dashboard.js:196 + minions.js:385) + save (dashboard.js:8753/8983/9102), so any code path that touches config.json runs the sanitizer."
   }
 ]
-

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.2049",
+  "version": "0.1.2050",
   "description": "Multi-agent AI dev team that runs from ~/.minions/ — five autonomous agents share a single engine, dashboard, and knowledge base",
   "bin": {
     "minions": "bin/minions.js"