@oh-my-pi/pi-coding-agent 5.0.0 → 5.0.1

package/CHANGELOG.md

@@ -2,6 +2,14 @@
 
 ## [Unreleased]
 
+## [5.0.1] - 2026-01-12
+### Changed
+
+- Replaced wasm-vips with Photon for more stable WASM image processing
+- Added graceful fallback to original images when image resizing fails
+- Added error handling for image conversion failures in interactive mode to prevent crashes
+- Replace wasm-vips with Photon for more stable WASM image processing (fixes worker thread crashes)
+
 ## [5.0.0] - 2026-01-12
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "5.0.0",
+	"version": "5.0.1",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"ompConfig": {
@@ -39,11 +39,12 @@
 		"prepublishOnly": "bun run generate-template && bun run clean && bun run build"
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-agent-core": "5.0.0",
-		"@oh-my-pi/pi-ai": "5.0.0",
-		"@oh-my-pi/pi-git-tool": "5.0.0",
-		"@oh-my-pi/pi-tui": "5.0.0",
+		"@oh-my-pi/pi-agent-core": "5.0.1",
+		"@oh-my-pi/pi-ai": "5.0.1",
+		"@oh-my-pi/pi-git-tool": "5.0.1",
+		"@oh-my-pi/pi-tui": "5.0.1",
 		"@openai/agents": "^0.3.7",
+		"@silvia-odwyer/photon": "^0.3.3",
 		"@sinclair/typebox": "^0.34.46",
 		"ajv": "^8.17.1",
 		"chalk": "^5.5.0",
@@ -60,7 +61,6 @@
 		"node-html-parser": "^6.1.13",
 		"smol-toml": "^1.6.0",
 		"strip-ansi": "^7.1.2",
-		"wasm-vips": "^0.0.16",
 		"winston": "^3.17.0",
 		"winston-daily-rotate-file": "^5.0.0",
 		"zod": "^4.3.5"

package/src/cli/file-processor.ts

@@ -52,13 +52,22 @@ export async function processFileArguments(fileArgs: string[], options?: Process
 			let dimensionNote: string | undefined;
 
 			if (_autoResizeImages) {
-				const resized = await resizeImage({ type: "image", data: base64Content, mimeType });
-				dimensionNote = formatDimensionNote(resized);
-				attachment = {
-					type: "image",
-					mimeType: resized.mimeType,
-					data: resized.data,
-				};
+				try {
+					const resized = await resizeImage({ type: "image", data: base64Content, mimeType });
+					dimensionNote = formatDimensionNote(resized);
+					attachment = {
+						type: "image",
+						mimeType: resized.mimeType,
+						data: resized.data,
+					};
+				} catch {
+					// Fall back to original image on resize failure
+					attachment = {
+						type: "image",
+						mimeType,
+						data: base64Content,
+					};
+				}
 			} else {
 				attachment = {
 					type: "image",

package/src/core/system-prompt.ts

@@ -632,6 +632,10 @@ export interface BuildSystemPromptOptions {
 
 /** Build the system prompt with tools, guidelines, and context */
 export async function buildSystemPrompt(options: BuildSystemPromptOptions = {}): Promise<string> {
+	if (process.env.NULL_PROMPT === "true") {
+		return "";
+	}
+
 	const {
 		customPrompt,
 		tools,

package/src/core/tools/read.ts

@@ -518,19 +518,27 @@ export function createReadTool(session: ToolSession): AgentTool<typeof readSchem
 							const base64 = Buffer.from(buffer).toString("base64");
 
 							if (autoResizeImages) {
-								// Resize image if needed
-								const resized = await resizeImage({ type: "image", data: base64, mimeType });
-								const dimensionNote = formatDimensionNote(resized);
-
-								let textNote = `Read image file [${resized.mimeType}]`;
-								if (dimensionNote) {
-									textNote += `\n${dimensionNote}`;
+								// Resize image if needed - catch errors from WASM
+								try {
+									const resized = await resizeImage({ type: "image", data: base64, mimeType });
+									const dimensionNote = formatDimensionNote(resized);
+
+									let textNote = `Read image file [${resized.mimeType}]`;
+									if (dimensionNote) {
+										textNote += `\n${dimensionNote}`;
+									}
+
+									content = [
+										{ type: "text", text: textNote },
+										{ type: "image", data: resized.data, mimeType: resized.mimeType },
+									];
+								} catch {
+									// Fall back to original image on resize failure
+									content = [
+										{ type: "text", text: `Read image file [${mimeType}]` },
+										{ type: "image", data: base64, mimeType },
+									];
 								}
-
-								content = [
-									{ type: "text", text: textNote },
-									{ type: "image", data: resized.data, mimeType: resized.mimeType },
-								];
 							} else {
 								content = [
 									{ type: "text", text: `Read image file [${mimeType}]` },

package/src/modes/interactive/components/tool-execution.ts

@@ -261,15 +261,19 @@ export class ToolExecutionComponent extends Container {
 			if (img.mimeType === "image/png") continue;
 			if (this.convertedImages.has(i)) continue;
 
-			// Convert async
+			// Convert async - catch errors from WASM processing
 			const index = i;
-			convertToPng(img.data, img.mimeType).then((converted) => {
-				if (converted) {
-					this.convertedImages.set(index, converted);
-					this.updateDisplay();
-					this.ui.requestRender();
-				}
-			});
+			convertToPng(img.data, img.mimeType)
+				.then((converted) => {
+					if (converted) {
+						this.convertedImages.set(index, converted);
+						this.updateDisplay();
+						this.ui.requestRender();
+					}
+				})
+				.catch(() => {
+					// Ignore conversion failures - display will use original image format
+				});
 		}
 	}
 

package/src/utils/image-convert.ts

@@ -1,11 +1,11 @@
 import { logger } from "../core/logger";
 import { convertToPngWithImageMagick } from "./image-magick";
-import { Vips } from "./vips";
+import { getPhoton } from "./photon";
 
 /**
  * Convert image to PNG format for terminal display.
  * Kitty graphics protocol requires PNG format (f=100).
- * Uses wasm-vips if available, falls back to ImageMagick (magick/convert).
+ * Uses Photon (Rust/WASM) if available, falls back to ImageMagick.
  */
 export async function convertToPng(
 	base64Data: string,
@@ -17,20 +17,20 @@ export async function convertToPng(
 	}
 
 	try {
-		const { Image } = await Vips();
-		const image = Image.newFromBuffer(Buffer.from(base64Data, "base64"));
+		const photon = await getPhoton();
+		const image = photon.PhotonImage.new_from_byteslice(new Uint8Array(Buffer.from(base64Data, "base64")));
 		try {
-			const pngBuffer = image.writeToBuffer(".png");
+			const pngBuffer = image.get_bytes();
 			return {
 				data: Buffer.from(pngBuffer).toString("base64"),
 				mimeType: "image/png",
 			};
 		} finally {
-			image.delete();
+			image.free();
 		}
 	} catch (error) {
-		// wasm-vips failed, try ImageMagick fallback
-		logger.error("Failed to convert image to PNG with wasm-vips", {
+		// Photon failed, try ImageMagick fallback
+		logger.error("Failed to convert image to PNG with Photon", {
 			error: error instanceof Error ? error.message : String(error),
 		});
 	}

package/src/utils/image-resize.ts

@@ -1,7 +1,7 @@
 import type { ImageContent } from "@oh-my-pi/pi-ai";
 import { logger } from "../core/logger";
 import { getImageDimensionsWithImageMagick, resizeWithImageMagick } from "./image-magick";
-import { Vips } from "./vips";
+import { getPhoton } from "./photon";
 
 export interface ImageResizeOptions {
 	maxWidth?: number; // Default: 2000
@@ -31,7 +31,7 @@ const DEFAULT_OPTIONS: Required<ImageResizeOptions> = {
 };
 
 /**
- * Fallback resize using ImageMagick when wasm-vips is unavailable.
+ * Fallback resize using ImageMagick when Photon is unavailable.
  */
 async function resizeImageWithImageMagick(
 	img: ImageContent,
@@ -85,7 +85,7 @@ function pickSmaller(
  * Resize an image to fit within the specified max dimensions and file size.
  * Returns the original image if it already fits within the limits.
  *
- * Uses wasm-vips for image processing. Falls back to ImageMagick if unavailable.
+ * Uses Photon (Rust/WASM) for image processing. Falls back to ImageMagick if unavailable.
  *
  * Strategy for staying under maxBytes:
  * 1. First resize to maxWidth/maxHeight
@@ -95,18 +95,19 @@ function pickSmaller(
  */
 export async function resizeImage(img: ImageContent, options?: ImageResizeOptions): Promise<ResizedImage> {
 	const opts = { ...DEFAULT_OPTIONS, ...options };
-	const buffer = Buffer.from(img.data, "base64");
+	const inputBuffer = Buffer.from(img.data, "base64");
 
 	try {
-		const { Image } = await Vips();
-		const image = Image.newFromBuffer(buffer);
+		const photon = await getPhoton();
+		const image = photon.PhotonImage.new_from_byteslice(new Uint8Array(inputBuffer));
+
 		try {
-			const originalWidth = image.width;
-			const originalHeight = image.height;
+			const originalWidth = image.get_width();
+			const originalHeight = image.get_height();
 			const format = img.mimeType?.split("/")[1] ?? "png";
 
 			// Check if already within all limits (dimensions AND size)
-			const originalSize = buffer.length;
+			const originalSize = inputBuffer.length;
 			if (originalWidth <= opts.maxWidth && originalHeight <= opts.maxHeight && originalSize <= opts.maxBytes) {
 				return {
 					data: img.data,
@@ -138,18 +139,19 @@ export async function resizeImage(img: ImageContent, options?: ImageResizeOption
 				height: number,
 				jpegQuality: number,
 			): { buffer: Uint8Array; mimeType: string } {
-				const scale = Math.min(width / originalWidth, height / originalHeight);
-				const resized = image!.resize(scale);
-
-				const pngBuffer = resized.writeToBuffer(".png");
-				const jpegBuffer = resized.writeToBuffer(".jpg", { Q: jpegQuality });
+				const resized = photon.resize(image, width, height, photon.SamplingFilter.Lanczos3);
 
-				resized.delete();
+				try {
+					const pngBuffer = resized.get_bytes();
+					const jpegBuffer = resized.get_bytes_jpeg(jpegQuality);
 
-				return pickSmaller(
-					{ buffer: pngBuffer, mimeType: "image/png" },
-					{ buffer: jpegBuffer, mimeType: "image/jpeg" },
-				);
+					return pickSmaller(
+						{ buffer: pngBuffer, mimeType: "image/png" },
+						{ buffer: jpegBuffer, mimeType: "image/jpeg" },
+					);
+				} finally {
+					resized.free();
+				}
 			}
 
 			// Try to produce an image under maxBytes
@@ -229,10 +231,10 @@ export async function resizeImage(img: ImageContent, options?: ImageResizeOption
 				wasResized: true,
 			};
 		} finally {
-			image.delete();
+			image.free();
 		}
 	} catch (error) {
-		logger.error("Failed to resize image with wasm-vips", {
+		logger.error("Failed to resize image with Photon", {
 			error: error instanceof Error ? error.message : String(error),
 		});
 		return resizeImageWithImageMagick(img, opts);

package/src/utils/photon.ts

@@ -0,0 +1,25 @@
+import type { base64_to_image, PhotonImage, resize as photonResize, SamplingFilter } from "@silvia-odwyer/photon";
+
+let _photon: typeof import("@silvia-odwyer/photon") | undefined;
+let _initialized = false;
+
+/**
+ * Get the initialized Photon module.
+ * Lazily imports and initializes the WASM module on first use.
+ */
+export async function getPhoton(): Promise<typeof import("@silvia-odwyer/photon")> {
+	if (_photon && _initialized) return _photon;
+
+	const photon = await import("@silvia-odwyer/photon");
+
+	// Initialize the WASM module (default export is the init function)
+	if (!_initialized) {
+		await photon.default();
+		_initialized = true;
+	}
+
+	_photon = photon;
+	return _photon;
+}
+
+export type { PhotonImage, SamplingFilter, photonResize, base64_to_image };

package/src/utils/vips.ts

@@ -1,23 +0,0 @@
-import type realVips from "wasm-vips";
-import { logger } from "../core/logger";
-
-// Cached vips instance
-let _vips: Promise<typeof realVips> | undefined;
-
-/**
- * Get the vips instance.
- * @returns The vips instance.
- */
-export function Vips(): Promise<typeof realVips> {
-	if (_vips) return _vips;
-
-	let instance: Promise<typeof realVips> | undefined;
-	try {
-		instance = import("wasm-vips").then((mod) => (mod.default ?? mod)());
-	} catch (error) {
-		logger.error("Failed to import wasm-vips", { error: error instanceof Error ? error.message : String(error) });
-		instance = Promise.reject(error);
-	}
-	_vips = instance;
-	return instance;
-}