ui: give the startup block a static settled form - #121
Merged
Conversation
…pens with what happened The settled header rendered the session's LIVE state, so reading an old session opened with "Session asleep" as its first line. It now compresses the start into one static line, the way a collapsed tool run does.
Contributor
There was a problem hiding this comment.
Important
Looks good to me! 👍
Reviewed b5ee90d in 7 minutes, 34 seconds.
- Reviewed
1commit with313lines of code in2files - Ran
1review agent producing0comments where0were posted - This pipeline runs no gatekeeper, so findings are posted as written.
- View full details on ellipsis.dev
This review was created by . You can tag
@ellipsis in this pull request.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
✓ Session asleepas its first line. It now shows a static record of the start:✦ Cloud agent session on ellipsis.dev+⎿ Sandbox ready in 42s · 31 log lines, compressed the way a collapsed tool run is.Starting cloud agent…;session_idle/session_resumedsettle the block without rewriting its headline.Sandbox started · N log lineswhen no duration can be derived (old feeds with nophase_timings, wakes).Test plan
bun run typecheckbun run test(467 pass)agent session connecton an old, asleep session: opening line reads as a past event, no live statusagent session connecton a fresh start: live headline ticks, build/setup log still streams, then collapses to the summaryImportant
Changes the settled startup block to show a static one-line summary instead of rendering live session status.
When a session startup finishes, the block now displays a compressed record of the start event — e.g.
Cloud agent session on ellipsis.devwithSandbox ready in 42s · 31 log lines— the way a collapsed tool run does. Previously it would render live status (e.g. "Session asleep"), which made old sessions open with "Session asleep" as the first line of the conversation, which is not what happened first.Starting cloud agent…while the session is coming up; the three settlement events (session_idle,session_resumed,sandbox_ready) now settle the block without rewriting its headline.SandboxState.readySecondsto track sandbox provisioning duration fromphase_timings; settlement builds a static one-line summary withsandboxSummary()instead of a live status.sandboxRows()now renders two forms: a live version with ticking headline + log tail, and a settled version with the static summary line under the opener.This description was created by
for b5ee90d. It will automatically update as commits are pushed.