aboutsummaryrefslogtreecommitdiffhomepage
path: root/crates/shirabe/src/repository/repository_set.rs
diff options
context:
space:
mode:
authornsfisis <nsfisis@gmail.com>2026-06-07 10:33:53 +0900
committernsfisis <nsfisis@gmail.com>2026-06-07 10:33:53 +0900
commit971824aa15334fd12d08ae0f441f6bf6079344c3 (patch)
treed59892042f478c65d980d96f62ef633644fd8fea /crates/shirabe/src/repository/repository_set.rs
parent0d45c403c20ac8950f2877a21f9cd4f0ca9bf784 (diff)
downloadphp-shirabe-971824aa15334fd12d08ae0f441f6bf6079344c3.tar.gz
php-shirabe-971824aa15334fd12d08ae0f441f6bf6079344c3.tar.zst
php-shirabe-971824aa15334fd12d08ae0f441f6bf6079344c3.zip
feat(shirabe): resolve phase-b TODOs with shared ownership
Replace TODO(phase-b) placeholders (todo!() and commented-out code) with real implementations: - Share JsonFile via Rc<RefCell<JsonFile>> so JsonConfigSource and the owning command can hold the same instance (base_config_command, config_command, repository_command, require_command, create_project, remove_command, factory) - Change InstallerInterface methods (is_installed, download, prepare, cleanup, get_install_path) to &mut self so initialize_vendor_dir can run, propagated to all installer implementations - Pass io/config/filesystem/process by clone instead of moving or stubbing (auth_helper, svn_driver, curl_downloader, library_installer) - Make TransportException Clone and store it by value in VcsRepository - Clone operations in Transaction sort, root_aliases/temporary_constraints in RepositorySet::create_pool, and share CompletePackage via handle in PlatformRepository - Wire up set_option, set_requires/set_dev_requires, installation manager setters, BumpCommand::set_composer, and clean_backups/set_local_phar
Diffstat (limited to 'crates/shirabe/src/repository/repository_set.rs')
-rw-r--r--crates/shirabe/src/repository/repository_set.rs33
1 files changed, 23 insertions, 10 deletions
diff --git a/crates/shirabe/src/repository/repository_set.rs b/crates/shirabe/src/repository/repository_set.rs
index 3cc4b9f..367a46a 100644
--- a/crates/shirabe/src/repository/repository_set.rs
+++ b/crates/shirabe/src/repository/repository_set.rs
@@ -459,7 +459,7 @@ impl RepositorySet {
/// @param list<string>|null $allowedTypes Only packages of those types are allowed if set to non-null
pub fn create_pool(
&mut self,
- request: Request,
+ mut request: Request,
io: std::rc::Rc<std::cell::RefCell<dyn IOInterface>>,
event_dispatcher: Option<std::rc::Rc<std::cell::RefCell<EventDispatcher>>>,
pool_optimizer: Option<PoolOptimizer>,
@@ -467,17 +467,34 @@ impl RepositorySet {
allowed_types: Option<Vec<String>>,
security_advisory_pool_filter: Option<SecurityAdvisoryPoolFilter>,
) -> Result<Pool> {
+ let root_aliases = self
+ .root_aliases
+ .iter()
+ .map(|(name, versions)| {
+ let versions = versions
+ .iter()
+ .map(|(version, entry)| {
+ let mut fields = IndexMap::new();
+ fields.insert("alias".to_string(), entry.alias.clone());
+ fields.insert(
+ "alias_normalized".to_string(),
+ entry.alias_normalized.clone(),
+ );
+ (version.clone(), fields)
+ })
+ .collect();
+ (name.clone(), versions)
+ })
+ .collect();
let mut pool_builder = PoolBuilder::new(
self.acceptable_stabilities.clone(),
self.stability_flags.clone(),
- // TODO(phase-b): clone root_aliases into PoolBuilder's expected type
- todo!("self.root_aliases.clone()"),
+ root_aliases,
self.root_references.clone(),
io,
event_dispatcher,
pool_optimizer,
- // TODO(phase-b): clone temporary_constraints
- todo!("self.temporary_constraints.clone()"),
+ self.temporary_constraints.clone(),
security_advisory_pool_filter,
);
pool_builder.set_ignored_types(ignored_types);
@@ -501,11 +518,7 @@ impl RepositorySet {
self.locked = true;
- // TODO(phase-b): pool_builder.build_pool takes owned Vec and &mut Request; revisit sharing model
- pool_builder.build_pool(
- todo!("share self.repositories"),
- todo!("share request as &mut"),
- )
+ pool_builder.build_pool(self.repositories.clone(), &mut request)
}
/// Create a pool for dependency resolution from the packages in this repository set.