From ea39e2301824435d45db88f95867dbe692aaf21e Mon Sep 17 00:00:00 2001 From: nsfisis Date: Mon, 17 Aug 2026 04:55:59 +0900 Subject: fix(platform): expand the %VAR% form in expand_path() The alternation that stands in for the original conditional subpattern reports the branch it did not take as an empty string, so `dvar` was always present and won over `pvar`, leaving %VAR% unexpanded. `\w+` cannot capture an empty string, so an empty branch means it did not participate. Co-Authored-By: Claude Opus 5 (1M context) --- crates/shirabe/src/util/platform.rs | 4 +++- crates/shirabe/tests/util/platform_test.rs | 3 --- 2 files changed, 3 insertions(+), 4 deletions(-) (limited to 'crates/shirabe') diff --git a/crates/shirabe/src/util/platform.rs b/crates/shirabe/src/util/platform.rs index f45294d5..4336de53 100644 --- a/crates/shirabe/src/util/platform.rs +++ b/crates/shirabe/src/util/platform.rs @@ -95,12 +95,14 @@ impl Platform { // Regex pattern compatibility: // The original pattern uses a conditional subpattern to make the trailing `%` required // only for the `%VAR%` form. The Rust regex crate does not support conditionals, so the - // two forms are written as an explicit alternation: `$VAR` or `%VAR%`. + // two forms are written as an explicit alternation: `$VAR` or `%VAR%`. The branch that did + // not participate is reported as an empty string, which `\w+` can never capture. Preg::replace_callback( php_regex!(r"#^(?:\$(?P\w+)|%(?P\w+)%)(?P.*)#"), |matches: &indexmap::IndexMap| -> String { let var = matches .get(&CaptureKey::ByName("dvar".to_string())) + .filter(|dvar| !dvar.is_empty()) .or_else(|| matches.get(&CaptureKey::ByName("pvar".to_string()))) .map(|s| s.as_str()) .unwrap_or(""); diff --git a/crates/shirabe/tests/util/platform_test.rs b/crates/shirabe/tests/util/platform_test.rs index 6c941ab2..be2764fe 100644 --- a/crates/shirabe/tests/util/platform_test.rs +++ b/crates/shirabe/tests/util/platform_test.rs @@ -4,9 +4,6 @@ use shirabe::util::platform::Platform; use shirabe_php_shim::defined; #[test] -#[ignore = "Preg::replace_callback doesn't set PREG_UNMATCHED_AS_NULL, so the non-participating \ -alternation branch (dvar) is captured as an empty string instead of absent; Platform::expand_path's \ -matches.get(dvar).or_else(pvar) then picks the empty dvar over pvar for the %VAR% form"] fn test_expand_path() { Platform::put_env("TESTENV", "/home/test"); assert_eq!( -- cgit v1.3.1-4-g156e