From 0b06f54103490e3ce5658e82bbc0119633e26cd8 Mon Sep 17 00:00:00 2001 From: nsfisis Date: Fri, 22 May 2026 01:29:48 +0900 Subject: refactor(composer): unify Composer/PartialComposer via Rc handles Model PHP's `Composer extends PartialComposer` as a PartialOrFullComposer enum and merge partial_composer.rs into composer.rs. Introduce ComposerHandle / PartialComposerHandle (plus their Weak variants) so the graph can be shared, and build it at once with Rc::new_cyclic in the factory to resolve the back-reference cycles. Co-Authored-By: Claude Opus 4.7 (1M context) --- crates/shirabe/src/package/locker.rs | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) (limited to 'crates/shirabe/src/package/locker.rs') diff --git a/crates/shirabe/src/package/locker.rs b/crates/shirabe/src/package/locker.rs index 5dd7ab4..ecd96e3 100644 --- a/crates/shirabe/src/package/locker.rs +++ b/crates/shirabe/src/package/locker.rs @@ -40,7 +40,7 @@ pub struct Locker { /// @var JsonFile lock_file: JsonFile, /// @var InstallationManager - installation_manager: InstallationManager, + installation_manager: std::rc::Rc>, /// @var string hash: String, /// @var string @@ -62,7 +62,7 @@ impl Locker { pub fn new( io: Box, lock_file: JsonFile, - installation_manager: InstallationManager, + installation_manager: std::rc::Rc>, composer_file_contents: &str, process: std::rc::Rc>, ) -> Self { @@ -824,7 +824,10 @@ impl Locker { return Ok(None); } - let path = self.installation_manager.get_install_path(package); + let path = self + .installation_manager + .borrow_mut() + .get_install_path(package); if path.is_none() { return Ok(None); } -- cgit v1.3.1