Skip to content

fix(monitoring): improve realtime request table clarity - #454

Merged
seakee merged 1 commit into
mainfrom
fix/realtime-monitoring-table
Jul 28, 2026
Merged

fix(monitoring): improve realtime request table clarity#454
seakee merged 1 commit into
mainfrom
fix/realtime-monitoring-table

Conversation

@seakee

@seakee seakee commented Jul 28, 2026

Copy link
Copy Markdown
Owner

Summary

Improve realtime request table readability and restore TPS to the stable total-latency definition.
This removes misleading generation-rate spikes while keeping request timing and usage details accessible.

Scope

  • Frontend panel
  • Manager Server
  • CPA panel mode
  • Full Docker mode
  • Native packages / release
  • Docs / Wiki
  • CI / build / tooling

Changes

  • Use total request latency as the single TPS calculation and display one TPS value per row.
  • Improve compact column alignment, token usage drilldown, status diagnostics, and tooltip readability.

User Impact

Realtime request rows show a stable TPS value without misleading high generation-rate spikes.
Usage and failed-request details are easier to scan, while successful status cells stay concise.

Compatibility / Runtime Notes

  • CPA panel mode: Supported; frontend-only presentation changes.
  • Manager Server mode: Supported; no API or storage contract changes.
  • Full Docker / native packages: Supported; rebuilt panel is embedded through the existing packaging flow.

Data / Security Notes

N/A. This PR changes frontend presentation and derived metrics only; persisted usage data and sensitive fields are unchanged.

Risk / Rollback

Risk level: Low

Rollback notes: Revert commit 844e67c to restore the previous realtime table presentation.

Verification

  • Type check
  • Lint
  • Tests
  • Build
  • Manual UI check
  • Docs/link check
  • Not applicable, docs-only

Commands / evidence:

  • npm --workspace apps/web run test -- model/eventRows.test.ts components/RealtimeEventsPanel.test.tsx (26 passed)
  • npm run type-check
  • npm run lint
  • npm run build
  • git diff --check

Screenshots / Recordings

N/A — no browser automation runtime was available. The visible layout was iterated against user-provided screenshots.

Docs

  • README / README_CN updated for user-visible capabilities
  • Matching docs manual and navigation updated
  • Demo fixtures, screenshots, and deep links reviewed
  • Release notes needed
  • Not needed — explanation included below

Docs decision: This PR refines an existing realtime monitoring table and does not add a user-facing capability that requires documentation.

Related

Refs #453

Refine realtime request metrics, status diagnostics, and token usage details.

Keep TPS based on total latency to avoid misleading generation-rate spikes.

Improve compact table alignment and tooltip readability without changing stored data.
@seakee
seakee merged commit 00cbd26 into main Jul 28, 2026
9 checks passed
@seakee
seakee deleted the fix/realtime-monitoring-table branch July 28, 2026 08:02
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