@oh-my-pi/pi-agent-core 18.14.2 → 18.15.0

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## [Unreleased]
 
+## [18.15.0] - 2026-11-01
+
+### Fixed
+
+- Fixed OpenAI remote compaction (`/responses/compact`) replaying backend-only `internal_chat_message_metadata_passthrough` metadata that Codex rejects as an unknown parameter ([#17590](https://github.com/can1357/oh-my-pi/pull/17590) by [@briankeefe](https://github.com/briankeefe))
+
 ## [18.14.2] - 2026-10-31
 
 ### Added

package/dist/types/agent.d.ts

@@ -2,6 +2,7 @@ import { type ApiKey, type AssistantMessage, type AssistantMessageEvent, type Co
 import type { Dialect } from "@oh-my-pi/pi-ai/dialect";
 import type { HarmonyAuditEvent } from "@oh-my-pi/pi-ai/utils/harmony-leak";
 import type { AppendOnlyContextManager } from "./append-only-context.js";
+import { SentToolDefinitions } from "./sent-tool-definitions.js";
 import { Tokenizer } from "./tokenizer.js";
 import type { AgentBeforeModelCall, AgentContext, AgentEvent, AgentLoopConfig, AgentMessage, AgentState, AgentTool, AgentToolContext, AgentTurnEndContext, AsideBoundary, AsideMessage, PrepareQueuedMessages, SpeculativeToolExecutionConfig, StreamFn, ToolCallContext, ToolChoiceDirective } from "./types.js";
 export declare class AgentBusyError extends Error {
@@ -64,6 +65,11 @@ export interface AgentOptions {
      * Shared provider state map for session-scoped transport/session caches.
      */
     providerSessionState?: Map<string, ProviderSessionState>;
+    /**
+     * Tool definitions already sent, for an Agent that replays another Agent's request history
+     * (see {@link Agent.copySentToolDefinitions}). Defaults to an empty store.
+     */
+    sentToolDefinitions?: SentToolDefinitions;
     /**
      * Resolves an API key or resolver dynamically for each LLM call.
      * Useful for expiring tokens and model-scoped credential routing.
@@ -295,6 +301,11 @@ export declare class Agent {
      * Set the prompt cache key forwarded to providers.
      */
     set promptCacheKey(value: string | undefined);
+    /**
+     * A detached copy of the tool definitions this agent has sent (`AgentOptions.sentToolDefinitions`), without the
+     * names `keep` rejects.
+     */
+    copySentToolDefinitions(keep?: (name: string) => boolean): SentToolDefinitions;
     /**
      * Fields added to every log record written while this agent runs (the
      * owning session's `sessionId`), so in-process agents' records stay apart.

package/dist/types/sent-tool-definitions.d.ts

@@ -8,6 +8,11 @@ export declare class SentToolDefinitions {
     #private;
     /** Remember the definitions a request is about to send. */
     record(tools: readonly Tool[]): void;
+    /**
+     * A detached copy: an Agent that replays another's request history declares the same withdrawn tools. `keep`
+     * leaves out a definition the copy must never declare.
+     */
+    copy(keep?: (name: string) => boolean): SentToolDefinitions;
     /**
      * `context` with {@link Context.inactiveTools} set to the definitions for names the latest
      * `requestControls.tools.declared` in its messages holds that are not in its `tools`; cleared when

package/dist/types/types.d.ts

@@ -968,7 +968,7 @@ export type ToolApprovalDecision = ToolTier | {
      *  prompt policies to the tool it dispatches into. */
     policyKey?: string;
 };
-export type ToolApproval = ToolApprovalDecision | ((args: unknown) => ToolApprovalDecision);
+export type ToolApproval = ToolApprovalDecision | ((args: unknown, context?: AgentToolContext) => ToolApprovalDecision);
 /**
  * Context passed to tool execution.
  * Apps can extend via declaration merging.

package/package.json

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

package/src/agent.ts

@@ -176,6 +176,11 @@ export interface AgentOptions {
 	 * Shared provider state map for session-scoped transport/session caches.
 	 */
 	providerSessionState?: Map<string, ProviderSessionState>;
+	/**
+	 * Tool definitions already sent, for an Agent that replays another Agent's request history
+	 * (see {@link Agent.copySentToolDefinitions}). Defaults to an empty store.
+	 */
+	sentToolDefinitions?: SentToolDefinitions;
 
 	/**
 	 * Resolves an API key or resolver dynamically for each LLM call.
@@ -568,6 +573,7 @@ export class Agent {
 		this.#deadline = opts.deadline;
 		this.#promptCacheKey = opts.promptCacheKey;
 		this.#providerSessionState = opts.providerSessionState;
+		if (opts.sentToolDefinitions) this.#sentToolDefinitions = opts.sentToolDefinitions;
 		this.#thinkingBudgets = opts.thinkingBudgets;
 		this.#temperature = opts.temperature;
 		this.#topP = opts.topP;
@@ -642,6 +648,14 @@ export class Agent {
 		this.#promptCacheKey = value;
 	}
 
+	/**
+	 * A detached copy of the tool definitions this agent has sent (`AgentOptions.sentToolDefinitions`), without the
+	 * names `keep` rejects.
+	 */
+	copySentToolDefinitions(keep?: (name: string) => boolean): SentToolDefinitions {
+		return this.#sentToolDefinitions.copy(keep);
+	}
+
 	/**
 	 * Fields added to every log record written while this agent runs (the
 	 * owning session's `sessionId`), so in-process agents' records stay apart.

package/src/compaction/openai.ts

@@ -45,6 +45,7 @@ import {
 	getOpenAIResponsesHistoryItems,
 	getOpenAIResponsesHistoryPayload,
 	normalizeResponsesToolCallId,
+	stripOpenAIResponsesInternalMetadataForReplay,
 	stripOpenAIResponsesOutputOnlyStatusesForReplay,
 	truncateResponseItemId,
 } from "@oh-my-pi/pi-ai/utils";
@@ -846,6 +847,9 @@ export function buildOpenAiNativeHistory(
 		msgIndex++;
 	}
 
+	for (let index = 0; index < input.length; index++) {
+		input[index] = stripOpenAIResponsesInternalMetadataForReplay(input[index]!);
+	}
 	return stripOpenAIResponsesOutputOnlyStatusesForReplay(hoistInterleavedResponsesToolBatchMessages(input));
 }
 

package/src/sent-tool-definitions.ts

@@ -13,6 +13,16 @@ export class SentToolDefinitions {
 		for (const tool of tools) this.#byName.set(tool.name, tool);
 	}
 
+	/**
+	 * A detached copy: an Agent that replays another's request history declares the same withdrawn tools. `keep`
+	 * leaves out a definition the copy must never declare.
+	 */
+	copy(keep: (name: string) => boolean = () => true): SentToolDefinitions {
+		const copy = new SentToolDefinitions();
+		copy.record([...this.#byName.values()].filter(tool => keep(tool.name)));
+		return copy;
+	}
+
 	/**
 	 * `context` with {@link Context.inactiveTools} set to the definitions for names the latest
 	 * `requestControls.tools.declared` in its messages holds that are not in its `tools`; cleared when

package/src/types.ts

@@ -1087,7 +1087,7 @@ export type ToolApprovalDecision =
 			 *  prompt policies to the tool it dispatches into. */
 			policyKey?: string;
 	  };
-export type ToolApproval = ToolApprovalDecision | ((args: unknown) => ToolApprovalDecision);
+export type ToolApproval = ToolApprovalDecision | ((args: unknown, context?: AgentToolContext) => ToolApprovalDecision);
 
 /**
  * Context passed to tool execution.