@prmichaelsen/remember-mcp 2.7.2 → 2.7.5

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.4] - 2026-02-17
+
+### Changed
+
+- **Enhanced Vectorization**: Added `title` and `summary` to vectorized source properties
+  - User memory schema: Now vectorizes `content`, `title`, `summary`, and `observation`
+  - Space memory schema: Now vectorizes `content`, `title`, `summary`, and `observation`
+  - Improves semantic search by including titles and summaries in vector embeddings
+  - Note: `title` and `summary` are optional fields, handled gracefully by vectorizer
+
+### Technical Details
+
+- Modified: `src/weaviate/schema.ts` (line 49)
+- Modified: `src/weaviate/space-schema.ts` (line 101)
+- Changed `sourceProperties` from `['content', 'observation']` to `['content', 'title', 'summary', 'observation']`
+- Applies to all new collections created after this version
+- Existing collections retain their original vectorizer configuration
+
+---
+
+## [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

@@ -699,8 +699,9 @@ async function createMemoryCollection(userId) {
     // Vectorizer configuration
     vectorizers: weaviate2.configure.vectorizer.text2VecOpenAI({
       model: "text-embedding-3-small",
-      // Vectorize both memory content and relationship observations
-      sourceProperties: ["content", "observation"]
+      // Vectorize content, title, summary, and observation for semantic search
+      // Note: title and summary are optional fields
+      sourceProperties: ["content", "title", "summary", "observation"]
     }),
     properties: [
       // Discriminator
@@ -1948,7 +1949,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

@@ -767,8 +767,9 @@ async function createMemoryCollection(userId) {
     // Vectorizer configuration
     vectorizers: weaviate2.configure.vectorizer.text2VecOpenAI({
       model: "text-embedding-3-small",
-      // Vectorize both memory content and relationship observations
-      sourceProperties: ["content", "observation"]
+      // Vectorize content, title, summary, and observation for semantic search
+      // Note: title and summary are optional fields
+      sourceProperties: ["content", "title", "summary", "observation"]
     }),
     properties: [
       // Discriminator
@@ -2016,7 +2017,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.5",
   "description": "Multi-tenant memory system MCP server with vector search and relationships",
   "main": "dist/server.js",
   "type": "module",
@@ -50,6 +50,7 @@
     "@modelcontextprotocol/sdk": "^1.0.4",
     "@prmichaelsen/firebase-admin-sdk-v8": "^2.2.0",
     "@prmichaelsen/mcp-auth": "^7.0.4",
+    "@prmichaelsen/remember-mcp": "^2.7.3",
     "dotenv": "^16.4.5",
     "weaviate-client": "^3.2.0"
   },

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

package/src/weaviate/schema.ts

@@ -45,8 +45,9 @@ export async function createMemoryCollection(userId: string): Promise<void> {
     // Vectorizer configuration
     vectorizers: weaviate.configure.vectorizer.text2VecOpenAI({
       model: 'text-embedding-3-small',
-      // Vectorize both memory content and relationship observations
-      sourceProperties: ['content', 'observation'],
+      // Vectorize content, title, summary, and observation for semantic search
+      // Note: title and summary are optional fields
+      sourceProperties: ['content', 'title', 'summary', 'observation'],
     }),
 
     properties: [

package/src/weaviate/space-schema.ts

@@ -97,8 +97,9 @@ async function createSpaceCollection(
     // Vectorizer configuration
     vectorizers: weaviate.configure.vectorizer.text2VecOpenAI({
       model: 'text-embedding-3-small',
-      // Vectorize content for semantic search
-      sourceProperties: ['content', 'observation'],
+      // Vectorize content, title, summary, and observation for semantic search
+      // Note: title and summary are optional fields
+      sourceProperties: ['content', 'title', 'summary', 'observation'],
     }),
 
     properties: [