Remove dead code and modernize validation rules

InteractsWithQueue:
- Remove unreachable else branch in fail() method
- After type narrowing, $exception is Throwable|null, so the
  instanceof Throwable || is_null() condition is always true
- Remove unused InvalidArgumentException import

ExcludeIf/ProhibitedIf:
- Change parameter type from mixed to bool|Closure to match property type
- Remove redundant runtime validation (PHP type system handles it)
- Remove unused InvalidArgumentException import
- Update tests to expect TypeError instead of InvalidArgumentException
  (both correctly reject invalid types, just different exception)

Author: binaryfire

src/queue/src/InteractsWithQueue.php

@@ -10,7 +10,6 @@
 use Hypervel\Queue\Contracts\Job as JobContract;
 use Hypervel\Queue\Exceptions\ManuallyFailedException;
 use Hypervel\Queue\Jobs\FakeJob;
-use InvalidArgumentException;
 use PHPUnit\Framework\Assert as PHPUnit;
 use RuntimeException;
 use Throwable;
@@ -52,13 +51,8 @@ public function fail(string|Throwable|null $exception = null): void
             $exception = new ManuallyFailedException($exception);
         }
 
-        if ($exception instanceof Throwable || is_null($exception)) {
-            if ($this->job) {
-                $this->job->fail($exception);
-                return;
-            }
-        } else {
-            throw new InvalidArgumentException('The fail method requires a string or an instance of Throwable.');
+        if ($this->job) {
+            $this->job->fail($exception);
         }
     }
 

src/validation/src/Rules/ExcludeIf.php

@@ -5,7 +5,6 @@
 namespace Hypervel\Validation\Rules;
 
 use Closure;
-use InvalidArgumentException;
 use Stringable;
 
 class ExcludeIf implements Stringable
@@ -17,18 +16,10 @@ class ExcludeIf implements Stringable
 
     /**
      * Create a new exclude validation rule based on a condition.
-     *
-     * @param bool|Closure $condition
-     *
-     * @throws InvalidArgumentException
      */
-    public function __construct(mixed $condition)
+    public function __construct(bool|Closure $condition)
     {
-        if ($condition instanceof Closure || is_bool($condition)) {
-            $this->condition = $condition;
-        } else {
-            throw new InvalidArgumentException('The provided condition must be a callable or boolean.');
-        }
+        $this->condition = $condition;
     }
 
     /**

src/validation/src/Rules/ProhibitedIf.php

@@ -5,7 +5,6 @@
 namespace Hypervel\Validation\Rules;
 
 use Closure;
-use InvalidArgumentException;
 use Stringable;
 
 class ProhibitedIf implements Stringable
@@ -17,18 +16,10 @@ class ProhibitedIf implements Stringable
 
     /**
      * Create a new prohibited validation rule based on a condition.
-     *
-     * @param bool|Closure $condition
-     *
-     * @throws InvalidArgumentException
      */
-    public function __construct(mixed $condition)
+    public function __construct(bool|Closure $condition)
     {
-        if ($condition instanceof Closure || is_bool($condition)) {
-            $this->condition = $condition;
-        } else {
-            throw new InvalidArgumentException('The provided condition must be a callable or boolean.');
-        }
+        $this->condition = $condition;
     }
 
     /**

tests/Validation/ValidationExcludeIfTest.php

@@ -9,9 +9,9 @@
 use Hypervel\Translation\Translator;
 use Hypervel\Validation\Rules\ExcludeIf;
 use Hypervel\Validation\Validator;
-use InvalidArgumentException;
 use PHPUnit\Framework\TestCase;
 use stdClass;
+use TypeError;
 
 /**
  * @internal
@@ -52,8 +52,8 @@ public function testItValidatesCallableAndBooleanAreAcceptableArguments()
             try {
                 new ExcludeIf($condition);
                 $this->fail('The ExcludeIf constructor must not accept ' . gettype($condition));
-            } catch (InvalidArgumentException $exception) {
-                $this->assertEquals('The provided condition must be a callable or boolean.', $exception->getMessage());
+            } catch (TypeError) {
+                $this->assertTrue(true); // Invalid types correctly rejected by PHP type system
             }
         }
     }

tests/Validation/ValidationProhibitedIfTest.php

@@ -9,9 +9,9 @@
 use Hypervel\Translation\Translator;
 use Hypervel\Validation\Rules\ProhibitedIf;
 use Hypervel\Validation\Validator;
-use InvalidArgumentException;
 use PHPUnit\Framework\TestCase;
 use stdClass;
+use TypeError;
 
 /**
  * @internal
@@ -52,8 +52,8 @@ public function testItValidatesCallableAndBooleanAreAcceptableArguments()
             try {
                 new ProhibitedIf($condition);
                 $this->fail('The ProhibitedIf constructor must not accept ' . gettype($condition));
-            } catch (InvalidArgumentException $exception) {
-                $this->assertEquals('The provided condition must be a callable or boolean.', $exception->getMessage());
+            } catch (TypeError) {
+                $this->assertTrue(true); // Invalid types correctly rejected by PHP type system
             }
         }
     }