@prmichaelsen/remember-mcp 2.6.9 → 2.6.11

package/CHANGELOG.md

@@ -5,6 +5,33 @@ 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.11] - 2026-02-17
+
+### Changed
+
+- **Enhanced Logging for `executePublishMemory()`**: Added detailed property logging
+  - Logs all property keys fetched from original memory
+  - Logs property keys being inserted into public collection
+  - Logs content length, title, and other key fields
+  - Changed fetch result log from debug to info level
+  - Helps diagnose if memory content is being fetched and copied correctly
+
+---
+
+## [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
@@ -4149,14 +4158,20 @@ async function executePublishMemory(request, userId) {
       userCollection,
       request.payload.memory_id
     );
-    logger.debug("Original memory fetch result", {
+    logger.info("Original memory fetch result", {
       function: "executePublishMemory",
       found: !!originalMemory,
       memoryId: request.payload.memory_id,
       hasProperties: !!originalMemory?.properties,
       propertyCount: originalMemory?.properties ? Object.keys(originalMemory.properties).length : 0,
+      propertyKeys: originalMemory?.properties ? Object.keys(originalMemory.properties) : [],
       hasTitle: !!originalMemory?.properties?.title,
-      hasContent: !!originalMemory?.properties?.content
+      hasContent: !!originalMemory?.properties?.content,
+      hasUserId: !!originalMemory?.properties?.user_id,
+      hasTags: !!originalMemory?.properties?.tags,
+      hasWeight: !!originalMemory?.properties?.weight,
+      contentLength: originalMemory?.properties?.content?.length || 0,
+      titleValue: originalMemory?.properties?.title || "NO_TITLE"
     });
     if (!originalMemory) {
       logger.info("Original memory not found", {
@@ -4224,7 +4239,13 @@ async function executePublishMemory(request, userId) {
       spaceCount: request.payload.spaces?.length || 0,
       memoryId: request.payload.memory_id,
       hasUserId: !!publishedMemory.user_id,
-      hasAuthorId: !!publishedMemory.author_id
+      hasAuthorId: !!publishedMemory.author_id,
+      publishedMemoryKeys: Object.keys(publishedMemory),
+      publishedMemoryKeyCount: Object.keys(publishedMemory).length,
+      hasContent: !!publishedMemory.content,
+      hasTitle: !!publishedMemory.title,
+      contentLength: publishedMemory.content?.length || 0,
+      titleValue: publishedMemory.title || "NO_TITLE"
     });
     const result = await publicCollection.data.insert(publishedMemory);
     logger.info("Memory published successfully", {

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
@@ -4217,14 +4226,20 @@ async function executePublishMemory(request, userId) {
       userCollection,
       request.payload.memory_id
     );
-    logger.debug("Original memory fetch result", {
+    logger.info("Original memory fetch result", {
       function: "executePublishMemory",
       found: !!originalMemory,
       memoryId: request.payload.memory_id,
       hasProperties: !!originalMemory?.properties,
       propertyCount: originalMemory?.properties ? Object.keys(originalMemory.properties).length : 0,
+      propertyKeys: originalMemory?.properties ? Object.keys(originalMemory.properties) : [],
       hasTitle: !!originalMemory?.properties?.title,
-      hasContent: !!originalMemory?.properties?.content
+      hasContent: !!originalMemory?.properties?.content,
+      hasUserId: !!originalMemory?.properties?.user_id,
+      hasTags: !!originalMemory?.properties?.tags,
+      hasWeight: !!originalMemory?.properties?.weight,
+      contentLength: originalMemory?.properties?.content?.length || 0,
+      titleValue: originalMemory?.properties?.title || "NO_TITLE"
     });
     if (!originalMemory) {
       logger.info("Original memory not found", {
@@ -4292,7 +4307,13 @@ async function executePublishMemory(request, userId) {
       spaceCount: request.payload.spaces?.length || 0,
       memoryId: request.payload.memory_id,
       hasUserId: !!publishedMemory.user_id,
-      hasAuthorId: !!publishedMemory.author_id
+      hasAuthorId: !!publishedMemory.author_id,
+      publishedMemoryKeys: Object.keys(publishedMemory),
+      publishedMemoryKeyCount: Object.keys(publishedMemory).length,
+      hasContent: !!publishedMemory.content,
+      hasTitle: !!publishedMemory.title,
+      contentLength: publishedMemory.content?.length || 0,
+      titleValue: publishedMemory.title || "NO_TITLE"
     });
     const result = await publicCollection.data.insert(publishedMemory);
     logger.info("Memory published successfully", {

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.11",
   "description": "Multi-tenant memory system MCP server with vector search and relationships",
   "main": "dist/server.js",
   "type": "module",

package/src/tools/confirm.ts

@@ -156,14 +156,20 @@ async function executePublishMemory(
       request.payload.memory_id
     );
     
-    logger.debug('Original memory fetch result', {
+    logger.info('Original memory fetch result', {
       function: 'executePublishMemory',
       found: !!originalMemory,
       memoryId: request.payload.memory_id,
       hasProperties: !!originalMemory?.properties,
       propertyCount: originalMemory?.properties ? Object.keys(originalMemory.properties).length : 0,
+      propertyKeys: originalMemory?.properties ? Object.keys(originalMemory.properties) : [],
       hasTitle: !!originalMemory?.properties?.title,
       hasContent: !!originalMemory?.properties?.content,
+      hasUserId: !!originalMemory?.properties?.user_id,
+      hasTags: !!originalMemory?.properties?.tags,
+      hasWeight: !!originalMemory?.properties?.weight,
+      contentLength: originalMemory?.properties?.content?.length || 0,
+      titleValue: originalMemory?.properties?.title || 'NO_TITLE',
     });
 
     if (!originalMemory) {
@@ -246,6 +252,12 @@ async function executePublishMemory(
       memoryId: request.payload.memory_id,
       hasUserId: !!(publishedMemory as any).user_id,
       hasAuthorId: !!publishedMemory.author_id,
+      publishedMemoryKeys: Object.keys(publishedMemory),
+      publishedMemoryKeyCount: Object.keys(publishedMemory).length,
+      hasContent: !!publishedMemory.content,
+      hasTitle: !!publishedMemory.title,
+      contentLength: publishedMemory.content?.length || 0,
+      titleValue: publishedMemory.title || 'NO_TITLE',
     });
     
     // Insert directly into unified public collection

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);
+  }
 }
 
 /**