seekstone 0.13.1 → 0.14.0

package/README.md

@@ -135,7 +135,7 @@ Other MCP clients (Windsurf, Cline, …) take the Option 3 JSON block in their o
 | `replace_in_note` | Find and replace text in the note body — literal or regex, whole-word, case sensitivity, optional `limit` (replaces **all** occurrences by default), dry-run preview. |
 | `append_periodic_note` | Append to today's periodic note, creating it from a template if it doesn't yet exist. |
 
-The content-editing tools (`append_note`, `patch_note`, `patch_frontmatter`, `replace_in_note`, `append_periodic_note`, and `create_note` with `overwrite: true`) support optional **compare-and-swap**: pass the `contentHash` you got from `read_note` as `prevHash` and the write fails cleanly if the note changed underneath you, instead of silently discarding the concurrent edit.
+Every write tool (`append_note`, `patch_note`, `patch_frontmatter`, `replace_in_note`, `rename_heading`, `move_note`, `delete_note`, `append_periodic_note`, and `create_note` with `overwrite: true`) supports optional **compare-and-swap**: pass the `contentHash` you got from `read_note` as `prevHash` and the call fails cleanly if the note changed underneath you — no silently discarded concurrent edit, no moving or deleting content you haven't seen. Every mutating result returns the new `contentHash`, so chained edits need no re-reads.
 
 ---
 

package/dist/index.js

@@ -701,7 +701,7 @@ ${stringifyYaml(input.frontmatter, { lineWidth: 0 })}---
 }
 
 // src/tools/delete_note.ts
-import { access as access2, mkdir as mkdir2, rename as rename2, rm } from "fs/promises";
+import { access as access2, mkdir as mkdir2, readFile as readFile3, rename as rename2, rm } from "fs/promises";
 import { basename as basename2, extname as extname2, join } from "path";
 import { z as z5 } from "zod";
 
@@ -742,6 +742,9 @@ var DeleteNoteInput = z5.object({
   path: z5.string().describe("Vault-relative path of the note to delete."),
   permanent: z5.boolean().default(false).describe(
     "Permanently remove the note instead of moving it to the vault .trash/ folder. Defaults to false \u2014 deletes are recoverable."
+  ),
+  prevHash: z5.string().optional().describe(
+    "Optional compare-and-swap guard: the contentHash from a prior read. Fails with hash_conflict if the note changed since \u2014 so you never delete content you have not seen."
   )
 });
 async function freeTrashPath(vaultRoot2, noteBase) {
@@ -761,11 +764,12 @@ async function deleteNote(ctx2, rawInput) {
   const input = DeleteNoteInput.parse(rawInput);
   const absPath = resolveVaultPath(ctx2.vaultRoot, input.path);
   assertWritable(ctx2.policy, input.path);
+  const raw = await readFile3(absPath, "utf8");
+  if (input.prevHash !== void 0) assertHashMatch(raw, input.prevHash, input.path);
   let trashedTo;
   if (input.permanent) {
     await rm(absPath);
   } else {
-    await access2(absPath);
     await mkdir2(join(ctx2.vaultRoot, ".trash"), { recursive: true });
     trashedTo = await freeTrashPath(ctx2.vaultRoot, basename2(input.path));
     await rename2(absPath, join(ctx2.vaultRoot, trashedTo));
@@ -775,7 +779,11 @@ async function deleteNote(ctx2, rawInput) {
     ctx2.index.discard(input.path);
     ctx2.notes.delete(input.path);
   }
-  const result = { path: input.path, permanent: input.permanent };
+  const result = {
+    path: input.path,
+    permanent: input.permanent,
+    contentHash: contentHash(raw)
+  };
   if (trashedTo !== void 0) result.trashedTo = trashedTo;
   return result;
 }
@@ -903,7 +911,7 @@ function listTags(ctx2, input) {
 }
 
 // src/tools/move_note.ts
-import { access as access3, mkdir as mkdir3, rename as rename3 } from "fs/promises";
+import { access as access3, mkdir as mkdir3, readFile as readFile4, rename as rename3 } from "fs/promises";
 import { dirname as dirname2, join as join2 } from "path";
 import { z as z10 } from "zod";
 
@@ -1006,6 +1014,9 @@ var MoveNoteInput = z10.object({
   overwrite: z10.boolean().default(false).describe("Overwrite the destination if it exists. Defaults to false."),
   rewriteLinks: z10.boolean().default(true).describe(
     "Rewrite wikilinks and markdown links in other notes that point at the moved note, so nothing breaks. Defaults to true; pass false to move the file only."
+  ),
+  prevHash: z10.string().optional().describe(
+    "Optional compare-and-swap guard on the source note: the contentHash from a prior read. Fails with hash_conflict if the note changed since."
   )
 });
 async function moveNote(ctx2, rawInput) {
@@ -1014,11 +1025,13 @@ async function moveNote(ctx2, rawInput) {
   const absTo = resolveVaultPath(ctx2.vaultRoot, input.to);
   assertWritable(ctx2.policy, input.from);
   assertWritable(ctx2.policy, input.to);
+  let raw;
   try {
-    await access3(absFrom);
+    raw = await readFile4(absFrom, "utf8");
   } catch {
     throw new Error(`Note not found: ${input.from}`);
   }
+  if (input.prevHash !== void 0) assertHashMatch(raw, input.prevHash, input.from);
   if (!input.overwrite) {
     try {
       await access3(absTo);
@@ -1041,7 +1054,6 @@ async function moveNote(ctx2, rawInput) {
   }
   await mkdir3(dirname2(absTo), { recursive: true });
   await rename3(absFrom, absTo);
-  const raw = ctx2.notes.get(input.from)?.raw ?? "";
   removeNoteBacklinks(ctx2, input.from);
   if (ctx2.notes.has(input.from)) {
     ctx2.index.discard(input.from);
@@ -1079,13 +1091,19 @@ async function moveNote(ctx2, rawInput) {
     }
   }
   ctx2.backlinks.delete(input.from);
-  const result = { from: input.from, to: input.to, notesRewritten, linksRewritten };
+  const result = {
+    from: input.from,
+    to: input.to,
+    notesRewritten,
+    linksRewritten,
+    contentHash: contentHash(raw)
+  };
   if (skipped.length > 0) result.skipped = skipped;
   return result;
 }
 
 // src/tools/outline_note.ts
-import { readFile as readFile3 } from "fs/promises";
+import { readFile as readFile5 } from "fs/promises";
 
 // ../core/src/outline.ts
 var HEADING_RE = /^(#{1,6})\s+(.+)/;
@@ -1143,7 +1161,7 @@ var OutlineNoteInput = z11.object({
 });
 async function outlineNote(ctx2, input) {
   const absPath = resolveVaultPath(ctx2.vaultRoot, input.path);
-  const raw = await readFile3(absPath, "utf8");
+  const raw = await readFile5(absPath, "utf8");
   return buildOutline(raw, {
     includeBlocks: input.includeBlocks,
     includeSizes: input.includeSizes
@@ -1151,7 +1169,7 @@ async function outlineNote(ctx2, input) {
 }
 
 // src/tools/patch_frontmatter.ts
-import { readFile as readFile4 } from "fs/promises";
+import { readFile as readFile6 } from "fs/promises";
 import { parseDocument as parseDocument2 } from "yaml";
 import { z as z12 } from "zod";
 var PatchFrontmatterInput = z12.object({
@@ -1166,7 +1184,7 @@ var PatchFrontmatterInput = z12.object({
 async function patchFrontmatter(ctx2, input) {
   const absPath = resolveVaultPath(ctx2.vaultRoot, input.path);
   assertWritable(ctx2.policy, input.path);
-  const original = await readFile4(absPath, "utf8");
+  const original = await readFile6(absPath, "utf8");
   if (input.prevHash !== void 0) assertHashMatch(original, input.prevHash, input.path);
   const fm = parseFrontmatter(original);
   const keysChanged = [];
@@ -1226,7 +1244,7 @@ ${original}`;
 }
 
 // src/tools/patch_note.ts
-import { readFile as readFile5 } from "fs/promises";
+import { readFile as readFile7 } from "fs/promises";
 import { z as z13 } from "zod";
 var PatchNoteInput = z13.object({
   path: z13.string().describe("Vault-relative path to the note."),
@@ -1257,7 +1275,7 @@ function lineStart(raw, offset) {
 async function patchNote(ctx2, input) {
   const absPath = resolveVaultPath(ctx2.vaultRoot, input.path);
   assertWritable(ctx2.policy, input.path);
-  const raw = await readFile5(absPath, "utf8");
+  const raw = await readFile7(absPath, "utf8");
   if (input.prevHash !== void 0) assertHashMatch(raw, input.prevHash, input.path);
   const fm = parseFrontmatter(raw);
   const originalFmRegion = raw.slice(0, fm.bodyStart);
@@ -1357,7 +1375,7 @@ ${input.content}${contentNl}`;
   };
 }
 async function verifyFrontmatter(absPath, originalFmRegion) {
-  const written = await readFile5(absPath, "utf8");
+  const written = await readFile7(absPath, "utf8");
   if (written.slice(0, originalFmRegion.length) !== originalFmRegion) {
     throw new Error("Write-safety violation: frontmatter region changed unexpectedly");
   }
@@ -1372,7 +1390,7 @@ function updateCache(ctx2, path2, newContent, bodyStart) {
 }
 
 // src/tools/periodic_note.ts
-import { access as access4, mkdir as mkdir4, readFile as readFile6 } from "fs/promises";
+import { access as access4, mkdir as mkdir4, readFile as readFile8 } from "fs/promises";
 import { dirname as dirname3, join as join3 } from "path";
 import { z as z14 } from "zod";
 var PERIOD = z14.enum(["daily", "weekly", "monthly", "quarterly", "yearly"]);
@@ -1385,7 +1403,7 @@ var DEFAULTS = {
 };
 async function readJson(absPath) {
   try {
-    return JSON.parse(await readFile6(absPath, "utf8"));
+    return JSON.parse(await readFile8(absPath, "utf8"));
   } catch {
     return null;
   }
@@ -1471,7 +1489,7 @@ async function getPeriodicNote(ctx2, rawInput) {
         ctx2.vaultRoot,
         cfg.template.endsWith(".md") ? cfg.template : `${cfg.template}.md`
       );
-      body = await readFile6(templateAbs, "utf8");
+      body = await readFile8(templateAbs, "utf8");
     } catch {
     }
   }
@@ -1498,7 +1516,7 @@ async function appendPeriodicNote(ctx2, rawInput) {
   assertWritable(ctx2.policy, path2);
   let original = "";
   try {
-    original = await readFile6(abs, "utf8");
+    original = await readFile8(abs, "utf8");
     if (input.prevHash !== void 0) assertHashMatch(original, input.prevHash, path2);
   } catch (err) {
     if (err.code !== "ENOENT") throw err;
@@ -1671,7 +1689,7 @@ function queryNotes(ctx2, input) {
 }
 
 // src/tools/read_note.ts
-import { readFile as readFile7 } from "fs/promises";
+import { readFile as readFile9 } from "fs/promises";
 import { z as z16 } from "zod";
 var ReadNoteInput = z16.object({
   path: z16.string().describe('Vault-relative path to the note, e.g. "Daily Notes/2026-05-29.md".'),
@@ -1689,7 +1707,7 @@ var ReadNoteInput = z16.object({
 });
 async function readNote(ctx2, input) {
   const absPath = resolveVaultPath(ctx2.vaultRoot, input.path);
-  const raw = await readFile7(absPath, "utf8");
+  const raw = await readFile9(absPath, "utf8");
   const noteBytes = Buffer.byteLength(raw, "utf8");
   const hash = contentHash(raw);
   if (input.section === void 0 && input.block === void 0 && input.lines === void 0) {
@@ -1777,7 +1795,7 @@ async function readNote(ctx2, input) {
 }
 
 // src/tools/rename_heading.ts
-import { readFile as readFile8 } from "fs/promises";
+import { readFile as readFile10 } from "fs/promises";
 import { join as join4 } from "path";
 import { z as z17 } from "zod";
 var RenameHeadingInput = z17.object({
@@ -1818,7 +1836,7 @@ async function renameHeading(ctx2, rawInput) {
   assertWritable(ctx2.policy, input.path);
   let raw;
   try {
-    raw = await readFile8(absPath, "utf8");
+    raw = await readFile10(absPath, "utf8");
   } catch {
     throw new Error(`Note not found: ${input.path}`);
   }
@@ -1854,7 +1872,7 @@ async function renameHeading(ctx2, rawInput) {
   newRaw = newRaw.slice(0, fm.bodyStart) + selfRewrite.content;
   const originalFmRegion = raw.slice(0, fm.bodyStart);
   await atomicWrite(absPath, newRaw);
-  const written = await readFile8(absPath, "utf8");
+  const written = await readFile10(absPath, "utf8");
   if (written.slice(0, originalFmRegion.length) !== originalFmRegion) {
     throw new Error("Write-safety violation: frontmatter region changed unexpectedly");
   }
@@ -1900,7 +1918,7 @@ async function renameHeading(ctx2, rawInput) {
 }
 
 // src/tools/replace_in_note.ts
-import { readFile as readFile9 } from "fs/promises";
+import { readFile as readFile11 } from "fs/promises";
 import { z as z18 } from "zod";
 var ReplaceInNoteInput = z18.object({
   path: z18.string().describe("Vault-relative path to the note."),
@@ -1937,7 +1955,7 @@ async function replaceInNote(ctx2, input) {
   } catch (err) {
     throw new Error(`Invalid regex: ${err instanceof Error ? err.message : String(err)}`);
   }
-  const raw = await readFile9(absPath, "utf8");
+  const raw = await readFile11(absPath, "utf8");
   if (input.prevHash !== void 0) assertHashMatch(raw, input.prevHash, input.path);
   const fm = parseFrontmatter(raw);
   const originalFmRegion = raw.slice(0, fm.bodyStart);
@@ -1954,7 +1972,10 @@ async function replaceInNote(ctx2, input) {
       path: input.path,
       replacements: toReplace.length,
       matches: matches2,
-      frontmatterUnchanged: true
+      frontmatterUnchanged: true,
+      // Nothing was written — hash the content as read, so chained edits can
+      // proceed from a dry run without a re-read.
+      contentHash: contentHash(raw)
     };
   }
   const singleRe = new RegExp(re.source, re.flags.replace("g", ""));
@@ -1969,7 +1990,7 @@ async function replaceInNote(ctx2, input) {
   newBody += body.slice(lastIndex);
   const newContent = `${originalFmRegion}${newBody}`;
   await atomicWrite(absPath, newContent);
-  const written = await readFile9(absPath, "utf8");
+  const written = await readFile11(absPath, "utf8");
   if (written.slice(0, originalFmRegion.length) !== originalFmRegion) {
     throw new Error("Write-safety violation: frontmatter region changed unexpectedly");
   }
@@ -2212,7 +2233,7 @@ async function run(ctx2, name, args) {
 }
 
 // src/index/build.ts
-import { readFile as readFile10 } from "fs/promises";
+import { readFile as readFile12 } from "fs/promises";
 import { basename as basename3, extname as extname4 } from "path";
 
 // ../core/src/walk.ts
@@ -2294,7 +2315,7 @@ async function buildIndex(vaultRoot2) {
   await Promise.all(
     noteEntries.map(async (entry) => {
       try {
-        const raw = await readFile10(entry.absPath, "utf8");
+        const raw = await readFile12(entry.absPath, "utf8");
         const fm = parseFrontmatter(raw);
         const inlineTags = extractInlineTags(fm.body);
         const fmTags = frontmatterTags(fm.data);
@@ -2347,7 +2368,7 @@ function buildBacklinks(notes2) {
 
 // src/init.ts
 import { execFileSync } from "child_process";
-import { access as access5, copyFile, mkdir as mkdir5, readdir, readFile as readFile11, stat as stat2, writeFile as writeFile2 } from "fs/promises";
+import { access as access5, copyFile, mkdir as mkdir5, readdir, readFile as readFile13, stat as stat2, writeFile as writeFile2 } from "fs/promises";
 import path, { dirname as dirname4, join as join6 } from "path";
 function parseInitArgs(argv) {
   const opts = { write: false, client: "desktop" };
@@ -2491,7 +2512,7 @@ async function runInit(opts, deps) {
       ]
     };
   }
-  const rf = deps.readFile ?? readFile11;
+  const rf = deps.readFile ?? readFile13;
   let vaultPath = opts.vault ?? deps.env.SEEKSTONE_VAULT;
   if (!vaultPath) {
     const registryPath = obsidianRegistryPath(deps.platform, {
@@ -2981,6 +3002,10 @@ var ALL_TOOLS = [
         permanent: {
           type: "boolean",
           description: "Permanently remove instead of moving to .trash/. Defaults to false."
+        },
+        prevHash: {
+          type: "string",
+          description: "Optional compare-and-swap guard: the contentHash from a prior read. Fails with hash_conflict if the note changed since \u2014 so you never delete content you have not seen."
         }
       },
       required: ["path"]
@@ -3002,6 +3027,10 @@ var ALL_TOOLS = [
         rewriteLinks: {
           type: "boolean",
           description: "Rewrite references in other notes to follow the move. Defaults to true; pass false to move the file only."
+        },
+        prevHash: {
+          type: "string",
+          description: "Optional compare-and-swap guard on the source note: the contentHash from a prior read. Fails with hash_conflict if the note changed since."
         }
       },
       required: ["from", "to"]
@@ -3259,7 +3288,7 @@ function visibleTools(policy2) {
 
 // src/watcher.ts
 import { realpathSync } from "fs";
-import { readFile as readFile12 } from "fs/promises";
+import { readFile as readFile14 } from "fs/promises";
 import { join as join7, relative as relative2, sep as sep3 } from "path";
 import chokidar from "chokidar";
 function startWatcher(ctx2, log2, opts) {
@@ -3272,7 +3301,7 @@ function startWatcher(ctx2, log2, opts) {
   const toRel = (abs) => relative2(vaultRoot2, abs).split(sep3).join("/");
   async function upsert(relPath, op) {
     try {
-      const raw = await readFile12(join7(vaultRoot2, relPath), "utf8");
+      const raw = await readFile14(join7(vaultRoot2, relPath), "utf8");
       removeNoteBacklinks(ctx2, relPath);
       upsertDoc(ctx2, buildDoc(relPath, raw));
       addNoteBacklinks(ctx2, relPath, raw);
@@ -3324,7 +3353,7 @@ function startWatcher(ctx2, log2, opts) {
 }
 
 // src/index.ts
-var VERSION = true ? "0.13.1" : "0.0.0-dev";
+var VERSION = true ? "0.14.0" : "0.0.0-dev";
 var intent = parseCliIntent(process.argv.slice(2));
 if (intent === "version") {
   process.stdout.write(`${VERSION}