@yemi33/minions 0.1.1693 → 0.1.1695

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 0.1.1695 (2026-05-04)
+
+### Features
+- harden command center stream retry (#2016)
+
 ## 0.1.1693 (2026-05-04)
 
 ### Features

package/README.md

@@ -406,7 +406,7 @@ No bash or shell involved — Node spawns Node directly. Dependency branches are
 - **Working directory** — project root (agent creates worktrees as needed)
 - **MCP servers** — inherited from `~/.claude.json` (no extra config needed)
 - **Full tool access** — all built-in tools plus all MCP tools
-- **Permission mode** — `bypassPermissions` (no interactive prompts)
+- **Permission bypass** — runtime-owned; Claude uses `--dangerously-skip-permissions`, while Copilot uses `--autopilot --allow-all --no-ask-user` (no dashboard `permissionMode` control)
 - **Output format** — `stream-json` (real-time streaming for live dashboard + completion recovery)
 
 ### Post-Completion

package/dashboard/js/command-center.js

@@ -8,6 +8,7 @@ var CC_TITLE_MAX_LENGTH = 40;
 var _ccTabs = [];         // [{id, title, sessionId, messages: [{role, html}]}]
 var _ccActiveTabId = null;
 var _ccOpen = false;
+var _ccRetrySeq = 0;
 // Per-tab sending state stored on tab objects: tab._sending, tab._queue, tab._abortController
 // Legacy globals for backward compat (badge, drawer close check)
 var _ccSending = false; // true if active tab is sending (UI indicator only)
@@ -139,6 +140,79 @@ async function _ccDashboardHealth() {
   }
 }
 
+function _ccIsReconnectableStreamError(err) {
+  if (!err) return false;
+  var name = String(err.name || '').toLowerCase();
+  var message = String(err.message || err || '').toLowerCase();
+  if (name === 'aborterror') return true;
+  return message === 'failed to fetch'
+    || message === 'load failed'
+    || message.includes('networkerror')
+    || message.includes('network request failed')
+    || message.includes('the internet connection appears to be offline')
+    || message.includes('network connection was lost')
+    || message.includes('cancelled')
+    || message.includes('canceled')
+    || message.includes('aborted');
+}
+
+function _ccJsArg(value) {
+  return escHtml(JSON.stringify(value == null ? '' : String(value)));
+}
+
+function _ccStoreRetryRequest(tab, tabId, message) {
+  if (!tab) return { id: '', tabId: tabId || '', message: String(message || '') };
+  var id = 'cc-retry-' + Date.now().toString(36) + '-' + (++_ccRetrySeq);
+  var request = { id: id, tabId: tabId || tab.id, message: String(message || ''), createdAt: Date.now() };
+  if (!tab._retryRequests) tab._retryRequests = {};
+  tab._retryRequests[id] = request;
+  tab._lastRetryRequestId = id;
+  tab._retryRequest = request;
+  return request;
+}
+
+function _ccFindRetryRequest(tab, retryId) {
+  if (!tab) return null;
+  if (retryId && tab._retryRequests && tab._retryRequests[retryId]) return tab._retryRequests[retryId];
+  if (tab._retryRequest) return tab._retryRequest;
+  if (tab._lastRetryRequestId && tab._retryRequests) return tab._retryRequests[tab._lastRetryRequestId] || null;
+  return null;
+}
+
+function _ccForgetRetryRequest(tab, retryId) {
+  if (!tab || !retryId) return;
+  if (tab._retryRequests) delete tab._retryRequests[retryId];
+  if (tab._lastRetryRequestId === retryId) delete tab._lastRetryRequestId;
+  if (tab._retryRequest && tab._retryRequest.id === retryId) delete tab._retryRequest;
+}
+
+function _ccRemoveRetryMessage(tab, retryId) {
+  var removed = false;
+  if (tab && retryId) {
+    for (var i = tab.messages.length - 1; i >= 0; i--) {
+      if (tab.messages[i] && tab.messages[i]._retryId === retryId) {
+        tab.messages.splice(i, 1);
+        removed = true;
+        break;
+      }
+    }
+  }
+  if (retryId && tab && tab.id === _ccActiveTabId) {
+    var msgs = document.getElementById('cc-messages');
+    if (msgs) {
+      Array.prototype.slice.call(msgs.children).some(function(child) {
+        if (child.getAttribute && child.getAttribute('data-cc-retry-id') === retryId) {
+          child.remove();
+          removed = true;
+          return true;
+        }
+        return false;
+      });
+    }
+  }
+  return removed;
+}
+
 function _ccFindPinTarget(query) {
   for (var i = 0; i < (inboxData || []).length; i++) {
     if (inboxData[i].name.toLowerCase().includes(query)) {
@@ -159,6 +233,7 @@ function _ccFindPinTarget(query) {
 function ccAbort() {
   var tab = _ccActiveTab();
   if (tab && tab._abortController) {
+    tab._userAborted = true;
     try {
       fetch('/api/command-center/abort', {
         method: 'POST',
@@ -290,6 +365,7 @@ function ccCloseTab(id) {
         body: JSON.stringify({ tabId: id })
       }).catch(function() {});
     } catch {}
+    closingTab._userAborted = true;
     if (closingTab._abortController) { closingTab._abortController.abort(); closingTab._abortController = null; }
     closingTab._sending = false;
     closingTab._queue = [];
@@ -406,7 +482,7 @@ function ccUpdateSessionIndicator() {
   }
 }
 
-function ccAddMessage(role, html, skipSave, targetTabId) {
+function ccAddMessage(role, html, skipSave, targetTabId, meta) {
   var isUser = role === 'user';
   var isSystem = role === 'system';
   var isAction = role === 'action';
@@ -418,6 +494,7 @@ function ccAddMessage(role, html, skipSave, targetTabId) {
     var el = document.getElementById('cc-messages');
     var div = document.createElement('div');
     div.className = isAssistant ? 'cc-msg-assistant' : '';
+    if (meta && meta.retryId) div.setAttribute('data-cc-retry-id', meta.retryId);
     div.style.cssText = 'padding:8px 12px;border-radius:8px;font-size:12px;line-height:1.6;max-width:95%;' +
       (isUser ? 'background:var(--blue);color:#fff;align-self:flex-end' : isSystem ? 'align-self:center;max-width:100%' : isAction ? 'align-self:flex-start;padding:2px 0' : 'background:var(--surface2);color:var(--text);align-self:flex-start;border:1px solid var(--border);position:relative');
     div.innerHTML = (isAssistant && !html.includes('color:var(--red)') && !html.includes('cc-queued-pill') ? llmCopyBtn() : '') + html;
@@ -428,7 +505,9 @@ function ccAddMessage(role, html, skipSave, targetTabId) {
   if (!skipSave) {
     var tab = targetTab;
     if (tab) {
-      tab.messages.push({ role: role, html: html });
+      var msg = { role: role, html: html };
+      if (meta && meta.retryId) msg._retryId = meta.retryId;
+      tab.messages.push(msg);
       // Auto-title from first user message
       if (role === 'user' && tab.title === 'New chat') {
         var tmp = document.createElement('div');
@@ -519,13 +598,14 @@ async function _ccDoSend(message, skipUserMsg, forceTabId) {
   activeTab._sending = true;
   activeTab._sendStartedAt = Date.now();
   activeTab._abortController = new AbortController();
+  activeTab._userAborted = false;
   _ccSending = true;
   ccRenderTabBar();
   var _wasAborted = false;
   try { localStorage.setItem('cc-sending', JSON.stringify({ sending: true, startedAt: Date.now() })); } catch {}
 
   // Scoped helper — always targets the originating tab, even if user switches tabs
-  function addMsg(role, html, skipSave) { ccAddMessage(role, html, skipSave, activeTabId); }
+  function addMsg(role, html, skipSave, meta) { ccAddMessage(role, html, skipSave, activeTabId, meta); }
 
   if (!skipUserMsg) addMsg('user', escHtml(message));
 
@@ -615,9 +695,11 @@ async function _ccDoSend(message, skipUserMsg, forceTabId) {
       var seconds = Math.round((Date.now() - ccStartTime) / 1000);
       return '<div style="font-size:9px;color:var(--muted);margin-top:6px;display:flex;justify-content:flex-end;padding-right:30px">' + label.replace('{seconds}', seconds) + '</div>';
     }
-    function _ccRetryControls(extraHtml, showReload) {
+    function _ccRetryControls(retryRequest, extraHtml, showReload) {
+      var retryTabId = retryRequest && retryRequest.tabId ? retryRequest.tabId : activeTabId;
+      var retryId = retryRequest && retryRequest.id ? retryRequest.id : '';
       return (extraHtml || '') +
-        '<button onclick="ccRetryLast()" style="margin-top:6px;padding:4px 12px;background:var(--surface2);border:1px solid var(--border);border-radius:4px;color:var(--blue);cursor:pointer;font-size:11px">Retry</button>' +
+        '<button onclick="ccRetryLast(' + _ccJsArg(retryTabId) + ',' + _ccJsArg(retryId) + ')" style="margin-top:6px;padding:4px 12px;background:var(--surface2);border:1px solid var(--border);border-radius:4px;color:var(--blue);cursor:pointer;font-size:11px">Retry</button>' +
         (showReload ? ' <button onclick="location.reload()" style="margin-top:6px;padding:4px 12px;background:var(--orange);color:#fff;border:none;border-radius:4px;cursor:pointer;font-size:11px">Reload Page</button>' : '') +
         ' <button onclick="ccNewTab()" style="margin-top:6px;padding:4px 12px;background:var(--surface2);border:1px solid var(--border);border-radius:4px;color:var(--muted);cursor:pointer;font-size:11px">New Session</button>';
     }
@@ -748,11 +830,12 @@ async function _ccDoSend(message, skipUserMsg, forceTabId) {
       if (!consume.interrupted) break;
       if (!consume.reconnectable || reconnectAttempts >= 2) {
         _cleanupStreamDiv();
-        var streamEndedHint = '<div style="font-size:10px;color:var(--muted);margin-top:4px">The response stream ended before completion. Retry to continue from the last user message.</div>';
+        var streamEndedHint = '<div style="font-size:10px;color:var(--muted);margin-top:4px">The response stream ended before completion. Retry to resend the interrupted message.</div>';
+        var streamEndedRetry = _ccStoreRetryRequest(activeTab, activeTabId, message);
         if (streamedText) {
-          addMsg('assistant', renderMd(streamedText) + _ccElapsedFooter('Stream interrupted after {seconds}s') + _ccRetryControls(streamEndedHint, false));
+          addMsg('assistant', renderMd(streamedText) + _ccElapsedFooter('Stream interrupted after {seconds}s') + _ccRetryControls(streamEndedRetry, streamEndedHint, false), false, { retryId: streamEndedRetry.id });
         } else {
-          addMsg('assistant', '<span style="color:var(--red)">The response stream ended before completion.</span>' + _ccRetryControls(streamEndedHint, false));
+          addMsg('assistant', '<span style="color:var(--red)">The response stream ended before completion.</span>' + _ccRetryControls(streamEndedRetry, streamEndedHint, false), false, { retryId: streamEndedRetry.id });
         }
         break;
       }
@@ -762,8 +845,9 @@ async function _ccDoSend(message, skipUserMsg, forceTabId) {
         var reconnectHint = reconnectHealth.restarted
           ? '<div style="font-size:10px;color:var(--muted);margin-top:4px">Dashboard restarted while this response was streaming. Reload the page to reconnect to the new instance.</div>'
           : '<div style="font-size:10px;color:var(--muted);margin-top:4px">The request stream was interrupted, but the dashboard is still reachable. Retry or start a new session.</div>';
+        var reconnectRetry = _ccStoreRetryRequest(activeTab, activeTabId, message);
         addMsg('assistant', (streamedText ? renderMd(streamedText) + _ccElapsedFooter('Stream interrupted after {seconds}s') : '') +
-          _ccRetryControls(reconnectHint, reconnectHealth.restarted));
+          _ccRetryControls(reconnectRetry, reconnectHint, reconnectHealth.restarted), false, { retryId: reconnectRetry.id });
         break;
       }
       reconnectAttempts++;
@@ -776,7 +860,7 @@ async function _ccDoSend(message, skipUserMsg, forceTabId) {
     }
   } catch (e) {
     _cleanupStreamDiv();
-    if (e.name === 'AbortError') {
+    if (activeTab && activeTab._userAborted) {
       _wasAborted = true;
       if (streamedText) {
         addMsg('assistant', renderMd(streamedText) + _ccElapsedFooter('Stopped after {seconds}s'));
@@ -784,7 +868,7 @@ async function _ccDoSend(message, skipUserMsg, forceTabId) {
         addMsg('assistant', '<span style="color:var(--red);font-size:11px">Stopped</span>');
       }
     } else {
-      var isNetworkError = e.message === 'Failed to fetch' || e.message.includes('NetworkError');
+      var isNetworkError = _ccIsReconnectableStreamError(e);
       var dashboardHealth = isNetworkError ? await _ccDashboardHealth() : { reachable: false, restarted: false };
       var connectionHint = '';
       if (isNetworkError) {
@@ -794,12 +878,13 @@ async function _ccDoSend(message, skipUserMsg, forceTabId) {
             ? '<div style="font-size:10px;color:var(--muted);margin-top:4px">The request stream was interrupted, but the dashboard is still reachable. Retry or start a new session.</div>'
             : '<div style="font-size:10px;color:var(--muted);margin-top:4px">Dashboard connection lost. Reload the page to reconnect.</div>';
       }
+      var errorRetry = _ccStoreRetryRequest(activeTab, activeTabId, message);
       addMsg('assistant', (streamedText ? renderMd(streamedText) + _ccElapsedFooter('Stream interrupted after {seconds}s') : '') +
         '<span style="color:var(--red)">Error: ' + escHtml(e.message) + '</span>' +
-        _ccRetryControls(connectionHint, isNetworkError && (!dashboardHealth.reachable || dashboardHealth.restarted)));
+        _ccRetryControls(errorRetry, connectionHint, isNetworkError && (!dashboardHealth.reachable || dashboardHealth.restarted)), false, { retryId: errorRetry.id });
     }
   } finally {
-    if (activeTab) { activeTab._sending = false; activeTab._abortController = null; activeTab._429retries = 0; delete activeTab._streamedText; delete activeTab._toolsUsed; delete activeTab._sendStartedAt; }
+    if (activeTab) { activeTab._sending = false; activeTab._abortController = null; activeTab._429retries = 0; delete activeTab._streamedText; delete activeTab._toolsUsed; delete activeTab._sendStartedAt; delete activeTab._userAborted; }
     _ccSending = (_ccTabs.some(function(t) { return t._sending; }));
     // Mark tab unread if response completed on a background tab or while drawer is closed
     if (activeTab && !_wasAborted && (activeTab.id !== _ccActiveTabId || !_ccOpen)) activeTab._unread = true;
@@ -813,28 +898,36 @@ async function _ccDoSend(message, skipUserMsg, forceTabId) {
   return _wasAborted;
 }
 
-function ccRetryLast() {
-  // Find the last user message and resend it
-  var tab = _ccActiveTab();
+function ccRetryLast(tabId, retryId) {
+  var tab = tabId ? _ccTabs.find(function(t) { return t.id === tabId; }) : _ccActiveTab();
   if (!tab) return;
-  var last = tab.messages.filter(function(m) { return m.role === 'user'; }).pop();
-  if (!last) return;
-  // Extract text from the HTML (strip tags)
-  var tmp = document.createElement('div');
-  tmp.innerHTML = last.html;
-  var text = tmp.textContent || tmp.innerText || '';
+  var retryRequest = _ccFindRetryRequest(tab, retryId);
+  var text = retryRequest ? retryRequest.message : '';
+  if (!text) {
+    var last = tab.messages.filter(function(m) { return m.role === 'user'; }).pop();
+    if (!last) return;
+    // Backward-compatible fallback for retry buttons rendered before retry context existed.
+    var tmp = document.createElement('div');
+    tmp.innerHTML = last.html;
+    text = tmp.textContent || tmp.innerText || '';
+  }
   if (!text.trim()) return;
-  // Remove the error message (last assistant message)
-  var el = document.getElementById('cc-messages');
-  if (el?.lastElementChild) el.lastElementChild.remove();
-  tab.messages = tab.messages.slice(0, -1); // remove error from history
+  var removed = _ccRemoveRetryMessage(tab, retryId);
+  if (!removed && tab.id === _ccActiveTabId) {
+    // Legacy fallback for old controls without retry ids: remove the visible error card.
+    var el = document.getElementById('cc-messages');
+    if (el?.lastElementChild) el.lastElementChild.remove();
+    tab.messages = tab.messages.slice(0, -1);
+  }
+  _ccForgetRetryRequest(tab, retryId);
+  ccSaveState();
   // Resend, then drain queue
-  _ccDoSend(text.trim()).then(async function() {
-    var retryTab = _ccActiveTab();
+  _ccDoSend(text.trim(), false, tab.id).then(async function() {
+    var retryTab = _ccTabs.find(function(t) { return t.id === tab.id; });
     while (retryTab && retryTab._queue && retryTab._queue.length > 0) {
       var next = retryTab._queue.shift();
       _renderQueueIndicator();
-      await _ccDoSend(next);
+      await _ccDoSend(next, false, retryTab.id);
     }
   });
 }

package/dashboard/js/render-agents.js

@@ -54,7 +54,6 @@ function renderAgents(agents) {
       })()}
       ${a._blockingToolCall ? `<div style="margin-top:4px;padding:4px 8px;background:rgba(130,160,210,0.13);border:1px solid rgba(130,160,210,0.3);border-radius:4px;font-size:10px;color:var(--muted)">&#x23F3; Blocking tool call (${escapeHtml(a._blockingToolCall.tool)}) &mdash; silent ${Math.round(a._blockingToolCall.silentMs/60000)}min, timeout in ${Math.round(a._blockingToolCall.remainingMs/60000)}min</div>` : ''}
       ${a._warning ? `<div style="margin-top:4px;padding:4px 8px;background:rgba(210,153,34,0.15);border:1px solid rgba(210,153,34,0.3);border-radius:4px;font-size:10px;color:var(--yellow)">&#x26A0; ${escapeHtml(a._warning)}</div>` : ''}
-      ${a._permissionMode && a._permissionMode !== 'bypassPermissions' && !a._warning ? `<div style="margin-top:4px;font-size:9px;color:var(--muted)">Permission mode: ${escapeHtml(a._permissionMode)}</div>` : ''}
       ${a.resultSummary ? `<div class="agent-result" title="${escapeHtml(a.resultSummary)}">${renderMd(a.resultSummary.slice(0, 200))}${a.resultSummary.length > 200 ? '...' : ''}</div>` : ''}
     </div>
   `).join('');

package/dashboard/js/settings.js

@@ -258,20 +258,12 @@ async function openSettings() {
     '</div>' +
 
     '<h3 style="font-size:13px;color:var(--blue);margin-bottom:8px">Claude CLI</h3>' +
-    '<div style="display:grid;grid-template-columns:1fr 1fr;gap:8px;margin-bottom:16px">' +
+    '<div style="display:grid;grid-template-columns:1fr 1fr;gap:8px;margin-bottom:8px">' +
       settingsField('Output Format', 'set-outputFormat', c.outputFormat || 'stream-json', '', '') +
-      settingsField('Allowed Tools', 'set-allowedTools', c.allowedTools || '', '', 'Tools agents can use without permission prompts. Add MCP tools here if using non-bypass mode (e.g. mcp__azure-ado__*)') +
+      settingsField('Allowed Tools', 'set-allowedTools', c.allowedTools || '', '', 'Claude allow-list passed through for compatibility; runtime bypass flags are adapter-owned.') +
     '</div>' +
-    '<div style="margin-bottom:16px">' +
-      '<label style="font-size:10px;color:var(--muted);display:block;margin-bottom:2px">Permission Mode <span style="opacity:0.6">(how agents handle tool approvals)</span></label>' +
-      '<select id="set-permissionMode" 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="bypassPermissions"' + ((c.permissionMode || 'bypassPermissions') === 'bypassPermissions' ? ' selected' : '') + '>Bypass (recommended) — agents run autonomously without permission prompts</option>' +
-        '<option value="auto"' + ((c.permissionMode) === 'auto' ? ' selected' : '') + '>Auto — auto-approve safe tools, prompt for risky ones (agents may hang on risky tools)</option>' +
-        '<option value="default"' + ((c.permissionMode) === 'default' ? ' selected' : '') + '>Default — prompt for every tool (agents WILL hang — not recommended)</option>' +
-      '</select>' +
-      '<div id="set-permissionMode-warn" style="font-size:9px;margin-top:4px;padding:4px 8px;border-radius:4px;' + ((c.permissionMode && c.permissionMode !== 'bypassPermissions') ? 'display:block;background:rgba(248,81,73,0.1);color:var(--red)' : 'display:none') + '">' +
-        '\u26A0 Tools listed in Allowed Tools above are auto-approved even in non-bypass modes. Agents will only hang if they try to use a tool NOT on that list (e.g. MCP tools). In bypass mode, all tools are approved automatically.' +
-      '</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>' +
@@ -314,20 +306,6 @@ async function openSettings() {
   document.getElementById('modal-body').style.whiteSpace = '';
   document.getElementById('modal').classList.add('open');
 
-  // Wire permission mode warning toggle
-  var pmSelect = document.getElementById('set-permissionMode');
-  if (pmSelect) pmSelect.addEventListener('change', function() {
-    var warn = document.getElementById('set-permissionMode-warn');
-    if (!warn) return;
-    if (this.value !== 'bypassPermissions') {
-      warn.style.display = 'block';
-      warn.style.background = 'rgba(248,81,73,0.1)';
-      warn.style.color = 'var(--red)';
-    } else {
-      warn.style.display = 'none';
-    }
-  });
-
   // ── 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.
@@ -595,7 +573,6 @@ async function saveSettings() {
     const claudePayload = {
       outputFormat: document.getElementById('set-outputFormat').value,
       allowedTools: document.getElementById('set-allowedTools').value,
-      permissionMode: document.getElementById('set-permissionMode').value,
     };
 
     const teamsPayload = {

package/dashboard.js

@@ -5405,6 +5405,12 @@ What would you like to discuss or change? When you're happy, say "approve" and I
     } catch (e) { return jsonReply(res, e.statusCode || 500, { error: e.message }); }
   }
 
+  function settingsClaudeConfig(config) {
+    const claude = { ...shared.DEFAULT_CLAUDE, ...(config.claude || {}) };
+    delete claude.permissionMode;
+    return claude;
+  }
+
   async function handleSettingsRead(req, res) {
     try {
       const config = queries.getConfig();
@@ -5414,7 +5420,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       if (engine.prPollCommentsEvery === undefined && engine.adoPollCommentsEvery !== undefined) engine.prPollCommentsEvery = engine.adoPollCommentsEvery;
       return jsonReply(res, 200, {
         engine,
-        claude: { ...shared.DEFAULT_CLAUDE, ...(config.claude || {}) },
+        claude: settingsClaudeConfig(config),
         agents: config.agents || {},
         teams: { ...shared.ENGINE_DEFAULTS.teams, ...(config.teams || {}) },
         projects: (config.projects || []).map(p => ({
@@ -5437,6 +5443,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       if (!config.engine) config.engine = {};
       if (!config.claude) config.claude = {};
       if (!config.agents) config.agents = {};
+      delete config.claude.permissionMode;
 
       const _clamped = [];
       const _engineModelDiscovery = require('./engine/model-discovery');
@@ -5607,10 +5614,6 @@ What would you like to discuss or change? When you're happy, say "approve" and I
         for (const key of ['allowedTools', 'outputFormat']) {
           if (body.claude[key] !== undefined) config.claude[key] = String(body.claude[key]);
         }
-        if (body.claude.permissionMode !== undefined) {
-          const valid = ['bypassPermissions', 'auto', 'default'];
-          config.claude.permissionMode = valid.includes(body.claude.permissionMode) ? body.claude.permissionMode : 'bypassPermissions';
-        }
       }
 
       if (body.agents) {

package/docs/auto-discovery.md

@@ -250,10 +250,12 @@ Combines:
 ```bash
 claude -p --system-prompt-file <sysprompt-file> \
   --output-format stream-json --max-turns 100 --verbose \
-  --permission-mode bypassPermissions
+  --dangerously-skip-permissions
 # Prompt text is piped via stdin (not passed as an arg).
 # Agent dispatches route through engine/spawn-agent.js; CC / doc-chat use a direct
 # spawn path in engine/llm.js that bypasses spawn-agent.js entirely.
+# Permission bypass is runtime-owned: Copilot uses
+# --autopilot --allow-all --no-ask-user instead.
 ```
 
 - Process runs in the worktree directory (or rootDir for reviews)

package/docs/blog-first-successful-dispatch.md

@@ -63,9 +63,9 @@ proc.stdin.end();
 
 Fix: switched to `--output-format stream-json` — streams events as they happen.
 
-### Permission Mode
+### Permission Bypass
 
-Agents would hang waiting for permission prompts (invisible in headless mode). Added `--permission-mode bypassPermissions`.
+Agents would hang waiting for permission prompts (invisible in headless mode). Modern Minions keeps bypass behavior inside runtime adapters: Claude emits `--dangerously-skip-permissions`; Copilot emits `--autopilot --allow-all --no-ask-user`. The legacy `config.claude.permissionMode` key is ignored.
 
 ### CLAUDECODE Environment Variable
 
@@ -85,7 +85,7 @@ When the engine restarts, the in-memory `activeProcesses` Map is lost. Active di
 
 **Dispatch 1773292681199** — Dallas, central work item, auto-route:
 
-1. Engine spawns `node spawn-agent.js prompt.md sysprompt.md --output-format stream-json --verbose --permission-mode bypassPermissions`
+1. Engine spawns `node spawn-agent.js prompt.md sysprompt.md --runtime claude --output-format stream-json --verbose`
 2. spawn-agent.js resolves `cli.js`, spawns `node cli.js -p --system-prompt <content> ...`
 3. Prompt piped via stdin — no shell interpretation
 4. MCP servers connect (azure-ado, azure-kusto, mobile, DevBox)

package/docs/deprecated.json

@@ -22,5 +22,13 @@
     "reason": "These cadence settings gate both ADO and GitHub PR polling, so the ADO-prefixed names are misleading.",
     "locations": ["engine.js read-side fallback from config.engine.adoPollStatusEvery/adoPollCommentsEvery", "dashboard.js handleSettingsRead/handleSettingsUpdate alias fallback for old keys"],
     "cleanup": "Remove the adoPoll* alias fallback reads after existing configs have been migrated to prPollStatusEvery/prPollCommentsEvery."
+  },
+  {
+    "id": "config-claude-permission-mode",
+    "summary": "config.claude.permissionMode ignored; runtime adapters own permission bypass flags",
+    "deprecated": "2026-05-04",
+    "reason": "Claude and Copilot require different non-interactive bypass flags, so a shared Claude config field was misleading and no longer controls spawns.",
+    "locations": ["dashboard.js settings update strips config.claude.permissionMode", "dashboard/js/settings.js no longer renders a Permission Mode selector"],
+    "cleanup": "After old configs have been rewritten through settings, remove the deprecated-field preflight warning entry for permissionMode."
   }
 ]

package/engine/copilot-models.json

@@ -1,5 +1,5 @@
 {
   "runtime": "copilot",
   "models": null,
-  "cachedAt": "2026-05-04T03:30:25.499Z"
+  "cachedAt": "2026-05-04T05:14:26.307Z"
 }

package/engine/queries.js

@@ -428,25 +428,11 @@ function getAgentStatus(agentId) {
     if (active._blockingToolCall) {
       result._blockingToolCall = active._blockingToolCall;
     }
-    // Detect permission-waiting and in-flight tools: read only head+tail of live-output.log (max 2KB total)
+    // Detect in-flight tools: read only head+tail of live-output.log (max 2KB total)
     try {
       const liveLogPath = path.join(AGENTS_DIR, agentId, 'live-output.log');
       const { head, tail } = readHeadTail(liveLogPath, 1024);
       if (head) {
-        // Check init message (in head) for permission mode
-        const initMatch = head.match(/"permissionMode"\s*:\s*"([^"]+)"/);
-        if (initMatch && initMatch[1] !== 'bypassPermissions') {
-          result._permissionMode = initMatch[1];
-        }
-        // Check if agent has been silent for >60s (use tail for recent activity)
-        const lastLine = tail.trimEnd().split('\n').pop();
-        if (lastLine && lastLine.includes('"type":"assistant"') && lastLine.includes('"tool_use"')) {
-          const liveStat = fs.statSync(liveLogPath);
-          const silentMs = Date.now() - liveStat.mtimeMs;
-          if (silentMs > 60000 && result._permissionMode) {
-            result._warning = 'Possibly waiting for permission approval — agent is not in bypass mode';
-          }
-        }
         // Detect in-flight tool calls (task_started with no task_notification)
         const inFlight = detectInFlightTool(tail);
         if (inFlight && inFlight.description) {
@@ -570,7 +556,6 @@ function getAgents(config) {
       completed_at: s.completed_at || null,
       _blockingToolCall: s._blockingToolCall || null,
       _warning: s._warning || null,
-      _permissionMode: s._permissionMode || null,
       chartered, inboxCount: inboxFiles.length + steeringInboxFiles.length
     };
   });

package/engine/shared.js

@@ -851,7 +851,7 @@ const ENGINE_DEFAULTS = {
   // Backward-compat: keep `engine.claude.*` field family deprecation tracker. Listed here so preflight
   // knows which subkeys to flag as deprecated. Do not consume `claude.*` in new code — use the runtime
   // adapter system (engine/runtimes/) and the resolveAgent*/resolveCc* helpers instead.
-  _deprecatedConfigClaudeFields: ['binary', 'outputFormat', 'allowedTools', 'maxTurns', 'effort', 'budgetCap'],
+  _deprecatedConfigClaudeFields: ['binary', 'outputFormat', 'allowedTools', 'permissionMode', 'maxTurns', 'effort', 'budgetCap'],
   // Teams integration — config.teams shape: { enabled, appId, appPassword, certPath, privateKeyPath, tenantId, notifyEvents, ccMirror, inboxPollInterval }
   // Auth modes: (1) appId + appPassword (client secret), or (2) appId + certPath + privateKeyPath + tenantId (certificate)
   teams: {

package/minions.js

@@ -470,6 +470,7 @@ async function initMinions({ skipScan = false, scanRoot, scanDepth } = {}) {
     if (config.engine[k] === undefined) config.engine[k] = v;
   }
   if (!config.claude) config.claude = {};
+  delete config.claude.permissionMode;
   for (const [k, v] of Object.entries(DEFAULT_CLAUDE)) {
     if (config.claude[k] === undefined) config.claude[k] = v;
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.1693",
+  "version": "0.1.1695",
   "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"