fix(phpstan): resolve type safety issues for static analysis

Add proper type guards and remove unused dependencies to pass PHPStan
level 9 analysis.

- Add type guards for mixed array values in CalculateQueueMetricsAction
- Add type assertions for command option values
- Remove unused QueueMetricsRepository dependency from RecordJobStartAction
- Handle mixed types from getMetrics() return values safely

All 118 tests still passing with strict type safety.

Author: sylvesterdamgaard

src/Actions/CalculateQueueMetricsAction.php

@@ -50,11 +50,13 @@ public function execute(string $connection, string $queue): void
             $jobClass = $job['jobClass'];
             $metrics = $this->jobRepository->getMetrics($jobClass, $connection, $queue);
 
-            $totalProcessed += $metrics['total_processed'];
-            $totalFailed += $metrics['total_failed'];
-            $totalDurationMs += $metrics['total_duration_ms'];
+            $totalProcessed += is_int($metrics['total_processed']) ? $metrics['total_processed'] : 0;
+            $totalFailed += is_int($metrics['total_failed']) ? $metrics['total_failed'] : 0;
+            $totalDurationMs += is_float($metrics['total_duration_ms']) || is_int($metrics['total_duration_ms'])
+                ? (float) $metrics['total_duration_ms']
+                : 0.0;
 
-            if ($metrics['last_processed_at'] !== null) {
+            if ($metrics['last_processed_at'] instanceof \Carbon\Carbon) {
                 if ($lastProcessedAt === null || $metrics['last_processed_at']->greaterThan($lastProcessedAt)) {
                     $lastProcessedAt = $metrics['last_processed_at'];
                 }

src/Actions/RecordJobStartAction.php

@@ -6,7 +6,6 @@
 
 use Carbon\Carbon;
 use PHPeek\LaravelQueueMetrics\Repositories\Contracts\JobMetricsRepository;
-use PHPeek\LaravelQueueMetrics\Repositories\Contracts\QueueMetricsRepository;
 
 /**
  * Record when a job starts processing.
@@ -15,7 +14,6 @@
 {
     public function __construct(
         private JobMetricsRepository $repository,
-        private QueueMetricsRepository $queueMetricsRepository,
     ) {}
 
     public function execute(

src/Console/CalculateQueueMetricsCommand.php

@@ -43,8 +43,10 @@ public function handle(): int
         try {
             if ($connection !== null && $queue !== null) {
                 // Calculate for specific queue
-                $this->info("Calculating metrics for {$connection}:{$queue}...");
-                $this->action->execute($connection, $queue);
+                $connectionStr = is_string($connection) ? $connection : '';
+                $queueStr = is_string($queue) ? $queue : '';
+                $this->info("Calculating metrics for {$connectionStr}:{$queueStr}...");
+                $this->action->execute($connectionStr, $queueStr);
                 $this->info('✓ Metrics calculated successfully');
 
                 return self::SUCCESS;