From 7b5d26921546e1f5364385325f8c098e88a20b30 Mon Sep 17 00:00:00 2001
From: Joseph Magly <1159087+jmagly@users.noreply.github.com>
Date: Sat, 1 Aug 2026 16:02:27 -0400
Subject: [PATCH] feat(ui): add local-agent disconnect control

---
 docs/index.html                               | 19 ++++++++++++++++++-
 .../local-api-hardening-static.test.ts        |  8 ++++++++
 2 files changed, 26 insertions(+), 1 deletion(-)

diff --git a/docs/index.html b/docs/index.html
index 7ce7abec..77ef5c5e 100755
--- a/docs/index.html
+++ b/docs/index.html
@@ -26365,7 +26365,8 @@ <h5 style="color: #ff8800; margin: 0 0 0.5rem 0;">${cs.category} <span style="co
       + pill(status, color)
       + (connected ? '<button onclick="setActiveLocalAgent(\''+a.id+'\')" title="Drive this agent as the LLM backbone for missions and analysis" style="padding:6px 11px;font-size:11px;font-weight:700;background:'+(isActive?'linear-gradient(135deg,#2fd6ff,#5b8cff)':'rgba(255,255,255,0.06)')+';border:1px solid '+(isActive?'transparent':'rgba(0,200,255,0.3)')+';border-radius:6px;color:'+(isActive?'#001624':'#bcd')+';cursor:pointer;white-space:nowrap;">'+(isActive?'★ Active':'☆ Use')+'</button>' : '')
       + (a.ready ? '<button onclick="pingOneLocalAgent(\''+a.id+'\')" title="send a real one-shot to verify it actually responds (spends a little quota)" style="padding:6px 11px;font-size:11px;font-weight:600;background:rgba(255,255,255,0.06);border:1px solid rgba(0,200,255,0.3);border-radius:6px;color:#bcd;cursor:pointer;white-space:nowrap;">Test</button>' : '')
-      + (a.ready ? '<button onclick="connectOneLocalAgent(\''+a.id+'\')" style="padding:6px 13px;font-size:11px;font-weight:700;background:'+(connected?'rgba(var(--brand-rgb),0.12)':'linear-gradient(135deg,#2fd6ff,#5b8cff)')+';border:'+(connected?'1px solid rgba(var(--brand-rgb),0.4)':'none')+';border-radius:6px;color:'+(connected?'var(--brand)':'#001624')+';cursor:pointer;white-space:nowrap;">'+(connected?'✓ Connected':'Connect')+'</button>' : '')
+      + (connected ? '<button onclick="disconnectLocalAgent(\''+a.id+'\')" title="Disconnect this agent and remove its saved reconnect preference" style="padding:6px 13px;font-size:11px;font-weight:700;background:rgba(255,80,80,0.1);border:1px solid rgba(255,80,80,0.4);border-radius:6px;color:#ff8a8a;cursor:pointer;white-space:nowrap;">Disconnect</button>'
+                   : (a.ready ? '<button onclick="connectOneLocalAgent(\''+a.id+'\')" style="padding:6px 13px;font-size:11px;font-weight:700;background:linear-gradient(135deg,#2fd6ff,#5b8cff);border:none;border-radius:6px;color:#001624;cursor:pointer;white-space:nowrap;">Connect</button>' : ''))
     + '</div>';
   }
 
@@ -26495,6 +26496,22 @@ <h5 style="color: #ff8800; margin: 0 0 0.5rem 0;">${cs.category} <span style="co
   };
   window.connectOneLocalAgent = function(id) { connect([id]); };
 
+  window.disconnectLocalAgent = async function(id) {
+    try {
+      const data = await api('/api/agents/local/disconnect', { id: id });
+      delete pingStatus[id];
+      const ids = Array.isArray(data.connected) ? data.connected : connectedIds.filter(function(x){ return x !== id; });
+      if (getActiveAgent() === id) setActiveAgentStored('');
+      persistConnected(ids);
+      render(lastDetected, ids);
+      if (window.toast) toast(id+' disconnected', 'success');
+      if (typeof window.refreshSystemStatus === 'function') window.refreshSystemStatus();
+      if (typeof updatePreflightChecklist === 'function') { try { updatePreflightChecklist(); } catch(e) {} }
+    } catch(e) {
+      if (window.toast) toast('Disconnect failed: '+e.message, 'error');
+    }
+  };
+
   // Pin (or unpin) which connected agent drives missions + analysis as the LLM backbone.
   // Clicking the active one again clears the pin (back to the auto codex→hermes→claude order).
   window.setActiveLocalAgent = function(id) {
diff --git a/src/__tests__/local-api-hardening-static.test.ts b/src/__tests__/local-api-hardening-static.test.ts
index e256b5ad..efee3b19 100644
--- a/src/__tests__/local-api-hardening-static.test.ts
+++ b/src/__tests__/local-api-hardening-static.test.ts
@@ -108,6 +108,14 @@ describe('local API authorization hardening invariants', () => {
     expect(uiSource).toMatch(/\['codex', 'mock', 'local', 'local-agent'\]/);
   });
 
+  it('Settings can disconnect a local agent without silently reconnecting it', () => {
+    expect(uiSource).toMatch(/onclick="disconnectLocalAgent\(\\'/);
+    expect(uiSource).toContain("api('/api/agents/local/disconnect', { id: id })");
+    expect(uiSource).toContain("if (getActiveAgent() === id) setActiveAgentStored('')");
+    expect(uiSource).toContain('persistConnected(ids)');
+    expect(uiSource).toContain('delete pingStatus[id]');
+  });
+
   it('Full Auto resumes with the exact approved receipt instead of minting another', () => {
     const start = uiSource.indexOf('async function generalFullAuto(');
     const end = uiSource.indexOf('/**\n         * REQUEST SITREP', start);