From cd25c3e193f05a5e89bca2a1c706c85fdc9c9155 Mon Sep 17 00:00:00 2001
From: nsfisis <nsfisis@gmail.com>
Date: Sat, 6 Jun 2026 02:13:59 +0900
Subject: 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>
---
 crates/shirabe/src/repository/filesystem_repository.rs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'crates/shirabe/src/repository/filesystem_repository.rs')

diff --git a/crates/shirabe/src/repository/filesystem_repository.rs b/crates/shirabe/src/repository/filesystem_repository.rs
index db9a8c6..c373bd9 100644
--- a/crates/shirabe/src/repository/filesystem_repository.rs
+++ b/crates/shirabe/src/repository/filesystem_repository.rs
@@ -463,7 +463,7 @@ impl FilesystemRepository {
 
     /// @param array<string, string> $installPaths
     fn generate_installed_versions(
-        &self,
+        &mut self,
         installation_manager: &InstallationManager,
         install_paths: &IndexMap<String, Option<String>>,
         dev_mode: bool,
@@ -478,7 +478,7 @@ impl FilesystemRepository {
         ));
         let mut packages: Vec<PackageInterfaceHandle> = self
             .inner
-            .get_packages()
+            .get_packages()?
             .into_iter()
             .map(|p| p.into())
             .collect();
-- 
cgit v1.3.1