@yemi33/minions 0.1.1825 → 0.1.1826

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 0.1.1826 (2026-05-09)
+
+### Fixes
+- always open dashboard if no live tab reconnects after restart
+
 ## 0.1.1825 (2026-05-09)
 
 ### Fixes

package/bin/minions.js

@@ -86,6 +86,43 @@ function hasRecentDashboardBrowserTab(minionsHome, now = Date.now()) {
   } catch { return false; }
 }
 
+// Clear the browser-presence cache. Pre-restart beacons can outlive the
+// browser window that produced them (closed Edge, locked-screen RDP session,
+// stale heuristic timer) and falsely tell `restart` to skip the auto-open. We
+// wipe it during restart so the post-restart probe only counts beacons that
+// arrive AFTER the new dashboard is up — i.e. from a still-living tab.
+function _clearDashboardBrowserState(minionsHome) {
+  try {
+    const fp = path.join(minionsHome, 'engine', 'dashboard-browser.json');
+    fs.writeFileSync(fp, JSON.stringify({ tabs: {}, updatedAt: new Date().toISOString() }));
+  } catch {}
+}
+
+async function _waitForBrowserReconnect(minionsHome, { afterMs, timeoutMs = 5000, pollMs = 500 } = {}) {
+  const start = Date.now();
+  while (Date.now() - start < timeoutMs) {
+    try {
+      const fp = path.join(minionsHome, 'engine', 'dashboard-browser.json');
+      const state = JSON.parse(fs.readFileSync(fp, 'utf8'));
+      const tabs = state && state.tabs ? Object.values(state.tabs) : [];
+      if (tabs.some(t => Number(t && t.lastSeen) > afterMs)) return true;
+    } catch { /* file may be temporarily missing during write */ }
+    await new Promise(r => setTimeout(r, pollMs));
+  }
+  return false;
+}
+
+function _openInBrowser(url) {
+  try {
+    if (process.platform === 'win32') execSync(`start "" "${url}"`, { stdio: 'ignore', windowsHide: true });
+    else if (process.platform === 'darwin') execSync(`open "${url}"`, { stdio: 'ignore' });
+    else execSync(`xdg-open "${url}"`, { stdio: 'ignore' });
+  } catch (e) {
+    console.log(`  Could not auto-open browser: ${e.message}`);
+    console.log(`  Please open ${url} manually.`);
+  }
+}
+
 /**
  * Read the engine's recorded PID from engine/control.json. Returns null if
  * the file is missing/corrupt or the PID isn't a positive integer.
@@ -411,12 +448,12 @@ function init() {
   if (isUpgrade && skipStart) return;
 
   // Auto-start on fresh install; direct force-upgrade restarts automatically.
-  // Probe before kill so we suppress browser auto-open only when a browser tab
-  // was recently polling the dashboard. A bare/orphan dashboard process on the
-  // port is not enough; cold starts should still open the UI.
   const dashWasUp = isPortListening(DASH_PORT);
-  const suppressDashboardOpen = !forceOpen && dashWasUp && hasRecentDashboardBrowserTab(MINIONS_HOME);
+  const restartStartMs = Date.now();
   if (isUpgrade) {
+    // Clear stale beacons so the post-restart probe only counts tabs that
+    // reconnect to the NEW dashboard.
+    _clearDashboardBrowserState(MINIONS_HOME);
     try { execSync(`node "${path.join(MINIONS_HOME, 'engine.js')}" stop`, { stdio: 'ignore', cwd: MINIONS_HOME, timeout: 10000, windowsHide: true }); } catch {}
     // Free the dashboard port too — without this the new dashboard EADDRINUSE-dies
     // silently and the user keeps running stale code from the old dashboard process.
@@ -431,10 +468,28 @@ function init() {
   engineProc.unref();
   console.log(`  Engine started (PID: ${engineProc.pid})`);
 
-  const dashProc = spawnDashboard(suppressDashboardOpen);
+  // Always suppress dashboard's self-open — we decide here after the health
+  // check based on whether an existing browser tab actually reconnects.
+  const dashProc = spawnDashboard(true);
   console.log(`  Dashboard started (PID: ${dashProc.pid})`);
   console.log(`  Dashboard: http://localhost:${DASH_PORT}`);
 
+  void (async () => {
+    let shouldOpen = forceOpen || !dashWasUp;
+    if (!shouldOpen) {
+      // Hot upgrade: give an existing browser tab up to 5s to reconnect via
+      // its 4s auto-refresh poll. If nothing beacons, the tab is gone — open.
+      const reconnected = await _waitForBrowserReconnect(MINIONS_HOME, {
+        afterMs: restartStartMs, timeoutMs: 5000,
+      });
+      shouldOpen = !reconnected;
+    }
+    if (shouldOpen) {
+      console.log(`  Opening dashboard in browser...`);
+      _openInBrowser(`http://localhost:${DASH_PORT}`);
+    }
+  })();
+
   // Next steps guidance
   console.log(`
   Next steps:
@@ -689,11 +744,11 @@ if (!cmd || cmd === 'help' || cmd === '--help' || cmd === '-h') {
   // `--cli` / `--model` flags forward to `engine.js start` so the runtime
   // fleet flips before the daemon spawns (P-6b3f9c2e AC: works on restart).
   ensureInstalled();
-  // Probe before kill so we suppress browser auto-open only when a browser tab
-  // was recently polling the dashboard. A bare/orphan dashboard process on the
-  // port is not enough; cold starts should still open the UI.
   const dashWasUp = isPortListening(DASH_PORT);
-  const suppressDashboardOpen = !forceOpen && dashWasUp && hasRecentDashboardBrowserTab(MINIONS_HOME);
+  // Mark the restart boundary so post-restart beacon timestamps are unambiguous,
+  // and clear stale beacons (browsers that closed without notifying).
+  const restartStartMs = Date.now();
+  _clearDashboardBrowserState(MINIONS_HOME);
   // Layered kill — each step is best-effort, layered so the next still runs if
   // one fails. Goal: the old engine is gone before we spawn a new one, even if
   // PowerShell is unavailable, the engine is hung, or its cmdline doesn't match.
@@ -712,7 +767,9 @@ if (!cmd || cmd === 'help' || cmd === '--help' || cmd === '-h') {
   });
   engineProc.unref();
   console.log(`\n  Engine started (PID: ${engineProc.pid})`);
-  const dashProc = spawnDashboard(suppressDashboardOpen);
+  // Always tell the dashboard to skip its own auto-open — we'll decide here
+  // after observing whether an existing browser tab reconnects.
+  const dashProc = spawnDashboard(true);
   console.log(`  Dashboard started (PID: ${dashProc.pid})`);
   console.log(`  Dashboard: http://localhost:${DASH_PORT}`);
   console.log('  Verifying restart health...');
@@ -725,7 +782,20 @@ if (!cmd || cmd === 'help' || cmd === '--help' || cmd === '-h') {
       console.error(formatRestartHealthError(result));
       process.exit(1);
     }
-    console.log(`  Restart verified: engine PID ${result.engine.pid}; dashboard healthy.\n`);
+    console.log(`  Restart verified: engine PID ${result.engine.pid}; dashboard healthy.`);
+
+    let shouldOpen = forceOpen || !dashWasUp;
+    if (!shouldOpen) {
+      const reconnected = await _waitForBrowserReconnect(MINIONS_HOME, {
+        afterMs: restartStartMs, timeoutMs: 5000,
+      });
+      shouldOpen = !reconnected;
+    }
+    if (shouldOpen) {
+      console.log(`  Opening dashboard in browser...`);
+      _openInBrowser(`http://localhost:${DASH_PORT}`);
+    }
+    console.log('');
   })().catch(err => {
     console.error(`\n  ERROR: Restart verification failed: ${err.message}\n`);
     process.exit(1);

package/engine/copilot-models.json

@@ -1,5 +1,5 @@
 {
   "runtime": "copilot",
   "models": null,
-  "cachedAt": "2026-05-09T20:04:56.707Z"
+  "cachedAt": "2026-05-09T22:56:52.587Z"
 }

package/package.json

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