fix: prompt for unsaved changes in new projects

src/components/video-editor/VideoEditor.test.ts

@@ -0,0 +1,26 @@
+import { describe, expect, it } from "vitest";
+import { hasUnsavedProjectChanges } from "./VideoEditor";
+
+describe("hasUnsavedProjectChanges", () => {
+	it("treats a new unsaved project with edits as dirty", () => {
+		expect(hasUnsavedProjectChanges('{"editor":{"trimRegions":[1]}}', null)).toBe(true);
+	});
+
+	it("treats a saved project with unchanged snapshot as clean", () => {
+		const snapshot = '{"editor":{"trimRegions":[1]}}';
+		expect(hasUnsavedProjectChanges(snapshot, snapshot)).toBe(false);
+	});
+
+	it("treats a saved project with changed snapshot as dirty", () => {
+		expect(
+			hasUnsavedProjectChanges(
+				'{"editor":{"trimRegions":[1,2]}}',
+				'{"editor":{"trimRegions":[1]}}',
+			),
+		).toBe(true);
+	});
+
+	it("treats missing current snapshot as clean", () => {
+		expect(hasUnsavedProjectChanges(null, null)).toBe(false);
+	});
+});

src/components/video-editor/VideoEditor.tsx

@@ -60,6 +60,21 @@ import {
 } from "./types";
 import VideoPlayback, { VideoPlaybackRef } from "./VideoPlayback";
 
+export function hasUnsavedProjectChanges(
+	currentProjectSnapshot: string | null,
+	lastSavedSnapshot: string | null,
+): boolean {
+	if (!currentProjectSnapshot) {
+		return false;
+	}
+
+	if (lastSavedSnapshot === null) {
+		return true;
+	}
+
+	return currentProjectSnapshot !== lastSavedSnapshot;
+}
+
 export default function VideoEditor() {
 	const {
 		state: editorState,
@@ -295,12 +310,7 @@ export default function VideoEditor() {
 		gifSizePreset,
 	]);
 
-	const hasUnsavedChanges = Boolean(
-		currentProjectPath &&
-			currentProjectSnapshot &&
-			lastSavedSnapshot &&
-			currentProjectSnapshot !== lastSavedSnapshot,
-	);
+	const hasUnsavedChanges = hasUnsavedProjectChanges(currentProjectSnapshot, lastSavedSnapshot);
 
 	useEffect(() => {
 		async function loadInitialData() {