Style: update pint rules and apply styles

pint.json

@@ -6,11 +6,28 @@
         "clean_namespace": true,
         "declare_strict_types": true,
         "list_syntax": true,
+        "no_empty_phpdoc": true,
+        "no_superfluous_elseif": true,
+        "no_superfluous_phpdoc_tags": {
+            "remove_inheritdoc": true
+        },
+        "no_unneeded_braces": true,
         "no_useless_nullsafe_operator": true,
         "no_whitespace_before_comma_in_array": true,
         "normalize_index_brace": true,
         "nullable_type_declaration": true,
         "nullable_type_declaration_for_default_null_value": true,
+        "ordered_attributes": true,
+        "ordered_traits": true,
+        "ordered_types": {
+            "null_adjustment": "always_last"
+        },
+        "phpdoc_summary": true,
+        "phpdoc_trim": true,
+        "phpdoc_types": true,
+        "phpdoc_types_order": {
+            "null_adjustment": "always_last"
+        },
         "simple_to_complex_string_variable": true,
         "ternary_to_null_coalescing": true,
         "trailing_comma_in_multiline": {

src/Application/Bus/CommandDispatcher.php

@@ -23,16 +23,10 @@
 class CommandDispatcher implements ICommandDispatcher
 {
     /**
-     * @var array<string|callable>
+     * @var array<callable|string>
      */
     private array $pipes = [];
 
-    /**
-     * CommandDispatcher constructor.
-     *
-     * @param CommandHandlerContainer $handlers
-     * @param PipeContainer|null $middleware
-     */
     public function __construct(
         private readonly CommandHandlerContainer $handlers,
         private readonly ?PipeContainer $middleware = null,
@@ -42,8 +36,7 @@ public function __construct(
     /**
      * Dispatch messages through the provided pipes.
      *
-     * @param array<string|callable> $pipes
-     * @return void
+     * @param array<callable|string> $pipes
      */
     public function through(array $pipes): void
     {
@@ -52,9 +45,6 @@ public function through(array $pipes): void
         $this->pipes = $pipes;
     }
 
-    /**
-     * @inheritDoc
-     */
     public function dispatch(Command $command): Result
     {
         $pipeline = PipelineBuilder::make($this->middleware)
@@ -71,7 +61,6 @@ public function dispatch(Command $command): Result
     }
 
     /**
-     * @param Command $command
      * @return Result<mixed>
      */
     private function execute(Command $command): Result

src/Application/Bus/CommandHandler.php

@@ -19,18 +19,10 @@
 
 final readonly class CommandHandler implements ICommandHandler
 {
-    /**
-     * CommandHandler constructor.
-     *
-     * @param object $handler
-     */
     public function __construct(private object $handler)
     {
     }
 
-    /**
-     * @inheritDoc
-     */
     public function __invoke(Command $command): Result
     {
         assert(method_exists($this->handler, 'execute'), sprintf(
@@ -46,9 +38,6 @@ public function __invoke(Command $command): Result
         return $result;
     }
 
-    /**
-     * @inheritDoc
-     */
     public function middleware(): array
     {
         if ($this->handler instanceof DispatchThroughMiddleware) {

src/Application/Bus/CommandHandlerContainer.php

@@ -13,31 +13,28 @@
 namespace CloudCreativity\Modules\Application\Bus;
 
 use Closure;
+use CloudCreativity\Modules\Application\ApplicationException;
 use CloudCreativity\Modules\Contracts\Application\Bus\CommandHandlerContainer as ICommandHandlerContainer;
-use RuntimeException;
+use CloudCreativity\Modules\Contracts\Toolkit\Messages\Command;
 
 final class CommandHandlerContainer implements ICommandHandlerContainer
 {
     /**
-     * @var array<string, Closure>
+     * @var array<class-string<Command>, Closure>
      */
     private array $bindings = [];
 
     /**
      * Bind a command handler into the container.
      *
-     * @param string $commandClass
-     * @param Closure $binding
-     * @return void
+     * @param class-string<Command> $commandClass
+     * @param Closure(): object $binding
      */
     public function bind(string $commandClass, Closure $binding): void
     {
         $this->bindings[$commandClass] = $binding;
     }
 
-    /**
-     * @inheritDoc
-     */
     public function get(string $commandClass): CommandHandler
     {
         $factory = $this->bindings[$commandClass] ?? null;
@@ -48,6 +45,6 @@ public function get(string $commandClass): CommandHandler
             return new CommandHandler($innerHandler);
         }
 
-        throw new RuntimeException('No command handler bound for command class: ' . $commandClass);
+        throw new ApplicationException('No command handler bound for command class: ' . $commandClass);
     }
 }

src/Application/Bus/CommandQueuer.php

@@ -18,18 +18,10 @@
 
 class CommandQueuer implements ICommandQueuer
 {
-    /**
-     * CommandQueuer constructor.
-     *
-     * @param Queue $queue
-     */
     public function __construct(private readonly Queue $queue)
     {
     }
 
-    /**
-     * @inheritDoc
-     */
     public function queue(Command $command): void
     {
         $this->queue->push($command);

src/Application/Bus/Middleware/ExecuteInUnitOfWork.php

@@ -22,20 +22,14 @@
 final readonly class ExecuteInUnitOfWork implements CommandMiddleware
 {
     /**
-     * ExecuteInUnitOfWork constructor.
-     *
-     * @param UnitOfWorkManager $unitOfWorkManager
-     * @param int $attempts
+     * @param int<1, max> $attempts
      */
     public function __construct(
         private UnitOfWorkManager $unitOfWorkManager,
         private int $attempts = 1,
     ) {
     }
 
-    /**
-     * @inheritDoc
-     */
     public function __invoke(Command $command, Closure $next): Result
     {
         try {

src/Application/Bus/Middleware/FlushDeferredEvents.php

@@ -21,18 +21,10 @@
 
 final readonly class FlushDeferredEvents implements CommandMiddleware
 {
-    /**
-     * FlushDeferredEvents constructor.
-     *
-     * @param DeferredDispatcher $dispatcher
-     */
     public function __construct(private DeferredDispatcher $dispatcher)
     {
     }
 
-    /**
-     * @inheritDoc
-     */
     public function __invoke(Command $command, Closure $next): Result
     {
         try {

src/Application/Bus/Middleware/LogMessageDispatch.php

@@ -25,14 +25,6 @@
 
 final readonly class LogMessageDispatch implements BusMiddleware
 {
-    /**
-     * LogMessageDispatch constructor.
-     *
-     * @param LoggerInterface $logger
-     * @param string $dispatchLevel
-     * @param string $dispatchedLevel
-     * @param ContextFactory $context
-     */
     public function __construct(
         private LoggerInterface $logger,
         private string $dispatchLevel = LogLevel::DEBUG,
@@ -41,9 +33,6 @@ public function __construct(
     ) {
     }
 
-    /**
-     * @inheritDoc
-     */
     public function __invoke(Command|Query $message, Closure $next): Result
     {
         $name = ModuleBasename::tryFrom($message)?->toString() ?? $message::class;

src/Application/Bus/Middleware/SetupBeforeDispatch.php

@@ -21,17 +21,12 @@
 final readonly class SetupBeforeDispatch implements BusMiddleware
 {
     /**
-     * SetupBeforeDispatch constructor.
-     *
      * @param Closure(): ?Closure(): void $callback
      */
     public function __construct(private Closure $callback)
     {
     }
 
-    /**
-     * @inheritDoc
-     */
     public function __invoke(Command|Query $message, Closure $next): Result
     {
         $tearDown = ($this->callback)();

src/Application/Bus/Middleware/TearDownAfterDispatch.php

@@ -21,17 +21,12 @@
 final readonly class TearDownAfterDispatch implements BusMiddleware
 {
     /**
-     * TearDownAfterDispatch constructor.
-     *
      * @param Closure(): void $callback
      */
     public function __construct(private Closure $callback)
     {
     }
 
-    /**
-     * @inheritDoc
-     */
     public function __invoke(Command|Query $message, Closure $next): Result
     {
         try {

src/Application/Bus/Middleware/ValidateCommand.php

@@ -24,22 +24,14 @@ abstract class ValidateCommand implements CommandMiddleware
     /**
      * Get the rules for the validation.
      *
-     * @return iterable<string|callable>
+     * @return iterable<callable|string>
      */
     abstract protected function rules(): iterable;
 
-    /**
-     * ValidateCommand constructor.
-     *
-     * @param Validator $validator
-     */
     public function __construct(private readonly Validator $validator)
     {
     }
 
-    /**
-     * @inheritDoc
-     */
     public function __invoke(Command $command, Closure $next): IResult
     {
         $errors = $this->validator

src/Application/Bus/Middleware/ValidateQuery.php

@@ -24,22 +24,14 @@ abstract class ValidateQuery implements QueryMiddleware
     /**
      * Get the rules for the validation.
      *
-     * @return iterable<string|callable>
+     * @return iterable<callable|string>
      */
     abstract protected function rules(): iterable;
 
-    /**
-     * ValidateQuery constructor.
-     *
-     * @param Validator $validator
-     */
     public function __construct(private readonly Validator $validator)
     {
     }
 
-    /**
-     * @inheritDoc
-     */
     public function __invoke(Query $query, Closure $next): IResult
     {
         $errors = $this->validator

src/Application/Bus/QueryDispatcher.php

@@ -23,16 +23,10 @@
 class QueryDispatcher implements IQueryDispatcher
 {
     /**
-     * @var array<string|callable>
+     * @var array<callable|string>
      */
     private array $pipes = [];
 
-    /**
-     * QueryDispatcher constructor.
-     *
-     * @param QueryHandlerContainer $handlers
-     * @param PipeContainer|null $middleware
-     */
     public function __construct(
         private readonly QueryHandlerContainer $handlers,
         private readonly ?PipeContainer $middleware = null,
@@ -42,8 +36,7 @@ public function __construct(
     /**
      * Dispatch messages through the provided pipes.
      *
-     * @param array<string|callable> $pipes
-     * @return void
+     * @param array<callable|string> $pipes
      */
     public function through(array $pipes): void
     {
@@ -52,9 +45,6 @@ public function through(array $pipes): void
         $this->pipes = $pipes;
     }
 
-    /**
-     * @inheritDoc
-     */
     public function dispatch(Query $query): Result
     {
         $pipeline = PipelineBuilder::make($this->middleware)
@@ -71,7 +61,6 @@ public function dispatch(Query $query): Result
     }
 
     /**
-     * @param Query $query
      * @return Result<mixed>
      */
     private function execute(Query $query): Result

src/Application/Bus/QueryHandler.php

@@ -19,18 +19,10 @@
 
 final readonly class QueryHandler implements IQueryHandler
 {
-    /**
-     * QueryHandler constructor.
-     *
-     * @param object $handler
-     */
     public function __construct(private object $handler)
     {
     }
 
-    /**
-     * @inheritDoc
-     */
     public function __invoke(Query $query): Result
     {
         assert(method_exists($this->handler, 'execute'), sprintf(
@@ -46,9 +38,6 @@ public function __invoke(Query $query): Result
         return $result;
     }
 
-    /**
-     * @inheritDoc
-     */
     public function middleware(): array
     {
         if ($this->handler instanceof DispatchThroughMiddleware) {