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

diff --git a/crates/shirabe/src/command/search_command.rs b/crates/shirabe/src/command/search_command.rs
index 2c5d27d..593b927 100644
--- a/crates/shirabe/src/command/search_command.rs
+++ b/crates/shirabe/src/command/search_command.rs
@@ -96,7 +96,7 @@ impl SearchCommand {
         {
             all_repos.push(r.clone());
         }
-        let repos = CompositeRepository::new(all_repos);
+        let mut repos = CompositeRepository::new(all_repos);
 
         // TODO(plugin): dispatch CommandEvent for search command
         let command_event = CommandEvent::new(PluginEvents::COMMAND, "search", input, output);
@@ -136,7 +136,7 @@ impl SearchCommand {
             query = preg_quote(&query, None);
         }
 
-        let results = repos.search(query, mode, r#type);
+        let results = repos.search(query, mode, r#type)?;
 
         if results.len() > 0 && format == "text" {
             let width = self.get_terminal_width();