From e583112899cbea7494ffdd73d7de380dd5f808c4 Mon Sep 17 00:00:00 2001
From: nsfisis <nsfisis@gmail.com>
Date: Wed, 10 Jun 2026 00:54:22 +0900
Subject: feat(phase-c): resolve exception-handling phase-b TODOs

* Catch specific exception types instead of broad/placeholder handling.
* Drop the shim Countable trait.
---
 crates/shirabe/src/repository/composite_repository.rs | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

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

diff --git a/crates/shirabe/src/repository/composite_repository.rs b/crates/shirabe/src/repository/composite_repository.rs
index 97fad44..dafedfc 100644
--- a/crates/shirabe/src/repository/composite_repository.rs
+++ b/crates/shirabe/src/repository/composite_repository.rs
@@ -62,13 +62,16 @@ impl CompositeRepository {
     }
 }
 
-impl shirabe_php_shim::Countable for CompositeRepository {
-    fn count(&self) -> i64 {
-        self.repositories.iter().map(|r| r.count()).sum()
+impl RepositoryInterface for CompositeRepository {
+    fn count(&self) -> anyhow::Result<usize> {
+        let mut total = 0;
+        for repository in &self.repositories {
+            total += repository.count()?;
+        }
+
+        Ok(total)
     }
-}
 
-impl RepositoryInterface for CompositeRepository {
     fn get_repo_name(&self) -> String {
         let names: Vec<String> = self
             .repositories
-- 
cgit v1.3.1