From 8766b9dbc981c827d49658b7bc79d860031c0493 Mon Sep 17 00:00:00 2001 From: nsfisis Date: Mon, 20 Jul 2026 05:58:29 +0900 Subject: fix(update-command): un-ignore test_interactive_tmp via assoc select choices PHP's UpdateCommand::getPackagesInteractively passes $autocompleterValues keyed by package name to $io->select, so the selection resolves to package names. The port passed only the keys as a list, making select resolve to a numeric index that the update then treated as an unknown package. The old ignore reason (non-interactive terminal error) no longer applied. Co-Authored-By: Claude Fable 5 --- crates/shirabe/src/command/update_command.rs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'crates/shirabe/src') diff --git a/crates/shirabe/src/command/update_command.rs b/crates/shirabe/src/command/update_command.rs index a20d288f..d6c20b45 100644 --- a/crates/shirabe/src/command/update_command.rs +++ b/crates/shirabe/src/command/update_command.rs @@ -705,10 +705,10 @@ impl UpdateCommand { let select_result = io.select( "Select packages: (Select more than one value separated by comma) ".to_string(), - PhpMixed::List( + PhpMixed::Array( autocompleter_values - .keys() - .map(|k| PhpMixed::String(k.clone())) + .iter() + .map(|(k, v)| (k.clone(), PhpMixed::String(v.clone()))) .collect(), ), PhpMixed::Bool(false), -- cgit v1.3.1