From 6fa5a175860553d757b781cf97c94655fcb30e2f Mon Sep 17 00:00:00 2001 From: Marshall Bowers Date: Tue, 17 Dec 2024 11:09:10 -0500 Subject: [PATCH] assistant2: Only add context if it is non-empty (#22141) This PR fixes an issue where we were attaching empty message content even if there was no context. Release Notes: - N/A --- crates/assistant2/src/context.rs | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/crates/assistant2/src/context.rs b/crates/assistant2/src/context.rs index b83e3bad12cad8fc940627a9bb5f50139861bf03..7e335f41379dc11a345d02ffb8b2cff946f0b7d0 100644 --- a/crates/assistant2/src/context.rs +++ b/crates/assistant2/src/context.rs @@ -73,5 +73,7 @@ pub fn attach_context_to_message( context_text.push_str(&thread_context); } - message.content.push(MessageContent::Text(context_text)); + if !context_text.is_empty() { + message.content.push(MessageContent::Text(context_text)); + } }