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

package/CHANGELOG.md

@@ -2,6 +2,26 @@
 
 ## [Unreleased]
 
+## [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
+
+- `SentToolDefinitions.inactiveFor(messages, tools)` is replaced by `withInactiveTools(context)`, which also clears a stale `inactiveTools`; `transformProviderContext` now receives the request's `inactiveTools` ([#17492](https://github.com/can1357/oh-my-pi/pull/17492))
+
+### Fixed
+
+- Long tool-call intents no longer block tool execution when intent tracing is enabled ([#13140](https://github.com/can1357/oh-my-pi/issues/13140)).
+
 ## [18.14.0] - 2026-10-29
 
 ### Breaking Changes

package/dist/types/compaction/compaction-v2-streaming.d.ts

@@ -85,6 +85,7 @@ export declare function requestCompactionV2Streaming(model: Model, apiKey: strin
     providerSessionState?: Map<string, ProviderSessionState>;
     codexCompaction?: CodexCompactionContext;
     preferWebsockets?: boolean;
+    codexAccessPrograms?: boolean;
 }): Promise<CompactionV2Response>;
 /** Build Codex-style V2 replacement history from retention candidates plus compaction output. */
 export declare function buildCompactionV2ReplacementHistory(input: unknown[], compactionItem: Record<string, unknown>, retainedMessageBudget?: number): {

package/dist/types/compaction/compaction.d.ts

@@ -256,6 +256,8 @@ export interface SummaryOptions {
     providerSessionState?: Map<string, ProviderSessionState>;
     /** Whether Codex remote compaction should prefer the provider WebSocket transport. */
     preferWebsockets?: boolean;
+    /** Daybreak Blue request on Codex remote compaction; see `StreamOptions.codexAccessPrograms`. */
+    codexAccessPrograms?: boolean;
     /** Classification shared by every provider request in this logical compaction. */
     codexCompaction?: CodexCompactionContext;
     /** Provider-visible tools for remote compaction transports that replay native tool history. */

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

@@ -1,4 +1,4 @@
-import type { Message, Tool } from "@oh-my-pi/pi-ai";
+import type { Context, Tool } from "@oh-my-pi/pi-ai";
 /**
  * Last wire definition this Agent sent for each tool name, so a provider that keeps
  * withdrawn tools declared (Anthropic `tool_removal`) can re-declare them byte-identically.
@@ -9,9 +9,10 @@ export declare class SentToolDefinitions {
     /** Remember the definitions a request is about to send. */
     record(tools: readonly Tool[]): void;
     /**
-     * Definitions for names the latest `requestControls.tools.declared` in `messages` holds
-     * that are not in `active`; undefined when none. Names never sent by this Agent are
-     * skipped: the provider drops them from the declaration.
+     * `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
+     * there are none or the request declares no tools. Names never sent by this Agent are skipped: the
+     * provider drops them from the declaration.
      */
-    inactiveFor(messages: readonly Message[], active: readonly Tool[]): Tool[] | undefined;
+    withInactiveTools(context: Context): Context;
 }

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

package/package.json

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

package/src/agent-loop.ts

@@ -1071,12 +1071,6 @@ function resolveIntentMode(intent: AgentTool["intent"]): "require" | "optional"
 	return "require";
 }
 
-/**
- * Longest `i` value accepted as an intent; anything past this is a tool
- * payload the model put in the wrong field, not an intent.
- */
-const MAX_INTENT_LENGTH = 200;
-
 function extractIntent(args: Record<string, unknown>): { intent?: string; strippedArgs: Record<string, unknown> } {
 	const { [INTENT_FIELD]: intent, ...strippedArgs } = args;
 	if (typeof intent !== "string") {
@@ -1976,6 +1970,9 @@ async function prepareProviderCall(
 			}),
 		};
 	}
+	// Before the transform too, so a request-size clamp in it measures the withdrawn definitions the provider
+	// re-declares; recomputed below against what the transform returns.
+	if (config.sentToolDefinitions) llmContext = config.sentToolDefinitions.withInactiveTools(llmContext);
 	if (config.transformProviderContext) {
 		llmContext = await config.transformProviderContext(llmContext, model);
 	}
@@ -1991,10 +1988,9 @@ async function prepareProviderCall(
 		};
 	}
 	// After `transformProviderContext`, so the recorded definitions are exactly what the provider receives.
-	if (config.sentToolDefinitions && llmContext.tools) {
-		config.sentToolDefinitions.record(llmContext.tools);
-		const inactiveTools = config.sentToolDefinitions.inactiveFor(llmContext.messages, llmContext.tools);
-		if (inactiveTools) llmContext = { ...llmContext, inactiveTools };
+	if (config.sentToolDefinitions) {
+		if (llmContext.tools) config.sentToolDefinitions.record(llmContext.tools);
+		llmContext = config.sentToolDefinitions.withInactiveTools(llmContext);
 	}
 	return { model, context: llmContext, promptToolWireTools, ownedDialect };
 }
@@ -3105,11 +3101,6 @@ async function prepareToolCallDispatch(
 				? { intent: undefined, strippedArgs: argsForExecution }
 				: extractIntent(toolCall.arguments);
 			argsForExecution = strippedArgs;
-			if (intent !== undefined && intent.length > MAX_INTENT_LENGTH && tool) {
-				entry.args = strippedArgs;
-				entry.validationErrorMessage = `\`${INTENT_FIELD}\` is a short intent label (at most ${MAX_INTENT_LENGTH} chars); the value you sent is ${intent.length} chars. The tool was not run. Put that content in the tool's own parameters and retry with a brief \`${INTENT_FIELD}\`.`;
-				continue;
-			}
 			if (intent) {
 				toolCall.intent = intent;
 			} else if (typeof tool?.intent === "function") {
@@ -3411,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> => {
@@ -3450,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

@@ -959,12 +959,11 @@ export class Agent {
 					pruneDescriptions: this.#pruneToolDescriptions,
 				}) ?? []);
 		let context: Context = { systemPrompt, messages, tools };
+		// Side requests reuse the main loop's sent definitions without recording their own; attached before the
+		// transform too, so a request-size clamp in it measures them.
+		context = this.#sentToolDefinitions.withInactiveTools(context);
 		if (this.#transformProviderContext) context = await this.#transformProviderContext(context, model);
-		// Side requests reuse the main loop's sent definitions without recording their own.
-		if (context.tools?.length) {
-			const inactiveTools = this.#sentToolDefinitions.inactiveFor(context.messages, context.tools);
-			if (inactiveTools) context = { ...context, inactiveTools };
-		}
+		context = this.#sentToolDefinitions.withInactiveTools(context);
 		return context;
 	}
 

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-v2-streaming.ts

@@ -274,6 +274,7 @@ export async function requestCompactionV2Streaming(
 		providerSessionState?: Map<string, ProviderSessionState>;
 		codexCompaction?: CodexCompactionContext;
 		preferWebsockets?: boolean;
+		codexAccessPrograms?: boolean;
 	},
 ): Promise<CompactionV2Response> {
 	let fetchImpl: FetchImpl = options?.fetch ?? globalThis.fetch;
@@ -316,6 +317,7 @@ export async function requestCompactionV2Streaming(
 				providerSessionState: options?.providerSessionState,
 				codexCompaction: options?.codexCompaction,
 				preferWebsockets: options?.preferWebsockets,
+				codexAccessPrograms: options?.codexAccessPrograms,
 			});
 		} catch (err) {
 			const error = err instanceof Error ? err : new Error(String(err));
@@ -352,6 +354,7 @@ async function attemptCompactionV2Streaming(
 		providerSessionState?: Map<string, ProviderSessionState>;
 		codexCompaction?: CodexCompactionContext;
 		preferWebsockets?: boolean;
+		codexAccessPrograms?: boolean;
 	},
 ): Promise<CompactionV2Response> {
 	// Faithful to Codex: append the compaction trigger as the final input item
@@ -377,6 +380,7 @@ async function attemptCompactionV2Streaming(
 			sessionId: request.sessionId,
 			providerSessionState: options.providerSessionState,
 			preferWebsockets: options.preferWebsockets,
+			codexAccessPrograms: options.codexAccessPrograms,
 			responsesLite: model.useResponsesLite,
 			codexCompaction: createOpenAICodexCompactionRequestContext({
 				context: options.codexCompaction,

package/src/compaction/compaction.ts

@@ -91,6 +91,7 @@ import {
 	escapeSummaryBoundaryTags,
 	extractFileOpsFromMessage,
 	type FileOperations,
+	incompleteOutputError,
 	SUMMARIZATION_SYSTEM_PROMPT,
 	serializeConversationForSummary,
 	stripReadSelector,
@@ -750,6 +751,8 @@ export interface SummaryOptions {
 	providerSessionState?: Map<string, ProviderSessionState>;
 	/** Whether Codex remote compaction should prefer the provider WebSocket transport. */
 	preferWebsockets?: boolean;
+	/** Daybreak Blue request on Codex remote compaction; see `StreamOptions.codexAccessPrograms`. */
+	codexAccessPrograms?: boolean;
 	/** Classification shared by every provider request in this logical compaction. */
 	codexCompaction?: CodexCompactionContext;
 	/** Provider-visible tools for remote compaction transports that replay native tool history. */
@@ -1073,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")
@@ -1180,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")
@@ -1303,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",
@@ -1765,6 +1776,7 @@ export async function compact(
 				: options?.promptCacheKey,
 		providerSessionState: options?.providerSessionState,
 		preferWebsockets: options?.preferWebsockets,
+		codexAccessPrograms: options?.codexAccessPrograms,
 		codexCompaction: options?.codexCompaction,
 		tools: options?.tools,
 		fetch: options?.fetch,
@@ -2000,6 +2012,7 @@ export async function compact(
 						fetch: summaryOptions.fetch,
 						providerSessionState: summaryOptions.providerSessionState,
 						preferWebsockets: summaryOptions.preferWebsockets,
+						codexAccessPrograms: summaryOptions.codexAccessPrograms,
 						codexCompaction: summaryOptions.codexCompaction,
 					}),
 				{ signal },
@@ -2431,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/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

@@ -1,4 +1,4 @@
-import type { Message, Tool } from "@oh-my-pi/pi-ai";
+import type { Context, Message, Tool } from "@oh-my-pi/pi-ai";
 
 /**
  * Last wire definition this Agent sent for each tool name, so a provider that keeps
@@ -14,11 +14,18 @@ export class SentToolDefinitions {
 	}
 
 	/**
-	 * Definitions for names the latest `requestControls.tools.declared` in `messages` holds
-	 * that are not in `active`; undefined when none. Names never sent by this Agent are
-	 * skipped: the provider drops them from the declaration.
+	 * `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
+	 * there are none or the request declares no tools. Names never sent by this Agent are skipped: the
+	 * provider drops them from the declaration.
 	 */
-	inactiveFor(messages: readonly Message[], active: readonly Tool[]): Tool[] | undefined {
+	withInactiveTools(context: Context): Context {
+		const inactiveTools = context.tools?.length ? this.#inactiveFor(context.messages, context.tools) : undefined;
+		if (!inactiveTools && !context.inactiveTools) return context;
+		return { ...context, inactiveTools };
+	}
+
+	#inactiveFor(messages: readonly Message[], active: readonly Tool[]): Tool[] | undefined {
 		let declared: readonly string[] | undefined;
 		for (let index = messages.length - 1; index >= 0; index--) {
 			const message = messages[index];

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