fix(viewer): singleton guard prevents port drift on reactivation

When the host gateway restarts and re-calls activate(), the previous
ViewerServer instance may still hold the port. Without cleanup, the new
instance hits EADDRINUSE and drifts to 18800+, causing Memory unavailable.

- Add module-level singleton tracking for ViewerServer
- Stop previous viewer instance before creating new one in activate()
- Ensure server.stop() fully releases the HTTP port

Fixes #1430

Author: kagura-agent

apps/memos-local-openclaw/index.ts

@@ -33,6 +33,11 @@ import { MEMORY_GUIDE_SKILL_MD } from "./src/skill/bundled-memory-guide";
 import { Telemetry } from "./src/telemetry";
 
 
+// Module-level singleton tracking for viewer/hub to prevent port drift on re-registration
+let _previousViewer: ViewerServer | null = null;
+let _previousHubServer: HubServer | null = null;
+let _previousStore: import('./src/storage/sqlite').SqliteStore | null = null;
+
 /** Remove near-duplicate hits based on summary word overlap (>70%). Keeps first (highest-scored) hit. */
 function deduplicateHits<T extends { summary: string }>(hits: T[]): T[] {
   const kept: T[] = [];
@@ -2365,6 +2370,20 @@ Groups: ${groupNames.length > 0 ? groupNames.join(", ") : "(none)"}`,
 
     const derivedHubPort = gatewayPort + 11;
 
+    // Cleanup previous instances to prevent port drift (issue #1430)
+    if (_previousViewer) {
+      try { _previousViewer.stop(); } catch (e) { api.logger.warn(`memos-local: previous viewer cleanup: ${e}`); }
+      _previousViewer = null;
+    }
+    if (_previousHubServer) {
+      try { _previousHubServer.stop(); } catch (e) { api.logger.warn(`memos-local: previous hub cleanup: ${e}`); }
+      _previousHubServer = null;
+    }
+    if (_previousStore) {
+      try { _previousStore.close(); } catch (e) { api.logger.warn(`memos-local: previous store cleanup: ${e}`); }
+      _previousStore = null;
+    }
+
     const viewer = new ViewerServer({
       store,
       embedder,
@@ -2378,6 +2397,10 @@ Groups: ${groupNames.length > 0 ? groupNames.join(", ") : "(none)"}`,
       ? new HubServer({ store, log: ctx.log, config: ctx.config, dataDir: stateDir, embedder, defaultHubPort: derivedHubPort })
       : null;
 
+    _previousViewer = viewer;
+    _previousHubServer = hubServer;
+    _previousStore = store;
+
     // ─── Service lifecycle ───
 
     let serviceStarted = false;

apps/memos-local-openclaw/src/viewer/server.ts

@@ -211,13 +211,18 @@ export class ViewerServer {
     }
   }
 
-  stop(): void {
+  stop(): Promise<void> {
     this.stopHubHeartbeat();
     this.stopNotifPoll();
     for (const c of this.notifSSEClients) { try { c.end(); } catch {} }
     this.notifSSEClients = [];
-    this.server?.close();
-    this.server = null;
+    if (!this.server) return Promise.resolve();
+    return new Promise<void>((resolve) => {
+      this.server!.close(() => {
+        this.server = null;
+        resolve();
+      });
+    });
   }
 
   getResetToken(): string {