diff options
| author | nsfisis <nsfisis@gmail.com> | 2026-06-26 00:38:27 +0900 |
|---|---|---|
| committer | nsfisis <nsfisis@gmail.com> | 2026-06-26 00:38:27 +0900 |
| commit | 90c7acbd8e6eb1ddaf8680de6a1ef9f2417d50ce (patch) | |
| tree | dac10f7452ab58e36c2795e18c23edffe3045c50 /crates/shirabe/src/dependency_resolver/operation/install_operation.rs | |
| parent | e0ba36807619780ff645efe0117281d32d5ace2a (diff) | |
| download | php-shirabe-90c7acbd8e6eb1ddaf8680de6a1ef9f2417d50ce.tar.gz php-shirabe-90c7acbd8e6eb1ddaf8680de6a1ef9f2417d50ce.tar.zst php-shirabe-90c7acbd8e6eb1ddaf8680de6a1ef9f2417d50ce.zip | |
feat(dependency-resolver): wire operation get_package and unblock solver tests
Override OperationInterface::get_package for Install/Uninstall/MarkAlias*
operations so trait-object dispatch no longer hits the default todo!(), and
implement Rule::is_caused_by_lock's locked-repository lookup via
Request::get_locked_repository (LockArrayRepository::get_packages is infallible).
Change PackageInterface::get_type to return String (matching PHP getType():
string) so AliasPackage can delegate live to its aliasOf handle across the
RefCell instead of an impossible &str borrow.
Un-ignores 25 SolverTest cases; 2 alias cases stay ignored pending a real
solver alias-resolution discrepancy.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Diffstat (limited to 'crates/shirabe/src/dependency_resolver/operation/install_operation.rs')
| -rw-r--r-- | crates/shirabe/src/dependency_resolver/operation/install_operation.rs | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/crates/shirabe/src/dependency_resolver/operation/install_operation.rs b/crates/shirabe/src/dependency_resolver/operation/install_operation.rs index ca3286c..ef49239 100644 --- a/crates/shirabe/src/dependency_resolver/operation/install_operation.rs +++ b/crates/shirabe/src/dependency_resolver/operation/install_operation.rs @@ -48,6 +48,10 @@ impl OperationInterface for InstallOperation { fn as_install_operation(&self) -> Option<&InstallOperation> { Some(self) } + + fn get_package(&self) -> PackageInterfaceHandle { + self.package.clone() + } } impl std::fmt::Display for InstallOperation { |