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

package/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 ## [Unreleased]
 
+## [15.5.10] - 2026-05-28
+
+### Added
+
+- Added `/drop-images` slash command that strips every `ImageContent` block from the current session's branch — `user`/`developer`/`custom`/`hookMessage`/`toolResult` content arrays plus `toolResult.details.images` and `fileMention.files[].image` — rewrites the session JSONL, rebuilds the agent's in-memory message list, tears down Codex Responses provider sessions, and rebuilds the TUI chat container so the change is visible immediately. ACP clients receive the same handler (returns `"Dropped N images …"` / `"No images found …"` through `runtime.output`). Stripping content that would leave a `toolResult` or `user` message with zero blocks inserts a single `[image removed]` placeholder so providers do not reject empty content arrays.
+
+### Fixed
+
+- Fixed compaction surfacing raw HTTP 401/403 envelopes (e.g. `Compaction failed: 401 {"type":"error","error":{"type":"authentication_error",…}}`) instead of routing to an authenticated fallback model. The compaction layer now attaches the provider-reported HTTP status onto the thrown error, and `AgentSession`'s auth-failure detector branches on `error.status === 401 || 403` in addition to the existing `auth_unavailable` regex. When a fallback model role (e.g. `modelRoles.smol`) is configured, compaction retries it transparently; otherwise the user sees the actionable "Compaction requires usable credentials for …" hint instead of the raw provider envelope.
+
 ## [15.5.8] - 2026-05-28
 
 ### Breaking Changes

package/dist/types/session/agent-session.d.ts

@@ -667,6 +667,21 @@ export declare class AgentSession {
      * Saves to settings.
      */
     setInterruptMode(mode: "immediate" | "wait"): void;
+    /**
+     * Strip image content blocks from every message on the current branch and
+     * persist the rewrite. Walks `SessionManager.getBranch()` in place — both
+     * `SessionMessageEntry.message` and `CustomMessageEntry.content` arrays
+     * are mutated, then `rewriteEntries` durably commits the new shape. The
+     * agent's runtime view is rebuilt from the freshly-mutated entries so any
+     * provider sessions caching message identity (Codex Responses) are torn
+     * down to force a clean replay on the next turn.
+     *
+     * No-op when the branch carries no images; returns `{ removed: 0 }` and
+     * skips the disk rewrite.
+     */
+    dropImages(): Promise<{
+        removed: number;
+    }>;
     /**
      * Manually compact the session context.
      * Aborts current agent operation first.

package/dist/types/session/messages.d.ts

@@ -55,6 +55,16 @@ export declare const INTERNAL_DETAILS_FIELDS: readonly ["__pendingDisplayTag"];
  *  (null/non-object, or no listed fields present) so callers don't pay a
  *  clone cost on the common path. */
 export declare function stripInternalDetailsFields<T>(details: T | undefined): T | undefined;
+/**
+ * Strip image content blocks from `message` in place. Returns the count of
+ * images removed across `content` (every role that carries `ImageContent`) and
+ * any tool-result `details.images` payload. Callers MUST rewrite session
+ * entries (`SessionManager.rewriteEntries`) and replay them through
+ * `Agent.replaceMessages` afterwards so persisted state and provider-side
+ * caches stay aligned with the mutated tree — `stripImagesFromMessage` is a
+ * pure local mutation and intentionally does neither.
+ */
+export declare function stripImagesFromMessage(message: AgentMessage): number;
 /**
  * Message type for bash executions via the ! command.
  */

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "15.5.8",
+	"version": "15.5.10",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://omp.sh",
 	"author": "Can Boluk",
@@ -47,13 +47,13 @@
 		"@agentclientprotocol/sdk": "0.21.0",
 		"@babel/parser": "^7.29.3",
 		"@mozilla/readability": "^0.6.0",
-		"@oh-my-pi/hashline": "15.5.8",
-		"@oh-my-pi/omp-stats": "15.5.8",
-		"@oh-my-pi/pi-agent-core": "15.5.8",
-		"@oh-my-pi/pi-ai": "15.5.8",
-		"@oh-my-pi/pi-natives": "15.5.8",
-		"@oh-my-pi/pi-tui": "15.5.8",
-		"@oh-my-pi/pi-utils": "15.5.8",
+		"@oh-my-pi/hashline": "15.5.10",
+		"@oh-my-pi/omp-stats": "15.5.10",
+		"@oh-my-pi/pi-agent-core": "15.5.10",
+		"@oh-my-pi/pi-ai": "15.5.10",
+		"@oh-my-pi/pi-natives": "15.5.10",
+		"@oh-my-pi/pi-tui": "15.5.10",
+		"@oh-my-pi/pi-utils": "15.5.10",
 		"@puppeteer/browsers": "^2.13.0",
 		"@types/turndown": "5.0.6",
 		"@xterm/headless": "^6.0.0",

package/src/session/agent-session.ts

@@ -196,6 +196,7 @@ import {
 	type PythonExecutionMessage,
 	readPendingDisplayTag,
 	SILENT_ABORT_MARKER,
+	stripImagesFromMessage,
 } from "./messages";
 import { formatSessionDumpText } from "./session-dump-format";
 import type {
@@ -5310,6 +5311,55 @@ export class AgentSession {
 		return result;
 	}
 
+	/**
+	 * Strip image content blocks from every message on the current branch and
+	 * persist the rewrite. Walks `SessionManager.getBranch()` in place — both
+	 * `SessionMessageEntry.message` and `CustomMessageEntry.content` arrays
+	 * are mutated, then `rewriteEntries` durably commits the new shape. The
+	 * agent's runtime view is rebuilt from the freshly-mutated entries so any
+	 * provider sessions caching message identity (Codex Responses) are torn
+	 * down to force a clean replay on the next turn.
+	 *
+	 * No-op when the branch carries no images; returns `{ removed: 0 }` and
+	 * skips the disk rewrite.
+	 */
+	async dropImages(): Promise<{ removed: number }> {
+		const branchEntries = this.sessionManager.getBranch();
+		let removed = 0;
+		for (const entry of branchEntries) {
+			if (entry.type === "message") {
+				removed += stripImagesFromMessage(entry.message);
+				continue;
+			}
+			if (entry.type === "custom_message" && typeof entry.content !== "string") {
+				const kept: typeof entry.content = [];
+				let dropped = 0;
+				for (const part of entry.content) {
+					if (part.type === "image") {
+						dropped++;
+					} else {
+						kept.push(part);
+					}
+				}
+				if (dropped > 0) {
+					if (kept.length === 0) {
+						kept.push({ type: "text", text: "[image removed]" });
+					}
+					entry.content = kept;
+					removed += dropped;
+				}
+			}
+		}
+		if (removed === 0) {
+			return { removed: 0 };
+		}
+		await this.sessionManager.rewriteEntries();
+		const sessionContext = this.buildDisplaySessionContext();
+		this.agent.replaceMessages(sessionContext.messages);
+		this.#closeCodexProviderSessionsForHistoryRewrite();
+		return { removed };
+	}
+
 	/**
 	 * Manually compact the session context.
 	 * Aborts current agent operation first.
@@ -6379,6 +6429,14 @@ export class AgentSession {
 	}
 	#isCompactionAuthFailure(error: unknown): boolean {
 		if (!(error instanceof Error)) return false;
+		// Real provider 401/403 — surfaced as `.status` by the compaction layer
+		// (see `createSummarizationError` in packages/agent/src/compaction/compaction.ts).
+		// Without this branch, an expired/revoked Anthropic key would bypass the
+		// authenticated-fallback path and dump the raw HTTP body into the UI.
+		const status = (error as Error & { status?: number }).status;
+		if (status === 401 || status === 403) return true;
+		// pi-native gateway synthetic for "no credential configured" (issue #986).
+		// Carries no HTTP status, so the legacy message regex stays.
 		return /auth_unavailable|no auth available/i.test(error.message);
 	}
 

package/src/session/messages.ts

@@ -114,6 +114,98 @@ function getPrunedToolResultContent(message: ToolResultMessage): (TextContent |
 	return [{ type: "text", text }];
 }
 
+/** Result of filtering image blocks out of a `(TextContent | ImageContent)[]` array. */
+interface StripContentResult {
+	content: (TextContent | ImageContent)[];
+	removed: number;
+}
+
+function stripImagesFromArrayContent(content: (TextContent | ImageContent)[]): StripContentResult {
+	let removed = 0;
+	const kept: (TextContent | ImageContent)[] = [];
+	for (const part of content) {
+		if (part.type === "image") {
+			removed++;
+		} else {
+			kept.push(part);
+		}
+	}
+	if (removed === 0) {
+		return { content, removed };
+	}
+	// Avoid emitting an empty `content` array — providers reject zero-block user/tool
+	// messages and the LLM still needs to see *something* where the image used to be.
+	if (kept.length === 0) {
+		kept.push({ type: "text", text: "[image removed]" });
+	}
+	return { content: kept, removed };
+}
+
+/**
+ * Strip image content blocks from `message` in place. Returns the count of
+ * images removed across `content` (every role that carries `ImageContent`) and
+ * any tool-result `details.images` payload. Callers MUST rewrite session
+ * entries (`SessionManager.rewriteEntries`) and replay them through
+ * `Agent.replaceMessages` afterwards so persisted state and provider-side
+ * caches stay aligned with the mutated tree — `stripImagesFromMessage` is a
+ * pure local mutation and intentionally does neither.
+ */
+export function stripImagesFromMessage(message: AgentMessage): number {
+	switch (message.role) {
+		case "user":
+		case "developer":
+		case "custom":
+		case "hookMessage": {
+			if (typeof message.content === "string") return 0;
+			const { content, removed } = stripImagesFromArrayContent(message.content);
+			if (removed > 0) {
+				// All four roles type `content` as `string | (TextContent | ImageContent)[]`;
+				// TypeScript can't narrow the assignment across the union, so cast once.
+				(message as { content: typeof content }).content = content;
+			}
+			return removed;
+		}
+		case "toolResult": {
+			let removed = 0;
+			const { content, removed: contentRemoved } = stripImagesFromArrayContent(message.content);
+			if (contentRemoved > 0) {
+				message.content = content;
+				removed += contentRemoved;
+			}
+			const details = message.details as { images?: unknown } | null | undefined;
+			if (details && Array.isArray(details.images)) {
+				const original = details.images as unknown[];
+				const kept: unknown[] = [];
+				for (const candidate of original) {
+					const looksLikeImageBlock =
+						!!candidate && typeof candidate === "object" && (candidate as { type?: unknown }).type === "image";
+					if (looksLikeImageBlock) {
+						removed++;
+					} else {
+						kept.push(candidate);
+					}
+				}
+				if (kept.length !== original.length) {
+					details.images = kept;
+				}
+			}
+			return removed;
+		}
+		case "fileMention": {
+			let removed = 0;
+			for (const file of message.files) {
+				if (file.image) {
+					file.image = undefined;
+					removed++;
+				}
+			}
+			return removed;
+		}
+		default:
+			return 0;
+	}
+}
+
 /**
  * Message type for bash executions via the ! command.
  */

package/src/slash-commands/builtin-registry.ts

@@ -802,6 +802,30 @@ const BUILTIN_SLASH_COMMAND_REGISTRY: ReadonlyArray<SlashCommandSpec> = [
 			await runtime.ctx.handleCompactCommand(customInstructions);
 		},
 	},
+	{
+		name: "drop-images",
+		description: "Strip every image from this session's history",
+		acpDescription: "Drop all images from the conversation history",
+		handle: async (_command, runtime) => {
+			const { removed } = await runtime.session.dropImages();
+			await runtime.output(
+				removed === 0
+					? "No images found in this session."
+					: `Dropped ${removed} image${removed === 1 ? "" : "s"} from this session.`,
+			);
+			return commandConsumed();
+		},
+		handleTui: async (_command, runtime) => {
+			runtime.ctx.editor.setText("");
+			const { removed } = await runtime.ctx.session.dropImages();
+			if (removed === 0) {
+				runtime.ctx.showStatus("No images found in this session.");
+				return;
+			}
+			runtime.ctx.rebuildChatFromMessages();
+			runtime.ctx.showStatus(`Dropped ${removed} image${removed === 1 ? "" : "s"} from this session.`);
+		},
+	},
 	{
 		name: "handoff",
 		description: "Hand off session context to a new session",