Repository navigation
Conversation
The recovery toast's Discard button deleted the whole recording folder straight away. Ask first, with the same prompt the library's delete button and the GPUI app's Discard already use. Fixes CapSoftware#2365
AatmanAJ
marked this pull request as ready for review
October 11, 2026 09:03
This branch has not been deployed
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.
Fixes #2365
In the Tauri app (
apps/desktop), the recovery toast's Discard button deleted the whole recording folder as soon as it was clicked. In #2365, that's how a 38-minute recording was lost after recovery kept failing. That recovery failure is #2386, fixed in #2458.Discard now asks first, using the prompt the library's delete button already uses: "Are you sure you want to delete this recording?" The GPUI app already shows this same confirmation before its Discard (
MainWindow::process_incomplete_recording), so the shipping app needs no change.Testing
tsc -p apps/desktop --noEmitpass.dialog:defaultis granted to every window (capabilities/default.json), soask()works in the main window where the toast renders.bun run dev:desktopon macOS 27.0.1:Discarded incomplete recordingline, at the moment of the Yes.NeedsRemux. A force-killedInProgressStudio recording is markedFailed: "Recording crashed"at the next launch (resume_uploads), and the recovery toast skipsFailedrecordings, so it never offers Discard for one. That is existing behaviour, unrelated to this change.
Confidence Score: 5/5
The PR appears safe to merge; no actionable issues were found.Summary
RecoveryToast.tsxnow asks before discarding an incomplete recording.No actionable issues found. Review was based on source inspection; no runtime or visual checks were performed.
Reviews (1) · Last reviewed commit: "fix(desktop): confirm before discarding ..." · Reviewed by Greptile