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

diff --git a/crates/shirabe/src/command/audit_command.rs b/crates/shirabe/src/command/audit_command.rs
index 786843e..a28c344 100644
--- a/crates/shirabe/src/command/audit_command.rs
+++ b/crates/shirabe/src/command/audit_command.rs
@@ -156,11 +156,9 @@ impl AuditCommand {
                     code: 0,
                 }.into());
             }
-            return Ok(CanonicalPackagesTrait::get_packages(
-                &*locker
-                    .get_locked_repository(!input.get_option("no-dev").as_bool().unwrap_or(false))?
-                    .borrow(),
-            ));
+            let locked_repo = locker
+                .get_locked_repository(!input.get_option("no-dev").as_bool().unwrap_or(false))?;
+            return locked_repo.borrow_mut().get_canonical_packages();
         }
 
         let _root_pkg = composer.get_package();