aboutsummaryrefslogtreecommitdiffhomepage
path: root/crates/shirabe/src/package/loader
diff options
context:
space:
mode:
authornsfisis <nsfisis@gmail.com>2026-05-22 01:29:48 +0900
committernsfisis <nsfisis@gmail.com>2026-05-22 01:43:48 +0900
commit0b06f54103490e3ce5658e82bbc0119633e26cd8 (patch)
tree687b075131d3679725e77e0931ff7c503a6c3034 /crates/shirabe/src/package/loader
parent2914770fba6b3cc03a68fae493f60470a41962ec (diff)
downloadphp-shirabe-0b06f54103490e3ce5658e82bbc0119633e26cd8.tar.gz
php-shirabe-0b06f54103490e3ce5658e82bbc0119633e26cd8.tar.zst
php-shirabe-0b06f54103490e3ce5658e82bbc0119633e26cd8.zip
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) <noreply@anthropic.com>
Diffstat (limited to 'crates/shirabe/src/package/loader')
-rw-r--r--crates/shirabe/src/package/loader/root_package_loader.rs8
1 files changed, 4 insertions, 4 deletions
diff --git a/crates/shirabe/src/package/loader/root_package_loader.rs b/crates/shirabe/src/package/loader/root_package_loader.rs
index 231c807..5d66931 100644
--- a/crates/shirabe/src/package/loader/root_package_loader.rs
+++ b/crates/shirabe/src/package/loader/root_package_loader.rs
@@ -27,7 +27,7 @@ use crate::util::ProcessExecutor;
#[derive(Debug)]
pub struct RootPackageLoader {
inner: ArrayLoader,
- manager: RepositoryManager,
+ manager: std::rc::Rc<std::cell::RefCell<RepositoryManager>>,
config: std::rc::Rc<std::cell::RefCell<Config>>,
version_guesser: VersionGuesser,
io: Option<Box<dyn IOInterface>>,
@@ -35,7 +35,7 @@ pub struct RootPackageLoader {
impl RootPackageLoader {
pub fn new(
- manager: RepositoryManager,
+ manager: std::rc::Rc<std::cell::RefCell<RepositoryManager>>,
config: std::rc::Rc<std::cell::RefCell<Config>>,
parser: Option<VersionParser>,
version_guesser: Option<VersionGuesser>,
@@ -281,10 +281,10 @@ impl RootPackageLoader {
let repos = RepositoryFactory::default_repos(
None,
Some(std::rc::Rc::clone(&self.config)),
- Some(&mut self.manager),
+ Some(&mut *self.manager.borrow_mut()),
)?;
for (_, repo) in repos {
- self.manager.add_repository(repo);
+ self.manager.borrow_mut().add_repository(repo);
}
// TODO(phase-b): Config::get_repositories returns IndexMap<String, PhpMixed>, but
// set_repositories expects Vec<IndexMap<String, PhpMixed>>; pass empty placeholder.