From 632ed8528c3566d67dda2804beb6d9e4e9511a72 Mon Sep 17 00:00:00 2001 From: nsfisis Date: Mon, 20 Jul 2026 07:09:41 +0900 Subject: fix(require-command): drop input borrows before determine_requirements The Ref temporaries created by input.borrow() inside the argument expressions of the determine_requirements call lived until the end of the whole call statement, so ConsoleIO::ask_question's borrow_mut() on the same shared input RefCell panicked with "RefCell already borrowed" when the command prompted for packages. Hoist the argument computations into locals so no borrow is held across the call, and un-ignore the run_require CLI test. Co-Authored-By: Claude Fable 5 --- crates/shirabe/src/lib.rs | 1 - 1 file changed, 1 deletion(-) (limited to 'crates/shirabe/src/lib.rs') diff --git a/crates/shirabe/src/lib.rs b/crates/shirabe/src/lib.rs index 2db47fcd..8d0a58bc 100644 --- a/crates/shirabe/src/lib.rs +++ b/crates/shirabe/src/lib.rs @@ -196,7 +196,6 @@ mod cli_tests { run_reinstall => "reinstall", run_remove => "remove", run_repository => "repository", - #[ignore = "currently panics"] run_require => "require", run_run_script => "run-script", run_search => "search", -- cgit v1.3.1