refactor(php-shim): give parse_url a typed UrlComponents result

parse_url now returns Option<UrlComponents> instead of a PhpMixed array,
and the component-selecting overload with the PHP_URL_* constants is gone:
callers read the field they want. Two call sites change behaviour as a
result, both towards PHP:

* CurlDownloader::handle_redirect tested scheme and host with is_null(), so
  an unparsable Location header (PhpMixed::Bool(false)) counted as an
  absolute URL. PHP's truthiness test sends it to the relative-path branch.
* Url::get_origin appended a literal port 0, which PHP treats as falsy and
  leaves off.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

1 files changed, 5 insertions, 12 deletions

diff --git a/crates/shirabe/src/repository/vcs/github_driver.rs b/crates/shirabe/src/repository/vcs/github_driver.rs
index 56ff3853..341d9f18 100644
--- a/crates/shirabe/src/repository/vcs/github_driver.rs
+++ b/crates/shirabe/src/repository/vcs/github_driver.rs
@@ -17,9 +17,9 @@ use indexmap::IndexMap;
 use shirabe_pcre::{CaptureKey, Preg};
 use shirabe_php_shim::Catch as _;
 use shirabe_php_shim::{
-    InvalidArgumentException, PhpMixed, RuntimeException, array_diff, array_key_exists, array_map,
+    InvalidArgumentException, PhpMixed, RuntimeException, array_diff, array_map,
     array_search_mixed, base64_decode, basename, empty, explode, extension_loaded, in_array_loose,
-    parse_url_all, php_regex, strpos, strtolower, substr, trim, urlencode,
+    parse_url, php_regex, strpos, strtolower, substr, trim, urlencode,
 };
 
 #[derive(Debug)]
@@ -666,19 +666,12 @@ impl GitHubDriver {
                     );
                 }
                 "custom" => {
-                    let bits = parse_url_all(&item_url);
-                    if matches!(bits, PhpMixed::Bool(false)) {
+                    let Some(bits) = parse_url(&item_url) else {
                         keys_to_remove.push(key_idx);
                         continue;
-                    }
-
-                    let bits_map = match bits {
-                        PhpMixed::Array(m) => m,
-                        _ => IndexMap::new(),
                     };
-                    if !array_key_exists("scheme", &bits_map)
-                        && !array_key_exists("host", &bits_map)
-                    {
+
+                    if bits.scheme.is_none() && bits.host.is_none() {
                         if Preg::is_match(php_regex!(r"{^[a-z0-9-]++\.[a-z]{2,3}$}"), &item_url) {
                             result[key_idx].insert(
                                 "url".to_string(),