Job-detail log retention depended on the active display sort instead of the stored log order. After switching to oldest-first, the next refresh could discard newly fetched lines while still advancing the last-seen timestamp.

Keep the in-memory log buffer in ascending timestamp order and always trim from the oldest side. Sorting now affects rendering only, so refreshes and backfills retain the newest log entries regardless of the selected view.

Author: thomashoneyman

dashboard/src/Dashboard/Component/JobDetail.purs

@@ -490,8 +490,7 @@ handleAction = case _ of
       let newLogs = Array.filter (isNewerThan state.lastLogTimestamp) info.logs
       when (not (Array.null newLogs)) do
         let lastTs = map _.timestamp (Array.last newLogs)
-        -- Logs are always stored in ASC order; new logs are appended at the end.
-        let combined = capLogs state.logSortOrder (state.allLogs <> newLogs)
+        let combined = keepNewestLogs (state.allLogs <> newLogs)
         H.modify_ _ { allLogs = combined, lastLogTimestamp = lastTs }
       -- Update job status and logUntil from the refreshed data
       H.modify_ _ { job = Just job, logUntil = info.finishedAt, currentTime = Just now }
@@ -536,14 +535,11 @@ logPageSize = 200
 logTotalPages :: Int -> Int
 logTotalPages total = max 1 (((total - 1) / logPageSize) + 1)
 
--- | Cap an array of logs (stored in ASC order) to `maxLogEntries`, trimming
--- | from the end furthest from the user's current view.
-capLogs :: SortOrder -> Array LogLine -> Array LogLine
-capLogs sortOrder logs =
+-- | Cap an array of logs stored in ASC order by keeping the newest entries.
+keepNewestLogs :: Array LogLine -> Array LogLine
+keepNewestLogs logs =
   if Array.length logs <= maxLogEntries then logs
-  else case sortOrder of
-    ASC -> Array.take maxLogEntries logs
-    DESC -> Array.drop (Array.length logs - maxLogEntries) logs
+  else Array.drop (Array.length logs - maxLogEntries) logs
 
 -- | Maximum number of pagination requests when fetching all remaining logs.
 maxPaginationIterations :: Int
@@ -611,7 +607,7 @@ fetchAllRemainingLogs = go 0
                   pure unit
                 else do
                   currentState <- H.get
-                  let combined = capLogs currentState.logSortOrder (currentState.allLogs <> newLogs)
+                  let combined = keepNewestLogs (currentState.allLogs <> newLogs)
                   let lastTs = map _.timestamp (Array.last combined)
                   H.modify_ _ { allLogs = combined, lastLogTimestamp = lastTs }
                   go (iteration + 1)