fix(redis): use KEYS command instead of SCAN for pattern matching

Problem:
Laravel's PhpRedisConnection has inconsistent behavior between KEYS and SCAN:
- KEYS command: Automatically adds Laravel's Redis prefix to patterns
- SCAN command: Does NOT add prefix, requires full pattern

This caused scanKeys() to return 0 results even when keys existed:
- Pattern sent: laravel_database_queue_metrics:jobs:*
- SCAN looked for: laravel_database_queue_metrics:jobs:* (no auto-prefix)
- KEYS would look for: laravel_database_queue_metrics:jobs:* (auto-prefixed)

Solution:
- Changed scanKeys() to use KEYS command instead of SCAN
- KEYS automatically handles prefix, making behavior consistent
- Simplified MetricsQueryService to use manager->key() pattern

Note: KEYS is fine for our use case because:
1. We use specific patterns (jobs:*:*:*), not wildcard scans
2. Number of job types is limited (not millions of keys)
3. It's called infrequently (overview endpoint)

For production systems with millions of keys, SCAN would be
preferred, but that would require handling phpredis prefix manually.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: sylvesterdamgaard

src/Services/MetricsQueryService.php

@@ -97,15 +97,14 @@ public function getOverview(): array
         // Scan Redis for all job metrics keys and aggregate
         $manager = app(\PHPeek\LaravelQueueMetrics\Storage\StorageManager::class);
 
-        // Build full pattern including Laravel Redis prefix
-        $redisPrefix = config('database.redis.options.prefix', '');
-        $ourPrefix = config('queue-metrics.storage.prefix');
-        $fullPattern = $redisPrefix . $ourPrefix . ':jobs:*';
+        // Laravel's Redis connection automatically adds prefix for KEYS command
+        // So we only use our package prefix, not Laravel's Redis prefix
+        $pattern = $manager->key('jobs', '*', '*', '*');
 
         $driver = $manager->driver();
 
-        // Get all job metrics keys - scanKeys expects the FULL pattern with Laravel prefix
-        $keys = $driver->scanKeys($fullPattern);
+        // Get all job metrics keys
+        $keys = $driver->scanKeys($pattern);
 
         // Sum metrics from all keys
         foreach ($keys as $key) {

src/Storage/RedisStorageDriver.php

@@ -140,17 +140,11 @@ public function expire(string $key, int $seconds): bool
 
     public function scanKeys(string $pattern): array
     {
-        $keys = [];
-        $cursor = '0';
-
-        do {
-            /** @var array{0: string, 1: array<string>} */
-            $result = $this->redis->scan($cursor, ['match' => $pattern, 'count' => 100]);
-            [$cursor, $found] = $result;
-            $keys = array_merge($keys, $found);
-        } while ($cursor !== '0');
-
-        return $keys;
+        // Use KEYS command instead of SCAN for pattern matching
+        // Laravel's Redis connection automatically adds prefix to patterns for KEYS
+        // but NOT for SCAN, causing inconsistent behavior with phpredis
+        /** @var array<string> */
+        return $this->redis->command('keys', [$pattern]) ?: [];
     }
 
     public function pipeline(callable $callback): void