Merge pull request #205 from EtienneLescot/fix/issue-197-windows-paths

Fix Windows cursor telemetry path resolution

Author: siddharthvaddem

src/components/video-editor/VideoEditor.tsx

@@ -419,15 +419,17 @@ export default function VideoEditor() {
 		let mounted = true;
 
 		async function loadCursorTelemetry() {
-			if (!videoPath) {
+			const sourcePath = videoSourcePath ?? (videoPath ? fromFileUrl(videoPath) : null);
+
+			if (!sourcePath) {
 				if (mounted) {
 					setCursorTelemetry([]);
 				}
 				return;
 			}
 
 			try {
-				const result = await window.electronAPI.getCursorTelemetry(fromFileUrl(videoPath));
+				const result = await window.electronAPI.getCursorTelemetry(sourcePath);
 				if (mounted) {
 					setCursorTelemetry(result.success ? result.samples : []);
 				}
@@ -444,7 +446,7 @@ export default function VideoEditor() {
 		return () => {
 			mounted = false;
 		};
-	}, [videoPath]);
+	}, [videoPath, videoSourcePath]);
 
 	function togglePlayPause() {
 		const playback = videoPlaybackRef.current;

src/components/video-editor/projectPersistence.ts

@@ -61,9 +61,9 @@ function clamp(value: number, min: number, max: number) {
 export function toFileUrl(filePath: string): string {
 	const normalized = filePath.replace(/\\/g, "/");
 	if (normalized.match(/^[a-zA-Z]:/)) {
-		return `file:///${normalized}`;
+		return `file:///${encodeURI(normalized)}`;
 	}
-	return `file://${normalized}`;
+	return `file://${encodeURI(normalized)}`;
 }
 
 export function fromFileUrl(fileUrl: string): string {
@@ -73,9 +73,20 @@ export function fromFileUrl(fileUrl: string): string {
 
 	try {
 		const url = new URL(fileUrl);
-		return decodeURIComponent(url.pathname);
+		const pathname = decodeURIComponent(url.pathname);
+
+		if (url.host && url.host !== "localhost") {
+			return `//${url.host}${pathname}`;
+		}
+
+		if (/^\/[a-zA-Z]:/.test(pathname)) {
+			return pathname.slice(1);
+		}
+
+		return pathname;
 	} catch {
-		return fileUrl.replace(/^file:\/\//, "");
+		const fallbackPath = decodeURIComponent(fileUrl.replace(/^file:\/\//, ""));
+		return fallbackPath.replace(/^\/([a-zA-Z]:)/, "$1");
 	}
 }