fix: prevent tab_switch from printing "Browser closed"

The condition `data.get("tabId").is_some()` matched any response
containing tabId, including tab_switch. Now only checks for
`data.get("closed").is_some()` and adds a dedicated tab_switch
output handler showing "Switched to tab [id] (url)".

Author: DJRHails

cli/src/commands.rs

@@ -2862,7 +2862,7 @@ mod tests {
     #[test]
     fn test_network_har_requires_subcommand() {
         let result = parse_command(&args("network har"), &default_flags());
-        assert!(matches!(result, Err(ParseError::MissingArguments { .. }));
+        assert!(matches!(result, Err(ParseError::MissingArguments { .. })));
     }
 
     #[test]

cli/src/output.rs

@@ -401,6 +401,26 @@ pub fn print_response_with_opts(resp: &Response, action: Option<&str>, opts: &Ou
             }
             return;
         }
+        // Tab switch
+        if action == Some("tab_switch") {
+            if let Some(tab_id) = data.get("tabId").and_then(|v| v.as_i64()) {
+                if let Some(url) = data.get("url").and_then(|v| v.as_str()) {
+                    println!(
+                        "{} Switched to tab [{}] ({})",
+                        color::success_indicator(),
+                        tab_id,
+                        url
+                    );
+                } else {
+                    println!(
+                        "{} Switched to tab [{}]",
+                        color::success_indicator(),
+                        tab_id
+                    );
+                }
+                return;
+            }
+        }
         // New tab/window
         if let Some(tab_id) = data.get("tabId").and_then(|v| v.as_i64()) {
             if let Some(total) = data.get("total").and_then(|v| v.as_i64()) {
@@ -556,7 +576,7 @@ pub fn print_response_with_opts(resp: &Response, action: Option<&str>, opts: &Ou
             return;
         }
         // Closed (browser or tab)
-        if data.get("closed").is_some() || data.get("tabId").is_some() {
+        if data.get("closed").is_some() {
             let label = match action {
                 Some("tab_close") => {
                     if let Some(closed_id) = data.get("tabId").and_then(|v| v.as_i64()) {