@vheins/local-memory-mcp 0.19.3 → 0.19.5

package/dist/{chunk-R45NRES5.js → chunk-ZSGQQS5F.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'");
@@ -3208,7 +3229,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 +3519,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 +3563,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 +4288,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 +4309,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 +4437,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 +4518,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 +4649,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 +4759,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 +4770,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 +4787,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 +5661,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,
@@ -6683,7 +6722,9 @@ 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"
 });
@@ -6906,10 +6947,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(),