fix(curl-downloader): unlink partial file on redirect-without-location failure

PHP's handleRedirect() throws a bare TransportException when the
Location header is missing, and the caller's single catch block always
unlinks the `~` partial file via rejectJob(). The Rust decide() loop
splits each failure path into its own branch and had unlinked on every
other one, but missed this branch, leaking the partial file.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

1 files changed, 6 insertions, 1 deletions

diff --git a/crates/shirabe/src/util/http/curl_downloader.rs b/crates/shirabe/src/util/http/curl_downloader.rs
index c0b968bd..454836e7 100644
--- a/crates/shirabe/src/util/http/curl_downloader.rs
+++ b/crates/shirabe/src/util/http/curl_downloader.rs
@@ -403,7 +403,12 @@ impl CurlDownloader {
                     });
                 }
                 Ok(_) => {}
-                Err(e) => return Ok(Decision::Failed(e)),
+                Err(e) => {
+                    if let Some(filename) = filename {
+                        unlink_silent(&format!("{}~", filename));
+                    }
+                    return Ok(Decision::Failed(e));
+                }
             }
         }