Format

Author: sebastiaanluca

rector.php

@@ -5,6 +5,7 @@
 use Rector\CodingStyle\Rector\Catch_\CatchExceptionNameMatchingTypeRector;
 use Rector\CodingStyle\Rector\Stmt\NewlineAfterStatementRector;
 use Rector\Config\RectorConfig;
+use Rector\DeadCode\Rector\ClassMethod\RemoveUnusedPrivateMethodRector;
 use Rector\EarlyReturn\Rector\If_\ChangeOrIfContinueToMultiContinueRector;
 use Rector\Php74\Rector\Closure\ClosureToArrowFunctionRector;
 use Rector\Php81\Rector\Array_\FirstClassCallableRector;
@@ -47,4 +48,7 @@
         ClosureToArrowFunctionRector::class,
         FirstClassCallableRector::class,
         NewlineAfterStatementRector::class,
+        RemoveUnusedPrivateMethodRector::class => [
+            __DIR__.'/tests/MethodsTest.php',
+        ],
     ]);

src/Pipe.php

@@ -6,12 +6,8 @@
 
 class Pipe
 {
-    protected mixed $value;
-
-    public function __construct(mixed $value)
+    public function __construct(protected mixed $value)
     {
-        $this->value = $value;
-
         if (! defined('PIPED_VALUE')) {
             define('PIPED_VALUE', 'PIPED_VALUE-'.uniqid('', true));
         }

src/PipeProxy.php

@@ -8,14 +8,7 @@
 
 class PipeProxy
 {
-    protected Pipe $item;
-    protected object $object;
-
-    public function __construct(Pipe $item, object $object)
-    {
-        $this->item = $item;
-        $this->object = $object;
-    }
+    public function __construct(protected Pipe $item, protected object $object) {}
 
     public function __call(string $method, array $arguments): Pipe
     {

tests/MethodsTest.php

@@ -88,7 +88,7 @@ public function it can transform a value using a closure(): void
         $this->assertSame(
             'prefixed-string',
             take('string')
-                ->pipe(function (string $value) {
+                ->pipe(function (string $value): string {
                     return 'prefixed-'.$value;
                 })
                 ->get()
@@ -147,6 +147,11 @@ public function it can transform a value using a proxied public class 
         );
     }
 
+    public function uppercase(string $value): string
+    {
+        return mb_strtoupper($value);
+    }
+
     /**
      * @test
      */
@@ -174,6 +179,16 @@ public function it can transform a value using a proxied private class
         );
     }
 
+    private function lowercase(string $value): string
+    {
+        return mb_strtolower($value);
+    }
+
+    private function join(string ...$values): string
+    {
+        return implode('-', $values);
+    }
+
     /**
      * @test
      */
@@ -199,19 +214,4 @@ public function it can transform a value while accepting pipe parameters
                 ->get()
         );
     }
-
-    public function uppercase(string $value): string
-    {
-        return mb_strtoupper($value);
-    }
-
-    private function lowercase(string $value): string
-    {
-        return mb_strtolower($value);
-    }
-
-    private function join(string ...$values): string
-    {
-        return implode('-', $values);
-    }
 }