From 20d665bb1247500d514f05b160d01f5f8e223a38 Mon Sep 17 00:00:00 2001 From: nsfisis Date: Thu, 4 Jun 2026 22:37:18 +0900 Subject: feat(loader): wire RootPackageLoader::load end-to-end Resolve the remaining todo!()/placeholder sites in RootPackageLoader::load so the root package loads with real data: - Unbox config at use sites and change load()'s parameter from IndexMap> to IndexMap, matching ArrayLoader::load / VersionGuesser::guess_version and dropping the redundant box/unbox round-trip at the factory call site. - Expose replace_version through CompletePackage/RootPackage inherent delegation and a RootPackageHandle method (PHP Package::replaceVersion, inherited), and apply it for the auto-versioned default. - Collect require/require-dev links via getter dispatch and build the pretty-string map feeding extractAliases/StabilityFlags/References. Also reconcile the package repositories type with Config: change CompletePackageInterface::{get,set}_repositories from Vec> to IndexMap (PHP's array), wire setRepositories(config.getRepositories()), and dump repositories verbatim as a keyed array (matching PHP ArrayDumper) instead of forcing a list. Co-Authored-By: Claude Opus 4.8 --- crates/shirabe/src/package/root_package.rs | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'crates/shirabe/src/package/root_package.rs') diff --git a/crates/shirabe/src/package/root_package.rs b/crates/shirabe/src/package/root_package.rs index 0fd50f1..fc04d25 100644 --- a/crates/shirabe/src/package/root_package.rs +++ b/crates/shirabe/src/package/root_package.rs @@ -39,6 +39,10 @@ impl RootPackage { aliases: Vec::new(), } } + + pub fn replace_version(&mut self, version: String, pretty_version: String) { + self.inner.replace_version(version, pretty_version); + } } impl RootPackageInterface for RootPackage { @@ -136,11 +140,11 @@ impl CompletePackageInterface for RootPackage { CompletePackageInterface::set_scripts(&mut self.inner, scripts) } - fn get_repositories(&self) -> Vec> { + fn get_repositories(&self) -> IndexMap { CompletePackageInterface::get_repositories(&self.inner) } - fn set_repositories(&mut self, repositories: Vec>) { + fn set_repositories(&mut self, repositories: IndexMap) { CompletePackageInterface::set_repositories(&mut self.inner, repositories) } -- cgit v1.3.1