@remnic/plugin-claude-code 10.0.39 → 10.0.41

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "remnic",
   "description": "Universal memory for AI agents — automatic recall, observation, and cross-agent knowledge sharing",
-  "version": "10.0.39",
+  "version": "10.0.41",
   "author": {
     "name": "Joshua Warren"
   },

package/hooks/bin/remnic-cc-hook.test.cjs

@@ -122,7 +122,7 @@ function cursorPath(home, sessionId) {
 test("session-start: healthy server returns recall context with codingContext cleared outside a repo", async () => {
   const home = mkHome();
   const { server, port, calls } = await startServer((req, res) => {
-    if (req.url === "/engram/v1/health") return res.writeHead(200).end("ok");
+    if (req.url === "/engram/v1/live") return res.writeHead(200).end("ok");
     if (req.url === "/engram/v1/recall") {
       return res.writeHead(200, { "Content-Type": "application/json" }).end(
         JSON.stringify({ context: "remembered preferences", count: 3, mode: "auto" }),
@@ -154,7 +154,7 @@ test("session-start: health probe carries the bearer token (auth-gated daemons)"
   const home = mkHome();
   let healthAuth = "unset";
   const { server, port } = await startServer((req, res) => {
-    if (req.url === "/engram/v1/health") {
+    if (req.url === "/engram/v1/live") {
       healthAuth = req.headers.authorization || null;
       return res.writeHead(200).end("ok");
     }
@@ -176,10 +176,85 @@ test("session-start: health probe carries the bearer token (auth-gated daemons)"
   }
 });
 
-test("session-start: REMNIC_DAEMON_URL routes recall to the remote base URL (#2483)", async () => {
+test("session-start: hung diagnostic /health does not stall the live probe (#3257)", async () => {
+  const home = mkHome();
+  const { server, port, calls } = await startServer((req, res) => {
+    // The expensive diagnostic route never answers; only /live is healthy.
+    if (req.url === "/engram/v1/health") return;
+    if (req.url === "/engram/v1/live") return res.writeHead(200).end("ok");
+    if (req.url === "/engram/v1/recall") {
+      return res.writeHead(200, { "Content-Type": "application/json" }).end(
+        JSON.stringify({ context: "live recall", count: 1, mode: "auto" }),
+      );
+    }
+    res.writeHead(404).end();
+  });
+  try {
+    const { json } = await runHook("session-start", { session_id: "s1", cwd: home }, { port, home });
+    assert.equal(json.continue, true);
+    assert.match(json.hookSpecificOutput.additionalContext, /live recall/);
+    assert.ok(calls.some((c) => c.url === "/engram/v1/recall"), "recall reached the daemon");
+  } finally {
+    server.close();
+    fs.rmSync(home, { recursive: true, force: true });
+  }
+});
+
+test("session-start: 404 live falls back to legacy health within one budget", async () => {
   const home = mkHome();
   const { server, port, calls } = await startServer((req, res) => {
+    if (req.url === "/engram/v1/live") return res.writeHead(404).end();
     if (req.url === "/engram/v1/health") return res.writeHead(200).end("ok");
+    if (req.url === "/engram/v1/recall") {
+      return res.writeHead(200, { "Content-Type": "application/json" }).end(
+        JSON.stringify({ context: "legacy recall", count: 1, mode: "auto" }),
+      );
+    }
+    res.writeHead(404).end();
+  });
+  try {
+    const { json } = await runHook("session-start", { session_id: "s1", cwd: home }, { port, home });
+    assert.equal(json.continue, true);
+    assert.match(json.hookSpecificOutput.additionalContext, /legacy recall/);
+    const probeUrls = calls.filter((c) => c.url === "/engram/v1/live" || c.url === "/engram/v1/health").map((c) => c.url);
+    assert.deepEqual(probeUrls, ["/engram/v1/live", "/engram/v1/health"]);
+  } finally {
+    server.close();
+    fs.rmSync(home, { recursive: true, force: true });
+  }
+});
+
+test("session-start: 503 live is a final answer with no legacy retry", async () => {
+  const home = mkHome();
+  const { server, port, calls } = await startServer((req, res) => {
+    if (req.url === "/engram/v1/live") return res.writeHead(503).end();
+    // A 503 must NOT be re-probed on the legacy route, even when it would pass.
+    if (req.url === "/engram/v1/health") return res.writeHead(200).end("ok");
+    if (req.url === "/engram/v1/recall") {
+      return res.writeHead(200, { "Content-Type": "application/json" }).end(
+        JSON.stringify({ context: "ctx", count: 1, mode: "auto" }),
+      );
+    }
+    res.writeHead(404).end();
+  });
+  try {
+    const { json } = await runHook("session-start", { session_id: "s1", cwd: home }, { port, home });
+    assert.match(json.hookSpecificOutput.additionalContext, /daemon not running/);
+    assert.ok(!calls.some((c) => c.url === "/engram/v1/recall"), "recall is skipped");
+    assert.deepEqual(
+      calls.filter((c) => c.url.startsWith("/engram/v1/")).map((c) => c.url),
+      ["/engram/v1/live", "/engram/v1/live"],
+    );
+  } finally {
+    server.close();
+    fs.rmSync(home, { recursive: true, force: true });
+  }
+});
+
+test("session-start: REMNIC_DAEMON_URL routes recall to the remote base URL (#2483)", async () => {
+  const home = mkHome();
+  const { server, port, calls } = await startServer((req, res) => {
+    if (req.url === "/engram/v1/live") return res.writeHead(200).end("ok");
     if (req.url === "/engram/v1/recall") {
       return res.writeHead(200, { "Content-Type": "application/json" }).end(
         JSON.stringify({ context: "remote recall", count: 1, mode: "auto" }),
@@ -217,7 +292,7 @@ test("session-start: REMNIC_DAEMON_URL routes recall to the remote base URL (#24
 test("recall body targets REMNIC_NAMESPACE when set, and omits namespace when unset", async () => {
   const mk = () =>
     startServer((req, res) => {
-      if (req.url === "/engram/v1/health") return res.writeHead(200).end("ok");
+      if (req.url === "/engram/v1/live") return res.writeHead(200).end("ok");
       if (req.url === "/engram/v1/recall") {
         return res
           .writeHead(200, { "Content-Type": "application/json" })
@@ -274,7 +349,7 @@ test("namespace targeting: ENGRAM_NAMESPACE fallback (recall) and observe body b
   {
     const home = mkHome();
     const { server, port, calls } = await startServer((req, res) => {
-      if (req.url === "/engram/v1/health") return res.writeHead(200).end("ok");
+      if (req.url === "/engram/v1/live") return res.writeHead(200).end("ok");
       if (req.url === "/engram/v1/recall") {
         return res
           .writeHead(200, { "Content-Type": "application/json" })
@@ -325,7 +400,7 @@ test("session-start: falls back to minimal mode when full recall fails", async (
   const home = mkHome();
   let recallHits = 0;
   const { server, port, calls } = await startServer((req, res) => {
-    if (req.url === "/engram/v1/health") return res.writeHead(200).end("ok");
+    if (req.url === "/engram/v1/live") return res.writeHead(200).end("ok");
     if (req.url === "/engram/v1/recall") {
       recallHits += 1;
       if (recallHits === 1) return res.writeHead(500).end("boom");
@@ -349,7 +424,7 @@ test("session-start: falls back to minimal mode when full recall fails", async (
 test("session-start: no token → claude-code install hint, no recall call", async () => {
   const home = mkHome();
   const { server, port, calls } = await startServer((req, res) => {
-    if (req.url === "/engram/v1/health") return res.writeHead(200).end("ok");
+    if (req.url === "/engram/v1/live") return res.writeHead(200).end("ok");
     res.writeHead(200).end("{}");
   });
   try {
@@ -617,7 +692,7 @@ test("token resolution: claude-code connector wins over openclaw in tokens.json"
     }),
   );
   const { server, port, calls } = await startServer((req, res) => {
-    if (req.url === "/engram/v1/health") return res.writeHead(200).end("ok");
+    if (req.url === "/engram/v1/live") return res.writeHead(200).end("ok");
     if (req.url === "/engram/v1/recall") return res.writeHead(200).end("{}");
     res.writeHead(200).end("{}");
   });
@@ -644,7 +719,7 @@ test("token resolution: legacy ~/.engram/tokens.json is the read fallback", asyn
     JSON.stringify({ tokens: [{ connector: "openclaw", token: "legacy-tok" }] }),
   );
   const { server, port, calls } = await startServer((req, res) => {
-    if (req.url === "/engram/v1/health") return res.writeHead(200).end("ok");
+    if (req.url === "/engram/v1/live") return res.writeHead(200).end("ok");
     if (req.url === "/engram/v1/recall") return res.writeHead(200).end("{}");
     res.writeHead(200).end("{}");
   });
@@ -671,7 +746,7 @@ test("token resolution: legacy ~/.engram/tokens.json is the read fallback", asyn
 function authEnvServer() {
   const seen = { health: "unset", recall: "unset" };
   return startServer((req, res) => {
-    if (req.url === "/engram/v1/health") {
+    if (req.url === "/engram/v1/live") {
       seen.health = req.headers.authorization || null;
       return res.writeHead(200).end("ok");
     }

package/hooks/bin/remnic-hook-core.cjs

@@ -366,34 +366,48 @@ function run(config) {
   // ignore) and green-light a probe whose recall then 401s.
   //
   // When the daemon has an auth token configured, every route — including
-  // /engram/v1/health — returns 401 to unauthenticated requests, so an
+  // /engram/v1/live — returns 401 to unauthenticated requests, so an
   // unauthenticated probe makes the hook wrongly report "daemon not running"
   // and skip recall/observe. Unauthenticated daemons ignore the header.
+  //
+  // One aggregate budget: readiness-gated /engram/v1/live first, and only a
+  // 404 (daemon predating the live route) retries the legacy diagnostic
+  // /engram/v1/health with the remaining time. 401/503 are final answers.
+  // Diagnostic health can outlive this liveness probe's deadline, which is
+  // why the gate probes liveness at all (#3257).
   function httpHealthy(timeoutMs, token) {
     if (DAEMON_URL === null) return Promise.resolve(false);
     const transport = DAEMON_URL.protocol === "https:" ? https : http;
-    return new Promise((resolve) => {
+    const headers = token ? { Authorization: `Bearer ${token}` } : {};
+    const deadline = Date.now() + timeoutMs;
+    const probe = (path, budget) => new Promise((resolve) => {
       const req = transport.request(
         {
           protocol: DAEMON_URL.protocol,
           hostname: DAEMON_URL.hostname,
           port: DAEMON_URL.port || (DAEMON_URL.protocol === "https:" ? 443 : 80),
-          path: DAEMON_BASE_PATH + "/engram/v1/health",
+          path: DAEMON_BASE_PATH + path,
           method: "GET",
-          headers: token ? { Authorization: `Bearer ${token}` } : {},
+          headers,
         },
         (res) => {
           res.resume();
-          resolve(res.statusCode >= 200 && res.statusCode < 300);
+          resolve(res.statusCode || 0);
         },
       );
-      req.on("error", () => resolve(false));
-      req.setTimeout(timeoutMs, () => {
+      req.on("error", () => resolve(0));
+      req.setTimeout(budget, () => {
         req.destroy();
-        resolve(false);
+        resolve(0);
       });
       req.end();
     });
+    return probe("/engram/v1/live", timeoutMs).then((status) => {
+      if (status !== 404) return status >= 200 && status < 300;
+      const remaining = deadline - Date.now();
+      if (remaining <= 0) return false;
+      return probe("/engram/v1/health", remaining).then((legacy) => legacy >= 200 && legacy < 300);
+    });
   }
 
   // ── git coding-context (mirrors @remnic/core git-context.ts) ───────────────

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remnic/plugin-claude-code",
-  "version": "10.0.39",
+  "version": "10.0.41",
   "description": "Remnic memory plugin for Claude Code — hooks, skills, MCP integration",
   "type": "module",
   "license": "MIT",