Skip to content

Conversation

@ariane-emory
Copy link
Owner

What does this PR do?

How did you verify your code works?

pschiel and others added 8 commits January 25, 2026 03:54
Add Details action to timeline dialog with Insert key

- Add dialog.push() method to support stacking dialogs
- Add Insert keybind in timeline to open message details
- Details dialog opens on top of timeline, Esc returns to timeline

merged inspect dialog into timelinetokens
- Added support for scrolling to assistant messages when selected from timeline
- Modified AssistantMessage component in TUI to use single root box instead of fragment
- Added anchor IDs to assistant messages in web UI components
- Extended Message interface to support optional ID prop for DOM anchoring
- Added scrollToAnyMessage function to handle both user and assistant message navigation
- Added pendingAssistantMessage state for deferred assistant message scrolling
- Updated session-turn component to pass anchor IDs to assistant message items

This enables timeline to correctly scroll viewport to show selected assistant messages, matching the behavior that already existed for user messages.
Fix viewport scrolling for agent messages in session timeline
- Added useKeyboard hook to intercept n and p keys in DialogTimeline
- Implemented navigation that skips assistant messages and only navigates user messages
- Pressing n moves to next user message in timeline
- Pressing p moves to previous user message in timeline
- Navigation respects message boundaries and works correctly with filters
- Tracks selected message ID to maintain state between navigation methods
Fixed navigation to properly move selection highlight in timeline dialog
by using selectRef.moveToValue() instead of calling onMove directly.
Keys now appear in help text and skip assistant messages.
feat: Add next / previous user message keys to the timeline
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.

3 participants