aboutsummaryrefslogtreecommitdiffhomepage
path: root/crates/shirabe/src/command/package_discovery_trait.rs
diff options
context:
space:
mode:
authornsfisis <nsfisis@gmail.com>2026-06-07 13:37:11 +0900
committernsfisis <nsfisis@gmail.com>2026-06-07 13:37:11 +0900
commit86961b16b2f5c9c26a776193934d13ff87ab7fea (patch)
tree5a45b3aed39a6413e126c7992a8ac6ebcd30b0ad /crates/shirabe/src/command/package_discovery_trait.rs
parent14138dde4a8c874bdc498edd25d1a2d7dc9022be (diff)
downloadphp-shirabe-86961b16b2f5c9c26a776193934d13ff87ab7fea.tar.gz
php-shirabe-86961b16b2f5c9c26a776193934d13ff87ab7fea.tar.zst
php-shirabe-86961b16b2f5c9c26a776193934d13ff87ab7fea.zip
refactor(phase-c): resolve shared-ownership TODOs via handle/Rc clones
Resolve the resolvable subset of category A (shared ownership / non-cloneable PHP class) TODOs by leaning on values that are already shared behind Rc/handle wrappers, where cloning preserves PHP reference semantics: - solver: call IgnoreListPlatformRequirementFilter::filter_constraint with a cloned AnyConstraint (a Clone enum) and propagate the Result - update_command: filter PlatformRepository out of the repository manager's handles into a CompositeRepository (array_filter equivalent) - package_discovery_trait: pass the real platform_requirement_filter (Rc clone) instead of substituting ignore_nothing() - installation_manager: pass the original full operation list (Vec<Rc<_>> clone) as all_operations - file_downloader: swap self.io to NullIO and restore via std::mem::replace - auditor: reuse the PackageInterfaceHandle list across the advisory and abandoned-package queries - process_executor: drop the unused, lossy Clone impl - config / array_repository: demote settled RefCell-design markers to comments Remaining category A items (factory installer wiring, purge_packages handle bridge, installer cache identity, reinstall flow, plugin command discovery) stay as TODOs since correct resolution needs structural refactors. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Diffstat (limited to 'crates/shirabe/src/command/package_discovery_trait.rs')
-rw-r--r--crates/shirabe/src/command/package_discovery_trait.rs6
1 files changed, 2 insertions, 4 deletions
diff --git a/crates/shirabe/src/command/package_discovery_trait.rs b/crates/shirabe/src/command/package_discovery_trait.rs
index cd6c5a0..58361b5 100644
--- a/crates/shirabe/src/command/package_discovery_trait.rs
+++ b/crates/shirabe/src/command/package_discovery_trait.rs
@@ -531,8 +531,7 @@ pub trait PackageDiscoveryTrait {
name,
None,
preferred_stability,
- // TODO(phase-b): Box<dyn ...> cannot be cloned; original PHP shares reference
- Some(PlatformRequirementFilterFactory::ignore_nothing()),
+ Some(platform_requirement_filter.clone()),
0,
None,
shirabe_php_shim::PhpMixed::Null,
@@ -620,8 +619,7 @@ pub trait PackageDiscoveryTrait {
name,
None,
preferred_stability,
- // TODO(phase-b): Box<dyn ...> cannot be cloned; reusing factory result
- Some(PlatformRequirementFilterFactory::ignore_nothing()),
+ Some(platform_requirement_filter.clone()),
RepositorySet::ALLOW_UNACCEPTABLE_STABILITIES,
None,
shirabe_php_shim::PhpMixed::Null,