@oh-my-pi/pi-coding-agent 16.0.4 → 16.0.6

package/src/extensibility/plugins/legacy-pi-compat.ts

@@ -64,14 +64,14 @@ const packageRootCache = new Map<string, string | null>();
 const packageImportsCache = new Map<string, Record<string, unknown> | null>();
 const PACKAGE_IMPORT_EXCLUDED = Symbol("packageImportExcluded");
 
-// Extensions that imported `@sinclair/typebox` directly used to resolve against a
-// real `@sinclair/typebox` install. The runtime dep was replaced with the Zod-backed
-// shim under `extensibility/typebox.ts`; plugins still importing the public name
-// are redirected to that shim so existing extensions keep working without code
-// changes. Submodules like `@sinclair/typebox/compiler` are intentionally not
-// remapped — those expose TypeBox-only APIs the shim does not provide and plugins
-// relying on them must vendor `@sinclair/typebox` directly.
-const TYPEBOX_SPECIFIER_FILTER = /^@sinclair\/typebox$/;
+// Extensions that imported TypeBox directly used to resolve against a real
+// `@sinclair/typebox` or `typebox` install. The runtime dep was replaced with
+// the Zod-backed shim under `extensibility/typebox.ts`; plugins still importing
+// either public name are redirected to that shim so existing extensions keep
+// working without code changes. Submodules like `@sinclair/typebox/compiler`
+// are intentionally not remapped — those expose TypeBox-only APIs the shim does
+// not provide and plugins relying on them must vendor TypeBox directly.
+const TYPEBOX_SPECIFIER_FILTER = /^(?:@sinclair\/typebox|typebox)$/;
 
 // Compat shim and bundled-package paths used in compiled-binary mode. The shim
 // paths must point at files that ship inside the bunfs root; in dev /
@@ -295,14 +295,14 @@ function rewriteLegacyPiImports(source: string): string {
 	);
 }
 
-// Match the bare `@sinclair/typebox` import specifier (static + dynamic).
-// Subpath imports like `@sinclair/typebox/compiler` are intentionally excluded —
-// they expose TypeBox-only APIs the Zod-backed shim does not provide.
-const TYPEBOX_IMPORT_SPECIFIER_REGEX = /((?:from\s+|import\s+|import\s*\(\s*)["'])(@sinclair\/typebox)(["'])/g;
+// Match the bare TypeBox import specifiers (static + dynamic). Subpath imports
+// like `@sinclair/typebox/compiler` are intentionally excluded — they expose
+// TypeBox-only APIs the Zod-backed shim does not provide.
+const TYPEBOX_IMPORT_SPECIFIER_REGEX = /((?:from\s+|import\s+|import\s*\(\s*)["'])(@sinclair\/typebox|typebox)(["'])/g;
 
 /**
  * Rewrite the extension-owned specifiers OMP must host-resolve — legacy
- * `@(scope)/pi-*`, bare `@sinclair/typebox`, and package `imports` aliases like
+ * `@(scope)/pi-*`, bare TypeBox packages, and package `imports` aliases like
  * `#src/*` — to absolute `file://` URLs. Every other specifier (relative
  * siblings and third-party dependencies) is left untouched so Bun resolves it
  * natively from the extension's real on-disk location.

package/src/extensibility/shared-events.ts

@@ -93,6 +93,17 @@ export interface SessionShutdownEvent {
 	type: "session_shutdown";
 }
 
+/** Fired when a main-agent turn is about to settle; handlers may request one continuation turn. */
+export interface SessionStopEvent {
+	type: "session_stop";
+	messages: AgentMessage[];
+	turn_id: number;
+	last_assistant_message?: AgentMessage;
+	session_id: string;
+	session_file?: string;
+	stop_hook_active: boolean;
+}
+
 /** Preparation data for tree navigation (used by session_before_tree event) */
 export interface TreePreparation {
 	/** Node being switched to */
@@ -145,6 +156,7 @@ export type SessionEvent =
 	| SessionBeforeCompactEvent
 	| SessionCompactingEvent
 	| SessionCompactEvent
+	| SessionStopEvent
 	| SessionShutdownEvent
 	| SessionBeforeTreeEvent
 	| SessionTreeEvent
@@ -328,6 +340,18 @@ export interface SessionCompactingResult {
 	preserveData?: Record<string, unknown>;
 }
 
+/** Return type for `session_stop` handlers */
+export interface SessionStopEventResult {
+	/** Continue the main session with additional context before settling */
+	continue?: boolean;
+	/** OMP-native model-visible context for the continuation */
+	additionalContext?: string;
+	/** Claude/Codex-compatible block decision; maps to a continuation */
+	decision?: "block";
+	/** Claude/Codex-compatible model-visible continuation reason */
+	reason?: string;
+}
+
 /** Return type for `session_before_tree` handlers */
 export interface SessionBeforeTreeResult {
 	/** If true, cancel the navigation entirely */

package/src/extensibility/tool-proxy.ts

@@ -14,7 +14,10 @@ export function applyToolProxy<TTool extends object>(tool: TTool, wrapper: objec
 			Object.defineProperty(wrapper, key, {
 				get() {
 					const value = (tool as Record<PropertyKey, unknown>)[key];
-					return typeof value === "function" ? value.bind(tool) : value;
+					// Bind real methods so `this` is preserved through the wrapper, but leave
+					// callable values that aren't plain functions untouched — notably an ArkType
+					// `Type` (the `parameters` schema) is callable yet lacks `Function.prototype.bind`.
+					return typeof value === "function" && typeof value.bind === "function" ? value.bind(tool) : value;
 				},
 				enumerable: true,
 				configurable: true,