@danielsimonjr/memory-mcp 11.0.1 → 11.1.1

package/dist/server/toolDefinitions.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"toolDefinitions.d.ts","sourceRoot":"","sources":["../../src/server/toolDefinitions.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH;;GAEG;AACH,MAAM,WAAW,cAAc;IAC7B,IAAI,EAAE,MAAM,CAAC;IACb,WAAW,EAAE,MAAM,CAAC;IACpB,WAAW,EAAE;QACX,IAAI,EAAE,QAAQ,CAAC;QACf,UAAU,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;QACpC,QAAQ,CAAC,EAAE,MAAM,EAAE,CAAC;QACpB,oBAAoB,CAAC,EAAE,OAAO,CAAC;KAChC,CAAC;CACH;AAED;;;GAGG;AACH,eAAO,MAAM,eAAe,EAAE,cAAc,EAygC3C,CAAC"}
+{"version":3,"file":"toolDefinitions.d.ts","sourceRoot":"","sources":["../../src/server/toolDefinitions.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH;;GAEG;AACH,MAAM,WAAW,cAAc;IAC7B,IAAI,EAAE,MAAM,CAAC;IACb,WAAW,EAAE,MAAM,CAAC;IACpB,WAAW,EAAE;QACX,IAAI,EAAE,QAAQ,CAAC;QACf,UAAU,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;QACpC,QAAQ,CAAC,EAAE,MAAM,EAAE,CAAC;QACpB,oBAAoB,CAAC,EAAE,OAAO,CAAC;KAChC,CAAC;CACH;AAED;;;GAGG;AACH,eAAO,MAAM,eAAe,EAAE,cAAc,EA6gC3C,CAAC"}

package/dist/server/toolDefinitions.js

@@ -204,6 +204,7 @@ export const toolDefinitions = [
                         anchorTimestamps: { type: 'boolean', default: true },
                         extractKeywords: { type: 'boolean', default: false },
                     },
+                    additionalProperties: false,
                 },
                 persist: {
                     type: 'boolean',
@@ -236,7 +237,7 @@ export const toolDefinitions = [
     },
     {
         name: 'search_by_date_range',
-        description: 'Search entities within a date range, with optional filtering by entity type and tags',
+        description: 'Search entities within a date range, with optional filtering by entity type and tags. At least one of startDate or endDate should be provided.',
         inputSchema: {
             type: 'object',
             properties: {
@@ -354,6 +355,7 @@ export const toolDefinitions = [
                             description: 'Weight for metadata filtering (default: 0.2)',
                         },
                     },
+                    additionalProperties: false,
                 },
                 filters: {
                     type: 'object',
@@ -375,10 +377,12 @@ export const toolDefinitions = [
                                 start: { type: 'string', description: 'Start date (ISO 8601)' },
                                 end: { type: 'string', description: 'End date (ISO 8601)' },
                             },
+                            additionalProperties: false,
                         },
                         minImportance: { type: 'number', description: 'Minimum importance score (0-10)' },
                         maxImportance: { type: 'number', description: 'Maximum importance score (0-10)' },
                     },
+                    additionalProperties: false,
                 },
                 limit: { type: 'number', description: 'Maximum results to return (default: 10)' },
             },

package/dist/server/toolHandlers.d.ts

@@ -10,8 +10,11 @@
 import { formatToolResponse, type ManagerContext } from '@danielsimonjr/memoryjs';
 /**
  * Tool response type for MCP SDK compatibility.
+ * Extends base response with optional isError flag for error framing.
  */
-export type ToolResponse = ReturnType<typeof formatToolResponse>;
+export type ToolResponse = ReturnType<typeof formatToolResponse> & {
+    isError?: boolean;
+};
 /**
  * Tool handler function signature.
  */
@@ -20,8 +23,10 @@ export type ToolHandler = (ctx: ManagerContext, args: Record<string, unknown>) =
  * Registry of all tool handlers keyed by tool name.
  * Handlers call managers directly for reduced abstraction layers.
  *
- * Note: Large-response tools (read_graph, search_nodes, get_subtree, open_nodes)
- * are wrapped with automatic response compression for payloads >256KB.
+ * Note: Tools that can return unbounded result sets (read_graph, search_nodes,
+ * get_subtree, open_nodes) are wrapped with withCompression() for payloads >256KB.
+ * Filtered/limited search tools (search_nodes_ranked, fuzzy_search, etc.) are not
+ * wrapped because their results are bounded by query specificity or limit params.
  */
 export declare const toolHandlers: Record<string, ToolHandler>;
 /**
@@ -30,8 +35,7 @@ export declare const toolHandlers: Record<string, ToolHandler>;
  * @param name - Tool name to call
  * @param args - Tool arguments
  * @param ctx - Manager context with all manager instances
- * @returns Tool response
- * @throws Error if tool name is unknown
+ * @returns Tool response (includes isError: true on failure)
  */
 export declare function handleToolCall(name: string, args: Record<string, unknown>, ctx: ManagerContext): Promise<ToolResponse>;
 //# sourceMappingURL=toolHandlers.d.ts.map

package/dist/server/toolHandlers.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"toolHandlers.d.ts","sourceRoot":"","sources":["../../src/server/toolHandlers.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,EACL,kBAAkB,EAwBlB,KAAK,cAAc,EACpB,MAAM,yBAAyB,CAAC;AAIjC;;GAEG;AACH,MAAM,MAAM,YAAY,GAAG,UAAU,CAAC,OAAO,kBAAkB,CAAC,CAAC;AAEjE;;GAEG;AACH,MAAM,MAAM,WAAW,GAAG,CACxB,GAAG,EAAE,cAAc,EACnB,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,KAC1B,OAAO,CAAC,YAAY,CAAC,CAAC;AAyC3B;;;;;;GAMG;AACH,eAAO,MAAM,YAAY,EAAE,MAAM,CAAC,MAAM,EAAE,WAAW,CA0tBpD,CAAC;AAEF;;;;;;;;GAQG;AACH,wBAAsB,cAAc,CAClC,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC7B,GAAG,EAAE,cAAc,GAClB,OAAO,CAAC,YAAY,CAAC,CAMvB"}
+{"version":3,"file":"toolHandlers.d.ts","sourceRoot":"","sources":["../../src/server/toolHandlers.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,EACL,kBAAkB,EAwBlB,KAAK,cAAc,EACpB,MAAM,yBAAyB,CAAC;AAIjC;;;GAGG;AACH,MAAM,MAAM,YAAY,GAAG,UAAU,CAAC,OAAO,kBAAkB,CAAC,GAAG;IAAE,OAAO,CAAC,EAAE,OAAO,CAAA;CAAE,CAAC;AAEzF;;GAEG;AACH,MAAM,MAAM,WAAW,GAAG,CACxB,GAAG,EAAE,cAAc,EACnB,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,KAC1B,OAAO,CAAC,YAAY,CAAC,CAAC;AAyC3B;;;;;;;;GAQG;AACH,eAAO,MAAM,YAAY,EAAE,MAAM,CAAC,MAAM,EAAE,WAAW,CA4tBpD,CAAC;AAEF;;;;;;;GAOG;AACH,wBAAsB,cAAc,CAClC,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC7B,GAAG,EAAE,cAAc,GAClB,OAAO,CAAC,YAAY,CAAC,CAiBvB"}

package/dist/server/toolHandlers.js

@@ -46,8 +46,10 @@ async function withCompression(handler) {
  * Registry of all tool handlers keyed by tool name.
  * Handlers call managers directly for reduced abstraction layers.
  *
- * Note: Large-response tools (read_graph, search_nodes, get_subtree, open_nodes)
- * are wrapped with automatic response compression for payloads >256KB.
+ * Note: Tools that can return unbounded result sets (read_graph, search_nodes,
+ * get_subtree, open_nodes) are wrapped with withCompression() for payloads >256KB.
+ * Filtered/limited search tools (search_nodes_ranked, fuzzy_search, etc.) are not
+ * wrapped because their results are bounded by query specificity or limit params.
  */
 export const toolHandlers = {
     // ==================== ENTITY HANDLERS ====================
@@ -111,7 +113,9 @@ export const toolHandlers = {
             };
         });
         if (persist) {
-            // Update entities with normalized observations
+            // TODO: This bypasses the manager layer and writes directly to storage.
+            // May leave in-memory cache stale. Ideally, normalization logic should
+            // be moved into a memoryjs manager method. See code review issue #3.
             const updatedEntities = graph.entities.map(entity => {
                 const result = results.find(r => r.entityName === entity.name);
                 if (result && result.changes.length > 0) {
@@ -125,7 +129,7 @@ export const toolHandlers = {
             });
             await ctx.storage.saveGraph({
                 entities: updatedEntities,
-                relations: [...graph.relations],
+                relations: [...graph.relations], // spread needed: graph.relations is readonly
             });
         }
         return formatToolResponse({
@@ -658,13 +662,24 @@ export const toolHandlers = {
  * @param name - Tool name to call
  * @param args - Tool arguments
  * @param ctx - Manager context with all manager instances
- * @returns Tool response
- * @throws Error if tool name is unknown
+ * @returns Tool response (includes isError: true on failure)
  */
 export async function handleToolCall(name, args, ctx) {
     const handler = toolHandlers[name];
     if (!handler) {
-        throw new Error(`Unknown tool: ${name}`);
+        return {
+            content: [{ type: 'text', text: `Error: Unknown tool: ${name}` }],
+            isError: true,
+        };
+    }
+    try {
+        return await handler(ctx, args);
+    }
+    catch (error) {
+        const message = error instanceof Error ? error.message : String(error);
+        return {
+            content: [{ type: 'text', text: `Error: ${message}` }],
+            isError: true,
+        };
     }
-    return handler(ctx, args);
 }

package/dist/types/index.d.ts

@@ -9,5 +9,5 @@
  * import { Entity, Relation, KnowledgeGraph, SearchResult } from './types/index.js';
  * ```
  */
-export type { Entity, Relation, KnowledgeGraph, ReadonlyKnowledgeGraph, SearchResult, SavedSearch, BooleanQueryNode, DocumentVector, TFIDFIndex, FuzzyCacheKey, BooleanCacheEntry, PaginatedCacheEntry, TokenizedEntity, GraphStats, ValidationReport, ValidationIssue, ValidationWarning, CacheCompressionStats, ArchiveResultExtended, ExportFilter, ExportOptions, ExportResult, ImportResult, CompressionResult, BackupOptions, BackupResult, RestoreResult, BackupMetadataExtended, BackupInfoExtended, TagAlias, IGraphStorage, StorageConfig, LowercaseData, TraversalOptions, TraversalResult, PathResult, ConnectedComponentsResult, CentralityResult, WeightedRelation, EmbeddingMode, EmbeddingService, SemanticSearchResult, IVectorStore, VectorSearchResult, EmbeddingConfig, SemanticIndexOptions, LongRunningOperationOptions, BatchOperationType, BatchOperation, BatchResult, BatchOptions, GraphEventType, GraphEventBase, EntityCreatedEvent, EntityUpdatedEvent, EntityDeletedEvent, RelationCreatedEvent, RelationDeletedEvent, ObservationAddedEvent, ObservationDeletedEvent, GraphSavedEvent, GraphLoadedEvent, GraphEvent, GraphEventListener, GraphEventMap, SearchMethod, QueryCostEstimate, AutoSearchResult, QueryCostEstimatorOptions, PreparedEntity, SymbolicFilters, HybridSearchOptions, HybridSearchResult, ExtractedEntity, TemporalRange, QueryAnalysis, SubQuery, QueryPlan, } from './types.js';
+export type { Entity, Relation, KnowledgeGraph, ReadonlyKnowledgeGraph, SearchResult, SavedSearch, BooleanQueryNode, DocumentVector, TFIDFIndex, FuzzyCacheKey, BooleanCacheEntry, PaginatedCacheEntry, TokenizedEntity, GraphStats, ValidationReport, ValidationIssue, ValidationWarning, CacheCompressionStats, ArchiveResultExtended, ExportFilter, ExportOptions, ExportResult, ImportResult, CompressionResult, BackupOptions, BackupResult, RestoreResult, BackupMetadataExtended, BackupInfoExtended, TagAlias, IGraphStorage, StorageConfig, LowercaseData, TraversalOptions, TraversalResult, PathResult, ConnectedComponentsResult, CentralityResult, WeightedRelation, EmbeddingService, SemanticSearchResult, IVectorStore, VectorSearchResult, EmbeddingConfig, SemanticIndexOptions, } from './types.js';
 //# sourceMappingURL=index.d.ts.map

package/dist/types/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/types/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,YAAY,EAEV,MAAM,EACN,QAAQ,EACR,cAAc,EACd,sBAAsB,EAEtB,YAAY,EACZ,WAAW,EACX,gBAAgB,EAChB,cAAc,EACd,UAAU,EAEV,aAAa,EACb,iBAAiB,EACjB,mBAAmB,EACnB,eAAe,EAEf,UAAU,EACV,gBAAgB,EAChB,eAAe,EACf,iBAAiB,EACjB,qBAAqB,EAErB,qBAAqB,EAErB,YAAY,EACZ,aAAa,EACb,YAAY,EACZ,YAAY,EACZ,iBAAiB,EAEjB,aAAa,EACb,YAAY,EACZ,aAAa,EACb,sBAAsB,EACtB,kBAAkB,EAElB,QAAQ,EAER,aAAa,EACb,aAAa,EACb,aAAa,EAEb,gBAAgB,EAChB,eAAe,EACf,UAAU,EACV,yBAAyB,EACzB,gBAAgB,EAChB,gBAAgB,EAEhB,aAAa,EACb,gBAAgB,EAChB,oBAAoB,EACpB,YAAY,EACZ,kBAAkB,EAClB,eAAe,EACf,oBAAoB,EAEpB,2BAA2B,EAE3B,kBAAkB,EAClB,cAAc,EACd,WAAW,EACX,YAAY,EAEZ,cAAc,EACd,cAAc,EACd,kBAAkB,EAClB,kBAAkB,EAClB,kBAAkB,EAClB,oBAAoB,EACpB,oBAAoB,EACpB,qBAAqB,EACrB,uBAAuB,EACvB,eAAe,EACf,gBAAgB,EAChB,UAAU,EACV,kBAAkB,EAClB,aAAa,EAEb,YAAY,EACZ,iBAAiB,EACjB,gBAAgB,EAChB,yBAAyB,EAEzB,cAAc,EAEd,eAAe,EACf,mBAAmB,EACnB,kBAAkB,EAElB,eAAe,EACf,aAAa,EACb,aAAa,EACb,QAAQ,EACR,SAAS,GACV,MAAM,YAAY,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/types/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,YAAY,EAEV,MAAM,EACN,QAAQ,EACR,cAAc,EACd,sBAAsB,EAEtB,YAAY,EACZ,WAAW,EACX,gBAAgB,EAChB,cAAc,EACd,UAAU,EAEV,aAAa,EACb,iBAAiB,EACjB,mBAAmB,EACnB,eAAe,EAEf,UAAU,EACV,gBAAgB,EAChB,eAAe,EACf,iBAAiB,EACjB,qBAAqB,EAErB,qBAAqB,EAErB,YAAY,EACZ,aAAa,EACb,YAAY,EACZ,YAAY,EACZ,iBAAiB,EAEjB,aAAa,EACb,YAAY,EACZ,aAAa,EACb,sBAAsB,EACtB,kBAAkB,EAElB,QAAQ,EAER,aAAa,EACb,aAAa,EACb,aAAa,EAEb,gBAAgB,EAChB,eAAe,EACf,UAAU,EACV,yBAAyB,EACzB,gBAAgB,EAChB,gBAAgB,EAEhB,gBAAgB,EAChB,oBAAoB,EACpB,YAAY,EACZ,kBAAkB,EAClB,eAAe,EACf,oBAAoB,GACrB,MAAM,YAAY,CAAC"}