Handle empty logs and assign unique client-side IDs

Added a check in `LogWatcher.cs` to skip processing empty lines
after stripping the CRI prefix, improving efficiency.

Enhanced `ParseContainerLogFormat` in `LogParser.cs` to handle
cases where CRI logs contain only flags (`F` or `P`) without
messages, returning an empty string for such cases. Addressed
edge cases involving ANSI escape sequences.

Introduced `nextClientId` in `log-processing.service.ts` to
generate unique IDs for logs without one (e.g., SignalR logs).
Updated `transformLogs` to assign these IDs, ensuring all logs
have consistent identifiers.

Author: Timothy Dodd

src/LogMkAgent/Services/LogWatcher.cs

@@ -551,6 +551,10 @@ private async Task ReadNewLinesAsync(PodInfo info, CancellationToken stoppingTok
         // Parse container format (timestamp stdout/stderr message)
         var processedLine = ParseContainerLogFormat(line, cleanLine);
 
+        // Skip empty lines after CRI prefix stripping
+        if (string.IsNullOrWhiteSpace(processedLine))
+            return null;
+
         var logLevel = GetLogLevelCached(processedLine);
 
         // If no log level was detected, default based on the CRI stream type:

src/LogMkCommon/LogParser.cs

@@ -163,25 +163,35 @@ public static string ParseContainerLogFormat(string originalLine, string cleanLi
             // Check for the F/P flag after stdout/stderr
             var thirdSpace = originalLine.IndexOf(' ', secondSpace + 1);
             if (thirdSpace <= 0)
+            {
+                // No space after stdout/stderr - check if remainder is just a flag (F/P)
+                var remainder = originalLine.Substring(secondSpace + 1);
+                if (remainder == "F" || remainder == "P")
+                    return string.Empty; // CRI line with empty message
                 return cleanLine;
+            }
 
             var flag = originalLine.Substring(secondSpace + 1, thirdSpace - secondSpace - 1);
             if (flag == "F" || flag == "P")
             {
-                // Find the actual log message after the flag
-                var fourthSpace = originalLine.IndexOf(' ', thirdSpace + 1);
-                if (fourthSpace > 0)
+                // The actual message starts right after the flag space
+                // CRI format: "timestamp stdout F message" - message is everything after "F "
+                if (thirdSpace + 1 >= originalLine.Length)
                 {
-                    // We need to find this position in the clean line
-                    // Count how many characters we need to skip in the clean line
-                    var prefixToSkip = originalLine.Substring(0, fourthSpace + 1);
-                    var cleanPrefixToSkip = RemoveANSIEscapeRegex.Replace(prefixToSkip, string.Empty);
-                    
-                    if (cleanPrefixToSkip.Length < cleanLine.Length)
-                    {
-                        return cleanLine.Substring(cleanPrefixToSkip.Length);
-                    }
+                    // Nothing after the flag - empty message
+                    return string.Empty;
+                }
+
+                var prefixToSkip = originalLine.Substring(0, thirdSpace + 1);
+                var cleanPrefixToSkip = RemoveANSIEscapeRegex.Replace(prefixToSkip, string.Empty);
+
+                if (cleanPrefixToSkip.Length < cleanLine.Length)
+                {
+                    return cleanLine.Substring(cleanPrefixToSkip.Length);
                 }
+
+                // Clean line is shorter than prefix (edge case with ANSI sequences)
+                return string.Empty;
             }
             else
             {

src/LogMkWeb/src/app/_services/log-processing.service.ts

@@ -22,13 +22,17 @@ export interface ExtractedFilters {
 })
 export class LogProcessingService {
 
+  // Counter for generating unique client-side IDs for logs that arrive without one (e.g. SignalR real-time logs)
+  private nextClientId = -1;
+
   /**
    * Core log transformation: assign pod colors and clean log lines
    * This is the main entry point for transforming raw logs into display-ready logs
    */
   transformLogs(logs: Log[]): Log[] {
     return logs.map(log => ({
       ...log,
+      id: log.id || this.nextClientId--,
       podColor: this.getPodColor(log.pod),
       view: this.cleanLogLine(log.line),
     }));