@oh-my-pi/pi-mnemopi 18.14.2 → 18.16.0

package/dist/types/util/env.d.ts

@@ -5,6 +5,8 @@ export declare function envOptionalString(name: string, env?: Env): string | und
 export declare function envTruthy(name: string, env?: Env): boolean;
 export declare function envDisabled(name: string, env?: Env): boolean;
 export declare function envBool(name: string, defaultValue: boolean, env?: Env): boolean;
+/** Unset or blank → `defaultValue`; otherwise true only for `1`/`true`/`yes`/`on`, so a malformed value disables. */
+export declare function envFlag(name: string, defaultValue: boolean, env?: Env): boolean;
 export declare function envInt(name: string, defaultValue: number, env?: Env): number;
 export declare function envFloat(name: string, defaultValue: number, env?: Env): number;
 export declare function envOneOf<T extends string>(name: string, allowed: readonly T[], defaultValue: T, env?: Env): T;

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-mnemopi",
-	"version": "18.14.2",
+	"version": "18.16.0",
 	"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.14.2",
-		"@oh-my-pi/pi-catalog": "18.14.2",
-		"@oh-my-pi/pi-natives": "18.14.2",
-		"@oh-my-pi/pi-utils": "18.14.2"
+		"@oh-my-pi/pi-ai": "18.16.0",
+		"@oh-my-pi/pi-catalog": "18.16.0",
+		"@oh-my-pi/pi-natives": "18.16.0",
+		"@oh-my-pi/pi-utils": "18.16.0"
 	},
 	"peerDependencies": {
 		"fastembed": "3.0.0",

package/src/core/extraction.ts

@@ -1,4 +1,4 @@
-import { withLoopPhase } from "@oh-my-pi/pi-utils";
+import { isRecord, withLoopPhase } from "@oh-my-pi/pi-utils";
 import { getDiagnostics, safeForLog } from "./extraction/diagnostics";
 import { callHostLlm, getHostLlmBackend } from "./llm-backends";
 import {
@@ -12,29 +12,14 @@ import {
 } from "./local-llm";
 import { getMnemopiRuntimeOptions } from "./runtime-options";
 import { clipToWindow, DEFAULT_INPUT_CHARS } from "./text-window";
-
-const TRUE_VALUES: Record<string, true> = { "1": true, true: true, yes: true, on: true };
-
-function env(name: string): string {
-	return process.env[name] ?? "";
-}
-
-function envBool(name: string, defaultValue: boolean): boolean {
-	const value = env(name).trim().toLowerCase();
-	return value === "" ? defaultValue : TRUE_VALUES[value] === true;
-}
-
-function envInt(name: string, defaultValue: number): number {
-	const parsed = Number.parseInt(env(name), 10);
-	return Number.isFinite(parsed) ? parsed : defaultValue;
-}
+import { envFlag, envInt, envString } from "../util/env";
 
 function llmEnabled(): boolean {
-	return envBool("MNEMOPI_LLM_ENABLED", true);
+	return envFlag("MNEMOPI_LLM_ENABLED", true);
 }
 
 function hostLlmEnabled(): boolean {
-	return envBool("MNEMOPI_HOST_LLM_ENABLED", false);
+	return envFlag("MNEMOPI_HOST_LLM_ENABLED", false);
 }
 
 function llmMaxTokens(): number {
@@ -42,7 +27,7 @@ function llmMaxTokens(): number {
 }
 
 export const EXTRACTION_PROMPT_TEMPLATE =
-	env("MNEMOPI_EXTRACTION_PROMPT") ||
+	envString("MNEMOPI_EXTRACTION_PROMPT") ||
 	`You are an expert structured memory extractor for Mnemopi v3.0+ MEMORIA tables.
 The user message below may be in English, German, Russian, or another language.
 First detect the language, then extract ONLY high-signal, long-term relevant items.
@@ -159,10 +144,6 @@ function normalizeFactArray(items: unknown, options: FactArrayOptions): string[]
 	return out;
 }
 
-function isRecord(value: unknown): value is Record<string, unknown> {
-	return value !== null && typeof value === "object" && !Array.isArray(value);
-}
-
 function triplePart(value: unknown): string {
 	return typeof value === "string" ? normalizeFact(value) : "";
 }
@@ -341,8 +322,8 @@ async function tryHostExtraction(prompt: string): Promise<[boolean, string | nul
 		maxTokens: llmMaxTokens(),
 		temperature: 0,
 		timeout: 15,
-		provider: env("MNEMOPI_HOST_LLM_PROVIDER").trim() || null,
-		model: env("MNEMOPI_HOST_LLM_MODEL").trim() || null,
+		provider: envString("MNEMOPI_HOST_LLM_PROVIDER").trim() || null,
+		model: envString("MNEMOPI_HOST_LLM_MODEL").trim() || null,
 	});
 	const text = typeof raw === "string" ? raw.trim() : "";
 	return [true, text === "" ? null : text];

package/src/core/local-llm.ts

@@ -11,6 +11,7 @@ import {
 import { ProviderHttpError } from "@oh-my-pi/pi-ai/error";
 import { fetchWithRetry } from "@oh-my-pi/pi-utils";
 import { type CompleteOptions, callHostLlm, getHostLlmBackend } from "./llm-backends";
+import { envFlag, envInt, envString } from "../util/env";
 import {
 	getMnemopiRuntimeOptions,
 	isPiAiModel,
@@ -29,12 +30,6 @@ export const DEFAULT_MODEL_REPO =
 export const DEFAULT_MODEL_FILE =
 	ENV_MODEL_REPO !== "" && ENV_MODEL_FILE !== "" ? ENV_MODEL_FILE : "tinyllama-1.1b-chat-v1.0.Q4_K_M.gguf";
 
-const TRUE_VALUES: Record<string, true> = { "1": true, true: true, yes: true, on: true };
-
-function env(name: string): string {
-	return process.env[name] ?? "";
-}
-
 function activeLlmOptions() {
 	return getMnemopiRuntimeOptions()?.llm;
 }
@@ -48,16 +43,6 @@ function activePiAiModel(): Model<Api> | undefined {
 	return isPiAiModel(model) ? model : undefined;
 }
 
-function envBool(name: string, defaultValue: boolean): boolean {
-	const value = env(name).trim().toLowerCase();
-	return value === "" ? defaultValue : TRUE_VALUES[value] === true;
-}
-
-function envInt(name: string, defaultValue: number): number {
-	const parsed = Number.parseInt(env(name), 10);
-	return Number.isFinite(parsed) ? parsed : defaultValue;
-}
-
 function stripTrailingSlash(value: string): string {
 	let end = value.length;
 	while (end > 0 && value.charCodeAt(end - 1) === 47) {
@@ -74,7 +59,7 @@ function llmEnabled(): boolean {
 	if (activeCustomCompletion() !== undefined || activePiAiModel() !== undefined) {
 		return true;
 	}
-	return envBool("MNEMOPI_LLM_ENABLED", true);
+	return envFlag("MNEMOPI_LLM_ENABLED", true);
 }
 
 function llmMaxTokens(): number {
@@ -97,7 +82,7 @@ function hostLlmEnabled(): boolean {
 	if (active?.baseUrl !== undefined || (typeof active?.model === "string" && active.model !== "")) {
 		return false;
 	}
-	return envBool("MNEMOPI_HOST_LLM_ENABLED", false);
+	return envFlag("MNEMOPI_HOST_LLM_ENABLED", false);
 }
 
 function hostLlmContextTokens(): number {
@@ -109,7 +94,7 @@ function llmBaseUrl(): string {
 	if (active?.baseUrl !== undefined) {
 		return stripTrailingSlash(active.baseUrl);
 	}
-	return stripTrailingSlash(env("MNEMOPI_LLM_BASE_URL"));
+	return stripTrailingSlash(envString("MNEMOPI_LLM_BASE_URL"));
 }
 
 function llmModelName(): string {
@@ -117,7 +102,7 @@ function llmModelName(): string {
 	if (typeof model === "string") {
 		return model;
 	}
-	return env("MNEMOPI_LLM_MODEL") || "local";
+	return envString("MNEMOPI_LLM_MODEL") || "local";
 }
 
 function llmApiKey(): ApiKey {
@@ -125,11 +110,11 @@ function llmApiKey(): ApiKey {
 	if (active?.apiKey !== undefined) {
 		return active.apiKey;
 	}
-	return env("MNEMOPI_LLM_API_KEY");
+	return envString("MNEMOPI_LLM_API_KEY");
 }
 
 function sleepPrompt(): string {
-	return env("MNEMOPI_SLEEP_PROMPT").trim();
+	return envString("MNEMOPI_SLEEP_PROMPT").trim();
 }
 
 function memoryLines(memories: readonly string[]): string {
@@ -248,8 +233,8 @@ async function tryHostLlm(prompt: string, maxTokens: number, temperature: number
 		maxTokens,
 		temperature,
 		timeout: 15,
-		provider: env("MNEMOPI_HOST_LLM_PROVIDER").trim() || null,
-		model: env("MNEMOPI_HOST_LLM_MODEL").trim() || null,
+		provider: envString("MNEMOPI_HOST_LLM_PROVIDER").trim() || null,
+		model: envString("MNEMOPI_HOST_LLM_MODEL").trim() || null,
 	});
 	const text = typeof raw === "string" ? raw.trim() : "";
 	return [true, text === "" ? null : text];
@@ -418,7 +403,7 @@ async function summarizeChunk(
 		return null;
 	}
 
-	if (llmEnabled() && llmBaseUrl() !== "" && !envBool("MNEMOPI_FORCE_LOCAL", false)) {
+	if (llmEnabled() && llmBaseUrl() !== "" && !envFlag("MNEMOPI_FORCE_LOCAL", false)) {
 		const raw = await callRemoteLlm(prompt, 0.3, options);
 		if (raw !== null) {
 			const cleaned = cleanOutput(raw);
@@ -475,7 +460,7 @@ export async function complete(
 	if (attempted) {
 		return hostText;
 	}
-	if (llmEnabled() && llmBaseUrl() !== "" && !envBool("MNEMOPI_FORCE_LOCAL", false)) {
+	if (llmEnabled() && llmBaseUrl() !== "" && !envFlag("MNEMOPI_FORCE_LOCAL", false)) {
 		const remote = await callRemoteLlm(prompt, temperature, options);
 		return remote === null ? null : cleanOutput(remote) || null;
 	}

package/src/util/env.ts

@@ -36,6 +36,13 @@ export function envBool(name: string, defaultValue: boolean, env: Env = process.
 	return defaultValue;
 }
 
+/** Unset or blank → `defaultValue`; otherwise true only for `1`/`true`/`yes`/`on`, so a malformed value disables. */
+export function envFlag(name: string, defaultValue: boolean, env: Env = process.env): boolean {
+	const value = env[name]?.trim().toLowerCase();
+	if (!value) return defaultValue;
+	return TRUE_VALUES[value] === true;
+}
+
 export function envInt(name: string, defaultValue: number, env: Env = process.env): number {
 	const raw = env[name]?.trim();
 	if (!raw) return defaultValue;