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, 4 deletions

diff --git a/crates/shirabe/src/package/version/version_selector.rs b/crates/shirabe/src/package/version/version_selector.rs
index 31942f9..9e41f97 100644
--- a/crates/shirabe/src/package/version/version_selector.rs
+++ b/crates/shirabe/src/package/version/version_selector.rs
@@ -36,12 +36,11 @@ pub struct VersionSelector {
 impl VersionSelector {
     pub fn new(
         repository_set: RepositorySet,
-        platform_repo: Option<&crate::repository::PlatformRepository>,
+        platform_repo: Option<&mut crate::repository::PlatformRepository>,
     ) -> anyhow::Result<Self> {
         let mut platform_constraints: IndexMap<String, Vec<AnyConstraint>> = IndexMap::new();
         if let Some(platform_repo) = platform_repo {
-            for package in <PlatformRepository as RepositoryInterface>::get_packages(platform_repo)
-            {
+            for package in platform_repo.get_packages()? {
                 let constraint = SimpleConstraint::new(
                     "==".to_string(),
                     package.get_version().to_string(),
@@ -95,7 +94,7 @@ impl VersionSelector {
             &strtolower(package_name),
             constraint.as_ref().map(|c| c.clone()),
             repo_set_flags,
-        );
+        )?;
 
         let min_priority = *base_package::STABILITIES.get(preferred_stability).unwrap();
         candidates.sort_by(|a, b| {