@linxiraos/pi-mnemopi 1.2.1 → 1.3.0

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 ## [Unreleased]
 
+## [1.3.0] - 2026-10-18
+
+### Fixed
+
+- Fixed a failing `mnemopi` CLI command or MCP memory call dropping the embeddings of memories it had just stored ([#15804](https://github.com/can1357/oh-my-pi/pull/15804) by [@yuzu-octopus](https://github.com/yuzu-octopus))
+- Fixed polyphonic recall ranking unrelated memories above relevant ones when the query shared only a common word such as "that" with an extracted fact subject ([#15512](https://github.com/can1357/oh-my-pi/issues/15512)).
+- Fixed `MNEMOPI_<LABEL>_WEIGHT` and `MNEMOPI_TIER<N>_WEIGHT` having no effect on recall ranking ([#13824](https://github.com/can1357/oh-my-pi/pull/13824) by [@sjawhar](https://github.com/sjawhar))
+
 ## [1.2.0] - 2026-10-12
 
 - 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)).

package/THIRD-PARTY-NOTICES.txt

@@ -8,8 +8,8 @@ no rights in third-party code, fonts, data, or marks identified below.
 PYTHON DEPENDENCY DISTRIBUTION ROLE
 ===================================
 
-omp-rpc has no third-party runtime dependencies. robomp declares FastAPI,
-Uvicorn, HTTPX, Pydantic, pydantic-settings, python-dotenv, Click, and omp-rpc
+zeta-rpc has no third-party runtime dependencies. robomp declares FastAPI,
+Uvicorn, HTTPX, Pydantic, pydantic-settings, python-dotenv, Click, and zeta-rpc
 as ordinary installer-resolved dependencies. The Python wheels do not vendor
 those projects; each dependency remains a separately installed distribution
 carrying its own license metadata and files. Container/system assemblers must
@@ -79,9 +79,9 @@ uutils coreutils (https://github.com/uutils/coreutils), MIT
 --------------------------------------------------------------------------------
 Covers: base32, base64, basename, cat, cksum (shared checksum machinery),
 b2sum, md5sum, sha1sum, sha224sum, sha256sum, sha384sum, sha512sum, comm, cp,
-cut, date, dirname, head, hostname, ln, ls, mkdir, mktemp, mv, nproc, paste,
-printenv, readlink, realpath, rm, seq, sort, stat, tac, tail, tee, touch, tr,
-truncate, uname, uniq, wc, whoami, yes.
+cut, date, dirname, head, hostname, ln, ls, mkdir, mktemp, mv, nl, nproc,
+paste, printenv, readlink, realpath, rm, seq, sort, stat, tac, tail, tee,
+touch, tr, truncate, uname, uniq, wc, whoami, yes.
 
   Copyright (c) uutils developers
 
@@ -824,6 +824,56 @@ public licenses.
 
 Creative Commons may be contacted at creativecommons.org.
 
+-------------------------------------------------------------------------------
+scripts/brand/fonts/ANSI Shadow.flf, scripts/brand/fonts/mini.flf
+
+Two FIGlet font files from the standard FIGlet font distribution — the
+figlet.org font set shipped by figlet.js
+(https://github.com/patorjk/figlet.js/tree/master/fonts), fetched from
+figlet.js 1.8.0 — vendored so the welcome/splash brand art
+(`packages/tui/src/prompt/welcome.ts`) is generated reproducibly and offline
+by `bun scripts/brand/generate-logo-art.ts`. Both files are unmodified and
+retain their original FIGlet headers. They are build-time inputs only: no font
+file or font data is compiled into, or shipped with, a Zeta release artifact.
+The fonts' own headers carry their terms:
+
+- `ANSI Shadow.flf` — "ANSI Shadow". Author unknown ("Font Author: ?" in the
+  header); a TheDraw-archive font converted with the FIGlet font editor
+  (http://patorjk.com/figfont-editor). Distributed under the FIGlet font
+  license as carried by the standard FIGlet font set: free to use and
+  redistribute with the header intact.
+- `mini.flf` — "mini" by Glenn Chappell (4/93), from figlet release 2.1
+  (12 Aug 1994), "Includes ISO Latin-1"; modified by Paul Burton
+  <solution@earthlink.net> 12/96. Header grant: "Permission is hereby given to
+  modify this font, as long as the modifier's name is placed on a comment
+  line."
+
+FIGlet renderer — `figlet` 1.8.0 (https://github.com/patorjk/figlet.js), a
+root devDependency used only to generate the art at build time; it is not
+imported by any runtime bundle or shipped artifact.
+
+  MIT License
+
+  Copyright (C) 2014-present Patrick Gillespie and other contributors
+
+  Permission is hereby granted, free of charge, to any person obtaining a copy
+  of this software and associated documentation files (the "Software"), to deal
+  in the Software without restriction, including without limitation the rights
+  to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+  copies of the Software, and to permit persons to whom the Software is
+  furnished to do so, subject to the following conditions:
+
+  The above copyright notice and this permission notice shall be included in
+  all copies or substantial portions of the Software.
+
+  THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+  IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+  FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+  AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+  LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+  OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+  SOFTWARE.
+
 BUN / NPM DEPENDENCY INVENTORY
 ==============================
 This inventory is traversed from bun.lock, not from the host node_modules tree.

package/dist/types/cli.d.ts

@@ -16,6 +16,7 @@ export declare class CliError extends Error {
     constructor(message: string, exitCode?: number);
 }
 type CommandHandler = (args: readonly string[], context?: CliContext) => number | Promise<number>;
+export declare function withMemory<T>(context: CliContext | undefined, fn: (memory: BeamMemory) => T | Promise<T>): Promise<T>;
 export declare function memoryStats(memory: BeamMemory, dataDir?: string): Record<string, unknown>;
 export declare const cmdExport: CommandHandler;
 export declare const cmdImport: CommandHandler;

package/dist/types/config.d.ts

@@ -56,8 +56,11 @@ export declare function scratchpadMaxItems(env?: Env): number;
 export declare function recencyHalflifeHours(env?: Env): number;
 export declare function tier2Days(env?: Env): number;
 export declare function tier3Days(env?: Env): number;
+/** Recall-time score multiplier for an episodic row still at degradation tier 1. */
 export declare function tier1Weight(env?: Env): number;
+/** Recall-time score multiplier for an episodic row degraded to tier 2. */
 export declare function tier2Weight(env?: Env): number;
+/** Recall-time score multiplier for an episodic row degraded to tier 3 or beyond. */
 export declare function tier3Weight(env?: Env): number;
 export declare function degradeBatchSize(env?: Env): number;
 export declare function smartCompressEnabled(env?: Env): boolean;

package/dist/types/mcp-tools.d.ts

@@ -1,3 +1,4 @@
+import { BeamMemory } from "./core/beam/index.js";
 export type JsonPrimitive = string | number | boolean | null;
 export type JsonValue = JsonPrimitive | JsonValue[] | {
     [key: string]: JsonValue;
@@ -480,5 +481,6 @@ export declare const GRAPH_LINK_SCHEMA: {
     readonly required: readonly ["source_id", "target_id", "relationship"];
 };
 export declare const TOOLS: readonly ToolDefinition[];
+export declare function withBeam<T>(args: ToolArguments, fn: (beam: BeamMemory, bank: string) => T | Promise<T>): Promise<T>;
 export declare function handleToolCall(name: string, args?: ToolArguments): Promise<ToolResult>;
 export declare function getToolDefinitions(): readonly ToolDefinition[];

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@linxiraos/pi-mnemopi",
-	"version": "1.2.1",
+	"version": "1.3.0",
 	"description": "Local SQLite memory engine for Zeta agents",
 	"homepage": "https://linxira-os.github.io/zeta/",
 	"author": "Linxira-OS",
@@ -32,17 +32,17 @@
 	},
 	"scripts": {
 		"check": "oxlint . && oxfmt --check --no-error-on-unmatched-pattern 'src/**/*.{ts,tsx}' '{test,bench,examples,scripts}/**/*.ts' '*.ts' && bun run check:types",
-		"check:types": "tsgo -p tsconfig.json --noEmit",
+		"check:types": "tsc -p tsconfig.json --noEmit",
 		"lint": "oxlint .",
 		"test": "bun test --parallel",
 		"fix": "oxlint --fix --fix-suggestions . && bun run fmt",
 		"fmt": "oxfmt --no-error-on-unmatched-pattern 'src/**/*.{ts,tsx}' '{test,bench,examples,scripts}/**/*.ts' '*.ts'"
 	},
 	"dependencies": {
-		"@linxiraos/pi-ai": "1.2.1",
-		"@linxiraos/pi-catalog": "1.2.1",
-		"@linxiraos/pi-natives": "1.2.1",
-		"@linxiraos/pi-utils": "1.2.1"
+		"@linxiraos/pi-ai": "1.3.0",
+		"@linxiraos/pi-catalog": "1.3.0",
+		"@linxiraos/pi-natives": "1.3.0",
+		"@linxiraos/pi-utils": "1.3.0"
 	},
 	"peerDependencies": {
 		"fastembed": "3.0.0",

package/src/cli.ts

@@ -77,17 +77,22 @@ function getMemory(context?: CliContext): { memory: BeamMemory; owned: boolean }
 	return { memory: new BeamMemory({ dbPath: resolveDbPath(context) }), owned: true };
 }
 
-async function withMemory<T>(context: CliContext | undefined, fn: (memory: BeamMemory) => T | Promise<T>): Promise<T> {
+export async function withMemory<T>(
+	context: CliContext | undefined,
+	fn: (memory: BeamMemory) => T | Promise<T>,
+): Promise<T> {
 	const { memory, owned } = getMemory(context);
 	try {
-		const result = await fn(memory);
+		return await fn(memory);
+	} finally {
 		// Drain background fact-extraction and embedding tasks before close so
 		// short-lived owners (e.g. `mnemopi store …` / `mnemopi sleep …`) don't
 		// race the SQLite handle shut from under in-flight `embed()` writes.
-		if (owned) await memory.flushExtractions();
-		return result;
-	} finally {
-		if (owned) memory.close();
+		// Runs on both success and throw paths.
+		if (owned) {
+			await memory.flushExtractions();
+			memory.close();
+		}
 	}
 }
 

package/src/config.ts

@@ -170,16 +170,19 @@ export function tier3Days(env: Env = process.env): number {
 	return envInt("MNEMOPI_TIER3_DAYS", 180, env);
 }
 
+/** Recall-time score multiplier for an episodic row still at degradation tier 1. */
 export function tier1Weight(env: Env = process.env): number {
 	return envFloat("MNEMOPI_TIER1_WEIGHT", 1.0, env);
 }
 
+/** Recall-time score multiplier for an episodic row degraded to tier 2. */
 export function tier2Weight(env: Env = process.env): number {
-	return envFloat("MNEMOPI_TIER2_WEIGHT", 0.5, env);
+	return envFloat("MNEMOPI_TIER2_WEIGHT", 0.85, env);
 }
 
+/** Recall-time score multiplier for an episodic row degraded to tier 3 or beyond. */
 export function tier3Weight(env: Env = process.env): number {
-	return envFloat("MNEMOPI_TIER3_WEIGHT", 0.25, env);
+	return envFloat("MNEMOPI_TIER3_WEIGHT", 0.7, env);
 }
 
 export function degradeBatchSize(env: Env = process.env): number {

package/src/core/beam/recall.ts

@@ -1,5 +1,17 @@
 import { popLoopPhase, pushLoopPhase, withLoopPhase } from "@linxiraos/pi-utils";
-import { normalizedRecallWeights, polyphonicRecallEnabled, temporalHalflifeHours } from "../../config";
+import {
+	importedWeight,
+	inferredWeight,
+	normalizedRecallWeights,
+	polyphonicRecallEnabled,
+	statedWeight,
+	temporalHalflifeHours,
+	tier1Weight,
+	tier2Weight,
+	tier3Weight,
+	toolWeight,
+	unknownWeight,
+} from "../../config";
 import { hasCjk, matchesWordForm } from "../../util/regex";
 import { embedQuery } from "../embeddings";
 import { mmrRerank } from "../mmr";
@@ -63,16 +75,33 @@ type RecallMmrItem = {
 	readonly [key: string]: unknown;
 };
 
-const VERACITY_WEIGHTS: Record<string, number> = {
-	stated: 1.0,
-	true: 1.0,
-	likely_true: 1.0,
-	unknown: 0.8,
-	inferred: 0.7,
-	imported: 0.6,
-	tool: 0.5,
-	false: 0,
-};
+/**
+ * Score multipliers for a candidate's stored `veracity` label and, for episodic
+ * rows, its degradation tier. Resolved once per {@link recall} from the
+ * `MNEMOPI_<LABEL>_WEIGHT` and `MNEMOPI_TIER<N>_WEIGHT` readers in `config.ts`,
+ * clamped at 0 like the hybrid weights so a negative value cannot rank a row
+ * below one labelled `false`; `true`, `likely_true` and `false` have no override.
+ */
+interface TrustWeights {
+	readonly veracity: Readonly<Record<string, number>> & { readonly unknown: number };
+	readonly tiers: readonly [tier1: number, tier2: number, tier3: number];
+}
+
+function resolveTrustWeights(): TrustWeights {
+	return {
+		veracity: {
+			stated: Math.max(0, statedWeight()),
+			true: 1.0,
+			likely_true: 1.0,
+			unknown: Math.max(0, unknownWeight()),
+			inferred: Math.max(0, inferredWeight()),
+			imported: Math.max(0, importedWeight()),
+			tool: Math.max(0, toolWeight()),
+			false: 0,
+		},
+		tiers: [Math.max(0, tier1Weight()), Math.max(0, tier2Weight()), Math.max(0, tier3Weight())],
+	};
+}
 
 /**
  * Default per-result content preview cap enforced by {@link recall}. Content
@@ -671,6 +700,7 @@ function scoreCandidate(
 	queryTokens: readonly string[],
 	queryGroups: readonly (readonly string[])[],
 	weights: readonly [number, number, number],
+	trust: TrustWeights,
 	options: RecallOptionsInternal,
 ): RecallResult | null {
 	const content = asString(candidate.row.content);
@@ -714,11 +744,11 @@ function scoreCandidate(
 		score *= 1 + temporalWeight * temporalScore;
 	}
 	const veracity = asString(candidate.row.veracity) || "unknown";
-	const veracityWeight = VERACITY_WEIGHTS[veracity] ?? VERACITY_WEIGHTS.unknown ?? 0.8;
+	const veracityWeight = trust.veracity[veracity] ?? trust.veracity.unknown;
 	const degradationTier = candidate.tierLabel === "episodic" ? asNumber(candidate.row.tier, 1) : undefined;
 	if (candidate.tierLabel === "episodic") {
-		const tierWeight = degradationTier === 1 ? 1 : degradationTier === 2 ? 0.85 : 0.7;
-		score *= tierWeight;
+		const [tier1, tier2, tier3] = trust.tiers;
+		score *= degradationTier === 1 ? tier1 : degradationTier === 2 ? tier2 : tier3;
 	}
 	score *= veracityWeight * currentContentAdjustment(searchableContent, options.currentSensitive === true);
 	const preview = clipRecallContent(content, options.contentPreviewChars ?? RECALL_CONTENT_PREVIEW_CHARS);
@@ -938,9 +968,10 @@ export async function recall(
 		const tokens = expandedTokens(query, useSynonyms);
 		const tokenGroups = expandedTokenGroups(query, useSynonyms);
 		const candidates = collectMemoryCandidates(beam, query, topK, temporalOptions);
+		const trust = resolveTrustWeights();
 		const scored: RecallResult[] = [];
 		for (const candidate of candidates) {
-			const result = scoreCandidate(candidate, tokens, tokenGroups, weights, temporalOptions);
+			const result = scoreCandidate(candidate, tokens, tokenGroups, weights, trust, temporalOptions);
 			if (result !== null) scored.push(result);
 		}
 		scored.sort((left, right) => (right.score ?? 0) - (left.score ?? 0));

package/src/core/polyphonic-recall.ts

@@ -5,6 +5,7 @@ import { closeQuietly, type DatabasePath, openDatabase } from "../db";
 import { backfillConsolidatedFacts, ensureVeracityConsolidator } from "./beam/consolidate";
 import type { BeamMemoryState, JsonValue, Metadata, RecallResult, RecallTierLabel } from "./beam/types";
 import { EpisodicGraph } from "./episodic-graph";
+import { STOP_WORDS } from "./synonyms";
 import { VeracityConsolidator } from "./veracity-consolidation";
 
 export type PolyphonicVoice = "hybrid" | "vector" | "graph" | "fact" | "temporal";
@@ -229,15 +230,20 @@ function queryTokens(query: string): string[] {
 	return tokens;
 }
 
-/** Subject candidates for the fact voice: every 3+ char query word plus adjacent word pairs. */
+/**
+ * Subject candidates for the fact voice: every 3+ char query word plus adjacent word pairs.
+ * Stopwords never anchor a subject on their own — the regex extractor stores sentence-initial
+ * words like `That` as subjects, so a bare `that` in the query would pull in unrelated memories.
+ */
 function factSubjectCandidates(query: string): string[] {
 	const tokens = queryTokens(query);
 	const seen = new Set<string>();
 	for (let i = 0; i < tokens.length; i++) {
 		const word = tokens[i] as string;
-		if (word.length >= 3) seen.add(word);
+		const isContent = !STOP_WORDS.has(word);
+		if (isContent && word.length >= 3) seen.add(word);
 		const next = tokens[i + 1];
-		if (next !== undefined) seen.add(`${word} ${next}`);
+		if (next !== undefined && (isContent || !STOP_WORDS.has(next))) seen.add(`${word} ${next}`);
 	}
 	return [...seen];
 }

package/src/mcp-tools.ts

@@ -447,20 +447,24 @@ function sharedBeam(): BeamMemory {
 	return new BeamMemory({ sessionId: "mcp_shared_surface", dbPath });
 }
 
-async function withBeam<T>(args: ToolArguments, fn: (beam: BeamMemory, bank: string) => T | Promise<T>): Promise<T> {
+export async function withBeam<T>(
+	args: ToolArguments,
+	fn: (beam: BeamMemory, bank: string) => T | Promise<T>,
+): Promise<T> {
 	const bank = resolveBank(args);
 	const beam = createBeam(args, bank);
+	let result: T;
 	try {
-		const result = await fn(beam, bank);
+		result = await fn(beam, bank);
+	} finally {
 		// Drain background fact-extraction and embedding tasks before close so
 		// the SQLite handle stays open until in-flight `embed()` writes commit;
 		// otherwise the short-lived MCP `remember`/`update`/`sleep` paths race
 		// the close and silently drop the new dense-recall rows.
 		await beam.flushExtractions();
-		return result;
-	} finally {
 		beam.close();
 	}
+	return result;
 }
 function serialize(value: unknown): unknown {
 	if (value instanceof Date) return value.toISOString();
@@ -739,13 +743,14 @@ function surfaceLabel(content: string, kind: string): string {
 
 async function withSharedBeam<T>(fn: (beam: BeamMemory) => T | Promise<T>): Promise<T> {
 	const beam = sharedBeam();
+	let result: T;
 	try {
-		const result = await fn(beam);
-		await beam.flushExtractions();
-		return result;
+		result = await fn(beam);
 	} finally {
+		await beam.flushExtractions();
 		beam.close();
 	}
+	return result;
 }
 
 async function handleSharedRemember(args: ToolArguments): Promise<ToolResult> {