@prmichaelsen/remember-mcp 2.6.9 → 2.6.10

package/CHANGELOG.md

@@ -5,6 +5,20 @@ 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.6.10] - 2026-02-17
+
+### Fixed
+
+- **CRITICAL: `fetchMemoryWithAllProperties()` Graceful Fallback**: Fixed memory content not being copied during publish
+  - Added try-catch wrapper around property fetch with fallback
+  - If fetching with ALL_MEMORY_PROPERTIES fails, falls back to fetching without property specification
+  - Weaviate returns all properties that actually exist on the record when no returnProperties specified
+  - Fixes issue where `remember_confirm` → `executePublishMemory()` only copied comment fields
+  - Published memories now include all content (title, content, tags, etc.)
+  - Prevents data loss during memory publication to shared spaces
+
+---
+
 ## [2.6.9] - 2026-02-17
 
 ### Fixed

package/dist/server-factory.js

@@ -617,9 +617,18 @@ var ALL_MEMORY_PROPERTIES = [
   "moderation_flags"
 ];
 async function fetchMemoryWithAllProperties(collection, memoryId) {
-  return await collection.query.fetchObjectById(memoryId, {
-    returnProperties: ALL_MEMORY_PROPERTIES
-  });
+  try {
+    return await collection.query.fetchObjectById(memoryId, {
+      returnProperties: ALL_MEMORY_PROPERTIES
+    });
+  } catch (error) {
+    logger.warn("Failed to fetch with all properties, falling back to unspecified fetch", {
+      module: "weaviate-client",
+      memoryId,
+      error: error instanceof Error ? error.message : String(error)
+    });
+    return await collection.query.fetchObjectById(memoryId);
+  }
 }
 
 // src/firestore/init.ts

package/dist/server.js

@@ -663,9 +663,18 @@ var ALL_MEMORY_PROPERTIES = [
   "moderation_flags"
 ];
 async function fetchMemoryWithAllProperties(collection, memoryId) {
-  return await collection.query.fetchObjectById(memoryId, {
-    returnProperties: ALL_MEMORY_PROPERTIES
-  });
+  try {
+    return await collection.query.fetchObjectById(memoryId, {
+      returnProperties: ALL_MEMORY_PROPERTIES
+    });
+  } catch (error) {
+    logger.warn("Failed to fetch with all properties, falling back to unspecified fetch", {
+      module: "weaviate-client",
+      memoryId,
+      error: error instanceof Error ? error.message : String(error)
+    });
+    return await collection.query.fetchObjectById(memoryId);
+  }
 }
 
 // src/firestore/init.ts

package/dist/weaviate/client.d.ts

@@ -48,9 +48,14 @@ export declare const ALL_MEMORY_PROPERTIES: readonly ["user_id", "doc_type", "co
  * This utility ensures all memory properties are fetched consistently
  * across all tools, preventing bugs where properties are missing.
  *
+ * NOTE: Some properties may not exist on all records (e.g., old records
+ * created before schema updates). This function handles that gracefully
+ * by falling back to fetching without property specification if the
+ * full property query fails.
+ *
  * @param collection - Weaviate collection
  * @param memoryId - Memory ID to fetch
- * @returns Memory object with all properties
+ * @returns Memory object with all properties that exist on the record
  */
 export declare function fetchMemoryWithAllProperties(collection: any, memoryId: string): Promise<any>;
 /**

package/package.json

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

package/src/weaviate/client.ts

@@ -215,17 +215,36 @@ export const ALL_MEMORY_PROPERTIES = [
  * This utility ensures all memory properties are fetched consistently
  * across all tools, preventing bugs where properties are missing.
  *
+ * NOTE: Some properties may not exist on all records (e.g., old records
+ * created before schema updates). This function handles that gracefully
+ * by falling back to fetching without property specification if the
+ * full property query fails.
+ *
  * @param collection - Weaviate collection
  * @param memoryId - Memory ID to fetch
- * @returns Memory object with all properties
+ * @returns Memory object with all properties that exist on the record
  */
 export async function fetchMemoryWithAllProperties(
   collection: any,
   memoryId: string
 ) {
-  return await collection.query.fetchObjectById(memoryId, {
-    returnProperties: ALL_MEMORY_PROPERTIES,
-  });
+  try {
+    // Try to fetch with all properties specified
+    return await collection.query.fetchObjectById(memoryId, {
+      returnProperties: ALL_MEMORY_PROPERTIES,
+    });
+  } catch (error) {
+    // If that fails (e.g., property doesn't exist on this record),
+    // fetch without specifying properties - Weaviate will return
+    // all properties that actually exist on the record
+    logger.warn('Failed to fetch with all properties, falling back to unspecified fetch', {
+      module: 'weaviate-client',
+      memoryId,
+      error: error instanceof Error ? error.message : String(error),
+    });
+    
+    return await collection.query.fetchObjectById(memoryId);
+  }
 }
 
 /**