feat(ui): delete a checkpoint from the chat, beside Fork - #2804
Merged
Merged
Conversation
The line a checkpoint draws now carries Delete as well as Fork, behind a confirmation: the snapshot it pins is the only thing holding that space, and nothing brings it back. Chats already forked from it are untouched. `CheckpointService/DeleteCheckpoint` was already in the proto; this wires it through the api layer and the fixture backend. Mock checkpoints move to sessionStorage so a deletion there survives a reload, as the controller's does. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Signed-off-by: Nicholas Bucher <behappy54321@gmail.com>
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Fork is the line's action, so it is filled; Remove is outlined beside it and named for what it does to the snapshot rather than to the line. Both tooltips open downwards — above the button, the tooltip covered the control the reader was about to press again. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Signed-off-by: Nicholas Bucher <behappy54321@gmail.com>
This comment has been minimized.
This comment has been minimized.
The transcript's own gap to the next message sits below the line as well, so the two together read as a hole in the conversation. Half the room below it. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Signed-off-by: Nicholas Bucher <behappy54321@gmail.com>
Charlesthebird
marked this pull request as ready for review
September 11, 2026 16:50
The Checkpoint button's tooltip named itself and nothing else, so the one place a reader meets the idea did not say what it buys them. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Signed-off-by: Nicholas Bucher <behappy54321@gmail.com>
Collaborator
Author
|
Current as of 77c4a3e. summary-reel-v3.mp4🤖 written by Claude |
Charlesthebird
enabled auto-merge
September 11, 2026 16:54
EItanya
approved these changes
Sep 14, 2026
github-merge-queue
Bot
removed this pull request from the merge queue due to failed status checks
Sep 14, 2026
peterj
approved these changes
Sep 14, 2026
github-merge-queue
Bot
removed this pull request from the merge queue due to failed status checks
Sep 14, 2026
…rd/deleteCheckpoint Signed-off-by: Nicholas Bucher <behappy54321@gmail.com>
Charlesthebird
enabled auto-merge
September 14, 2026 13:22
github-merge-queue
Bot
removed this pull request from the merge queue due to failed status checks
Sep 14, 2026
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
3 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.
🤖 written by Claude (start)
A checkpoint's line now carries
DeletebesideFork, behind a confirmation. Deleting releases the snapshot that checkpoint pins; chats already forked from it keep working.UI only —
CheckpointService/DeleteCheckpointalready existed, so this wires it through the api layer and the fixture backend. Mock checkpoints move tosessionStorage, so a deletion in mock mode survives a reload the way the controller's does.Testing
ENABLE_MOCK_UI=true yarn dev, open a conversation, and save two checkpoints a turn apart.Deleteon the earlier line and confirm. That line goes; the other stays.🤖 written by Claude (end)