fix(downloader): restore late binding in getLocalChanges/update paths

PHP's FileDownloader::getLocalChanges and ::update call $this->download()
/ $this->install() / $this->remove() / $this->getInstallOperationAppendix(),
which late-bind to the concrete downloader class. The Rust port embeds the
parent as `inner`, so delegating these methods to FileDownloader pinned the
calls to FileDownloader's own implementations: `status` built the compare
tree without extracting the archive (flagging every file of dist-installed
packages as changed), and `update` re-installed the raw dist file instead
of extracting it.

Thread the concrete downloader in as `this: &dyn DownloaderInterface` via
shared helpers (base_get_local_changes / base_update) and pass `self` from
each delegating downloader.

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

1 files changed, 5 insertions, 2 deletions

diff --git a/crates/shirabe/src/downloader/gzip_downloader.rs b/crates/shirabe/src/downloader/gzip_downloader.rs
index 6b5f3872..70b9f7bb 100644
--- a/crates/shirabe/src/downloader/gzip_downloader.rs
+++ b/crates/shirabe/src/downloader/gzip_downloader.rs
@@ -146,7 +146,7 @@ impl ChangeReportInterface for GzipDownloader {
         package: PackageInterfaceHandle,
         path: &str,
     ) -> anyhow::Result<Option<String>> {
-        self.inner.get_local_changes(package, path)
+        self.inner.base_get_local_changes(self, package, path)
     }
 }
 
@@ -197,7 +197,10 @@ impl crate::downloader::DownloaderInterface for GzipDownloader {
         target: PackageInterfaceHandle,
         path: &str,
     ) -> anyhow::Result<Option<PhpMixed>> {
-        self.inner.update(initial, target, path).await
+        let appendix = self.get_install_operation_appendix(target.clone(), path);
+        self.inner
+            .base_update(self, appendix, initial, target, path)
+            .await
     }
 
     async fn remove(