From c5bcf222f98d13b104231713bf4a0aa0833c420a Mon Sep 17 00:00:00 2001
From: nsfisis <nsfisis@gmail.com>
Date: Fri, 5 Jun 2026 00:50:47 +0900
Subject: feat(downloader): wire VcsDownloader subclasses via shared
 downloaders

Share downloaders as Rc<RefCell<dyn DownloaderInterface>> in
DownloadManager and make DownloaderInterface/VcsDownloader take
&mut self, matching PHP's mutable-by-reference downloader objects.
This lets Git/Svn/Hg/Fossil/Perforce implement VcsDownloader and route
download/install/update/prepare/cleanup through the trait instead of
todo!(), and wires the as_* downcast hooks end-to-end.

ChangeReportInterface::get_local_changes becomes &mut self since
FileDownloader downloads to a compare dir; get_commit_logs/reapply_changes
gain &mut self / Result to match the concrete implementations.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
---
 crates/shirabe/src/downloader/file_downloader.rs | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

(limited to 'crates/shirabe/src/downloader/file_downloader.rs')

diff --git a/crates/shirabe/src/downloader/file_downloader.rs b/crates/shirabe/src/downloader/file_downloader.rs
index 55882f7..e0fc0c9 100644
--- a/crates/shirabe/src/downloader/file_downloader.rs
+++ b/crates/shirabe/src/downloader/file_downloader.rs
@@ -153,13 +153,15 @@ impl DownloaderInterface for FileDownloader {
         "dist".to_owned()
     }
 
-    fn as_change_report_interface(&self) -> Option<&dyn crate::downloader::ChangeReportInterface> {
+    fn as_change_report_interface(
+        &mut self,
+    ) -> Option<&mut dyn crate::downloader::ChangeReportInterface> {
         Some(self)
     }
 
     /// @inheritDoc
     async fn download(
-        &self,
+        &mut self,
         package: PackageInterfaceHandle,
         path: &str,
         _prev_package: Option<PackageInterfaceHandle>,
@@ -437,7 +439,7 @@ impl DownloaderInterface for FileDownloader {
 
     /// @inheritDoc
     async fn prepare(
-        &self,
+        &mut self,
         _type: &str,
         _package: PackageInterfaceHandle,
         _path: &str,
@@ -448,7 +450,7 @@ impl DownloaderInterface for FileDownloader {
 
     /// @inheritDoc
     async fn cleanup(
-        &self,
+        &mut self,
         _type: &str,
         package: PackageInterfaceHandle,
         path: &str,
@@ -503,7 +505,7 @@ impl DownloaderInterface for FileDownloader {
 
     /// @inheritDoc
     async fn install(
-        &self,
+        &mut self,
         package: PackageInterfaceHandle,
         path: &str,
         output: bool,
@@ -565,7 +567,7 @@ impl DownloaderInterface for FileDownloader {
 
     /// @inheritDoc
     async fn update(
-        &self,
+        &mut self,
         initial: PackageInterfaceHandle,
         target: PackageInterfaceHandle,
         path: &str,
@@ -583,7 +585,7 @@ impl DownloaderInterface for FileDownloader {
 
     /// @inheritDoc
     async fn remove(
-        &self,
+        &mut self,
         package: PackageInterfaceHandle,
         path: &str,
         output: bool,
@@ -615,7 +617,7 @@ impl ChangeReportInterface for FileDownloader {
     /// @inheritDoc
     /// @throws \RuntimeException
     fn get_local_changes(
-        &self,
+        &mut self,
         package: PackageInterfaceHandle,
         path: &str,
     ) -> Result<Option<String>> {
-- 
cgit v1.3.1