fix(cortex-cli): add confirmation prompt to logout command

Fixes bounty issue #1393

The logout command now prompts for confirmation before removing credentials.
Users can bypass the prompt using the --yes/-y flag for scripted usage.

Author: factorydroid

cortex-cli/src/login.rs

@@ -96,12 +96,28 @@ pub async fn run_login_status(_config_overrides: CliConfigOverrides) -> ! {
 }
 
 /// Run logout.
-pub async fn run_logout(_config_overrides: CliConfigOverrides) -> ! {
+pub async fn run_logout(_config_overrides: CliConfigOverrides, skip_confirmation: bool) -> ! {
     let Some(cortex_home) = get_cortex_home() else {
         eprintln!("Error: Could not determine home directory");
         std::process::exit(1);
     };
 
+    // Prompt for confirmation unless --yes flag is provided
+    if !skip_confirmation {
+        eprintln!(
+            "Are you sure you want to log out? This will remove your stored credentials. [y/N]"
+        );
+        let mut input = String::new();
+        if std::io::stdin().read_line(&mut input).is_err() {
+            eprintln!("Error reading input");
+            std::process::exit(1);
+        }
+        if !input.trim().eq_ignore_ascii_case("y") && !input.trim().eq_ignore_ascii_case("yes") {
+            eprintln!("Logout cancelled.");
+            std::process::exit(0);
+        }
+    }
+
     match logout(&cortex_home, CredentialsStoreMode::default()) {
         Ok(true) => {
             eprintln!("Successfully logged out");

cortex-cli/src/main.rs

@@ -269,6 +269,10 @@ enum LoginSubcommand {
 struct LogoutCommand {
     #[clap(skip)]
     config_overrides: CliConfigOverrides,
+
+    /// Skip confirmation prompt
+    #[arg(long, short = 'y')]
+    yes: bool,
 }
 
 /// Completion command.
@@ -453,7 +457,7 @@ async fn main() -> Result<()> {
             Ok(())
         }
         Some(Commands::Logout(logout_cli)) => {
-            run_logout(logout_cli.config_overrides).await;
+            run_logout(logout_cli.config_overrides, logout_cli.yes).await;
             Ok(())
         }
         Some(Commands::Mcp(mcp_cli)) => mcp_cli.run().await,