@prmichaelsen/remember-mcp 2.7.2 → 2.7.3

package/CHANGELOG.md

@@ -5,6 +5,34 @@ 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

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:", {

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:", {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@prmichaelsen/remember-mcp",
-  "version": "2.7.2",
+  "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/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:', {