@remnic/core 10.0.40 → 10.0.42

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remnic/core",
-  "version": "10.0.40",
+  "version": "10.0.42",
   "description": "Framework-agnostic Remnic memory engine — orchestrator, storage, extraction, search, trust zones",
   "type": "module",
   "main": "dist/index.js",
@@ -3317,7 +3317,7 @@
     "meilisearch": "^0.46.0",
     "node-gyp": "^12.3.0",
     "openai": "^6.0.0",
-    "undici": "^6.28.1",
+    "undici": "^6.29.1",
     "zod": "^3.25.28",
     "zod-to-json-schema": "^3.25.2"
   },
@@ -3347,7 +3347,7 @@
     "core"
   ],
   "peerDependencies": {
-    "@remnic/coding-graph": "^10.0.40"
+    "@remnic/coding-graph": "^10.0.42"
   },
   "peerDependenciesMeta": {
     "@remnic/coding-graph": {

package/src/access-service.ts

@@ -603,6 +603,7 @@ export interface EngramAccessTimelineResponse {
 }
 
 export interface EngramAccessEntitySummary {
+  id: string;
   name: string;
   type: string;
   updated: string;
@@ -3534,6 +3535,7 @@ export class EngramAccessService extends SupportPassportAccessServiceBase {
         if (!haystack.includes(query)) continue;
       }
       entities.push({
+        id: name,
         name: entity.name,
         type: entity.type,
         updated: entity.updated,

package/src/runtime/daemon-health.test.ts

@@ -0,0 +1,85 @@
+import assert from "node:assert/strict";
+import { once } from "node:events";
+import test from "node:test";
+import { Worker } from "node:worker_threads";
+import { checkDaemonHealth } from "./daemon-health.js";
+
+test("checkDaemonHealth probes /live without waiting on diagnostic /health", { timeout: 30_000 }, async (t) => {
+  // The synchronous probe blocks this thread, so HTTP must run on a worker.
+  const worker = new Worker(`
+    const { parentPort } = require("node:worker_threads");
+    const { createServer } = require("node:http");
+    const paths = [];
+    const server = createServer((request, response) => {
+      paths.push(request.url);
+      if (request.url === "/engram/v1/health") return;
+      response.writeHead(request.headers.authorization === "Bearer test-token" ? 200 : 401).end();
+    });
+    server.listen(0, "127.0.0.1", () => parentPort.postMessage(server.address().port));
+    parentPort.on("message", () => parentPort.postMessage(paths));
+  `, { eval: true });
+  t.after(() => worker.terminate());
+  let port = 0;
+  [port] = await once(worker, "message");
+  assert.ok(Number.isInteger(port) && port > 0 && port <= 65535);
+  assert.equal(checkDaemonHealth("127.0.0.1", port, "test-token"), true);
+  assert.equal(checkDaemonHealth("127.0.0.1", port, "wrong-token"), false);
+  const paths = once(worker, "message");
+  worker.postMessage("paths");
+  assert.deepEqual((await paths)[0], [
+    "/engram/v1/live",
+    "/engram/v1/live",
+    "/engram/v1/live",
+  ]);
+  await worker.terminate();
+  assert.equal(checkDaemonHealth("127.0.0.1", port, "test-token"), false);
+});
+
+test("checkDaemonHealth retries legacy /health only on 404 (#3257)", { timeout: 30_000 }, async (t) => {
+  const worker = new Worker(`
+    const { parentPort } = require("node:worker_threads");
+    const { createServer } = require("node:http");
+    const paths = [];
+    const server = createServer((request, response) => {
+      paths.push(request.url);
+      if (request.url === "/engram/v1/live") return response.writeHead(404).end();
+      response.writeHead(200).end();
+    });
+    server.listen(0, "127.0.0.1", () => parentPort.postMessage(server.address().port));
+    parentPort.on("message", () => parentPort.postMessage(paths));
+  `, { eval: true });
+  t.after(() => worker.terminate());
+  let port = 0;
+  [port] = await once(worker, "message");
+  assert.equal(checkDaemonHealth("127.0.0.1", port, "test-token"), true);
+  const paths = once(worker, "message");
+  worker.postMessage("paths");
+  assert.deepEqual((await paths)[0], [
+    "/engram/v1/live",
+    "/engram/v1/health",
+  ]);
+  await worker.terminate();
+});
+
+test("checkDaemonHealth treats 503 as final with no legacy retry (#3257)", { timeout: 30_000 }, async (t) => {
+  const worker = new Worker(`
+    const { parentPort } = require("node:worker_threads");
+    const { createServer } = require("node:http");
+    const paths = [];
+    const server = createServer((request, response) => {
+      paths.push(request.url);
+      if (request.url === "/engram/v1/live") return response.writeHead(503).end();
+      response.writeHead(200).end();
+    });
+    server.listen(0, "127.0.0.1", () => parentPort.postMessage(server.address().port));
+    parentPort.on("message", () => parentPort.postMessage(paths));
+  `, { eval: true });
+  t.after(() => worker.terminate());
+  let port = 0;
+  [port] = await once(worker, "message");
+  assert.equal(checkDaemonHealth("127.0.0.1", port, "test-token"), false);
+  const paths = once(worker, "message");
+  worker.postMessage("paths");
+  assert.deepEqual((await paths)[0], ["/engram/v1/live"]);
+  await worker.terminate();
+});

package/src/runtime/daemon-health.ts

@@ -16,6 +16,9 @@ export function checkDaemonHealth(
     if (!Number.isFinite(safePort) || safePort < 1 || safePort > 65535) return false;
 
     const bareHost = host.startsWith("[") && host.endsWith("]") ? host.slice(1, -1) : host;
+    // One 3 s aggregate budget: /engram/v1/live, and only a 404 (daemon older
+    // than the live route) retries the legacy diagnostic path with the
+    // remaining time. 401/503/other statuses are final answers, never re-probed.
     const script = [
       "const env = process['env'];",
       "const protocol = env.REMNIC_HEALTH_PROTOCOL;",
@@ -23,13 +26,23 @@ export function checkDaemonHealth(
       "const transport = require(protocol);",
       "const headers = {};",
       "if (env.REMNIC_HEALTH_TOKEN) headers['authorization'] = 'Bearer ' + env.REMNIC_HEALTH_TOKEN;",
-      "const req = transport.get({",
-      "  host: env.REMNIC_HEALTH_HOST,",
-      "  port: parseInt(env.REMNIC_HEALTH_PORT, 10),",
-      "  path: '/engram/v1/health', headers, timeout: 3000,",
-      "}, (res) => { process.exit(res.statusCode === 200 ? 0 : res.statusCode === 401 ? 2 : 1); });",
-      "req.on('error', () => process.exit(1));",
-      "req.on('timeout', () => { req.destroy(); process.exit(1); });",
+      "const exitFor = (status) => process.exit(status === 200 ? 0 : status === 401 ? 2 : 1);",
+      "const attempt = (path, timeoutMs, onStatus) => {",
+      "  const req = transport.get({",
+      "    host: env.REMNIC_HEALTH_HOST,",
+      "    port: parseInt(env.REMNIC_HEALTH_PORT, 10),",
+      "    path, headers, timeout: timeoutMs,",
+      "  }, (res) => { res.resume(); onStatus(res.statusCode); });",
+      "  req.on('error', () => process.exit(1));",
+      "  req.on('timeout', () => { req.destroy(); process.exit(1); });",
+      "};",
+      "const deadline = Date.now() + 3000;",
+      "attempt('/engram/v1/live', 3000, (status) => {",
+      "  if (status !== 404) exitFor(status);",
+      "  const remaining = deadline - Date.now();",
+      "  if (remaining <= 0) process.exit(1);",
+      "  attempt('/engram/v1/health', remaining, exitFor);",
+      "});",
     ].join("\n");
     const env: NodeJS.ProcessEnv = mergeEnv({
       REMNIC_HEALTH_HOST: bareHost,