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, 3 insertions, 3 deletions

diff --git a/crates/shirabe/src/command/suggests_command.rs b/crates/shirabe/src/command/suggests_command.rs
index 2091608..92a4288 100644
--- a/crates/shirabe/src/command/suggests_command.rs
+++ b/crates/shirabe/src/command/suggests_command.rs
@@ -87,11 +87,11 @@ impl SuggestsCommand {
             );
         }
 
-        let installed_repo = InstalledRepository::new(installed_repos);
+        let mut installed_repo = InstalledRepository::new(installed_repos);
         let mut reporter = SuggestedPackagesReporter::new(self.get_io().clone());
 
         let filter = input.get_argument("packages");
-        let mut packages = RepositoryInterface::get_packages(&installed_repo);
+        let mut packages = RepositoryInterface::get_packages(&mut installed_repo)?;
         let root_pkg_as_base: crate::package::BasePackageHandle =
             composer.get_package().clone().into();
         packages.push(root_pkg_as_base);
@@ -121,7 +121,7 @@ impl SuggestsCommand {
                 None
             };
 
-        reporter.output(mode, Some(&installed_repo), only_dependents_of);
+        reporter.output(mode, Some(&mut installed_repo), only_dependents_of)?;
 
         Ok(0)
     }