Replace tap by transform

Author: nyamsprod

Builder.php

@@ -16,6 +16,7 @@
 use BackedEnum;
 use League\Uri\Contracts\Conditionable;
 use League\Uri\Contracts\FragmentDirective;
+use League\Uri\Contracts\Transformable;
 use League\Uri\Contracts\UriComponentInterface;
 use League\Uri\Exceptions\SyntaxError;
 use SensitiveParameter;
@@ -29,7 +30,7 @@
 use function str_replace;
 use function strpos;
 
-final class Builder implements Conditionable
+final class Builder implements Conditionable, Transformable
 {
     private ?string $scheme = null;
     private ?string $username = null;
@@ -206,13 +207,11 @@ public function reset(): self
      * Executes the given callback with the current instance
      * and returns the current instance.
      *
-     * @param callable(self): void $callback
+     * @param callable(self): self $callback
      */
-    public function tap(callable $callback): self
+    public function transform(callable $callback): static
     {
-        $callback($this);
-
-        return $this;
+        return $callback($this);
     }
 
     public function when(callable|bool $condition, callable $onSuccess, ?callable $onFail = null): static

BuilderTest.php

@@ -141,28 +141,19 @@ public function test_tap_calls_callback_with_self_and_returns_same_instance(): v
         $builder = new Builder();
 
         $called = false;
-        $callback = function (Builder $b) use (&$called, $builder) {
+        $callback = function (Builder $b) use (&$called, $builder): Builder {
             $called = true;
             self::assertSame($builder, $b);
+
+            return $b->authority('example.com');
         };
 
-        $result = $builder->tap($callback);
+        $result = $builder->transform($callback);
 
         self::assertTrue($called);
         self::assertSame($builder, $result);
-    }
-
-    public function test_tap_allows_builder_mutation(): void
-    {
-        $builder = (new Builder())->scheme('http');
-        $builder->tap(function (Builder $b) {
-            $b->host('example.com');
-        });
-
-        $uri = $builder->build();
-
-        self::assertSame('example.com', $uri->getHost());
-        self::assertSame('http', $uri->getScheme());
+        self::assertSame('example.com', $builder->build()->getHost());
+        self::assertNull($builder->build()->getScheme());
     }
 
     public function test_authority_with_host_only(): void