@yemi33/minions 0.1.1922 → 0.1.1923

package/bin/minions.js

@@ -766,7 +766,8 @@ if (!cmd || cmd === 'help' || cmd === '--help' || cmd === '-h') {
   void (async () => {
     const result = await waitForRestartHealth({
       minionsHome: MINIONS_HOME,
-      dashboardUrl: `http://127.0.0.1:${DASH_PORT}/api/health`,
+      dashboardPid: dashProc.pid,
+      dashboardPort: DASH_PORT,
     });
     if (!result.ok) {
       console.error(formatRestartHealthError(result));

package/engine/copilot-models.json

@@ -0,0 +1,5 @@
+{
+  "runtime": "copilot",
+  "models": null,
+  "cachedAt": "2026-05-13T23:13:36.239Z"
+}

package/engine/restart-health.js

@@ -43,6 +43,25 @@ function isProcessAlive(pid) {
   }
 }
 
+function isPortListening(port) {
+  const n = Number(port);
+  if (!Number.isInteger(n) || n <= 0) return false;
+  try {
+    if (process.platform === 'win32') {
+      const out = execSync(`netstat -ano -p TCP`, {
+        encoding: 'utf8', windowsHide: true, timeout: 3000, maxBuffer: 4 * 1024 * 1024,
+      });
+      const re = new RegExp(`\\s127\\.0\\.0\\.1:${n}\\s+\\S+\\s+LISTENING`, 'i');
+      const re6 = new RegExp(`\\s\\[::1?\\]:${n}\\s+\\S+\\s+LISTENING`, 'i');
+      return re.test(out) || re6.test(out);
+    }
+    const out = execSync(`ss -ltn 'sport = :${n}' 2>/dev/null || netstat -ltn 2>/dev/null`, {
+      encoding: 'utf8', timeout: 3000, shell: true,
+    });
+    return new RegExp(`[:.]${n}\\b`).test(out);
+  } catch { return false; }
+}
+
 function httpGetJson(url, timeoutMs = 1000) {
   return new Promise(resolve => {
     let settled = false;
@@ -81,9 +100,12 @@ function httpGetJson(url, timeoutMs = 1000) {
 async function checkRestartHealth(options = {}) {
   const {
     minionsHome,
-    dashboardUrl = 'http://127.0.0.1:7331/api/health',
+    dashboardUrl,
+    dashboardPid,
+    dashboardPort = 7331,
     readControl = readEngineControl,
     isProcessAlive: isAlive = isProcessAlive,
+    isPortListening: portCheck = isPortListening,
     httpGetJson: getJson = httpGetJson,
   } = options;
 
@@ -92,9 +114,42 @@ async function checkRestartHealth(options = {}) {
   const engineAlive = pid ? isAlive(pid) : false;
   const engineOk = control && control.state === 'running' && engineAlive;
 
-  const dashboard = await getJson(dashboardUrl, 3000);
-  const dashboardStatus = dashboard && dashboard.json && dashboard.json.status;
-  const dashboardOk = !!(dashboard && dashboard.ok && dashboardStatus === 'healthy');
+  // Two strategies for the dashboard check:
+  //   1) Process + port-listening (preferred from `minions restart` since
+  //      `bin/minions.js` knows the dashboard PID it just spawned). This avoids
+  //      a roundtrip through the dashboard's Node event loop, which can be
+  //      blocked for 15-25s during a cold `getStatus()` rebuild while still
+  //      being healthy.
+  //   2) HTTP `/api/health` — legacy path retained for tests that mock
+  //      httpGetJson + dashboardUrl, and for external probes that genuinely
+  //      want a response-level check.
+  // Strategy 1 wins when `dashboardPid` is supplied; otherwise we fall back to
+  // HTTP using the default URL or whatever the caller injected.
+  let dashboardOk;
+  let dashboardDetail;
+  let dashboardKind;
+  let dashboardSnapshot;
+  if (dashboardPid != null && !dashboardUrl) {
+    dashboardKind = 'process';
+    const dpid = normalizePid(dashboardPid);
+    const dashAlive = dpid ? isAlive(dpid) : false;
+    const portOpen = portCheck(dashboardPort);
+    dashboardOk = !!(dashAlive && portOpen);
+    dashboardDetail = `pid=${dpid || 'none'} alive=${dashAlive ? 'yes' : 'no'} port=${dashboardPort} listening=${portOpen ? 'yes' : 'no'}`;
+    dashboardSnapshot = { kind: 'process', pid: dpid, alive: dashAlive, port: dashboardPort, listening: portOpen };
+  } else {
+    dashboardKind = 'http';
+    const url = dashboardUrl || `http://127.0.0.1:${dashboardPort}/api/health`;
+    const dashboard = await getJson(url, 3000);
+    const dashboardStatus = dashboard && dashboard.json && dashboard.json.status;
+    dashboardOk = !!(dashboard && dashboard.ok && dashboardStatus === 'healthy');
+    dashboardDetail = dashboard && dashboard.error
+      ? dashboard.error.message
+      : dashboard && dashboard.statusCode
+        ? `HTTP ${dashboard.statusCode}${dashboardStatus ? `, status=${dashboardStatus}` : ''}`
+        : 'no response';
+    dashboardSnapshot = { kind: 'http', url, ok: !!(dashboard && dashboard.ok), statusCode: dashboard && dashboard.statusCode, status: dashboardStatus };
+  }
 
   const errors = [];
   if (!engineOk) {
@@ -103,18 +158,16 @@ async function checkRestartHealth(options = {}) {
     errors.push(`Engine is not healthy (state=${state}, pid=${pidLabel}, alive=${engineAlive ? 'yes' : 'no'})`);
   }
   if (!dashboardOk) {
-    const detail = dashboard && dashboard.error
-      ? dashboard.error.message
-      : dashboard && dashboard.statusCode
-        ? `HTTP ${dashboard.statusCode}${dashboardStatus ? `, status=${dashboardStatus}` : ''}`
-        : 'no response';
-    errors.push(`Dashboard failed health check at ${dashboardUrl} (${detail})`);
+    const where = dashboardKind === 'http'
+      ? (dashboardUrl || `http://127.0.0.1:${dashboardPort}/api/health`)
+      : `dashboard pid=${normalizePid(dashboardPid) || 'none'} port=${dashboardPort}`;
+    errors.push(`Dashboard failed health check at ${where} (${dashboardDetail})`);
   }
 
   return {
     ok: engineOk && dashboardOk,
     engine: { state: control && control.state, pid, alive: engineAlive },
-    dashboard: { url: dashboardUrl, ok: !!(dashboard && dashboard.ok), statusCode: dashboard && dashboard.statusCode, status: dashboardStatus },
+    dashboard: dashboardSnapshot,
     errors,
   };
 }
@@ -163,6 +216,7 @@ module.exports = {
   _private: {
     httpGetJson,
     isProcessAlive,
+    isPortListening,
     readEngineControl,
     normalizePid,
   },

package/package.json

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