Show chat usage in assistant timestamp footer - #551
Conversation
|
Codex review: found issues before merge. Reviewed June 13, 2026, 1:30 AM ET / 05:30 UTC. Summary Reproducibility: not applicable. this is a feature/UI PR rather than a current-main bug report. Source inspection and screenshots verify the proposed display behavior, and no failing current-main reproduction is required. Review metrics: 3 noteworthy metrics.
Merge readiness Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch. Rank-up moves:
Risk before merge
Maintainer options:
Next step before merge
Security Review findings
Review detailsBest possible solution: Land the usage footer after required validation finishes, with either the chat explorations removal split out or an explicit maintainer decision that the persisted exploration preset surface is retired. Do we have a high-confidence way to reproduce the issue? Not applicable: this is a feature/UI PR rather than a current-main bug report. Source inspection and screenshots verify the proposed display behavior, and no failing current-main reproduction is required. Is this the best way to solve the issue? Partly: the shared formatter and latest-assistant metadata path are a maintainable way to add the usage footer. Bundling the feature with removal of the persisted chat explorations surface is not the safest merge path without explicit maintainer acceptance. Full review comments:
Overall correctness: patch is incorrect AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against d4284d43fb6a. Label changesLabel changes:
Label justifications:
Evidence reviewedWhat 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
|
|
ClawSweeper PR egg 🔥 Warming up: real-behavior proof passed; findings, security review, or rank-up moves are still in progress. Hatch commandComment Hatchability rules:
What is this egg doing here?
|
|
@clawsweeper re-review |
|
🦞🧹 I asked ClawSweeper to review this item again. Re-review progress:
|
896b132 to
2c8997f
Compare
ccd22e1 to
febe165
Compare
Add usage placement controls for the native chat explorations UI and render context usage in the selected chat/composer locations. Share usage formatting between chat and tray session cards, keep assistant usage monotonic, and round the tray usage bar fill. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Keep the usage display focused on the assistant timestamp footer and remove the exploration placement options from the PR surface. Hide the chat explorations launcher from Diagnostics and update the page contract test accordingly. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Prefer latest assistant entry metadata for timestamp footer usage and cover lower thread total regressions. Keep UseMemo dependencies nullable-safe after rebase. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
febe165 to
56dd05e
Compare
## Summary
Validation
./build.ps1dotnet test ./tests/OpenClaw.Shared.Tests/OpenClaw.Shared.Tests.csproj --no-restoredotnet test ./tests/OpenClaw.Tray.Tests/OpenClaw.Tray.Tests.csproj --no-restore