From 627aaead32dcf38bed255012fcf7965d9553f52e Mon Sep 17 00:00:00 2001 From: nsfisis Date: Sun, 9 Aug 2026 07:36:40 +0900 Subject: fix(dependency-resolver): query the PHP runtime for the php version Problem reported the platform php version from the shim's compile-time PHP_VERSION constant, and the shim's phpversion() hit todo!() for any extension. Go through the RPC bridge instead, as PHP's Composer\Platform\Runtime does, and drop the shim function that now has no callers. Co-Authored-By: Claude Opus 5 (1M context) --- crates/shirabe/src/dependency_resolver/problem.rs | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) (limited to 'crates/shirabe/src/dependency_resolver/problem.rs') diff --git a/crates/shirabe/src/dependency_resolver/problem.rs b/crates/shirabe/src/dependency_resolver/problem.rs index 3b9feba5..63cbc6fb 100644 --- a/crates/shirabe/src/dependency_resolver/problem.rs +++ b/crates/shirabe/src/dependency_resolver/problem.rs @@ -14,8 +14,8 @@ use shirabe_external_packages::composer::pcre::{CaptureKey, Preg}; use shirabe_external_packages::symfony::console::formatter::OutputFormatter; use shirabe_php_shim::{ CmpOp, LogicException, PhpMixed, defined, extension_loaded, implode, loosely_compare, - php_regex, phpversion, spl_object_hash, sprintf, str_replace, str_starts_with, stripos, strpos, - strtolower, substr, substr_count, version_compare, + php_regex, spl_object_hash, sprintf, str_replace, str_starts_with, stripos, strpos, strtolower, + substr, substr_count, version_compare, }; use shirabe_semver::constraint::AnyConstraint; use shirabe_semver::constraint::MultiConstraint; @@ -389,7 +389,7 @@ impl Problem { let version = Self::get_platform_package_version( pool, package_name, - &phpversion("").unwrap_or_default(), + &shirabe_php_rpc::phpversion("").unwrap_or_default(), ); let msg = format!( @@ -456,8 +456,6 @@ impl Problem { Self::constraint_to_text(constraint) ); - // Per-extension version info can't be known statically; query the real PHP - // runtime via the RPC bridge, as PHP's Composer\Platform\Runtime does. let runtime_version = shirabe_php_rpc::phpversion(&ext); let effective_version = match runtime_version { None => "0".to_string(), -- cgit v1.3.1-4-g156e