fix(compile): fix more random compile errors

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

1 files changed, 37 insertions, 31 deletions

diff --git a/crates/shirabe/src/command/reinstall_command.rs b/crates/shirabe/src/command/reinstall_command.rs
index 7e703c4..4ffad79 100644
--- a/crates/shirabe/src/command/reinstall_command.rs
+++ b/crates/shirabe/src/command/reinstall_command.rs
@@ -4,12 +4,13 @@ use std::any::Any;
 
 use anyhow::Result;
 use shirabe_external_packages::composer::pcre::preg::Preg;
-use shirabe_external_packages::symfony::console::input::input_interface::InputInterface;
-use shirabe_external_packages::symfony::console::output::output_interface::OutputInterface;
+use shirabe_external_packages::symfony::component::console::input::input_interface::InputInterface;
+use shirabe_external_packages::symfony::component::console::output::output_interface::OutputInterface;
 use shirabe_php_shim::InvalidArgumentException;
 
 use crate::command::base_command::{BaseCommand, BaseCommandData, HasBaseCommandData};
 use crate::composer::Composer;
+use crate::console::input::InputDefinitionItem;
 use crate::console::input::input_argument::InputArgument;
 use crate::console::input::input_option::InputOption;
 use crate::dependency_resolver::operation::install_operation::InstallOperation;
@@ -35,20 +36,20 @@ impl ReinstallCommand {
         self
             .set_name("reinstall")
             .set_description("Uninstalls and reinstalls the given package names")
-            .set_definition(vec![
-                InputOption::new("prefer-source", None, Some(InputOption::VALUE_NONE), "Forces installation from package sources when possible, including VCS information.", None),
-                InputOption::new("prefer-dist", None, Some(InputOption::VALUE_NONE), "Forces installation from package dist (default behavior).", None),
-                InputOption::new("prefer-install", None, Some(InputOption::VALUE_REQUIRED), "Forces installation from package dist|source|auto (auto chooses source for dev versions, dist for the rest).", None),
-                InputOption::new("no-autoloader", None, Some(InputOption::VALUE_NONE), "Skips autoloader generation", None),
-                InputOption::new("no-progress", None, Some(InputOption::VALUE_NONE), "Do not output download progress.", None),
-                InputOption::new("optimize-autoloader", Some(shirabe_php_shim::PhpMixed::String("o".to_string())), Some(InputOption::VALUE_NONE), "Optimize autoloader during autoloader dump", None),
-                InputOption::new("classmap-authoritative", Some(shirabe_php_shim::PhpMixed::String("a".to_string())), Some(InputOption::VALUE_NONE), "Autoload classes from the classmap only. Implicitly enables `--optimize-autoloader`.", None),
-                InputOption::new("apcu-autoloader", None, Some(InputOption::VALUE_NONE), "Use APCu to cache found/not-found classes.", None),
-                InputOption::new("apcu-autoloader-prefix", None, Some(InputOption::VALUE_REQUIRED), "Use a custom prefix for the APCu autoloader cache. Implicitly enables --apcu-autoloader", None),
-                InputOption::new("ignore-platform-req", None, Some(InputOption::VALUE_REQUIRED | InputOption::VALUE_IS_ARRAY), "Ignore a specific platform requirement (php & ext- packages).", None),
-                InputOption::new("ignore-platform-reqs", None, Some(InputOption::VALUE_NONE), "Ignore all platform requirements (php & ext- packages).", None),
-                InputOption::new("type", None, Some(InputOption::VALUE_REQUIRED | InputOption::VALUE_IS_ARRAY), "Filter packages to reinstall by type(s)", None),
-                InputArgument::new("packages", Some(InputArgument::IS_ARRAY), "List of package names to reinstall, can include a wildcard (*) to match any substring.", None),
+            .set_definition(&[
+                InputOption::new("prefer-source", None, Some(InputOption::VALUE_NONE), "Forces installation from package sources when possible, including VCS information.", None).unwrap().into().unwrap().into(),
+                InputOption::new("prefer-dist", None, Some(InputOption::VALUE_NONE), "Forces installation from package dist (default behavior).", None).unwrap().into().unwrap().into(),
+                InputOption::new("prefer-install", None, Some(InputOption::VALUE_REQUIRED), "Forces installation from package dist|source|auto (auto chooses source for dev versions, dist for the rest).", None).unwrap().into().unwrap().into(),
+                InputOption::new("no-autoloader", None, Some(InputOption::VALUE_NONE), "Skips autoloader generation", None).unwrap().into().unwrap().into(),
+                InputOption::new("no-progress", None, Some(InputOption::VALUE_NONE), "Do not output download progress.", None).unwrap().into().unwrap().into(),
+                InputOption::new("optimize-autoloader", Some(shirabe_php_shim::PhpMixed::String("o".to_string())), Some(InputOption::VALUE_NONE), "Optimize autoloader during autoloader dump", None).unwrap().into().unwrap().into(),
+                InputOption::new("classmap-authoritative", Some(shirabe_php_shim::PhpMixed::String("a".to_string())), Some(InputOption::VALUE_NONE), "Autoload classes from the classmap only. Implicitly enables `--optimize-autoloader`.", None).unwrap().into().unwrap().into(),
+                InputOption::new("apcu-autoloader", None, Some(InputOption::VALUE_NONE), "Use APCu to cache found/not-found classes.", None).unwrap().into().unwrap().into(),
+                InputOption::new("apcu-autoloader-prefix", None, Some(InputOption::VALUE_REQUIRED), "Use a custom prefix for the APCu autoloader cache. Implicitly enables --apcu-autoloader", None).unwrap().into().unwrap().into(),
+                InputOption::new("ignore-platform-req", None, Some(InputOption::VALUE_REQUIRED | InputOption::VALUE_IS_ARRAY), "Ignore a specific platform requirement (php & ext- packages).", None).unwrap().into().unwrap().into(),
+                InputOption::new("ignore-platform-reqs", None, Some(InputOption::VALUE_NONE), "Ignore all platform requirements (php & ext- packages).", None).unwrap().into().unwrap().into(),
+                InputOption::new("type", None, Some(InputOption::VALUE_REQUIRED | InputOption::VALUE_IS_ARRAY), "Filter packages to reinstall by type(s)", None).unwrap().into().unwrap().into(),
+                InputArgument::new("packages", Some(InputArgument::IS_ARRAY), "List of package names to reinstall, can include a wildcard (*) to match any substring.", None).unwrap().into().unwrap().into(),
             ])
             .set_help(
                 "The <info>reinstall</info> command looks up installed packages by name,\n\
@@ -155,8 +156,10 @@ impl ReinstallCommand {
 
         let mut install_order = indexmap::IndexMap::new();
         for (index, op) in install_operations.iter().enumerate() {
-            if let Some(install_op) = (op.as_any() as &dyn Any).downcast_ref::<InstallOperation>() {
-                if (install_op.get_package().as_any() as &dyn Any)
+            if let Some(install_op) = op.as_any().downcast_ref::<InstallOperation>() {
+                if install_op
+                    .get_package()
+                    .as_any()
                     .downcast_ref::<AliasPackage>()
                     .is_none()
                 {
@@ -178,19 +181,13 @@ impl ReinstallCommand {
         });
 
         // TODO(plugin): dispatch CommandEvent
-        let command_event = CommandEvent::new(
-            PluginEvents::COMMAND.to_string(),
-            "reinstall".to_string(),
-            input,
-            output,
-            vec![],
-            vec![],
-        );
+        let command_event = CommandEvent::new(PluginEvents::COMMAND, "reinstall", input, output);
         let event_dispatcher = composer.get_event_dispatcher();
         event_dispatcher.dispatch(Some(command_event.get_name()), None);
 
-        let config = composer.get_config();
-        let (prefer_source, prefer_dist) = self.get_preferred_install_options(config, input)?;
+        let config = std::rc::Rc::clone(composer.get_config());
+        let (prefer_source, prefer_dist) =
+            self.get_preferred_install_options(&*config.borrow(), input)?;
 
         let installation_manager = composer.get_installation_manager();
         let download_manager = composer.get_download_manager();
@@ -225,12 +222,17 @@ impl ReinstallCommand {
                 .get_option("optimize-autoloader")
                 .as_bool()
                 .unwrap_or(false)
-                || config.get("optimize-autoloader").as_bool().unwrap_or(false);
+                || config
+                    .borrow_mut()
+                    .get("optimize-autoloader")
+                    .as_bool()
+                    .unwrap_or(false);
             let authoritative = input
                 .get_option("classmap-authoritative")
                 .as_bool()
                 .unwrap_or(false)
                 || config
+                    .borrow_mut()
                     .get("classmap-authoritative")
                     .as_bool()
                     .unwrap_or(false);
@@ -243,14 +245,18 @@ impl ReinstallCommand {
                     .get_option("apcu-autoloader")
                     .as_bool()
                     .unwrap_or(false)
-                || config.get("apcu-autoloader").as_bool().unwrap_or(false);
+                || config
+                    .borrow_mut()
+                    .get("apcu-autoloader")
+                    .as_bool()
+                    .unwrap_or(false);
 
             let generator = composer.get_autoload_generator();
             generator.set_class_map_authoritative(authoritative);
             generator.set_apcu(apcu, apcu_prefix.as_deref());
             generator.set_platform_requirement_filter(self.get_platform_requirement_filter(input)?);
             generator.dump(
-                config,
+                &*config.borrow(),
                 local_repo,
                 package,
                 installation_manager,