Repository navigation
fix(fork): an edited agent message shows its edit time - #161
Merged
Merged
Conversation
The browser case now checks the time under a sent message after a rewrite: the row and its tooltip must show when the rewrite completed. It fails on main, where the row keeps the first send time. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
In a display mode, an agent message that a rewrite changed kept the time of its first send, while its "Edited" mark already knew the rewrite time. The message's time and tooltip now read that edit time when there is one. The row keeps its place, and its stored times stay at the first send, so turn order and "Worked for" do not change. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
This pull request makes an agent message that a rewrite changed show the rewrite's time, not the time it was first sent.
改了什么行为
Behaviour change: yes, a user who reads a thread in a display mode sees an agent message that
replace_send_to_userchanged dated by the time that rewrite completed, in the message's time and in its hover tooltip (web and desktop). Before, the message kept the time of its first send, and only the "Edited" mark's tooltip had the edit time.Why
The display modes already store the rewrite time on the message as
editedAt, for the "Edited" mark. The message's time line read onlyupdatedAt, which stays at the first send. It now readseditedAt ?? updatedAt.updatedAtitself stays at the first send. The turn fold reads it to work out an older turn's "Worked for" length. If it moved to the edit time, a turn that took 40 seconds would read as lasting until the later rewrite.Checks
How to undo
Revert the two commits. No data or setting changes.
Design notes
TimelineAssistantRows.tsx+2 −2, the two timestamp lines ofAssistantMessageMeta.