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

package/CHANGELOG.md

@@ -2,6 +2,22 @@
 
 ## [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
+
+- The soft steering signal a tool sees as `toolCall.steeringSignal` now aborts with the source of the queued message (`"user"`, `"irc"`, …) as its `reason`, so a tool can yield only to the sources it should ([#17537](https://github.com/can1357/oh-my-pi/issues/17537))
+
+### Fixed
+
+- Reject incomplete local compaction, turn-prefix, handoff, and branch summaries ending with `stopReason: "length"` instead of using truncated text as complete context. Omit incomplete display-only short summaries without discarding a completed full summary ([#17499](https://github.com/can1357/oh-my-pi/pull/17499) by [@Xytronix](https://github.com/Xytronix)).
+
 ## [18.14.1] - 2026-10-31
 
 ### Breaking Changes

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/compaction/utils.d.ts

@@ -10,6 +10,8 @@ export interface FileOperations {
     edited: Set<string>;
 }
 export declare function createFileOps(): FileOperations;
+/** Error text for a summarization call that hit its output limit (`stopReason: "length"`): partial text is never used. */
+export declare function incompleteOutputError(prefix: string): string;
 /**
  * Split a read-tool path into its base path and trailing selector, mirroring the
  * read tool's own splitter. Single source of the grammar in this package: the

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

@@ -588,7 +588,8 @@ export interface ToolCallContext {
     providerMetadata?: ToolCallProviderMetadata;
     /**
      * Cooperative steering signal: aborted when a queued user/steering message
-     * (or an interrupting peer IRC) is detected while this tool batch runs.
+     * (or an interrupting peer IRC) is detected while this tool batch runs. Its
+     * `reason` names the source: a {@link SteeringInterruptSource} or `"irc"`.
      * Unlike the hard abort signal it NEVER kills the tool — long-running
      * tools MAY observe it (via `ctx.toolCall.steeringSignal`) to finish early
      * or background themselves so the message injects promptly; ignoring it is
@@ -967,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.1",
+	"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.1",
-		"@oh-my-pi/pi-catalog": "18.14.1",
-		"@oh-my-pi/pi-natives": "18.14.1",
-		"@oh-my-pi/pi-utils": "18.14.1",
-		"@oh-my-pi/pi-wire": "18.14.1",
-		"@oh-my-pi/snapcompact": "18.14.1",
+		"@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.1",
+		"@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-loop.ts

@@ -3402,7 +3402,7 @@ async function executeToolCalls(
 		// also cascades: the freshly detached job's own completion re-triggers
 		// this check for the next command, so millisecond-long commands chain
 		// into separate background deliveries (#12869).
-		if (source === "irc" && softInterrupts) steeringSoftController.abort();
+		if (source === "irc" && softInterrupts) steeringSoftController.abort("irc");
 	};
 
 	const checkSteering = async (): Promise<void> => {
@@ -3441,7 +3441,7 @@ async function executeToolCalls(
 				interruptState.triggered = true;
 				interruptState.source = steeringSource ?? "unknown";
 				steeringAbortController.abort(TOOL_INTERRUPT_ABORT_REASON);
-				if (softInterrupts) steeringSoftController.abort();
+				if (softInterrupts) steeringSoftController.abort(interruptState.source);
 			}
 			return;
 		}

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/branch-summarization.ts

@@ -28,6 +28,7 @@ import {
 	createFileOps,
 	extractFileOpsFromMessage,
 	type FileOperations,
+	incompleteOutputError,
 	SUMMARIZATION_SYSTEM_PROMPT,
 	serializeConversationForSummary,
 	stripReadSelector,
@@ -386,6 +387,9 @@ export async function generateBranchSummary(
 	if (response.stopReason === "error") {
 		return { error: response.errorMessage || "Summarization failed" };
 	}
+	if (response.stopReason === "length") {
+		return { error: incompleteOutputError("Summarization failed") };
+	}
 
 	let summary = response.content
 		.filter((c): c is { type: "text"; text: string } => c.type === "text")

package/src/compaction/compaction.ts

@@ -91,6 +91,7 @@ import {
 	escapeSummaryBoundaryTags,
 	extractFileOpsFromMessage,
 	type FileOperations,
+	incompleteOutputError,
 	SUMMARIZATION_SYSTEM_PROMPT,
 	serializeConversationForSummary,
 	stripReadSelector,
@@ -1075,6 +1076,9 @@ async function summarizeConversationWindow(
 	if (response.stopReason === "error") {
 		throw createSummarizationError("Summarization failed", response);
 	}
+	if (response.stopReason === "length") {
+		throw new Error(incompleteOutputError("Summarization failed"));
+	}
 
 	const textContent = response.content
 		.filter((c): c is { type: "text"; text: string } => c.type === "text")
@@ -1182,6 +1186,9 @@ export async function generateHandoffFromContext(
 	if (response.stopReason === "error") {
 		throw createSummarizationError("Handoff generation failed", response);
 	}
+	if (response.stopReason === "length") {
+		throw new Error(incompleteOutputError("Handoff generation failed"));
+	}
 
 	return response.content
 		.filter((c): c is { type: "text"; text: string } => c.type === "text")
@@ -1305,8 +1312,10 @@ async function generateShortSummary(
 				case "error":
 					outcome = { kind: "failed", error: response.errorMessage || "Unknown error" };
 					break;
-				case "stop":
 				case "length":
+					outcome = { kind: "failed", error: incompleteOutputError("Short summary failed") };
+					break;
+				case "stop":
 				case "toolUse":
 					outcome = {
 						kind: "text",
@@ -2435,6 +2444,9 @@ async function generateTurnPrefixSummary(
 	if (response.stopReason === "error") {
 		throw createSummarizationError("Turn prefix summarization failed", response);
 	}
+	if (response.stopReason === "length") {
+		throw new Error(incompleteOutputError("Turn prefix summarization failed"));
+	}
 
 	return response.content
 		.filter((c): c is { type: "text"; text: string } => c.type === "text")

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/compaction/utils.ts

@@ -28,6 +28,11 @@ export function createFileOps(): FileOperations {
 	};
 }
 
+/** Error text for a summarization call that hit its output limit (`stopReason: "length"`): partial text is never used. */
+export function incompleteOutputError(prefix: string): string {
+	return `${prefix}: model output ended incomplete (stopReason: length)`;
+}
+
 // Read-tool selector grammar, mirrored from the conservative filesystem splitter in
 // packages/coding-agent/src/tools/path-utils.ts (splitPathAndSel). Keep in sync.
 // A trailing `:chunk` is a selector only when it is a line-range list

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

@@ -672,7 +672,8 @@ export interface ToolCallContext {
 	providerMetadata?: ToolCallProviderMetadata;
 	/**
 	 * Cooperative steering signal: aborted when a queued user/steering message
-	 * (or an interrupting peer IRC) is detected while this tool batch runs.
+	 * (or an interrupting peer IRC) is detected while this tool batch runs. Its
+	 * `reason` names the source: a {@link SteeringInterruptSource} or `"irc"`.
 	 * Unlike the hard abort signal it NEVER kills the tool — long-running
 	 * tools MAY observe it (via `ctx.toolCall.steeringSignal`) to finish early
 	 * or background themselves so the message injects promptly; ignoring it is
@@ -1086,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.