@yemi33/minions 0.1.2439 → 0.1.2441

package/dashboard/js/settings.js

@@ -1960,4 +1960,25 @@ window.MinionsSettings = { openSettings, saveSettings, addProject, removeProject
   } else {
     setTimeout(openWhenReady, 0);
   }
+  // FRESHNESS ON REOPEN (W-ms4ogdc800aa6931): the slim parent now CACHES this
+  // embedded /settings frame (created once, shown/hidden) instead of cold-booting
+  // it per open, so the iframe `load` — and thus the one-shot openSettings() boot
+  // above — fires only ONCE. Re-pull /api/settings + /api/features whenever the
+  // parent re-shows us (visible:true) so a value changed elsewhere isn't stale. A
+  // full openSettings() re-render is cheap here (the SPA is already booted — just
+  // two localhost fetches + a DOM render, no cold boot), so we prefer it over
+  // ad-hoc partial refresh. Reuses the SAME {source:'minions-slim-embed',
+  // type:'visibility'} channel refresh.js already honors (no second postMessage
+  // bus). Embed-mode gated by the isSettingsPath+isEmbed early return above, so a
+  // standalone /settings visit is completely unaffected.
+  try {
+    window.addEventListener('message', function(ev) {
+      if (!ev || ev.origin !== window.location.origin) return;
+      var d = ev.data;
+      if (!d || d.source !== 'minions-slim-embed' || d.type !== 'visibility') return;
+      if (d.visible && typeof openSettings === 'function') {
+        try { openSettings(); } catch (e) { /* transient — next show retries */ }
+      }
+    });
+  } catch (e) { /* defensive — never block the embed boot on this hook */ }
 })();

package/dashboard/slim/body.html

@@ -202,24 +202,24 @@
   </div>
 </div>
 
-<!-- Knowledge control panel — opened from the "Knowledge" status tile. One
-     unified box surfacing all three knowledge surfaces via tabs: Pinned
-     Context, Notes, and KB (knowledge base). Each tab is rendered lazily by
-     dashboard/slim/js/knowledge.js into #slim-knowledge-body. Backed by the
-     existing endpoints (no new server routes):
+<!-- Knowledge control panel — opened from the "Knowledge" status tile. Two
+     tabs: Pinned Context (slim-native, via pinned.js — there is no standalone
+     classic route for pinned, it lives on /home) and Notes & Knowledge, which
+     embeds the LITERAL classic /inbox screen (Notes Inbox + Team Notes +
+     Knowledge Base + memory search) in a chrome-off /inbox?embed=1 iframe
+     rather than a slim-native fork (CLAUDE.md "Slim UX vs Classic UX — reuse,
+     don't fork"). Rendered lazily by dashboard/slim/js/knowledge.js into
+     #slim-knowledge-body. Backed by the existing endpoints (no new routes):
        Pinned — /api/pinned (+ update/remove), surfaced via /api/status.
-       Notes  — /api/notes-full (read), /api/notes-save (edit notes.md),
-                /api/notes (add an inbox note).
-       KB     — /api/knowledge (list), /api/knowledge/:cat/:file (read),
-                /api/knowledge POST (create), /api/kb-pins(+/toggle). -->
+       Notes & Knowledge — the classic /inbox page (/api/notes*, /api/knowledge,
+                /api/kb-pins, memory search) inside the embedded iframe. -->
 <div class="modal-bg" id="slim-knowledge-modal">
   <div class="modal slim-knowledge-modal-inner">
     <div class="modal-header">
       <h3>Knowledge</h3>
       <div class="kn-tabs" id="slim-kn-tabs" role="tablist">
         <button class="kn-tab active" id="slim-kn-tab-pinned" data-kn-tab="pinned" type="button" role="tab" aria-selected="true" aria-controls="slim-knowledge-body">Pinned Context</button>
-        <button class="kn-tab" id="slim-kn-tab-notes" data-kn-tab="notes" type="button" role="tab" aria-selected="false" aria-controls="slim-knowledge-body">Notes</button>
-        <button class="kn-tab" id="slim-kn-tab-kb" data-kn-tab="kb" type="button" role="tab" aria-selected="false" aria-controls="slim-knowledge-body">KB</button>
+        <button class="kn-tab" id="slim-kn-tab-inbox" data-kn-tab="inbox" type="button" role="tab" aria-selected="false" aria-controls="slim-knowledge-body">Notes &amp; Knowledge</button>
       </div>
       <button id="slim-knowledge-close" class="icon-btn" title="Close" style="margin-left:auto">&times;</button>
     </div>

package/dashboard/slim/js/knowledge.js

@@ -1,34 +1,23 @@
 
-  // ── Knowledge control panel (Pinned Context + Notes + KB) ──────────
-  // One unified cockpit box over the three knowledge surfaces, replacing the
-  // old single "Pinned context" tile. Reuses the classic dashboard's existing
-  // endpoints (no new server routes):
-  //   Pinned — /api/pinned (+ update/remove); the Pinned-Context tab reuses the
-  //            editor + list renderer in pinned.js.
-  //   Notes  — /api/notes-full (read notes.md), /api/notes-save (edit notes.md),
-  //            /api/notes (add an inbox note for consolidation).
-  //   KB     — /api/knowledge (list), /api/knowledge/:cat/:file (read entry),
-  //            /api/knowledge POST (create), /api/kb-pins(+/toggle) (pin state).
+  // ── Knowledge control panel (Pinned Context + Notes & Knowledge) ────
+  // One cockpit box over the knowledge surfaces, replacing the old single
+  // "Pinned context" tile. Two tabs:
+  //   Pinned — slim-native (there is no standalone classic route for pinned;
+  //            it lives on /home). Reuses the editor + list renderer in
+  //            pinned.js, backed by /api/pinned (+ update/remove).
+  //   Notes & Knowledge — embeds the LITERAL classic /inbox screen (Notes
+  //            Inbox + Team Notes + Knowledge Base + memory search) in a
+  //            chrome-off /inbox?embed=1 iframe, per CLAUDE.md "Slim UX vs
+  //            Classic UX — reuse, don't fork". Zero forked rendering logic.
   //
   // Counts: pinned is live from the 5s status poll (renderKnowledgeTile is fed
   // by applyStatus). Notes + KB counts are fetched lazily — once shortly after
-  // load and again after any create/save — so the heavier /api/notes-full and
+  // load and again on modal open — so the heavier /api/notes-full and
   // /api/knowledge reads stay off the recurring poll.
 
   var _knowledgeCounts = { pinned: 0, notes: null, kb: null };
   var _knActiveTab = 'pinned';
-  var _kbData = null;      // cached /api/knowledge payload
-  var _kbPins = [];        // cached pinned KB keys ('knowledge/<cat>/<file>')
-
-  var KB_CAT_LABELS = {
-    architecture: 'Architecture', conventions: 'Conventions',
-    'project-notes': 'Project Notes', 'build-reports': 'Build Reports',
-    reviews: 'Reviews', learnings: 'Learnings', decisions: 'Decisions',
-    incidents: 'Incidents', 'api-notes': 'API Notes',
-  };
-  // Categories offered in the KB create form (mirrors the classic dashboard).
-  var KB_CREATE_CATS = ['architecture', 'conventions', 'project-notes', 'build-reports', 'reviews'];
-  var KB_LIST_CAP = 60;    // cap rendered rows; the KB can hold thousands
+  var _knInboxFrame = null;   // cached /inbox?embed=1 iframe (created once)
 
   // ── Cockpit tile ───────────────────────────────────────────────────
   // Called from applyStatus with the live pinned count each poll, and from the
@@ -68,35 +57,24 @@
     try {
       var kres = await fetch('/api/knowledge', { headers: { 'Accept': 'application/json' } });
       if (kres.ok) {
-        _kbData = await kres.json();
-        _knowledgeCounts.kb = _countKbEntries(_kbData);
+        _knowledgeCounts.kb = _countKbEntries(await kres.json());
       }
     } catch (e) { /* keep last-known */ }
     renderKnowledgeTile();
   }
 
-  async function loadKbPins() {
-    try {
-      var res = await fetch('/api/kb-pins', { headers: { 'Accept': 'application/json' } });
-      if (!res.ok) return;
-      var d = await res.json();
-      _kbPins = Array.isArray(d.pins) ? d.pins : [];
-    } catch (e) { _kbPins = _kbPins || []; }
-  }
-
-  function kbPinKey(cat, file) { return 'knowledge/' + cat + '/' + file; }
-  function isKbPinned(key) { return _kbPins.indexOf(key) !== -1; }
-
   // ── Modal open/close + tab switching ───────────────────────────────
   function openKnowledgeModal() {
     var modal = document.getElementById('slim-knowledge-modal');
     if (!modal) return;
-    renderKnowledgeTab();
+    renderKnowledgeTab();   // posts visibility=true to the embed if the inbox tab is active
     modal.classList.add('open');
   }
   function closeKnowledgeModal() {
     var modal = document.getElementById('slim-knowledge-modal');
     if (modal) modal.classList.remove('open');
+    // Suspend the embedded /inbox poll while the modal is closed.
+    _postKnEmbedVisibility(false);
   }
   function setKnowledgeTab(tab) {
     _knActiveTab = tab;
@@ -112,13 +90,81 @@
     if (body && activeTabId) body.setAttribute('aria-labelledby', activeTabId);
     renderKnowledgeTab();
   }
+  // Tab routing. The Pinned tab renders slim-native content into a transient
+  // container; the Notes & Knowledge tab shows a cached /inbox?embed=1 iframe
+  // (created once, kept mounted so a tab switch never reloads it).
   function renderKnowledgeTab() {
+    var c = _ensureKnContainers();
+    if (!c) return;
+    if (_knActiveTab === 'inbox') {
+      renderKnowledgeInboxTab();
+    } else {
+      c.host.hidden = true;
+      _postKnEmbedVisibility(false);
+      c.transient.textContent = '';
+      renderKnowledgePinnedTab(c.transient);
+    }
+  }
+
+  // Persistent containers inside #slim-knowledge-body: an embed host that holds
+  // the cached iframe (never wiped, so the iframe is never re-parented/reloaded)
+  // and a transient container for the Pinned tab's slim-native content. Built
+  // with createElement (no innerHTML) to satisfy the no-unsanitized lint gate.
+  function _ensureKnContainers() {
     var body = document.getElementById('slim-knowledge-body');
-    if (!body) return;
-    body.textContent = '';
-    if (_knActiveTab === 'notes') renderKnowledgeNotesTab(body);
-    else if (_knActiveTab === 'kb') renderKnowledgeKbTab(body);
-    else renderKnowledgePinnedTab(body);
+    if (!body) return null;
+    var host = document.getElementById('slim-kn-embed-host');
+    if (!host) {
+      host = document.createElement('div');
+      host.id = 'slim-kn-embed-host';
+      host.className = 'slim-kn-embed-host';
+      host.hidden = true;
+      body.appendChild(host);
+    }
+    var transient = document.getElementById('slim-kn-transient');
+    if (!transient) {
+      transient = document.createElement('div');
+      transient.id = 'slim-kn-transient';
+      transient.className = 'slim-kn-transient';
+      body.appendChild(transient);
+    }
+    return { host: host, transient: transient };
+  }
+
+  // Tell the embedded classic /inbox SPA whether its iframe is visible so it can
+  // suspend/resume its own /api/status poll (dashboard/js/refresh.js listens for
+  // this exact message shape). Same-origin, guarded defensively.
+  function _postKnEmbedVisibility(visible) {
+    try {
+      if (_knInboxFrame && _knInboxFrame.contentWindow) {
+        _knInboxFrame.contentWindow.postMessage(
+          { source: 'minions-slim-embed', type: 'visibility', visible: !!visible }, '*');
+      }
+    } catch (e) { /* defensive — same-origin means this should never throw */ }
+  }
+
+  // ── Tab: Notes & Knowledge (classic /inbox embed) ──────────────────
+  // Lazily creates ONE cached iframe pointing at the classic Notes+KB screen
+  // (/inbox?embed=1) and mounts it into the persistent embed host. Subsequent
+  // activations just re-show the same iframe (never recreated), so switching
+  // tabs never reloads it.
+  function renderKnowledgeInboxTab() {
+    var c = _ensureKnContainers();
+    if (!c) return;
+    c.transient.textContent = '';
+    if (!_knInboxFrame) {
+      var frame = document.createElement('iframe');
+      frame.className = 'slim-kn-embed';
+      frame.src = '/inbox?embed=1';
+      frame.title = 'Notes & Knowledge';
+      frame.addEventListener('load', function() {
+        _postKnEmbedVisibility(_knActiveTab === 'inbox');
+      });
+      c.host.appendChild(frame);
+      _knInboxFrame = frame;
+    }
+    c.host.hidden = false;
+    _postKnEmbedVisibility(true);
   }
 
   // ── Tab: Pinned Context ────────────────────────────────────────────
@@ -143,423 +189,10 @@
     renderSlimPinnedList();   // defined in pinned.js; renders into #slim-pinned-list
   }
 
-  // ── Tab: Notes ─────────────────────────────────────────────────────
-  function renderKnowledgeNotesTab(body) {
-    var intro = document.createElement('p');
-    intro.textContent = 'Consolidated team notes (notes.md). Edit and save, or add a note for the next consolidation sweep.';
-    body.appendChild(intro);
-
-    var msg = document.createElement('div');
-    msg.id = 'slim-notes-msg';
-    msg.className = 'kn-msg';
-    body.appendChild(msg);
-
-    var ta = document.createElement('textarea');
-    ta.id = 'slim-notes-content';
-    ta.className = 'linkpr-input kn-notes-textarea';
-    ta.rows = 12;
-    ta.value = 'Loading notes…';
-    ta.disabled = true;
-    body.appendChild(ta);
-
-    var saveRow = document.createElement('div');
-    saveRow.className = 'kn-toolbar kn-toolbar-end';
-    var saveBtn = document.createElement('button');
-    saveBtn.className = 'btn-primary';
-    saveBtn.type = 'button';
-    saveBtn.textContent = 'Save notes.md';
-    saveBtn.disabled = true;
-    saveBtn.addEventListener('click', saveKnowledgeNotes);
-    saveRow.appendChild(saveBtn);
-    body.appendChild(saveRow);
-
-    // Add-a-note (inbox) sub-form.
-    var addHead = document.createElement('div');
-    addHead.className = 'kn-section-head';
-    addHead.textContent = 'Add a note for consolidation';
-    body.appendChild(addHead);
-
-    var titleLabel = document.createElement('label');
-    titleLabel.className = 'linkpr-label';
-    titleLabel.textContent = 'Title';
-    var titleInput = document.createElement('input');
-    titleInput.id = 'slim-note-title';
-    titleInput.className = 'linkpr-input';
-    titleInput.type = 'text';
-    titleInput.placeholder = 'Short summary';
-    titleLabel.appendChild(titleInput);
-    body.appendChild(titleLabel);
-
-    var whatLabel = document.createElement('label');
-    whatLabel.className = 'linkpr-label';
-    whatLabel.textContent = 'Note';
-    var whatInput = document.createElement('textarea');
-    whatInput.id = 'slim-note-what';
-    whatInput.className = 'linkpr-input';
-    whatInput.rows = 3;
-    whatInput.placeholder = 'What did you learn / decide?';
-    whatLabel.appendChild(whatInput);
-    body.appendChild(whatLabel);
-
-    var addRow = document.createElement('div');
-    addRow.className = 'kn-toolbar kn-toolbar-end';
-    var addNoteMsg = document.createElement('span');
-    addNoteMsg.id = 'slim-note-msg';
-    addNoteMsg.className = 'kn-msg kn-msg-inline';
-    addRow.appendChild(addNoteMsg);
-    var addNoteBtn = document.createElement('button');
-    addNoteBtn.className = 'btn-secondary';
-    addNoteBtn.type = 'button';
-    addNoteBtn.textContent = 'Add note';
-    addNoteBtn.addEventListener('click', submitKnowledgeNote);
-    addRow.appendChild(addNoteBtn);
-    body.appendChild(addRow);
-
-    // Load notes.md into the textarea.
-    fetch('/api/notes-full', { headers: { 'Accept': 'text/plain' } })
-      .then(function(r) { return r.ok ? r.text() : Promise.reject(new Error('HTTP ' + r.status)); })
-      .then(function(text) {
-        if (document.getElementById('slim-notes-content') !== ta) return;  // tab switched away
-        ta.value = (text === 'No notes file found.') ? '' : text;
-        ta.disabled = false;
-        saveBtn.disabled = false;
-        _knowledgeCounts.notes = _countNotesEntries(text);
-        renderKnowledgeTile();
-      })
-      .catch(function(e) {
-        ta.value = '';
-        ta.disabled = false;
-        saveBtn.disabled = false;
-        if (msg) { msg.style.color = 'var(--red)'; msg.textContent = 'Could not load notes: ' + (e.message || e); }
-      });
-  }
-
-  async function saveKnowledgeNotes() {
-    var ta = document.getElementById('slim-notes-content');
-    var msg = document.getElementById('slim-notes-msg');
-    if (!ta) return;
-    if (msg) { msg.style.color = 'var(--muted)'; msg.textContent = 'Saving…'; }
-    try {
-      var res = await fetch('/api/notes-save', {
-        method: 'POST', headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ content: ta.value, file: 'notes.md' }),
-      });
-      var d = await res.json().catch(function() { return {}; });
-      if (!res.ok) throw new Error(d.error || ('HTTP ' + res.status));
-      if (msg) { msg.style.color = 'var(--green)'; msg.textContent = 'Saved.'; }
-      _knowledgeCounts.notes = _countNotesEntries(ta.value);
-      renderKnowledgeTile();
-    } catch (e) {
-      if (msg) { msg.style.color = 'var(--red)'; msg.textContent = 'Error: ' + (e && e.message ? e.message : 'failed'); }
-    }
-  }
-
-  async function submitKnowledgeNote() {
-    var titleEl = document.getElementById('slim-note-title');
-    var whatEl = document.getElementById('slim-note-what');
-    var msg = document.getElementById('slim-note-msg');
-    var title = titleEl ? (titleEl.value || '').trim() : '';
-    var what = whatEl ? (whatEl.value || '').trim() : '';
-    if (!title) {
-      if (msg) { msg.style.color = 'var(--red)'; msg.textContent = 'Title is required.'; }
-      return;
-    }
-    if (msg) { msg.style.color = 'var(--muted)'; msg.textContent = 'Adding…'; }
-    try {
-      var res = await fetch('/api/notes', {
-        method: 'POST', headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ title: title, what: what }),
-      });
-      var d = await res.json().catch(function() { return {}; });
-      if (!res.ok) throw new Error(d.error || ('HTTP ' + res.status));
-      if (titleEl) titleEl.value = '';
-      if (whatEl) whatEl.value = '';
-      if (msg) { msg.style.color = 'var(--green)'; msg.textContent = 'Note added for consolidation.'; }
-    } catch (e) {
-      if (msg) { msg.style.color = 'var(--red)'; msg.textContent = 'Error: ' + (e && e.message ? e.message : 'failed'); }
-    }
-  }
-
-  // ── Tab: KB ────────────────────────────────────────────────────────
-  function renderKnowledgeKbTab(body) {
-    var intro = document.createElement('p');
-    intro.textContent = 'Knowledge base — categorized entries written by consolidation. Click an entry to read it.';
-    body.appendChild(intro);
-
-    var toolbar = document.createElement('div');
-    toolbar.className = 'kn-toolbar';
-    var newBtn = document.createElement('button');
-    newBtn.className = 'linkpr-chip';
-    newBtn.type = 'button';
-    newBtn.textContent = '+ New entry';
-    newBtn.addEventListener('click', function() { toggleKbCreateForm(body); });
-    toolbar.appendChild(newBtn);
-    body.appendChild(toolbar);
-
-    var listMsg = document.createElement('div');
-    listMsg.id = 'slim-kb-list-msg';
-    listMsg.className = 'kn-msg';
-    body.appendChild(listMsg);
-
-    var listWrap = document.createElement('div');
-    listWrap.id = 'slim-kb-list';
-    body.appendChild(listWrap);
-
-    if (_kbData) renderKbList(listWrap);
-    else {
-      var loading = document.createElement('div');
-      loading.className = 'tile-empty';
-      loading.textContent = 'Loading knowledge base…';
-      listWrap.appendChild(loading);
-    }
-    // Always refresh list + pins on open so newly-swept entries appear.
-    Promise.all([
-      fetch('/api/knowledge', { headers: { 'Accept': 'application/json' } }).then(function(r) { return r.ok ? r.json() : null; }).catch(function() { return null; }),
-      loadKbPins(),
-    ]).then(function(results) {
-      if (results[0]) {
-        _kbData = results[0];
-        _knowledgeCounts.kb = _countKbEntries(_kbData);
-        renderKnowledgeTile();
-      }
-      if (_knActiveTab === 'kb') {
-        var lw = document.getElementById('slim-kb-list');
-        if (lw) renderKbList(lw);
-      }
-    });
-  }
-
-  function _kbAllItems() {
-    var items = [];
-    if (!_kbData) return items;
-    Object.keys(_kbData).forEach(function(cat) {
-      if (!Array.isArray(_kbData[cat])) return;
-      _kbData[cat].forEach(function(item) {
-        items.push({ file: item.file, category: cat, title: item.title, agent: item.agent, date: item.date, sortTs: item.sortTs, preview: item.preview });
-      });
-    });
-    return items;
-  }
-
-  function renderKbList(listWrap) {
-    listWrap.textContent = '';
-    var items = _kbAllItems();
-    if (!items.length) {
-      var empty = document.createElement('div');
-      empty.className = 'tile-empty';
-      empty.textContent = 'No knowledge entries yet. Notes are classified here after consolidation.';
-      listWrap.appendChild(empty);
-      return;
-    }
-    // Pinned first, then newest.
-    items.sort(function(a, b) {
-      var ap = isKbPinned(kbPinKey(a.category, a.file));
-      var bp = isKbPinned(kbPinKey(b.category, b.file));
-      if (ap !== bp) return ap ? -1 : 1;
-      return (b.sortTs || 0) - (a.sortTs || 0) || (b.date || '').localeCompare(a.date || '');
-    });
-    var shown = items.slice(0, KB_LIST_CAP);
-    shown.forEach(function(item) {
-      listWrap.appendChild(buildKbRow(item));
-    });
-    if (items.length > shown.length) {
-      var more = document.createElement('div');
-      more.className = 'tile-empty';
-      more.textContent = 'Showing ' + shown.length + ' of ' + items.length + ' entries (newest + pinned first).';
-      listWrap.appendChild(more);
-    }
-  }
-
-  function buildKbRow(item) {
-    var key = kbPinKey(item.category, item.file);
-    var pinned = isKbPinned(key);
-    var row = document.createElement('div');
-    row.className = 'kb-row' + (pinned ? ' kb-row-pinned' : '');
-
-    var top = document.createElement('div');
-    top.className = 'kb-row-top';
-
-    var pinBtn = document.createElement('button');
-    pinBtn.className = 'kb-pin-btn' + (pinned ? ' on' : '');
-    pinBtn.type = 'button';
-    pinBtn.title = pinned ? 'Unpin from KB' : 'Pin in KB';
-    pinBtn.textContent = pinned ? '★' : '☆';
-    pinBtn.addEventListener('click', function(ev) { ev.stopPropagation(); toggleKbPin(key); });
-    top.appendChild(pinBtn);
-
-    var title = document.createElement('span');
-    title.className = 'kb-row-title';
-    title.textContent = item.title || item.file || '(untitled)';
-    title.title = item.title || '';
-    top.appendChild(title);
-    row.appendChild(top);
-
-    var meta = document.createElement('div');
-    meta.className = 'kb-row-meta';
-    meta.textContent = [KB_CAT_LABELS[item.category] || item.category, item.agent || null, item.date || null]
-      .filter(Boolean).join(' · ');
-    row.appendChild(meta);
-
-    if (item.preview) {
-      var prev = document.createElement('div');
-      prev.className = 'kb-row-preview';
-      var t = String(item.preview);
-      prev.textContent = t.length > 200 ? t.slice(0, 200) + '…' : t;
-      row.appendChild(prev);
-    }
-
-    row.addEventListener('click', function() { openKbItem(item.category, item.file); });
-    return row;
-  }
-
-  async function toggleKbPin(key) {
-    var listMsg = document.getElementById('slim-kb-list-msg');
-    if (listMsg) { listMsg.style.color = ''; listMsg.textContent = ''; }
-    try {
-      var res = await fetch('/api/kb-pins/toggle', {
-        method: 'POST', headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ key: key }),
-      });
-      var d = await res.json().catch(function() { return {}; });
-      if (!res.ok) throw new Error(d.error || ('HTTP ' + res.status));
-      if (d.pinned) { if (_kbPins.indexOf(key) === -1) _kbPins.push(key); }
-      else { _kbPins = _kbPins.filter(function(k) { return k !== key; }); }
-      var lw = document.getElementById('slim-kb-list');
-      if (lw) renderKbList(lw);
-    } catch (e) {
-      if (listMsg) { listMsg.style.color = 'var(--red)'; listMsg.textContent = 'Pin toggle failed: ' + (e && e.message ? e.message : e); }
-    }
-  }
-
-  async function openKbItem(category, file) {
-    var body = document.getElementById('slim-knowledge-body');
-    if (!body) return;
-    body.textContent = '';
-    var backRow = document.createElement('div');
-    backRow.className = 'kn-toolbar';
-    var backBtn = document.createElement('button');
-    backBtn.className = 'btn-secondary';
-    backBtn.type = 'button';
-    backBtn.textContent = '← Back to list';
-    backBtn.addEventListener('click', function() { renderKnowledgeTab(); });
-    backRow.appendChild(backBtn);
-    body.appendChild(backRow);
-
-    var head = document.createElement('div');
-    head.className = 'kn-section-head';
-    head.textContent = file;
-    body.appendChild(head);
-
-    var pre = document.createElement('pre');
-    pre.className = 'kb-entry-content';
-    pre.textContent = 'Loading…';
-    body.appendChild(pre);
-
-    try {
-      var res = await fetch('/api/knowledge/' + encodeURIComponent(category) + '/' + encodeURIComponent(file));
-      if (!res.ok) throw new Error('HTTP ' + res.status);
-      var text = await res.text();
-      pre.textContent = text.replace(/^---[\s\S]*?---\n*/m, '');
-    } catch (e) {
-      pre.textContent = 'Failed to load entry: ' + (e && e.message ? e.message : e);
-    }
-  }
-
-  function toggleKbCreateForm(body) {
-    var existing = document.getElementById('slim-kb-create');
-    if (existing) { existing.parentNode.removeChild(existing); return; }
-    var form = document.createElement('div');
-    form.id = 'slim-kb-create';
-    form.className = 'kn-create-form';
-
-    var catLabel = document.createElement('label');
-    catLabel.className = 'linkpr-label';
-    catLabel.textContent = 'Category';
-    var catSel = document.createElement('select');
-    catSel.id = 'slim-kb-category';
-    catSel.className = 'linkpr-input';
-    KB_CREATE_CATS.forEach(function(c) {
-      var opt = document.createElement('option');
-      opt.value = c;
-      opt.textContent = KB_CAT_LABELS[c] || c;
-      catSel.appendChild(opt);
-    });
-    catLabel.appendChild(catSel);
-    form.appendChild(catLabel);
-
-    var titleLabel = document.createElement('label');
-    titleLabel.className = 'linkpr-label';
-    titleLabel.textContent = 'Title';
-    var titleInput = document.createElement('input');
-    titleInput.id = 'slim-kb-title';
-    titleInput.className = 'linkpr-input';
-    titleInput.type = 'text';
-    titleInput.placeholder = 'Entry title';
-    titleLabel.appendChild(titleInput);
-    form.appendChild(titleLabel);
-
-    var contentLabel = document.createElement('label');
-    contentLabel.className = 'linkpr-label';
-    contentLabel.textContent = 'Content';
-    var contentInput = document.createElement('textarea');
-    contentInput.id = 'slim-kb-content';
-    contentInput.className = 'linkpr-input';
-    contentInput.rows = 6;
-    contentInput.placeholder = 'Write your knowledge entry…';
-    contentLabel.appendChild(contentInput);
-    form.appendChild(contentLabel);
-
-    var row = document.createElement('div');
-    row.className = 'kn-toolbar kn-toolbar-end';
-    var msg = document.createElement('span');
-    msg.id = 'slim-kb-create-msg';
-    msg.className = 'kn-msg kn-msg-inline';
-    row.appendChild(msg);
-    var saveBtn = document.createElement('button');
-    saveBtn.className = 'btn-primary';
-    saveBtn.type = 'button';
-    saveBtn.textContent = 'Save entry';
-    saveBtn.addEventListener('click', submitKbEntry);
-    row.appendChild(saveBtn);
-    form.appendChild(row);
-
-    // Insert the form directly after the toolbar (before the list).
-    var listWrap = document.getElementById('slim-kb-list');
-    if (listWrap && listWrap.parentNode) listWrap.parentNode.insertBefore(form, listWrap);
-    else body.appendChild(form);
-  }
-
-  async function submitKbEntry() {
-    var catEl = document.getElementById('slim-kb-category');
-    var titleEl = document.getElementById('slim-kb-title');
-    var contentEl = document.getElementById('slim-kb-content');
-    var msg = document.getElementById('slim-kb-create-msg');
-    var category = catEl ? catEl.value : '';
-    var title = titleEl ? (titleEl.value || '').trim() : '';
-    var content = contentEl ? contentEl.value : '';
-    if (!title || !content.trim()) {
-      if (msg) { msg.style.color = 'var(--red)'; msg.textContent = 'Title and content are required.'; }
-      return;
-    }
-    if (msg) { msg.style.color = 'var(--muted)'; msg.textContent = 'Saving…'; }
-    try {
-      var res = await fetch('/api/knowledge', {
-        method: 'POST', headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ category: category, title: title, content: content }),
-      });
-      var d = await res.json().catch(function() { return {}; });
-      if (!res.ok) throw new Error(d.error || ('HTTP ' + res.status));
-      // Refresh the list + count, then re-render the KB tab.
-      var kres = await fetch('/api/knowledge', { headers: { 'Accept': 'application/json' } });
-      if (kres.ok) { _kbData = await kres.json(); _knowledgeCounts.kb = _countKbEntries(_kbData); renderKnowledgeTile(); }
-      if (_knActiveTab === 'kb') renderKnowledgeTab();
-    } catch (e) {
-      if (msg) { msg.style.color = 'var(--red)'; msg.textContent = 'Error: ' + (e && e.message ? e.message : 'failed'); }
-    }
-  }
-
   // ── Wiring ─────────────────────────────────────────────────────────
-  bindModalClose('slim-knowledge-modal', 'slim-knowledge-close');
+  // onClose fires on every dismiss path (close-X, backdrop, Esc) so the
+  // embedded /inbox poll is suspended whenever the modal closes.
+  bindModalClose('slim-knowledge-modal', 'slim-knowledge-close', function() { _postKnEmbedVisibility(false); });
   (function bindKnowledgeUi() {
     var tabBar = document.getElementById('slim-kn-tabs');
     if (tabBar) {

package/dashboard/slim/js/settings.js

@@ -18,16 +18,44 @@
   // — two stacked "Settings" bars was the duplicate-menu-bar bug
   // (W-mrpbzke4000ie609). Closing is driven from the embedded modal's own close
   // X (wired below to the parent), plus backdrop-click and Escape.
-  function openSlimSettings() {
-    var modal = document.getElementById('slim-settings-modal');
+  // CACHED settings iframe (W-ms4ogdc800aa6931). The old code discarded the
+  // frame on close and appended a BRAND-NEW `/settings?embed=1` iframe on every
+  // open, so each open cold-booted the entire classic SPA + full Settings render
+  // (/api/settings + /api/features + per-agent runtime dropdowns + projects +
+  // routing, plus the SPA's 4s /api/status poll) — the multi-second stall. We
+  // now mirror the cockpit-tile reuse contract (modals-tiles.js
+  // #_getOrCreateEmbedFrame / _showReusableEmbed): create the frame ONCE, keep
+  // it mounted (hidden), and show/hide it. A single background boot replaces N
+  // per-open cold boots; while hidden the embedded SPA's poll is suspended via
+  // the shared {source:'minions-slim-embed',type:'visibility'} message
+  // (refresh.js honors it in embed mode) so the cached frame doesn't poll
+  // /api/status forever off-screen.
+  var _slimSettingsFrame = null;
+
+  // Create (once) the cached settings iframe and return it. Built with
+  // createElement (no innerHTML) to satisfy the dashboard no-unsanitized lint
+  // gate. Starts hidden; a load-time visibility message suspends its poll until
+  // it is shown. Because the frame is cached, its `load` handler fires exactly
+  // ONCE — so the close/Escape wiring is installed here and persists across all
+  // hide/show cycles.
+  function _getOrCreateSettingsFrame() {
+    if (_slimSettingsFrame) return _slimSettingsFrame;
     var body = document.getElementById('slim-settings-body');
-    if (!modal || !body) return;
-    body.textContent = '';
+    if (!body) return null;
+    body.textContent = '';   // drop the "Loading settings…" placeholder (once)
     var frame = document.createElement('iframe');
     frame.className = 'slim-settings-embed';
     frame.src = '/settings?embed=1';
     frame.title = 'Settings';
+    frame.hidden = true;
     frame.addEventListener('load', function() {
+      // Suspend/resume the embedded SPA poll to match current visibility. On a
+      // visible:true the embedded classic Settings also re-pulls /api/settings +
+      // /api/features (see dashboard/js/settings.js#bootSettingsEmbed) — the
+      // same single postMessage channel, no second bus.
+      var modal = document.getElementById('slim-settings-modal');
+      var shown = !!(modal && modal.classList.contains('open'));
+      if (typeof _postEmbedVisibility === 'function') _postEmbedVisibility(frame, shown);
       var frameDocument = frame.contentDocument;
       if (!frameDocument) return;
       // Capture before the classic modal handler so only the parent modal closes.
@@ -53,20 +81,50 @@
       }
     });
     body.appendChild(frame);
+    _slimSettingsFrame = frame;
+    return frame;
+  }
+
+  function openSlimSettings() {
+    var modal = document.getElementById('slim-settings-modal');
+    var frame = _getOrCreateSettingsFrame();
+    if (!modal || !frame) return;
+    frame.hidden = false;
+    // Resume the embedded SPA poll + trigger a fresh /api/settings+/api/features
+    // re-pull (freshness on reopen — the cached frame's `load` fired only once,
+    // so without this a value changed elsewhere would show stale).
+    if (typeof _postEmbedVisibility === 'function') _postEmbedVisibility(frame, true);
     modal.classList.add('open');
   }
 
   function closeSlimSettings() {
-    document.getElementById('slim-settings-modal').classList.remove('open');
+    var modal = document.getElementById('slim-settings-modal');
+    if (!modal) return;
+    // Keep the frame MOUNTED for an instant reopen; just hide it and suspend its
+    // poll (so the cached frame stops polling /api/status in the background).
+    if (_slimSettingsFrame) {
+      if (typeof _postEmbedVisibility === 'function') _postEmbedVisibility(_slimSettingsFrame, false);
+      _slimSettingsFrame.hidden = true;
+    }
+    modal.classList.remove('open');
   }
 
   (function bindSettingsUi() {
     var btn = document.getElementById('slim-settings-btn');
     var backdrop = document.getElementById('slim-settings-modal');
-    if (btn) btn.addEventListener('click', openSlimSettings);
+    if (btn) {
+      btn.addEventListener('click', openSlimSettings);
+      // PREDICTIVE prewarm: the first hover / focus / press of the gear boots the
+      // cached settings frame before the click lands, so the open is an instant
+      // show. Fires at most once (the frame is cached thereafter).
+      var predictivePrewarm = function() { _getOrCreateSettingsFrame(); };
+      btn.addEventListener('pointerenter', predictivePrewarm, { once: true });
+      btn.addEventListener('pointerdown', predictivePrewarm, { once: true });
+      btn.addEventListener('focus', predictivePrewarm, { once: true });
+    }
     // No outer close button: the embedded classic Settings modal's own close X
-    // (wired to the parent in openSlimSettings) is the single close affordance,
-    // alongside backdrop-click and Escape below (W-mrpbzke4000ie609).
+    // (wired to the parent in _getOrCreateSettingsFrame) is the single close
+    // affordance, alongside backdrop-click and Escape below (W-mrpbzke4000ie609).
     if (backdrop) {
       backdrop.addEventListener('click', function(ev) {
         if (ev.target === backdrop) closeSlimSettings();
@@ -78,6 +136,22 @@
     });
   })();
 
+  // Deferred prewarm: boot the cached settings frame AFTER the cockpit's own hot
+  // embed prewarm (modals-tiles.js schedules that at ~800ms) so it never competes
+  // with slim cockpit initial paint. Idle-callback when available, else a delayed
+  // timeout — either way it runs well after first paint. First open then becomes
+  // an instant show instead of a cold boot.
+  (function scheduleSettingsPrewarm() {
+    var warm = function() { try { _getOrCreateSettingsFrame(); } catch (e) { /* skip prewarm */ } };
+    try {
+      if (typeof requestIdleCallback === 'function') {
+        requestIdleCallback(function() { setTimeout(warm, 1200); }, { timeout: 4000 });
+      } else {
+        setTimeout(warm, 1600);
+      }
+    } catch (e) { /* no timers available — skip prewarm, first open cold-boots */ }
+  })();
+
   // POST a feature-flag toggle; throws on a non-2xx so callers can branch on
   // failure. Used by the one-click returnToClassicDashboard().
   async function postFeatureToggle(id, enabled) {

package/dashboard/slim/styles.css

@@ -797,6 +797,12 @@
        outranks the class rules (no !important needed). */
     .slim-tile-embed-host, .slim-tile-transient { display: block; }
     #slim-tile-embed-host iframe[hidden] { display: none; }
+    /* Cached slim Settings frame (W-ms4ogdc800aa6931): like the tile embeds it is
+       created once and shown/hidden rather than cold-booted per open. The
+       .slim-settings-embed class rule above sets display:block, which outranks
+       [hidden]'s UA display:none — so a hidden cached frame must be forced back
+       to none. The id+attr selector outranks the class rule (no !important). */
+    #slim-settings-body iframe[hidden] { display: none; }
     /* Automation tile (Watches · Schedules · Pipelines) — a tabbed composite
        inside the shared wide tile-modal. The tab bar sits above the embedded
        classic screen, so the iframe height leaves room for it (vs the padless
@@ -867,8 +873,9 @@
     .pinned-row-actions .btn-secondary { padding: 4px 12px; font-size: var(--text-base); }
     .pinned-row-unpin { color: var(--red); border-color: var(--red); }
 
-    /* Knowledge control panel (slim-knowledge-modal): Pinned Context / Notes /
-       KB tabs in one box. Reuses .tile-* and .pinned-row primitives. */
+    /* Knowledge control panel (slim-knowledge-modal): Pinned Context (slim-
+       native) + Notes & Knowledge (embedded classic /inbox) tabs in one box.
+       Reuses .tile-* and .pinned-row primitives. */
     .slim-knowledge-modal-inner { width: 720px; max-width: calc(100vw - 32px); }
     .kn-tabs { display: flex; gap: 6px; margin-left: 16px; }
     .kn-tab {
@@ -899,51 +906,13 @@
     }
     .kn-msg { font-size: var(--text-base); min-height: 16px; color: var(--muted); }
     .kn-msg-inline { margin-right: auto; }
-    .kn-notes-textarea { font-family: var(--mono, monospace); white-space: pre; }
-    .kn-create-form {
-      border: 1px solid var(--border);
-      border-radius: var(--radius);
-      background: var(--surface2);
-      padding: 12px;
-      margin-bottom: 12px;
-    }
-    /* KB list rows */
-    .kb-row {
-      border: 1px solid var(--border);
-      border-radius: var(--radius);
-      background: var(--surface2);
-      padding: 8px 10px;
-      margin-bottom: 8px;
-      cursor: pointer;
-    }
-    .kb-row:hover { border-color: var(--blue); }
-    .kb-row-pinned { border-color: var(--amber); }
-    .kb-row-top { display: flex; align-items: center; gap: 8px; }
-    .kb-row-title {
-      flex: 1; min-width: 0;
-      font-weight: 600; font-size: var(--text-md); color: var(--text);
-      overflow: hidden; text-overflow: ellipsis; white-space: nowrap;
-    }
-    .kb-pin-btn {
-      flex: 0 0 auto;
-      border: none; background: transparent; cursor: pointer;
-      color: var(--muted); font-size: var(--text-lg); line-height: 1; padding: 0 2px;
-    }
-    .kb-pin-btn.on { color: var(--amber); }
-    .kb-pin-btn:hover { color: var(--amber); }
-    .kb-row-meta { font-size: var(--text-base); color: var(--muted); margin-top: 3px; }
-    .kb-row-preview {
-      margin-top: 5px;
-      font-size: var(--text-base); color: var(--muted); line-height: 1.5;
-      word-break: break-word;
-    }
-    .kb-entry-content {
-      white-space: pre-wrap; word-break: break-word;
-      font-family: var(--mono, monospace); font-size: var(--text-base);
-      color: var(--text); line-height: 1.55;
-      background: var(--surface2); border: 1px solid var(--border);
-      border-radius: var(--radius); padding: 12px; margin: 0;
-      max-height: 60vh; overflow-y: auto;
+    /* Notes & Knowledge tab: the embedded classic /inbox screen. The host div
+       is toggled via the [hidden] attribute — a plain div defaults to
+       display:block when shown and the UA [hidden] rule forces display:none
+       when hidden, so no extra host rule is needed. */
+    .slim-kn-embed {
+      display: block; width: 100%; height: 62vh; min-height: 360px;
+      border: 0; background: var(--bg);
     }
 
     .chat-thinking {

package/package.json

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