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

diff --git a/crates/shirabe/src/dependency_resolver/local_repo_transaction.rs b/crates/shirabe/src/dependency_resolver/local_repo_transaction.rs
index dc71562..5feb9eb 100644
--- a/crates/shirabe/src/dependency_resolver/local_repo_transaction.rs
+++ b/crates/shirabe/src/dependency_resolver/local_repo_transaction.rs
@@ -11,15 +11,15 @@ pub struct LocalRepoTransaction {
 
 impl LocalRepoTransaction {
     pub fn new(
-        locked_repository: &dyn RepositoryInterface,
-        local_repository: &dyn InstalledRepositoryInterface,
-    ) -> Self {
-        Self {
+        locked_repository: &mut dyn RepositoryInterface,
+        local_repository: &mut dyn InstalledRepositoryInterface,
+    ) -> anyhow::Result<Self> {
+        Ok(Self {
             inner: Transaction::new(
-                local_repository.get_packages(),
-                locked_repository.get_packages(),
+                local_repository.get_packages()?,
+                locked_repository.get_packages()?,
             ),
-        }
+        })
     }
 
     pub fn get_operations(