@prmichaelsen/remember-mcp 3.14.21 → 3.15.0

package/CHANGELOG.md

@@ -5,6 +5,14 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [3.15.0] - 2026-03-06
+
+### Fixed
+
+- Wire `MemoryIndexService` into `MemoryService` and `SpaceService` (task-174)
+- All new memories and space-published memories are now indexed in Firestore lookup table
+- Enables `resolveById()` cross-collection resolution and fixes 404s on published memories
+
 ## [3.14.21] - 2026-03-06
 
 ### Added

package/agent/tasks/unassigned/task-174-wire-memory-index-service.md

@@ -0,0 +1,97 @@
+# Task 174: Wire MemoryIndexService into Core Services
+
+**Milestone**: Unassigned (breaking change from remember-core v0.33.0)
+**Estimated Time**: 0.5-1 hour
+**Dependencies**: remember-core v0.33.0+
+**Status**: Not Started
+
+---
+
+## Objective
+
+Update `src/core-services.ts` to pass the now-required `MemoryIndexService` to both `MemoryService` and `SpaceService` constructors. remember-core v0.33.0 made `MemoryIndexService` a required parameter (no longer optional) in both services.
+
+---
+
+## Context
+
+remember-core task-117 made `MemoryIndexService` required in:
+- `MemoryService` constructor: 4th param `options.memoryIndex` (was optional, now required)
+- `SpaceService` constructor: new 6th positional param `memoryIndexService` (before the optional `options`)
+
+Currently `src/core-services.ts` line 52 constructs `MemoryService` with 3 args (no memoryIndex) and line 54 constructs `SpaceService` with `{ moderationClient }` as the 6th arg (which is now the 7th position).
+
+---
+
+## Steps
+
+### 1. Import MemoryIndexService
+
+Add `MemoryIndexService` to the import from `@prmichaelsen/remember-core`.
+
+### 2. Create singleton MemoryIndexService
+
+Add alongside the other singletons (line ~31):
+
+```typescript
+const memoryIndexService = new MemoryIndexService(coreLogger);
+```
+
+### 3. Update MemoryService constructor call
+
+Change line 52 from:
+```typescript
+memory: new MemoryService(collection, userId, coreLogger),
+```
+to:
+```typescript
+memory: new MemoryService(collection, userId, coreLogger, {
+  memoryIndex: memoryIndexService,
+  weaviateClient,
+}),
+```
+
+### 4. Update SpaceService constructor call
+
+Change line 54 from:
+```typescript
+space: new SpaceService(weaviateClient, collection, userId, tokenService, coreLogger, { moderationClient }),
+```
+to:
+```typescript
+space: new SpaceService(weaviateClient, collection, userId, tokenService, coreLogger, memoryIndexService, { moderationClient }),
+```
+
+### 5. Bump remember-core dependency
+
+Update `package.json` to require `@prmichaelsen/remember-core` >= 0.33.0.
+
+### 6. Build and verify
+
+- `tsc --noEmit` passes
+- All tests pass
+
+---
+
+## Verification
+
+- [ ] `MemoryIndexService` imported and instantiated as singleton
+- [ ] `MemoryService` receives `{ memoryIndex: memoryIndexService, weaviateClient }` in options
+- [ ] `SpaceService` receives `memoryIndexService` as 6th positional arg
+- [ ] `tsc --noEmit` passes
+- [ ] All tests pass
+
+---
+
+## Files Modified
+
+- `src/core-services.ts` — add import, singleton, update constructor calls
+- `package.json` — bump remember-core dependency
+
+---
+
+## Notes
+
+- Single file change (`core-services.ts`) — all service construction is centralized here
+- `weaviateClient` is already available in `createCoreServices()`, so passing it to MemoryService options enables `resolveById()` cross-collection resolution
+- This ensures all new memories created via MCP tools are indexed in the Firestore lookup table

package/dist/server-factory.js

@@ -1754,6 +1754,9 @@ function getUserPreferencesPath(userId) {
 function getCollectionRegistryPath() {
   return `${BASE}.collection_registry`;
 }
+function getMemoryIndexPath() {
+  return `${BASE}.memory_index`;
+}
 
 // node_modules/@prmichaelsen/remember-core/dist/services/preferences.service.js
 var PreferencesDatabaseService = class {
@@ -2143,20 +2146,18 @@ var MemoryService = class {
    * Requires `memoryIndex` and `weaviateClient` in constructor options.
    */
   async resolveById(memoryId) {
-    if (!this.options?.weaviateClient) {
+    if (!this.options.weaviateClient) {
       throw new Error("resolveById requires weaviateClient in options");
     }
-    if (this.options?.memoryIndex) {
-      const collectionName = await this.options.memoryIndex.lookup(memoryId);
-      if (collectionName) {
-        const col = this.options.weaviateClient.collections.get(collectionName);
-        const memory = await fetchMemoryWithAllProperties(col, memoryId);
-        if (memory?.properties) {
-          return {
-            memory: normalizeDoc({ id: memory.uuid, ...memory.properties }),
-            collectionName
-          };
-        }
+    const collectionName = await this.options.memoryIndex.lookup(memoryId);
+    if (collectionName) {
+      const col = this.options.weaviateClient.collections.get(collectionName);
+      const memory = await fetchMemoryWithAllProperties(col, memoryId);
+      if (memory?.properties) {
+        return {
+          memory: normalizeDoc({ id: memory.uuid, ...memory.properties }),
+          collectionName
+        };
       }
     }
     return { memory: null, collectionName: null };
@@ -2202,13 +2203,11 @@ var MemoryService = class {
     };
     const memoryId = await this.collection.data.insert({ properties });
     this.logger.info("Memory created", { memoryId, userId: this.userId });
-    if (this.options?.memoryIndex) {
-      try {
-        const collectionName = this.collection.name;
-        await this.options.memoryIndex.index(memoryId, collectionName);
-      } catch (err2) {
-        this.logger.warn?.(`[MemoryService] Index write failed for ${memoryId}: ${err2}`);
-      }
+    try {
+      const collectionName = this.collection.name;
+      await this.options.memoryIndex.index(memoryId, collectionName);
+    } catch (err2) {
+      this.logger.warn?.(`[MemoryService] Index write failed for ${memoryId}: ${err2}`);
     }
     return { memory_id: memoryId, created_at: now };
   }
@@ -3142,14 +3141,18 @@ var SpaceService = class {
   userId;
   confirmationTokenService;
   logger;
+  memoryIndexService;
   moderationClient;
-  constructor(weaviateClient, userCollection, userId, confirmationTokenService, logger2, options) {
+  memoryIndex;
+  constructor(weaviateClient, userCollection, userId, confirmationTokenService, logger2, memoryIndexService2, options) {
     this.weaviateClient = weaviateClient;
     this.userCollection = userCollection;
     this.userId = userId;
     this.confirmationTokenService = confirmationTokenService;
     this.logger = logger2;
+    this.memoryIndexService = memoryIndexService2;
     this.moderationClient = options?.moderationClient;
+    this.memoryIndex = memoryIndexService2;
   }
   // ── Content moderation helper ────────────────────────────────────────
   async checkModeration(content) {
@@ -3555,6 +3558,11 @@ var SpaceService = class {
         } else {
           await publicCollection.data.insert({ id: weaviateId, properties: publishedMemory });
         }
+        try {
+          await this.memoryIndex.index(weaviateId, "Memory_spaces_public");
+        } catch (err2) {
+          this.logger.warn?.(`[SpaceService] Index write failed for ${weaviateId}: ${err2}`);
+        }
         successfulPublications.push(`spaces: ${spaces.join(", ")}`);
       } catch (err2) {
         failedPublications.push(`spaces: ${err2 instanceof Error ? err2.message : String(err2)}`);
@@ -3591,6 +3599,11 @@ var SpaceService = class {
         } else {
           await groupCollection.data.insert({ id: weaviateId, properties: groupMemory });
         }
+        try {
+          await this.memoryIndex.index(weaviateId, groupCollectionName);
+        } catch (err2) {
+          this.logger.warn?.(`[SpaceService] Index write failed for ${weaviateId} in ${groupCollectionName}: ${err2}`);
+        }
         successfulPublications.push(`group: ${groupId}`);
       } catch (err2) {
         failedPublications.push(`group ${groupId}: ${err2 instanceof Error ? err2.message : String(err2)}`);
@@ -3831,6 +3844,39 @@ var REM_STATE_COLLECTION = `${BASE}.rem_state`;
 // node_modules/@prmichaelsen/remember-core/dist/services/rem.clustering.js
 import { Filters as Filters5 } from "weaviate-client";
 
+// node_modules/@prmichaelsen/remember-core/dist/services/memory-index.service.js
+var MemoryIndexService = class {
+  logger;
+  collectionPath;
+  constructor(logger2) {
+    this.logger = logger2;
+    this.collectionPath = getMemoryIndexPath();
+  }
+  /**
+   * Index a memory UUID → collection name mapping.
+   * Uses set() for idempotency (safe to re-index).
+   */
+  async index(memoryUuid, collectionName) {
+    const entry = {
+      collection_name: collectionName,
+      created_at: (/* @__PURE__ */ new Date()).toISOString()
+    };
+    await setDocument2(this.collectionPath, memoryUuid, entry);
+    this.logger.debug?.(`[MemoryIndex] Indexed ${memoryUuid} \u2192 ${collectionName}`);
+  }
+  /**
+   * Look up the collection name for a memory UUID.
+   * Returns null if the memory is not indexed.
+   */
+  async lookup(memoryUuid) {
+    const doc = await getDocument2(this.collectionPath, memoryUuid);
+    if (!doc)
+      return null;
+    const entry = doc;
+    return entry.collection_name ?? null;
+  }
+};
+
 // node_modules/@prmichaelsen/remember-core/dist/services/moderation.service.js
 import { createHash as createHash2 } from "crypto";
 function buildModerationPrompt(content) {
@@ -4053,6 +4099,7 @@ var coreLogger = createLogger("info");
 var tokenService = new ConfirmationTokenService(coreLogger);
 var preferencesService = new PreferencesDatabaseService(coreLogger);
 var moderationClient = process.env.ANTHROPIC_API_KEY ? createModerationClient({ apiKey: process.env.ANTHROPIC_API_KEY }) : void 0;
+var memoryIndexService = new MemoryIndexService(coreLogger);
 var coreServicesCache = /* @__PURE__ */ new Map();
 function createCoreServices(userId) {
   const cached = coreServicesCache.get(userId);
@@ -4061,9 +4108,12 @@ function createCoreServices(userId) {
   const collection = getMemoryCollection(userId);
   const weaviateClient = getWeaviateClient();
   const services = {
-    memory: new MemoryService(collection, userId, coreLogger),
+    memory: new MemoryService(collection, userId, coreLogger, {
+      memoryIndex: memoryIndexService,
+      weaviateClient
+    }),
     relationship: new RelationshipService(collection, userId, coreLogger),
-    space: new SpaceService(weaviateClient, collection, userId, tokenService, coreLogger, { moderationClient }),
+    space: new SpaceService(weaviateClient, collection, userId, tokenService, coreLogger, memoryIndexService, { moderationClient }),
     preferences: preferencesService,
     token: tokenService
   };

package/dist/server.js

@@ -1758,6 +1758,9 @@ function getUserPreferencesPath(userId) {
 function getCollectionRegistryPath() {
   return `${BASE}.collection_registry`;
 }
+function getMemoryIndexPath() {
+  return `${BASE}.memory_index`;
+}
 
 // node_modules/@prmichaelsen/remember-core/dist/services/preferences.service.js
 var PreferencesDatabaseService = class {
@@ -2147,20 +2150,18 @@ var MemoryService = class {
    * Requires `memoryIndex` and `weaviateClient` in constructor options.
    */
   async resolveById(memoryId) {
-    if (!this.options?.weaviateClient) {
+    if (!this.options.weaviateClient) {
       throw new Error("resolveById requires weaviateClient in options");
     }
-    if (this.options?.memoryIndex) {
-      const collectionName = await this.options.memoryIndex.lookup(memoryId);
-      if (collectionName) {
-        const col = this.options.weaviateClient.collections.get(collectionName);
-        const memory = await fetchMemoryWithAllProperties(col, memoryId);
-        if (memory?.properties) {
-          return {
-            memory: normalizeDoc({ id: memory.uuid, ...memory.properties }),
-            collectionName
-          };
-        }
+    const collectionName = await this.options.memoryIndex.lookup(memoryId);
+    if (collectionName) {
+      const col = this.options.weaviateClient.collections.get(collectionName);
+      const memory = await fetchMemoryWithAllProperties(col, memoryId);
+      if (memory?.properties) {
+        return {
+          memory: normalizeDoc({ id: memory.uuid, ...memory.properties }),
+          collectionName
+        };
       }
     }
     return { memory: null, collectionName: null };
@@ -2206,13 +2207,11 @@ var MemoryService = class {
     };
     const memoryId = await this.collection.data.insert({ properties });
     this.logger.info("Memory created", { memoryId, userId: this.userId });
-    if (this.options?.memoryIndex) {
-      try {
-        const collectionName = this.collection.name;
-        await this.options.memoryIndex.index(memoryId, collectionName);
-      } catch (err2) {
-        this.logger.warn?.(`[MemoryService] Index write failed for ${memoryId}: ${err2}`);
-      }
+    try {
+      const collectionName = this.collection.name;
+      await this.options.memoryIndex.index(memoryId, collectionName);
+    } catch (err2) {
+      this.logger.warn?.(`[MemoryService] Index write failed for ${memoryId}: ${err2}`);
     }
     return { memory_id: memoryId, created_at: now };
   }
@@ -3146,14 +3145,18 @@ var SpaceService = class {
   userId;
   confirmationTokenService;
   logger;
+  memoryIndexService;
   moderationClient;
-  constructor(weaviateClient, userCollection, userId, confirmationTokenService, logger2, options) {
+  memoryIndex;
+  constructor(weaviateClient, userCollection, userId, confirmationTokenService, logger2, memoryIndexService2, options) {
     this.weaviateClient = weaviateClient;
     this.userCollection = userCollection;
     this.userId = userId;
     this.confirmationTokenService = confirmationTokenService;
     this.logger = logger2;
+    this.memoryIndexService = memoryIndexService2;
     this.moderationClient = options?.moderationClient;
+    this.memoryIndex = memoryIndexService2;
   }
   // ── Content moderation helper ────────────────────────────────────────
   async checkModeration(content) {
@@ -3559,6 +3562,11 @@ var SpaceService = class {
         } else {
           await publicCollection.data.insert({ id: weaviateId, properties: publishedMemory });
         }
+        try {
+          await this.memoryIndex.index(weaviateId, "Memory_spaces_public");
+        } catch (err2) {
+          this.logger.warn?.(`[SpaceService] Index write failed for ${weaviateId}: ${err2}`);
+        }
         successfulPublications.push(`spaces: ${spaces.join(", ")}`);
       } catch (err2) {
         failedPublications.push(`spaces: ${err2 instanceof Error ? err2.message : String(err2)}`);
@@ -3595,6 +3603,11 @@ var SpaceService = class {
         } else {
           await groupCollection.data.insert({ id: weaviateId, properties: groupMemory });
         }
+        try {
+          await this.memoryIndex.index(weaviateId, groupCollectionName);
+        } catch (err2) {
+          this.logger.warn?.(`[SpaceService] Index write failed for ${weaviateId} in ${groupCollectionName}: ${err2}`);
+        }
         successfulPublications.push(`group: ${groupId}`);
       } catch (err2) {
         failedPublications.push(`group ${groupId}: ${err2 instanceof Error ? err2.message : String(err2)}`);
@@ -3835,6 +3848,39 @@ var REM_STATE_COLLECTION = `${BASE}.rem_state`;
 // node_modules/@prmichaelsen/remember-core/dist/services/rem.clustering.js
 import { Filters as Filters5 } from "weaviate-client";
 
+// node_modules/@prmichaelsen/remember-core/dist/services/memory-index.service.js
+var MemoryIndexService = class {
+  logger;
+  collectionPath;
+  constructor(logger2) {
+    this.logger = logger2;
+    this.collectionPath = getMemoryIndexPath();
+  }
+  /**
+   * Index a memory UUID → collection name mapping.
+   * Uses set() for idempotency (safe to re-index).
+   */
+  async index(memoryUuid, collectionName) {
+    const entry = {
+      collection_name: collectionName,
+      created_at: (/* @__PURE__ */ new Date()).toISOString()
+    };
+    await setDocument2(this.collectionPath, memoryUuid, entry);
+    this.logger.debug?.(`[MemoryIndex] Indexed ${memoryUuid} \u2192 ${collectionName}`);
+  }
+  /**
+   * Look up the collection name for a memory UUID.
+   * Returns null if the memory is not indexed.
+   */
+  async lookup(memoryUuid) {
+    const doc = await getDocument2(this.collectionPath, memoryUuid);
+    if (!doc)
+      return null;
+    const entry = doc;
+    return entry.collection_name ?? null;
+  }
+};
+
 // node_modules/@prmichaelsen/remember-core/dist/services/moderation.service.js
 import { createHash as createHash2 } from "crypto";
 function buildModerationPrompt(content) {
@@ -4057,6 +4103,7 @@ var coreLogger = createLogger("info");
 var tokenService = new ConfirmationTokenService(coreLogger);
 var preferencesService = new PreferencesDatabaseService(coreLogger);
 var moderationClient = process.env.ANTHROPIC_API_KEY ? createModerationClient({ apiKey: process.env.ANTHROPIC_API_KEY }) : void 0;
+var memoryIndexService = new MemoryIndexService(coreLogger);
 var coreServicesCache = /* @__PURE__ */ new Map();
 function createCoreServices(userId) {
   const cached = coreServicesCache.get(userId);
@@ -4065,9 +4112,12 @@ function createCoreServices(userId) {
   const collection = getMemoryCollection(userId);
   const weaviateClient = getWeaviateClient();
   const services = {
-    memory: new MemoryService(collection, userId, coreLogger),
+    memory: new MemoryService(collection, userId, coreLogger, {
+      memoryIndex: memoryIndexService,
+      weaviateClient
+    }),
     relationship: new RelationshipService(collection, userId, coreLogger),
-    space: new SpaceService(weaviateClient, collection, userId, tokenService, coreLogger, { moderationClient }),
+    space: new SpaceService(weaviateClient, collection, userId, tokenService, coreLogger, memoryIndexService, { moderationClient }),
     preferences: preferencesService,
     token: tokenService
   };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@prmichaelsen/remember-mcp",
-  "version": "3.14.21",
+  "version": "3.15.0",
   "description": "Multi-tenant memory system MCP server with vector search and relationships",
   "main": "dist/server.js",
   "type": "module",
@@ -50,7 +50,7 @@
     "@modelcontextprotocol/sdk": "^1.0.4",
     "@prmichaelsen/firebase-admin-sdk-v8": "^2.2.0",
     "@prmichaelsen/mcp-auth": "^7.0.4",
-    "@prmichaelsen/remember-core": "^0.32.1",
+    "@prmichaelsen/remember-core": "^0.33.1",
     "dotenv": "^16.4.5",
     "uuid": "^13.0.0",
     "weaviate-client": "^3.2.0"

package/src/core-services.ts

@@ -7,6 +7,7 @@
 
 import {
   MemoryService,
+  MemoryIndexService,
   RelationshipService,
   SpaceService,
   PreferencesDatabaseService,
@@ -33,6 +34,7 @@ const preferencesService = new PreferencesDatabaseService(coreLogger);
 const moderationClient: ModerationClient | undefined = process.env.ANTHROPIC_API_KEY
   ? createModerationClient({ apiKey: process.env.ANTHROPIC_API_KEY })
   : undefined;
+const memoryIndexService = new MemoryIndexService(coreLogger);
 
 /** Cached CoreServices per userId — avoids re-instantiation on every tool call */
 const coreServicesCache = new Map<string, CoreServices>();
@@ -49,9 +51,12 @@ export function createCoreServices(userId: string): CoreServices {
   const weaviateClient = getWeaviateClient();
 
   const services: CoreServices = {
-    memory: new MemoryService(collection, userId, coreLogger),
+    memory: new MemoryService(collection, userId, coreLogger, {
+      memoryIndex: memoryIndexService,
+      weaviateClient,
+    }),
     relationship: new RelationshipService(collection, userId, coreLogger),
-    space: new SpaceService(weaviateClient, collection, userId, tokenService, coreLogger, { moderationClient }),
+    space: new SpaceService(weaviateClient, collection, userId, tokenService, coreLogger, memoryIndexService, { moderationClient }),
     preferences: preferencesService,
     token: tokenService,
   };