@joshuaswarren/openclaw-engram 9.0.98 → 9.0.99

package/dist/index.js

@@ -13509,6 +13509,8 @@ var ENGRAM_REGISTERED_GUARD = "__openclawEngramRegistered";
 var ENGRAM_HOOK_APIS = "__openclawEngramHookApis";
 var ENGRAM_ACCESS_SERVICE = "__openclawEngramAccessService";
 var ENGRAM_ACCESS_HTTP_SERVER = "__openclawEngramAccessHttpServer";
+var ENGRAM_SERVICE_STARTED = "__openclawEngramServiceStarted";
+var ENGRAM_INIT_PROMISE = "__openclawEngramInitPromise";
 function loadPluginConfigFromFile() {
   try {
     const explicitConfigPath = process.env.OPENCLAW_ENGRAM_CONFIG_PATH || process.env.OPENCLAW_CONFIG_PATH;
@@ -13974,35 +13976,76 @@ Use this context naturally when relevant. Never quote or expose this memory cont
       registerCli(api, orchestrator);
     }
     let activeOpikExporter = null;
-    if (isFirstRegistration) api.registerService({
+    let didCountStart = false;
+    api.registerService({
       id: "openclaw-engram",
       start: async () => {
-        log.info("initializing engram memory system...");
-        await orchestrator.initialize();
-        activeOpikExporter = createOpikExporter({}, log);
-        if (activeOpikExporter) activeOpikExporter.subscribe();
-        if (orchestrator.config.transcriptEnabled) {
-          await orchestrator.transcript.cleanup(orchestrator.config.transcriptRetentionDays);
-        }
-        if (orchestrator.config.hourlySummariesEnabled && orchestrator.config.hourlySummaryCronAutoRegister) {
-          await ensureHourlySummaryCron(api);
-        } else if (orchestrator.config.hourlySummariesEnabled) {
-          log.info(
-            "hourly summaries enabled; cron auto-register is disabled. To schedule summaries, create an isolated/agentTurn cron job that calls `memory_summarize_hourly`."
-          );
+        if (globalThis[ENGRAM_INIT_PROMISE]) {
+          await globalThis[ENGRAM_INIT_PROMISE];
+          if (globalThis[ENGRAM_SERVICE_STARTED]) return;
         }
-        if (cfg.agentAccessHttp.enabled) {
+        if (globalThis[ENGRAM_SERVICE_STARTED]) {
+          log.debug("openclaw-engram: service.start() called again \u2014 skipping duplicate init");
+          return;
+        }
+        didCountStart = true;
+        const initPromise = (async () => {
           try {
-            const status = await accessHttpServer.start();
-            log.info(`engram access HTTP ready at http://${status.host}:${status.port}`);
+            log.info("initializing engram memory system...");
+            await orchestrator.initialize();
+            if (!didCountStart) return;
+            activeOpikExporter = createOpikExporter({}, log);
+            if (activeOpikExporter) activeOpikExporter.subscribe();
+            if (orchestrator.config.transcriptEnabled) {
+              await orchestrator.transcript.cleanup(orchestrator.config.transcriptRetentionDays);
+              if (!didCountStart) return;
+            }
+            if (orchestrator.config.hourlySummariesEnabled && orchestrator.config.hourlySummaryCronAutoRegister) {
+              await ensureHourlySummaryCron(api);
+              if (!didCountStart) return;
+            } else if (orchestrator.config.hourlySummariesEnabled) {
+              log.info(
+                "hourly summaries enabled; cron auto-register is disabled. To schedule summaries, create an isolated/agentTurn cron job that calls `memory_summarize_hourly`."
+              );
+            }
+            if (cfg.agentAccessHttp.enabled) {
+              if (!didCountStart) return;
+              try {
+                const status = await accessHttpServer.start();
+                log.info(`engram access HTTP ready at http://${status.host}:${status.port}`);
+              } catch (err) {
+                log.error("failed to start engram access HTTP server", err);
+              }
+            }
+            if (!didCountStart) return;
+            log.info("engram memory system ready");
           } catch (err) {
-            log.error("failed to start engram access HTTP server", err);
+            try {
+              activeOpikExporter?.unsubscribe();
+            } catch {
+            }
+            activeOpikExporter = null;
+            didCountStart = false;
+            globalThis[ENGRAM_SERVICE_STARTED] = false;
+            throw err;
           }
+        })();
+        globalThis[ENGRAM_INIT_PROMISE] = initPromise;
+        globalThis[ENGRAM_SERVICE_STARTED] = true;
+        try {
+          await initPromise;
+        } finally {
+          globalThis[ENGRAM_INIT_PROMISE] = null;
         }
-        log.info("engram memory system ready");
       },
       stop: async () => {
-        activeOpikExporter?.unsubscribe();
+        if (!didCountStart) return;
+        didCountStart = false;
+        try {
+          activeOpikExporter?.unsubscribe();
+        } catch (err) {
+          log.debug(`engram opik exporter unsubscribe failed: ${err}`);
+        }
         activeOpikExporter = null;
         try {
           await accessHttpServer.stop();
@@ -14013,6 +14056,7 @@ Use this context naturally when relevant. Never quote or expose this memory cont
         delete globalThis[ENGRAM_ACCESS_SERVICE];
         globalThis[ENGRAM_REGISTERED_GUARD] = false;
         globalThis[ENGRAM_HOOK_APIS] = /* @__PURE__ */ new WeakSet();
+        globalThis[ENGRAM_SERVICE_STARTED] = false;
         log.info("stopped");
       }
     });