aboutsummaryrefslogtreecommitdiffhomepage
path: root/crates/shirabe/src/command/suggests_command.rs
diff options
context:
space:
mode:
authornsfisis <nsfisis@gmail.com>2026-06-06 18:04:45 +0900
committernsfisis <nsfisis@gmail.com>2026-06-06 18:04:45 +0900
commitd5b313f388d424e60ae33ae58c1a65a65b24b8f1 (patch)
tree849ada9b7319b8eb85aab28e02f614023fea2105 /crates/shirabe/src/command/suggests_command.rs
parent71b432db3f115d17be498f36cd3efd40b4650a0b (diff)
downloadphp-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/suggests_command.rs')
-rw-r--r--crates/shirabe/src/command/suggests_command.rs35
1 files changed, 24 insertions, 11 deletions
diff --git a/crates/shirabe/src/command/suggests_command.rs b/crates/shirabe/src/command/suggests_command.rs
index 92a4288..d050ec7 100644
--- a/crates/shirabe/src/command/suggests_command.rs
+++ b/crates/shirabe/src/command/suggests_command.rs
@@ -42,8 +42,8 @@ impl SuggestsCommand {
pub fn 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>>,
) -> Result<i64> {
let composer = self.require_composer(None, None)?;
let mut composer = crate::command::composer_full_mut(&composer);
@@ -65,10 +65,13 @@ impl SuggestsCommand {
vec![],
platform_overrides,
)?));
- let locked_repo = composer
- .get_locker()
- .borrow_mut()
- .get_locked_repository(!input.get_option("no-dev").as_bool().unwrap_or(false))?;
+ let locked_repo = composer.get_locker().borrow_mut().get_locked_repository(
+ !input
+ .borrow()
+ .get_option("no-dev")
+ .as_bool()
+ .unwrap_or(false),
+ )?;
installed_repos.push(locked_repo.into());
} else {
// TODO(phase-b): Config::get returns PhpMixed; need to coerce to IndexMap<String, PhpMixed>
@@ -90,7 +93,7 @@ impl SuggestsCommand {
let mut installed_repo = InstalledRepository::new(installed_repos);
let mut reporter = SuggestedPackagesReporter::new(self.get_io().clone());
- let filter = input.get_argument("packages");
+ let filter = input.borrow().get_argument("packages");
let mut packages = RepositoryInterface::get_packages(&mut installed_repo)?;
let root_pkg_as_base: crate::package::BasePackageHandle =
composer.get_package().clone().into();
@@ -104,18 +107,28 @@ impl SuggestsCommand {
let mut mode = SuggestedPackagesReporter::MODE_BY_PACKAGE;
- if input.get_option("by-suggestion").as_bool().unwrap_or(false) {
+ if input
+ .borrow()
+ .get_option("by-suggestion")
+ .as_bool()
+ .unwrap_or(false)
+ {
mode = SuggestedPackagesReporter::MODE_BY_SUGGESTION;
}
- if input.get_option("by-package").as_bool().unwrap_or(false) {
+ if input
+ .borrow()
+ .get_option("by-package")
+ .as_bool()
+ .unwrap_or(false)
+ {
mode |= SuggestedPackagesReporter::MODE_BY_PACKAGE;
}
- if input.get_option("list").as_bool().unwrap_or(false) {
+ if input.borrow().get_option("list").as_bool().unwrap_or(false) {
mode = SuggestedPackagesReporter::MODE_LIST;
}
let only_dependents_of: Option<crate::package::PackageInterfaceHandle> =
- if empty(&filter) && !input.get_option("all").as_bool().unwrap_or(false) {
+ if empty(&filter) && !input.borrow().get_option("all").as_bool().unwrap_or(false) {
Some(composer.get_package().clone().into())
} else {
None