@prmichaelsen/remember-mcp 3.17.0 → 3.17.2

package/.github/workflows/publish.yml

@@ -26,7 +26,7 @@ jobs:
         run: npm run build
 
       - name: Test
-        run: node --max-old-space-size=4096 ./node_modules/.bin/jest --runInBand
+        run: npm test
 
       - name: Check if version changed
         id: version

package/CHANGELOG.md

@@ -5,6 +5,13 @@ 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.17.1] - 2026-03-09
+
+### Fixed
+
+- Move `server-factory.spec.ts` to `server-factory.e2e.ts` — test requires live Weaviate and was crashing CI workers
+- Revert CI test command to `npm test` (OOM was a symptom of the Weaviate connection failure, not memory)
+
 ## [3.17.0] - 2026-03-09
 
 ### Added

package/dist/server-factory.e2e.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=server-factory.e2e.d.ts.map

package/dist/server-factory.js

@@ -3786,6 +3786,49 @@ var RelationshipService = class {
       this.logger.error(`Failed to update relationship_count for ${memoryId}:`, { error: error?.message || String(error) });
     }
   }
+  /**
+   * Validate that memory IDs exist, belong to user, are memories, and not deleted.
+   * Returns validated entries with their current relationship_ids.
+   */
+  async validateMemoryIds(memoryIds) {
+    const checks = await Promise.all(memoryIds.map(async (memoryId) => {
+      const memory = await this.collection.query.fetchObjectById(memoryId, {
+        returnProperties: ["user_id", "doc_type", "relationship_ids", "deleted_at"]
+      });
+      if (!memory)
+        return { memoryId, error: "Memory not found" };
+      if (memory.properties.user_id !== this.userId)
+        return { memoryId, error: "Unauthorized" };
+      if (memory.properties.doc_type !== "memory")
+        return { memoryId, error: "Not a memory document" };
+      if (memory.properties.deleted_at)
+        return { memoryId, error: "Memory is deleted" };
+      return { memoryId, relationships: memory.properties.relationship_ids || [] };
+    }));
+    const errors = checks.filter((c) => "error" in c && !!c.error);
+    if (errors.length > 0) {
+      throw new Error(`Memory validation failed: ${errors.map((e) => `${e.memoryId}: ${e.error}`).join("; ")}`);
+    }
+    return checks;
+  }
+  /**
+   * Add bidirectional relationship_ids links from memories to a relationship.
+   */
+  async linkMemoriesToRelationship(relationshipId, validated, now) {
+    await Promise.all(validated.map(async (c) => {
+      try {
+        await this.collection.data.update({
+          id: c.memoryId,
+          properties: {
+            relationship_ids: [...c.relationships, relationshipId],
+            updated_at: now
+          }
+        });
+      } catch {
+        this.logger.warn(`Failed to update memory ${c.memoryId} with relationship`);
+      }
+    }));
+  }
   // ── Get by ID ────────────────────────────────────────────────────────
   async getById(relationshipId) {
     const result = await this.collection.query.fetchObjectById(relationshipId, {
@@ -3817,24 +3860,7 @@ var RelationshipService = class {
     if (input.memory_ids.length < 2) {
       throw new Error("At least 2 memory IDs are required to create a relationship");
     }
-    const checks = await Promise.all(input.memory_ids.map(async (memoryId) => {
-      const memory = await this.collection.query.fetchObjectById(memoryId, {
-        returnProperties: ["user_id", "doc_type", "relationship_ids", "deleted_at"]
-      });
-      if (!memory)
-        return { memoryId, error: "Memory not found" };
-      if (memory.properties.user_id !== this.userId)
-        return { memoryId, error: "Unauthorized" };
-      if (memory.properties.doc_type !== "memory")
-        return { memoryId, error: "Not a memory document" };
-      if (memory.properties.deleted_at)
-        return { memoryId, error: "Memory is deleted" };
-      return { memoryId, memory, relationships: memory.properties.relationship_ids || [] };
-    }));
-    const errors = checks.filter((c) => c.error);
-    if (errors.length > 0) {
-      throw new Error(`Memory validation failed: ${errors.map((e) => `${e.memoryId}: ${e.error}`).join("; ")}`);
-    }
+    const validated = await this.validateMemoryIds(input.memory_ids);
     const now = (/* @__PURE__ */ new Date()).toISOString();
     const properties = {
       user_id: this.userId,
@@ -3853,26 +3879,18 @@ var RelationshipService = class {
     };
     const relationshipId = await this.collection.data.insert({ properties });
     await Promise.all(input.memory_ids.map((memoryId) => this.updateRelationshipCount(memoryId, 1)));
-    await Promise.all(checks.filter((c) => !c.error && c.memory).map(async (c) => {
-      try {
-        await this.collection.data.update({
-          id: c.memoryId,
-          properties: {
-            relationship_ids: [...c.relationships || [], relationshipId],
-            updated_at: now
-          }
-        });
-      } catch {
-        this.logger.warn(`Failed to update memory ${c.memoryId} with relationship`);
-      }
-    }));
+    await this.linkMemoriesToRelationship(relationshipId, validated, now);
     this.logger.info("Relationship created", { relationshipId, memoryCount: input.memory_ids.length });
     return { relationship_id: relationshipId, memory_ids: input.memory_ids, created_at: now };
   }
   // ── Update ──────────────────────────────────────────────────────────
   async update(input) {
+    const fetchProps = ["user_id", "doc_type", "version"];
+    const needMemoryIds = input.add_memory_ids && input.add_memory_ids.length > 0;
+    if (needMemoryIds)
+      fetchProps.push("related_memory_ids");
     const existing = await this.collection.query.fetchObjectById(input.relationship_id, {
-      returnProperties: ["user_id", "doc_type", "version"]
+      returnProperties: fetchProps
     });
     if (!existing)
       throw new Error(`Relationship not found: ${input.relationship_id}`);
@@ -3906,12 +3924,31 @@ var RelationshipService = class {
       updates.tags = input.tags;
       updatedFields.push("tags");
     }
+    let newMemoryIds = [];
+    let validated = [];
+    if (needMemoryIds) {
+      const existingMemoryIds = new Set(existing.properties.related_memory_ids || []);
+      newMemoryIds = input.add_memory_ids.filter((id) => !existingMemoryIds.has(id));
+      if (newMemoryIds.length > 0) {
+        validated = await this.validateMemoryIds(newMemoryIds);
+        const allMemoryIds = [...existingMemoryIds, ...newMemoryIds];
+        updates.related_memory_ids = allMemoryIds;
+        updates.member_count = allMemoryIds.length;
+        updatedFields.push("related_memory_ids", "member_count");
+      }
+    }
     if (updatedFields.length === 0)
       throw new Error("No fields provided for update");
     const now = (/* @__PURE__ */ new Date()).toISOString();
     updates.updated_at = now;
     updates.version = existing.properties.version + 1;
     await this.collection.data.update({ id: input.relationship_id, properties: updates });
+    if (newMemoryIds.length > 0) {
+      await Promise.all([
+        this.linkMemoriesToRelationship(input.relationship_id, validated, now),
+        ...newMemoryIds.map((memoryId) => this.updateRelationshipCount(memoryId, 1))
+      ]);
+    }
     this.logger.info("Relationship updated", { relationshipId: input.relationship_id, updatedFields });
     return {
       relationship_id: input.relationship_id,

package/dist/server.js

@@ -3790,6 +3790,49 @@ var RelationshipService = class {
       this.logger.error(`Failed to update relationship_count for ${memoryId}:`, { error: error?.message || String(error) });
     }
   }
+  /**
+   * Validate that memory IDs exist, belong to user, are memories, and not deleted.
+   * Returns validated entries with their current relationship_ids.
+   */
+  async validateMemoryIds(memoryIds) {
+    const checks = await Promise.all(memoryIds.map(async (memoryId) => {
+      const memory = await this.collection.query.fetchObjectById(memoryId, {
+        returnProperties: ["user_id", "doc_type", "relationship_ids", "deleted_at"]
+      });
+      if (!memory)
+        return { memoryId, error: "Memory not found" };
+      if (memory.properties.user_id !== this.userId)
+        return { memoryId, error: "Unauthorized" };
+      if (memory.properties.doc_type !== "memory")
+        return { memoryId, error: "Not a memory document" };
+      if (memory.properties.deleted_at)
+        return { memoryId, error: "Memory is deleted" };
+      return { memoryId, relationships: memory.properties.relationship_ids || [] };
+    }));
+    const errors = checks.filter((c) => "error" in c && !!c.error);
+    if (errors.length > 0) {
+      throw new Error(`Memory validation failed: ${errors.map((e) => `${e.memoryId}: ${e.error}`).join("; ")}`);
+    }
+    return checks;
+  }
+  /**
+   * Add bidirectional relationship_ids links from memories to a relationship.
+   */
+  async linkMemoriesToRelationship(relationshipId, validated, now) {
+    await Promise.all(validated.map(async (c) => {
+      try {
+        await this.collection.data.update({
+          id: c.memoryId,
+          properties: {
+            relationship_ids: [...c.relationships, relationshipId],
+            updated_at: now
+          }
+        });
+      } catch {
+        this.logger.warn(`Failed to update memory ${c.memoryId} with relationship`);
+      }
+    }));
+  }
   // ── Get by ID ────────────────────────────────────────────────────────
   async getById(relationshipId) {
     const result = await this.collection.query.fetchObjectById(relationshipId, {
@@ -3821,24 +3864,7 @@ var RelationshipService = class {
     if (input.memory_ids.length < 2) {
       throw new Error("At least 2 memory IDs are required to create a relationship");
     }
-    const checks = await Promise.all(input.memory_ids.map(async (memoryId) => {
-      const memory = await this.collection.query.fetchObjectById(memoryId, {
-        returnProperties: ["user_id", "doc_type", "relationship_ids", "deleted_at"]
-      });
-      if (!memory)
-        return { memoryId, error: "Memory not found" };
-      if (memory.properties.user_id !== this.userId)
-        return { memoryId, error: "Unauthorized" };
-      if (memory.properties.doc_type !== "memory")
-        return { memoryId, error: "Not a memory document" };
-      if (memory.properties.deleted_at)
-        return { memoryId, error: "Memory is deleted" };
-      return { memoryId, memory, relationships: memory.properties.relationship_ids || [] };
-    }));
-    const errors = checks.filter((c) => c.error);
-    if (errors.length > 0) {
-      throw new Error(`Memory validation failed: ${errors.map((e) => `${e.memoryId}: ${e.error}`).join("; ")}`);
-    }
+    const validated = await this.validateMemoryIds(input.memory_ids);
     const now = (/* @__PURE__ */ new Date()).toISOString();
     const properties = {
       user_id: this.userId,
@@ -3857,26 +3883,18 @@ var RelationshipService = class {
     };
     const relationshipId = await this.collection.data.insert({ properties });
     await Promise.all(input.memory_ids.map((memoryId) => this.updateRelationshipCount(memoryId, 1)));
-    await Promise.all(checks.filter((c) => !c.error && c.memory).map(async (c) => {
-      try {
-        await this.collection.data.update({
-          id: c.memoryId,
-          properties: {
-            relationship_ids: [...c.relationships || [], relationshipId],
-            updated_at: now
-          }
-        });
-      } catch {
-        this.logger.warn(`Failed to update memory ${c.memoryId} with relationship`);
-      }
-    }));
+    await this.linkMemoriesToRelationship(relationshipId, validated, now);
     this.logger.info("Relationship created", { relationshipId, memoryCount: input.memory_ids.length });
     return { relationship_id: relationshipId, memory_ids: input.memory_ids, created_at: now };
   }
   // ── Update ──────────────────────────────────────────────────────────
   async update(input) {
+    const fetchProps = ["user_id", "doc_type", "version"];
+    const needMemoryIds = input.add_memory_ids && input.add_memory_ids.length > 0;
+    if (needMemoryIds)
+      fetchProps.push("related_memory_ids");
     const existing = await this.collection.query.fetchObjectById(input.relationship_id, {
-      returnProperties: ["user_id", "doc_type", "version"]
+      returnProperties: fetchProps
     });
     if (!existing)
       throw new Error(`Relationship not found: ${input.relationship_id}`);
@@ -3910,12 +3928,31 @@ var RelationshipService = class {
       updates.tags = input.tags;
       updatedFields.push("tags");
     }
+    let newMemoryIds = [];
+    let validated = [];
+    if (needMemoryIds) {
+      const existingMemoryIds = new Set(existing.properties.related_memory_ids || []);
+      newMemoryIds = input.add_memory_ids.filter((id) => !existingMemoryIds.has(id));
+      if (newMemoryIds.length > 0) {
+        validated = await this.validateMemoryIds(newMemoryIds);
+        const allMemoryIds = [...existingMemoryIds, ...newMemoryIds];
+        updates.related_memory_ids = allMemoryIds;
+        updates.member_count = allMemoryIds.length;
+        updatedFields.push("related_memory_ids", "member_count");
+      }
+    }
     if (updatedFields.length === 0)
       throw new Error("No fields provided for update");
     const now = (/* @__PURE__ */ new Date()).toISOString();
     updates.updated_at = now;
     updates.version = existing.properties.version + 1;
     await this.collection.data.update({ id: input.relationship_id, properties: updates });
+    if (newMemoryIds.length > 0) {
+      await Promise.all([
+        this.linkMemoriesToRelationship(input.relationship_id, validated, now),
+        ...newMemoryIds.map((memoryId) => this.updateRelationshipCount(memoryId, 1))
+      ]);
+    }
     this.logger.info("Relationship updated", { relationshipId: input.relationship_id, updatedFields });
     return {
       relationship_id: input.relationship_id,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@prmichaelsen/remember-mcp",
-  "version": "3.17.0",
+  "version": "3.17.2",
   "description": "Multi-tenant memory system MCP server with vector search and relationships",
   "main": "dist/server.js",
   "type": "module",
@@ -51,7 +51,7 @@
     "@google-cloud/vision": "^5.3.4",
     "@modelcontextprotocol/sdk": "^1.0.4",
     "@prmichaelsen/firebase-admin-sdk-v8": "^2.2.0",
-    "@prmichaelsen/remember-core": "^0.54.0",
+    "@prmichaelsen/remember-core": "^0.55.0",
     "dotenv": "^16.4.5",
     "uuid": "^13.0.0",
     "weaviate-client": "^3.2.0"

package/dist/server-factory.spec.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=server-factory.spec.d.ts.map