@yemi33/minions 0.1.2380 → 0.1.2382

package/bin/minions.js

@@ -336,7 +336,11 @@ function _sqliteSpawnFlags() {
  *  CLI's intent (EADDRINUSE retries still apply inside the dashboard if a
  *  race opens between the CLI's port-free check and the actual bind). */
 function spawnDashboard(requestedPort) {
-  const env = { ...process.env, MINIONS_NO_AUTO_OPEN: '1' };
+  const env = {
+    ...process.env,
+    MINIONS_NO_AUTO_OPEN: '1',
+    MINIONS_REQUIRE_DASHBOARD_PORT: '1',
+  };
   const out = _openStdioLog('dashboard-stdio.log');
   const err = _openStdioLog('dashboard-stdio.log');
   const args = [..._sqliteSpawnFlags(), path.join(MINIONS_HOME, 'dashboard.js')];
@@ -444,19 +448,43 @@ function spawnFullStackAndVerify({ rest, forceOpen, dashWasUp, restartStartMs })
     // failure rather than a false "healthy".
     const actualPort = await _waitForDashboardPortFile(MINIONS_HOME, 12000, 150, dashProc.pid) || requested.port;
     if (actualPort !== requested.port) {
-      console.log(`  Dashboard bound to alternate port ${actualPort} (requested ${requested.port} was busy)`);
+      console.error(`\n  ERROR: Dashboard bound to unexpected port ${actualPort}; required ${requested.port}.`);
+      console.error('  Refusing to verify a split-port Minions topology. Run `minions restart` again.');
+      _cleanupFailedStackSpawn({
+        enginePid: engineProc.pid,
+        dashboardPid: dashProc.pid,
+        supervisorPid: supProc.pid,
+        dashboardPort: actualPort,
+      });
+      process.exit(1);
     }
+    const supervisorInternals = require(path.join(PKG_ROOT, 'engine', 'supervisor'))._internals;
     const result = await waitForRestartHealth({
       minionsHome: MINIONS_HOME,
+      expectedEnginePid: engineProc.pid,
       dashboardPid: dashProc.pid,
       dashboardPort: actualPort,
+      supervisorPid: supProc.pid,
       timeoutMs: _resolveRestartHealthTimeoutMs(),
       // Verify the process bound to the port IS the dashboard we just spawned
       // (beacon pid match), not a stale leftover still holding it.
       requireBeaconOwner: true,
+      requireExactTopology: true,
+      listProcessPids: supervisorInternals.listNodePidsMatching,
+      topologyScripts: {
+        engine: path.join(MINIONS_HOME, 'engine.js'),
+        dashboard: path.join(MINIONS_HOME, 'dashboard.js'),
+        supervisor: path.join(MINIONS_HOME, 'engine', 'supervisor.js'),
+      },
     });
     if (!result.ok) {
       console.error(formatRestartHealthError(result));
+      _cleanupFailedStackSpawn({
+        enginePid: engineProc.pid,
+        dashboardPid: dashProc.pid,
+        supervisorPid: supProc.pid,
+        dashboardPort: actualPort,
+      });
       process.exit(1);
     }
     console.log(`  Restart verified: engine PID ${result.engine.pid}; dashboard healthy.`);
@@ -479,6 +507,17 @@ function spawnFullStackAndVerify({ rest, forceOpen, dashWasUp, restartStartMs })
   });
 }
 
+function _cleanupFailedStackSpawn({ enginePid, dashboardPid, supervisorPid, dashboardPort }) {
+  writeStopIntent('restart verification failed');
+  for (const pid of [supervisorPid, dashboardPid, enginePid]) killPidOnly(pid);
+  for (const pid of [supervisorPid, dashboardPid, enginePid]) waitForPidDeath(pid, 5000);
+  if (dashboardPort) {
+    killByPort(dashboardPort);
+    waitForPortRelease(dashboardPort, 5000);
+  }
+  shared.clearDashboardPortFile(MINIONS_HOME);
+}
+
 /** Poll engine/dashboard-port.json until it appears (dashboard wrote it on
  *  listen) or the timeout elapses. Returns the bound port or null on
  *  timeout. Stale files left behind by a crashed dashboard are tolerated —

package/dashboard/js/refresh.js

@@ -138,7 +138,7 @@ const RENDER_VERSIONS = {
   // Bumped 4→5 for the clickable checkout-mode pill + picker (W-mr1b67zi0006b788).
   // Bumped 5→6 for multi-select hybrid liveValidation.type support (W-mr2m1ute000a9c01).
   // Bumped 6→7 for the liveValidation.autoDispatch toggle + pill "· auto-validate" suffix (W-mr2q361a00097e5c).
-  projects: 7,
+  projects: 9,
   notes: 1,
   prd: 3,
   prs: 3,
@@ -151,8 +151,11 @@ const RENDER_VERSIONS = {
   dispatch: 2,
   engineLog: 2,
   metrics: 1,
-  // Bumped 8->9: row actions now preserve work-item source scope when IDs collide.
-  workItems: 9,
+  // Bumped 7→8: WI detail modal's branch pill moved out of the Artifacts
+  // chip row into its own non-interactive "Branch" field (W-mr2qjr3b0002522f).
+  // Bumped 8→9 for paused pre-dispatch evaluation guidance.
+  // Bumped 9→10: row actions now preserve work-item source scope when IDs collide.
+  workItems: 10,
   skills: 1,
   commands: 1,
   mcpServers: 1,
@@ -840,6 +843,9 @@ function _processStatusUpdate(data, opts) {
       .catch(function () { /* keep render even if managed fetch failed — getLastItems() returns the last good cache (or []) */ })
       .then(function () { try { renderKeepProcesses(); } catch {} });
   }
+  if (typeof renderWorktreeQuarantineRefs === 'function') {
+    _safeRender('worktreeQuarantineRefs', function () { renderWorktreeQuarantineRefs(); });
+  }
   // Work items now come from /api/work-items — a dedicated fresh-JSON
   // endpoint that re-runs getWorkItems() server-side on every request
   // with input-mtime ETag (issue #2949). The previous /state/ + stale-

package/dashboard/js/render-other.js

@@ -105,13 +105,13 @@ function _renderProjectBranch(p) {
 // "Live checkout" (live — agents run in-place inside the operator's working
 // tree). Driven by p.checkoutMode (shared.CHECKOUT_MODES; engine defaults
 // unset → 'worktree'). Live is the riskier/special mode (capped to one mutating
-// dispatch per project, refuses on a dirty tree) so it gets a more prominent
-// color than the muted worktree pill.
+// dispatch per project, with configurable dirty-tree recovery) so it gets a
+// more prominent color than the muted worktree pill.
 //
 // Hybrid live-validation: when checkoutMode is 'live' AND p.liveValidationType
-// is set, coding WIs author in isolated worktrees and only the named validation
-// type runs in-place on the live checkout. This is a materially different
-// dispatch profile from full-live, so it gets its own pill.
+// is set, only selected WI types run in-place and all others use isolated
+// worktrees. This is a materially different dispatch profile from full-live,
+// so it gets its own pill.
 // W-mr1b67zi0006b788: the pill is clickable — opens an inline picker (see
 // _openCheckoutModeMenu below) to switch between Worktrees / Live checkout /
 // Hybrid dispatch modes without leaving the Projects view. Shared
@@ -125,6 +125,10 @@ function _renderWorktreeModePill(p) {
   const common = ' data-checkout-pill="' + name + '" role="button" tabindex="0" aria-haspopup="true" aria-label="Change checkout mode"';
   if (p.checkoutMode === 'live') {
     const types = _liveValidationTypesArray(p);
+    if (p.liveValidationInvalid) {
+      const effectiveTypes = types.length > 0 ? types.join(', ') : 'none';
+      return ' <span class="project-mode-pill project-mode-hybrid project-mode-pill-clickable project-warn"' + common + ' title="Invalid hybrid configuration — unsupported settings are ignored. Effective live work-item types: ' + escHtml(effectiveTypes) + '. Click to repair.">⚠ Hybrid · invalid</span>';
+    }
     if (types.length > 0) {
       const typesText = types.join(', ');
       const vt = escapeHtml(typesText);
@@ -134,20 +138,19 @@ function _renderWorktreeModePill(p) {
       const autoDispatch = p.liveValidationAutoDispatch === true;
       const autoSuffix = autoDispatch ? ' · auto-validate' : '';
       const autoTitleBit = autoDispatch
-        ? ' A validation work item is automatically dispatched after each coding work item completes.'
-        : ' Auto-validation is OFF — validation must be dispatched manually after each coding work item completes.';
-      return ' <span class="project-mode-pill project-mode-hybrid project-mode-pill-clickable"' + common + ' title="Hybrid live-validation — coding work items author in isolated worktrees; only the &quot;' + vt + '&quot; validation type(s) run in-place on the live checkout (capped to one mutating dispatch, refused on a dirty tree).' + escHtml(autoTitleBit) + ' Click to change.">⚡ Hybrid · ' + vt + escHtml(autoSuffix) + '</span>';
+        ? ' A test work item is automatically dispatched after each eligible coding work item completes.'
+        : ' Auto-validation is OFF — validation must be dispatched manually.';
+      return ' <span class="project-mode-pill project-mode-hybrid project-mode-pill-clickable"' + common + ' title="Hybrid dispatch — only the &quot;' + vt + '&quot; work-item type(s) run in-place; all others use isolated worktrees. Live work is capped to one mutating dispatch, and dirty-tree handling follows the configured auto-stash/reset policy.' + escHtml(autoTitleBit) + ' Click to change.">⚡ Hybrid · ' + vt + escHtml(autoSuffix) + '</span>';
     }
-    return ' <span class="project-mode-pill project-mode-live project-mode-pill-clickable"' + common + ' title="Live-checkout dispatch mode — agents run in-place inside the project working tree (no isolated worktree); capped to one mutating dispatch and refused on a dirty tree. Click to change.">⚡ Live checkout</span>';
+    return ' <span class="project-mode-pill project-mode-live project-mode-pill-clickable"' + common + ' title="Live-checkout dispatch mode — agents run in-place inside the project working tree (no isolated worktree); capped to one mutating dispatch. Dirty-tree handling follows the configured auto-stash/reset policy. Click to change.">⚡ Live checkout</span>';
   }
   return ' <span class="project-mode-pill project-mode-isolated project-mode-pill-clickable"' + common + ' title="Worktree dispatch mode (default) — each agent runs in its own git worktree. Click to change.">Worktrees</span>';
 }
 
-// Work-item types eligible for hybrid live-validation (W-mr1b67zi0006b788).
-// Mirrors the set the server documents for project.liveValidation.type;
-// intentionally excludes the illustrative "build-and-test" example from the
-// server comments since that isn't a real WORK_TYPE value.
-const CHECKOUT_MODE_HYBRID_TYPES = ['implement', 'fix', 'docs', 'decompose', 'explore', 'ask', 'review', 'test', 'verify', 'setup'];
+// Canonical work-item types that can meaningfully use the live checkout.
+// Read-only root tasks are omitted because hybrid routing cannot change where
+// they run. `build-and-test` is a playbook alias; `test` is the WI type.
+const CHECKOUT_MODE_HYBRID_TYPES = ['test', 'implement', 'implement:large', 'fix', 'build-fix-complex', 'docs', 'decompose', 'review', 'verify', 'setup'];
 const CHECKOUT_MODE_HYBRID_DEFAULT_TYPE = 'test';
 
 function _findProjectInLastStatus(name) {
@@ -230,9 +233,10 @@ function _openCheckoutModeMenu(projectName, anchorEl) {
       desc.textContent = descText;
       item.appendChild(desc);
     }
+    const liveTypes = _liveValidationTypesArray(project);
     const isActive = (mode === 'worktree' && project.checkoutMode !== 'live')
-      || (mode === 'live' && project.checkoutMode === 'live' && _liveValidationTypesArray(project).length === 0)
-      || (mode === 'hybrid' && project.checkoutMode === 'live' && _liveValidationTypesArray(project).length > 0);
+      || (mode === 'live' && project.checkoutMode === 'live' && liveTypes.length === 0 && !project.liveValidationInvalid)
+      || (mode === 'hybrid' && project.checkoutMode === 'live' && (liveTypes.length > 0 || project.liveValidationInvalid));
     if (isActive) {
       item.setAttribute('aria-checked', 'true');
       item.classList.add('checkout-mode-menu-item-active');
@@ -254,8 +258,8 @@ function _openCheckoutModeMenu(projectName, anchorEl) {
   }
 
   addItem('Worktrees (default)', 'worktree', 'Each dispatch runs in its own isolated git worktree. Fully parallel, never touches your working directory.');
-  addItem('Live checkout', 'live', 'Agents run in-place in the project folder. One mutating dispatch at a time; refuses to run on a dirty tree.');
-  addItem('Hybrid dispatch…', 'hybrid', 'Coding work authors in isolated worktrees; only a chosen validation step (e.g. build-and-test) runs on the real checkout.');
+  addItem('Live checkout', 'live', 'Agents run in-place in the project folder. One mutating dispatch at a time; dirty-tree handling follows the auto-stash/reset policy.');
+  addItem('Hybrid dispatch…', 'hybrid', 'Only chosen work-item types (normally test) use the real checkout; all others use isolated worktrees.');
 
   document.body.appendChild(menu);
   const rect = anchorEl.getBoundingClientRect();
@@ -286,7 +290,7 @@ function _renderCheckoutModeHybridStep(menu, projectName) {
 
   const label = document.createElement('div');
   label.className = 'checkout-mode-menu-item';
-  label.textContent = 'Work-item type(s) to validate live:';
+  label.textContent = 'Work-item type(s) to run live:';
   menu.appendChild(label);
 
   const currentTypes = _liveValidationTypesArray(project);
@@ -325,10 +329,23 @@ function _renderCheckoutModeHybridStep(menu, projectName) {
   autoDispatchCb.className = 'checkout-mode-menu-checkbox checkout-mode-menu-autodispatch-checkbox';
   autoDispatchCb.checked = project.liveValidationAutoDispatch === true;
   const autoDispatchText = document.createElement('span');
-  autoDispatchText.textContent = 'Automatically validate after each coding work item completes';
+  autoDispatchText.textContent = 'Automatically run a test validation after each eligible coding work item completes';
   autoDispatchRow.appendChild(autoDispatchCb);
   autoDispatchRow.appendChild(autoDispatchText);
   menu.appendChild(autoDispatchRow);
+  const testTypeCheckbox = checkboxes.find(function(cb) {
+    return cb.value === CHECKOUT_MODE_HYBRID_DEFAULT_TYPE;
+  });
+  function syncAutoDispatchType() {
+    if (autoDispatchCb.checked && testTypeCheckbox) testTypeCheckbox.checked = true;
+  }
+  autoDispatchCb.addEventListener('change', syncAutoDispatchType);
+  if (testTypeCheckbox) {
+    testTypeCheckbox.addEventListener('change', function() {
+      if (!testTypeCheckbox.checked) autoDispatchCb.checked = false;
+    });
+  }
+  syncAutoDispatchType();
 
   const actions = document.createElement('div');
   actions.className = 'checkout-mode-menu-actions';
@@ -349,9 +366,9 @@ function _renderCheckoutModeHybridStep(menu, projectName) {
   applyBtn.className = 'checkout-mode-menu-btn checkout-mode-menu-btn-primary';
   applyBtn.textContent = 'Apply';
   applyBtn.addEventListener('click', function() {
+    const autoDispatch = autoDispatchCb.checked;
     const types = checkboxes.filter(function(cb) { return cb.checked; }).map(function(cb) { return cb.value; });
     if (types.length === 0) return; // require at least one selected type
-    const autoDispatch = autoDispatchCb.checked;
     _closeCheckoutModeMenu();
     _applyCheckoutModeChange(projectName, 'live', types, autoDispatch);
   });
@@ -392,8 +409,8 @@ async function _applyCheckoutModeChange(projectName, newMode, liveValidationType
   // explicitly in the confirm message rather than burying it in the label.
   const autoDispatchNote = typesForLabel.length > 0
     ? (autoDispatchOn
-      ? ' Auto-validation will be turned ON — a validation work item will be dispatched automatically after each coding work item completes, and coding agents will skip local builds/tests.'
-      : ' Auto-validation will be OFF — you will need to dispatch validation manually after each coding work item completes.')
+      ? ' Auto-validation will be turned ON — a test work item will be dispatched automatically after each eligible coding work item completes, and eligible coding agents will skip local builds/tests.'
+      : ' Auto-validation will be OFF — you will need to dispatch validation manually.')
     : '';
   const ok = await confirmDialog({
     title: 'Change checkout mode?',
@@ -407,12 +424,14 @@ async function _applyCheckoutModeChange(projectName, newMode, liveValidationType
   const prevMode = project.checkoutMode;
   const prevType = project.liveValidationType;
   const prevAutoDispatch = project.liveValidationAutoDispatch;
+  const prevInvalid = project.liveValidationInvalid;
   const nextType = (newMode === 'live' && liveValidationType) ? liveValidationType : null;
   // Optimistic flip BEFORE awaiting the API call (per dashboard convention —
   // see projectChipRemove / removePinnedNote); reverted in the catch below.
   project.checkoutMode = newMode;
   project.liveValidationType = nextType;
   project.liveValidationAutoDispatch = nextType ? autoDispatchOn : false;
+  project.liveValidationInvalid = false;
   if (window._lastStatus && Array.isArray(window._lastStatus.projects)) renderProjects(window._lastStatus.projects);
 
   try {
@@ -434,6 +453,7 @@ async function _applyCheckoutModeChange(projectName, newMode, liveValidationType
     project.checkoutMode = prevMode;
     project.liveValidationType = prevType;
     project.liveValidationAutoDispatch = prevAutoDispatch;
+    project.liveValidationInvalid = prevInvalid;
     if (window._lastStatus && Array.isArray(window._lastStatus.projects)) renderProjects(window._lastStatus.projects);
     showToast('cmd-toast', 'Failed to update checkout mode: ' + (err && err.message || err), false);
   }
@@ -1230,3 +1250,41 @@ async function killKeepPid(agentId, pid) {
 }
 
 window.MinionsKeepProcesses = { renderKeepProcesses, killKeepPid, mountKeepProcessesPanel, unmountKeepProcessesPanel };
+
+async function renderWorktreeQuarantineRefs() {
+  const root = document.getElementById('worktree-quarantine-refs-content');
+  const count = document.getElementById('worktree-quarantine-refs-count');
+  if (!root) return;
+  let html;
+  try {
+    const res = await fetch('/api/worktree-quarantine-refs');
+    const data = await res.json();
+    if (!res.ok) throw new Error(data.error || String(res.status));
+    const items = Array.isArray(data.items) ? data.items : [];
+    if (count) count.textContent = String(items.length);
+    if (!items.length) {
+      html = '<p class="empty">No quarantined work refs found.</p>';
+    } else {
+      html = '<table style="width:100%;border-collapse:collapse"><thead><tr>' +
+        '<th style="text-align:left">Project</th><th style="text-align:left">Backup ref</th>' +
+        '<th style="text-align:left">SHA</th><th style="text-align:left">Created</th></tr></thead><tbody>' +
+        items.map(function (item) {
+          if (item.error) {
+            return '<tr><td>' + escHtml(item.project || '') + '</td><td colspan="3" style="color:var(--red)">' + escHtml(item.error) + '</td></tr>';
+          }
+          return '<tr><td>' + escHtml(item.project || '') + '</td>' +
+            '<td><code>' + escHtml(item.ref || '') + '</code></td>' +
+            '<td><code>' + escHtml(String(item.sha || '').slice(0, 12)) + '</code></td>' +
+            '<td>' + escHtml(item.createdAt || '') + '</td></tr>';
+        }).join('') +
+        '</tbody></table>';
+    }
+  } catch (e) {
+    if (count) count.textContent = '?';
+    html = '<span style="color:var(--red)">Failed to load: ' + escHtml(e.message) + '</span>';
+  }
+  // eslint-disable-next-line no-unsanitized/method -- structural HTML only; API fields are escaped above
+  root.replaceChildren(document.createRange().createContextualFragment(html));
+}
+
+window.MinionsWorktreeQuarantineRefs = { renderWorktreeQuarantineRefs };

package/dashboard/js/render-work-items.js

@@ -52,10 +52,17 @@ function needsAttentionInfo(item) {
   if (item._preDispatchEval && item._preDispatchEval.valid === false) {
     var rawReason = String(item._preDispatchEval.reason || '(no reason recorded)');
     var firstLine = rawReason.split(/\r?\n/, 1)[0] || rawReason;
+    var exhausted = item._preDispatchEval.exhausted === true;
+    var prefix = exhausted
+      ? 'Pre-dispatch evaluation paused after ' + (item._preDispatchEval.rejectCount || '?') + ' unchanged rejections: '
+      : 'Pre-dispatch evaluation rejected: ';
+    var fullReason = exhausted
+      ? rawReason + '\n\nEdit the work item description to resume evaluation, or cancel it.'
+      : rawReason;
     return {
       kind: 'pre_dispatch_eval',
-      short: 'Pre-dispatch evaluation rejected: ' + firstLine,
-      full: rawReason,
+      short: prefix + firstLine,
+      full: fullReason,
     };
   }
 
@@ -1065,7 +1072,9 @@ function viewAgentOutput(logPath) {
     });
 }
 
+let _inboxNoteOpenSeq = 0;
 function openInboxNote(filename) {
+  var openSeq = ++_inboxNoteOpenSeq;
   var idx = (inboxData || []).findIndex(function(item) { return item.name === filename; });
   if (idx >= 0) {
     openModal(idx);
@@ -1083,29 +1092,61 @@ function openInboxNote(filename) {
   // 'archive:<name>' token). Fetch it from notes/archive/ and show it in the
   // modal instead of bouncing to the inbox page. Falls back to the inbox page
   // only if the file is genuinely gone (e.g. merged into notes.md).
+  var title = filename + ' (archived)';
+  var filePath = 'notes/archive/' + filename;
+  var titleEl = document.getElementById('modal-title');
+  var bodyEl = document.getElementById('modal-body');
+  if (titleEl) titleEl.textContent = title;
+  if (bodyEl) {
+    bodyEl.innerHTML = '<p style="color:var(--muted)">Loading...</p>';
+    bodyEl.style.fontFamily = "'Segoe UI', system-ui, sans-serif";
+    bodyEl.style.whiteSpace = 'normal';
+  }
+  _modalDocContext = { title: title, content: '', selection: '' };
+  _modalFilePath = filePath;
+  var qaEl = document.getElementById('modal-qa');
+  if (qaEl) qaEl.style.display = 'none';
+  var modalEl = document.getElementById('modal');
+  if (modalEl) modalEl.classList.add('open');
+
+  // A note can be superseded by another note before its archive fetch resolves,
+  // or by a different artifact stacked above it. Guard both cases so delayed
+  // success and failure handlers only affect the selection that started them.
+  var applyIfCurrent = function(fn) {
+    if (openSeq !== _inboxNoteOpenSeq ||
+        _modalFilePath !== filePath ||
+        !modalEl ||
+        !modalEl.classList.contains('open')) return false;
+    if (typeof withTopFrame === 'function') return withTopFrame('note', filename, fn);
+    fn();
+    return true;
+  };
+
   fetch('/state/notes/archive/' + encodeURIComponent(filename))
     .then(function(r) { if (!r.ok) throw new Error('HTTP ' + r.status); return r.text(); })
     .then(function(content) {
-      var titleEl = document.getElementById('modal-title');
-      var bodyEl = document.getElementById('modal-body');
-      if (!titleEl || !bodyEl) return;
-      titleEl.textContent = filename + ' (archived)';
-      // eslint-disable-next-line no-unsanitized/property -- reason: renderMd() escapes note content before assembling HTML; filename is set via textContent
-      bodyEl.innerHTML = '<div style="font-size:var(--text-md);line-height:1.7;color:var(--muted)">' + renderMd(content) + '</div>';
-      // Wire Doc Chat for the deeplinked archive note, mirroring the live-inbox
-      // openModal() path (render-prs.js). Without this the archive branch opened
-      // the note as plain text with no Q&A panel, so deeplinking into a note
-      // from a (completed) work item sometimes silently dropped doc-chat.
-      _modalDocContext = { title: filename + ' (archived)', content: content, selection: '' };
-      _modalFilePath = 'notes/archive/' + filename; showModalQa();
-      document.getElementById('modal').classList.add('open');
-      // P-34fa5d79 — Source-WI chip for archive notes. Same convention as the
-      // live-inbox branch above; parsed off the freshly-fetched content.
-      _wiInjectSourceChipFromFrontmatter(content, 'modal-body');
+      applyIfCurrent(function() {
+        if (!titleEl || !bodyEl) return;
+        titleEl.textContent = title;
+        // eslint-disable-next-line no-unsanitized/property -- reason: renderMd() escapes note content before assembling HTML; filename is set via textContent
+        bodyEl.innerHTML = '<div style="font-size:var(--text-md);line-height:1.7;color:var(--muted)">' + renderMd(content) + '</div>';
+        // Wire Doc Chat for the deeplinked archive note, mirroring the live-inbox
+        // openModal() path (render-prs.js). Without this the archive branch opened
+        // the note as plain text with no Q&A panel, so deeplinking into a note
+        // from a (completed) work item sometimes silently dropped doc-chat.
+        _modalDocContext = { title: title, content: content, selection: '' };
+        _modalFilePath = filePath;
+        showModalQa();
+        // P-34fa5d79 — Source-WI chip for archive notes. Same convention as the
+        // live-inbox branch above; parsed off the freshly-fetched content.
+        _wiInjectSourceChipFromFrontmatter(content, 'modal-body');
+      });
     })
     .catch(function() {
-      closeModal();
-      switchPage('inbox');
+      applyIfCurrent(function() {
+        closeModal();
+        switchPage('inbox');
+      });
     });
 }
 

package/dashboard/js/settings.js

@@ -277,16 +277,18 @@ async function openSettings() {
     // (honors the legacy worktreeMode field). Chip is hidden by default and
     // toggled reactively below.
     // W-mr9mx93d000x1dda — third option "Hybrid dispatch": checkoutMode:'live'
-    // PLUS a liveValidation block (coding WIs author in isolated worktrees; only
-    // the chosen validation type(s) run in-place on the live checkout, with an
-    // optional auto-dispatch-after-coding toggle). This mirrors the Projects-bar
+    // PLUS a liveValidation block (only chosen work-item types run in-place;
+    // all others use isolated worktrees, with an optional test auto-dispatch
+    // after eligible coding work). This mirrors the Projects-bar
     // picker (render-other.js _openCheckoutModeMenu / _renderCheckoutModeHybridStep)
     // so both surfaces configure the SAME liveValidation shape identically.
     var currentWtMode = (p.checkoutMode === 'live') ? 'live' : 'worktree';
     var lvTypes = (window._liveValidationTypesArray ? window._liveValidationTypesArray(p) : []);
-    // 3-way UI mode: worktree / live / hybrid. Hybrid = live + >=1 validation type.
+    // 3-way UI mode: worktree / live / hybrid. Keep invalid persisted hybrid
+    // blocks in the hybrid editor so saving can repair rather than hide them as
+    // full-live.
     var currentCheckoutMode = (currentWtMode === 'live')
-      ? (lvTypes.length > 0 ? 'hybrid' : 'live')
+      ? ((lvTypes.length > 0 || p.liveValidationInvalid) ? 'hybrid' : 'live')
       : 'worktree';
     var hybridTypes = window.CHECKOUT_MODE_HYBRID_TYPES || [];
     var hybridDefaultType = window.CHECKOUT_MODE_HYBRID_DEFAULT_TYPE || 'test';
@@ -303,26 +305,26 @@ async function openSettings() {
     var hybridPanel =
       '<div data-checkout-hybrid-panel="' + escHtml(p.name) + '" style="' + (currentCheckoutMode === 'hybrid' ? '' : 'display:none;') + 'margin-top:6px">' +
         '<div style="padding:6px 8px;background:rgba(234,179,8,0.12);border:1px solid var(--yellow);border-radius:4px;color:var(--yellow);font-size:var(--text-xs);line-height:1.4;margin-bottom:6px">' +
-          '⚠ Hybrid mode: coding work items author in isolated worktrees; only the chosen validation type(s) run in-place on the live checkout (one mutating dispatch at a time; refused on a dirty tree).' +
+          '⚠ Hybrid mode: only the chosen work-item type(s) run in-place on the live checkout; all others use isolated worktrees (one mutating dispatch at a time; dirty-tree recovery follows the live-checkout auto-stash/reset settings).' +
         '</div>' +
-        '<div style="font-size:var(--text-sm);color:var(--muted);margin-bottom:2px">Work-item type(s) to validate live:</div>' +
+        '<div style="font-size:var(--text-sm);color:var(--muted);margin-bottom:2px">Work-item type(s) to run live:</div>' +
         '<div style="margin-bottom:4px">' + hybridTypeChecks + '</div>' +
         '<label style="display:inline-flex;align-items:center;gap:6px;font-size:var(--text-sm);color:var(--text);margin-top:2px">' +
           '<input type="checkbox" id="set-liveValidationAutoDispatch-' + escHtml(p.name) + '"' + (hybridAutoPre ? ' checked' : '') + '>' +
-          '<span>Automatically validate after each coding work item completes</span>' +
+          '<span>Automatically run a test validation after each eligible coding work item completes</span>' +
         '</label>' +
-        '<div data-checkout-hybrid-error="' + escHtml(p.name) + '" style="display:none;margin-top:6px;color:var(--red);font-size:var(--text-xs)">Select at least one work-item type to validate live.</div>' +
+        '<div data-checkout-hybrid-error="' + escHtml(p.name) + '" style="display:none;margin-top:6px;color:var(--red);font-size:var(--text-xs)">Select at least one work-item type to run live.</div>' +
       '</div>';
     var worktreeModeBlock =
       '<div data-search="' + escHtml(wtModeSearch) + '" style="margin-bottom:6px">' +
         '<label style="font-size:var(--text-sm);color:var(--muted);display:block;margin-bottom:2px">Checkout mode</label>' +
         '<select id="set-checkoutMode-' + escHtml(p.name) + '" data-checkout-mode-select="' + escHtml(p.name) + '" style="width:100%;padding:4px 6px;background:var(--surface);border:1px solid var(--border);border-radius:4px;color:var(--text);font-size:var(--text-md)">' +
           '<option value="worktree" title="Each dispatch runs in its own isolated git worktree. Fully parallel, never touches your working directory."' + (currentCheckoutMode === 'worktree' ? ' selected' : '') + '>Worktree (default)</option>' +
-          '<option value="live" title="Agents run in-place in the project folder. One mutating dispatch at a time; refuses to run on a dirty tree."' + (currentCheckoutMode === 'live' ? ' selected' : '') + '>Live checkout</option>' +
-          '<option value="hybrid" title="Coding work authors in isolated worktrees; only a chosen validation step runs on the real checkout."' + (currentCheckoutMode === 'hybrid' ? ' selected' : '') + '>Hybrid dispatch</option>' +
+          '<option value="live" title="Agents run in-place in the project folder. One mutating dispatch at a time; dirty-tree handling follows the auto-stash/reset policy."' + (currentCheckoutMode === 'live' ? ' selected' : '') + '>Live checkout</option>' +
+          '<option value="hybrid" title="Only chosen work-item types run on the real checkout; all others use isolated worktrees."' + (currentCheckoutMode === 'hybrid' ? ' selected' : '') + '>Hybrid dispatch</option>' +
         '</select>' +
         '<div data-checkout-mode-chip="' + escHtml(p.name) + '" style="' + (currentCheckoutMode === 'live' ? '' : 'display:none;') + 'margin-top:6px;padding:6px 8px;background:rgba(234,179,8,0.12);border:1px solid var(--yellow);border-radius:4px;color:var(--yellow);font-size:var(--text-xs);line-height:1.4">' +
-          '⚠ Live mode: dispatches run directly in this repo\'s checkout. Only one mutating dispatch runs at a time. Dirty working trees block dispatch — commit or stash before running.' +
+          '⚠ Live mode: dispatches run directly in this repo\'s checkout. Only one mutating dispatch runs at a time. Dirty-tree behavior follows the auto-stash/reset settings below.' +
         '</div>' +
         hybridPanel +
       '</div>';
@@ -474,6 +476,7 @@ async function openSettings() {
       // 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('Propagate CLAUDE.md to non-Claude runtimes', 'set-propagateClaudeMdForNonClaudeRuntimes', e.propagateClaudeMdForNonClaudeRuntimes !== false, 'Copilot/Codex have NO native CLAUDE.md auto-load. When ON (default), the engine bounded-discovers the nearest applicable repo-authored CLAUDE.md files for a dispatch and injects them as a "Project instructions (CLAUDE.md)" context layer so load-bearing "keep in sync" rules and blessed tooling reach the agent. Claude is always skipped (its CLI reads CLAUDE.md itself). Orthogonal to each runtime\'s native AGENTS.md discovery — CLAUDE.md carries no split-brain risk because Copilot/Codex never read it themselves. Per-project override via project.propagateClaudeMdForNonClaudeRuntimes.') +
       settingsToggle('Copilot: reasoning summaries', 'set-copilotReasoningSummaries', !!e.copilotReasoningSummaries, '--enable-reasoning-summaries (Anthropic-family models only)') +
     '</div>' +
     '<div class="settings-grid-2">' +
@@ -774,6 +777,17 @@ async function openSettings() {
       if (hybridPanel) hybridPanel.style.display = (sel.value === 'hybrid') ? '' : 'none';
     });
   });
+  document.querySelectorAll('[id^="set-liveValidationAutoDispatch-"]').forEach(function(autoDispatch) {
+    autoDispatch.addEventListener('change', function() {
+      if (!autoDispatch.checked) return;
+      const projectName = autoDispatch.id.slice('set-liveValidationAutoDispatch-'.length);
+      const esc = (window.CSS && CSS.escape ? CSS.escape(projectName) : projectName);
+      const testType = document.querySelector(
+        '[data-live-validation-type="' + esc + '"][value="test"]',
+      );
+      if (testType) testType.checked = true;
+    });
+  });
 }
 
 // Lazily open/close the per-agent charter editor in the Settings Agents table.
@@ -960,19 +974,16 @@ async function loadModelsForAgent(agentId, runtimeName, currentValue) {
     cell.innerHTML = '<input ' + baseAttrs + ' value="' + escHtml(currentValue || '') + '" placeholder="' + escHtml(runtimeName) + ' default" style="' + baseStyle + '">';
     return;
   }
-  let opts = '<option value=""' + (!currentValue ? ' selected' : '') + '>(fleet/default)</option>';
+  const listId = 'agent-model-options-' + agentId;
+  let opts = '';
   for (const m of models) {
     const id = m.id || m.name || '';
     if (!id) continue;
-    const label = m.name && m.name !== id ? (id + ' — ' + m.name) : id;
-    opts += '<option value="' + escHtml(id) + '"' + (id === currentValue ? ' selected' : '') + '>' + escHtml(label) + '</option>';
-  }
-  // Preserve unknown saved values so a user-set custom ID survives the next save.
-  if (currentValue && !models.some(m => (m.id || m.name) === currentValue)) {
-    opts += '<option value="' + escHtml(currentValue) + '" selected>' + escHtml(currentValue) + ' (custom — invalid for ' + escHtml(runtimeName) + '?)</option>';
+    const label = m.name && m.name !== id ? m.name : '';
+    opts += '<option value="' + escHtml(id) + '"' + (label ? ' label="' + escHtml(label) + '"' : '') + '></option>';
   }
   // eslint-disable-next-line no-unsanitized/property -- reason: structural HTML is a string literal; all user data wrapped in escHtml() (fields: agentId, model id, model label, currentValue, runtimeName)
-  cell.innerHTML = '<select ' + baseAttrs + ' style="' + baseStyle + '">' + opts + '</select>';
+  cell.innerHTML = '<input ' + baseAttrs + ' list="' + escHtml(listId) + '" value="' + escHtml(currentValue || '') + '" placeholder="' + escHtml(runtimeName) + ' default" autocomplete="off" style="' + baseStyle + '"><datalist id="' + escHtml(listId) + '">' + opts + '</datalist>';
 }
 
 function settingsToggle(label, id, checked, hint) {
@@ -1178,6 +1189,7 @@ async function saveSettings() {
       claudeFallbackModel: (document.getElementById('set-claudeFallbackModel')?.value ?? '').trim(),
       copilotFallbackModel: (document.getElementById('set-copilotFallbackModel')?.value ?? '').trim(),
       copilotDisableBuiltinMcps: !!document.getElementById('set-copilotDisableBuiltinMcps')?.checked,
+      propagateClaudeMdForNonClaudeRuntimes: !!document.getElementById('set-propagateClaudeMdForNonClaudeRuntimes')?.checked,
       copilotStreamMode: document.getElementById('set-copilotStreamMode')?.value || 'on',
       copilotReasoningSummaries: !!document.getElementById('set-copilotReasoningSummaries')?.checked,
       claudePreApproveWorkspaceMcps: !!document.getElementById('set-claudePreApproveWorkspaceMcps')?.checked,
@@ -1244,13 +1256,23 @@ async function saveSettings() {
         const types = [];
         typeEls.forEach(function(el) { if (el && el.checked) types.push(el.value); });
         const errEl = document.querySelector('[data-checkout-hybrid-error="' + esc + '"]');
+        const autoEl = document.getElementById('set-liveValidationAutoDispatch-' + p.name);
+        const autoDispatch = autoEl?.checked === true;
         if (types.length === 0) {
-          hybridValidationErrors.push(p.name);
-          if (errEl) errEl.style.display = '';
+          hybridValidationErrors.push(p.name + ': select at least one live type');
+          if (errEl) {
+            errEl.textContent = 'Select at least one work-item type to run live.';
+            errEl.style.display = '';
+          }
+        } else if (autoDispatch && types.indexOf('test') === -1) {
+          hybridValidationErrors.push(p.name + ': auto-validation requires test');
+          if (errEl) {
+            errEl.textContent = 'Automatic validation requires the test type.';
+            errEl.style.display = '';
+          }
         } else {
           if (errEl) errEl.style.display = 'none';
-          const autoEl = document.getElementById('set-liveValidationAutoDispatch-' + p.name);
-          liveValidation = { type: types, autoDispatch: !!(autoEl && autoEl.checked) };
+          liveValidation = { type: types, autoDispatch };
         }
       }
       // W-mqtvnnj1000357fa — per-project live-checkout auto-stash override.
@@ -1280,7 +1302,7 @@ async function saveSettings() {
     });
 
     if (hybridValidationErrors.length > 0) {
-      throw new Error('Hybrid dispatch needs at least one validation work-item type for: ' + hybridValidationErrors.join(', '));
+      throw new Error('Invalid hybrid dispatch configuration: ' + hybridValidationErrors.join(', '));
     }
 
     // Save config

package/dashboard/pages/engine.html

@@ -26,6 +26,12 @@
         </h2>
         <div id="keep-processes-content"><p class="empty">No agents have left processes running. Set <code>meta.keep_processes: true</code> on a work item to enable.</p></div>
       </section>
+      <section id="worktree-quarantine-refs-section">
+        <h2>Quarantined Work <span class="count" id="worktree-quarantine-refs-count">0</span>
+          <span class="qa-section-subtitle">local backup refs available for manual recovery</span>
+        </h2>
+        <div id="worktree-quarantine-refs-content"><p class="empty">No quarantined work refs found.</p></div>
+      </section>
       <section id="managed-processes-section">
         <h2>Managed Processes <span class="count" id="managed-processes-count">0</span>
           <span class="qa-section-subtitle">engine-managed long-running services</span>