diff options
| author | nsfisis <nsfisis@gmail.com> | 2026-06-04 22:02:44 +0900 |
|---|---|---|
| committer | nsfisis <nsfisis@gmail.com> | 2026-06-04 22:02:44 +0900 |
| commit | f429396f1cada5b7096d12c3cfaeea113018406c (patch) | |
| tree | e831bebe8d20fe02243a24d18ed72259fe7e6ff7 /crates/shirabe/src/repository | |
| parent | 813a1b30aab5034512fd4ae4bee0acb1ece414dd (diff) | |
| download | php-shirabe-f429396f1cada5b7096d12c3cfaeea113018406c.tar.gz php-shirabe-f429396f1cada5b7096d12c3cfaeea113018406c.tar.zst php-shirabe-f429396f1cada5b7096d12c3cfaeea113018406c.zip | |
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>
Diffstat (limited to 'crates/shirabe/src/repository')
| -rw-r--r-- | crates/shirabe/src/repository/handle.rs | 4 | ||||
| -rw-r--r-- | crates/shirabe/src/repository/lock_array_repository.rs | 12 | ||||
| -rw-r--r-- | crates/shirabe/src/repository/repository_interface.rs | 4 |
3 files changed, 13 insertions, 7 deletions
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!() - } } |
