Improve log cleanup, DB handling, and JSON highlighting

- Execute `RunRetentionCleanupAsync` immediately on startup in `LogRetentionService` with added exception handling.
- Reduce log deletion batch size from 10,000 to 5,000 to lower DB load.
- Add retry logic with exponential backoff for deadlocks during log deletion.
- Explicitly open DB connection in `UpdateCurrentHourSummaryAsync` to ensure readiness before transactions.
- Refine JSON number regex in `HighlightLogPipe` to avoid matching times.

Author: Timothy Dodd

src/LogMkApi/Services/LogRetentionService.cs

@@ -27,6 +27,20 @@ protected override async Task ExecuteAsync(CancellationToken stoppingToken)
     {
         _logger.LogInformation("LogRetentionService started");
 
+        // Run immediately on startup
+        try
+        {
+            await RunRetentionCleanupAsync(stoppingToken);
+        }
+        catch (OperationCanceledException) when (stoppingToken.IsCancellationRequested)
+        {
+            return;
+        }
+        catch (Exception ex)
+        {
+            _logger.LogError(ex, "Error during initial retention cleanup");
+        }
+
         while (!stoppingToken.IsCancellationRequested)
         {
             var now = DateTime.UtcNow;
@@ -81,16 +95,30 @@ private async Task RunRetentionCleanupAsync(CancellationToken cancellationToken)
             retentionDays, cutoffDate);
 
         var totalDeleted = 0L;
-        const int batchSize = 10000;
+        const int batchSize = 5000;
 
         using var db = _dbFactory.CreateConnection();
 
         // Batch delete from Log table
+        const int maxRetries = 3;
         while (!cancellationToken.IsCancellationRequested)
         {
-            var deleted = await db.ExecuteAsync(
-                $"DELETE FROM {db.GetQuotedTableName<Log>()} WHERE LogDate < @cutoffDate LIMIT {batchSize}",
-                new { cutoffDate });
+            int deleted = 0;
+            for (var attempt = 1; attempt <= maxRetries; attempt++)
+            {
+                try
+                {
+                    deleted = await db.ExecuteAsync(
+                        $"DELETE FROM {db.GetQuotedTableName<Log>()} WHERE LogDate < @cutoffDate LIMIT {batchSize}",
+                        new { cutoffDate });
+                    break;
+                }
+                catch (MySql.Data.MySqlClient.MySqlException ex) when (ex.Number == 1213 && attempt < maxRetries)
+                {
+                    _logger.LogWarning("Deadlock on retention delete attempt {Attempt}, retrying after delay...", attempt);
+                    await Task.Delay(500 * attempt, cancellationToken);
+                }
+            }
 
             if (deleted == 0)
                 break;

src/LogMkApi/Services/LogSummaryHourlyBackgroundService.cs

@@ -58,6 +58,7 @@ private async Task UpdateCurrentHourSummaryAsync()
         {
             using (var connection = _dbFactory.CreateConnection())
             {
+                connection.Open();
                 var date = DateTime.UtcNow.Date;
                 var hour = DateTime.UtcNow.Hour - 1;
 

src/LogMkWeb/src/app/_pages/main-log-page/_services/highlight.directive.ts

@@ -41,8 +41,8 @@ export class HighlightLogPipe implements PipeTransform {
   // JSON string values (quoted strings not followed by colon)
   [/:\s*"([^"]*)"/g, ': "<span class="json-string">$1</span>"'],
 
-  // JSON numbers
-  [/:\s*(-?\d+(?:\.\d+)?(?:[eE][+-]?\d+)?)\b/g, ': <span class="json-number">$1</span>'],
+  // JSON numbers (negative lookbehind for digits to avoid matching times like 02:04:07)
+  [/(?<!\d):\s*(-?\d+(?:\.\d+)?(?:[eE][+-]?\d+)?)\b/g, ': <span class="json-number">$1</span>'],
 
   // JSON booleans
   [/:\s*(true|false)\b/g, ': <span class="json-boolean">$1</span>'],