@oh-my-pi/pi-agent-core 18.3.1 → 18.3.2

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 ## [Unreleased]
 
+## [18.3.2] - 2026-09-25
+
+### Fixed
+
+- Fixed tool calls that put their payload in the intent field `i` (for example a file body in `write`) silently running with the leftover arguments; they now fail with an error telling the model to retry ([#13140](https://github.com/can1357/oh-my-pi/issues/13140), [#13141](https://github.com/can1357/oh-my-pi/pull/13141) by [@radkawar](https://github.com/radkawar))
+- Fixed the Anthropic compaction failure log omitting why no compaction block came back; it now names the stop reason ([#13300](https://github.com/can1357/oh-my-pi/pull/13300) by [@alphastorm](https://github.com/alphastorm))
+
 ## [18.3.1] - 2026-09-25
 
 ### Added

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

@@ -82,6 +82,15 @@ export interface SupersedePruneConfig {
     /** Tool-result protection matchers (same contract as {@link PruneConfig.protectedTools}). */
     protectedTools: ProtectedToolMatcher[];
 }
+/**
+ * Generic age-based pruning floor. Below this, blanking a result to
+ * `[Output truncated - N tokens]` recovers nothing — the placeholder itself
+ * costs ~8 tokens, so a sub-floor result grows the context (and churns the
+ * prompt cache) instead of shrinking it. Superseded/useless results keep their
+ * own rules: useless already drops no-savings candidates, superseded prunes for
+ * correctness regardless of size.
+ */
+export declare const MIN_PRUNE_TOKENS = 50;
 /**
  * Prune superseded tool results (e.g. stale `read` outputs replaced by a newer
  * read of the same file) and, when `pruneUseless` is set, results their tool

package/dist/types/compaction/transcript-tokens.d.ts

@@ -37,6 +37,14 @@ export interface TranscriptUsageAnchor {
  * the session-entry walkers.
  */
 export declare function isTranscriptUsageAnchor(message: AgentMessage): message is AssistantMessage;
+/** Options for {@link findTranscriptUsageAnchor}. */
+export interface TranscriptUsageAnchorOptions {
+    /**
+     * Also treat usage reported at or before the newest `prunedAt` as stale,
+     * the same rule {@link findRequestUsageAnchor} applies to request contexts.
+     */
+    skipPrunedAnchors?: boolean;
+}
 /**
  * Newest assistant turn in `messages[fromIndex..]` whose usage can anchor the
  * transcript, or `undefined` when none qualifies (fresh context, or every
@@ -45,7 +53,7 @@ export declare function isTranscriptUsageAnchor(message: AgentMessage): message
  * `fromIndex` excludes turns whose usage is stale — anything a compaction
  * summarized away describes a prompt that is no longer sent.
  */
-export declare function findTranscriptUsageAnchor(messages: readonly AgentMessage[], fromIndex?: number): TranscriptUsageAnchor | undefined;
+export declare function findTranscriptUsageAnchor(messages: readonly AgentMessage[], fromIndex?: number, options?: TranscriptUsageAnchorOptions): TranscriptUsageAnchor | undefined;
 /**
  * Newest assistant turn in a provider request's `messages` whose usage still
  * describes the prefix it sits on, or `undefined` when none does.
@@ -67,6 +75,8 @@ export interface TranscriptTokenOptions {
      * accounting is governed separately by {@link countFromIndex}.
      */
     anchorFromIndex?: number;
+    /** Forwarded to {@link findTranscriptUsageAnchor}. */
+    skipPrunedAnchors?: boolean;
     /**
      * First message whose content is counted locally when no anchor is found.
      * Defaults to 0 (count the whole transcript), which is what a floor

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-agent-core",
-	"version": "18.3.1",
+	"version": "18.3.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.3.1",
-		"@oh-my-pi/pi-catalog": "18.3.1",
-		"@oh-my-pi/pi-natives": "18.3.1",
-		"@oh-my-pi/pi-utils": "18.3.1",
-		"@oh-my-pi/pi-wire": "18.3.1",
-		"@oh-my-pi/snapcompact": "18.3.1",
+		"@oh-my-pi/pi-ai": "18.3.2",
+		"@oh-my-pi/pi-catalog": "18.3.2",
+		"@oh-my-pi/pi-natives": "18.3.2",
+		"@oh-my-pi/pi-utils": "18.3.2",
+		"@oh-my-pi/pi-wire": "18.3.2",
+		"@oh-my-pi/snapcompact": "18.3.2",
 		"@opentelemetry/api": "^1.9.1"
 	},
 	"devDependencies": {
-		"@oh-my-pi/omptype": "18.3.1",
+		"@oh-my-pi/omptype": "18.3.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

@@ -39,6 +39,7 @@ import {
 	getStreamingPartialJson,
 	kCursorExecResolved,
 } from "@oh-my-pi/pi-ai/utils/block-symbols";
+import { schemaDefinesProperty } from "@oh-my-pi/pi-ai/utils/schema/json-schema-validator";
 import { stamp } from "@oh-my-pi/pi-ai/utils/schema/stamps";
 import {
 	createHarmonyAuditEvent,
@@ -1022,6 +1023,13 @@ function resolveIntentMode(intent: AgentTool["intent"]): "require" | "optional"
 	return "require";
 }
 
+/**
+ * Longest `i` value accepted as an intent. The injected field is described as
+ * a "concise intent" (INTENT_FIELD_DESCRIPTION); anything past this is a tool
+ * payload the model put in the wrong field, not a label.
+ */
+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") {
@@ -2830,6 +2838,19 @@ async function prepareToolCallDispatch(
 		if (intentTracing) {
 			const { intent, strippedArgs } = extractIntent(toolCall.arguments);
 			argsForExecution = strippedArgs;
+			// A payload in `i` would be stripped and the tool run with the leftover
+			// args. Unknown tools fall through to the not-found error; a tool that
+			// owns `i` as a real parameter has nowhere else to put the value.
+			if (
+				intent !== undefined &&
+				intent.length > MAX_INTENT_LENGTH &&
+				tool &&
+				!schemaDefinesProperty(toolWireSchema(tool), INTENT_FIELD)
+			) {
+				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/compaction/anthropic.ts

@@ -255,7 +255,7 @@ export async function requestAnthropicNativeCompaction(
 		throw new Error(
 			response.stopDetails?.type === "compaction"
 				? "Anthropic compaction returned no signed summary"
-				: "Anthropic compaction response carried no compaction block",
+				: `Anthropic compaction response carried no compaction block (stop reason: ${response.stopDetails?.type ?? response.stopReason})`,
 		);
 	}
 	return {

package/src/compaction/pruning.ts

@@ -120,7 +120,7 @@ function createPrunedNotice(tokens: number): string {
  * own rules: useless already drops no-savings candidates, superseded prunes for
  * correctness regardless of size.
  */
-const MIN_PRUNE_TOKENS = 50;
+export const MIN_PRUNE_TOKENS = 50;
 
 function getToolResultMessage(entry: SessionEntry): ToolResultMessage | undefined {
 	if (entry.type !== "message") return undefined;

package/src/compaction/transcript-tokens.ts

@@ -47,6 +47,31 @@ export function isTranscriptUsageAnchor(message: AgentMessage): message is Assis
 	return assistant.usage !== undefined && hasContextTokenUsage(assistant.usage);
 }
 
+/**
+ * Newest `prunedAt` in `messages`, or `-Infinity` when nothing was pruned.
+ *
+ * A pruned tool result was rewritten in place, so every usage report made at
+ * or before this time still counts the removed bytes.
+ */
+function newestPrunedAt(messages: readonly AgentMessage[]): number {
+	let rewriteAt = Number.NEGATIVE_INFINITY;
+	for (const message of messages) {
+		if (message.role === "toolResult" && message.prunedAt !== undefined) {
+			rewriteAt = Math.max(rewriteAt, message.prunedAt);
+		}
+	}
+	return rewriteAt;
+}
+
+/** Options for {@link findTranscriptUsageAnchor}. */
+export interface TranscriptUsageAnchorOptions {
+	/**
+	 * Also treat usage reported at or before the newest `prunedAt` as stale,
+	 * the same rule {@link findRequestUsageAnchor} applies to request contexts.
+	 */
+	skipPrunedAnchors?: boolean;
+}
+
 /**
  * Newest assistant turn in `messages[fromIndex..]` whose usage can anchor the
  * transcript, or `undefined` when none qualifies (fresh context, or every
@@ -58,10 +83,12 @@ export function isTranscriptUsageAnchor(message: AgentMessage): message is Assis
 export function findTranscriptUsageAnchor(
 	messages: readonly AgentMessage[],
 	fromIndex = 0,
+	options?: TranscriptUsageAnchorOptions,
 ): TranscriptUsageAnchor | undefined {
+	const rewriteAt = options?.skipPrunedAnchors === true ? newestPrunedAt(messages) : Number.NEGATIVE_INFINITY;
 	for (let index = messages.length - 1; index >= fromIndex; index--) {
 		const message = messages[index];
-		if (!isTranscriptUsageAnchor(message)) continue;
+		if (!isTranscriptUsageAnchor(message) || message.timestamp <= rewriteAt) continue;
 		return { index, message, tokens: calculateContextTokens(message.usage) };
 	}
 	return undefined;
@@ -105,6 +132,8 @@ export interface TranscriptTokenOptions {
 	 * accounting is governed separately by {@link countFromIndex}.
 	 */
 	anchorFromIndex?: number;
+	/** Forwarded to {@link findTranscriptUsageAnchor}. */
+	skipPrunedAnchors?: boolean;
 	/**
 	 * First message whose content is counted locally when no anchor is found.
 	 * Defaults to 0 (count the whole transcript), which is what a floor
@@ -132,7 +161,9 @@ export function estimateTranscriptTokens(
 ): number {
 	const estimateOptions: MessageCountOptions | undefined =
 		options?.excludeEncryptedReasoning === true ? { excludeEncryptedReasoning: true } : undefined;
-	const anchor = findTranscriptUsageAnchor(messages, options?.anchorFromIndex ?? 0);
+	const anchor = findTranscriptUsageAnchor(messages, options?.anchorFromIndex ?? 0, {
+		skipPrunedAnchors: options?.skipPrunedAnchors,
+	});
 	let total = anchor?.tokens ?? 0;
 	for (let index = anchor ? anchor.index + 1 : (options?.countFromIndex ?? 0); index < messages.length; index++) {
 		total += tokenizer.countMessage(messages[index], estimateOptions);