fix(cli): show valid approval policy values in --ask-for-approval help

Fixes bounty issue #1424

The --ask-for-approval flag now displays valid options (untrusted, on-failure,
on-request, never) in the help text using clap's value_parser attribute.
Invalid values are also rejected with a clear error message.

Also fixes formatting issues in the file.

Author: factorydroid

cortex-cli/src/main.rs

@@ -78,7 +78,7 @@ struct InteractiveArgs {
     sandbox_mode: Option<String>,
 
     /// Configure when the model requires human approval
-    #[arg(long = "ask-for-approval", short = 'a')]
+    #[arg(long = "ask-for-approval", short = 'a', value_parser = ["untrusted", "on-failure", "on-request", "never"])]
     approval_policy: Option<String>,
 
     /// Convenience alias for low-friction sandboxed automatic execution
@@ -860,13 +860,13 @@ async fn run_serve(serve_cli: ServeCommand) -> Result<()> {
     let result = cortex_app_server::run(config).await;
 
     // Stop mDNS advertising on shutdown
-    if let Some(ref mut mdns) = mdns_service {
-        if let Err(e) = mdns.stop_advertising().await {
-            eprintln!("Warning: Failed to stop mDNS advertising: {e}");
-        }
+    if let Some(ref mut mdns) = mdns_service
+        && let Err(e) = mdns.stop_advertising().await
+    {
+        eprintln!("Warning: Failed to stop mDNS advertising: {e}");
     }
 
-    result.map_err(Into::into)
+    result
 }
 
 async fn run_servers(servers_cli: ServersCommand) -> Result<()> {