From 524c513c03b77bb71bdaa230a4134bf4325e04e3 Mon Sep 17 00:00:00 2001 From: Hunter Bown Date: Sun, 10 May 2026 10:15:09 -0500 Subject: [PATCH] chore: workspace clippy + missing-field fix-up for preflight MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Two small workspace-clippy gaps that snuck through the per-crate sweeps in this branch: * `crates/cli/src/lib.rs` — the OpenAI-provider passthrough test was building a `ResolvedRuntimeOptions` literal directly and missed the `yolo: Option` field that landed earlier on this branch in 665801bb8 (`fix(cli): forward --yolo to TUI binary`). Set to `None` to match the test's non-yolo intent. * `crates/tui/src/mcp.rs` — the new `reload_if_config_changed` swap test was using `iter().any(|n| *n == "new")`, which is rust-1.94 clippy's `manual_contains` lint. Switched to `names.contains(&"new")`. `cargo clippy --workspace --all-targets --all-features --locked -- -D warnings` is now green. Co-Authored-By: Claude Opus 4.7 (1M context) --- crates/cli/src/lib.rs | 1 + crates/tui/src/mcp.rs | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/crates/cli/src/lib.rs b/crates/cli/src/lib.rs index 973b30fe..58f07bff 100644 --- a/crates/cli/src/lib.rs +++ b/crates/cli/src/lib.rs @@ -2516,6 +2516,7 @@ mod tests { telemetry: false, approval_policy: None, sandbox_mode: None, + yolo: None, http_headers: std::collections::BTreeMap::new(), }; diff --git a/crates/tui/src/mcp.rs b/crates/tui/src/mcp.rs index 4c828dd2..e65973df 100644 --- a/crates/tui/src/mcp.rs +++ b/crates/tui/src/mcp.rs @@ -2781,7 +2781,7 @@ mod tests { assert!(reloaded, "content-changed config must trigger reload"); let names = pool.server_names(); assert!( - names.iter().any(|n| *n == "new"), + names.contains(&"new"), "expected new server in pool after reload, got {names:?}" ); }