@prmichaelsen/remember-mcp 2.7.8 → 2.7.10

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.9] - 2026-02-17
+
+### Added
+
+- **Bidirectional Linking for Published Memories**
+  - Original memories now store `space_memory_id` after publishing
+  - Enables correlation between source memory and published space memory
+  - Added automatic update of original memory after successful publish
+  - Non-critical update - publish still succeeds if update fails
+  - Improves traceability and enables future features (unpublish, sync)
+
+### Technical Details
+
+- Modified: `src/tools/confirm.ts` (lines 277-296)
+- Added `userCollection.data.update()` call after publish
+- Updates original memory with `space_memory_id` field
+- Includes error handling with warning log (non-blocking)
+- Maintains backward compatibility
+
+---
+
+## [2.7.8] - 2026-02-17
+
+### Fixed
+
+- **CRITICAL: Fixed Shared Space Discovery Bug**
+  - Search and query tools now correctly filter by `doc_type: 'memory'` instead of `'space_memory'`
+  - The `space_memory` concept was removed from the architecture
+  - Search tools were filtering for `doc_type: 'space_memory'` but published memories had `doc_type: 'memory'`
+  - Now all users can discover memories published to shared spaces like The Void
+  - Root cause: Lines 159 in `search-space.ts` and 147 in `query-space.ts` filtered for wrong doc_type
+  - Fix: Changed filters from `'space_memory'` to `'memory'` to match actual data
+
+### Technical Details
+
+- Modified: `src/tools/search-space.ts` (line 159)
+- Modified: `src/tools/query-space.ts` (line 147)
+- Changed filter from `.equal('space_memory')` to `.equal('memory')`
+- Published memories keep `doc_type: 'memory'` (as intended)
+
+### Impact
+
+- **Fixes**: Shared space discovery now works correctly
+- **Unblocks**: Core shared spaces functionality (The Void)
+- **Note**: No data migration needed - existing published memories are correct
+
+---
+
 ## [2.7.7] - 2026-02-17
 
 ### Fixed

package/agent/progress.yaml

@@ -2,11 +2,11 @@
 
 project:
   name: remember-mcp
-  version: 2.6.3
+  version: 2.7.8
   started: 2026-02-11
   status: in_progress
   current_milestone: M12
-  last_updated: 2026-02-16
+  last_updated: 2026-02-17
 
 milestones:
   - id: M1
@@ -523,6 +523,19 @@ progress:
   overall: 50%
 
 recent_work:
+  - date: 2026-02-17
+    description: CRITICAL BUG FIX - Shared Space Discovery Now Works (v2.7.8)
+    items:
+      - 🐛 CRITICAL: Fixed shared space discovery bug
+      - ✅ Search/query tools now filter by doc_type: 'memory' (was 'space_memory')
+      - ✅ space_memory concept was removed from architecture
+      - ✅ All users can now discover memories in The Void
+      - ✅ Root cause: search-space.ts and query-space.ts filtered for wrong doc_type
+      - ✅ Changed filters from 'space_memory' to 'memory'
+      - ✅ Build successful, all 81 tests passing
+      - ✅ No data migration needed - existing memories are correct
+      - 📋 Released v2.7.8 with critical bug fix
+      
   - date: 2026-02-17
     description: Fixed Build Error & Completed Task 66 - Comment Field Initialization
     items:
@@ -544,20 +557,25 @@ recent_work:
       - 📋 Ready for M12 completion: Tasks 58-59 remaining
       
   - date: 2026-02-17
-    description: ACP Initialization Complete - Project Context Loaded
+    description: ACP Initialization Complete - Ready for M12 Completion
     items:
       - ✅ ACP initialization workflow executed via @acp.init command
       - ⚠️  ACP updates available (v1.4.3 with improved script color output)
-      - ✅ All agent documentation reviewed (23 design docs, 11 milestones, 66 tasks, 5 patterns)
+      - ✅ All agent documentation reviewed (30 design docs, 12 milestones, 67 tasks, 7 patterns)
       - ✅ Project status verified: M1-M4, M10-M11 complete (100%), M12 in progress (60%)
-      - ✅ Version 2.6.7 confirmed (latest release)
-      - ✅ 53 unit tests passing (1 skipped integration test)
-      - ⚠️  TypeScript compilation error: Missing 'poetry' in content-types.ts
+      - ✅ Version 2.7.7 confirmed (latest release)
+      - ✅ 81 unit tests passing (1 skipped integration test)
+      - ✅ Test coverage: 33.12% overall
+      - ✅ TypeScript compiles without errors
+      - ✅ Build successful (dual export: server + factory)
       - ✅ All 17 MCP tools implemented and working
       - ✅ M12 Comment System: 3/5 tasks complete (schema, search, query updates)
+      - ✅ Task 66 completed: Comment fields initialized in create/update tools
       - 📋 Remaining M12 tasks: Task 58 (unit tests), Task 59 (documentation)
-      - 📋 Build blocked by TypeScript error in content-types.ts
-      - 📋 Need to fix 'poetry' content type metadata
+      - 📋 Ready to complete M12 milestone
+      - 📋 Multi-tenant architecture with per-user isolation operational
+      - 📋 Vector search with Weaviate + metadata with Firestore working
+      - 📋 Shared spaces with multi-space support functional
       
   - date: 2026-02-16
     description: Comment System, Structured Logging, and Safety Guidelines (v2.6.0-v2.6.2)

package/dist/server-factory.js

@@ -4330,7 +4330,7 @@ async function executePublishMemory(request, userId) {
       attribution: "user",
       // Merge additional tags with original tags
       tags: [...originalTags, ...additionalTags]
-      // Keep doc_type as 'memory' (don't change to 'space_memory')
+      // Keep doc_type as 'memory' (space_memory concept was removed)
       // Keep original created_at, updated_at, version (don't overwrite)
     };
     logger.info("Inserting memory into Memory_public", {
@@ -4355,6 +4355,26 @@ async function executePublishMemory(request, userId) {
       spaceMemoryId: result,
       spaces: request.payload.spaces
     });
+    try {
+      await userCollection.data.update({
+        id: request.payload.memory_id,
+        properties: {
+          space_memory_id: result
+        }
+      });
+      logger.info("Updated original memory with space_memory_id", {
+        function: "executePublishMemory",
+        memoryId: request.payload.memory_id,
+        spaceMemoryId: result
+      });
+    } catch (updateError) {
+      logger.warn("Failed to update original memory with space_memory_id", {
+        function: "executePublishMemory",
+        memoryId: request.payload.memory_id,
+        spaceMemoryId: result,
+        error: updateError instanceof Error ? updateError.message : String(updateError)
+      });
+    }
     return JSON.stringify(
       {
         success: true,
@@ -4542,7 +4562,7 @@ async function handleSearchSpace(args, userId) {
     const publicCollection = await ensurePublicCollection(weaviateClient);
     const filterList = [];
     filterList.push(publicCollection.filter.byProperty("spaces").containsAny(args.spaces));
-    filterList.push(publicCollection.filter.byProperty("doc_type").equal("space_memory"));
+    filterList.push(publicCollection.filter.byProperty("doc_type").equal("memory"));
     if (args.content_type) {
       filterList.push(publicCollection.filter.byProperty("type").equal(args.content_type));
     }
@@ -4698,7 +4718,7 @@ async function handleQuerySpace(args, userId) {
     const publicCollection = await ensurePublicCollection(weaviateClient);
     const filterList = [];
     filterList.push(publicCollection.filter.byProperty("spaces").containsAny(args.spaces));
-    filterList.push(publicCollection.filter.byProperty("doc_type").equal("space_memory"));
+    filterList.push(publicCollection.filter.byProperty("doc_type").equal("memory"));
     if (args.content_type) {
       filterList.push(publicCollection.filter.byProperty("type").equal(args.content_type));
     }

package/dist/server.js

@@ -4398,7 +4398,7 @@ async function executePublishMemory(request, userId) {
       attribution: "user",
       // Merge additional tags with original tags
       tags: [...originalTags, ...additionalTags]
-      // Keep doc_type as 'memory' (don't change to 'space_memory')
+      // Keep doc_type as 'memory' (space_memory concept was removed)
       // Keep original created_at, updated_at, version (don't overwrite)
     };
     logger.info("Inserting memory into Memory_public", {
@@ -4423,6 +4423,26 @@ async function executePublishMemory(request, userId) {
       spaceMemoryId: result,
       spaces: request.payload.spaces
     });
+    try {
+      await userCollection.data.update({
+        id: request.payload.memory_id,
+        properties: {
+          space_memory_id: result
+        }
+      });
+      logger.info("Updated original memory with space_memory_id", {
+        function: "executePublishMemory",
+        memoryId: request.payload.memory_id,
+        spaceMemoryId: result
+      });
+    } catch (updateError) {
+      logger.warn("Failed to update original memory with space_memory_id", {
+        function: "executePublishMemory",
+        memoryId: request.payload.memory_id,
+        spaceMemoryId: result,
+        error: updateError instanceof Error ? updateError.message : String(updateError)
+      });
+    }
     return JSON.stringify(
       {
         success: true,
@@ -4610,7 +4630,7 @@ async function handleSearchSpace(args, userId) {
     const publicCollection = await ensurePublicCollection(weaviateClient);
     const filterList = [];
     filterList.push(publicCollection.filter.byProperty("spaces").containsAny(args.spaces));
-    filterList.push(publicCollection.filter.byProperty("doc_type").equal("space_memory"));
+    filterList.push(publicCollection.filter.byProperty("doc_type").equal("memory"));
     if (args.content_type) {
       filterList.push(publicCollection.filter.byProperty("type").equal(args.content_type));
     }
@@ -4766,7 +4786,7 @@ async function handleQuerySpace(args, userId) {
     const publicCollection = await ensurePublicCollection(weaviateClient);
     const filterList = [];
     filterList.push(publicCollection.filter.byProperty("spaces").containsAny(args.spaces));
-    filterList.push(publicCollection.filter.byProperty("doc_type").equal("space_memory"));
+    filterList.push(publicCollection.filter.byProperty("doc_type").equal("memory"));
     if (args.content_type) {
       filterList.push(publicCollection.filter.byProperty("type").equal(args.content_type));
     }

package/package.json

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

package/src/tools/confirm.ts

@@ -243,7 +243,7 @@ async function executePublishMemory(
       attribution: 'user' as const,
       // Merge additional tags with original tags
       tags: [...originalTags, ...additionalTags],
-      // Keep doc_type as 'memory' (don't change to 'space_memory')
+      // Keep doc_type as 'memory' (space_memory concept was removed)
       // Keep original created_at, updated_at, version (don't overwrite)
     };
 
@@ -274,6 +274,30 @@ async function executePublishMemory(
       spaces: request.payload.spaces,
     });
 
+    // Update original memory with space_memory_id for bidirectional linking
+    try {
+      await userCollection.data.update({
+        id: request.payload.memory_id,
+        properties: {
+          space_memory_id: result,
+        },
+      });
+      
+      logger.info('Updated original memory with space_memory_id', {
+        function: 'executePublishMemory',
+        memoryId: request.payload.memory_id,
+        spaceMemoryId: result,
+      });
+    } catch (updateError) {
+      logger.warn('Failed to update original memory with space_memory_id', {
+        function: 'executePublishMemory',
+        memoryId: request.payload.memory_id,
+        spaceMemoryId: result,
+        error: updateError instanceof Error ? updateError.message : String(updateError),
+      });
+      // Don't fail the publish if this update fails - it's not critical
+    }
+
     // Return minimal response with spaces array
     return JSON.stringify(
       {

package/src/tools/query-space.ts

@@ -143,8 +143,8 @@ export async function handleQuerySpace(
     // Filter by spaces array (memory must be in at least one requested space)
     filterList.push(publicCollection.filter.byProperty('spaces').containsAny(args.spaces));
 
-    // Filter by doc_type (space_memory)
-    filterList.push(publicCollection.filter.byProperty('doc_type').equal('space_memory'));
+    // Filter by doc_type (memory) - space_memory concept was removed
+    filterList.push(publicCollection.filter.byProperty('doc_type').equal('memory'));
 
     // Apply content type filter
     if (args.content_type) {

package/src/tools/search-space.ts

@@ -155,8 +155,8 @@ export async function handleSearchSpace(
     // Filter by spaces array (memory must be in at least one requested space)
     filterList.push(publicCollection.filter.byProperty('spaces').containsAny(args.spaces));
 
-    // Filter by doc_type (space_memory)
-    filterList.push(publicCollection.filter.byProperty('doc_type').equal('space_memory'));
+    // Filter by doc_type (memory) - space_memory concept was removed
+    filterList.push(publicCollection.filter.byProperty('doc_type').equal('memory'));
 
     // Apply content type filter
     if (args.content_type) {