refactor(repository): make read methods fallible and take &mut self

Change RepositoryInterface and WritableRepositoryInterface read methods
(find_package, find_packages, get_packages, load_packages, search,
get_providers, get_canonical_packages) to take &mut self and return
anyhow::Result, so lazy-loading repositories such as ComposerRepository
can perform fallible I/O and mutate internal state on access. Update all
implementors and call sites to propagate the Result and pass mutable
references.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

1 files changed, 2 insertions, 2 deletions

diff --git a/crates/shirabe/src/repository/filesystem_repository.rs b/crates/shirabe/src/repository/filesystem_repository.rs
index db9a8c6..c373bd9 100644
--- a/crates/shirabe/src/repository/filesystem_repository.rs
+++ b/crates/shirabe/src/repository/filesystem_repository.rs
@@ -463,7 +463,7 @@ impl FilesystemRepository {
 
     /// @param array<string, string> $installPaths
     fn generate_installed_versions(
-        &self,
+        &mut self,
         installation_manager: &InstallationManager,
         install_paths: &IndexMap<String, Option<String>>,
         dev_mode: bool,
@@ -478,7 +478,7 @@ impl FilesystemRepository {
         ));
         let mut packages: Vec<PackageInterfaceHandle> = self
             .inner
-            .get_packages()
+            .get_packages()?
             .into_iter()
             .map(|p| p.into())
             .collect();