@oh-my-pi/pi-mnemopi 18.11.0 → 18.11.2

package/CHANGELOG.md

@@ -2,6 +2,18 @@
 
 ## [Unreleased]
 
+## [18.11.2] - 2026-10-16
+
+### Fixed
+
+- 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))
+
+## [18.11.1] - 2026-10-15
+
+### 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))
+
 ## [18.11.0] - 2026-10-14
 
 ### Fixed

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": "@oh-my-pi/pi-mnemopi",
-	"version": "18.11.0",
+	"version": "18.11.2",
 	"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.11.0",
-		"@oh-my-pi/pi-catalog": "18.11.0",
-		"@oh-my-pi/pi-natives": "18.11.0",
-		"@oh-my-pi/pi-utils": "18.11.0"
+		"@oh-my-pi/pi-ai": "18.11.2",
+		"@oh-my-pi/pi-catalog": "18.11.2",
+		"@oh-my-pi/pi-natives": "18.11.2",
+		"@oh-my-pi/pi-utils": "18.11.2"
 	},
 	"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 "@oh-my-pi/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/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> {