From 6d53dd16759df8b2cde09047ddcf460652034fd9 Mon Sep 17 00:00:00 2001 From: nsfisis Date: Sat, 27 Jun 2026 04:52:00 +0900 Subject: refactor(command): replace composer_full_mut with shared composer_full The compiler confirms none of the call sites invoke a &mut self method on the returned Composer, so the exclusive RefMut borrow was never needed and only risked borrow check conflict. Switch every site to the shared composer_full borrow and drop the now-dead composer_full_mut helper. Co-Authored-By: Claude Opus 4.8 (1M context) --- crates/shirabe/src/command/update_command.rs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'crates/shirabe/src/command/update_command.rs') diff --git a/crates/shirabe/src/command/update_command.rs b/crates/shirabe/src/command/update_command.rs index 6a74371..d22fbec 100644 --- a/crates/shirabe/src/command/update_command.rs +++ b/crates/shirabe/src/command/update_command.rs @@ -117,7 +117,7 @@ impl Command for UpdateCommand { } let composer_handle = self.require_composer(None, None)?; - let composer = crate::command::composer_full(&composer_handle); + let composer = crate::composer::composer_full(&composer_handle); if !HttpDownloader::is_curl_enabled() { io.write_error3( @@ -578,7 +578,7 @@ impl UpdateCommand { .into()); } - let composer_ref = crate::command::composer_full(composer); + let composer_ref = crate::composer::composer_full(composer); let platform_req_filter = self.get_platform_requirement_filter(input); let stability_flags = composer_ref.get_package().get_stability_flags(); let requires = array_merge_map( @@ -715,7 +715,7 @@ impl UpdateCommand { } fn create_version_selector(&self, composer: &PartialComposerHandle) -> Result { - let composer = crate::command::composer_full(composer); + let composer = crate::composer::composer_full(composer); let root_aliases: Vec = composer .get_package() .get_aliases() -- cgit v1.3.1