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

diff --git a/crates/shirabe/src/repository/writable_repository_interface.rs b/crates/shirabe/src/repository/writable_repository_interface.rs
index abbcbe7..ae62e1e 100644
--- a/crates/shirabe/src/repository/writable_repository_interface.rs
+++ b/crates/shirabe/src/repository/writable_repository_interface.rs
@@ -12,7 +12,7 @@ pub trait WritableRepositoryInterface: RepositoryInterface {
 
     fn remove_package(&mut self, package: PackageInterfaceHandle) -> Result<()>;
 
-    fn get_canonical_packages(&self) -> Vec<PackageInterfaceHandle>;
+    fn get_canonical_packages(&mut self) -> Result<Vec<PackageInterfaceHandle>>;
 
     fn reload(&mut self);