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

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 ## [Unreleased]
 
+## [18.4.10] - 2026-10-02
+
+### Fixed
+
+- `streamProxy` no longer finalizes a cut-off tool-call argument buffer into an executable auto-closed preview; such a call gets the parse-error arguments, so the tool is not run and the model receives the parse error ([#13868](https://github.com/can1357/oh-my-pi/pull/13868) by [@alphastorm](https://github.com/alphastorm))
+- OpenAI remote compaction no longer sends stored native tool calls whose names are blank, longer than 128 characters, or contain whitespace or control characters. The outputs that answer those calls are dropped as well ([#13985](https://github.com/can1357/oh-my-pi/pull/13985) by [@Xytronix](https://github.com/Xytronix)).
+
 ## [18.4.6] - 2026-10-01
 
 ### Added

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-agent-core",
-	"version": "18.4.8",
+	"version": "18.4.10",
 	"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.8",
-		"@oh-my-pi/pi-catalog": "18.4.8",
-		"@oh-my-pi/pi-natives": "18.4.8",
-		"@oh-my-pi/pi-utils": "18.4.8",
-		"@oh-my-pi/pi-wire": "18.4.8",
-		"@oh-my-pi/snapcompact": "18.4.8",
+		"@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",
 		"@opentelemetry/api": "^1.9.1"
 	},
 	"devDependencies": {
-		"@oh-my-pi/omptype": "18.4.8",
+		"@oh-my-pi/omptype": "18.4.10",
 		"@opentelemetry/context-async-hooks": "^2.9.0",
 		"@opentelemetry/sdk-trace-base": "^2.9.0",
 		"@types/bun": "^1.3.14"

package/src/compaction/openai.ts

@@ -36,6 +36,7 @@ import type {
 	ProviderSessionState,
 } from "@oh-my-pi/pi-ai/types";
 import {
+	dropMalformedOpenAIResponsesToolCalls,
 	getOpenAIResponsesHistoryItems,
 	getOpenAIResponsesHistoryPayload,
 	normalizeResponsesToolCallId,
@@ -537,7 +538,10 @@ export function buildOpenAiNativeHistory(
 						}
 					}
 				}
-				const historyItems = adaptComputerHistoryForCompaction(rawHistoryItems, model.supportsComputerUse === true);
+				const historyItems = adaptComputerHistoryForCompaction(
+					dropMalformedOpenAIResponsesToolCalls(rawHistoryItems),
+					model.supportsComputerUse === true,
+				);
 				input.push(...historyItems);
 				addOpenAiCallIds(historyItems, knownCallIds, customCallIds, computerCallIds);
 				msgIndex++;
@@ -589,7 +593,7 @@ export function buildOpenAiNativeHistory(
 					}
 				}
 				const historyItems = adaptComputerHistoryForCompaction(
-					providerPayload.items,
+					dropMalformedOpenAIResponsesToolCalls(providerPayload.items),
 					model.supportsComputerUse === true,
 				);
 				if (providerPayload.dt) {

package/src/proxy.ts

@@ -20,7 +20,8 @@ import {
 	type StreamingPartialJsonCarrier,
 	setStreamingPartialJson,
 } from "@oh-my-pi/pi-ai/utils/block-symbols";
-import { parseStreamingJson, parseStreamingJsonThrottled, readSseJson } from "@oh-my-pi/pi-utils";
+import { parseToolCallArguments } from "@oh-my-pi/pi-ai/utils/tool-call-arguments";
+import { parseStreamingJsonThrottled, readSseJson } from "@oh-my-pi/pi-utils";
 
 // Event stream adapter for proxy SSE events
 export class ProxyMessageEventStream extends EventStream<AssistantMessageEvent, AssistantMessage> {
@@ -378,7 +379,7 @@ function processProxyEvent(
 				// throttle may have skipped trailing deltas, so the last
 				// parsed arguments can lag the accumulated buffer.
 				const acc = partialJsonByIndex.get(proxyEvent.contentIndex);
-				if (acc !== undefined && acc.length > 0) content.arguments = parseStreamingJson(acc) || {};
+				if (acc !== undefined && acc.length > 0) content.arguments = parseToolCallArguments(acc) || {};
 				partialJsonByIndex.delete(proxyEvent.contentIndex);
 				parsedLenByIndex.delete(proxyEvent.contentIndex);
 				clearStreamingPartialJson(content);
@@ -427,6 +428,6 @@ function finalizeBufferedArguments(partial: AssistantMessage, partialJsonByIndex
 	for (const [index, acc] of partialJsonByIndex) {
 		if (acc.length === 0) continue;
 		const content = partial.content[index];
-		if (content?.type === "toolCall") content.arguments = parseStreamingJson(acc) || {};
+		if (content?.type === "toolCall") content.arguments = parseToolCallArguments(acc) || {};
 	}
 }