@prmichaelsen/remember-mcp 2.7.1 → 2.7.3

package/CHANGELOG.md

@@ -5,6 +5,54 @@ 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).
 
+## [2.7.3] - 2026-02-17
+
+### Fixed
+
+- **CRITICAL: Fixed `remember_update_memory` "Memory not found" error**
+  - Changed `handleUpdateMemory()` to use `fetchMemoryWithAllProperties()` wrapper
+  - Previously used direct `collection.query.fetchObjectById()` call without fallback
+  - Direct call failed when querying memories with certain property configurations (e.g., `parent_id: ""`)
+  - Wrapper provides graceful fallback for schema evolution and property incompatibilities
+  - Matches pattern used in other working tools like `handlePublish()`
+
+### Root Cause
+
+- `handleUpdateMemory()` bypassed the `fetchMemoryWithAllProperties()` abstraction layer
+- Direct `fetchObjectById()` without property specification can fail on edge case property values
+- Error was caught and converted to misleading "Memory not found" message
+- Wrapper's try-catch with fallback handles these cases gracefully
+- This fix ensures consistent behavior across all memory operations
+
+### Technical Details
+
+- Modified: `src/tools/update-memory.ts` (lines 8, 140)
+- Added import: `fetchMemoryWithAllProperties` from `weaviate/client.js`
+- Replaced: `collection.query.fetchObjectById(id)` → `fetchMemoryWithAllProperties(collection, id)`
+- Verified working in local testing with memories containing `parent_id: ""`
+
+---
+
+## [2.7.2] - 2026-02-17
+
+### Fixed
+
+- **CRITICAL: Fixed Insert API Call Format**: Wrap properties in `{properties: ...}` object
+  - Changed `publicCollection.data.insert(publishedMemory)` to `publicCollection.data.insert({properties: publishedMemory})`
+  - Weaviate insert API expects `{properties: {...}}` format, not properties directly
+  - This is why ALL inserts were creating documents with zero properties
+  - The properties were being ignored because they weren't in the expected format
+
+### Root Cause
+
+- Weaviate client `insert()` API signature: `insert({properties: {...}, vectors?: ..., id?: ...})`
+- We were passing properties directly: `insert(properties)`
+- Weaviate accepted the call but ignored the properties (wrong format)
+- Created documents with UUID but zero properties
+- This explains ALL the empty document issues across all schema approaches
+
+---
+
 ## [2.7.1] - 2026-02-17
 
 ### Fixed

package/dist/server-factory.js

@@ -1948,7 +1948,7 @@ async function handleUpdateMemory(args, userId) {
     const collection = getMemoryCollection(userId);
     let existingMemory;
     try {
-      existingMemory = await collection.query.fetchObjectById(args.memory_id);
+      existingMemory = await fetchMemoryWithAllProperties(collection, args.memory_id);
     } catch (fetchError) {
       const fetchErrorMsg = fetchError instanceof Error ? fetchError.message : String(fetchError);
       logger.error("Failed to fetch memory for update:", {
@@ -4071,7 +4071,9 @@ async function executePublishMemory(request, userId) {
       contentLength: publishedMemory.content?.length || 0,
       titleValue: publishedMemory.title || "NO_TITLE"
     });
-    const result = await publicCollection.data.insert(publishedMemory);
+    const result = await publicCollection.data.insert({
+      properties: publishedMemory
+    });
     logger.info("Memory published successfully", {
       function: "executePublishMemory",
       spaceMemoryId: result,

package/dist/server.js

@@ -2016,7 +2016,7 @@ async function handleUpdateMemory(args, userId) {
     const collection = getMemoryCollection(userId);
     let existingMemory;
     try {
-      existingMemory = await collection.query.fetchObjectById(args.memory_id);
+      existingMemory = await fetchMemoryWithAllProperties(collection, args.memory_id);
     } catch (fetchError) {
       const fetchErrorMsg = fetchError instanceof Error ? fetchError.message : String(fetchError);
       logger.error("Failed to fetch memory for update:", {
@@ -4139,7 +4139,9 @@ async function executePublishMemory(request, userId) {
       contentLength: publishedMemory.content?.length || 0,
       titleValue: publishedMemory.title || "NO_TITLE"
     });
-    const result = await publicCollection.data.insert(publishedMemory);
+    const result = await publicCollection.data.insert({
+      properties: publishedMemory
+    });
     logger.info("Memory published successfully", {
       function: "executePublishMemory",
       spaceMemoryId: result,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@prmichaelsen/remember-mcp",
-  "version": "2.7.1",
+  "version": "2.7.3",
   "description": "Multi-tenant memory system MCP server with vector search and relationships",
   "main": "dist/server.js",
   "type": "module",

package/src/tools/confirm.ts

@@ -263,7 +263,10 @@ async function executePublishMemory(
     });
     
     // Insert directly into unified public collection
-    const result = await publicCollection.data.insert(publishedMemory as any);
+    // CRITICAL: Weaviate insert API expects {properties: {...}}, not the properties directly!
+    const result = await publicCollection.data.insert({
+      properties: publishedMemory,
+    });
     
     logger.info('Memory published successfully', {
       function: 'executePublishMemory',

package/src/tools/update-memory.ts

@@ -5,6 +5,7 @@
 
 import type { Memory, MemoryUpdate } from '../types/memory.js';
 import { getMemoryCollection } from '../weaviate/schema.js';
+import { fetchMemoryWithAllProperties } from '../weaviate/client.js';
 import { logger } from '../utils/logger.js';
 import { handleToolError, withErrorHandling } from '../utils/error-handler.js';
 import { isValidContentType } from '../constants/content-types.js';
@@ -133,9 +134,10 @@ export async function handleUpdateMemory(
     // Get existing memory - fetch ALL properties for replace operation
     // We need the full object to use replace() instead of update()
     // (Weaviate bug: update() only persists if vectorized fields change)
+    // Use fetchMemoryWithAllProperties() to handle schema evolution gracefully
     let existingMemory;
     try {
-      existingMemory = await collection.query.fetchObjectById(args.memory_id);
+      existingMemory = await fetchMemoryWithAllProperties(collection, args.memory_id);
     } catch (fetchError) {
       const fetchErrorMsg = fetchError instanceof Error ? fetchError.message : String(fetchError);
       logger.error('Failed to fetch memory for update:', {