fix(agents): resolve clippy collapsible-if and expect-used lints

Collapse nested if blocks into let-chains (collapsible_if) in runner.rs
and tool_parsing.rs. Allow expect_used in tool_grammar test module where
panicking on None is the desired test behavior.

Author: penso

crates/agents/src/runner.rs

@@ -756,19 +756,19 @@ pub async fn run_agent_loop_with_context(
 
         // Fallback: parse tool calls from model text if the provider returned
         // no structured tool calls (some providers/models emit text-based calls).
-        if response.tool_calls.is_empty() {
-            if let Some(ref text) = response.text {
-                let (parsed, remaining) = parse_tool_calls_from_text(text);
-                if !parsed.is_empty() {
-                    info!(
-                        native_tools,
-                        count = parsed.len(),
-                        first_tool = %parsed[0].name,
-                        "parsed tool call(s) from text fallback"
-                    );
-                    response.text = remaining;
-                    response.tool_calls = parsed;
-                }
+        if response.tool_calls.is_empty()
+            && let Some(ref text) = response.text
+        {
+            let (parsed, remaining) = parse_tool_calls_from_text(text);
+            if !parsed.is_empty() {
+                info!(
+                    native_tools,
+                    count = parsed.len(),
+                    first_tool = %parsed[0].name,
+                    "parsed tool call(s) from text fallback"
+                );
+                response.text = remaining;
+                response.tool_calls = parsed;
             }
         }
 

crates/agents/src/tool_parsing.rs

@@ -59,10 +59,10 @@ pub fn parse_tool_calls_from_text(text: &str) -> (Vec<ToolCall>, Option<String>)
     blocks.sort_by_key(|b| b.start);
     let mut merged: Vec<ParsedBlock> = Vec::with_capacity(blocks.len());
     for block in blocks {
-        if let Some(last) = merged.last() {
-            if block.start < last.end {
-                continue; // overlapping — skip
-            }
+        if let Some(last) = merged.last()
+            && block.start < last.end
+        {
+            continue; // overlapping — skip
         }
         merged.push(block);
     }

crates/providers/src/local_gguf/tool_grammar.rs

@@ -73,6 +73,7 @@ ws ::= [ \t\n]*
 }
 
 #[cfg(test)]
+#[allow(clippy::expect_used)]
 mod tests {
     use super::*;