@prmichaelsen/remember-mcp 1.0.4 → 1.0.5

package/dist/server-factory.js

@@ -1407,103 +1407,6 @@ async function handleCreateMemory(args, userId, context) {
   }
 }
 
-// src/utils/weaviate-filters.ts
-import { Filters } from "weaviate-client";
-function buildCombinedSearchFilters(collection, filters) {
-  const memoryFilters = buildDocTypeFilters(collection, "memory", filters);
-  const relationshipFilters = buildDocTypeFilters(collection, "relationship", filters);
-  const validFilters = [memoryFilters, relationshipFilters].filter((f) => f !== void 0 && f !== null);
-  if (validFilters.length === 0) {
-    return void 0;
-  } else if (validFilters.length === 1) {
-    return validFilters[0];
-  } else {
-    return combineFiltersWithOr(validFilters);
-  }
-}
-function buildDocTypeFilters(collection, docType, filters) {
-  const filterList = [];
-  filterList.push(
-    collection.filter.byProperty("doc_type").equal(docType)
-  );
-  if (docType === "memory" && filters?.types && filters.types.length > 0) {
-    if (filters.types.length === 1) {
-      filterList.push(
-        collection.filter.byProperty("type").equal(filters.types[0])
-      );
-    } else {
-      filterList.push(
-        collection.filter.byProperty("type").containsAny(filters.types)
-      );
-    }
-  }
-  if (filters?.weight_min !== void 0) {
-    filterList.push(
-      collection.filter.byProperty("weight").greaterThanOrEqual(filters.weight_min)
-    );
-  }
-  if (filters?.weight_max !== void 0) {
-    filterList.push(
-      collection.filter.byProperty("weight").lessThanOrEqual(filters.weight_max)
-    );
-  }
-  if (filters?.trust_min !== void 0) {
-    filterList.push(
-      collection.filter.byProperty("trust").greaterThanOrEqual(filters.trust_min)
-    );
-  }
-  if (filters?.trust_max !== void 0) {
-    filterList.push(
-      collection.filter.byProperty("trust").lessThanOrEqual(filters.trust_max)
-    );
-  }
-  if (filters?.date_from) {
-    filterList.push(
-      collection.filter.byProperty("created_at").greaterThanOrEqual(new Date(filters.date_from))
-    );
-  }
-  if (filters?.date_to) {
-    filterList.push(
-      collection.filter.byProperty("created_at").lessThanOrEqual(new Date(filters.date_to))
-    );
-  }
-  if (filters?.tags && filters.tags.length > 0) {
-    if (filters.tags.length === 1) {
-      filterList.push(
-        collection.filter.byProperty("tags").containsAny([filters.tags[0]])
-      );
-    } else {
-      filterList.push(
-        collection.filter.byProperty("tags").containsAny(filters.tags)
-      );
-    }
-  }
-  return combineFiltersWithAnd(filterList);
-}
-function buildMemoryOnlyFilters(collection, filters) {
-  return buildDocTypeFilters(collection, "memory", filters);
-}
-function combineFiltersWithAnd(filters) {
-  const validFilters = filters.filter((f) => f !== void 0 && f !== null);
-  if (validFilters.length === 0) {
-    return void 0;
-  }
-  if (validFilters.length === 1) {
-    return validFilters[0];
-  }
-  return Filters.and(...validFilters);
-}
-function combineFiltersWithOr(filters) {
-  const validFilters = filters.filter((f) => f !== void 0 && f !== null);
-  if (validFilters.length === 0) {
-    return void 0;
-  }
-  if (validFilters.length === 1) {
-    return validFilters[0];
-  }
-  return Filters.or(...validFilters);
-}
-
 // src/tools/search-memory.ts
 var searchMemoryTool = {
   name: "remember_search_memory",
@@ -1607,19 +1510,15 @@ async function handleSearchMemory(args, userId) {
     const alpha = args.alpha ?? 0.7;
     const limit = args.limit ?? 10;
     const offset = args.offset ?? 0;
-    const filters = includeRelationships ? buildCombinedSearchFilters(collection, args.filters) : buildMemoryOnlyFilters(collection, args.filters);
     const searchOptions = {
       alpha,
       limit: limit + offset
       // Get extra for offset
     };
-    if (filters) {
-      searchOptions.filters = filters;
-    }
     logger.info("Weaviate query", {
       query: args.query,
-      searchOptions: JSON.stringify(searchOptions, null, 2),
-      hasFilters: !!filters
+      searchOptions: JSON.stringify(searchOptions, null, 2)
+      // hasFilters: !!filters,
     });
     const results = await collection.query.hybrid(args.query, searchOptions);
     const paginatedResults = results.objects.slice(offset);
@@ -1652,8 +1551,9 @@ async function handleSearchMemory(args, userId) {
     });
     return JSON.stringify(searchResult, null, 2);
   } catch (error) {
-    logger.error("Failed to search memories:", error);
-    throw new Error(`Failed to search memories: ${error instanceof Error ? error.message : String(error)}`);
+    const errorMessage = error instanceof Error ? error.message : String(error);
+    logger.error("Failed to search memories:", { error: errorMessage, userId, query: args.query });
+    throw new Error(`Failed to search memories: ${errorMessage}`);
   }
 }
 
@@ -2012,6 +1912,100 @@ async function handleFindSimilar(args, userId) {
   }
 }
 
+// src/utils/weaviate-filters.ts
+import { Filters } from "weaviate-client";
+function buildCombinedSearchFilters(collection, filters) {
+  const memoryFilters = buildDocTypeFilters(collection, "memory", filters);
+  const relationshipFilters = buildDocTypeFilters(collection, "relationship", filters);
+  const validFilters = [memoryFilters, relationshipFilters].filter((f) => f !== void 0 && f !== null);
+  if (validFilters.length === 0) {
+    return void 0;
+  } else if (validFilters.length === 1) {
+    return validFilters[0];
+  } else {
+    return combineFiltersWithOr(validFilters);
+  }
+}
+function buildDocTypeFilters(collection, docType, filters) {
+  const filterList = [];
+  filterList.push(
+    collection.filter.byProperty("doc_type").equal(docType)
+  );
+  if (docType === "memory" && filters?.types && filters.types.length > 0) {
+    if (filters.types.length === 1) {
+      filterList.push(
+        collection.filter.byProperty("type").equal(filters.types[0])
+      );
+    } else {
+      filterList.push(
+        collection.filter.byProperty("type").containsAny(filters.types)
+      );
+    }
+  }
+  if (filters?.weight_min !== void 0) {
+    filterList.push(
+      collection.filter.byProperty("weight").greaterThanOrEqual(filters.weight_min)
+    );
+  }
+  if (filters?.weight_max !== void 0) {
+    filterList.push(
+      collection.filter.byProperty("weight").lessThanOrEqual(filters.weight_max)
+    );
+  }
+  if (filters?.trust_min !== void 0) {
+    filterList.push(
+      collection.filter.byProperty("trust").greaterThanOrEqual(filters.trust_min)
+    );
+  }
+  if (filters?.trust_max !== void 0) {
+    filterList.push(
+      collection.filter.byProperty("trust").lessThanOrEqual(filters.trust_max)
+    );
+  }
+  if (filters?.date_from) {
+    filterList.push(
+      collection.filter.byProperty("created_at").greaterThanOrEqual(new Date(filters.date_from))
+    );
+  }
+  if (filters?.date_to) {
+    filterList.push(
+      collection.filter.byProperty("created_at").lessThanOrEqual(new Date(filters.date_to))
+    );
+  }
+  if (filters?.tags && filters.tags.length > 0) {
+    if (filters.tags.length === 1) {
+      filterList.push(
+        collection.filter.byProperty("tags").containsAny([filters.tags[0]])
+      );
+    } else {
+      filterList.push(
+        collection.filter.byProperty("tags").containsAny(filters.tags)
+      );
+    }
+  }
+  return combineFiltersWithAnd(filterList);
+}
+function combineFiltersWithAnd(filters) {
+  const validFilters = filters.filter((f) => f !== void 0 && f !== null);
+  if (validFilters.length === 0) {
+    return void 0;
+  }
+  if (validFilters.length === 1) {
+    return validFilters[0];
+  }
+  return Filters.and(...validFilters);
+}
+function combineFiltersWithOr(filters) {
+  const validFilters = filters.filter((f) => f !== void 0 && f !== null);
+  if (validFilters.length === 0) {
+    return void 0;
+  }
+  if (validFilters.length === 1) {
+    return validFilters[0];
+  }
+  return Filters.or(...validFilters);
+}
+
 // src/tools/query-memory.ts
 var queryMemoryTool = {
   name: "remember_query_memory",

package/dist/server.js

@@ -1336,103 +1336,6 @@ async function handleCreateMemory(args, userId, context) {
   }
 }
 
-// src/utils/weaviate-filters.ts
-import { Filters } from "weaviate-client";
-function buildCombinedSearchFilters(collection, filters) {
-  const memoryFilters = buildDocTypeFilters(collection, "memory", filters);
-  const relationshipFilters = buildDocTypeFilters(collection, "relationship", filters);
-  const validFilters = [memoryFilters, relationshipFilters].filter((f) => f !== void 0 && f !== null);
-  if (validFilters.length === 0) {
-    return void 0;
-  } else if (validFilters.length === 1) {
-    return validFilters[0];
-  } else {
-    return combineFiltersWithOr(validFilters);
-  }
-}
-function buildDocTypeFilters(collection, docType, filters) {
-  const filterList = [];
-  filterList.push(
-    collection.filter.byProperty("doc_type").equal(docType)
-  );
-  if (docType === "memory" && filters?.types && filters.types.length > 0) {
-    if (filters.types.length === 1) {
-      filterList.push(
-        collection.filter.byProperty("type").equal(filters.types[0])
-      );
-    } else {
-      filterList.push(
-        collection.filter.byProperty("type").containsAny(filters.types)
-      );
-    }
-  }
-  if (filters?.weight_min !== void 0) {
-    filterList.push(
-      collection.filter.byProperty("weight").greaterThanOrEqual(filters.weight_min)
-    );
-  }
-  if (filters?.weight_max !== void 0) {
-    filterList.push(
-      collection.filter.byProperty("weight").lessThanOrEqual(filters.weight_max)
-    );
-  }
-  if (filters?.trust_min !== void 0) {
-    filterList.push(
-      collection.filter.byProperty("trust").greaterThanOrEqual(filters.trust_min)
-    );
-  }
-  if (filters?.trust_max !== void 0) {
-    filterList.push(
-      collection.filter.byProperty("trust").lessThanOrEqual(filters.trust_max)
-    );
-  }
-  if (filters?.date_from) {
-    filterList.push(
-      collection.filter.byProperty("created_at").greaterThanOrEqual(new Date(filters.date_from))
-    );
-  }
-  if (filters?.date_to) {
-    filterList.push(
-      collection.filter.byProperty("created_at").lessThanOrEqual(new Date(filters.date_to))
-    );
-  }
-  if (filters?.tags && filters.tags.length > 0) {
-    if (filters.tags.length === 1) {
-      filterList.push(
-        collection.filter.byProperty("tags").containsAny([filters.tags[0]])
-      );
-    } else {
-      filterList.push(
-        collection.filter.byProperty("tags").containsAny(filters.tags)
-      );
-    }
-  }
-  return combineFiltersWithAnd(filterList);
-}
-function buildMemoryOnlyFilters(collection, filters) {
-  return buildDocTypeFilters(collection, "memory", filters);
-}
-function combineFiltersWithAnd(filters) {
-  const validFilters = filters.filter((f) => f !== void 0 && f !== null);
-  if (validFilters.length === 0) {
-    return void 0;
-  }
-  if (validFilters.length === 1) {
-    return validFilters[0];
-  }
-  return Filters.and(...validFilters);
-}
-function combineFiltersWithOr(filters) {
-  const validFilters = filters.filter((f) => f !== void 0 && f !== null);
-  if (validFilters.length === 0) {
-    return void 0;
-  }
-  if (validFilters.length === 1) {
-    return validFilters[0];
-  }
-  return Filters.or(...validFilters);
-}
-
 // src/tools/search-memory.ts
 var searchMemoryTool = {
   name: "remember_search_memory",
@@ -1536,19 +1439,15 @@ async function handleSearchMemory(args, userId) {
     const alpha = args.alpha ?? 0.7;
     const limit = args.limit ?? 10;
     const offset = args.offset ?? 0;
-    const filters = includeRelationships ? buildCombinedSearchFilters(collection, args.filters) : buildMemoryOnlyFilters(collection, args.filters);
     const searchOptions = {
       alpha,
       limit: limit + offset
       // Get extra for offset
     };
-    if (filters) {
-      searchOptions.filters = filters;
-    }
     logger.info("Weaviate query", {
       query: args.query,
-      searchOptions: JSON.stringify(searchOptions, null, 2),
-      hasFilters: !!filters
+      searchOptions: JSON.stringify(searchOptions, null, 2)
+      // hasFilters: !!filters,
     });
     const results = await collection.query.hybrid(args.query, searchOptions);
     const paginatedResults = results.objects.slice(offset);
@@ -1581,8 +1480,9 @@ async function handleSearchMemory(args, userId) {
     });
     return JSON.stringify(searchResult, null, 2);
   } catch (error) {
-    logger.error("Failed to search memories:", error);
-    throw new Error(`Failed to search memories: ${error instanceof Error ? error.message : String(error)}`);
+    const errorMessage = error instanceof Error ? error.message : String(error);
+    logger.error("Failed to search memories:", { error: errorMessage, userId, query: args.query });
+    throw new Error(`Failed to search memories: ${errorMessage}`);
   }
 }
 
@@ -1941,6 +1841,100 @@ async function handleFindSimilar(args, userId) {
   }
 }
 
+// src/utils/weaviate-filters.ts
+import { Filters } from "weaviate-client";
+function buildCombinedSearchFilters(collection, filters) {
+  const memoryFilters = buildDocTypeFilters(collection, "memory", filters);
+  const relationshipFilters = buildDocTypeFilters(collection, "relationship", filters);
+  const validFilters = [memoryFilters, relationshipFilters].filter((f) => f !== void 0 && f !== null);
+  if (validFilters.length === 0) {
+    return void 0;
+  } else if (validFilters.length === 1) {
+    return validFilters[0];
+  } else {
+    return combineFiltersWithOr(validFilters);
+  }
+}
+function buildDocTypeFilters(collection, docType, filters) {
+  const filterList = [];
+  filterList.push(
+    collection.filter.byProperty("doc_type").equal(docType)
+  );
+  if (docType === "memory" && filters?.types && filters.types.length > 0) {
+    if (filters.types.length === 1) {
+      filterList.push(
+        collection.filter.byProperty("type").equal(filters.types[0])
+      );
+    } else {
+      filterList.push(
+        collection.filter.byProperty("type").containsAny(filters.types)
+      );
+    }
+  }
+  if (filters?.weight_min !== void 0) {
+    filterList.push(
+      collection.filter.byProperty("weight").greaterThanOrEqual(filters.weight_min)
+    );
+  }
+  if (filters?.weight_max !== void 0) {
+    filterList.push(
+      collection.filter.byProperty("weight").lessThanOrEqual(filters.weight_max)
+    );
+  }
+  if (filters?.trust_min !== void 0) {
+    filterList.push(
+      collection.filter.byProperty("trust").greaterThanOrEqual(filters.trust_min)
+    );
+  }
+  if (filters?.trust_max !== void 0) {
+    filterList.push(
+      collection.filter.byProperty("trust").lessThanOrEqual(filters.trust_max)
+    );
+  }
+  if (filters?.date_from) {
+    filterList.push(
+      collection.filter.byProperty("created_at").greaterThanOrEqual(new Date(filters.date_from))
+    );
+  }
+  if (filters?.date_to) {
+    filterList.push(
+      collection.filter.byProperty("created_at").lessThanOrEqual(new Date(filters.date_to))
+    );
+  }
+  if (filters?.tags && filters.tags.length > 0) {
+    if (filters.tags.length === 1) {
+      filterList.push(
+        collection.filter.byProperty("tags").containsAny([filters.tags[0]])
+      );
+    } else {
+      filterList.push(
+        collection.filter.byProperty("tags").containsAny(filters.tags)
+      );
+    }
+  }
+  return combineFiltersWithAnd(filterList);
+}
+function combineFiltersWithAnd(filters) {
+  const validFilters = filters.filter((f) => f !== void 0 && f !== null);
+  if (validFilters.length === 0) {
+    return void 0;
+  }
+  if (validFilters.length === 1) {
+    return validFilters[0];
+  }
+  return Filters.and(...validFilters);
+}
+function combineFiltersWithOr(filters) {
+  const validFilters = filters.filter((f) => f !== void 0 && f !== null);
+  if (validFilters.length === 0) {
+    return void 0;
+  }
+  if (validFilters.length === 1) {
+    return validFilters[0];
+  }
+  return Filters.or(...validFilters);
+}
+
 // src/tools/query-memory.ts
 var queryMemoryTool = {
   name: "remember_query_memory",

package/package.json

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

package/src/tools/search-memory.ts

@@ -127,9 +127,9 @@ export async function handleSearchMemory(
 
     // Build filters using v3 API
     // Use OR logic to search both memories and relationships
-    const filters = includeRelationships
-      ? buildCombinedSearchFilters(collection, args.filters)
-      : buildMemoryOnlyFilters(collection, args.filters);
+    // const filters = includeRelationships
+    //   ? buildCombinedSearchFilters(collection, args.filters)
+    //   : buildMemoryOnlyFilters(collection, args.filters);
 
     // Build search options
     const searchOptions: any = {
@@ -138,15 +138,15 @@ export async function handleSearchMemory(
     };
 
     // Add filters if present
-    if (filters) {
-      searchOptions.filters = filters;
-    }
+    // if (filters) {
+    //   searchOptions.filters = filters;
+    // }
 
     // Log the query for debugging
     logger.info('Weaviate query', {
       query: args.query,
       searchOptions: JSON.stringify(searchOptions, null, 2),
-      hasFilters: !!filters,
+      // hasFilters: !!filters,
     });
 
     // Perform hybrid search with Weaviate v3 API
@@ -191,7 +191,8 @@ export async function handleSearchMemory(
 
     return JSON.stringify(searchResult, null, 2);
   } catch (error) {
-    logger.error('Failed to search memories:', error);
-    throw new Error(`Failed to search memories: ${error instanceof Error ? error.message : String(error)}`);
+    const errorMessage = error instanceof Error ? error.message : String(error);
+    logger.error('Failed to search memories:', { error: errorMessage, userId, query: args.query });
+    throw new Error(`Failed to search memories: ${errorMessage}`);
   }
 }