Skip to content

Runtime fixes 2/5: Bound Logs page retention - #432

Merged
guido4096 merged 1 commit into
danielkucera:masterfrom
guido4096:fix/runtime-log-retention
Oct 31, 2026
Merged

guido4096 merged 1 commit into
danielkucera:masterfrom
guido4096:fix/runtime-log-retention

Conversation

@guido4096

@guido4096 guido4096 commented Oct 31, 2026 •

Copy link
Copy Markdown
Collaborator

On a long-running Logs page, duplicate keys remained in seenEntries after their log entries were evicted. Once the retention limit was reached, incoming batches could also trigger rebuilding the complete displayed log.

Keep duplicate keys aligned with retained entries and remove evicted DOM rows incrementally. Preserve filtering, clearing and restart markers. Add a regression that runs the actual page script through repeated retention-limit crossings.

Validation

  • 170 host tests passed on this standalone branch (macOS, POSIX enabled, original pinned eBUS revision f730dce).
  • Whitespace checks passed. CI and a fresh firmware build for this isolated branch are pending.
  • For the standalone macOS test run, the two mock-file fixes from part 1/5 were applied as temporary, uncommitted test-only overrides. Those files are not included in this PR.

The regression covers bounded retained keys/entries, incremental row eviction, filtering, clearing and restart markers.

Browser check before marking ready

Leave the Logs page open through repeated crossings of the 2,000-entry limit; verify that new logs, filtering and clearing continue to work and browser memory settles. The host regression is not a browser soak-test result.

Related runtime fixes

Part 2/5 of the single-commit replacements for #430, split at the maintainer's request. Every PR targets master directly and contains only its corresponding change.

Part Scope PR / branch
1/5 Correct host-test mocks #431
2/5 Bound Logs page retention #432
3/5 Schedule diagnostics by elapsed time #433
4/5 Protect string-pool IDs and concurrent access #434
5/5 Buffer HTTP streams and stop after send failures #435

Suggested merge order: the host-test mock fix first, then the other four changes independently. The five patches together reconstruct the exact tracked file tree of #430.

Source commit in #430: f552144.

Keep duplicate keys in sync with retained entries and remove only evicted DOM rows. Cover long-running retention, filtering and restart markers.
@guido4096
guido4096 merged commit 7fdf22d into danielkucera:master Oct 31, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants