From 6e0c8b45170eb295cfbba514a4e8aca00efb43aa Mon Sep 17 00:00:00 2001 From: Ahmed Ibrahim Date: Tue, 23 Jun 2026 00:20:23 -0700 Subject: [PATCH] Handle additional tools in image URL validation (#29577) ## Why `ResponseItem::AdditionalTools` was added without updating app-server image URL validation. The exhaustive match therefore prevents app-server and downstream targets from compiling on `main`. ## What changed Treat `AdditionalTools` like the other response items that cannot contain input-image URLs. --- codex-rs/app-server/src/request_processors/turn_processor.rs | 1 + 1 file changed, 1 insertion(+) diff --git a/codex-rs/app-server/src/request_processors/turn_processor.rs b/codex-rs/app-server/src/request_processors/turn_processor.rs index f7b4f7c14..4b6e0fda3 100644 --- a/codex-rs/app-server/src/request_processors/turn_processor.rs +++ b/codex-rs/app-server/src/request_processors/turn_processor.rs @@ -58,6 +58,7 @@ fn validate_response_item_image_urls(items: &[ResponseItem]) -> Result<(), JSONR | ResponseItem::Compaction { .. } | ResponseItem::CompactionTrigger { .. } | ResponseItem::ContextCompaction { .. } + | ResponseItem::AdditionalTools { .. } | ResponseItem::Other => false, }) { return Err(invalid_request(REMOTE_IMAGE_URL_ERROR));