Fix uri parsing idempotency

CHANGELOG.md

@@ -10,6 +10,10 @@
 
 - _Scheme less_ urls now keep their leading `//` when converted to strings
 
+### Fixed
+
+- Url parsing idempotency with partially encoded characters
+
 ## 5.3.0 - 2026-05-14
 
 ### Changed

src/Authority/UserInformation/Password.php

@@ -54,8 +54,13 @@ public static function none(): self
      */
     public static function parsed(Uri|Concrete $parsed): self
     {
-        /** @psalm-suppress ImpureMethodCall */
-        $password = $parsed->getPassword();
+        if ($parsed instanceof Uri) {
+            /** @psalm-suppress ImpureMethodCall */
+            $password = $parsed->getRawPassword();
+        } else {
+            /** @psalm-suppress ImpureMethodCall */
+            $password = $parsed->getPassword();
+        }
 
         return match ($password) {
             null => self::none(),

src/Authority/UserInformation/User.php

@@ -53,8 +53,13 @@ public static function none(): self
      */
     public static function parsed(Uri|Concrete $parsed): self
     {
-        /** @psalm-suppress ImpureMethodCall */
-        $user = $parsed->getUsername();
+        if ($parsed instanceof Uri) {
+            /** @psalm-suppress ImpureMethodCall */
+            $user = $parsed->getRawUsername();
+        } else {
+            /** @psalm-suppress ImpureMethodCall */
+            $user = $parsed->getUsername();
+        }
 
         return match ($user) {
             null => self::none(),

src/Fragment.php

@@ -43,8 +43,13 @@ public static function none(): self
      */
     public static function parsed(Uri|Concrete $parsed): self
     {
-        /** @psalm-suppress ImpureMethodCall */
-        $fragment = $parsed->getFragment();
+        if ($parsed instanceof Uri) {
+            /** @psalm-suppress ImpureMethodCall */
+            $fragment = $parsed->getRawFragment();
+        } else {
+            /** @psalm-suppress ImpureMethodCall */
+            $fragment = $parsed->getFragment();
+        }
 
         return match ($fragment) {
             null => self::none(),

src/Query.php

@@ -43,8 +43,13 @@ public static function none(): self
      */
     public static function parsed(Uri|Concrete $parsed): self
     {
-        /** @psalm-suppress ImpureMethodCall */
-        $query = $parsed->getQuery();
+        if ($parsed instanceof Uri) {
+            /** @psalm-suppress ImpureMethodCall */
+            $query = $parsed->getRawQuery();
+        } else {
+            /** @psalm-suppress ImpureMethodCall */
+            $query = $parsed->getQuery();
+        }
 
         return match ($query) {
             null => self::none(),

src/Scheme.php

@@ -43,7 +43,7 @@ public static function parsed(
         #[\SensitiveParameter] string $origin,
     ): self {
         /** @psalm-suppress ImpureMethodCall */
-        $scheme = $parsed->getScheme();
+        $scheme = $parsed->getRawScheme();
 
         return match ($scheme) {
             null => match (\str_starts_with($origin, '//')) {

tests/UrlTest.php

@@ -339,6 +339,14 @@ public function testSchemeLessUrls(): BlackBox\Proof
             });
     }
 
+    public function testParsingIdempotencyOfPartiallyEncodedUrls()
+    {
+        $this->assertSame(
+            'http://h.tld/a%20b?x=%2e',
+            Url::of(Url::of('http://h.tld/a b?x=%2e')->toString())->toString(),
+        );
+    }
+
     public static function cases(): array
     {
         return [