mcp-memory-bucket 0.12.3 → 0.12.4

package/dist/src/memory/repository.js

@@ -629,9 +629,11 @@ export class MemoryRepository {
         const existing = await this.get(folder, filename);
         if (!existing)
             throw new Error(`memory doc "${filename}" not found`);
-        // `paused` is local-cache-only and must never reach writeMarkdownFile — split it off of
-        // `existing` before spreading the rest into the frontmatter that gets written to disk.
-        const { paused: existingPaused, ...existingForFile } = existing;
+        // `paused` is local-cache-only and `body` is written separately as the markdown body, not
+        // frontmatter — both must be split off of `existing` before spreading the rest into
+        // `merged`, or they'd ride along into the MemoryFrontmatter written to disk (duplicating the
+        // body inside its own frontmatter as a `body:` key).
+        const { paused: existingPaused, body: _existingBody, ...existingForFile } = existing;
         const merged = {
             ...existingForFile,
             ...frontmatter,

package/dist/src/skills/repository.js

@@ -531,9 +531,11 @@ export class SkillRepository {
         const existing = await this.get(name, folder);
         if (!existing)
             throw new Error(`skill with name "${name}" not found`);
-        // `paused` is local-cache-only and must never reach writeMarkdownFile — split it off of
-        // `existing` before spreading the rest into the frontmatter that gets written to disk.
-        const { paused: existingPaused, ...existingForFile } = existing;
+        // `paused` is local-cache-only and `body` is written separately as the markdown body, not
+        // frontmatter — both must be split off of `existing` before spreading the rest into
+        // `merged`, or they'd ride along into the SkillFrontmatter written to disk (duplicating the
+        // body inside its own frontmatter as a `body:` key).
+        const { paused: existingPaused, body: _existingBody, ...existingForFile } = existing;
         // Builtin skills (e.g. memory-bucket-authoring) are the server's own always-present
         // documentation, not user content — deprecating them would hide guidance every session needs.
         const deprecated = this.isBuiltin(existing) ? existing.deprecated : frontmatter?.deprecated;
@@ -585,7 +587,9 @@ export class SkillRepository {
         const newFilePath = path.join(newDir, 'SKILL.md');
         // Rename changes the skill's id, so it becomes a fresh cache row — paused (local-only,
         // keyed by id) does not carry over, same as it wouldn't survive deleting the cache file.
-        const { paused: _existingPaused, ...existingForFile } = existing;
+        // `body` is split off too, for the same reason as in update() — it must not ride along into
+        // the SkillFrontmatter written to disk.
+        const { paused: _existingPaused, body: _existingBody, ...existingForFile } = existing;
         const merged = { ...existingForFile, name: newName };
         const fileContents = formatMarkdownFile(stripSourcePath(merged), existing.body);
         const remote = this.remoteFor(existing.folder);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mcp-memory-bucket",
-  "version": "0.12.3",
+  "version": "0.12.4",
   "type": "module",
   "description": "MCP server exposing skill_* (reusable coding patterns) and memory_* (point-in-time working context) tools over a markdown+frontmatter source, cached into SQLite at runtime.",
   "repository": {