From b141dd23d7c0a4c0ae31ae8a8698c3c82276f4d6 Mon Sep 17 00:00:00 2001
From: EtienneLescot <215859519+EtienneLescot@users.noreply.github.com>
Date: Sat, 20 Jun 2026 06:15:19 +0000
Subject: [PATCH 1/4] Add export preview diagnostics

---
 src/components/video-editor/VideoEditor.tsx   | 56 +++++++++++++++++++
 src/components/video-editor/VideoPlayback.tsx | 39 ++++++++++++-
 2 files changed, 94 insertions(+), 1 deletion(-)

diff --git a/src/components/video-editor/VideoEditor.tsx b/src/components/video-editor/VideoEditor.tsx
index 445a09060..1d5c92c3d 100644
--- a/src/components/video-editor/VideoEditor.tsx
+++ b/src/components/video-editor/VideoEditor.tsx
@@ -174,6 +174,31 @@ function buildSaveDiagnosticMessage(formatLabel: "GIF" | "Video", reason?: strin
 	return `${formatLabel} export save failed${reason ? `\nReason: ${reason}` : ""}`;
 }
 
+function getPreviewVideoDiagnostics(video: HTMLVideoElement | null) {
+	if (!video) {
+		return { present: false };
+	}
+
+	return {
+		present: true,
+		src: video.currentSrc || video.src,
+		readyState: video.readyState,
+		networkState: video.networkState,
+		error: video.error
+			? {
+					code: video.error.code,
+					message: video.error.message,
+				}
+			: null,
+		currentTime: Number.isFinite(video.currentTime) ? video.currentTime : null,
+		duration: Number.isFinite(video.duration) ? video.duration : null,
+		paused: video.paused,
+		ended: video.ended,
+		videoWidth: video.videoWidth,
+		videoHeight: video.videoHeight,
+	};
+}
+
 const CAPTION_WORD_CHOICES = [1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12] as const;
 
 export default function VideoEditor() {
@@ -1855,6 +1880,31 @@ export default function VideoEditor() {
 			setExportError(null);
 			setExportedFilePath(null);
 
+			const previewBeforeExport = getPreviewVideoDiagnostics(video);
+			console.info("[VideoEditor] export started", {
+				format: settings.format,
+				sourcePath: videoSourcePath ?? videoPath,
+				videoPath,
+				webcamVideoPath,
+				preview: previewBeforeExport,
+				settings: {
+					exportQuality: settings.quality || exportQuality,
+					aspectRatio,
+					padding,
+					borderRadius,
+					shadowIntensity,
+					showBlur,
+					motionBlurAmount,
+					cropRegion,
+					zoomRegions: zoomRegions.length,
+					trimRegions: trimRegions.length,
+					speedRegions: speedRegions.length,
+					annotations: annotationRegions.length,
+					effectiveShowCursor,
+					cursorSize,
+				},
+			});
+
 			try {
 				const wasPlaying = isPlaying;
 				if (wasPlaying) {
@@ -2080,6 +2130,12 @@ export default function VideoEditor() {
 					toast.error(t("errors.exportFailedWithError", { error: message }));
 				}
 			} finally {
+				console.info("[VideoEditor] export finished", {
+					sourcePath: videoSourcePath ?? videoPath,
+					videoPath,
+					previewBefore: previewBeforeExport,
+					previewAfter: getPreviewVideoDiagnostics(videoPlaybackRef.current?.video ?? null),
+				});
 				setIsExporting(false);
 				exporterRef.current = null;
 				// Reset so the next export can reopen the dialog (second export
diff --git a/src/components/video-editor/VideoPlayback.tsx b/src/components/video-editor/VideoPlayback.tsx
index 1b4ad5263..fbb2767ae 100644
--- a/src/components/video-editor/VideoPlayback.tsx
+++ b/src/components/video-editor/VideoPlayback.tsx
@@ -215,6 +215,26 @@ function enableAllPreviewAudioTracks(video: HTMLVideoElement) {
 	}
 }
 
+function getVideoElementDiagnostics(video: HTMLVideoElement) {
+	return {
+		src: video.currentSrc || video.src,
+		readyState: video.readyState,
+		networkState: video.networkState,
+		error: video.error
+			? {
+					code: video.error.code,
+					message: video.error.message,
+				}
+			: null,
+		currentTime: Number.isFinite(video.currentTime) ? video.currentTime : null,
+		duration: Number.isFinite(video.duration) ? video.duration : null,
+		paused: video.paused,
+		ended: video.ended,
+		videoWidth: video.videoWidth,
+		videoHeight: video.videoHeight,
+	};
+}
+
 const VideoPlayback = forwardRef<VideoPlaybackRef, VideoPlaybackProps>(
 	(
 		{
@@ -345,6 +365,16 @@ const VideoPlayback = forwardRef<VideoPlaybackRef, VideoPlaybackProps>(
 		const speedRegionsRef = useRef<SpeedRegion[]>([]);
 		const motionBlurAmountRef = useRef(motionBlurAmount);
 		const cursorOverlayRef = useRef<PixiCursorOverlay | null>(null);
+
+		const logPreviewVideoEvent = useCallback(
+			(event: React.SyntheticEvent<HTMLVideoElement>) => {
+				console.info(`[VideoPlayback] preview video ${event.type}`, {
+					videoPath,
+					diagnostics: getVideoElementDiagnostics(event.currentTarget),
+				});
+			},
+			[videoPath],
+		);
 		const showCursorRef = useRef(showCursor);
 		const cursorSizeRef = useRef(cursorSize);
 		const cursorSmoothingRef = useRef(cursorSmoothing);
@@ -2152,7 +2182,14 @@ const VideoPlayback = forwardRef<VideoPlaybackRef, VideoPlaybackProps>(
 							forceResolveDuration(e.currentTarget);
 						}
 					}}
-					onError={() => onError("Failed to load video")}
+					onEmptied={logPreviewVideoEvent}
+					onStalled={logPreviewVideoEvent}
+					onSuspend={logPreviewVideoEvent}
+					onAbort={logPreviewVideoEvent}
+					onError={(event) => {
+						logPreviewVideoEvent(event);
+						onError("Failed to load video");
+					}}
 				/>
 				{supplementalAudioPath && (
 					<audio ref={supplementalAudioRef} src={supplementalAudioPath} preload="auto" />

From aabdc0d0a036efe0dc7500e7f3f99348a8dd1c8c Mon Sep 17 00:00:00 2001
From: EtienneLescot <etienne.lescot@kilo.ai>
Date: Mon, 22 Jun 2026 09:40:48 +0000
Subject: [PATCH 2/4] Forward renderer console output to main process
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Adds `webContents.on('console-message', …)` forwarding on every
browser window created in `electron/main.ts`. Renderer
`console.info` / `console.warn` / `console.error` calls now show
up in the terminal where `npm run dev` is running, prefixed with
`[renderer INFO|WARNING|ERROR]` so they are easy to grep out from
the surrounding OpenH264 / Wayland noise.

This makes the export-preview diagnostics from PR #11 actually
visible to users following terminal instructions — without it the
`[VideoEditor]` / `[VideoPlayback]` lines only land in DevTools,
which is exactly the gap that caused the first Manjaro run to
produce no usable data (cfinnberg, issue #8 comment 4757223195).

Uses the non-deprecated event-based signature
(`details: { level, message, … }`) with `level` already typed as
"info" | "warning" | "error" | "debug".

Smoke-tested locally with a minimal Electron script: `console.info`,
`console.warn`, and `console.error` from a `data:` URL all
appear on stdout.
---
 electron/main.ts | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/electron/main.ts b/electron/main.ts
index c2d7a4d48..827e7725c 100644
--- a/electron/main.ts
+++ b/electron/main.ts
@@ -126,6 +126,30 @@ if (hasSingleInstanceLock) {
 	app.quit();
 }
 
+// Forward renderer console output to the main process so it appears in the
+// terminal where `npm run dev` is running. Without this, `console.info` /
+// `console.warn` / `console.error` calls in the renderer only show up in
+// DevTools, which is invisible when the user is following terminal
+// instructions (see issue #8 and PR #11).
+app.on("browser-window-created", (_event, window) => {
+	window.webContents.on("console-message", (details) => {
+		// New API: details.level is "info" | "warning" | "error" | "debug".
+		// Skip debug to keep the terminal readable; the user can still inspect
+		// DevTools for that.
+		const { level, message } = details;
+		if (level === "debug") return;
+		const tag = level.toUpperCase();
+		const line = `[renderer ${tag}] ${message}`;
+		if (level === "error") {
+			console.error(line);
+		} else if (level === "warning") {
+			console.warn(line);
+		} else {
+			console.log(line);
+		}
+	});
+});
+
 function isEditorWindow(window: BrowserWindow) {
 	return window.webContents.getURL().includes("windowType=editor");
 }

From b1c5ee5f62e08e081ea5806b89ea709386b3496a Mon Sep 17 00:00:00 2001
From: EtienneLescot <etienne.lescot@users.noreply.github.com>
Date: Mon, 22 Jun 2026 14:22:27 +0200
Subject: [PATCH 3/4] fix(analytics): inline JSON.stringify in
 [VideoEditor]/[VideoPlayback] logs

The console-message event in Electron's webContents only delivers the
first argument as a string in details.message. The original PR #11
passed the diagnostic object as the second argument to console.info,
which got collapsed to [object Object] in the forwarded log line -
defeating the whole point of capturing structured diagnostic data.

This folds the payload into the first argument via template literal +
JSON.stringify, so the structured src/readyState/networkState/error/
dimensions/duration/currentTime reaches stdout through the existing
console-message forwarding.

Also rebase onto current main so the branch keeps moving forward.
---
 src/components/video-editor/VideoEditor.tsx   | 62 ++++++++++---------
 src/components/video-editor/VideoPlayback.tsx | 10 +--
 2 files changed, 39 insertions(+), 33 deletions(-)

diff --git a/src/components/video-editor/VideoEditor.tsx b/src/components/video-editor/VideoEditor.tsx
index 1d5c92c3d..3b7afb360 100644
--- a/src/components/video-editor/VideoEditor.tsx
+++ b/src/components/video-editor/VideoEditor.tsx
@@ -1881,29 +1881,31 @@ export default function VideoEditor() {
 			setExportedFilePath(null);
 
 			const previewBeforeExport = getPreviewVideoDiagnostics(video);
-			console.info("[VideoEditor] export started", {
-				format: settings.format,
-				sourcePath: videoSourcePath ?? videoPath,
-				videoPath,
-				webcamVideoPath,
-				preview: previewBeforeExport,
-				settings: {
-					exportQuality: settings.quality || exportQuality,
-					aspectRatio,
-					padding,
-					borderRadius,
-					shadowIntensity,
-					showBlur,
-					motionBlurAmount,
-					cropRegion,
-					zoomRegions: zoomRegions.length,
-					trimRegions: trimRegions.length,
-					speedRegions: speedRegions.length,
-					annotations: annotationRegions.length,
-					effectiveShowCursor,
-					cursorSize,
-				},
-			});
+			console.info(
+				`[VideoEditor] export started ${JSON.stringify({
+					format: settings.format,
+					sourcePath: videoSourcePath ?? videoPath,
+					videoPath,
+					webcamVideoPath,
+					preview: previewBeforeExport,
+					settings: {
+						exportQuality: settings.quality || exportQuality,
+						aspectRatio,
+						padding,
+						borderRadius,
+						shadowIntensity,
+						showBlur,
+						motionBlurAmount,
+						cropRegion,
+						zoomRegions: zoomRegions.length,
+						trimRegions: trimRegions.length,
+						speedRegions: speedRegions.length,
+						annotations: annotationRegions.length,
+						effectiveShowCursor,
+						cursorSize,
+					},
+				})}`,
+			);
 
 			try {
 				const wasPlaying = isPlaying;
@@ -2130,12 +2132,14 @@ export default function VideoEditor() {
 					toast.error(t("errors.exportFailedWithError", { error: message }));
 				}
 			} finally {
-				console.info("[VideoEditor] export finished", {
-					sourcePath: videoSourcePath ?? videoPath,
-					videoPath,
-					previewBefore: previewBeforeExport,
-					previewAfter: getPreviewVideoDiagnostics(videoPlaybackRef.current?.video ?? null),
-				});
+				console.info(
+					`[VideoEditor] export finished ${JSON.stringify({
+						sourcePath: videoSourcePath ?? videoPath,
+						videoPath,
+						previewBefore: previewBeforeExport,
+						previewAfter: getPreviewVideoDiagnostics(videoPlaybackRef.current?.video ?? null),
+					})}`,
+				);
 				setIsExporting(false);
 				exporterRef.current = null;
 				// Reset so the next export can reopen the dialog (second export
diff --git a/src/components/video-editor/VideoPlayback.tsx b/src/components/video-editor/VideoPlayback.tsx
index fbb2767ae..64967184f 100644
--- a/src/components/video-editor/VideoPlayback.tsx
+++ b/src/components/video-editor/VideoPlayback.tsx
@@ -368,10 +368,12 @@ const VideoPlayback = forwardRef<VideoPlaybackRef, VideoPlaybackProps>(
 
 		const logPreviewVideoEvent = useCallback(
 			(event: React.SyntheticEvent<HTMLVideoElement>) => {
-				console.info(`[VideoPlayback] preview video ${event.type}`, {
-					videoPath,
-					diagnostics: getVideoElementDiagnostics(event.currentTarget),
-				});
+				console.info(
+					`[VideoPlayback] preview video ${event.type} ${JSON.stringify({
+						videoPath,
+						diagnostics: getVideoElementDiagnostics(event.currentTarget),
+					})}`,
+				);
 			},
 			[videoPath],
 		);

From 306570579c94e56d7cd09d42d5faebed942baaeb Mon Sep 17 00:00:00 2001
From: EtienneLescot <etiennelescot@gmail.com>
Date: Thu, 25 Jun 2026 20:18:26 +0200
Subject: [PATCH 4/4] [codex] trap uncaught errors to surface stack for #8
 .geometry null deref
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Mirrors the cfinnberg-reported failure log by capturing filename, lineno,
colno, and stack from window 'error' and 'unhandledrejection' events so the
next repro run produces a stack trace, not just the bare TypeError message.

One-shot diagnostic — to be removed once the .geometry null deref is pinned
and fixed.
---
 src/main.tsx | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/src/main.tsx b/src/main.tsx
index 365bdc7b2..ac448233e 100644
--- a/src/main.tsx
+++ b/src/main.tsx
@@ -4,6 +4,27 @@ import App from "./App.tsx";
 import { I18nProvider } from "./contexts/I18nContext";
 import "./index.css";
 
+// === One-shot uncaught trap — capture stack traces for the .geometry null deref in #8.
+// Delete after the diagnostic run that closes out the issue's root cause.
+const __errTrap = (tag: string) => (e: ErrorEvent | PromiseRejectionEvent) => {
+	const isRej = "reason" in e;
+	const err = isRej ? (e as PromiseRejectionEvent).reason : (e as ErrorEvent).error;
+	const evt = e as ErrorEvent;
+	console.error(
+		`[VideoEditor] ${tag}`,
+		JSON.stringify({
+			message: err?.message ?? String(err),
+			source: isRej ? null : evt.filename,
+			lineno: isRej ? null : evt.lineno,
+			colno: isRej ? null : evt.colno,
+			stack: err?.stack ?? null,
+		}),
+	);
+};
+window.addEventListener("error", __errTrap("uncaught"));
+window.addEventListener("unhandledrejection", __errTrap("unhandledrejection"));
+// === end trap ===
+
 const windowType = new URLSearchParams(window.location.search).get("windowType") || "";
 if (
 	windowType === "hud-overlay" ||