Merge branch 'main' into test-class-style-fallthrough

xHeaven · xHeaven · commit fecc915f400d · 2025-12-12T16:32:15.000+01:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,7 +2,26 @@
 
 All notable changes to this project will be documented in this file.
 
-## [2.13.0](https://github.com/tempestphp/tempest-framework/compare/v2.12.0..2.13.0)  —  2025-12-04
+## [2.14.0](https://github.com/tempestphp/tempest-framework/compare/v2.13.0..2.14.0)  —  2025-12-12
+
+### 🚨 Breaking changes
+
+- **console**: [**breaking**] allow `--force` to bypass `CautionMiddleware` (#1804) ([bccf92f](https://github.com/tempestphp/tempest-framework/commit/bccf92fb3e88c8ba35b1f89a8f893031a072df96))
+
+### 🚀 Features
+
+- **session**: add redis session manager (#1790) ([eb7150b](https://github.com/tempestphp/tempest-framework/commit/eb7150b8c4226397feee5311b0033c5f05fd9be1))
+
+### 🐛 Bug fixes
+
+- **core**: fix class guards not using the autoloader (#1808) ([450b2e7](https://github.com/tempestphp/tempest-framework/commit/450b2e7058852044d6257dc98c92b3366cb7e2fc))
+- **database**: support route binding through `IsDatabaseModel` (#1794) ([3556acb](https://github.com/tempestphp/tempest-framework/commit/3556acb1538a26cb21b33fb0e0d15180b64527c9))
+- **database**: support pagination with joins and relations (#1801) ([0b52ffd](https://github.com/tempestphp/tempest-framework/commit/0b52ffd2cd18db89bb27fa3ace5b409e90f20f7e))
+- **router**: add null checks and fix route parameter handling (#1778) ([c89c345](https://github.com/tempestphp/tempest-framework/commit/c89c345da59a97612c4ac5807cafee0b5e1b1da9))
+- **view**: throw exception when parsing xml views with `short_open_tag` enabled (#1795) ([30b2a6f](https://github.com/tempestphp/tempest-framework/commit/30b2a6ff8f8d4d108b334109748c96fe56ad8bde))
+
+
+## [2.13.0](https://github.com/tempestphp/tempest-framework/compare/v2.12.0..v2.13.0)  —  2025-12-04
 
 ### 🚀 Features
 
diff --git a/packages/auth/src/Installer/AuthenticationInstaller.php b/packages/auth/src/Installer/AuthenticationInstaller.php
@@ -15,7 +15,7 @@
 use function Tempest\src_path;
 use function Tempest\Support\Namespace\to_fqcn;
 
-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {
+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {
     final class AuthenticationInstaller implements Installer
     {
         use PublishesFiles;
diff --git a/packages/cache/src/Commands/CacheClearCommand.php b/packages/cache/src/Commands/CacheClearCommand.php
@@ -22,7 +22,7 @@
 use function Tempest\Support\arr;
 use function Tempest\Support\str;
 
-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {
+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {
     final readonly class CacheClearCommand
     {
         use HasConsole;
diff --git a/packages/cache/src/Commands/CacheStatusCommand.php b/packages/cache/src/Commands/CacheStatusCommand.php
@@ -22,7 +22,7 @@
 
 use function Tempest\Support\arr;
 
-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {
+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {
     final readonly class CacheStatusCommand
     {
         use HasConsole;
diff --git a/packages/command-bus/src/HandleAsyncCommand.php b/packages/command-bus/src/HandleAsyncCommand.php
@@ -14,7 +14,7 @@
 
 use function Tempest\Support\arr;
 
-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {
+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {
     final readonly class HandleAsyncCommand
     {
         use HasConsole;
diff --git a/packages/command-bus/src/MonitorAsyncCommands.php b/packages/command-bus/src/MonitorAsyncCommands.php
@@ -13,7 +13,7 @@
 
 use function Tempest\Support\arr;
 
-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {
+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {
     final readonly class MonitorAsyncCommands
     {
         use HasConsole;
diff --git a/packages/console/src/Console.php b/packages/console/src/Console.php
@@ -157,4 +157,11 @@ public function supportsPrompting(): bool;
      * Forces the console to not be interactive.
      */
     public function disablePrompting(): self;
+
+    /**
+     * Whether the console is in forced mode (skipping confirmations).
+     */
+    public bool $isForced {
+        get;
+    }
 }
diff --git a/packages/console/src/GenericConsole.php b/packages/console/src/GenericConsole.php
@@ -39,7 +39,7 @@ final class GenericConsole implements Console
 
     private ?string $label = null;
 
-    private bool $isForced = false;
+    private(set) bool $isForced = false;
 
     private bool $supportsPrompting = true;
 
diff --git a/packages/console/src/Middleware/CautionMiddleware.php b/packages/console/src/Middleware/CautionMiddleware.php
@@ -25,6 +25,10 @@ public function __invoke(Invocation $invocation, ConsoleMiddlewareCallable $next
         $environment = $this->appConfig->environment;
 
         if ($environment->isProduction() || $environment->isStaging()) {
+            if ($this->console->isForced) {
+                return $next($invocation);
+            }
+
             $continue = $this->console->confirm('This command might be destructive. Do you wish to continue?');
 
             if (! $continue) {
diff --git a/packages/container/src/Commands/ContainerShowCommand.php b/packages/container/src/Commands/ContainerShowCommand.php
@@ -18,7 +18,7 @@
 use function Tempest\Support\Str\before_last;
 use function Tempest\Support\Str\contains;
 
-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {
+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {
     final readonly class ContainerShowCommand
     {
         public function __construct(
diff --git a/packages/container/src/Commands/MakeInitializerCommand.php b/packages/container/src/Commands/MakeInitializerCommand.php
@@ -13,7 +13,7 @@
 use Tempest\Generation\ClassManipulator;
 use Tempest\Generation\DataObjects\StubFile;
 
-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {
+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {
     final class MakeInitializerCommand
     {
         use PublishesFiles;
diff --git a/packages/core/src/Commands/DiscoveryClearCommand.php b/packages/core/src/Commands/DiscoveryClearCommand.php
@@ -8,7 +8,7 @@
 use Tempest\Console\ConsoleCommand;
 use Tempest\Core\DiscoveryCache;
 
-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {
+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {
     final readonly class DiscoveryClearCommand
     {
         public function __construct(
diff --git a/packages/core/src/Commands/DiscoveryGenerateCommand.php b/packages/core/src/Commands/DiscoveryGenerateCommand.php
@@ -19,7 +19,7 @@
 
 use function Tempest\env;
 
-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {
+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {
     final readonly class DiscoveryGenerateCommand
     {
         use HasConsole;
diff --git a/packages/core/src/Commands/DiscoveryStatusCommand.php b/packages/core/src/Commands/DiscoveryStatusCommand.php
@@ -15,7 +15,7 @@
 use function Tempest\root_path;
 use function Tempest\Support\str;
 
-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {
+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {
     final readonly class DiscoveryStatusCommand
     {
         public function __construct(
diff --git a/packages/core/src/Commands/InstallCommand.php b/packages/core/src/Commands/InstallCommand.php
@@ -14,7 +14,7 @@
 
 use function Tempest\Support\arr;
 
-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {
+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {
     final readonly class InstallCommand
     {
         use HasConsole;
diff --git a/packages/core/src/FrameworkKernel.php b/packages/core/src/FrameworkKernel.php
@@ -108,7 +108,7 @@ public function createContainer(): Container
 
     public function loadComposer(): self
     {
-        if (class_exists(GenericProcessExecutor::class, false)) {
+        if (class_exists(GenericProcessExecutor::class)) {
             $processExecutor = new GenericProcessExecutor();
         } else {
             $processExecutor = null;
@@ -249,7 +249,7 @@ public function registerExceptionHandler(): self
         }
 
         // TODO: refactor to not have a hard-coded dependency on these exception handlers
-        if (! class_exists(ConsoleExceptionHandler::class, false) || ! class_exists(HttpExceptionHandler::class, false)) {
+        if (! class_exists(ConsoleExceptionHandler::class) || ! class_exists(HttpExceptionHandler::class)) {
             return $this;
         }
 
diff --git a/packages/core/src/Kernel.php b/packages/core/src/Kernel.php
@@ -8,7 +8,7 @@
 
 interface Kernel
 {
-    public const string VERSION = '2.13.0';
+    public const string VERSION = '2.14.0';
 
     public string $root {
         get;
diff --git a/packages/cryptography/src/GenerateSigningKeyCommand.php b/packages/cryptography/src/GenerateSigningKeyCommand.php
@@ -13,7 +13,7 @@
 
 use function Tempest\root_path;
 
-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {
+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {
     final readonly class GenerateSigningKeyCommand
     {
         public function __construct(
diff --git a/packages/discovery/src/Commands/MakeDiscoveryCommand.php b/packages/discovery/src/Commands/MakeDiscoveryCommand.php
@@ -12,7 +12,7 @@
 use Tempest\Generation\ClassManipulator;
 use Tempest\Generation\DataObjects\StubFile;
 
-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {
+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {
     final class MakeDiscoveryCommand
     {
         use PublishesFiles;
diff --git a/packages/reflection/src/PropertyReflector.php b/packages/reflection/src/PropertyReflector.php
@@ -98,13 +98,15 @@ public function getIterableType(): ?TypeReflector
             return null;
         }
 
-        preg_match('/@var ([\\\\\w]+)\[]/', $doc, $match);
+        if (preg_match('/@var\s+([\\\\\w]+)\[\]/', $doc, $match)) {
+            return new TypeReflector(ltrim($match[1], '\\'));
+        }
 
-        if (! isset($match[1])) {
-            return null;
+        if (preg_match('/@var\s+(?:list|array)<([\\\\\w]+)>/', $doc, $match)) {
+            return new TypeReflector(ltrim($match[1], '\\'));
         }
 
-        return new TypeReflector(ltrim($match[1], '\\'));
+        return null;
     }
 
     public function isUninitialized(object $object): bool
@@ -151,7 +153,7 @@ public function hasDefaultValue(): bool
 
         $hasDefaultValue = $this->reflectionProperty->hasDefaultValue();
 
-        $hasPromotedDefaultValue = $this->isPromoted() && $constructorParameters[$this->getName()]->isDefaultValueAvailable();
+        $hasPromotedDefaultValue = $this->isPromoted() && isset($constructorParameters[$this->getName()]) && $constructorParameters[$this->getName()]->isDefaultValueAvailable();
 
         return $hasDefaultValue || $hasPromotedDefaultValue;
     }
diff --git a/packages/reflection/src/TypeReflector.php b/packages/reflection/src/TypeReflector.php
@@ -6,7 +6,6 @@
 
 use BackedEnum;
 use DateTimeInterface;
-use Exception;
 use Generator;
 use Iterator;
 use ReflectionClass as PHPReflectionClass;
@@ -232,7 +231,13 @@ private function resolveDefinition(PHPReflector|PHPReflectionType|string $reflec
         }
 
         if ($reflector instanceof PHPReflectionParameter || $reflector instanceof PHPReflectionProperty) {
-            return $this->resolveDefinition($reflector->getType());
+            $type = $reflector->getType();
+
+            if ($type === null) {
+                return 'mixed';
+            }
+
+            return $this->resolveDefinition($type);
         }
 
         if ($reflector instanceof PHPReflectionClass) {
@@ -257,7 +262,9 @@ private function resolveDefinition(PHPReflector|PHPReflectionType|string $reflec
             ));
         }
 
-        throw new Exception('Could not resolve type');
+        throw new \InvalidArgumentException(
+            sprintf('Could not resolve type for reflector of type: %s', get_debug_type($reflector)),
+        );
     }
 
     private function resolveIsNullable(PHPReflectionType|PHPReflector|string $reflector): bool
@@ -267,7 +274,9 @@ private function resolveIsNullable(PHPReflectionType|PHPReflector|string $reflec
         }
 
         if ($reflector instanceof PHPReflectionParameter || $reflector instanceof PHPReflectionProperty) {
-            return $reflector->getType()->allowsNull();
+            $type = $reflector->getType();
+
+            return $type === null || $type->allowsNull();
         }
 
         if ($reflector instanceof PHPReflectionType) {
diff --git a/packages/router/src/Commands/MakeControllerCommand.php b/packages/router/src/Commands/MakeControllerCommand.php
@@ -12,7 +12,7 @@
 use Tempest\Generation\DataObjects\StubFile;
 use Tempest\Router\Stubs\ControllerStub;
 
-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {
+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {
     final class MakeControllerCommand
     {
         use PublishesFiles;
diff --git a/packages/router/src/Commands/RoutesCommand.php b/packages/router/src/Commands/RoutesCommand.php
@@ -17,7 +17,7 @@
 use function Tempest\Support\Str\after_last;
 use function Tempest\Support\Str\before_last;
 
-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {
+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {
     final readonly class RoutesCommand
     {
         public function __construct(
diff --git a/packages/router/src/Commands/ServeCommand.php b/packages/router/src/Commands/ServeCommand.php
@@ -8,7 +8,7 @@
 use Tempest\Intl\Number;
 use Tempest\Support\Str;
 
-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {
+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {
     final readonly class ServeCommand
     {
         #[ConsoleCommand(
diff --git a/packages/router/src/RouteBindingInitializer.php b/packages/router/src/RouteBindingInitializer.php
@@ -31,6 +31,10 @@ public function initialize(ClassReflector $class, null|string|UnitEnum $tag, Con
             }
         }
 
+        if ($parameter === null) {
+            throw new RouteBindingFailed();
+        }
+
         $object = $class->callStatic('resolve', $matchedRoute->params[$parameter->getName()]);
 
         if ($object === null) {
diff --git a/packages/router/src/Routing/Construction/RoutingTree.php b/packages/router/src/Routing/Construction/RoutingTree.php
@@ -51,7 +51,7 @@ private function isOptionalSegment(string $segment): bool
 
     private function stripOptionalMarker(string $segment): string
     {
-        return str_replace('?', '', $segment);
+        return preg_replace('/^\{\?/', '{', $segment);
     }
 
     /** @return array<string, MatchingRegex> */
diff --git a/packages/router/src/UriGenerator.php b/packages/router/src/UriGenerator.php
@@ -178,6 +178,10 @@ public function isCurrentUri(array|string|MethodReflector $action, mixed ...$par
     {
         $action = $this->normalizeActionToUri($action);
 
+        if (! $this->container->has(MatchedRoute::class)) {
+            return false;
+        }
+
         $matchedRoute = $this->container->get(MatchedRoute::class);
         $candidateUri = $this->createUri($action, ...[...$matchedRoute->params, ...$params]);
         $currentUri = $this->createUri([$matchedRoute->route->handler->getDeclaringClass()->getName(), $matchedRoute->route->handler->getName()]);
diff --git a/packages/view/src/Commands/ClearViewCacheCommand.php b/packages/view/src/Commands/ClearViewCacheCommand.php
@@ -10,7 +10,7 @@
 use Tempest\Container\Container;
 use Tempest\View\ViewCache;
 
-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {
+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {
     final readonly class ClearViewCacheCommand
     {
         use HasConsole;
diff --git a/packages/view/src/Commands/MakeViewCommand.php b/packages/view/src/Commands/MakeViewCommand.php
@@ -17,7 +17,7 @@
 
 use function Tempest\Support\str;
 
-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {
+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {
     final class MakeViewCommand
     {
         use PublishesFiles;
diff --git a/packages/vite-plugin-tempest/package.json b/packages/vite-plugin-tempest/package.json
@@ -1,7 +1,7 @@
 {
 	"name": "vite-plugin-tempest",
 	"type": "module",
-	"version": "2.13.0",
+	"version": "2.14.0",
 	"author": "Enzo Innocenzi",
 	"license": "MIT",
 	"sideEffects": false,

Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@`
`15`	`15`	`use function Tempest\src_path;`
`16`	`16`	`use function Tempest\Support\Namespace\to_fqcn;`
`17`	`17`
`18`		`-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {`
	`18`	`+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {`
`19`	`19`	`final class AuthenticationInstaller implements Installer`
`20`	`20`	`{`
`21`	`21`	`use PublishesFiles;`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@`
`22`	`22`	`use function Tempest\Support\arr;`
`23`	`23`	`use function Tempest\Support\str;`
`24`	`24`
`25`		`-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {`
	`25`	`+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {`
`26`	`26`	`final readonly class CacheClearCommand`
`27`	`27`	`{`
`28`	`28`	`use HasConsole;`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@`
`14`	`14`
`15`	`15`	`use function Tempest\Support\arr;`
`16`	`16`
`17`		`-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {`
	`17`	`+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {`
`18`	`18`	`final readonly class HandleAsyncCommand`
`19`	`19`	`{`
`20`	`20`	`use HasConsole;`
Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@`
`13`	`13`
`14`	`14`	`use function Tempest\Support\arr;`
`15`	`15`
`16`		`-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {`
	`16`	`+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {`
`17`	`17`	`final readonly class MonitorAsyncCommands`
`18`	`18`	`{`
`19`	`19`	`use HasConsole;`
Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@`
`18`	`18`	`use function Tempest\Support\Str\before_last;`
`19`	`19`	`use function Tempest\Support\Str\contains;`
`20`	`20`
`21`		`-if (class_exists(\Tempest\Console\ConsoleCommand::class, false)) {`
	`21`	`+if (class_exists(\Tempest\Console\ConsoleCommand::class)) {`
`22`	`22`	`final readonly class ContainerShowCommand`
`23`	`23`	`{`
`24`	`24`	`public function __construct(`