chore: Adjust for mago

Author: valb-mig

src/Application/UseCase/Dice/RollDice/RollDiceUseCase.php

@@ -12,7 +12,12 @@
 
 final class RollDiceUseCase
 {
-    /** @return Result<RollDiceOutput> */
+    /**
+     * @param  RollDiceInput $input
+     * @return Result<RollDiceOutput>
+     * @throws \Random\RandomException if the system entropy source fails.
+     * @throws \LogicException if an invalid symbol somehow bypasses DiceModifier::fromString.
+     */
     public static function handle(RollDiceInput $input): Result
     {
         $total = 0;

src/Domain/Actions/Dice/RollDiceAction.php

@@ -12,6 +12,7 @@ final class RollDiceAction
      * Returns the result of rolling a dice with the given number of sides.
      * @param Dice $dice The dice to roll.
      * @return int The result of the roll.
+     * @throws \Random\RandomException if the system entropy source fails.
      */
     public static function roll(Dice $dice): int
     {

src/Domain/Entities/Session.php

@@ -12,15 +12,10 @@ final class Session
      * @param string $name The name of the session
      * @param Identifier $identifier The unique identifier for the session
      * @param \DateTime $createdAt The date and time when the session was created
-     * @throws \InvalidArgumentException if the session name is empty
      */
     public function __construct(
         public readonly string $name,
         public readonly Identifier $identifier,
         public readonly \DateTime $createdAt,
-    ) {
-        if (empty($name)) {
-            throw new \InvalidArgumentException('Session name cannot be empty');
-        }
-    }
+    ) {}
 }

src/Domain/ValueObjects/DiceModifier.php

@@ -45,14 +45,16 @@ public static function fromString(string $modifier): self
     /**
      * @param int $rollValue
      * @return int
+     * @throws \LogicException if an invalid symbol somehow bypasses fromString
      */
     public function apply(int $rollValue): int
     {
         return (int) ceil(match ($this->symbol) {
             '+' => $rollValue + $this->value,
             '-' => $rollValue - $this->value,
             '*', 'x' => $rollValue * $this->value,
-            '/' => $this->value !== 0 ? $rollValue / $this->value : $rollValue,
+            '/' => $rollValue / $this->value,
+            default => throw new \LogicException("Unexpected symbol: {$this->symbol}"),
         });
     }
 }