@yemi33/minions 0.1.1929 → 0.1.1930

package/dashboard/js/command-center.js

@@ -182,24 +182,45 @@ async function _ccDashboardHealth() {
 // Triggered by the CC "Restart Minions" recovery button when a stale dashboard
 // connection is killing CC streams with "Failed to fetch". Spawns the same
 // `minions restart` flow as the CLI command (kills + respawns engine AND
-// dashboard). The browser auto-reloads via refresh.js when it sees the new
-// dashboardStartedAt; if the POST itself fails (dashboard truly unreachable),
-// fall back to a plain location.reload() so behavior is never worse than before.
+// dashboard). Then polls /api/health until the new dashboard is online before
+// reloading — fixed timers either fired too early (reload hits dead port) or
+// fired after the fetch threw (dashboard killed mid-response → 500ms reload
+// to nothing).
 async function ccRestartMinions(btn) {
   if (btn) { try { btn.disabled = true; btn.textContent = 'Restarting...'; } catch {} }
+  // Fire and forget the POST. We do NOT await it — the dashboard often kills
+  // its own process before the response is flushed, so the fetch throws even
+  // though the restart child is happily running. Whatever the POST does, the
+  // next step (wait-for-healthy) is the truth.
   try {
-    var res = await fetch('/api/dashboard/restart', { method: 'POST', headers: { 'Content-Type': 'application/json' } });
-    if (!res.ok) throw new Error('HTTP ' + res.status);
-    if (btn) { try { btn.textContent = 'Restarting Minions — page will reload...'; } catch {} }
-    // The dashboard process is about to be killed. refresh.js's status poll
-    // will detect the new dashboardStartedAt and call location.reload() once
-    // the new dashboard is online; this fallback handles the case where the
-    // poll hasn't fired (e.g., user closed the browser tab and reopened).
-    setTimeout(function() { try { location.reload(); } catch {} }, 8000);
-  } catch (e) {
-    if (btn) { try { btn.textContent = 'Restart failed — reloading...'; } catch {} }
-    setTimeout(function() { try { location.reload(); } catch {} }, 500);
+    fetch('/api/dashboard/restart', { method: 'POST', headers: { 'Content-Type': 'application/json' } })
+      .catch(function() { /* dashboard process likely killed mid-response — expected */ });
+  } catch { /* network layer threw before fetch even queued — also expected */ }
+  if (btn) { try { btn.textContent = 'Restarting Minions — waiting for new dashboard...'; } catch {} }
+  var startedAt = Date.now();
+  var DEADLINE_MS = 60000;
+  var INTERVAL_MS = 500;
+  function pollHealth() {
+    fetch('/api/health', { cache: 'no-store' }).then(function(res) {
+      if (res && res.ok) {
+        if (btn) { try { btn.textContent = 'Dashboard online — reloading...'; } catch {} }
+        try { location.reload(); } catch {}
+        return;
+      }
+      throw new Error('not ok');
+    }).catch(function() {
+      if (Date.now() - startedAt > DEADLINE_MS) {
+        if (btn) { try { btn.textContent = 'Restart timed out — reloading...'; } catch {} }
+        try { location.reload(); } catch {}
+        return;
+      }
+      setTimeout(pollHealth, INTERVAL_MS);
+    });
   }
+  // Don't poll immediately — give the restart child a moment to actually kill
+  // the old dashboard. If we polled at t=0 we'd hit the OLD (dying) dashboard
+  // and prematurely reload before the new one is up.
+  setTimeout(pollHealth, 2000);
 }
 
 function _ccIsReconnectableStreamError(err) {

package/engine/copilot-models.json

@@ -1,5 +1,5 @@
 {
   "runtime": "copilot",
   "models": null,
-  "cachedAt": "2026-05-14T02:50:03.274Z"
+  "cachedAt": "2026-05-14T02:53:42.873Z"
 }

package/package.json

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