Render tool calls fully when resuming a session#152
Merged
Conversation
When a user resumes a session, the tool calls that were executed previously ought to appear just as they did during the live conversation — with headers, output, overlays, and toggles. Showing only '[N tool calls]' discards the very context that makes a session history useful. This replaces the count-and-summarize approach with a result index built in a single pass over the message vector. Each tool call is then rendered through the same display-tool-start and display-tool-end machinery used during streaming, so the history and the live view are visually identical. Tool calls whose results are absent (aborted sessions) render the header alone.
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
1 participant
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.
When a user resumes a session, the tool calls that were executed
previously ought to appear just as they did during the live
conversation — with headers, output, overlays, and toggles.
Showing only '[N tool calls]' discards the very context that makes
a session history useful.
This replaces the count-and-summarize approach with a result index
built in a single pass over the message vector. Each tool call is
then rendered through the same display-tool-start and
display-tool-end machinery used during streaming, so the history
and the live view are visually identical. Tool calls whose results
are absent (aborted sessions) render the header alone.