From 0c46b56555212a6fe09abf2775af67b9701ef26b Mon Sep 17 00:00:00 2001 From: Edvard Date: Tue, 17 Feb 2026 16:11:15 -0500 Subject: [PATCH] fix(agent): satisfy clippy::if_not_else lint in tool history push Flip conditional to use positive check (is_empty) in the if-branch to resolve clippy::if_not_else error in CI strict delta lint gate. Co-Authored-By: Claude Opus 4.6 --- src/agent/loop_.rs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/agent/loop_.rs b/src/agent/loop_.rs index 0fefeaf..b9221d8 100644 --- a/src/agent/loop_.rs +++ b/src/agent/loop_.rs @@ -829,7 +829,9 @@ pub(crate) async fn run_tool_call_loop( // reconstruct proper OpenAI-format tool_calls and tool result messages. // Prompt mode: use XML-based text format as before. history.push(ChatMessage::assistant(assistant_history_content)); - if !native_tool_calls.is_empty() { + if native_tool_calls.is_empty() { + history.push(ChatMessage::user(format!("[Tool results]\n{tool_results}"))); + } else { for (native_call, result) in native_tool_calls.iter().zip(individual_results.iter()) { let tool_msg = serde_json::json!({ "tool_call_id": native_call.id, @@ -837,8 +839,6 @@ pub(crate) async fn run_tool_call_loop( }); history.push(ChatMessage::tool(tool_msg.to_string())); } - } else { - history.push(ChatMessage::user(format!("[Tool results]\n{tool_results}"))); } }