aboutsummaryrefslogtreecommitdiffhomepage
path: root/crates/shirabe/src/command/base_dependency_command.rs
diff options
context:
space:
mode:
authornsfisis <nsfisis@gmail.com>2026-05-22 01:29:48 +0900
committernsfisis <nsfisis@gmail.com>2026-05-22 01:43:48 +0900
commit0b06f54103490e3ce5658e82bbc0119633e26cd8 (patch)
tree687b075131d3679725e77e0931ff7c503a6c3034 /crates/shirabe/src/command/base_dependency_command.rs
parent2914770fba6b3cc03a68fae493f60470a41962ec (diff)
downloadphp-shirabe-0b06f54103490e3ce5658e82bbc0119633e26cd8.tar.gz
php-shirabe-0b06f54103490e3ce5658e82bbc0119633e26cd8.tar.zst
php-shirabe-0b06f54103490e3ce5658e82bbc0119633e26cd8.zip
refactor(composer): unify Composer/PartialComposer via Rc handles
Model PHP's `Composer extends PartialComposer` as a PartialOrFullComposer enum and merge partial_composer.rs into composer.rs. Introduce ComposerHandle / PartialComposerHandle (plus their Weak variants) so the graph can be shared, and build it at once with Rc::new_cyclic in the factory to resolve the back-reference cycles. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Diffstat (limited to 'crates/shirabe/src/command/base_dependency_command.rs')
-rw-r--r--crates/shirabe/src/command/base_dependency_command.rs12
1 files changed, 8 insertions, 4 deletions
diff --git a/crates/shirabe/src/command/base_dependency_command.rs b/crates/shirabe/src/command/base_dependency_command.rs
index 7ef3b15..498c589 100644
--- a/crates/shirabe/src/command/base_dependency_command.rs
+++ b/crates/shirabe/src/command/base_dependency_command.rs
@@ -47,7 +47,8 @@ pub trait BaseDependencyCommand: BaseCommand {
output: &dyn OutputInterface,
inverted: bool,
) -> anyhow::Result<i64> {
- let mut composer = self.require_composer(None, None)?;
+ let composer = self.require_composer(None, None)?;
+ let mut composer = crate::command::composer_full_mut(&composer);
// TODO(plugin): dispatch CommandEvent(PluginEvents::COMMAND, self.get_name(), input, output) via composer.get_event_dispatcher()
let mut repos: Vec<Box<dyn RepositoryInterface>> = vec![];
@@ -56,7 +57,8 @@ pub trait BaseDependencyCommand: BaseCommand {
)));
if input.get_option("locked").as_bool().unwrap_or(false) {
- let locker = composer.get_locker_mut();
+ let locker = composer.get_locker().clone();
+ let mut locker = locker.borrow_mut();
if !locker.is_locked() {
return Err(anyhow::anyhow!(UnexpectedValueException {
@@ -78,7 +80,9 @@ pub trait BaseDependencyCommand: BaseCommand {
platform_overrides,
)?));
} else {
- let local_repo = composer.get_repository_manager().get_local_repository();
+ let repository_manager = composer.get_repository_manager().clone();
+ let repository_manager = repository_manager.borrow();
+ let local_repo = repository_manager.get_local_repository();
let root_pkg = composer.get_package();
if local_repo.get_packages().len() == 0
@@ -142,7 +146,7 @@ pub trait BaseDependencyCommand: BaseCommand {
let default_repos = CompositeRepository::new(
RepositoryFactory::default_repos(
Some(self.get_io()),
- Some(std::rc::Rc::clone(composer.get_config())),
+ Some(composer.get_config()),
// TODO(phase-b): get_repository_manager returns &; default_repos needs &mut
Some(todo!("share repository_manager as &mut")),
)?