aboutsummaryrefslogtreecommitdiffhomepage
path: root/crates/shirabe
diff options
context:
space:
mode:
authornsfisis <nsfisis@gmail.com>2026-06-22 03:56:30 +0900
committernsfisis <nsfisis@gmail.com>2026-06-22 03:56:30 +0900
commitc3bf6c567ba2eac2830fa6eddcf4fadf619eb17b (patch)
tree753a13fc244b1b9fe7280176927d056c530922bc /crates/shirabe
parent1628e4b1e03c00a97fd4804d76790fa3972153aa (diff)
downloadphp-shirabe-c3bf6c567ba2eac2830fa6eddcf4fadf619eb17b.tar.gz
php-shirabe-c3bf6c567ba2eac2830fa6eddcf4fadf619eb17b.tar.zst
php-shirabe-c3bf6c567ba2eac2830fa6eddcf4fadf619eb17b.zip
refactor(console): drop unported LazyCommand
Composer never uses Symfony's command loader, so LazyCommand is never instantiated on any Composer execution path. Remove the stub port and the instanceof branches that guarded against it.
Diffstat (limited to 'crates/shirabe')
-rw-r--r--crates/shirabe/src/console/application.rs13
1 files changed, 3 insertions, 10 deletions
diff --git a/crates/shirabe/src/console/application.rs b/crates/shirabe/src/console/application.rs
index 01515a1..d0e82ae 100644
--- a/crates/shirabe/src/console/application.rs
+++ b/crates/shirabe/src/console/application.rs
@@ -9,7 +9,6 @@ use shirabe_external_packages::seld::json_lint::ParsingException;
use shirabe_external_packages::symfony::console::application::Application as BaseApplication;
use shirabe_external_packages::symfony::console::command::Command as SymfonyCommand;
use shirabe_external_packages::symfony::console::command::help_command::HelpCommand;
-use shirabe_external_packages::symfony::console::command::lazy_command::LazyCommand;
use shirabe_external_packages::symfony::console::command::signalable_command_interface::SignalableCommandInterface;
use shirabe_external_packages::symfony::console::command_loader::command_loader_interface::CommandLoaderInterface;
use shirabe_external_packages::symfony::console::completion::completion_input::CompletionInput;
@@ -291,7 +290,7 @@ impl Application {
return Ok(None);
}
- // if (!$command instanceof LazyCommand) { $command->getDefinition(); }
+ // LazyCommand is intentionally not ported.
command.borrow().get_definition();
if command.borrow().get_name().is_none() {
@@ -1669,11 +1668,7 @@ impl Application {
}
}
- // if ($command instanceof LazyCommand) $command = $command->getCommand();
- // TODO(review): LazyCommand is a distinct type from SymfonyCommand here; PHP unwraps the real
- // command. The `commands` map stores Rc<RefCell<dyn SymfonyCommand>>, so the LazyCommand-unwrap path
- // needs a design decision about how lazy commands are represented.
- let _ = std::marker::PhantomData::<LazyCommand>;
+ // LazyCommand is intentionally not ported.
application.borrow_mut().running_command = Some(command.clone());
// do_run_command invokes the command's run(), which calls back into the application
@@ -1853,9 +1848,7 @@ impl Application {
return Ok(None);
}
- // if (!$command instanceof LazyCommand) { $command->getDefinition(); }
- // TODO(review): LazyCommand vs SymfonyCommand type distinction; eager definition probe omitted
- // pending lazy-command representation decision.
+ // LazyCommand is intentionally not ported.
command.borrow().get_definition();
if command.borrow().get_name().is_none() {