@yemi33/minions 0.1.2390 → 0.1.2391

package/dashboard/js/settings.js

@@ -33,6 +33,38 @@ function _isCurrentModelLoad(scope, key, token) {
   return _modelLoadEpochs[scope][key] === token;
 }
 
+// Per-open dedup cache for /api/runtimes/<name>/models payloads.
+// initRuntimeFleetUI() fans out one model fetch per agent PLUS default + CC.
+// When the whole fleet shares a runtime (the common case) those are
+// byte-identical requests, and each one makes the server reloadConfig() from
+// disk + re-run model discovery. Collapsing them to one request per distinct
+// runtime per Settings open removes the N+2 → 1 duplicate round-trips (measured
+// 10 → 1 for an 8-agent single-runtime fleet). forceRefresh (the ↻ button)
+// bypasses the cache and repopulates it with the fresh list. Transient
+// failures are NOT cached so a blip can't poison every agent cell for the open.
+let _modelFetchCache = new Map();
+function _resetModelFetchCache() { _modelFetchCache = new Map(); }
+function _fetchRuntimeModelsPayload(runtimeName, forceRefresh) {
+  if (!runtimeName) return Promise.resolve({ models: null });
+  if (!forceRefresh) {
+    const cached = _modelFetchCache.get(runtimeName);
+    if (cached) return cached;
+  }
+  const suffix = forceRefresh ? '/models/refresh' : '/models';
+  const p = fetch('/api/runtimes/' + encodeURIComponent(runtimeName) + suffix, forceRefresh ? { method: 'POST' } : undefined)
+    .then(res => (res.ok ? res.json() : null))
+    .catch(() => null)
+    .then(payload => {
+      if (!payload || !Array.isArray(payload.models)) {
+        if (_modelFetchCache.get(runtimeName) === p) _modelFetchCache.delete(runtimeName);
+        return payload || { models: null };
+      }
+      return payload;
+    });
+  _modelFetchCache.set(runtimeName, p);
+  return p;
+}
+
 async function openSettings() {
   document.getElementById('modal-title').textContent = 'Settings';
   document.getElementById('modal-body').innerHTML = '<p style="color:var(--muted)">Loading...</p>';
@@ -719,8 +751,16 @@ async function openSettings() {
   // hides unmatched rows so operators can see results without tab-hopping.
   const searchInput = document.getElementById('settings-search');
   if (searchInput) {
+    // Debounce the filter: _applySettingsSearch runs several full-DOM
+    // querySelectorAll scans and reads textContent of every [data-search] row
+    // (a forced reflow). Firing that per-keystroke made typing janky on the
+    // large Settings DOM. The field itself stays instant; only the filter
+    // computation is coalesced to the last keystroke in a ~120ms window.
+    let _searchDebounce = null;
     searchInput.addEventListener('input', function() {
-      _applySettingsSearch(searchInput.value || '');
+      const val = searchInput.value || '';
+      if (_searchDebounce) clearTimeout(_searchDebounce);
+      _searchDebounce = setTimeout(function() { _applySettingsSearch(val); }, 120);
     });
   }
 
@@ -822,6 +862,9 @@ async function initRuntimeFleetUI(engineCfg, agentsCfg) {
   const ccCliSelect = document.getElementById('set-ccCli');
   if (!cliSelect || !ccCliSelect) return;
 
+  // Fresh cache per Settings open so a config change between opens is picked up.
+  _resetModelFetchCache();
+
   // Fetch the registry; render an empty fallback on failure so the rest of the
   // settings panel still works.
   let runtimes = [];
@@ -917,11 +960,7 @@ async function loadModelsForRuntime(runtimeName, inputId, currentValue, forceRef
     return;
   }
   let payload = { models: null };
-  try {
-    const suffix = forceRefresh ? '/models/refresh' : '/models';
-    const res = await fetch('/api/runtimes/' + encodeURIComponent(runtimeName) + suffix, forceRefresh ? { method: 'POST' } : undefined);
-    if (res.ok) payload = await res.json();
-  } catch { /* fall through to free-text */ }
+  payload = await _fetchRuntimeModelsPayload(runtimeName, forceRefresh);
 
   if (!_isCurrentModelLoad('runtime', inputId, token)) return;
   const models = Array.isArray(payload.models) ? payload.models : null;
@@ -962,10 +1001,7 @@ async function loadModelsForAgent(agentId, runtimeName, currentValue) {
     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 */ }
+  payload = await _fetchRuntimeModelsPayload(runtimeName, false);
 
   if (!_isCurrentModelLoad('agent', agentId, token)) return;
   const models = Array.isArray(payload.models) ? payload.models : null;

package/package.json

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