fix: clippy needless_return and fmt compliance

Remove unneeded return in utils.rs (crates/tui/src/utils.rs:256) that was caught by clippy on the new commit. Also run cargo fmt to satisfy format checks.

crates/tui/src/tui/ui.rs

@@ -508,8 +508,7 @@ pub async fn run_tui(config: &Config, options: TuiOptions) -> Result<()> {
     // switch), so a brand-new session would otherwise leave it None and both
     // exec_shell shell_env hooks and ToolCallBefore gate would silently no-op.
     if app.runtime_services.hook_executor.is_none() {
-        app.runtime_services.hook_executor =
-            Some(std::sync::Arc::new(app.hooks.clone()));
+        app.runtime_services.hook_executor = Some(std::sync::Arc::new(app.hooks.clone()));
     }
     app.runtime_services = RuntimeToolServices {
         shell_manager: Some(shell_manager),
@@ -8883,4 +8882,4 @@ fn parse_semver(v: &str) -> Option<(u32, u32, u32)> {
 }
 
 #[cfg(test)]
-mod tests;
+mod tests;

crates/tui/src/utils.rs

@@ -253,7 +253,7 @@ fn browser_open_command(url: &str) -> Result<Command> {
     {
         let mut cmd = Command::new("cmd");
         cmd.args(["/C", "start", "", url]);
-        return Ok(cmd);
+        Ok(cmd)
     }
 
     #[cfg(not(any(target_os = "macos", target_os = "linux", target_os = "windows")))]