@oh-my-pi/pi-utils 17.0.4 → 17.0.5

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 ## [Unreleased]
 
+## [17.0.5] - 2026-07-18
+
+### Changed
+
+- Updated `installRuntimeModuleResolver` to return an uninstaller function that restores the stock `node:module` resolver once all runtime roots are unregistered.
+- Added documentation regarding a known limitation with Bun 1.3.14's `createRequire` behavior when the module resolver patch is active.
+
 ## [17.0.2] - 2026-07-17
 
 ### Added

package/dist/types/env.d.ts

@@ -64,6 +64,20 @@ export declare function isTerminalHeadless(): boolean;
  * so callers can restore exact prior state (`const prev = setTerminalHeadless(false); … setTerminalHeadless(prev);`).
  */
 export declare function setTerminalHeadless(headless: boolean): boolean;
+/**
+ * True when this process runs an interactive coding-agent host — the only
+ * context where the operator can browse the Agent Hub and focus a live
+ * subagent's session (`SessionFocusController`), so a subagent's session title
+ * can become operator-visible. Off by default (print/RPC/ACP/eval/SDK/`bun
+ * test` never render a focusable session tree); the interactive entrypoint
+ * flips it on with {@link setInteractiveHost}.
+ */
+export declare function isInteractiveHost(): boolean;
+/**
+ * Set the interactive-host flag and return the previous value so callers can
+ * restore exact prior state. See {@link isInteractiveHost}.
+ */
+export declare function setInteractiveHost(interactive: boolean): boolean;
 /**
  * True when this code is running inside a `bun build --compile` standalone
  * binary. Detects via the embedded virtual-filesystem path markers

package/dist/types/runtime-install.d.ts

@@ -31,15 +31,25 @@ export interface RuntimeResolverOptions {
  * runtime caches. Stock resolution is tried first and kept for anything
  * outside the registered roots (bundled imports, node builtins, host or
  * extension trees). Multiple runtime roots may register; they are consulted
- * in registration order.
+ * in registration order. Returns an uninstaller that drops the registration
+ * and restores the stock resolver once no registrations remain.
  *
  * One stock "success" is distrusted: the compiled-binary resolver ignores
  * `main`/`exports` for real-FS packages (Bun #1763), so a package shipping
  * its TS source next to `dist/` (e.g. `@huggingface/hub`'s root `index.ts`)
  * resolves to the wrong file. When the stock hit lands inside a registered
  * runtime root, the manifest-aware resolution wins.
+ *
+ * KNOWN LIMITATION (Bun 1.3.14): while any JS override of
+ * `Module._resolveFilename` is installed, Bun routes `createRequire(...)`
+ * resolution through it with `parent === undefined` — the requester context is
+ * never passed, so relative requires from a `createRequire` require fail with
+ * "Cannot find module './x' from ''". The override cannot recover what it is
+ * never given. Keep this patch scoped to dedicated worker/runtime processes
+ * (tiny-inference, fastembed); never install it in the main agent process,
+ * where legacy-pi extensions rely on `createRequire` relative requires.
  */
-export declare function installRuntimeModuleResolver({ runtimeNodeModules, stubs }: RuntimeResolverOptions): void;
+export declare function installRuntimeModuleResolver({ runtimeNodeModules, stubs }: RuntimeResolverOptions): () => void;
 /** Pinned dependency set materialized into a runtime cache directory. */
 export interface RuntimeInstallSpec {
     dependencies: Record<string, string>;

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-utils",
-	"version": "17.0.4",
+	"version": "17.0.5",
 	"description": "Shared utilities for pi packages",
 	"homepage": "https://omp.sh",
 	"author": "Can Boluk",
@@ -31,7 +31,7 @@
 		"fmt": "biome format --write ."
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-natives": "17.0.4",
+		"@oh-my-pi/pi-natives": "17.0.5",
 		"handlebars": "^4.7.9",
 		"winston": "^3.19.0",
 		"winston-daily-rotate-file": "^5.0.0"

package/src/env.ts

@@ -207,6 +207,30 @@ export function setTerminalHeadless(headless: boolean): boolean {
 	return previous;
 }
 
+let interactiveHost = false;
+
+/**
+ * True when this process runs an interactive coding-agent host — the only
+ * context where the operator can browse the Agent Hub and focus a live
+ * subagent's session (`SessionFocusController`), so a subagent's session title
+ * can become operator-visible. Off by default (print/RPC/ACP/eval/SDK/`bun
+ * test` never render a focusable session tree); the interactive entrypoint
+ * flips it on with {@link setInteractiveHost}.
+ */
+export function isInteractiveHost(): boolean {
+	return interactiveHost;
+}
+
+/**
+ * Set the interactive-host flag and return the previous value so callers can
+ * restore exact prior state. See {@link isInteractiveHost}.
+ */
+export function setInteractiveHost(interactive: boolean): boolean {
+	const previous = interactiveHost;
+	interactiveHost = interactive;
+	return previous;
+}
+
 /**
  * True when this code is running inside a `bun build --compile` standalone
  * binary. Detects via the embedded virtual-filesystem path markers

package/src/runtime-install.ts

@@ -194,24 +194,41 @@ export interface RuntimeResolverOptions {
  * runtime caches. Stock resolution is tried first and kept for anything
  * outside the registered roots (bundled imports, node builtins, host or
  * extension trees). Multiple runtime roots may register; they are consulted
- * in registration order.
+ * in registration order. Returns an uninstaller that drops the registration
+ * and restores the stock resolver once no registrations remain.
  *
  * One stock "success" is distrusted: the compiled-binary resolver ignores
  * `main`/`exports` for real-FS packages (Bun #1763), so a package shipping
  * its TS source next to `dist/` (e.g. `@huggingface/hub`'s root `index.ts`)
  * resolves to the wrong file. When the stock hit lands inside a registered
  * runtime root, the manifest-aware resolution wins.
+ *
+ * KNOWN LIMITATION (Bun 1.3.14): while any JS override of
+ * `Module._resolveFilename` is installed, Bun routes `createRequire(...)`
+ * resolution through it with `parent === undefined` — the requester context is
+ * never passed, so relative requires from a `createRequire` require fail with
+ * "Cannot find module './x' from ''". The override cannot recover what it is
+ * never given. Keep this patch scoped to dedicated worker/runtime processes
+ * (tiny-inference, fastembed); never install it in the main agent process,
+ * where legacy-pi extensions rely on `createRequire` relative requires.
  */
-export function installRuntimeModuleResolver({ runtimeNodeModules, stubs = {} }: RuntimeResolverOptions): void {
+export function installRuntimeModuleResolver({ runtimeNodeModules, stubs = {} }: RuntimeResolverOptions): () => void {
 	const registry = resolverRegistry();
 	const existing = registry.find(entry => entry.runtimeNodeModules === runtimeNodeModules);
 	if (existing) Object.assign(existing.stubs, stubs);
 	else registry.push({ runtimeNodeModules, stubs: { ...stubs } });
 
 	const resolver = (Module as unknown as { default?: ModuleResolver } & ModuleResolver).default ?? Module;
-	const target = resolver as unknown as ModuleResolver & { [PATCHED]?: boolean };
-	if (target[PATCHED]) return;
-	const original = target._resolveFilename.bind(target);
+	const target = resolver as unknown as ModuleResolver & { [PATCHED]?: () => void };
+	const uninstall = (): void => {
+		const entries = resolverRegistry();
+		const index = entries.findIndex(entry => entry.runtimeNodeModules === runtimeNodeModules);
+		if (index !== -1) entries.splice(index, 1);
+		if (entries.length === 0) target[PATCHED]?.();
+	};
+	if (target[PATCHED]) return uninstall;
+	const pristine = target._resolveFilename;
+	const original = pristine.bind(target);
 	target._resolveFilename = (request: string, parent: unknown, isMain: boolean, options?: unknown): string => {
 		let stockResolved: string | null = null;
 		let stockError: unknown;
@@ -256,7 +273,11 @@ export function installRuntimeModuleResolver({ runtimeNodeModules, stubs = {} }:
 		if (stockResolved) return stockResolved;
 		throw stockError;
 	};
-	target[PATCHED] = true;
+	target[PATCHED] = () => {
+		target._resolveFilename = pristine;
+		delete target[PATCHED];
+	};
+	return uninstall;
 }
 
 /** Pinned dependency set materialized into a runtime cache directory. */