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/repository/package_repository.rs b/crates/shirabe/src/repository/package_repository.rs
index b5280a8..83e13d0 100644
--- a/crates/shirabe/src/repository/package_repository.rs
+++ b/crates/shirabe/src/repository/package_repository.rs
@@ -82,12 +82,12 @@ impl PackageRepository {
 }
 
 impl AdvisoryProviderInterface for PackageRepository {
-    fn has_security_advisories(&self) -> bool {
-        !self.security_advisories.is_empty()
+    fn has_security_advisories(&mut self) -> anyhow::Result<bool> {
+        Ok(!self.security_advisories.is_empty())
     }
 
     fn get_security_advisories(
-        &self,
+        &mut self,
         package_constraint_map: IndexMap<String, AnyConstraint>,
         allow_partial_advisories: bool,
     ) -> anyhow::Result<SecurityAdvisoryResult> {