Repository navigation
Conversation
Each archived row gets a Delete button next to Unarchive. It opens the
same confirmation as the sidebar ("Delete session?") and deletes only
that chat; Delete all keeps its own dialog. One delete runs at a time.
Also records the T3 watch review through t3code@67fdc53823.
Inspired by: pingdotgg/t3code@c864b1c1e1
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.
What T3 added (pingdotgg/t3code@c864b1c1e1): the archive page has a Delete action on each archived thread, next to Unarchive, so you no longer have to restore a thread just to delete it.
How Godspeed does it: Settings → Archived already had Unarchive on each row and a confirmed Delete all. Each row now also has a Delete button, styled like Unarchive (it turns red on hover). It opens the same card as the sidebar's single delete: "Delete session?", "“<title>” will be permanently deleted. This can’t be undone.", and a Delete button. It deletes only that chat with
Mutate deleteChat. Delete all keeps its own dialog. Both share one confirm/task (DeleteConfirm), so only one delete runs at a time, and while it runs the row Delete buttons are dimmed. Escape closes either dialog before it can close Settings. Like Delete all, this doesn't remove a worktree; the chat's checkout is left alone.Also records the T3 watch review through t3code@67fdc53823: ledger rows in
T3-WATCH.md, plus a Done row inT3-GAPS.md.Screenshots: not available (the agent runs headless).
Tests run
cargo build -p godspeed: okcargo check --workspace --tests: okcargo test -p zeron-ui --lib: 2179 passed. New tests:deleting_one_row_deletes_only_that_chatanddelete_copy_names_one_row_and_counts_delete_all.Please check by eye
Note:
T3-WATCH.mdmay conflict with open PR #94 (both add rows at the top of the ledger). Keep both sets of rows. The marker should be the newer one (67fdc53823).