MTP: Instant cancel via USB SIC abort

- Bump `mtp-rs` from 0.9.1 to 0.11.0 for `FileDownload::cancel()` support
- On cancel, call `download.cancel(DEFAULT_CANCEL_TIMEOUT)` to cleanly abort the USB transfer (~300ms) instead of draining the entire stream
- Fixes regression where breaking mid-stream corrupted the USB session, making the device unresponsive until reconnect

Author: vdavid

Cargo.lock

@@ -109,7 +109,7 @@ mime_guess = "2"
 
 [target.'cfg(any(target_os = "macos", target_os = "linux"))'.dependencies]
 # MTP (Android device) support via pure Rust implementation
-mtp-rs = "0.9.1"
+mtp-rs = "0.11.0"
 # USB hotplug detection for MTP device watcher
 nusb = "0.2.3"
 bytes = "1"

apps/desktop/src-tauri/Cargo.toml

@@ -137,10 +137,8 @@ impl MtpConnectionManager {
                 message: format!("Failed to create local file: {}", e),
             })?;
 
-        // Write chunks to file (must complete before releasing device lock).
-        // MTP USB transfers must be fully consumed — breaking mid-stream corrupts
-        // the session and makes the device unresponsive for subsequent operations.
-        // On cancel: stop writing to disk, but keep draining the USB stream.
+        // Write chunks to file, checking for cancellation between chunks.
+        // On cancel: use mtp-rs's USB SIC cancel to abort the transfer cleanly (~300ms).
         let mut bytes_written = 0u64;
         let mut cancelled = false;
         while let Some(chunk_result) = download.next_chunk().await {
@@ -149,26 +147,29 @@ impl MtpConnectionManager {
                 message: format!("Download error: {}", e),
             })?;
 
-            if !cancelled {
-                file.write_all(&chunk).await.map_err(|e| MtpConnectionError::Other {
-                    device_id: device_id.to_string(),
-                    message: format!("Failed to write local file: {}", e),
-                })?;
-
-                bytes_written += chunk.len() as u64;
-
-                // Report progress and check for cancellation
-                if let Some(ref cb) = on_progress
-                    && cb(bytes_written, total_size).is_break()
-                {
-                    cancelled = true;
-                    // Don't break — keep draining the USB stream to keep the session healthy
-                }
+            file.write_all(&chunk).await.map_err(|e| MtpConnectionError::Other {
+                device_id: device_id.to_string(),
+                message: format!("Failed to write local file: {}", e),
+            })?;
+
+            bytes_written += chunk.len() as u64;
+
+            // Report progress and check for cancellation
+            if let Some(ref cb) = on_progress
+                && cb(bytes_written, total_size).is_break()
+            {
+                cancelled = true;
+                break;
             }
-            // else: cancelled, just drain remaining chunks without writing
         }
 
-        // Release device lock after download completes
+        // On cancellation, abort the USB transfer cleanly before releasing the lock
+        if cancelled {
+            let _ = download.cancel(mtp_rs::DEFAULT_CANCEL_TIMEOUT).await;
+        }
+
+        // Release device lock after download completes (or is cancelled)
+        drop(download);
         drop(storage);
         drop(device);