amalgm 0.1.251 → 0.1.252

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "amalgm",
-  "version": "0.1.251",
+  "version": "0.1.252",
   "description": "Amalgm local computer runtime: login, MCP, chat, events, previews, and tunnels.",
   "license": "UNLICENSED",
   "private": false,

package/runtime/scripts/amalgm-mcp/server/routes/health.js

@@ -3,36 +3,10 @@
 const { activeAgentConversations } = require('../../agents/talk');
 
 // /healthz is a liveness signal: the parent supervisor kills this process
-// after three slow answers, so the handler must never touch disk. Counts are
-// informational — serve a cached snapshot and refresh it off the request path.
-const COUNTS_TTL_MS = 30_000;
-let cachedCounts = { automations: 0, enabledAutomations: 0, apps: 0, customAgents: 0 };
-let countsRefreshedAt = 0;
-let countsRefreshing = false;
-
-function refreshCountsSoon() {
-  if (countsRefreshing || Date.now() - countsRefreshedAt < COUNTS_TTL_MS) return;
-  countsRefreshing = true;
-  setImmediate(() => {
-    try {
-      const { loadAgents } = require('../../agents/store');
-      const { loadApps } = require('../../apps/store');
-      const { listAutomations } = require('../../automations/store');
-      const automations = listAutomations();
-      cachedCounts = {
-        automations: automations.length,
-        enabledAutomations: automations.filter((automation) => automation.enabled !== false).length,
-        apps: loadApps().apps.length,
-        customAgents: loadAgents().agents.length,
-      };
-      countsRefreshedAt = Date.now();
-    } catch {
-      // Counts stay stale; liveness must not depend on store reads succeeding.
-    } finally {
-      countsRefreshing = false;
-    }
-  });
-}
+// after three slow answers. It therefore performs no I/O and starts no
+// follow-up work. Operational counts belong on diagnostic routes: allowing a
+// liveness probe to inspect the  user's state store made a large, healthy
+// installation look dead by blocking its own event loop.
 
 async function handleHealthRoutes(ctx) {
   if (ctx.pathname !== '/healthz') return false;
@@ -43,10 +17,8 @@ async function handleHealthRoutes(ctx) {
     runtime_label: process.env.AMALGM_RUNTIME_LABEL || 'main',
     runtime_instance_id: process.env.AMALGM_RUNTIME_INSTANCE_ID || '',
     pid: process.pid,
-    ...cachedCounts,
     activeAgentConversations: activeAgentConversations.size,
   });
-  refreshCountsSoon();
   return true;
 }
 

package/runtime/scripts/amalgm-mcp/tests/health.test.js

@@ -0,0 +1,20 @@
+'use strict';
+
+const test = require('node:test');
+const assert = require('node:assert/strict');
+
+test('/healthz is an I/O-free liveness response', async () => {
+  const { handleHealthRoutes } = require('../server/routes/health');
+  const replies = [];
+  const handled = await handleHealthRoutes({
+    pathname: '/healthz',
+    sendJson: (status, body) => replies.push({ status, body }),
+  });
+
+  assert.equal(handled, true);
+  assert.equal(replies.length, 1);
+  assert.equal(replies[0].status, 200);
+  assert.equal(replies[0].body.status, 'ok');
+  assert.equal(Object.hasOwn(replies[0].body, 'automations'), false);
+  assert.equal(Object.hasOwn(replies[0].body, 'apps'), false);
+});