memorix 0.9.0 → 0.9.1

package/CHANGELOG.md

@@ -2,6 +2,11 @@
 
 All notable changes to this project will be documented in this file.
 
+## [0.9.1] — 2026-02-25
+
+### Fixed
+- **Defensive parameter coercion** — All 24 MCP tools now gracefully handle string-encoded arrays and numbers (e.g., `"[16]"` → `[16]`, `"20"` → `20`). Fixes compatibility with Claude Code CLI's known serialization bug ([#5504](https://github.com/anthropics/claude-code/issues/5504), [#26027](https://github.com/anthropics/claude-code/issues/26027)) and non-Anthropic models (GLM, etc.) that may produce incorrectly typed tool call arguments. Codex, Windsurf, and Cursor were already unaffected.
+
 ## [0.9.0] — 2026-02-24
 
 ### Added

package/dist/cli/index.js

@@ -5337,6 +5337,58 @@ __export(server_exports2, {
 import { watch } from "fs";
 import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { z } from "zod";
+function coerceNumberArray(val) {
+  if (Array.isArray(val)) return val.map(Number);
+  if (typeof val === "string") {
+    try {
+      const parsed = JSON.parse(val);
+      if (Array.isArray(parsed)) return parsed.map(Number);
+    } catch {
+    }
+  }
+  return [];
+}
+function coerceNumber(val, fallback) {
+  if (typeof val === "number") return val;
+  if (typeof val === "string") {
+    const n = Number(val);
+    if (!Number.isNaN(n)) return n;
+  }
+  return fallback;
+}
+function coerceStringArray(val) {
+  if (Array.isArray(val)) return val.map(String);
+  if (typeof val === "string") {
+    try {
+      const parsed = JSON.parse(val);
+      if (Array.isArray(parsed)) return parsed.map(String);
+    } catch {
+    }
+  }
+  return [];
+}
+function coerceObjectArray(val) {
+  if (Array.isArray(val)) {
+    return val.map((item) => {
+      if (typeof item === "string") {
+        try {
+          return JSON.parse(item);
+        } catch {
+          return item;
+        }
+      }
+      return item;
+    });
+  }
+  if (typeof val === "string") {
+    try {
+      const parsed = JSON.parse(val);
+      if (Array.isArray(parsed)) return parsed;
+    } catch {
+    }
+  }
+  return [];
+}
 async function createMemorixServer(cwd, existingServer) {
   const project = detectProject(cwd);
   if (project.id === "__invalid__") {
@@ -5472,6 +5524,9 @@ async function createMemorixServer(cwd, existingServer) {
       }
     },
     async ({ entityName, type, title, narrative, facts, filesModified, concepts, topicKey }) => {
+      const safeFacts = facts ? coerceStringArray(facts) : void 0;
+      const safeFiles = filesModified ? coerceStringArray(filesModified) : void 0;
+      const safeConcepts = concepts ? coerceStringArray(concepts) : void 0;
       await graphManager.createEntities([
         { name: entityName, entityType: "auto", observations: [] }
       ]);
@@ -5487,9 +5542,9 @@ async function createMemorixServer(cwd, existingServer) {
         type,
         title,
         narrative,
-        facts,
-        filesModified,
-        concepts,
+        facts: safeFacts,
+        filesModified: safeFiles,
+        concepts: safeConcepts,
         projectId: project.id,
         topicKey,
         sessionId
@@ -5497,11 +5552,11 @@ async function createMemorixServer(cwd, existingServer) {
       await graphManager.addObservations([
         { entityName, contents: [`[#${obs.id}] ${title}`] }
       ]);
-      const extracted = extractEntities([title, narrative, ...facts ?? []].join(" "));
+      const extracted = extractEntities([title, narrative, ...safeFacts ?? []].join(" "));
       const autoRelCount = await createAutoRelations(obs, extracted, graphManager);
       const enrichmentParts = [];
-      const autoFiles = obs.filesModified.filter((f) => !(filesModified ?? []).includes(f));
-      const autoConcepts = obs.concepts.filter((c) => !(concepts ?? []).includes(c));
+      const autoFiles = obs.filesModified.filter((f) => !(safeFiles ?? []).includes(f));
+      const autoConcepts = obs.concepts.filter((c) => !(safeConcepts ?? []).includes(c));
       if (autoFiles.length > 0) enrichmentParts.push(`+${autoFiles.length} files extracted`);
       if (autoConcepts.length > 0) enrichmentParts.push(`+${autoConcepts.length} concepts enriched`);
       if (autoRelCount > 0) enrichmentParts.push(`+${autoRelCount} relations auto-created`);
@@ -5565,11 +5620,13 @@ Use this as the \`topicKey\` parameter in \`memorix_store\` to enable upsert beh
       }
     },
     async ({ query, limit, type, maxTokens, scope, since, until }) => {
+      const safeLimit = limit != null ? coerceNumber(limit, 20) : void 0;
+      const safeMaxTokens = maxTokens != null ? coerceNumber(maxTokens, 0) : void 0;
       const result = await compactSearch({
         query,
-        limit,
+        limit: safeLimit,
         type,
-        maxTokens,
+        maxTokens: safeMaxTokens,
         since,
         until,
         // Default to current project scope; 'global' removes the project filter
@@ -5602,11 +5659,14 @@ Use this as the \`topicKey\` parameter in \`memorix_store\` to enable upsert beh
       }
     },
     async ({ anchorId, depthBefore, depthAfter }) => {
+      const safeAnchor = coerceNumber(anchorId, 0);
+      const safeBefore = depthBefore != null ? coerceNumber(depthBefore, 3) : void 0;
+      const safeAfter = depthAfter != null ? coerceNumber(depthAfter, 3) : void 0;
       const result = await compactTimeline(
-        anchorId,
+        safeAnchor,
         void 0,
-        depthBefore,
-        depthAfter
+        safeBefore,
+        safeAfter
       );
       return {
         content: [
@@ -5628,12 +5688,13 @@ Use this as the \`topicKey\` parameter in \`memorix_store\` to enable upsert beh
       }
     },
     async ({ ids }) => {
-      const result = await compactDetail(ids);
+      const safeIds = coerceNumberArray(ids);
+      const result = await compactDetail(safeIds);
       return {
         content: [
           {
             type: "text",
-            text: result.documents.length > 0 ? result.formatted : `No observations found for IDs: ${ids.join(", ")}`
+            text: result.documents.length > 0 ? result.formatted : `No observations found for IDs: ${safeIds.join(", ")}`
           }
         ]
       };
@@ -5735,7 +5796,8 @@ Archived memories can be restored manually if needed.` }]
       }
     },
     async ({ entities }) => {
-      const result = await graphManager.createEntities(entities);
+      const safeEntities = coerceObjectArray(entities);
+      const result = await graphManager.createEntities(safeEntities);
       return {
         content: [{ type: "text", text: JSON.stringify(result, null, 2) }]
       };
@@ -5755,7 +5817,8 @@ Archived memories can be restored manually if needed.` }]
       }
     },
     async ({ relations }) => {
-      const result = await graphManager.createRelations(relations);
+      const safeRelations = coerceObjectArray(relations);
+      const result = await graphManager.createRelations(safeRelations);
       return {
         content: [{ type: "text", text: JSON.stringify(result, null, 2) }]
       };
@@ -5774,7 +5837,8 @@ Archived memories can be restored manually if needed.` }]
       }
     },
     async ({ observations: observations2 }) => {
-      const result = await graphManager.addObservations(observations2);
+      const safeObs = coerceObjectArray(observations2);
+      const result = await graphManager.addObservations(safeObs);
       return {
         content: [{ type: "text", text: JSON.stringify(result, null, 2) }]
       };
@@ -5790,7 +5854,8 @@ Archived memories can be restored manually if needed.` }]
       }
     },
     async ({ entityNames }) => {
-      await graphManager.deleteEntities(entityNames);
+      const safeNames = coerceStringArray(entityNames);
+      await graphManager.deleteEntities(safeNames);
       return {
         content: [{ type: "text", text: "Entities deleted successfully" }]
       };
@@ -5809,7 +5874,8 @@ Archived memories can be restored manually if needed.` }]
       }
     },
     async ({ deletions }) => {
-      await graphManager.deleteObservations(deletions);
+      const safeDeletions = coerceObjectArray(deletions);
+      await graphManager.deleteObservations(safeDeletions);
       return {
         content: [{ type: "text", text: "Observations deleted successfully" }]
       };
@@ -5829,7 +5895,8 @@ Archived memories can be restored manually if needed.` }]
       }
     },
     async ({ relations }) => {
-      await graphManager.deleteRelations(relations);
+      const safeRelations = coerceObjectArray(relations);
+      await graphManager.deleteRelations(safeRelations);
       return {
         content: [{ type: "text", text: "Relations deleted successfully" }]
       };
@@ -5875,7 +5942,8 @@ Archived memories can be restored manually if needed.` }]
       }
     },
     async ({ names }) => {
-      const graph = await graphManager.openNodes(names);
+      const safeNames = coerceStringArray(names);
+      const graph = await graphManager.openNodes(safeNames);
       return {
         content: [{ type: "text", text: JSON.stringify(graph, null, 2) }]
       };
@@ -6154,9 +6222,10 @@ ${skill.content}` }]
       }
     },
     async ({ action, threshold }) => {
+      const safeThreshold = threshold != null ? coerceNumber(threshold, 0.45) : void 0;
       const { findConsolidationCandidates: findConsolidationCandidates2, executeConsolidation: executeConsolidation2 } = await Promise.resolve().then(() => (init_consolidation(), consolidation_exports));
       if (action === "preview") {
-        const clusters = await findConsolidationCandidates2(projectDir2, project.id, { threshold });
+        const clusters = await findConsolidationCandidates2(projectDir2, project.id, { threshold: safeThreshold });
         if (clusters.length === 0) {
           return { content: [{ type: "text", text: "\u2705 No consolidation candidates found. Your memories are already clean!" }] };
         }
@@ -6172,7 +6241,7 @@ ${skill.content}` }]
         lines2.push(`> Run with \`action: "execute"\` to merge. This will remove **${totalMergeable}** duplicate observations.`);
         return { content: [{ type: "text", text: lines2.join("\n") }] };
       }
-      const result = await executeConsolidation2(projectDir2, project.id, { threshold });
+      const result = await executeConsolidation2(projectDir2, project.id, { threshold: safeThreshold });
       if (result.clustersFound === 0) {
         return { content: [{ type: "text", text: "\u2705 No consolidation needed. Memories are already clean!" }] };
       }
@@ -6257,8 +6326,9 @@ ${summary ? "Summary saved for next session context injection." : "No summary pr
       }
     },
     async ({ limit }) => {
+      const safeLimit = limit != null ? coerceNumber(limit, 3) : 3;
       const { getSessionContext: getSessionContext2, listSessions: listSessions2 } = await Promise.resolve().then(() => (init_session(), session_exports));
-      const context = await getSessionContext2(projectDir2, project.id, limit ?? 3);
+      const context = await getSessionContext2(projectDir2, project.id, safeLimit);
       const sessions = await listSessions2(projectDir2, project.id);
       const activeSessions = sessions.filter((s) => s.status === "active");
       const completedSessions = sessions.filter((s) => s.status === "completed");
@@ -6355,7 +6425,7 @@ ${json}
       }
     },
     async ({ port: dashboardPort }) => {
-      const portNum = dashboardPort || 3210;
+      const portNum = dashboardPort != null ? coerceNumber(dashboardPort, 3210) : 3210;
       const url = `http://localhost:${portNum}`;
       if (dashboardRunning) {
         const { createConnection } = await import("net");