From 28777b0c77a75d5b5c42685f168c488cd1cd48a8 Mon Sep 17 00:00:00 2001
From: dave <futurechimp@users.noreply.github.com>
Date: Tue, 14 Apr 2026 09:52:51 +0000
Subject: [PATCH] fix: simplify boolean in validate_working_dir to satisfy
 clippy nonminimal_bool

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 server/src/http/mcp/shell_tools.rs | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/server/src/http/mcp/shell_tools.rs b/server/src/http/mcp/shell_tools.rs
index a1c01fa3..81e0d8b5 100644
--- a/server/src/http/mcp/shell_tools.rs
+++ b/server/src/http/mcp/shell_tools.rs
@@ -87,9 +87,10 @@ fn validate_working_dir(working_dir: &str, ctx: &AppContext) -> Result<PathBuf,
     let merge_workspace = project_root.join(".huskies").join("merge_workspace");
     let canonical_mw = merge_workspace.canonicalize().unwrap_or_default();
 
-    if !canonical_wd.starts_with(&canonical_wt)
-        && !(!canonical_mw.as_os_str().is_empty() && canonical_wd.starts_with(&canonical_mw))
-    {
+    let in_worktrees = canonical_wd.starts_with(&canonical_wt);
+    let in_merge_ws =
+        !canonical_mw.as_os_str().is_empty() && canonical_wd.starts_with(&canonical_mw);
+    if !in_worktrees && !in_merge_ws {
         return Err(format!(
             "working_dir must be inside .huskies/worktrees/ or .huskies/merge_workspace/. Got: {working_dir}"
         ));