@oh-my-pi/pi-utils 18.9.1 → 18.10.0

package/CHANGELOG.md

@@ -2,6 +2,26 @@
 
 ## [Unreleased]
 
+## [18.10.0] - 2026-10-12
+
+### Added
+
+- Added `getMnemopiEmbedRuntimeDir()` for the shared mnemopi embedding server endpoints and logs.
+
+### Fixed
+
+- Fixed `ChildProcess.kill()` silently leaving the child running when the native process handle could not be resolved or used; it now signals the child directly and escalates to SIGKILL after the grace period.
+
+## [18.9.2] - 2026-10-11
+
+### Changed
+
+- Improved streaming JSON parsing performance for tool arguments containing many string values.
+
+### Fixed
+
+- Fixed parsing of long strings that contain closing delimiters other than double or single quotes.
+
 ## [18.9.0] - 2026-10-11
 
 ### Breaking Changes

package/dist/types/dirs.d.ts

@@ -320,6 +320,8 @@ export declare function getDebugLogPath(agentDir?: string): string;
 export declare function getSecretPlaceholderKeyPath(): string;
 /** Directory holding the per-model tiny-worker sockets and logs (~/.omp/run/tiny; XDG default: $XDG_STATE_HOME/omp/run/tiny). */
 export declare function getTinyWorkerRuntimeDir(): string;
+/** Directory holding the per-model mnemopi embedding server endpoints and logs (~/.omp/run/mnemopi; XDG default: $XDG_STATE_HOME/omp/run/mnemopi). */
+export declare function getMnemopiEmbedRuntimeDir(): string;
 /** Root directory containing every per-project daemon runtime scope (~/.omp/run/daemons; XDG default: $XDG_STATE_HOME/omp/run/daemons). */
 export declare function getDaemonRuntimeRoot(): string;
 /** Get the daemon runtime directory for a project (~/.omp/run/daemons/<hash>; XDG default: $XDG_STATE_HOME/omp/run/daemons/<hash>). */

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-utils",
-	"version": "18.9.1",
+	"version": "18.10.0",
 	"description": "Shared utilities for pi packages",
 	"keywords": [
 		"cli",
@@ -57,7 +57,7 @@
 		"fmt": "oxfmt --no-error-on-unmatched-pattern 'src/**/*.{ts,tsx}' '{test,bench,examples,scripts}/**/*.ts' '*.ts'"
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-natives": "18.9.1"
+		"@oh-my-pi/pi-natives": "18.10.0"
 	},
 	"devDependencies": {
 		"@types/bun": "^1.3.14"

package/src/dirs.ts

@@ -1082,6 +1082,11 @@ export function getTinyWorkerRuntimeDir(): string {
 	return dirs.rootSubdir(path.join("run", "tiny"), "state");
 }
 
+/** Directory holding the per-model mnemopi embedding server endpoints and logs (~/.omp/run/mnemopi; XDG default: $XDG_STATE_HOME/omp/run/mnemopi). */
+export function getMnemopiEmbedRuntimeDir(): string {
+	return dirs.rootSubdir(path.join("run", "mnemopi"), "state");
+}
+
 /** Root directory containing every per-project daemon runtime scope (~/.omp/run/daemons; XDG default: $XDG_STATE_HOME/omp/run/daemons). */
 export function getDaemonRuntimeRoot(): string {
 	return dirs.rootSubdir(path.join("run", "daemons"), "state");

package/src/json-lexer.ts

@@ -186,6 +186,17 @@ const enum QuoteLook {
  */
 export class JsonLexer {
 	pos: number;
+	/**
+	 * Shared across {@link string} calls: `src` has no backslash in
+	 * `[#backslashFrom, #backslashAt)` and `#backslashAt` is a backslash or
+	 * `src.length`. An escape-free run can extend past its string's closing
+	 * quote to the end of the document, so searching per string would rescan
+	 * the same tail for every sibling string (quadratic on objects with many
+	 * short values). `src` is immutable, so the range never goes stale; a
+	 * cursor outside it (e.g. a rollback) re-searches.
+	 */
+	#backslashFrom = 0;
+	#backslashAt = -1;
 
 	constructor(
 		readonly src: string,
@@ -228,13 +239,16 @@ export class JsonLexer {
 		// malformed structure fails loudly instead of silently swallowing
 		// commas/colons or sibling members.
 		const lenient = quote === SQUOTE || this.mode === "streaming";
-		const quoteChar = quote === QUOTE ? '"' : "'";
+		// Any delimiter code unit, not just `"`/`'`; a value above 0xFFFF matches
+		// no code unit, so such a string never closes.
+		const quoteChar = String.fromCharCode(quote);
 		// Next quote / backslash at or after the cursor, cached across iterations
 		// and re-searched only once the cursor passes them, so every character is
 		// scanned at most once per kind and the bulk path stays linear even on
 		// escape-dense input (re-searching the far closing quote after each
-		// escape would be quadratic). `n` means "none before end of input".
-		let nextQuote = -1;
+		// escape would be quadratic). `n` means "none before end of input". The
+		// backslash search result is shared across calls via `#backslashAt`.
+		let nextQuote = quoteChar.charCodeAt(0) === quote ? -1 : n;
 		let nextBackslash = -1;
 		while (i < n) {
 			// Bulk-skip a run of ordinary characters: jump to the nearest quote or
@@ -249,8 +263,12 @@ export class JsonLexer {
 					nextQuote = q === -1 ? n : q;
 				}
 				if (nextBackslash < i) {
-					const b = s.indexOf("\\", i);
-					nextBackslash = b === -1 ? n : b;
+					if (i < this.#backslashFrom || i > this.#backslashAt) {
+						const b = s.indexOf("\\", i);
+						this.#backslashFrom = i;
+						this.#backslashAt = b === -1 ? n : b;
+					}
+					nextBackslash = this.#backslashAt;
 				}
 				i = nextQuote < nextBackslash ? nextQuote : nextBackslash;
 				if (i >= n) break;

package/src/ptree.ts

@@ -7,7 +7,7 @@
  * - Convenience helpers: captureText / execText, AbortSignal, timeouts.
  */
 
-import { Process } from "@oh-my-pi/pi-natives";
+import { Process, type ProcessTerminateOptions } from "@oh-my-pi/pi-natives";
 import type { Spawn, Subprocess } from "bun";
 
 type InMask = "pipe" | "ignore" | Buffer | Uint8Array | null;
@@ -387,12 +387,40 @@ export class ChildProcess<In extends InMask = InMask> {
 						? { group: true }
 						: undefined
 					: { gracefulMs, group: this.#terminateGroup };
-			this.#terminating = (this.#windowsRootProcess ?? Process.fromPid(this.proc.pid))
-				?.terminate(options)
-				?.catch(e => void e);
+			this.#terminating = this.#terminateTree(options);
 		}
 	}
 
+	/**
+	 * Native tree termination. A native handle that cannot be resolved or used
+	 * — the PID left the process table, or a `bun test --isolate` realm whose
+	 * napi classes another realm superseded ("Value is not an instance of class
+	 * `Process`") — must not leave the child running, so signal it directly.
+	 */
+	#terminateTree(options: ProcessTerminateOptions | undefined): Promise<boolean | void> {
+		const gracefulMs = options?.gracefulMs ?? 1000;
+		try {
+			const root = this.#windowsRootProcess ?? Process.fromPid(this.proc.pid);
+			if (root) return root.terminate(options).catch(() => this.#signalChild(gracefulMs));
+		} catch {}
+		return this.#signalChild(gracefulMs);
+	}
+
+	/** SIGTERM the child (its group when it leads one), then SIGKILL if it outlives `gracefulMs`. */
+	async #signalChild(gracefulMs: number): Promise<void> {
+		const signal = (name: NodeJS.Signals) => {
+			try {
+				if (this.#terminateGroup && process.platform !== "win32") process.kill(-this.proc.pid, name);
+				else this.proc.kill(name);
+			} catch {}
+		};
+		if (gracefulMs >= 0) {
+			signal("SIGTERM");
+			await Promise.race([this.proc.exited, Bun.sleep(gracefulMs)]);
+		}
+		if (this.proc.exitCode === null && this.proc.signalCode === null) signal("SIGKILL");
+	}
+
 	async #hardKillSubreaperTree(root: Process): Promise<void> {
 		try {
 			const deadline = Date.now() + SUBREAPER_KILL_WINDOW_MS;