diff options
| author | nsfisis <nsfisis@gmail.com> | 2026-06-06 18:04:45 +0900 |
|---|---|---|
| committer | nsfisis <nsfisis@gmail.com> | 2026-06-06 18:04:45 +0900 |
| commit | d5b313f388d424e60ae33ae58c1a65a65b24b8f1 (patch) | |
| tree | 849ada9b7319b8eb85aab28e02f614023fea2105 /crates/shirabe/src/command/base_dependency_command.rs | |
| parent | 71b432db3f115d17be498f36cd3efd40b4650a0b (diff) | |
| download | php-shirabe-d5b313f388d424e60ae33ae58c1a65a65b24b8f1.tar.gz php-shirabe-d5b313f388d424e60ae33ae58c1a65a65b24b8f1.tar.zst php-shirabe-d5b313f388d424e60ae33ae58c1a65a65b24b8f1.zip | |
refactor(command): share Input/OutputInterface via Rc<RefCell>
Convert InputInterface and OutputInterface parameters from &dyn/&mut dyn
references to Rc<RefCell<dyn ...>> shared ownership across the command,
console, and IO layers, matching the Phase C shared-ownership approach
already used for IOInterface.
Co-Authored-By: Claude Opus 4.8 (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.rs | 31 |
1 files changed, 22 insertions, 9 deletions
diff --git a/crates/shirabe/src/command/base_dependency_command.rs b/crates/shirabe/src/command/base_dependency_command.rs index 4f929c8..4a5b854 100644 --- a/crates/shirabe/src/command/base_dependency_command.rs +++ b/crates/shirabe/src/command/base_dependency_command.rs @@ -45,8 +45,8 @@ pub trait BaseDependencyCommand: BaseCommand { fn do_execute( &mut self, - input: &dyn InputInterface, - output: &dyn OutputInterface, + input: std::rc::Rc<std::cell::RefCell<dyn InputInterface>>, + output: std::rc::Rc<std::cell::RefCell<dyn OutputInterface>>, inverted: bool, ) -> anyhow::Result<i64> { let composer = self.require_composer(None, None)?; @@ -60,7 +60,12 @@ pub trait BaseDependencyCommand: BaseCommand { )), )]; - if input.get_option("locked").as_bool().unwrap_or(false) { + if input + .borrow() + .get_option("locked") + .as_bool() + .unwrap_or(false) + { let locker = composer.get_locker().clone(); let mut locker = locker.borrow_mut(); @@ -91,7 +96,7 @@ pub trait BaseDependencyCommand: BaseCommand { if local_repo.get_packages()?.len() == 0 && (root_pkg.get_requires().len() > 0 || root_pkg.get_dev_requires().len() > 0) { - output.writeln( + output.borrow().writeln( "<warning>No dependencies installed. Try running composer install or update, or use --locked.</warning>", shirabe_external_packages::symfony::console::output::OUTPUT_NORMAL, ); @@ -118,12 +123,14 @@ pub trait BaseDependencyCommand: BaseCommand { let mut installed_repo = InstalledRepository::new(repos); let needle = input + .borrow() .get_argument(Self::ARGUMENT_PACKAGE) .as_string() .unwrap_or_default() .to_string(); - let text_constraint: String = if input.has_argument(Self::ARGUMENT_CONSTRAINT) { + let text_constraint: String = if input.borrow().has_argument(Self::ARGUMENT_CONSTRAINT) { input + .borrow() .get_argument(Self::ARGUMENT_CONSTRAINT) .as_string() .unwrap_or("*") @@ -242,11 +249,13 @@ pub trait BaseDependencyCommand: BaseCommand { }; let render_tree = input + .borrow() .get_option(Self::OPTION_TREE) .as_bool() .unwrap_or(false); let recursive = render_tree || input + .borrow() .get_option(Self::OPTION_RECURSIVE) .as_bool() .unwrap_or(false); @@ -294,7 +303,7 @@ pub trait BaseDependencyCommand: BaseCommand { } if inverted - && input.has_argument(Self::ARGUMENT_CONSTRAINT) + && input.borrow().has_argument(Self::ARGUMENT_CONSTRAINT) && !PlatformRepository::is_platform_package(&needle) { let mut composer_command = "update"; @@ -322,7 +331,11 @@ pub trait BaseDependencyCommand: BaseCommand { Ok(r#return) } - fn print_table(&self, output: &dyn OutputInterface, results: Vec<DependentsEntry>) { + fn print_table( + &self, + output: std::rc::Rc<std::cell::RefCell<dyn OutputInterface>>, + results: Vec<DependentsEntry>, + ) { let mut table: Vec<Vec<String>> = vec![]; let mut doubles: IndexMap<String, bool> = IndexMap::new(); let mut results = results; @@ -383,7 +396,7 @@ pub trait BaseDependencyCommand: BaseCommand { self.render_table(table_as_mixed, output); } - fn init_styles(&mut self, output: &dyn OutputInterface) { + fn init_styles(&mut self, output: std::rc::Rc<std::cell::RefCell<dyn OutputInterface>>) { *self.colors_mut() = vec