Skip to content

Commit d743d17

Browse files
committed
fix(codebuff): track prevHour only for time-only timestamps
Fixed prevHour to only update for time-only timestamps (HH:MM PM). RFC3339 timestamps retain their timezone, so comparing their hours with local time-only timestamps can cause incorrect date advances. Reset prevHour when a non-time-only timestamp is encountered to prevent cross-format rollover errors.
1 parent 342e4b8 commit d743d17

1 file changed

Lines changed: 8 additions & 1 deletion

File tree

internal/parser/codebuff.go

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -480,8 +480,15 @@ func parseCodebuffMessages(
480480
)
481481
}
482482
}
483+
// Only track prevHour for time-only timestamps to avoid
484+
// incorrect rollover when formats are mixed. Reset prevHour
485+
// when a non-time-only timestamp is encountered.
483486
if !ts.IsZero() {
484-
prevHour = ts.Hour()
487+
if isTimeOnly {
488+
prevHour = ts.Hour()
489+
} else {
490+
prevHour = -1
491+
}
485492
}
486493

487494
if !ts.IsZero() {

0 commit comments

Comments
 (0)