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

package/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 ## [Unreleased]
 
+## [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/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/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-agent-core",
-	"version": "18.14.0",
+	"version": "18.14.1",
 	"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.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",
 		"@opentelemetry/api": "^1.9.1"
 	},
 	"devDependencies": {
-		"@oh-my-pi/omptype": "18.14.0",
+		"@oh-my-pi/omptype": "18.14.1",
 		"@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") {

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/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

@@ -750,6 +750,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. */
@@ -1765,6 +1767,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 +2003,7 @@ export async function compact(
 						fetch: summaryOptions.fetch,
 						providerSessionState: summaryOptions.providerSessionState,
 						preferWebsockets: summaryOptions.preferWebsockets,
+						codexAccessPrograms: summaryOptions.codexAccessPrograms,
 						codexCompaction: summaryOptions.codexCompaction,
 					}),
 				{ signal },

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];