From f429396f1cada5b7096d12c3cfaeea113018406c Mon Sep 17 00:00:00 2001
From: nsfisis <nsfisis@gmail.com>
Date: Thu, 4 Jun 2026 22:02:44 +0900
Subject: feat(locker): wire get_locked_repository via LockArrayRepository

Implement LockArrayRepository::new/add_package and
CanonicalPackagesTrait::get_packages (delegating to inner
ArrayRepository), expose add_package on LockArrayRepositoryHandle, and
add set_root_package_alias on CompleteAliasPackageHandle.

With these in place, Locker::get_locked_repository is fully wired:
load each locked package, register it (plus its aliasOf for
AliasPackages) in package_by_name, then build CompleteAliasPackage
handles for lock-file aliases via as_complete_package() narrowing.

Drop the unused RepositoryInterface::clone_box default and the stale
inherent LockArrayRepository::clone_box (no callers).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
---
 crates/shirabe/src/repository/handle.rs                |  4 ++++
 crates/shirabe/src/repository/lock_array_repository.rs | 12 +++++++++---
 crates/shirabe/src/repository/repository_interface.rs  |  4 ----
 3 files changed, 13 insertions(+), 7 deletions(-)

(limited to 'crates/shirabe/src/repository')

diff --git a/crates/shirabe/src/repository/handle.rs b/crates/shirabe/src/repository/handle.rs
index 38a4f28..73d4493 100644
--- a/crates/shirabe/src/repository/handle.rs
+++ b/crates/shirabe/src/repository/handle.rs
@@ -202,6 +202,10 @@ impl LockArrayRepositoryHandle {
         self.0.borrow_mut()
     }
 
+    pub fn add_package(&self, package: PackageInterfaceHandle) -> anyhow::Result<()> {
+        self.0.borrow().add_package(package)
+    }
+
     pub fn ptr_eq(&self, other: &Self) -> bool {
         Rc::ptr_eq(&self.0, &other.0)
     }
diff --git a/crates/shirabe/src/repository/lock_array_repository.rs b/crates/shirabe/src/repository/lock_array_repository.rs
index f9b64a7..08e888a 100644
--- a/crates/shirabe/src/repository/lock_array_repository.rs
+++ b/crates/shirabe/src/repository/lock_array_repository.rs
@@ -19,13 +19,19 @@ pub struct LockArrayRepository {
 
 impl CanonicalPackagesTrait for LockArrayRepository {
     fn get_packages(&self) -> Vec<PackageInterfaceHandle> {
-        todo!()
+        RepositoryInterface::get_packages(&self.inner)
     }
 }
 
 impl LockArrayRepository {
-    pub fn clone_box(&self) -> Box<dyn RepositoryInterface> {
-        todo!()
+    pub fn new(packages: Vec<PackageInterfaceHandle>) -> anyhow::Result<Self> {
+        Ok(Self {
+            inner: ArrayRepository::new(packages)?,
+        })
+    }
+
+    pub fn add_package(&self, package: PackageInterfaceHandle) -> anyhow::Result<()> {
+        self.inner.add_package(package)
     }
 }
 
diff --git a/crates/shirabe/src/repository/repository_interface.rs b/crates/shirabe/src/repository/repository_interface.rs
index 21914f6..e75f54e 100644
--- a/crates/shirabe/src/repository/repository_interface.rs
+++ b/crates/shirabe/src/repository/repository_interface.rs
@@ -109,8 +109,4 @@ pub trait RepositoryInterface: Countable + std::fmt::Debug {
     fn set_self_handle(&self, weak: crate::repository::RepositoryInterfaceWeakHandle) {
         let _ = weak;
     }
-
-    fn clone_box(&self) -> Box<dyn RepositoryInterface> {
-        todo!()
-    }
 }
-- 
cgit v1.3.1