@oh-my-pi/pi-agent-core 18.12.3 → 18.12.4

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## [Unreleased]
 
+## [18.12.4] - 2026-10-20
+
+### Fixed
+
+- Enabled native compaction for supported Bedrock Claude models ([#16343](https://github.com/can1357/oh-my-pi/pull/16343) by [@anatoli-tsinovoy](https://github.com/anatoli-tsinovoy))
+
 ## [18.12.3] - 2026-10-19
 
 ### Added

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

@@ -14,6 +14,8 @@ export declare const ANTHROPIC_COMPACTION_PRESERVE_KEY = "anthropicCompaction";
 export interface AnthropicCompactionPreserveData {
     provider: string;
     content: string;
+    /** Opaque signed tool-change records; replayed without normalization. */
+    toolChanges?: Record<string, unknown>[];
     /** Signature attached to an on-demand block; replayed verbatim. */
     signature?: string;
     /** Legacy threshold block state; replay-only. */
@@ -35,9 +37,11 @@ export interface AnthropicCompactionPreserveData {
  * lineage the beta documents); endpoint eligibility is resolved the way the
  * provider routes requests, so a Foundry or `ANTHROPIC_BASE_URL` reroute of a
  * first-party model is excluded unless the route opted in with
- * `remoteCompaction.enabled`.
+ * `remoteCompaction.enabled`. Bedrock Converse models compact through a
+ * Bedrock Runtime side request; their ordinary turns stay on Converse.
  */
-export declare function shouldUseAnthropicNativeCompaction(model: Model): model is Model<"anthropic-messages">;
+export declare function shouldUseAnthropicNativeCompaction(model: Model): model is Model<"anthropic-messages" | "bedrock-converse-stream">;
+type AnthropicNativeCompactionModel = Model<"anthropic-messages" | "bedrock-converse-stream">;
 export declare function getPreservedAnthropicCompactionData(preserveData: Record<string, unknown> | undefined): AnthropicCompactionPreserveData | undefined;
 /** Set or strip the Anthropic compaction slot; a new compaction never inherits a stale summary. */
 export declare function withAnthropicCompactionPreserveData(preserveData: Record<string, unknown> | undefined, compaction: AnthropicCompactionPreserveData | undefined): Record<string, unknown> | undefined;
@@ -67,6 +71,7 @@ export interface AnthropicNativeCompactionRequest {
 export interface AnthropicNativeCompactionResponse {
     content: string;
     signature: string;
+    toolChanges?: Record<string, unknown>[];
     usage: Usage;
     model: string;
 }
@@ -81,4 +86,5 @@ export interface AnthropicNativeCompactionOptions extends Pick<SimpleStreamOptio
  * without a summary is a native failure, including tool use, refusals and
  * output limits; the configured method order can then choose a fallback.
  */
-export declare function requestAnthropicNativeCompaction(model: Model<"anthropic-messages">, apiKey: ApiKey, request: AnthropicNativeCompactionRequest, signal: AbortSignal | undefined, options: AnthropicNativeCompactionOptions): Promise<AnthropicNativeCompactionResponse>;
+export declare function requestAnthropicNativeCompaction(model: AnthropicNativeCompactionModel, apiKey: ApiKey, request: AnthropicNativeCompactionRequest, signal: AbortSignal | undefined, options: AnthropicNativeCompactionOptions): Promise<AnthropicNativeCompactionResponse>;
+export {};

package/package.json

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

package/src/compaction/anthropic.ts

@@ -19,7 +19,10 @@ import type {
 } from "@oh-my-pi/pi-ai";
 import * as AIError from "@oh-my-pi/pi-ai/error";
 import { resolveAnthropicCompactionEffort } from "@oh-my-pi/pi-ai/providers/anthropic";
-import { supportsAnthropicCompaction } from "@oh-my-pi/pi-ai/providers/anthropic-compaction";
+import {
+	supportsAnthropicCompaction,
+	supportsBedrockConverseCompaction,
+} from "@oh-my-pi/pi-ai/providers/anthropic-compaction";
 import { isRecord, prompt } from "@oh-my-pi/pi-utils";
 import { type InstrumentedChatSpanOptions, instrumentedCompleteSimple } from "../telemetry";
 import type { AgentMessage } from "../types";
@@ -31,6 +34,8 @@ export const ANTHROPIC_COMPACTION_PRESERVE_KEY = "anthropicCompaction";
 export interface AnthropicCompactionPreserveData {
 	provider: string;
 	content: string;
+	/** Opaque signed tool-change records; replayed without normalization. */
+	toolChanges?: Record<string, unknown>[];
 	/** Signature attached to an on-demand block; replayed verbatim. */
 	signature?: string;
 	/** Legacy threshold block state; replay-only. */
@@ -51,18 +56,28 @@ function isAnthropicMessagesModel(model: Model): model is Model<"anthropic-messa
 	return model.api === "anthropic-messages";
 }
 
+function isBedrockConverseModel(model: Model): model is Model<"bedrock-converse-stream"> {
+	return model.api === "bedrock-converse-stream";
+}
+
 /**
  * Whether a model compacts through the Anthropic compaction beta. Model
  * eligibility is catalog policy (`compat.supportsServerCompaction`, the
  * lineage the beta documents); endpoint eligibility is resolved the way the
  * provider routes requests, so a Foundry or `ANTHROPIC_BASE_URL` reroute of a
  * first-party model is excluded unless the route opted in with
- * `remoteCompaction.enabled`.
+ * `remoteCompaction.enabled`. Bedrock Converse models compact through a
+ * Bedrock Runtime side request; their ordinary turns stay on Converse.
  */
-export function shouldUseAnthropicNativeCompaction(model: Model): model is Model<"anthropic-messages"> {
-	return isAnthropicMessagesModel(model) && supportsAnthropicCompaction(model);
+export function shouldUseAnthropicNativeCompaction(
+	model: Model,
+): model is Model<"anthropic-messages" | "bedrock-converse-stream"> {
+	if (isAnthropicMessagesModel(model)) return supportsAnthropicCompaction(model);
+	return isBedrockConverseModel(model) && supportsBedrockConverseCompaction(model);
 }
 
+type AnthropicNativeCompactionModel = Model<"anthropic-messages" | "bedrock-converse-stream">;
+
 export function getPreservedAnthropicCompactionData(
 	preserveData: Record<string, unknown> | undefined,
 ): AnthropicCompactionPreserveData | undefined {
@@ -70,6 +85,9 @@ export function getPreservedAnthropicCompactionData(
 	if (!isRecord(candidate)) return undefined;
 	if (typeof candidate.provider !== "string" || candidate.provider.length === 0) return undefined;
 	if (typeof candidate.content !== "string" || candidate.content.length === 0) return undefined;
+	const toolChanges = candidate.toolChanges;
+	// The signature covers `tool_changes`: a block missing or mangling them cannot replay.
+	if (toolChanges !== undefined && (!Array.isArray(toolChanges) || !toolChanges.every(isRecord))) return undefined;
 	return {
 		provider: candidate.provider,
 		content: candidate.content,
@@ -90,6 +108,7 @@ export function getPreservedAnthropicCompactionData(
 					),
 				}
 			: {}),
+		...(toolChanges !== undefined ? { toolChanges } : {}),
 		...(candidate.exactTail === true ? { exactTail: true as const } : {}),
 		...(typeof candidate.model === "string" ? { model: candidate.model } : {}),
 		...(typeof candidate.usedTokens === "number" ? { usedTokens: candidate.usedTokens } : {}),
@@ -124,6 +143,7 @@ export function getAnthropicCompactionPayload(
 		...(preserved.signature ? { signature: preserved.signature } : {}),
 		...(preserved.encryptedContent ? { encryptedContent: preserved.encryptedContent } : {}),
 		...(preserved.filesText ? { filesText: preserved.filesText } : {}),
+		...(preserved.toolChanges ? { toolChanges: preserved.toolChanges } : {}),
 		...(preserved.retainedFiles?.length ? { retainedFiles: preserved.retainedFiles } : {}),
 		...(preserved.exactTail ? { exactTail: true as const } : {}),
 	};
@@ -196,6 +216,7 @@ export interface AnthropicNativeCompactionRequest {
 export interface AnthropicNativeCompactionResponse {
 	content: string;
 	signature: string;
+	toolChanges?: Record<string, unknown>[];
 	usage: Usage;
 	model: string;
 }
@@ -224,7 +245,7 @@ export interface AnthropicNativeCompactionOptions
  * output limits; the configured method order can then choose a fallback.
  */
 export async function requestAnthropicNativeCompaction(
-	model: Model<"anthropic-messages">,
+	model: AnthropicNativeCompactionModel,
 	apiKey: ApiKey,
 	request: AnthropicNativeCompactionRequest,
 	signal: AbortSignal | undefined,
@@ -237,7 +258,9 @@ export async function requestAnthropicNativeCompaction(
 			apiKey,
 			signal,
 			maxTokens: request.maxTokens,
-			reasoning: resolveAnthropicCompactionEffort(model, request.context.messages, request.reasoning),
+			reasoning: isAnthropicMessagesModel(model)
+				? resolveAnthropicCompactionEffort(model, request.context.messages, request.reasoning)
+				: request.reasoning,
 			initiatorOverride: options.initiatorOverride,
 			metadata: options.metadata,
 			fetch: options.fetch,
@@ -279,6 +302,7 @@ export async function requestAnthropicNativeCompaction(
 	return {
 		content: payload.content,
 		signature: payload.signature,
+		...(payload.toolChanges ? { toolChanges: payload.toolChanges } : {}),
 		usage: response.usage,
 		model: response.model,
 	};

package/src/compaction/compaction.ts

@@ -2024,6 +2024,7 @@ export async function compact(
 	let nativeFirstKeptEntryId = firstKeptEntryId;
 	let nativeUsedTokens: number | undefined;
 	let nativeRetainedFiles: AnthropicCompactionFiles[] | undefined;
+	let nativeToolChanges: Record<string, unknown>[] | undefined;
 	if (!usedRemoteCompaction && settings.remoteEnabled !== false && shouldUseAnthropicNativeCompaction(model)) {
 		const previousNative = getPreservedAnthropicCompactionData(previousPreserveData);
 		// Lead with the previous summary as the live context renders it:
@@ -2052,6 +2053,7 @@ export async function compact(
 											: {}),
 										...(previousNative.filesText ? { filesText: previousNative.filesText } : {}),
 										...(previousNative.retainedFiles ? { retainedFiles: previousNative.retainedFiles } : {}),
+										...(previousNative.toolChanges ? { toolChanges: previousNative.toolChanges } : {}),
 										...(previousNative.exactTail ? { exactTail: true as const } : {}),
 									}
 								: undefined,
@@ -2148,6 +2150,7 @@ export async function compact(
 			);
 			nativeSummary = remote.content;
 			nativeSignature = remote.signature;
+			nativeToolChanges = remote.toolChanges;
 			nativeUsedTokens = calculatePromptTokens(remote.usage);
 			usedRemoteCompaction = true;
 		} catch (err) {
@@ -2235,16 +2238,18 @@ export async function compact(
 	const { readFiles, modifiedFiles } = computeFileLists(fileOps);
 	summary = upsertFileOperations(summary, readFiles, modifiedFiles, fileOps.read);
 	if (nativeSummary !== undefined) {
-		// The replayed block stays byte-identical to the API's summary so it
-		// matches its signature. The harness file lists above travel
-		// separately: the converter replaces the summary message with the
-		// block and skips its text, so they would otherwise be invisible to
-		// this provider. Every other provider keeps reading the entry text.
+		// The replayed block (summary and any `tool_changes`) stays
+		// byte-identical to the API's output so it matches its signature. The
+		// harness file lists above travel separately: the converter replaces the
+		// summary message with the block and skips its text, so they would
+		// otherwise be invisible to this provider. Every other provider keeps
+		// reading the entry text.
 		const filesText = upsertFileOperations("", readFiles, modifiedFiles, fileOps.read) || undefined;
 		preserveData = withAnthropicCompactionPreserveData(preserveData, {
 			provider: model.provider,
 			content: nativeSummary,
 			...(nativeSignature ? { signature: nativeSignature } : {}),
+			...(nativeToolChanges ? { toolChanges: nativeToolChanges } : {}),
 			...(filesText ? { filesText } : {}),
 			...(nativeRetainedFiles ? { retainedFiles: nativeRetainedFiles } : {}),
 			exactTail: true,