From 60bfd667e4a98f5463a5bd5f2d0259bd0eeb0a5e Mon Sep 17 00:00:00 2001 From: nsfisis Date: Wed, 3 Jun 2026 23:06:16 +0900 Subject: feat(downloader): wire as_* downcasts for file/archive/path/git downloaders Override DownloaderInterface's as_change_report_interface / as_vcs_capable_downloader_interface / as_dvcs_downloader_interface so PHP-style instanceof checks resolve to the concrete sub-interface: FileDownloader and the six archive downloaders (Zip/Tar/Gzip/Xz/Rar/Phar) plus PathDownloader gain ChangeReportInterface (archives/path delegate to the inner FileDownloader), PathDownloader exposes VcsCapableDownloaderInterface, and GitDownloader exposes DvcsDownloaderInterface. The default None impls remain correct for downloaders that do not implement a given sub-interface, so their stale TODO markers are dropped. VCS downloaders' ChangeReport/VcsCapable conformance follows separately. Co-Authored-By: Claude Opus 4.8 --- crates/shirabe/src/downloader/phar_downloader.rs | 15 +++++++++++++++ 1 file changed, 15 insertions(+) (limited to 'crates/shirabe/src/downloader/phar_downloader.rs') diff --git a/crates/shirabe/src/downloader/phar_downloader.rs b/crates/shirabe/src/downloader/phar_downloader.rs index 6a4ac97..46fdd84 100644 --- a/crates/shirabe/src/downloader/phar_downloader.rs +++ b/crates/shirabe/src/downloader/phar_downloader.rs @@ -3,6 +3,7 @@ use crate::cache::Cache; use crate::config::Config; use crate::downloader::ArchiveDownloader; +use crate::downloader::ChangeReportInterface; use crate::downloader::DownloaderInterface; use crate::downloader::FileDownloader; use crate::event_dispatcher::EventDispatcher; @@ -63,12 +64,26 @@ impl PharDownloader { } } +impl ChangeReportInterface for PharDownloader { + fn get_local_changes( + &self, + package: PackageInterfaceHandle, + path: &str, + ) -> Result> { + self.inner.get_local_changes(package, path) + } +} + #[async_trait::async_trait(?Send)] impl DownloaderInterface for PharDownloader { fn get_installation_source(&self) -> String { self.inner.get_installation_source() } + fn as_change_report_interface(&self) -> Option<&dyn crate::downloader::ChangeReportInterface> { + Some(self) + } + async fn download( &self, package: PackageInterfaceHandle, -- cgit v1.3.1