wip

Author: brendt

packages/upgrade/config/sets/level/up-to-tempest-30.php

@@ -0,0 +1,13 @@
+<?php
+
+declare(strict_types=1);
+
+use Rector\Config\RectorConfig;
+use Tempest\Upgrade\Set\TempestSetList;
+
+return static function (RectorConfig $config): void {
+    $config->sets([
+        TempestSetList::TEMPEST_20,
+        TempestSetList::TEMPEST_30,
+    ]);
+};

packages/upgrade/config/sets/tempest30.php

@@ -0,0 +1,11 @@
+<?php
+
+use Rector\Config\RectorConfig;
+use Tempest\Upgrade\Tempest3\WriteableRouteRector;
+
+return static function (RectorConfig $config): void {
+    $config->importNames();
+    $config->importShortClasses();
+
+    $config->rule(WriteableRouteRector::class);
+};

packages/upgrade/src/Set/TempestLevelSetList.php

@@ -7,4 +7,5 @@
 final class TempestLevelSetList
 {
     public const string UP_TO_TEMPEST_20 = __DIR__ . '/../../config/sets/level/up-to-tempest-20.php';
+    public const string UP_TO_TEMPEST_30 = __DIR__ . '/../../config/sets/level/up-to-tempest-30.php';
 }

packages/upgrade/src/Set/TempestSetList.php

@@ -7,4 +7,5 @@
 final class TempestSetList
 {
     public const string TEMPEST_20 = __DIR__ . '/../../config/sets/tempest20.php';
+    public const string TEMPEST_30 = __DIR__ . '/../../config/sets/tempest30.php';
 }

packages/upgrade/src/Tempest3/WriteableRouteRector.php

@@ -0,0 +1,43 @@
+<?php
+
+namespace Tempest\Upgrade\Tempest3;
+
+use PhpParser\Modifiers;
+use PhpParser\Node;
+use Rector\Rector\AbstractRector;
+use Tempest\Router\Route;
+
+final class WriteableRouteRector extends AbstractRector
+{
+    public function getNodeTypes(): array
+    {
+        return [
+            Node\Stmt\Class_::class,
+        ];
+    }
+
+    public function refactor(Node $node): void
+    {
+        if (! $node instanceof Node\Stmt\Class_) {
+            return;
+        }
+
+        // Check whether this class implements Tempest\Router\Route
+        $implements = $node->implements;
+
+        $implementsRoute = array_find_key(
+            $implements,
+            static fn (Node\Name $name) => $name->toString() === Route::class,
+        );
+
+        if ($implementsRoute === null) {
+            return;
+        }
+
+        if (! $node->isReadonly()) {
+            return;
+        }
+
+        $node->flags = $node->flags & ~Modifiers::READONLY;
+    }
+}

packages/upgrade/tests/Tempest30/Fixtures/CustomRoute.input.php

@@ -0,0 +1,16 @@
+<?php
+
+namespace Tempest\Upgrade\Tests\Tempest30\Fixtures;
+
+use Tempest\Http\Method;
+use Tempest\Router\Route;
+
+final readonly class CustomRoute implements Route
+{
+    public function __construct(
+        public Method $method,
+        public string $uri,
+        public array $middleware = [],
+        public array $without = [],
+    ) {}
+}

packages/upgrade/tests/Tempest30/Tempest30RectorTest.php

@@ -0,0 +1,20 @@
+<?php
+
+namespace Tempest\Upgrade\Tests\Tempest30;
+
+use PHPUnit\Framework\TestCase;
+use Tempest\Upgrade\Tests\RectorTester;
+
+final class Tempest30RectorTest extends TestCase
+{
+    private RectorTester $rector {
+        get => new RectorTester(__DIR__ . '/tempest30_rector.php');
+    }
+
+    public function test_writeable_routes(): void
+    {
+        $this->rector
+            ->runFixture(__DIR__ . '/Fixtures/CustomRoute.input.php')
+            ->assertContains('final class CustomRoute implements Route');
+    }
+}

packages/upgrade/tests/Tempest30/tempest30_rector.php

@@ -0,0 +1,9 @@
+<?php
+
+use Rector\Config\RectorConfig;
+use Tempest\Upgrade\Set\TempestSetList;
+
+return RectorConfig::configure()
+    ->withSets([
+        TempestSetList::TEMPEST_30,
+    ]);