fix(mcp): let the editor read an imported .oga, keep a created project's id when its checkpoint fails, check an existing video for private content

Part of #1180

Author: EtienneLescot

electron/ipc/handlers.ts

@@ -274,6 +274,7 @@ const ALLOWED_IMPORT_AUDIO_EXTENSIONS = new Set([
 	".aac",
 	".flac",
 	".ogg",
+	".oga",
 	".opus",
 ]);
 

electron/mcp/media-tools.ts

@@ -443,11 +443,17 @@ export function createMediaTools(deps: MediaToolDeps): MediaTools {
 
 	/** A fresh project's checkpoint, so a client can edit it at once and take it back in one step. */
 	async function checkpointOf(projectId: string): Promise<string | null> {
-		const checkpoint = await deps.runTool("createCheckpoint", { projectId });
-		const checkpointText = checkpoint.content[0]?.type === "text" ? checkpoint.content[0].text : "";
-		return checkpoint.isError
-			? null
-			: ((JSON.parse(checkpointText) as { checkpointId?: string }).checkpointId ?? null);
+		// Never fails the call: the project exists either way, and its id is the answer.
+		try {
+			const checkpoint = await deps.runTool("createCheckpoint", { projectId });
+			const checkpointText =
+				checkpoint.content[0]?.type === "text" ? checkpoint.content[0].text : "";
+			return checkpoint.isError
+				? null
+				: ((JSON.parse(checkpointText) as { checkpointId?: string }).checkpointId ?? null);
+		} catch {
+			return null;
+		}
 	}
 
 	/** Stops the take, then hands back its project with a checkpoint of it. */
@@ -838,8 +844,15 @@ export function createMediaTools(deps: MediaToolDeps): MediaTools {
 				}
 				const title = args.title?.trim();
 				if (args.videoPath === undefined) {
-					const created = await deps.documents.createProject(title ?? "");
-					const projectId = created.project.id;
+					let projectId: string;
+					try {
+						projectId = (await deps.documents.createProject(title ?? "")).project.id;
+					} catch (error) {
+						return text(
+							`No project was created: ${error instanceof Error ? error.message : String(error)}`,
+							true,
+						);
+					}
 					return text(JSON.stringify({ projectId, checkpointId: await checkpointOf(projectId) }));
 				}
 				const videoPath = args.videoPath;

skills/openscreen-demo/SKILL.md

@@ -68,7 +68,7 @@ Write a storyboard and show it to the user in a few lines, then carry on unless
 - **Real mouse** (computer-use, or any tool that moves the actual pointer): OpenScreen records the pointer, draws a clean cursor and already zooms on the clicks (see *Zoom on the action* below). If your mouse tool cannot click in the app's window (some only read browsers, for one), drive it from a script instead.
 - **A script** (Playwright, CDP): the screen pointer never moves, so the take has no cursor and no zoom. In the script, log every action as it runs: `at` is `Date.now()`, `x` and `y` are the action's position as fractions of the window, `click` is true for a click. Hand the log to `setPointerTrack` after `stopRecording`; its description gives the formula from a page's coordinates. A script also gives the cleanest take: same timing on every run, no hesitation.
 
-**A video you already have** needs no recording: `createProject` with its absolute `videoPath` lays it out as a project, and returns a `projectId` and a `checkpointId` as `stopRecording` does. Then go on from step 6.
+**A video you already have** needs no recording: `createProject` with its absolute `videoPath` lays it out as a project, and returns a `projectId` and a `checkpointId` as `stopRecording` does. Check it as a take in step 5, its `videoPath` included, then go on from step 6.
 
 ## 5. Check the take