feat: improve default redirect strategy

Author: innocenzi

src/Web/Documentation/DocumentationController.php

@@ -28,9 +28,11 @@ public function redirect(string $path): Redirect
     }
 
     #[Get('/documentation')]
-    public function index(): Redirect
+    #[Get('/docs')]
+    #[Get('/{version}')]
+    public function index(?string $version): Redirect
     {
-        $version = Version::default();
+        $version = Version::tryFromString($version);
 
         $category = arr(glob(__DIR__ . "/content/{$version->getUrlSegment()}/*", flags: GLOB_ONLYDIR))
             ->tap(fn (ImmutableArray $files) => $files->isEmpty() ? throw new \RuntimeException('Documentation has not been fetched. Run `tempest docs:pull`.') : null)

src/Web/Documentation/Version.php

@@ -11,18 +11,15 @@ enum Version: string
     case VERSION_1 = '1.x';
     case VERSION_2 = '2.x';
 
-    public function isNext(): bool
+    public static function default(): self
     {
-        return match ($this) {
-            self::VERSION_2 => true,
-            default => false,
-        };
+        return self::VERSION_1;
     }
 
-    public function isCurrent(): bool
+    public function isNext(): bool
     {
         return match ($this) {
-            self::VERSION_1 => true,
+            self::VERSION_2 => true,
             default => false,
         };
     }
@@ -37,28 +34,25 @@ public function getBranch(): string
 
     public function getUrlSegment(): string
     {
-        return match ($this) {
-            self::VERSION_1 => '1.x',
-            self::VERSION_2 => '2.x',
-        };
+        return $this->value;
     }
 
     public function isPrevious(): bool
     {
         return ! $this->isNext() && ! $this->isCurrent();
     }
 
+    public function isCurrent(): bool
+    {
+        return $this === self::default();
+    }
+
     public static function tryFromString(?string $case): ?static
     {
         return match ($case) {
             'default', 'current', 'main', null => self::default(),
-            'next' => self::VERSION_2,
+            'next' => array_find(self::cases(), fn (self $version) => $version->isNext()),
             default => self::tryFrom($case),
         };
     }
-
-    public static function default(): self
-    {
-        return self::VERSION_1;
-    }
 }