Fix: trimming an existing video no longer deletes the original file

Sources/ZoomItMacCore/Capture/RecordingController.swift

@@ -1132,12 +1132,60 @@ final class RecordingController {
         self.clipEditor = editor
         editor.present(tempURL: url, suggestedName: suggestedFilename(), onSave: { [weak self] editedURL in
             self?.clipEditor = nil
-            self?.savePanel(for: editedURL)
+            self?.saveTrimmedClip(editedURL: editedURL, originalURL: url)
         }, onCancel: { [weak self] in
             self?.clipEditor = nil
         })
     }
 
+    /// Action for saving a clip opened from an existing file (the Trim
+    /// workflow). If the editor exported a new temp file, that temp is moved
+    /// into place; if it returned the user's own original (no edits), the
+    /// original is copied so it is preserved — matching Windows ZoomIt, which
+    /// never deletes the source file.
+    enum TrimSaveAction: Equatable { case move, copy }
+
+    static func trimSaveAction(editedURL: URL, originalURL: URL) -> TrimSaveAction {
+        editedURL == originalURL ? .copy : .move
+    }
+
+    /// Saves a clip that was opened from an existing file, always preserving the
+    /// user's original source file.
+    private func saveTrimmedClip(editedURL: URL, originalURL: URL) {
+        let panel = NSSavePanel()
+        panel.nameFieldStringValue = suggestedFilename()
+        panel.allowedContentTypes = [.mpeg4Movie]
+        panel.canCreateDirectories = true
+        NSApp.activate(ignoringOtherApps: true)
+
+        let action = Self.trimSaveAction(editedURL: editedURL, originalURL: originalURL)
+        if panel.runModal() == .OK, let destination = panel.url {
+            if destination != originalURL {
+                try? FileManager.default.removeItem(at: destination)
+            }
+            do {
+                switch action {
+                case .move:
+                    // Move the exported temp file into place; original untouched.
+                    if destination != editedURL {
+                        try FileManager.default.moveItem(at: editedURL, to: destination)
+                    }
+                case .copy:
+                    // No edits: copy the user's original, preserving the source.
+                    if destination != editedURL {
+                        try FileManager.default.copyItem(at: editedURL, to: destination)
+                    }
+                }
+            } catch {
+                let alert = NSAlert(error: error)
+                alert.runModal()
+            }
+        } else if action == .move {
+            // Discard the temp export; never delete the user's original file.
+            try? FileManager.default.removeItem(at: editedURL)
+        }
+    }
+
     private func selectRegion(on display: DisplayDescriptor, completion: @escaping (CGRect?) -> Void) {
         Task { @MainActor in
             do {

Sources/ZoomItMacCore/SelfTest/SelfTestRunner.swift

@@ -52,6 +52,7 @@ public enum SelfTestRunner {
         try testStatusMenuOrderMatchesWindows()
         try testClipTransitionUpdatesOnChange()
         try testWebcamOverlayDragOrigin()
+        try testTrimSavePreservesOriginal()
         try testStaticZoomStaysAtOneX()
         try testPanoramaStitching()
         try testPanoramaTopSeamUsesSingleFramePixels()
@@ -573,6 +574,23 @@ public enum SelfTestRunner {
         try expect(moved == CGPoint(x: 150, y: 130), "Expected dragged origin to track the cursor, got \(moved)")
     }
 
+    /// Trimming an existing video and saving under a new name must NOT delete
+    /// the user's original file (it did, because the source was moved). When no
+    /// edits were made the editor returns the original URL and we copy it;
+    /// otherwise it returns an exported temp file that we move.
+    private static func testTrimSavePreservesOriginal() throws {
+        let original = URL(fileURLWithPath: "/tmp/original.mp4")
+
+        // No edits: editor hands back the original URL -> copy (preserve source).
+        try expect(RecordingController.trimSaveAction(editedURL: original, originalURL: original) == .copy,
+                   "Expected an unedited trim save to copy the original, preserving it")
+
+        // Edited: editor exported a temp file -> move it (original untouched).
+        let exported = URL(fileURLWithPath: "/tmp/ZoomIt-edit-1234.mp4")
+        try expect(RecordingController.trimSaveAction(editedURL: exported, originalURL: original) == .move,
+                   "Expected an edited trim save to move the exported temp file")
+    }
+
     private static func testBreakTimerLayout() throws {
         try expect(BreakTimerLayout.timerText(for: 601) == "10:01", "Expected positive break timer text to format as minutes and seconds")
         try expect(BreakTimerLayout.timerText(for: 0) == "0:00", "Expected zero break timer text")