@oh-my-pi/pi-agent-core 15.5.8 → 15.5.10

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## [Unreleased]
 
+## [15.5.10] - 2026-05-28
+
+### Fixed
+
+- Fixed compaction summarizer throws losing the provider's HTTP status. `generateSummary`, `generateHandoff`, `generateShortSummary`, and `generateTurnPrefixSummary` now route their `stopReason === "error"` throws through a `createSummarizationError` helper that copies `AssistantMessage.errorStatus` onto the thrown `Error` as `.status`, letting downstream consumers (e.g. `AgentSession.#isCompactionAuthFailure` in `@oh-my-pi/pi-coding-agent`) branch on real provider 401/403s without regex-scraping the message body.
+
 ## [15.5.0] - 2026-05-26
 ### Added
 

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-agent-core",
-	"version": "15.5.8",
+	"version": "15.5.10",
 	"description": "General-purpose agent with transport abstraction, state management, and attachment support",
 	"homepage": "https://omp.sh",
 	"author": "Can Boluk",
@@ -35,9 +35,9 @@
 		"fmt": "biome format --write ."
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-ai": "15.5.8",
-		"@oh-my-pi/pi-natives": "15.5.8",
-		"@oh-my-pi/pi-utils": "15.5.8",
+		"@oh-my-pi/pi-ai": "15.5.10",
+		"@oh-my-pi/pi-natives": "15.5.10",
+		"@oh-my-pi/pi-utils": "15.5.10",
 		"@opentelemetry/api": "^1.9.0"
 	},
 	"devDependencies": {

package/src/compaction/compaction.ts

@@ -550,6 +550,23 @@ function resolveCompactionEffort(model: Model, level: ThinkingLevel | undefined)
 	return clampThinkingLevelForModel(model, requested);
 }
 
+/**
+ * Build the error thrown when an LLM summarization call ends with
+ * `stopReason === "error"`. Carries the provider's HTTP `errorStatus`
+ * onto a top-level `.status` field so callers (notably
+ * `AgentSession.#isCompactionAuthFailure`) can branch on 401/403 without
+ * regex-scraping `error.message`. The `auth_unavailable` synthetic
+ * (pi-native gateway) does not populate `errorStatus`, hence the legacy
+ * message-based check is still required upstream — see issue #986.
+ */
+function createSummarizationError(prefix: string, response: AssistantMessage): Error {
+	const error: Error & { status?: number } = new Error(`${prefix}: ${response.errorMessage || "Unknown error"}`);
+	if (response.errorStatus !== undefined) {
+		error.status = response.errorStatus;
+	}
+	return error;
+}
+
 /**
  * Generate a summary of the conversation using the LLM.
  * If previousSummary is provided, uses the update prompt to merge.
@@ -649,7 +666,7 @@ export async function generateSummary(
 	);
 
 	if (response.stopReason === "error") {
-		throw new Error(`Summarization failed: ${response.errorMessage || "Unknown error"}`);
+		throw createSummarizationError("Summarization failed", response);
 	}
 
 	const textContent = response.content
@@ -731,7 +748,7 @@ export async function generateHandoff(
 	);
 
 	if (response.stopReason === "error") {
-		throw new Error(`Handoff generation failed: ${response.errorMessage || "Unknown error"}`);
+		throw createSummarizationError("Handoff generation failed", response);
 	}
 
 	return response.content
@@ -790,7 +807,7 @@ async function generateShortSummary(
 	);
 
 	if (response.stopReason === "error") {
-		throw new Error(`Short summary failed: ${response.errorMessage || "Unknown error"}`);
+		throw createSummarizationError("Short summary failed", response);
 	}
 
 	return response.content
@@ -1128,7 +1145,7 @@ async function generateTurnPrefixSummary(
 	);
 
 	if (response.stopReason === "error") {
-		throw new Error(`Turn prefix summarization failed: ${response.errorMessage || "Unknown error"}`);
+		throw createSummarizationError("Turn prefix summarization failed", response);
 	}
 
 	return response.content