@oh-my-pi/pi-mnemopi 16.3.12 → 16.3.13

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-mnemopi",
-	"version": "16.3.12",
+	"version": "16.3.13",
 	"description": "Local SQLite memory engine for Oh My Pi agents",
 	"homepage": "https://omp.sh",
 	"author": "Can Boluk",
@@ -39,9 +39,9 @@
 		"fmt": "biome format --write ."
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-ai": "16.3.12",
-		"@oh-my-pi/pi-catalog": "16.3.12",
-		"@oh-my-pi/pi-utils": "16.3.12",
+		"@oh-my-pi/pi-ai": "16.3.13",
+		"@oh-my-pi/pi-catalog": "16.3.13",
+		"@oh-my-pi/pi-utils": "16.3.13",
 		"lru-cache": "11.5.1"
 	},
 	"peerDependencies": {

package/src/core/beam/store.ts

@@ -665,7 +665,46 @@ export function get(beam: BeamMemoryState, memoryId: string): Row | null {
 			WHERE id = ? AND (session_id = ? OR scope = 'global')
 		`)
 		.get(memoryId, beam.sessionId) as Row | null | undefined;
-	return episodic == null ? null : { ...episodic, metadata: episodic.metadata_json, memory_store: "episodic" };
+	if (episodic != null) return { ...episodic, metadata: episodic.metadata_json, memory_store: "episodic" };
+
+	return getFact(beam, memoryId);
+}
+
+/**
+ * Read-only resolution for ids minted from the `facts` table. `recall`
+ * surfaces `facts.fact_id` as a result id (`factRecall`), so `get` must
+ * resolve those ids too — otherwise every surfaced fact id is a dead end
+ * for the read path (issue #4725). Visibility mirrors `factRecall`:
+ * same-session facts plus explicitly global ones (`scope` is an optional
+ * column on `facts`; `SELECT *` tolerates banks without it, in which case
+ * only same-session facts resolve). The row is shaped like the
+ * working/episodic hits with the full triple as content;
+ * `memory_store: "fact"` marks it read-only — no update/forget/invalidate
+ * path mutates `facts`.
+ */
+function getFact(beam: BeamMemoryState, memoryId: string): Row | null {
+	const fact = beam.db.prepare("SELECT * FROM facts WHERE fact_id = ?").get(memoryId) as Row | null | undefined;
+	if (fact == null) return null;
+	if (fact.session_id !== beam.sessionId && fact.scope !== "global") return null;
+	const subject = typeof fact.subject === "string" ? fact.subject : "";
+	const predicate = typeof fact.predicate === "string" ? fact.predicate : "";
+	const object = typeof fact.object === "string" ? fact.object : "";
+	return {
+		id: fact.fact_id,
+		content: [subject, predicate, object].filter(part => part.length > 0).join(" "),
+		source: "facts",
+		timestamp: fact.timestamp ?? null,
+		session_id: fact.session_id ?? null,
+		importance: fact.confidence ?? null,
+		metadata: JSON.stringify({
+			subject,
+			predicate,
+			object,
+			source_msg_id: fact.source_msg_id ?? null,
+		}),
+		created_at: fact.created_at ?? null,
+		memory_store: "fact",
+	};
 }
 
 export function forgetWorking(beam: BeamMemoryState, memoryId: string): boolean {