improve(ui): calmer sessions roster with dot status and drawer-level runtime#102664
Conversation
|
Codex review: needs maintainer review before merge. Reviewed July 9, 2026, 6:50 AM ET / 10:50 UTC. Summary PR surface: Source +34, Tests -7, Docs 0. Total +27 across 64 files. Reproducibility: not applicable. as a bug reproduction; this is UI polish. The PR body screenshots and source diff show the requested visual behavior, and tests cover the runtime-search and drawer-stat paths. Review metrics: none identified. Stored data model Root-cause cluster Members:
Proposal only: this assessment does not dispatch repair, suppress jobs, mutate sibling items, close, or merge anything. Merge readiness Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch. Risk before merge
Maintainer options:
Next step before merge
Maintainer decision needed
Security Review detailsBest possible solution: Land the focused UI polish after maintainer acceptance of the status/runtime presentation direction and exact-head validation remains green; otherwise narrow it to the accepted subset. Do we have a high-confidence way to reproduce the issue? Not applicable as a bug reproduction; this is UI polish. The PR body screenshots and source diff show the requested visual behavior, and tests cover the runtime-search and drawer-stat paths. Is this the best way to solve the issue? Yes: this is the right implementation layer because the Sessions view, Sessions CSS, docs sentence, and focused UI tests own the changed presentation. Product acceptance remains the separate maintainer decision. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against 0a8677ec9948. Label changesLabel changes:
Label justifications:
Evidence reviewedPR surface: Source +34, Tests -7, Docs 0. Total +27 across 64 files. View PR surface stats
What I checked:
Likely related people:
What the crustacean ranks mean
Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics. How this review workflow works
|
…runtime Feedback pass on the sessions redesign: status pills become a plain colored dot + label (green pulse for live, green dot with neutral label for done, muted idle, red failed) in both the roster and the drawer hero, and the Runtime column moves out of the table into the row drawer as Runtime (agent runtime) plus Run duration. Runtime stays searchable. Responsive breakpoints reflow for the 7-column roster and the actions column shrinks to the chevron and row menu that landed in #102564. Closes #102654
178d4ab to
6bf7ed9
Compare
|
Merged via squash.
|
…runtime (openclaw#102664) Feedback pass on the sessions redesign: status pills become a plain colored dot + label (green pulse for live, green dot with neutral label for done, muted idle, red failed) in both the roster and the drawer hero, and the Runtime column moves out of the table into the row drawer as Runtime (agent runtime) plus Run duration. Runtime stays searchable. Responsive breakpoints reflow for the 7-column roster and the actions column shrinks to the chevron and row menu that landed in openclaw#102564. Closes openclaw#102654
Closes #102654
What Problem This Solves
Feedback on the redesigned Sessions page (#102523): the status pills added chrome the roster does not need — next to kind badges and label chips, rows read as a wall of pills — and the Runtime column (agent runtime id such as
pi/codex) is rarely consulted while scanning, costing ~130px per row while per-session detail already has a home in the row drawer.Why This Change Was Made
Continues the calmer-roster direction: status becomes typographic (colored dot + plain label, no pill), and the roster keeps only scan-relevant columns while the drawer absorbs diagnostic detail. The row actions menu that landed in #102564 already collapsed the action buttons, so the actions column also shrinks to the chevron plus menu and its header goes visually quiet (screen-reader label kept).
User Impact
Evidence
Validation:
uiVitest project on Blacksmith Testbox (Linux, Node 24): 146 files / 2368 tests passed.pnpm check:changeddelegated to Testbox: all lanes green (format, oxlint, tsgo, guards).