@yemi33/minions 0.1.2011 → 0.1.2012

package/dashboard.js

@@ -1919,7 +1919,7 @@ async function _handleStatusRequest(req, res) {
   }
 }
 
-// Periodic push for engine-driven changes (dispatch.json, control.json) that bypass invalidateStatusCache
+// Periodic push for engine-driven changes (dispatch.json, work-items.json, pull-requests.json) that bypass invalidateStatusCache
 setInterval(() => {
   if (_statusStreamClients.size === 0) return;
   const data = getStatusJson();
@@ -8098,6 +8098,11 @@ What would you like to discuss or change? When you're happy, say "approve" and I
   async function handleEngineRestart(req, res) {
     try {
       const newPid = restartEngine();
+      // control.json is no longer in the mtime-tracked list (W-mpg8aapw001d7e0c)
+      // because heartbeat noise dominated legitimate state changes and defeated
+      // the /api/status ETag short-circuit. Explicit invalidation here keeps the
+      // restart visible on the next poll without re-tracking control.json.
+      invalidateStatusCache();
       return jsonReply(res, 200, { ok: true, pid: newPid });
     } catch (e) { return jsonReply(res, e.statusCode || 500, { error: e.message }); }
   }

package/engine/queries.js

@@ -1822,6 +1822,19 @@ function resetProjectGitStatusCache() {
  *      automatically.
  *
  * Files intentionally NOT tracked:
+ *   - `engine/log.json` — append-only log written many times per tick
+ *     (PR polls, dispatch transitions, heartbeat). Its mtime advances
+ *     every few seconds in steady state, which by itself was enough to
+ *     bust the ETag cache on every poll and reduce the /api/status 304
+ *     short-circuit to 0% hit rate (W-mpg8aapw001d7e0c). Log tail is
+ *     surfaced by `getEngineLog()` and accepts the 10 s `FAST_STATE_TTL`
+ *     backstop — log entries are not UX-blocking.
+ *   - `engine/control.json` — heartbeat writes advance mtime every tick
+ *     (~60 s) plus on any control-side mutation, dominating legitimate
+ *     state changes. Engine state (running/paused/stopped) is surfaced
+ *     by `getEngineState()`; the dashboard-side mutating handlers
+ *     (`handleEngineRestart` etc.) call `invalidateStatusCache()`
+ *     directly, and the 10 s TTL covers CLI-driven control-file edits.
  *   - `engine/state.json` — surfaced via `getEngineState()` but changes
  *     only on engine startup / reconcile. Negligible benefit.
  *   - `engine/cooldowns.json`, `engine/pr-links.json`, `engine/pending-
@@ -1841,11 +1854,10 @@ function resetProjectGitStatusCache() {
 function getStatusFastStateMtimePaths(config) {
   const projects = getProjects(config || getConfig());
   const files = [
-    // Engine-level state surfaced by getDispatchQueue / inline engine block /
-    // getEngineLog / getMetrics.
+    // Engine-level state surfaced by getDispatchQueue / getMetrics.
+    // `control.json` and `log.json` are intentionally omitted — see the
+    // "Files intentionally NOT tracked" section above (W-mpg8aapw001d7e0c).
     DISPATCH_PATH,
-    CONTROL_PATH,
-    LOG_PATH,
     path.join(ENGINE_DIR, 'metrics.json'),
     // Watches surfaced by watchesMod.getWatches() (W-mpftp7na000td0f4 fix).
     path.join(ENGINE_DIR, 'watches.json'),

package/package.json

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