feat(steps): polish UI and add sub-agent + terminal-jump support#75
Merged
johannesjo merged 2 commits intomainfrom Apr 16, 2026
Merged
feat(steps): polish UI and add sub-agent + terminal-jump support#75johannesjo merged 2 commits intomainfrom
johannesjo merged 2 commits intomainfrom
Conversation
UI polish: - Replace `→ \`...\`` formatting around `next` with a clean chevron + italic - Add status-color spine on left edge of every step card - Remove fixed-length truncation; let summary/next/detail wrap naturally and rely on CSS ellipsis for the collapsed history rows - Insert a subtle divider between adjacent history entries when status changes phase (e.g. investigating → implementing) - Hover reveals copy buttons on summary and detail Sub-agent tree (minimal): - New optional `agent_id` field on StepEntry for entries written on behalf of delegated work - Steps panel indents and adds a dashed agent badge for entries with an agent_id; instruction prompt teaches the agent to populate it Jump to terminal moment: - TerminalView exposes mark/jump bookmarks backed by xterm IMarker - TaskAITerminal registers a marker for each new step as it arrives - Steps panel shows a hover affordance on each step that scrolls the AI terminal back to the moment the step was recorded
- CopyButton: clear pending reset timeout on unmount so the callback doesn't fire into a disposed component - AgentBadge: cap max-width and ellipsize so a long agent_id can't push copy/jump controls off the row - Lifecycle: TerminalView signals onStepNavReady(undefined) on cleanup; TaskAITerminal resets stepNav and lastMarkedLen so the new TerminalView (after agent restart) re-backfills markers for every existing step - Drop the early-return that advanced lastMarkedLen while no terminal was mounted — we now wait for the API before incrementing the watermark - Extract JumpButton component; both call sites now share it - Introduce SUB_AGENT_INDENT_PX constant so collapsed/expanded/latest views agree on how far to indent sub-agent rows
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
2 participants
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.
UI polish:
→ \...`formatting aroundnext` with a clean chevron + italicand rely on CSS ellipsis for the collapsed history rows
changes phase (e.g. investigating → implementing)
Sub-agent tree (minimal):
agent_idfield on StepEntry for entries written onbehalf of delegated work
agent_id; instruction prompt teaches the agent to populate it
Jump to terminal moment:
terminal back to the moment the step was recorded