@yemi33/minions 0.1.1599 → 0.1.1601

package/CHANGELOG.md

@@ -1,18 +1,22 @@
 # Changelog
 
-## 0.1.1599 (2026-04-28)
+## 0.1.1601 (2026-04-28)
 
 ### Features
 - match runtime tags to actual logos (pixel-crab Claude, mascot Copilot)
 - replace runtime text tag with inline SVG logos
 
 ### Fixes
+- runtime-aware model picker + cross-runtime validation
 -  keep cc stream final text complete
 - switch Copilot icon to outline style for cleaner inline read
 - un-invert Copilot — purple silhouette + white cutouts
 - redraw Copilot icon to match actual mascot — vertical eye pills, not grill bars
 - invert Copilot icon colors for better dark-theme visibility
 
+### Other
+- Stream CC chunks incrementally
+
 ## 0.1.1592 (2026-04-28)
 
 ### Features

package/dashboard/js/command-center.js

@@ -54,6 +54,22 @@ function _ccActiveTab() {
   return _ccTabs.find(function(t) { return t.id === _ccActiveTabId; }) || null;
 }
 
+function _ccMergeStreamText(prev, incoming) {
+  var current = prev || '';
+  var next = incoming || '';
+  if (!current) return next;
+  if (!next) return current;
+  if (next === current) return current;
+  if (next.startsWith(current)) return next;
+  if (current.startsWith(next)) return current;
+  for (var overlap = Math.min(current.length, next.length); overlap > 0; overlap--) {
+    if (current.slice(-overlap) === next.slice(0, overlap)) {
+      return current + next.slice(overlap);
+    }
+  }
+  return current + '\n\n' + next;
+}
+
 async function _ccDashboardHealth() {
   var controller = new AbortController();
   var timer = setTimeout(function() { controller.abort(); }, 3000);
@@ -580,7 +596,7 @@ async function _ccDoSend(message, skipUserMsg, forceTabId) {
 
     async function _handleEvent(evt) {
       if (evt.type === 'chunk') {
-        streamedText = evt.text;
+        streamedText = _ccMergeStreamText(streamedText, evt.text || '');
         if (activeTab) activeTab._streamedText = streamedText;
         updateStreamDiv();
       } else if (evt.type === 'heartbeat') {

package/dashboard/js/refresh.js

@@ -35,6 +35,20 @@ function _changed(key, value) {
   return true;
 }
 
+function _formatCcPowerLabel(autoMode) {
+  var runtime = autoMode && autoMode.ccCli ? String(autoMode.ccCli) : 'claude';
+  var runtimeLabel = runtime.charAt(0).toUpperCase() + runtime.slice(1);
+  var model = autoMode && autoMode.ccModel ? String(autoMode.ccModel) : '';
+  return 'Ask anything, dispatch work, manage plans — powered by ' + runtimeLabel + (model ? ' (' + model + ')' : '');
+}
+
+function _formatCcDrawerLabel(autoMode) {
+  var runtime = autoMode && autoMode.ccCli ? String(autoMode.ccCli) : 'claude';
+  var runtimeLabel = runtime.charAt(0).toUpperCase() + runtime.slice(1);
+  var model = autoMode && autoMode.ccModel ? String(autoMode.ccModel) : '';
+  return runtimeLabel + (model ? ' (' + model + ')' : '') + '-powered. Full minions context. Enter to send, Shift+Enter for newline.';
+}
+
 function _processStatusUpdate(data) {
   // Detect fresh install — clear stale browser state if install ID changed
   if (data.installId) {
@@ -53,6 +67,10 @@ function _processStatusUpdate(data) {
   if (autoEl) autoEl.innerHTML = data.autoMode?.approvePlans
     ? '<span style="font-size:9px;font-weight:600;padding:1px 6px;border-radius:3px;background:rgba(63,185,80,0.15);color:var(--green);border:1px solid rgba(63,185,80,0.3)">AUTO-APPROVE</span>'
     : '';
+  const ccLabelEl = document.getElementById('cmd-powered-by');
+  if (ccLabelEl) ccLabelEl.textContent = _formatCcPowerLabel(data.autoMode);
+  const ccDrawerLabelEl = document.getElementById('cc-powered-by');
+  if (ccDrawerLabelEl) ccDrawerLabelEl.textContent = _formatCcDrawerLabel(data.autoMode);
   const threshEl = document.getElementById('inbox-threshold');
   if (threshEl && data.autoMode?.inboxThreshold) threshEl.textContent = data.autoMode.inboxThreshold;
 

package/dashboard/js/settings.js

@@ -35,7 +35,11 @@ async function openSettings() {
         // <select> populated from /api/runtimes once the registry resolves.
         '<input value="' + escHtml(a.cli || '') + '" placeholder="' + escHtml(fleetCliLabel) + ' (fleet)" disabled style="width:100%;padding:4px 6px;background:var(--surface);border:1px solid var(--border);border-radius:4px;color:var(--muted);font-size:11px">' +
       '</td>' +
-      '<td><input data-agent="' + escHtml(id) + '" data-field="model" value="' + escHtml(a.model || '') + '" placeholder="' + escHtml(fleetModelLabel) + ' (fleet)" style="width:120px;padding:4px 6px;background:var(--surface);border:1px solid var(--border);border-radius:4px;color:var(--text);font-size:11px"></td>' +
+      '<td data-runtime-model="' + escHtml(id) + '" style="min-width:140px">' +
+        // Loading placeholder — initRuntimeFleetUI() replaces this with a
+        // <select> populated from /api/runtimes/<resolved-cli>/models.
+        '<input value="' + escHtml(a.model || '') + '" placeholder="' + escHtml(fleetModelLabel) + ' (fleet)" disabled style="width:120px;padding:4px 6px;background:var(--surface);border:1px solid var(--border);border-radius:4px;color:var(--muted);font-size:11px">' +
+      '</td>' +
       '<td><input data-agent="' + escHtml(id) + '" data-field="monthlyBudgetUsd" value="' + escHtml(a.monthlyBudgetUsd != null ? String(a.monthlyBudgetUsd) : '') + '" placeholder="unlimited" style="width:70px;padding:4px 6px;background:var(--surface);border:1px solid var(--border);border-radius:4px;color:var(--text);font-size:11px;text-align:right"></td>' +
     '</tr>';
   }).join('');
@@ -347,6 +351,26 @@ async function initRuntimeFleetUI(engineCfg, agentsCfg) {
         ).join('') +
       '</select>';
   }
+  // Hydrate per-agent model dropdowns. The model list is keyed off the
+  // agent's RESOLVED runtime: per-agent override → fleet default. Without
+  // this the input was free-text and a user could (and did) save an agent
+  // with cli=claude + model=<some gpt> — invalid combination that crashed
+  // dispatch. Refreshing on CLI change clears stale model values.
+  const fleetDefaultCli = engineCfg.defaultCli || 'claude';
+  for (const cell of cliCells) {
+    const agentId = cell.getAttribute('data-runtime-cli');
+    const agent = (agentsCfg || {})[agentId] || {};
+    const resolvedCli = agent.cli || fleetDefaultCli;
+    loadModelsForAgent(agentId, resolvedCli, agent.model || '');
+    // CLI dropdown change → refresh that agent's model dropdown to match.
+    const sel = cell.querySelector('select[data-field="cli"]');
+    if (sel) {
+      sel.addEventListener('change', () => {
+        const newCli = sel.value || fleetDefaultCli;
+        loadModelsForAgent(agentId, newCli, ''); // clear value: previous model may not exist for the new runtime
+      });
+    }
+  }
 
   // Models load for the resolved default + CC CLIs. ccCli falls back to
   // defaultCli when unset — same rule as resolveCcCli().
@@ -409,6 +433,47 @@ async function loadModelsForRuntime(runtimeName, inputId, currentValue) {
   wrap.innerHTML = '<select id="' + inputId + '" style="width:100%;padding:4px 6px;background:var(--surface);border:1px solid var(--border);border-radius:4px;color:var(--text);font-size:12px">' + opts + '</select>';
 }
 
+/**
+ * Per-agent model hydrator. Replaces the placeholder input in the cell
+ * `[data-runtime-model="<agentId>"]` with a <select> of valid models for the
+ * given runtime. Output element keeps `data-agent` + `data-field="model"` so
+ * the existing save flow picks it up unchanged. Free-text input fallback
+ * when the runtime returns no model list (Claude / discovery disabled).
+ */
+async function loadModelsForAgent(agentId, runtimeName, currentValue) {
+  const cell = document.querySelector('[data-runtime-model="' + agentId + '"]');
+  if (!cell) return;
+  const baseAttrs = 'data-agent="' + escHtml(agentId) + '" data-field="model"';
+  const baseStyle = 'width:120px;padding:4px 6px;background:var(--surface);border:1px solid var(--border);border-radius:4px;color:var(--text);font-size:11px';
+  if (!runtimeName) {
+    cell.innerHTML = '<input ' + baseAttrs + ' value="' + escHtml(currentValue || '') + '" placeholder="(no runtime)" disabled style="' + baseStyle + ';color:var(--muted)">';
+    return;
+  }
+  let payload = { models: null };
+  try {
+    const res = await fetch('/api/runtimes/' + encodeURIComponent(runtimeName) + '/models');
+    if (res.ok) payload = await res.json();
+  } catch { /* fall through to free-text */ }
+
+  const models = Array.isArray(payload.models) ? payload.models : null;
+  if (!models || models.length === 0) {
+    cell.innerHTML = '<input ' + baseAttrs + ' value="' + escHtml(currentValue || '') + '" placeholder="' + escHtml(runtimeName) + ' default" style="' + baseStyle + '">';
+    return;
+  }
+  let opts = '<option value=""' + (!currentValue ? ' selected' : '') + '>(fleet/default)</option>';
+  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>';
+  }
+  cell.innerHTML = '<select ' + baseAttrs + ' style="' + baseStyle + '">' + opts + '</select>';
+}
+
 function settingsToggle(label, id, checked, hint) {
   return '<div 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">' +

package/dashboard/layout.html

@@ -43,7 +43,7 @@
       <textarea id="cc-input" rows="2" placeholder="Ask anything or give a command..." style="flex:1;padding:8px 10px;background:var(--bg);border:1px solid var(--border);border-radius:6px;color:var(--text);font-size:12px;resize:none;font-family:inherit" onkeydown="if(event.key==='Enter'&&!event.shiftKey){event.preventDefault();ccSend()}"></textarea>
       <button onclick="ccSend()" style="background:var(--blue);color:#fff;border:none;border-radius:6px;padding:8px 14px;font-size:12px;font-weight:600;cursor:pointer;align-self:flex-end">Send</button>
     </div>
-    <div style="font-size:9px;color:var(--muted);margin-top:4px">Sonnet-powered. Full minions context. Enter to send, Shift+Enter for newline.</div>
+    <div id="cc-powered-by" style="font-size:9px;color:var(--muted);margin-top:4px">Full minions context. Enter to send, Shift+Enter for newline.</div>
   </div>
 </div>
 

package/dashboard/pages/home.html

@@ -10,7 +10,7 @@
         <div class="cmd-meta" id="cmd-meta" style="display:none"></div>
         <div class="cmd-hints">
           <span style="color:var(--blue);font-weight:600">Command Center</span>
-          <span>Ask anything, dispatch work, manage plans — powered by Sonnet</span>
+          <span id="cmd-powered-by">Ask anything, dispatch work, manage plans</span>
           <button class="cmd-history-btn" onclick="cmdShowHistory()">Past Commands</button>
           <button class="pr-pager-btn" style="font-size:9px;padding:2px 8px;color:var(--green);border-color:var(--green)" onclick="openQuickNoteModal()">+ Note</button>
         </div>

package/dashboard.js

@@ -474,7 +474,8 @@ function getStatus() {
         decompose: CONFIG.engine?.autoDecompose !== false,
         tempAgents: !!CONFIG.engine?.allowTempAgents,
         inboxThreshold: CONFIG.engine?.inboxConsolidateThreshold || shared.ENGINE_DEFAULTS.inboxConsolidateThreshold,
-        ccModel: CONFIG.engine?.ccModel || shared.ENGINE_DEFAULTS.ccModel,
+        ccCli: shared.resolveCcCli(CONFIG.engine),
+        ccModel: shared.resolveCcModel(CONFIG.engine),
         ccEffort: CONFIG.engine?.ccEffort || shared.ENGINE_DEFAULTS.ccEffort,
       },
       initialized: !!(CONFIG.agents && Object.keys(CONFIG.agents).length > 0),
@@ -4917,13 +4918,58 @@ What would you like to discuss or change? When you're happy, say "approve" and I
           else if (_validCli(e.ccCli)) config.engine.ccCli = String(e.ccCli);
           else _clamped.push(`ccCli: "${e.ccCli}" not registered (kept previous value)`);
         }
+        // Validate fleet-level model assignments against the resolved runtime.
+        // This is where the bug bit: defaultCli=copilot + defaultModel=gpt-5.5
+        // (where gpt-5.5 doesn't actually exist) cascaded into every agent
+        // that didn't pin its own model. Reject when the model is known to
+        // belong to a different runtime than the one it'll spawn against.
+        const _engineModelDiscovery = require('./engine/model-discovery');
+        const _engineRuntimes = require('./engine/runtimes');
+        async function _validateFleetModel(modelStr, resolvedRuntime) {
+          if (!modelStr) return null;
+          let knownForResolved = null;
+          try {
+            const list = await _engineModelDiscovery.getRuntimeModels(resolvedRuntime, { config });
+            if (Array.isArray(list?.models) && list.models.length > 0) {
+              knownForResolved = new Set(list.models.map(m => m.id || m.name).filter(Boolean));
+            }
+          } catch { /* unknown runtime */ }
+          if (knownForResolved && !knownForResolved.has(modelStr)) {
+            return `not a valid model for runtime "${resolvedRuntime}" (known: ${[...knownForResolved].slice(0, 4).join(', ')}${knownForResolved.size > 4 ? '…' : ''})`;
+          }
+          if (!knownForResolved) {
+            // Free-text runtime (Claude). Reject only if model belongs to a different runtime's published list.
+            for (const rt of _engineRuntimes.listRuntimes()) {
+              if (rt === resolvedRuntime) continue;
+              try {
+                const otherList = await _engineModelDiscovery.getRuntimeModels(rt, { config });
+                if (Array.isArray(otherList?.models) && otherList.models.some(m => (m.id || m.name) === modelStr)) {
+                  return `belongs to runtime "${rt}" but resolved runtime is "${resolvedRuntime}" — incompatible combination`;
+                }
+              } catch { /* skip */ }
+            }
+          }
+          return null;
+        }
         if (e.defaultModel !== undefined) {
           if (_isClear(e.defaultModel)) delete config.engine.defaultModel;
-          else config.engine.defaultModel = String(e.defaultModel);
+          else {
+            const candidate = String(e.defaultModel);
+            const resolvedCli = config.engine.defaultCli || 'claude';
+            const rejection = await _validateFleetModel(candidate, resolvedCli);
+            if (rejection) _clamped.push(`engine.defaultModel: "${candidate}" ${rejection} — kept previous value`);
+            else config.engine.defaultModel = candidate;
+          }
         }
         if (e.ccModel !== undefined) {
           if (_isClear(e.ccModel)) delete config.engine.ccModel;
-          else config.engine.ccModel = String(e.ccModel);
+          else {
+            const candidate = String(e.ccModel);
+            const resolvedCli = config.engine.ccCli || config.engine.defaultCli || 'claude';
+            const rejection = await _validateFleetModel(candidate, resolvedCli);
+            if (rejection) _clamped.push(`engine.ccModel: "${candidate}" ${rejection} — kept previous value`);
+            else config.engine.ccModel = candidate;
+          }
         }
         if (e.claudeFallbackModel !== undefined) {
           if (_isClear(e.claudeFallbackModel)) delete config.engine.claudeFallbackModel;
@@ -4981,6 +5027,34 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       }
 
       if (body.agents) {
+        // Cache cross-runtime model lists once per request so we can reject
+        // claude+gpt-* / copilot+claude-* combinations before they crash a
+        // dispatch (see #model-validation: a stray engine.defaultModel='gpt-5.5'
+        // pinned every Claude agent into a 404 spawn loop).
+        const _modelDiscovery = require('./engine/model-discovery');
+        const _runtimeModelsCache = new Map(); // runtimeName → Set<modelId> (or null when unknown / Claude)
+        async function _modelsFor(runtimeName) {
+          if (_runtimeModelsCache.has(runtimeName)) return _runtimeModelsCache.get(runtimeName);
+          let set = null;
+          try {
+            const list = await _modelDiscovery.getRuntimeModels(runtimeName, { config });
+            if (Array.isArray(list?.models) && list.models.length > 0) {
+              set = new Set(list.models.map(m => m.id || m.name).filter(Boolean));
+            }
+          } catch { /* unknown runtime → free-text */ }
+          _runtimeModelsCache.set(runtimeName, set);
+          return set;
+        }
+        // Returns the runtime that "owns" this model, or null if no other
+        // runtime claims it. Catches "claude + gpt-5.5" by spotting that
+        // gpt-5.5 belongs to copilot's list.
+        async function _ownerOfModel(modelId) {
+          for (const rt of require('./engine/runtimes').listRuntimes()) {
+            const set = await _modelsFor(rt);
+            if (set && set.has(modelId)) return rt;
+          }
+          return null;
+        }
         for (const [id, updates] of Object.entries(body.agents)) {
           if (!config.agents[id]) continue;
           if (updates.role !== undefined) config.agents[id].role = String(updates.role);
@@ -5000,7 +5074,30 @@ What would you like to discuss or change? When you're happy, say "approve" and I
           }
           if (updates.model !== undefined) {
             if (updates.model === '' || updates.model === null) delete config.agents[id].model;
-            else config.agents[id].model = String(updates.model);
+            else {
+              const candidate = String(updates.model);
+              const resolvedCli = config.agents[id].cli || config.engine.defaultCli || 'claude';
+              const knownModels = await _modelsFor(resolvedCli);
+              // Two validation paths:
+              //   1. If the runtime publishes a model list, enforce membership.
+              //   2. If the runtime doesn't (Claude), still reject when the
+              //      model belongs to a DIFFERENT runtime's list — that's how
+              //      we catch claude+gpt-5.5 (gpt-5.5 is in Copilot's list).
+              let rejection = null;
+              if (knownModels && !knownModels.has(candidate)) {
+                rejection = `not a valid model for runtime "${resolvedCli}" (known: ${[...knownModels].slice(0, 4).join(', ')}${knownModels.size > 4 ? '…' : ''})`;
+              } else if (!knownModels) {
+                const owner = await _ownerOfModel(candidate);
+                if (owner && owner !== resolvedCli) {
+                  rejection = `belongs to runtime "${owner}" but agent uses "${resolvedCli}" — incompatible combination`;
+                }
+              }
+              if (rejection) {
+                _clamped.push(`agents.${id}.model: "${candidate}" ${rejection} — kept previous value`);
+              } else {
+                config.agents[id].model = candidate;
+              }
+            }
           }
           if (updates.maxBudgetUsd !== undefined) {
             if (updates.maxBudgetUsd === '' || updates.maxBudgetUsd === null) delete config.agents[id].maxBudgetUsd;

package/package.json

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