Skip to content

fix(dash): refresh stats on an interval and compact feed timestamps - #137

Merged
fu351 merged 1 commit into
mainfrom
fix/dash/live-stats-compact-ts
Jul 20, 2026
Merged

fix(dash): refresh stats on an interval and compact feed timestamps#137
fu351 merged 1 commit into
mainfrom
fix/dash/live-stats-compact-ts

Conversation

@fu351

@fu351 fu351 commented Jul 20, 2026

Copy link
Copy Markdown
Owner

Slice

  • Repo: doberman-core
  • Feature / Slice: dash polish follow-up (found while recording the README hero GIF)

What this PR does

The stats row was fetched once at page load and froze at those values while the live feed kept filling - visibly wrong on camera (feed shows 7 decisions, counters read 0). Stats now refresh every 5s via the same authed GET. Feed timestamps render as compact HH:MM:SS sliced from the ISO string (full precision stays in doberman log / the TUI).

Tests added (run in CI)

  • tests/unit/test_dash_polish.py: stats-refresh interval present; compact timestamp form present and the old full-ISO render gone. Full dash suite: 58 passed.

Public-release safety (doberman-core only)

  • Contains nothing from the not-allowed list
  • Core still builds/tests/runs with NO enterprise package installed

Security checklist

  • Fails closed on error / uncertainty (stats fetch failure renders 'stats unavailable', nothing else changes)
  • No secret, full file, or unredacted prompt logged or committed
  • Any guardrail/learning change is raise-only (N/A - UI only)
  • Every BLOCK/AUTH carries reason codes + a human explanation (N/A)
  • doberman-core does not import doberman_enterprise

Edge cases covered / Deviations from plan / Risks introduced

  • textContent-only rendering discipline unchanged (asserted by existing test); no new endpoints; missing/short ts renders '-'.

The stats row was fetched once at page load and froze there while the live feed kept filling - surfaced immediately when recording the README demo. Refresh it every 5s (same lightweight authed GET). Also slice feed timestamps to HH:MM:SS - the full ISO-with-microseconds string was noise; the precise timestamp stays available in doberman log and the TUI. No new endpoints, no auth/redaction changes.
@fu351
fu351 merged commit 679ea3a into main Jul 20, 2026
20 of 27 checks passed
@fu351
fu351 deleted the fix/dash/live-stats-compact-ts branch July 20, 2026 02:00
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.

1 participant