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/require_command.rs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'crates/shirabe/src/command/require_command.rs') diff --git a/crates/shirabe/src/command/require_command.rs b/crates/shirabe/src/command/require_command.rs index d2923c8..4f50e71 100644 --- a/crates/shirabe/src/command/require_command.rs +++ b/crates/shirabe/src/command/require_command.rs @@ -278,7 +278,7 @@ impl Command for RequireCommand { } let composer = self.require_composer(None, None)?; - let composer = crate::command::composer_full(&composer); + let composer = crate::composer::composer_full(&composer); let repository_manager = composer.get_repository_manager().clone(); let repository_manager = repository_manager.borrow(); let repos = repository_manager.get_repositories(); @@ -753,7 +753,7 @@ impl RequireCommand { // Update packages self.reset_composer()?; let composer_handle = self.require_composer(None, None)?; - let composer = crate::command::composer_full_mut(&composer_handle); + let composer = crate::composer::composer_full(&composer_handle); self.dependency_resolution_completed.set(false); // PHP: $composer->getEventDispatcher()->addListener(InstallerEvents::PRE_OPERATIONS_EXEC, @@ -1056,7 +1056,7 @@ impl RequireCommand { fixed: bool, ) -> Result { let composer = self.require_composer(None, None)?; - let composer = crate::command::composer_full_mut(&composer); + let composer = crate::composer::composer_full(&composer); let locker_is_locked = composer.get_locker().borrow_mut().is_locked(); let mut requirements: IndexMap = IndexMap::new(); let mut version_selector = VersionSelector::new( -- cgit v1.3.1