@vheins/local-memory-mcp 0.19.4 → 0.19.6

package/dist/{chunk-R45NRES5.js → chunk-IT2PAPJ6.js}

@@ -132,6 +132,7 @@ import fs3 from "fs";
 import os from "os";
 
 // src/mcp/storage/migrations.ts
+var SCHEMA_VERSION = 2;
 var MigrationManager = class {
   constructor(db) {
     this.db = db;
@@ -149,7 +150,25 @@ var MigrationManager = class {
   get(sql) {
     return this.db.prepare(sql).get();
   }
+  getSchemaVersion() {
+    try {
+      const row = this.get("SELECT version FROM _schema_version LIMIT 1");
+      return row ? row.version : 0;
+    } catch {
+      return 0;
+    }
+  }
+  setSchemaVersion(version) {
+    this.exec(`CREATE TABLE IF NOT EXISTS _schema_version (version INTEGER NOT NULL)`);
+    this.run("DELETE FROM _schema_version");
+    this.run("INSERT INTO _schema_version (version) VALUES (?)", version);
+  }
   migrate() {
+    const currentVersion = this.getSchemaVersion();
+    if (currentVersion >= SCHEMA_VERSION) {
+      logger.debug(`[Migration] Schema already at version ${SCHEMA_VERSION}, skipping`);
+      return;
+    }
     this.exec(`
       CREATE TABLE IF NOT EXISTS memories (
         id TEXT PRIMARY KEY,
@@ -599,6 +618,8 @@ var MigrationManager = class {
       this.run("UPDATE tasks SET task_code = substr(id, 1, 8) WHERE task_code IS NULL");
     } catch {
     }
+    this.setSchemaVersion(SCHEMA_VERSION);
+    logger.info(`[Migration] Schema upgraded to version ${SCHEMA_VERSION}`);
   }
   ensureMemoryTypeConstraint() {
     const tableSql = this.get("SELECT sql FROM sqlite_master WHERE type = 'table' AND name = 'memories'");
@@ -1049,7 +1070,15 @@ var BaseEntity = class {
       status: r.status || "active",
       is_global: r.is_global === 1,
       tags: this.safeJSONParse(r.tags, []),
-      metadata: this.safeJSONParse(r.metadata, {})
+      metadata: (() => {
+        const meta = this.safeJSONParse(r.metadata, {});
+        delete meta.structuredData;
+        return meta;
+      })(),
+      structuredData: (() => {
+        const meta = this.safeJSONParse(r.metadata, {});
+        return meta.structuredData ?? void 0;
+      })()
     };
   }
   rowToTask(row) {
@@ -1141,6 +1170,7 @@ var VALID_COLUMNS = /* @__PURE__ */ new Set([
 ]);
 var MemoryEntity = class extends BaseEntity {
   insert(entry) {
+    const mergedMeta = this.mergeStructuredData(entry.metadata, entry.structuredData);
     this.run(
       `INSERT INTO memories (
 				id, code, repo, owner, type, title, content, importance, folder, language,
@@ -1165,7 +1195,7 @@ var MemoryEntity = class extends BaseEntity {
         entry.status || "active",
         entry.is_global ? 1 : 0,
         entry.tags ? JSON.stringify(entry.tags) : null,
-        entry.metadata ? JSON.stringify(entry.metadata) : null,
+        mergedMeta ? JSON.stringify(mergedMeta) : null,
         entry.agent || "unknown",
         entry.role || "unknown",
         entry.model || "unknown",
@@ -1173,6 +1203,9 @@ var MemoryEntity = class extends BaseEntity {
       ]
     );
   }
+  mergeStructuredData(metadata, structuredData) {
+    return { ...metadata, structuredData: structuredData ?? {} };
+  }
   update(id, updates) {
     const fields = [];
     const values = [];
@@ -1198,6 +1231,12 @@ var MemoryEntity = class extends BaseEntity {
             fields.push("language = ?");
             values.push(scope.language);
           }
+        } else if (k === "structuredData") {
+          const existingRow = this.get("SELECT metadata FROM memories WHERE id = ?", [id]);
+          const existingMeta = existingRow ? this.safeJSONParse(existingRow.metadata, {}) : {};
+          const merged = { ...existingMeta, structuredData: val };
+          fields.push("metadata = ?");
+          values.push(JSON.stringify(merged));
         } else if (k === "tags" || k === "metadata") {
           fields.push(`${k} = ?`);
           values.push(JSON.stringify(val));
@@ -1301,6 +1340,7 @@ var MemoryEntity = class extends BaseEntity {
     return this.transaction(() => {
       let count = 0;
       for (const entry of entries) {
+        const mergedMeta = this.mergeStructuredData(entry.metadata, entry.structuredData);
         this.run(
           `INSERT INTO memories (
 						id, repo, owner, type, title, content, importance, folder, language,
@@ -1324,7 +1364,7 @@ var MemoryEntity = class extends BaseEntity {
             entry.status || "active",
             entry.is_global ? 1 : 0,
             entry.tags ? JSON.stringify(entry.tags) : null,
-            entry.metadata ? JSON.stringify(entry.metadata) : null,
+            mergedMeta ? JSON.stringify(mergedMeta) : null,
             entry.agent || "unknown",
             entry.role || "unknown",
             entry.model || "unknown",
@@ -3208,7 +3248,11 @@ var SQLiteStore = class _SQLiteStore {
     this.db.pragma("foreign_keys = ON");
     this.db.pragma("wal_autocheckpoint = 100");
     if (finalPath !== ":memory:") {
-      this._startupChecks(finalPath);
+      try {
+        this.db.pragma("wal_checkpoint(PASSIVE)");
+      } catch (err) {
+        logger.warn("[SQLiteStore] WAL checkpoint failed on startup", { error: String(err) });
+      }
     }
     const migrator = new MigrationManager(this.db);
     migrator.migrate();
@@ -3494,24 +3538,29 @@ function findPromptDir() {
   return path4.resolve(__dirname, "./definitions");
 }
 var PROMPT_DIR = findPromptDir();
+var promptCache = /* @__PURE__ */ new Map();
 function listPromptFiles() {
   if (!fs4.existsSync(PROMPT_DIR)) return [];
   return fs4.readdirSync(PROMPT_DIR).filter((file) => file.endsWith(".md")).map((file) => file.replace(/\.md$/, "")).sort();
 }
 function loadPromptFromMarkdown(name) {
+  const cached = promptCache.get(name);
+  if (cached) return cached;
   const filePath = path4.join(PROMPT_DIR, `${name}.md`);
   if (!fs4.existsSync(filePath)) {
     throw new Error(`Prompt file not found: ${filePath}`);
   }
   const fileContent = fs4.readFileSync(filePath, "utf-8");
   const { data, content } = matter(fileContent);
-  return {
+  const result = {
     name: data.name || name,
     description: data.description || "",
     arguments: data.arguments || [],
     agent: data.agent,
     content: content.trim()
   };
+  promptCache.set(name, result);
+  return result;
 }
 function findServerInstructionsDir() {
   const candidates = [
@@ -3533,14 +3582,17 @@ function findServerInstructionsDir() {
   return path4.resolve(__dirname, "./server");
 }
 var SERVER_DIR = findServerInstructionsDir();
+var serverInstructionsCache = null;
 function loadServerInstructions() {
+  if (serverInstructionsCache !== null) return serverInstructionsCache;
   const filePath = path4.join(SERVER_DIR, "instructions.md");
   if (!fs4.existsSync(filePath)) {
     throw new Error(`Server instructions file not found: ${filePath}`);
   }
   const fileContent = fs4.readFileSync(filePath, "utf-8");
   const { content } = matter(fileContent);
-  return content.trim();
+  serverInstructionsCache = content.trim();
+  return serverInstructionsCache;
 }
 
 // src/mcp/tools/definitions/memory.ts
@@ -4255,7 +4307,10 @@ var TASK_TOOL_DEFINITIONS = [
           properties: {
             owner: { type: "string", description: "Organization/namespace (e.g., GitHub org or username)" },
             repo: { type: "string", description: "Repository name" },
-            task_code: { type: "string", description: "Task code (e.g. TASK-001)" },
+            task_code: {
+              type: "string",
+              description: "Task code (e.g. PERF-1, TASK-001). Use instead of 'id' for string code lookup."
+            },
             structured: {
               type: "boolean",
               default: false,
@@ -4273,7 +4328,7 @@ var TASK_TOOL_DEFINITIONS = [
               type: "array",
               items: { type: "string" },
               minItems: 1,
-              description: "Array of task codes (e.g. TASK-001)"
+              description: "Array of task codes (e.g. PERF-1, TASK-001). Use instead of 'ids' when identifying tasks by code rather than UUID."
             },
             structured: {
               type: "boolean",
@@ -4401,7 +4456,7 @@ var TASK_TOOL_DEFINITIONS = [
   {
     name: "task-update",
     title: "Task Update",
-    description: "Update one or more tasks. Supports single update via 'id' or bulk update via 'ids'. Provide only the fields that need to be changed. MANDATORY WORKFLOW: You cannot move a task from 'pending' or 'blocked' directly to 'completed'. You MUST move it to 'in_progress' first. When changing status to 'completed', include 'est_tokens' with the estimated total tokens actually used for the task. You may also include 'commit_id' (git commit hash) and 'changed_files' (list of files changed) for traceability.",
+    description: "Update one or more tasks. Supports single update via 'id' (UUID) or 'task_code' (e.g. PERF-1), bulk via 'ids' (UUID array) or 'task_codes' (string array). Use 'task_code'/'task_codes' for human-readable identifiers, 'id'/'ids' for UUID lookups. Provide only the fields that need to be changed. MANDATORY WORKFLOW: Cannot move 'pending'/'blocked' \u2192 'completed' directly; MUST go through 'in_progress' first. Include 'est_tokens' when moving to 'completed'.",
     annotations: {
       readOnlyHint: false,
       idempotentHint: false,
@@ -4482,7 +4537,7 @@ var TASK_TOOL_DEFINITIONS = [
             ids: {
               type: "array",
               items: { type: "string", format: "uuid" },
-              description: "Task IDs (for bulk update)"
+              description: "Task UUIDs (for bulk update). NOT task codes \u2014 use 'task_codes' for PERF-1 style identifiers."
             },
             phase: { type: "string" },
             title: { type: "string", minLength: 3, maxLength: 100 },
@@ -4613,7 +4668,7 @@ var TASK_TOOL_DEFINITIONS = [
               type: "array",
               items: { type: "string" },
               minItems: 1,
-              description: "Array of task codes (e.g. TASK-001)"
+              description: "Array of task codes (e.g. PERF-1, TASK-001). Use instead of 'ids' when identifying tasks by code rather than UUID."
             },
             phase: { type: "string" },
             title: { type: "string", minLength: 3, maxLength: 100 },
@@ -4723,7 +4778,7 @@ var TASK_TOOL_DEFINITIONS = [
             ids: {
               type: "array",
               items: { type: "string", format: "uuid" },
-              description: "Task IDs (for bulk deletion)"
+              description: "Task UUIDs (for bulk deletion). NOT task codes \u2014 use 'task_codes' for PERF-1 style identifiers."
             },
             structured: { type: "boolean", default: false, description: "If true, returns structured JSON result." }
           }
@@ -4734,7 +4789,10 @@ var TASK_TOOL_DEFINITIONS = [
           properties: {
             owner: { type: "string", description: "Organization/namespace (e.g., GitHub org or username)" },
             repo: { type: "string", description: "Repository name" },
-            task_code: { type: "string", description: "Task code (e.g. TASK-001)" },
+            task_code: {
+              type: "string",
+              description: "Task code (e.g. PERF-1, TASK-001). Use instead of 'id' for string code lookup."
+            },
             structured: { type: "boolean", default: false, description: "If true, returns structured JSON result." }
           }
         },
@@ -4748,7 +4806,7 @@ var TASK_TOOL_DEFINITIONS = [
               type: "array",
               items: { type: "string" },
               minItems: 1,
-              description: "Array of task codes (e.g. TASK-001)"
+              description: "Array of task codes (e.g. PERF-1, TASK-001). Use instead of 'ids' when identifying tasks by code rather than UUID."
             },
             structured: { type: "boolean", default: false, description: "If true, returns structured JSON result." }
           }
@@ -5622,7 +5680,7 @@ var AGENT_TOOL_DEFINITIONS = [
   {
     name: "decision-log",
     title: "Decision Logger",
-    description: "Logs a structured decision as a memory entry. Accepts a summary of what was decided, the context/situation, the rationale, and optional alternatives considered. Internally reuses the memory-store handler with type='decision' and importance=4. Owner and repo are auto-inferred from session when omitted.",
+    description: "Logs a structured decision as a memory entry (type='decision', importance=4). Fields: summary (max 255 chars \u2014 becomes the memory title), context (situation/background, min 10 chars), rationale (why it was decided, min 10 chars), and alternatives (ARRAY of strings, NOT a single string \u2014 e.g. ['option A', 'option B']). Do NOT pass 'title' \u2014 that field does not exist on this tool. Owner and repo are auto-inferred from session when omitted.",
     annotations: {
       readOnlyHint: false,
       idempotentHint: false,
@@ -6418,6 +6476,28 @@ var SingleStandardSchema = z2.object({
   model: z2.string().optional()
 });
 var TaskStatusValues = TaskStatusSchema.options;
+function isPlainObject(v) {
+  return typeof v === "object" && v !== null && !Array.isArray(v) && Object.getPrototypeOf(v) === Object.prototype;
+}
+function isJsonSerializable(val) {
+  try {
+    JSON.stringify(val);
+    return true;
+  } catch {
+    return false;
+  }
+}
+var StructuredDataValue = z2.lazy(
+  () => z2.union([
+    z2.string(),
+    z2.number().refine((v) => Number.isFinite(v), "Number must be finite"),
+    z2.boolean(),
+    z2.null(),
+    z2.array(StructuredDataValue),
+    z2.record(z2.string(), StructuredDataValue).refine(isPlainObject, "Plain object required")
+  ])
+);
+var StructuredDataSchema = z2.unknown().refine(isJsonSerializable, { message: "Value must be JSON-serializable" }).pipe(z2.record(z2.string(), StructuredDataValue));
 
 // src/mcp/tools/schemas/memory.ts
 import { z as z3 } from "zod";
@@ -6499,8 +6579,8 @@ var MemoryAcknowledgeSchema = z3.object({
   message: "Either memory_id or code must be provided"
 });
 var MemoryRecapSchema = z3.object({
-  owner: z3.string().min(1),
-  repo: z3.string().min(1).transform(normalizeRepo),
+  owner: z3.string().min(1, "owner is required \u2014 provide it explicitly or configure MCP workspace roots"),
+  repo: z3.string().min(1, "repo is required \u2014 provide it explicitly or configure MCP workspace roots").transform(normalizeRepo),
   limit: z3.number().min(1).max(50).default(20),
   offset: z3.number().min(0).default(0),
   structured: z3.boolean().default(false)
@@ -6683,13 +6763,15 @@ var TaskUpdateSchema = z4.object({
   structured: z4.boolean().default(false)
 }).refine(
   (data) => data.id !== void 0 || data.ids !== void 0 || data.task_code !== void 0 || data.task_codes !== void 0,
-  { message: "Either 'id', 'ids', 'task_code', or 'task_codes' must be provided for update" }
+  {
+    message: "Either 'id' (UUID), 'ids' (array of UUIDs), 'task_code' (string code like PERF-1), or 'task_codes' (array of string codes) must be provided. Note: 'ids' expects UUID format, not task codes \u2014 use 'task_code'/'task_codes' for string identifiers."
+  }
 ).refine((data) => Object.keys(data).length > 2, {
   message: "At least one field besides repo and id/ids must be provided for update"
 });
 var TaskListSchema = z4.object({
-  owner: z4.string().min(1),
-  repo: z4.string().min(1).transform(normalizeRepo),
+  owner: z4.string().min(1, "owner is required \u2014 provide it explicitly or configure MCP workspace roots"),
+  repo: z4.string().min(1, "repo is required \u2014 provide it explicitly or configure MCP workspace roots").transform(normalizeRepo),
   status: TaskStatusListSchema.default("backlog,pending,in_progress,blocked"),
   phase: z4.string().optional(),
   query: z4.string().optional(),
@@ -6718,7 +6800,9 @@ var TaskDeleteSchema = z4.object({
   structured: z4.boolean().default(false)
 }).refine(
   (data) => data.id !== void 0 || data.ids !== void 0 || data.task_code !== void 0 || data.task_codes !== void 0,
-  { message: "Either 'id', 'ids', 'task_code', or 'task_codes' must be provided for deletion" }
+  {
+    message: "Either 'id' (UUID), 'ids' (array of UUIDs), 'task_code' (string code like PERF-1), or 'task_codes' (array of string codes) must be provided. Note: 'ids' expects UUID format, not task codes \u2014 use 'task_code'/'task_codes' for string identifiers."
+  }
 );
 var TaskGetSchema = z4.object({
   owner: z4.string().min(1),
@@ -6734,8 +6818,8 @@ var TaskGetSchema = z4.object({
 // src/mcp/tools/schemas/handoff.ts
 import { z as z5 } from "zod";
 var HandoffCreateSchema = z5.object({
-  owner: z5.string().min(1).optional().default(""),
-  repo: z5.string().min(1).transform(normalizeRepo),
+  owner: z5.string().min(1, "owner is required \u2014 provide it explicitly or configure MCP workspace roots"),
+  repo: z5.string().min(1, "repo is required \u2014 provide it explicitly or configure MCP workspace roots").transform(normalizeRepo),
   from_agent: z5.string().min(1),
   to_agent: z5.string().min(1).optional(),
   task_id: z5.string().uuid().optional(),
@@ -6758,8 +6842,8 @@ var HandoffUpdateSchema = z5.object({
   structured: z5.boolean().default(false)
 });
 var HandoffListSchema = z5.object({
-  owner: z5.string().min(1).optional().default(""),
-  repo: z5.string().min(1).transform(normalizeRepo),
+  owner: z5.string().min(1, "owner is required \u2014 provide it explicitly or configure MCP workspace roots"),
+  repo: z5.string().min(1, "repo is required \u2014 provide it explicitly or configure MCP workspace roots").transform(normalizeRepo),
   status: HandoffStatusSchema.optional(),
   from_agent: z5.string().min(1).optional(),
   to_agent: z5.string().min(1).optional(),
@@ -6768,8 +6852,8 @@ var HandoffListSchema = z5.object({
   structured: z5.boolean().default(false)
 });
 var TaskClaimSchema = z5.object({
-  owner: z5.string().min(1).optional().default(""),
-  repo: z5.string().min(1).transform(normalizeRepo),
+  owner: z5.string().min(1, "owner is required \u2014 provide it explicitly or configure MCP workspace roots"),
+  repo: z5.string().min(1, "repo is required \u2014 provide it explicitly or configure MCP workspace roots").transform(normalizeRepo),
   task_id: z5.string().uuid().optional(),
   task_code: z5.string().optional(),
   agent: z5.string().min(1),
@@ -6782,8 +6866,8 @@ var TaskClaimSchema = z5.object({
   message: "Provide either task_id or task_code, not both"
 });
 var ClaimListSchema = z5.object({
-  owner: z5.string().min(1).optional().default(""),
-  repo: z5.string().min(1).transform(normalizeRepo),
+  owner: z5.string().optional().default(""),
+  repo: z5.string().transform(normalizeRepo).optional().default(""),
   agent: z5.string().min(1).optional(),
   active_only: z5.boolean().default(true),
   limit: z5.number().min(1).max(100).default(20),
@@ -6791,8 +6875,8 @@ var ClaimListSchema = z5.object({
   structured: z5.boolean().default(false)
 });
 var ClaimReleaseSchema = z5.object({
-  owner: z5.string().min(1).optional().default(""),
-  repo: z5.string().min(1).transform(normalizeRepo),
+  owner: z5.string().min(1, "owner is required \u2014 provide it explicitly or configure MCP workspace roots"),
+  repo: z5.string().min(1, "repo is required \u2014 provide it explicitly or configure MCP workspace roots").transform(normalizeRepo),
   task_id: z5.string().uuid().optional(),
   task_code: z5.string().optional(),
   agent: z5.string().min(1).optional(),
@@ -6906,10 +6990,10 @@ var AgentContextSchema = z7.object({
   structured: z7.boolean().default(false)
 });
 var DecisionLogSchema = z7.object({
-  summary: z7.string().min(3).max(255, { message: "Summary must be at most 255 characters" }),
-  context: z7.string().min(10, { message: "Context must be at least 10 characters" }),
-  rationale: z7.string().min(10, { message: "Rationale must be at least 10 characters" }),
-  alternatives: z7.array(z7.string()).optional(),
+  summary: z7.string().min(3, { message: "Summary must be at least 3 characters (this becomes the memory title)" }).max(255, { message: "Summary must be at most 255 characters (this becomes the memory title)" }),
+  context: z7.string().min(10, { message: "Context must be at least 10 characters describing the situation" }),
+  rationale: z7.string().min(10, { message: "Rationale must be at least 10 characters explaining why the decision was made" }),
+  alternatives: z7.array(z7.string(), { message: "Alternatives must be an array of strings, e.g. ['option A', 'option B']" }).optional(),
   tags: z7.array(z7.string()).optional(),
   owner: z7.string().optional(),
   repo: z7.string().optional(),
@@ -7029,7 +7113,13 @@ async function handleHandoffCreate(args, storage) {
   });
 }
 async function handleHandoffList(args, storage) {
-  const validated = HandoffListSchema.parse(args);
+  const parsed = HandoffListSchema.safeParse(args);
+  if (!parsed.success) {
+    const missing = parsed.error.issues.filter((i) => i.path.some((p) => p === "owner" || p === "repo")).map((i) => i.message).filter(Boolean);
+    const msg = missing.length > 0 ? `Missing required fields: ${missing.join("; ")}. Pass owner/repo explicitly or configure MCP workspace roots so they can be auto-inferred.` : `Validation error: ${parsed.error.message}`;
+    return { content: [{ type: "text", text: msg }], isError: true };
+  }
+  const validated = parsed.data;
   const { owner, repo, status, from_agent, to_agent, limit, offset, structured } = validated;
   const handoffs = storage.handoffs.listHandoffs({
     owner,

package/dist/dashboard/server.js

@@ -15,7 +15,7 @@ import {
   handleTaskClaim,
   listResources,
   logger
-} from "../chunk-R45NRES5.js";
+} from "../chunk-IT2PAPJ6.js";
 
 // src/dashboard/server.ts
 import express from "express";

package/dist/mcp/server.js

@@ -61,7 +61,7 @@ import {
   parseRepoInput,
   rankCompletionValues,
   toContextSlug
-} from "../chunk-R45NRES5.js";
+} from "../chunk-IT2PAPJ6.js";
 
 // src/mcp/server.ts
 import { serveStdio } from "@modelcontextprotocol/server/stdio";
@@ -74,8 +74,8 @@ import path from "path";
 import { fileURLToPath } from "url";
 var __dirname = path.dirname(fileURLToPath(import.meta.url));
 var pkgVersion = "0.1.0";
-if ("0.19.4") {
-  pkgVersion = "0.19.4";
+if ("0.19.6") {
+  pkgVersion = "0.19.6";
 } else {
   let searchDir = __dirname;
   for (let i = 0; i < 5; i++) {
@@ -154,7 +154,6 @@ function generateNextCode(owner, repo, entityType, storage, batchCodes) {
 
 // src/mcp/tools/kg-archivist.ts
 import { randomUUID } from "crypto";
-import nlp from "compromise";
 var MAX_CONTENT_LENGTH = 5e3;
 var PRONOUNS = /* @__PURE__ */ new Set([
   "i",
@@ -354,8 +353,9 @@ function isExcludedNoun(candidate) {
   if (/^\d+$/.test(candidate)) return true;
   return false;
 }
-function extractEntities(content) {
+async function extractEntities(content) {
   if (!content || content.trim().length === 0) return [];
+  const { default: nlp } = await import("compromise");
   const text = content.length > MAX_CONTENT_LENGTH ? content.slice(0, MAX_CONTENT_LENGTH) : content;
   const doc = nlp(text);
   const seen = /* @__PURE__ */ new Set();
@@ -386,11 +386,11 @@ function extractEntities(content) {
   }
   return entities;
 }
-function saveExtractions(content, title, owner, repo, db2) {
+async function saveExtractions(content, title, owner, repo, db2) {
   if (!content || content.trim().length === 0) return;
   let entities;
   try {
-    entities = extractEntities(content);
+    entities = await extractEntities(content);
   } catch (err) {
     logger.warn("[KG-Archivist] Entity extraction failed, skipping", {
       error: String(err)
@@ -507,7 +507,7 @@ async function storeSingleMemory(params, db2, vectors2) {
     logger.warn("Failed to generate vector embedding", { error: String(error) });
   }
   try {
-    saveExtractions(entry.content, entry.title, entry.scope.owner, entry.scope.repo, db2);
+    await saveExtractions(entry.content, entry.title, entry.scope.owner, entry.scope.repo, db2);
   } catch (error) {
     logger.warn("[KG-Archivist] NLP extraction failed, memory stored without KG enrichment", {
       error: String(error)
@@ -614,7 +614,7 @@ async function handleMemoryStore(params, db2, vectors2) {
         logger.warn("Failed to generate vector embedding", { error: String(error) });
       }
       try {
-        saveExtractions(entry.content, entry.title, entry.scope.owner, entry.scope.repo, db2);
+        await saveExtractions(entry.content, entry.title, entry.scope.owner, entry.scope.repo, db2);
       } catch (error) {
         logger.warn("[KG-Archivist] NLP extraction failed, memory stored without KG enrichment", {
           error: String(error)
@@ -1118,7 +1118,13 @@ function extractAcceptedElicitationContent(result) {
 
 // src/mcp/tools/memory.recap.ts
 async function handleMemoryRecap(params, db2) {
-  const validated = MemoryRecapSchema.parse(params);
+  const parsed = MemoryRecapSchema.safeParse(params);
+  if (!parsed.success) {
+    const missing = parsed.error.issues.filter((i) => i.path.some((p) => p === "owner" || p === "repo")).map((i) => i.message).filter(Boolean);
+    const msg = missing.length > 0 ? `Missing required fields: ${missing.join("; ")}. Pass owner/repo explicitly or configure MCP workspace roots so they can be auto-inferred.` : `Validation error: ${parsed.error.message}`;
+    return { content: [{ type: "text", text: msg }], isError: true };
+  }
+  const validated = parsed.data;
   logger.info("[Tool] memory.recap", { repo: validated.repo, limit: validated.limit, offset: validated.offset });
   const stats = db2.memories.getStats(validated.owner, validated.repo);
   const total = db2.memories.getTotalCount(validated.owner, validated.repo, false, ["task_archive"]);
@@ -1241,7 +1247,13 @@ function capitalize3(str) {
   return str.charAt(0).toUpperCase() + str.slice(1);
 }
 async function handleTaskList(args, storage) {
-  const validated = TaskListSchema.parse(args);
+  const parsed = TaskListSchema.safeParse(args);
+  if (!parsed.success) {
+    const missing = parsed.error.issues.filter((i) => i.path.some((p) => p === "owner" || p === "repo")).map((i) => i.message).filter(Boolean);
+    const msg = missing.length > 0 ? `Missing required fields: ${missing.join("; ")}. Pass owner/repo explicitly or configure MCP workspace roots so they can be auto-inferred.` : `Validation error: ${parsed.error.message}`;
+    return { content: [{ type: "text", text: msg }], isError: true };
+  }
+  const validated = parsed.data;
   const { owner, repo, status, phase, query, limit, offset, structured: isStructuredRequest = false } = validated;
   let statuses = [];
   if (status !== "all") {
@@ -3419,6 +3431,55 @@ async function handleKGBackfill(args, db2) {
   };
   const scanRepos = repo ? [repo] : db2.system.listRepoNavigation().map((r) => r.repo);
   stats.reposScanned = scanRepos.length;
+  const pendingOps = [];
+  for (const r of scanRepos) {
+    const currentOwner = owner || "unknown";
+    if (source === "memories" || source === "both") {
+      const rows = db2.db.prepare("SELECT title, content FROM memories WHERE repo = ?").all(r);
+      for (let i = 0; i < rows.length; i++) {
+        const row = rows[i];
+        const text = `${row.content || ""} ${row.title || ""}`;
+        try {
+          const entities = await extractEntities(text);
+          if (entities.length > 0) {
+            pendingOps.push({
+              entities,
+              repo: r,
+              owner: currentOwner,
+              observationText: `Mentioned in memory: ${row.title || "untitled"}`
+            });
+          }
+        } catch {
+          stats.errors++;
+        }
+        stats.itemsProcessed++;
+        if ((i + 1) % 100 === 0) {
+          logger.info(`[kg-backfill] Processed ${i + 1}/${rows.length} memories in repo "${r}"`);
+        }
+      }
+    }
+    if (source === "standards" || source === "both") {
+      const rows = db2.db.prepare("SELECT title, content FROM coding_standards WHERE repo = ?").all(r);
+      for (let i = 0; i < rows.length; i++) {
+        const row = rows[i];
+        const text = `${row.content || ""} ${row.title || ""}`;
+        try {
+          const entities = await extractEntities(text);
+          if (entities.length > 0) {
+            pendingOps.push({
+              entities,
+              repo: r,
+              owner: currentOwner,
+              observationText: `Mentioned in standard: ${row.title || "untitled"}`
+            });
+          }
+        } catch {
+          stats.errors++;
+        }
+        stats.itemsProcessed++;
+      }
+    }
+  }
   const insertEntity = db2.db.prepare(
     `INSERT OR IGNORE INTO entities (name, type, description, repo, owner, created_at, updated_at)
 		 VALUES (?, ?, ?, ?, ?, ?, ?)`
@@ -3429,66 +3490,12 @@ async function handleKGBackfill(args, db2) {
   );
   const now = (/* @__PURE__ */ new Date()).toISOString();
   const transaction = db2.db.transaction(() => {
-    for (const r of scanRepos) {
-      const currentOwner = owner || "unknown";
-      if (source === "memories" || source === "both") {
-        const rows = db2.db.prepare("SELECT title, content FROM memories WHERE repo = ?").all(r);
-        for (let i = 0; i < rows.length; i++) {
-          const row = rows[i];
-          const content = `${row.content || ""} ${row.title || ""}`;
-          let entities;
-          try {
-            entities = extractEntities(content);
-          } catch {
-            stats.errors++;
-            continue;
-          }
-          for (const ent of entities) {
-            insertEntity.run(ent.name, ent.type, null, r, currentOwner, now, now);
-            stats.entitiesCreated++;
-            insertObservation.run(
-              randomUUID6(),
-              ent.name,
-              `Mentioned in memory: ${row.title || "untitled"}`,
-              r,
-              currentOwner,
-              now
-            );
-            stats.observationsCreated++;
-          }
-          stats.itemsProcessed++;
-          if ((i + 1) % 100 === 0) {
-            logger.info(`[kg-backfill] Processed ${i + 1}/${rows.length} memories in repo "${r}"`);
-          }
-        }
-      }
-      if (source === "standards" || source === "both") {
-        const rows = db2.db.prepare("SELECT title, content FROM coding_standards WHERE repo = ?").all(r);
-        for (let i = 0; i < rows.length; i++) {
-          const row = rows[i];
-          const content = `${row.content || ""} ${row.title || ""}`;
-          let entities;
-          try {
-            entities = extractEntities(content);
-          } catch {
-            stats.errors++;
-            continue;
-          }
-          for (const ent of entities) {
-            insertEntity.run(ent.name, ent.type, null, r, currentOwner, now, now);
-            stats.entitiesCreated++;
-            insertObservation.run(
-              randomUUID6(),
-              ent.name,
-              `Mentioned in standard: ${row.title || "untitled"}`,
-              r,
-              currentOwner,
-              now
-            );
-            stats.observationsCreated++;
-          }
-          stats.itemsProcessed++;
-        }
+    for (const op of pendingOps) {
+      for (const ent of op.entities) {
+        insertEntity.run(ent.name, ent.type, null, op.repo, op.owner, now, now);
+        stats.entitiesCreated++;
+        insertObservation.run(randomUUID6(), ent.name, op.observationText, op.repo, op.owner, now);
+        stats.observationsCreated++;
       }
     }
   });
@@ -3835,8 +3842,8 @@ function registerAllResources(server, store, _vectors, session) {
       mimeType: "application/json"
     },
     async (uri, _extra) => {
-      const repos2 = db2.system.listRepoNavigation();
-      const payload = JSON.stringify(repos2, null, 2);
+      const repos = db2.system.listRepoNavigation();
+      const payload = JSON.stringify(repos, null, 2);
       return {
         contents: [
           {
@@ -3869,15 +3876,29 @@ function registerAllResources(server, store, _vectors, session) {
       };
     }
   );
-  const repos = getRepos();
-  const tags = getTags();
+  function reposLazy() {
+    let cached = null;
+    return () => {
+      if (cached === null) cached = getRepos();
+      return cached;
+    };
+  }
+  function tagsLazy() {
+    let cached = null;
+    return () => {
+      if (cached === null) cached = getTags();
+      return cached;
+    };
+  }
+  const reposFn = reposLazy();
+  const tagsFn = tagsLazy();
   server.registerResource(
     "repository-memories",
     new ResourceTemplate("repository://{name}/memories{?search,type,tag,limit,offset}", {
       list: void 0,
       complete: {
-        name: async (value) => rankCompletionValues(repos, value),
-        tag: async (value) => rankCompletionValues(tags, value)
+        name: async (value) => rankCompletionValues(reposFn(), value),
+        tag: async (value) => rankCompletionValues(tagsFn(), value)
       }
     }),
     {
@@ -3941,7 +3962,7 @@ function registerAllResources(server, store, _vectors, session) {
     new ResourceTemplate("repository://{name}/tasks{?status,priority,limit,offset}", {
       list: void 0,
       complete: {
-        name: async (value) => rankCompletionValues(repos, value)
+        name: async (value) => rankCompletionValues(reposFn(), value)
       }
     }),
     {
@@ -4010,7 +4031,7 @@ function registerAllResources(server, store, _vectors, session) {
     new ResourceTemplate("repository://{name}/summary", {
       list: void 0,
       complete: {
-        name: async (value) => rankCompletionValues(repos, value)
+        name: async (value) => rankCompletionValues(reposFn(), value)
       }
     }),
     {
@@ -4038,7 +4059,7 @@ function registerAllResources(server, store, _vectors, session) {
     new ResourceTemplate("repository://{name}/actions{?limit,offset}", {
       list: void 0,
       complete: {
-        name: async (value) => rankCompletionValues(repos, value)
+        name: async (value) => rankCompletionValues(reposFn(), value)
       }
     }),
     {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vheins/local-memory-mcp",
-	"version": "0.19.4",
+	"version": "0.19.6",
 	"description": "MCP Local Memory Service for coding copilot agents",
 	"mcpName": "io.github.vheins/local-memory-mcp",
 	"type": "module",