add error handling for delete operation.

src/flipper_ble.rs

@@ -428,9 +428,10 @@ impl FlipperBle {
 
         if pb_response.1.command_status == flipper_pb::flipper::CommandStatus::OK.into() {
             Ok(())
+        } else if pb_response.1.command_status == flipper_pb::flipper::CommandStatus::ERROR_STORAGE_INVALID_NAME.into() {
+            Err("Invalid name specified!".into())
         } else {
-            info!("received response {:?}", pb_response.1.command_id);
-            Err("".into())
+            Err(format!("Flipper returned unexpected response: {:?}", pb_response).into())
         }
     }
 
@@ -456,7 +457,9 @@ impl FlipperBle {
         let pb_response = ProtobufCodec::parse_response(&response)?;
         debug!("response received: {:?}", pb_response);
 
-        // TODO: what is the response if you try to open a nonexistent file in a correctly named app?
+        // If you try to load a nonexistent file in an app, the app is
+        // the one that displays an error. No error is relayed back
+        // over RPC.
         if pb_response.1.command_status == flipper_pb::flipper::CommandStatus::OK.into() {
             Ok(())
         } else if pb_response.1.command_status == flipper_pb::flipper::CommandStatus::ERROR_INVALID_PARAMETERS.into() {

src/main.rs

@@ -122,7 +122,7 @@ async fn main() {
         },
 
         Commands::Launch { app, args } => {
-	    println!("running with args {:?}", args);
+	    //println!("running with args {:?}", args);
             match flipper.launch(app, args).await {
                 Ok(()) => {
                     info!("launched app successfully");
@@ -161,7 +161,7 @@ async fn main() {
                     info!("deleted file successfully");
                 },
                 Err(e) => {
-                    error!("failed to send file: {}", e);
+                    error!("failed to delete file: {}", e);
                 }
             };
 	},

src/protobuf_codec.rs

@@ -17,8 +17,8 @@ use crate::flipper_pb;
 //
 // This number also affects things like lag, and 350 is a good number
 // that seems to just work.
-pub const PROTOBUF_BLE_MTU_SIZE: usize = 350;
-
+//const PROTOBUF_BLE_MTU_SIZE: usize = 350;
+pub const PROTOBUF_BLE_MTU_SIZE: usize = 25;
 // number of file bytes to write per cycle
 const PROTOBUF_FILE_WRITE_CHUNK_SIZE: usize = 512;
 
@@ -344,7 +344,9 @@ impl ProtobufCodec {
     /// Returns a Vec<u8> of an encoded SetDatetimeRequest with the
     /// datetime arguments set to the fields in `datetime`. No need
     /// for chunking, this command is always the same size.
-    pub fn create_set_datetime_request_packet(&mut self, datetime: chrono::DateTime<chrono::FixedOffset>) -> Result<Vec<u8>, Box<dyn Error>> {
+    pub fn create_set_datetime_request_packet(
+        &mut self,
+        datetime: chrono::DateTime<chrono::FixedOffset>) -> Result<Vec<u8>, Box<dyn Error>> {
 
         // SetDatetimeRequest is a thin wrapper around
         // FuriHalRtcDateTime which itself is a thin wrapper around