@owrede/vault-memory 2.0.0-rc.5 → 2.0.0

package/CHANGELOG.md

@@ -10,6 +10,16 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 > with today's date, and a fresh `## [Unreleased]` block is started above it. See the
 > bottom of this file for the one-paragraph cut-a-release recipe.
 
+## [Unreleased]
+
+_Nothing yet._
+
+## [2.0.0] — 2026-06-25
+
+### Fixed
+
+- **Live indexer no longer crashes on duplicate-anchor sibling sections** (`ISSUE-indexer-duplicate-anchor.md`). A note containing two sibling sections with identical heading + body produces the same content-hash anchor and collided on `UNIQUE(note_id, anchor)`, aborting the entire `vault-memory index` / `index --full` run with `SQLITE_CONSTRAINT_UNIQUE`. `SectionsQueries` now uses `INSERT OR IGNORE` and a new `insertOneResolving()` that returns the surviving row's id on collision (so `parent_id` linkage still resolves); `buildSectionsForNote` uses it inside a per-note try/catch so one pathological note can never abort the whole vault's index. Mirrors the migration-010 backfill fix already shipped. Verified end-to-end: a full index of a vault with a colliding note exits 0.
+
 ## [2.0.0-rc.1] — 2026-05-19
 
 

package/dist/cli.js

@@ -3056,7 +3056,7 @@ var init_sections = __esm({
       constructor(db) {
         this.db = db;
         this._insert = db.prepare(`
-      INSERT INTO sections
+      INSERT OR IGNORE INTO sections
         (note_id, anchor, heading_path, heading_text, level,
          parent_id, ord, chunk_id_first, chunk_id_last, created_at)
       VALUES
@@ -3126,6 +3126,33 @@ var init_sections = __esm({
         tx(rows);
         return ids;
       }
+      /**
+       * Insert one section, collision-safe. Returns the id of the row that
+       * now owns (note_id, anchor): the freshly inserted row, or — when a
+       * same-anchor sibling already won the unique slot — that surviving
+       * row's id (so callers can resolve parent_id linkage). Mirrors the
+       * backfill behavior in src/sections/backfill.ts. The live indexer
+       * uses this instead of `insertMany` so duplicate-anchor sibling
+       * headings can't abort the whole index run
+       * (see ISSUE-indexer-duplicate-anchor.md).
+       */
+      insertOneResolving(r) {
+        const info = this._insert.run({
+          note_id: r.note_id,
+          anchor: r.anchor,
+          heading_path: r.heading_path,
+          heading_text: r.heading_text,
+          level: r.level,
+          parent_id: r.parent_id,
+          ord: r.ord,
+          chunk_id_first: r.chunk_id_first,
+          chunk_id_last: r.chunk_id_last,
+          created_at: Date.now()
+        });
+        if (info.changes > 0) return Number(info.lastInsertRowid);
+        const existing = this._getByAnchor.get(r.note_id, r.anchor);
+        return existing ? Number(existing.id) : null;
+      }
       deleteByNote(noteId) {
         return this._deleteByNote.run(noteId).changes;
       }
@@ -6352,7 +6379,14 @@ async function indexVault(vault, options) {
         chunkIdFragment: computeChunkIdFragment(c.text)
       }));
       const chunkIds = vault.db.chunks.insertBatch(noteId, chunkInputs);
-      buildSectionsForNote(vault, noteId, parsed.content, chunkIds);
+      try {
+        buildSectionsForNote(vault, noteId, parsed.content, chunkIds);
+      } catch (err) {
+        const message = err instanceof Error ? err.message : String(err);
+        console.error(
+          `[indexer:${vault.config.name}] section build failed for ${parsed.relativePath}: ${message} \u2014 skipping sections for this note`
+        );
+      }
       const embedResult = await options.ollama.embed({
         model: options.embeddingModel,
         texts: chunks.map((c) => c.text)
@@ -6517,15 +6551,15 @@ function buildSectionsForNote(vault, noteId, content, insertedChunkIds) {
       chunk_id_first: pair.first,
       chunk_id_last: pair.last
     };
-    const ids = vault.db.sections.insertMany([row]);
-    insertedIds.push(ids[0]);
+    insertedIds.push(vault.db.sections.insertOneResolving(row));
   }
   return insertedIds.length;
 }
 function mapChunksToSections(chunks, sectionRanges) {
-  const out = sectionRanges.map(
-    () => ({ first: null, last: null })
-  );
+  const out = sectionRanges.map(() => ({
+    first: null,
+    last: null
+  }));
   for (const chunk of chunks) {
     const offset = chunk.start_offset;
     let chosenIdx = null;