@oh-my-pi/pi-mnemopi 18.8.6 → 18.8.7

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## [Unreleased]
 
+## [18.8.7] - 2026-10-09
+
+### Fixed
+
+- Fixed proactive linking freezing the host for seconds per stored memory on large banks; a memory's graph links are now written in one commit instead of one per link ([#14998](https://github.com/can1357/oh-my-pi/issues/14998)).
+
 ## [18.8.5] - 2026-10-08
 
 ### Fixed

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-mnemopi",
-	"version": "18.8.6",
+	"version": "18.8.7",
 	"description": "Local SQLite memory engine for omp agents",
 	"homepage": "https://omp.sh",
 	"author": {
@@ -42,10 +42,10 @@
 		"fmt": "oxfmt --no-error-on-unmatched-pattern 'src/**/*.{ts,tsx}' '{test,bench,examples,scripts}/**/*.ts' '*.ts'"
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-ai": "18.8.6",
-		"@oh-my-pi/pi-catalog": "18.8.6",
-		"@oh-my-pi/pi-natives": "18.8.6",
-		"@oh-my-pi/pi-utils": "18.8.6"
+		"@oh-my-pi/pi-ai": "18.8.7",
+		"@oh-my-pi/pi-catalog": "18.8.7",
+		"@oh-my-pi/pi-natives": "18.8.7",
+		"@oh-my-pi/pi-utils": "18.8.7"
 	},
 	"peerDependencies": {
 		"fastembed": "3.0.0",

package/src/core/episodic-graph.ts

@@ -1,5 +1,5 @@
 import type { Database } from "bun:sqlite";
-import { closeQuietly, type DatabasePath, openDatabase } from "../db";
+import { closeQuietly, type DatabasePath, openDatabase, transaction } from "../db";
 
 export interface Gist {
 	readonly id: string;
@@ -506,83 +506,92 @@ export class EpisodicGraph {
 		const extractEntities = options.extractEntities ?? true;
 		const gist = this.extractGist(content, memoryId);
 		const facts = extractEntities ? this.extractFacts(content, memoryId) : [];
-		const edges: GraphEdge[] = [];
-		const timestamp = nowIso();
-
-		const previousMemoryIds = linkExisting ? this.knownMemoryIds(memoryId) : [];
-		this.storeGist(gist, memoryId);
-		const gistEdge = { source: memoryId, target: gist.id, edgeType: "ctx", weight: 1, timestamp };
-		this.addEdge(gistEdge);
-		edges.push(gistEdge);
-
-		for (const fact of facts) {
-			this.storeFact(fact, memoryId, sessionId);
-			const edge = {
-				source: gist.id,
-				target: fact.id,
-				edgeType: "rel",
-				weight: fact.confidence,
-				timestamp,
-			};
-			this.addEdge(edge);
-			edges.push(edge);
-		}
+		// One transaction for every gist/fact/edge write: autocommitted, each
+		// `addEdge` paid its own WAL commit (an fsync), so linking a memory
+		// against a large bank blocked the caller's event loop for seconds (#14998).
+		return transaction(this.db, () => {
+			const edges: GraphEdge[] = [];
+			const timestamp = nowIso();
+
+			const previousMemoryIds = linkExisting ? this.knownMemoryIds(memoryId) : [];
+			this.storeGist(gist, memoryId);
+			const gistEdge = { source: memoryId, target: gist.id, edgeType: "ctx", weight: 1, timestamp };
+			this.addEdge(gistEdge);
+			edges.push(gistEdge);
+
+			for (const fact of facts) {
+				this.storeFact(fact, memoryId, sessionId);
+				const edge = {
+					source: gist.id,
+					target: fact.id,
+					edgeType: "rel",
+					weight: fact.confidence,
+					timestamp,
+				};
+				this.addEdge(edge);
+				edges.push(edge);
+			}
 
-		if (linkExisting) {
-			const sourceTokens = contentTokenSet(content);
-			for (const otherId of previousMemoryIds) {
-				const otherContent = this.memoryContent(otherId, timestamp);
-				const lexicalScore = Math.round(jaccard(sourceTokens, contentTokenSet(otherContent)) * 1000) / 1000;
-				let wroteCtxEdge = false;
-				if (lexicalScore >= minLinkScore) {
-					const edge = {
-						source: memoryId,
-						target: otherId,
-						edgeType: "related_to",
-						weight: lexicalScore,
-						timestamp,
-					};
-					this.addEdge(edge);
-					edges.push(edge);
-					const ctxEdge = {
-						source: memoryId,
-						target: otherId,
-						edgeType: "ctx",
-						weight: lexicalScore,
-						timestamp,
-					};
-					this.addEdge(ctxEdge);
-					edges.push(ctxEdge);
-					wroteCtxEdge = true;
-				}
-				const entityScore = this.entityOverlapScore(memoryId, otherId);
-				if (entityScore > 0) {
-					const edge = {
-						source: memoryId,
-						target: otherId,
-						edgeType: "references",
-						weight: entityScore,
-						timestamp,
-					};
-					this.addEdge(edge);
-					edges.push(edge);
-				}
-				const contextualScore = Math.max(lexicalScore, entityScore, this.temporalContextScore(memoryId, otherId));
-				if (!wroteCtxEdge && contextualScore >= minLinkScore) {
-					const ctxEdge = {
-						source: memoryId,
-						target: otherId,
-						edgeType: "ctx",
-						weight: contextualScore,
-						timestamp,
-					};
-					this.addEdge(ctxEdge);
-					edges.push(ctxEdge);
+			if (linkExisting) {
+				const sourceTokens = contentTokenSet(content);
+				for (const otherId of previousMemoryIds) {
+					const otherContent = this.memoryContent(otherId, timestamp);
+					const lexicalScore = Math.round(jaccard(sourceTokens, contentTokenSet(otherContent)) * 1000) / 1000;
+					let wroteCtxEdge = false;
+					if (lexicalScore >= minLinkScore) {
+						const edge = {
+							source: memoryId,
+							target: otherId,
+							edgeType: "related_to",
+							weight: lexicalScore,
+							timestamp,
+						};
+						this.addEdge(edge);
+						edges.push(edge);
+						const ctxEdge = {
+							source: memoryId,
+							target: otherId,
+							edgeType: "ctx",
+							weight: lexicalScore,
+							timestamp,
+						};
+						this.addEdge(ctxEdge);
+						edges.push(ctxEdge);
+						wroteCtxEdge = true;
+					}
+					const entityScore = this.entityOverlapScore(memoryId, otherId);
+					if (entityScore > 0) {
+						const edge = {
+							source: memoryId,
+							target: otherId,
+							edgeType: "references",
+							weight: entityScore,
+							timestamp,
+						};
+						this.addEdge(edge);
+						edges.push(edge);
+					}
+					const contextualScore = Math.max(
+						lexicalScore,
+						entityScore,
+						this.temporalContextScore(memoryId, otherId),
+					);
+					if (!wroteCtxEdge && contextualScore >= minLinkScore) {
+						const ctxEdge = {
+							source: memoryId,
+							target: otherId,
+							edgeType: "ctx",
+							weight: contextualScore,
+							timestamp,
+						};
+						this.addEdge(ctxEdge);
+						edges.push(ctxEdge);
+					}
 				}
 			}
-		}
 
-		return { memoryId, gist, facts, edges };
+			return { memoryId, gist, facts, edges };
+		});
 	}
 	getStats(): GraphStats {
 		const gists = this.count("gists");

package/src/db.ts

@@ -119,10 +119,12 @@ export function transaction<T>(db: Database, fn: () => T): T {
 			state.depth--;
 		}
 	}
-
+	// Caller-owned Bun/SQLite transactions already batch these writes; do not
+	// commit or roll them back from this helper.
+	if (db.inTransaction) return fn();
+	db.exec("BEGIN DEFERRED");
 	state = { depth: 1 };
 	txDb[TX_STATE] = state;
-	db.exec("BEGIN DEFERRED");
 	try {
 		const result = fn();
 		state.depth = 0;
@@ -154,10 +156,10 @@ export async function transactionAsync<T>(db: Database, fn: () => Promise<T>): P
 			state.depth--;
 		}
 	}
-
+	if (db.inTransaction) return await fn();
+	db.exec("BEGIN DEFERRED");
 	state = { depth: 1 };
 	txDb[TX_STATE] = state;
-	db.exec("BEGIN DEFERRED");
 	try {
 		const result = await fn();
 		state.depth = 0;