toon-memory 1.6.1 → 1.6.3

package/dist/cli/setup.js

@@ -1,4 +1,4 @@
-import { existsSync, mkdirSync, readFileSync, writeFileSync, cpSync, unlinkSync, readdirSync, statSync } from "fs";
+import { existsSync, mkdirSync, readFileSync, writeFileSync, unlinkSync, readdirSync, statSync } from "fs";
 import { basename, dirname, join } from "path";
 import { fileURLToPath } from "url";
 import { execSync } from "child_process";
@@ -66,15 +66,11 @@ function detectAgents() {
   return agents2;
 }
 function installOpenCodeTools() {
-  const toolsDir = join(projectRoot, ".opencode", "tools");
   const memoryDir = join(projectRoot, ".opencode", "memory");
   const memoryFile = join(memoryDir, "data.toon");
-  if (!existsSync(toolsDir)) mkdirSync(toolsDir, { recursive: true });
   if (!existsSync(memoryDir)) mkdirSync(memoryDir, { recursive: true });
-  cpSync(join(sourceDir, "memory.ts"), join(toolsDir, "memory.ts"));
-  console.log("  Copied memory.ts to .opencode/tools/");
   if (!existsSync(memoryFile)) {
-    writeFileSync(memoryFile, "version: 1\nentries[0|]{id|category|key|content|file|tags|date}:\n");
+    writeFileSync(memoryFile, "version: 1\n[0|]\n");
     console.log("  Created .opencode/memory/data.toon");
   }
 }

package/mcp/server.js

@@ -27796,19 +27796,36 @@ server.registerTool(
     const id = generateId();
     const date5 = (/* @__PURE__ */ new Date()).toISOString().split("T")[0];
     const lines = data.split("\n");
-    let headerIdx = lines.findIndex((l) => l.startsWith("entries["));
+    let headerIdx = lines.findIndex((l) => l.startsWith("entries[") || /^\[\d+\|]/.test(l));
     if (headerIdx === -1) {
-      lines.push(`entries[0|]{id|category|key|content|file|tags|date}:`);
+      lines.push(`[0|]`);
       headerIdx = lines.length - 1;
     }
-    const match = lines[headerIdx].match(/entries\[(\d+)\|/);
-    const count = match ? parseInt(match[1]) : 0;
+    let existingIdx = -1;
+    for (let i = headerIdx + 1; i < lines.length; i++) {
+      const line = lines[i];
+      if (!line.startsWith("  ") || !line.includes("|")) continue;
+      if (line.startsWith("  summaries:")) break;
+      const parts = line.trim().split("|");
+      if (parts[2] === key) {
+        existingIdx = i;
+        break;
+      }
+    }
     const newEntry = `${id}|${category}|${key}|${content}|${file2 || ""}|${tags || ""}|${date5}`;
-    lines.splice(headerIdx + 1, 0, `  ${newEntry}`);
-    lines[headerIdx] = lines[headerIdx].replace(/entries\[\d+\|/, `[${count + 1}|`);
+    let action = "Guardado";
+    if (existingIdx !== -1) {
+      lines[existingIdx] = `  ${newEntry}`;
+      action = "Actualizado";
+    } else {
+      const match = lines[headerIdx].match(/\[(\d+)\|/);
+      const count = match ? parseInt(match[1]) : 0;
+      lines.splice(headerIdx + 1, 0, `  ${newEntry}`);
+      lines[headerIdx] = lines[headerIdx].replace(/\[\d+\|/, `[${count + 1}|`);
+    }
     writeMemory(lines.join("\n"));
     return {
-      content: [{ type: "text", text: `\u{1F9E0} Guardado: ${category}/${key} (${id})
+      content: [{ type: "text", text: `\u{1F9E0} ${action}: ${category}/${key} (${id})
 ${content}` }]
     };
   }
@@ -27827,7 +27844,7 @@ server.registerTool(
   },
   async ({ query, category, from_date, to_date }) => {
     const data = readMemory();
-    const lines = data.split("\n").filter((l) => l.startsWith("  ") && l.includes("|"));
+    const lines = data.split("\n").filter((l) => l.startsWith("  ") && l.includes("|") && !l.startsWith("summaries:"));
     const queryLower = query.toLowerCase();
     const results = lines.map((line) => {
       const trimmed = line.trim();
@@ -27862,19 +27879,19 @@ server.registerTool(
   async ({ key }) => {
     const data = readMemory();
     const lines = data.split("\n");
-    const headerIdx = lines.findIndex((l) => l.startsWith("entries["));
+    const headerIdx = lines.findIndex((l) => l.startsWith("entries[") || /^\[\d+\|]/.test(l));
     if (headerIdx === -1) {
       return { content: [{ type: "text", text: "No hay entradas en memoria" }] };
     }
-    const entryLines = lines.slice(headerIdx + 1).filter((l) => l.trim().length > 0);
+    const entryLines = lines.slice(headerIdx + 1).filter((l) => l.trim().length > 0 && !l.startsWith("summaries:"));
     const filtered = entryLines.filter((l) => {
       const parts = l.trim().split("|");
       return parts[0] !== key && parts[2] !== key;
     });
     const removed = entryLines.length - filtered.length;
-    const match = lines[headerIdx].match(/entries\[(\d+)\|/);
+    const match = lines[headerIdx].match(/\[(\d+)\|/);
     const count = match ? parseInt(match[1]) : 0;
-    lines[headerIdx] = lines[headerIdx].replace(/entries\[\d+\|/, `[${count - removed}|`);
+    lines[headerIdx] = lines[headerIdx].replace(/\[\d+\|/, `[${count - removed}|`);
     lines.splice(headerIdx + 1, entryLines.length, ...filtered.map((l) => `  ${l.trim()}`));
     writeMemory(lines.join("\n"));
     return {
@@ -27891,7 +27908,7 @@ server.registerTool(
   },
   async () => {
     const data = readMemory();
-    const lines = data.split("\n").filter((l) => l.startsWith("  ") && l.includes("|"));
+    const lines = data.split("\n").filter((l) => l.startsWith("  ") && l.includes("|") && !l.startsWith("  summaries:"));
     const entries = lines.map((l) => {
       const parts = l.trim().split("|");
       return { category: parts[1] || "unknown" };
@@ -27900,7 +27917,7 @@ server.registerTool(
     for (const e of entries) {
       byCategory[e.category] = (byCategory[e.category] || 0) + 1;
     }
-    const summaryLines = data.split("\n").filter((l) => l.includes(":") && !l.startsWith("  ") && !l.startsWith("version") && !l.startsWith("entries"));
+    const summaryLines = data.split("\n").filter((l) => l.includes(":") && !l.startsWith("  ") && !l.startsWith("version") && !l.startsWith("entries") && !/^\[\d+\|]/.test(l));
     const stats = [
       `Entradas totales: ${entries.length}`,
       `Res\xFAmenes de archivos: ${summaryLines.length}`,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "toon-memory",
-  "version": "1.6.1",
+  "version": "1.6.3",
   "description": "Persistent memory system for AI coding agents using TOON format (40% fewer tokens than JSON)",
   "type": "module",
   "bin": {

package/src/cli/setup.ts

@@ -116,10 +116,10 @@ function detectAgents(): Agent[] {
 }
 
 /**
- * Install custom tools and memory directory for OpenCode.
+ * Install memory directory for OpenCode.
  * 
- * Creates `.opencode/tools/` and `.opencode/memory/` directories,
- * copies `memory.ts` tool, and creates initial `data.toon` if needed.
+ * Creates `.opencode/memory/` directory and initial `data.toon` if needed.
+ * The MCP server handles all memory operations - no plugin needed.
  * 
  * @example
  * ```bash
@@ -127,18 +127,13 @@ function detectAgents(): Agent[] {
  * ```
  */
 function installOpenCodeTools(): void {
-  const toolsDir = join(projectRoot, ".opencode", "tools")
   const memoryDir = join(projectRoot, ".opencode", "memory")
   const memoryFile = join(memoryDir, "data.toon")
 
-  if (!existsSync(toolsDir)) mkdirSync(toolsDir, { recursive: true })
   if (!existsSync(memoryDir)) mkdirSync(memoryDir, { recursive: true })
 
-  cpSync(join(sourceDir, "memory.ts"), join(toolsDir, "memory.ts"))
-  console.log("  Copied memory.ts to .opencode/tools/")
-
   if (!existsSync(memoryFile)) {
-    writeFileSync(memoryFile, "version: 1\nentries[0|]{id|category|key|content|file|tags|date}:\n")
+    writeFileSync(memoryFile, "version: 1\n[0|]\n")
     console.log("  Created .opencode/memory/data.toon")
   }
 }

package/src/mcp/server.ts

@@ -310,22 +310,43 @@ server.registerTool(
     const date = new Date().toISOString().split("T")[0]
     const lines = data.split("\n")
 
-    let headerIdx = lines.findIndex((l) => l.startsWith("entries["))
+    let headerIdx = lines.findIndex((l) => l.startsWith("entries[") || /^\[\d+\|]/.test(l))
     if (headerIdx === -1) {
-      lines.push(`entries[0|]{id|category|key|content|file|tags|date}:`)
+      lines.push(`[0|]`)
       headerIdx = lines.length - 1
     }
 
-    const match = lines[headerIdx].match(/entries\[(\d+)\|/)
-    const count = match ? parseInt(match[1]) : 0
+    // Find existing entry with same key (upsert)
+    let existingIdx = -1
+    for (let i = headerIdx + 1; i < lines.length; i++) {
+      const line = lines[i]
+      if (!line.startsWith("  ") || !line.includes("|")) continue
+      if (line.startsWith("  summaries:")) break
+      const parts = line.trim().split("|")
+      if (parts[2] === key) {
+        existingIdx = i
+        break
+      }
+    }
+
     const newEntry = `${id}|${category}|${key}|${content}|${file || ""}|${tags || ""}|${date}`
+    let action = "Guardado"
 
-    lines.splice(headerIdx + 1, 0, `  ${newEntry}`)
-    lines[headerIdx] = lines[headerIdx].replace(/entries\[\d+\|/, `[${count + 1}|`)
+    if (existingIdx !== -1) {
+      // Update existing entry
+      lines[existingIdx] = `  ${newEntry}`
+      action = "Actualizado"
+    } else {
+      // Create new entry
+      const match = lines[headerIdx].match(/\[(\d+)\|/)
+      const count = match ? parseInt(match[1]) : 0
+      lines.splice(headerIdx + 1, 0, `  ${newEntry}`)
+      lines[headerIdx] = lines[headerIdx].replace(/\[\d+\|/, `[${count + 1}|`)
+    }
 
     writeMemory(lines.join("\n"))
     return {
-      content: [{ type: "text" as const, text: `🧠 Guardado: ${category}/${key} (${id})\n${content}` }],
+      content: [{ type: "text" as const, text: `🧠 ${action}: ${category}/${key} (${id})\n${content}` }],
     }
   }
 )
@@ -355,7 +376,7 @@ server.registerTool(
   },
   async ({ query, category, from_date, to_date }) => {
     const data = readMemory()
-    const lines = data.split("\n").filter((l) => l.startsWith("  ") && l.includes("|"))
+    const lines = data.split("\n").filter((l) => l.startsWith("  ") && l.includes("|") && !l.startsWith("summaries:"))
     const queryLower = query.toLowerCase()
 
     const results = lines
@@ -406,22 +427,22 @@ server.registerTool(
   async ({ key }) => {
     const data = readMemory()
     const lines = data.split("\n")
-    const headerIdx = lines.findIndex((l) => l.startsWith("entries["))
+    const headerIdx = lines.findIndex((l) => l.startsWith("entries[") || /^\[\d+\|]/.test(l))
 
     if (headerIdx === -1) {
       return { content: [{ type: "text" as const, text: "No hay entradas en memoria" }] }
     }
 
-    const entryLines = lines.slice(headerIdx + 1).filter((l) => l.trim().length > 0)
+    const entryLines = lines.slice(headerIdx + 1).filter((l) => l.trim().length > 0 && !l.startsWith("summaries:"))
     const filtered = entryLines.filter((l) => {
       const parts = l.trim().split("|")
       return parts[0] !== key && parts[2] !== key
     })
 
     const removed = entryLines.length - filtered.length
-    const match = lines[headerIdx].match(/entries\[(\d+)\|/)
+    const match = lines[headerIdx].match(/\[(\d+)\|/)
     const count = match ? parseInt(match[1]) : 0
-    lines[headerIdx] = lines[headerIdx].replace(/entries\[\d+\|/, `[${count - removed}|`)
+    lines[headerIdx] = lines[headerIdx].replace(/\[\d+\|/, `[${count - removed}|`)
     lines.splice(headerIdx + 1, entryLines.length, ...filtered.map((l) => `  ${l.trim()}`))
 
     writeMemory(lines.join("\n"))
@@ -449,7 +470,7 @@ server.registerTool(
   },
   async () => {
     const data = readMemory()
-    const lines = data.split("\n").filter((l) => l.startsWith("  ") && l.includes("|"))
+    const lines = data.split("\n").filter((l) => l.startsWith("  ") && l.includes("|") && !l.startsWith("  summaries:"))
     const entries = lines.map((l) => {
       const parts = l.trim().split("|")
       return { category: parts[1] || "unknown" }
@@ -460,7 +481,7 @@ server.registerTool(
       byCategory[e.category] = (byCategory[e.category] || 0) + 1
     }
 
-    const summaryLines = data.split("\n").filter((l) => l.includes(":") && !l.startsWith("  ") && !l.startsWith("version") && !l.startsWith("entries"))
+    const summaryLines = data.split("\n").filter((l) => l.includes(":") && !l.startsWith("  ") && !l.startsWith("version") && !l.startsWith("entries") && !/^\[\d+\|]/.test(l))
     const stats = [
       `Entradas totales: ${entries.length}`,
       `Resúmenes de archivos: ${summaryLines.length}`,