Make CLI more resilient for platform protocol errors (#645)

crates/cli-tools/CHANGELOG.md

@@ -10,6 +10,7 @@
 
 ### Minor
 
+- Handle more errors during platform discovery and `action::PlatformReboot`
 - Extend `fs::write()` first parameter to set the `OpenOptions` too
 - Add `error::root_cause_is()` to check the `anyhow::Error` root cause
 - Add `action::PlatformLock` for locking a platform protocol

crates/cli-tools/src/action.rs

@@ -375,7 +375,10 @@ async fn final_call<S: service::Service>(
     match connection.receive::<S>().await {
         Ok(x) => proof(x)?,
         Err(e) => {
-            if root_cause_is::<rusb::Error>(&e, |x| matches!(x, rusb::Error::NoDevice)) {
+            if root_cause_is::<rusb::Error>(&e, |x| {
+                use rusb::Error::*;
+                matches!(x, NoDevice | Pipe)
+            }) {
                 return Ok(());
             }
             if root_cause_is::<std::io::Error>(&e, |x| {

crates/cli-tools/src/action/protocol.rs

@@ -61,8 +61,8 @@ impl ProtocolUsb {
         let context = GlobalContext::default();
         let mut matches = Vec::new();
         for candidate in wasefire_protocol_usb::list(&context)? {
-            let mut connection = candidate.connect(timeout)?;
-            let info = connection.call::<service::PlatformInfo>(()).await?;
+            let Ok(mut connection) = candidate.connect(timeout) else { continue };
+            let Ok(info) = connection.call::<service::PlatformInfo>(()).await else { continue };
             if !self.matches(connection.device(), &info) {
                 continue;
             }