@oh-my-pi/pi-agent-core 18.4.10 → 18.4.12

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## [Unreleased]
 
+## [18.4.11] - 2026-10-02
+
+### Fixed
+
+- Fixed lenient argument validation for tools such as `yield`: malformed tool-call JSON is now reported to the model instead of causing the tool to run with empty arguments.
+
 ## [18.4.10] - 2026-10-02
 
 ### Fixed

package/dist/types/types.d.ts

@@ -1007,7 +1007,7 @@ export interface AgentTool<TParameters extends TSchema = TSchema, TDetails = any
      * before ordinary dispatch commits its result.
      */
     speculation?: ToolSpeculationPolicy;
-    /** If true, argument validation errors are non-fatal: raw args are passed to execute() instead of returning an error to the LLM. */
+    /** If true, schema validation errors are non-fatal: raw args are passed to execute() instead of returning an error to the LLM. Malformed argument JSON still returns the parse error. */
     lenientArgValidation?: boolean;
     /**
      * Whether the agent loop may abort this tool mid-execution — or skip it

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-agent-core",
-	"version": "18.4.10",
+	"version": "18.4.12",
 	"description": "General-purpose agent with transport abstraction, state management, and attachment support",
 	"homepage": "https://omp.sh",
 	"author": {
@@ -38,16 +38,16 @@
 		"fmt": "oxfmt --no-error-on-unmatched-pattern 'src/**/*.{ts,tsx}' '{test,bench,examples,scripts}/**/*.ts' '*.ts'"
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-ai": "18.4.10",
-		"@oh-my-pi/pi-catalog": "18.4.10",
-		"@oh-my-pi/pi-natives": "18.4.10",
-		"@oh-my-pi/pi-utils": "18.4.10",
-		"@oh-my-pi/pi-wire": "18.4.10",
-		"@oh-my-pi/snapcompact": "18.4.10",
+		"@oh-my-pi/pi-ai": "18.4.12",
+		"@oh-my-pi/pi-catalog": "18.4.12",
+		"@oh-my-pi/pi-natives": "18.4.12",
+		"@oh-my-pi/pi-utils": "18.4.12",
+		"@oh-my-pi/pi-wire": "18.4.12",
+		"@oh-my-pi/snapcompact": "18.4.12",
 		"@opentelemetry/api": "^1.9.1"
 	},
 	"devDependencies": {
-		"@oh-my-pi/omptype": "18.4.10",
+		"@oh-my-pi/omptype": "18.4.12",
 		"@opentelemetry/context-async-hooks": "^2.9.0",
 		"@opentelemetry/sdk-trace-base": "^2.9.0",
 		"@types/bun": "^1.3.14"

package/src/agent-loop.ts

@@ -2907,13 +2907,14 @@ async function prepareToolCallDispatch(
 				}
 				return validateToolArguments(tool, { ...toolCall, arguments: args });
 			} catch (validationError) {
-				if (tool?.lenientArgValidation) {
+				// Lenience covers schema mismatches; a parse failure has no args to hand over.
+				const parseFailed = "__parseError" in args;
+				if (tool?.lenientArgValidation && !parseFailed) {
 					const fallback = { ...args };
-					delete fallback.__parseError;
 					delete fallback.__rawJson;
 					return fallback;
 				}
-				entry.args = "__parseError" in args ? { __parseError: args.__parseError } : args;
+				entry.args = parseFailed ? { __parseError: args.__parseError } : args;
 				entry.validationErrorMessage =
 					validationError instanceof Error ? validationError.message : String(validationError);
 				return undefined;

package/src/speculative-execution.ts

@@ -714,12 +714,12 @@ export class SpeculativeOperationCoordinator {
 		try {
 			validatedArgs = validateToolArguments(tool, toolCall);
 		} catch {
-			if (!tool.lenientArgValidation) {
+			// Lenience covers schema mismatches; a parse failure has no args to hand over.
+			if (!tool.lenientArgValidation || "__parseError" in toolCall.arguments) {
 				this.ineligible(toolCall, "tool arguments are not valid", source, parentToolCallId);
 				return undefined;
 			}
-			validatedArgs = { ...(toolCall.arguments as Record<string, unknown>) };
-			delete validatedArgs.__parseError;
+			validatedArgs = { ...toolCall.arguments };
 			delete validatedArgs.__rawJson;
 		}
 		let executionArgs: Record<string, unknown>;

package/src/types.ts

@@ -1143,7 +1143,7 @@ export interface AgentTool<
 	 * before ordinary dispatch commits its result.
 	 */
 	speculation?: ToolSpeculationPolicy;
-	/** If true, argument validation errors are non-fatal: raw args are passed to execute() instead of returning an error to the LLM. */
+	/** If true, schema validation errors are non-fatal: raw args are passed to execute() instead of returning an error to the LLM. Malformed argument JSON still returns the parse error. */
 	lenientArgValidation?: boolean;
 	/**
 	 * Whether the agent loop may abort this tool mid-execution — or skip it