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/archive_command.rs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'crates/shirabe/src/command/archive_command.rs') diff --git a/crates/shirabe/src/command/archive_command.rs b/crates/shirabe/src/command/archive_command.rs index 3b27b4c..c2e9070 100644 --- a/crates/shirabe/src/command/archive_command.rs +++ b/crates/shirabe/src/command/archive_command.rs @@ -218,7 +218,7 @@ impl ArchiveCommand { ignore_filters: bool, composer: Option<&PartialComposerHandle>, ) -> Result { - let composer_guard = composer.map(crate::command::composer_full); + let composer_guard = composer.map(crate::composer::composer_full); let mut owned_archive_manager; let composer_archive_manager; let mut composer_archive_manager_ref; @@ -256,7 +256,7 @@ impl ArchiveCommand { } } else { let rc = self.require_composer(None, None)?; - let composer = crate::command::composer_full(&rc); + let composer = crate::composer::composer_full(&rc); composer.get_package().clone().into() }; @@ -299,7 +299,7 @@ impl ArchiveCommand { let repo; if let Some(composer) = self.try_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 local_repo = repository_manager.get_local_repository(); -- cgit v1.3.1