@hyperframes/studio-server 0.8.118 → 0.8.119

package/dist/index.js

@@ -4891,10 +4891,14 @@ function registerWaveformRoutes(api, adapter) {
 
 // src/routes/freezeFrame.ts
 import { execFile } from "child_process";
-import { closeSync as closeSync3, openSync as openSync3, readFileSync as readFileSync11, rmSync as rmSync5, statSync as statSync8 } from "fs";
-import { dirname as dirname5, join as join13, relative as relative4, sep as sep2 } from "path";
+import { readFileSync as readFileSync11, rmSync as rmSync5, statSync as statSync8 } from "fs";
+import { basename as basename2, dirname as dirname5, join as join13, relative as relative4, sep as sep2 } from "path";
 import { findFfBinary as findFfBinary3 } from "@hyperframes/parsers/ff-binaries";
-import { replaceFileAtomically as replaceFileAtomically3 } from "@hyperframes/core/atomic-file";
+import {
+  atomicTempPath,
+  createFileAtomically as createFileAtomically2,
+  replaceFileAtomically as replaceFileAtomically3
+} from "@hyperframes/core/atomic-file";
 
 // src/helpers/freezeFrame.ts
 import { createHash as createHash5, randomBytes } from "crypto";
@@ -4919,7 +4923,22 @@ function freezeFrameMediaTime(input) {
   return input.mediaStart + sourceTimeAt(rate, Math.max(0, input.playhead - input.clipStart));
 }
 function freezeExtractArgs(src, mediaTime, output) {
-  return ["-y", "-ss", String(round3(mediaTime)), "-i", src, "-frames:v", "1", output];
+  return [
+    "-y",
+    "-ss",
+    String(round3(mediaTime)),
+    "-i",
+    src,
+    "-frames:v",
+    "1",
+    "-c:v",
+    "png",
+    "-f",
+    "image2",
+    "-update",
+    "1",
+    output
+  ];
 }
 var randomStillToken = () => randomBytes(4).toString("hex");
 function freezeStillFileName(clipId, playhead, token = randomStillToken()) {
@@ -5092,15 +5111,38 @@ function readExpected(absPath, expectedVersion) {
   }
   return fileContentVersion(content) === expectedVersion ? { content } : { error: "file conflict", status: 409 };
 }
-function claimStillNameBeforeExtracting(path) {
+function readFrame(path) {
+  try {
+    return readFileSync11(path);
+  } catch (error) {
+    if (error.code === "ENOENT") return null;
+    throw error;
+  }
+}
+function publishStill(path, frame) {
   try {
-    closeSync3(openSync3(path, "wx"));
+    createFileAtomically2(path, frame);
     return true;
   } catch (error) {
     if (error.code === "EEXIST") return false;
     throw error;
   }
 }
+async function extractThenPublishWhole(extract, mediaPath, mediaTime, imagePath) {
+  const unseenByHistory = atomicTempPath(imagePath);
+  try {
+    const extracted = await extract(freezeExtractArgs(mediaPath, mediaTime, unseenByHistory));
+    const frame = extracted.ok ? readFrame(unseenByHistory) : null;
+    if (!frame?.length) {
+      const reason = extracted.ok ? "no frame at this time" : extracted.error ?? "ffmpeg failed";
+      return { error: `Could not extract the frame: ${reason}`, status: 500 };
+    }
+    if (publishStill(imagePath, frame)) return null;
+    return { error: `freeze still already exists: ${basename2(imagePath)}`, status: 409 };
+  } finally {
+    rmSync5(unseenByHistory, { force: true });
+  }
+}
 async function extractStill(projectDir, absPath, source, playhead, tools) {
   const fileDir = dirname5(absPath);
   const mediaPath = pinWithinProject(projectDir, relative4(projectDir, join13(fileDir, source.src)));
@@ -5112,15 +5154,13 @@ async function extractStill(projectDir, absPath, source, playhead, tools) {
   if (!imagePath || dirname5(imagePath) !== freezeDir) {
     return { error: `forbidden freeze path: ${fileName}`, status: 403 };
   }
-  if (!claimStillNameBeforeExtracting(imagePath))
-    return { error: `freeze still already exists: ${fileName}`, status: 409 };
-  const extracted = await tools.extract(freezeExtractArgs(mediaPath, source.mediaTime, imagePath));
-  const partialOrNoFrame = !extracted.ok || statSync8(imagePath).size === 0;
-  if (partialOrNoFrame) {
-    rmSync5(imagePath, { force: true });
-    const reason = extracted.ok ? "no frame at this time" : extracted.error ?? "ffmpeg failed";
-    return { error: `Could not extract the frame: ${reason}`, status: 500 };
-  }
+  const failed = await extractThenPublishWhole(
+    tools.extract,
+    mediaPath,
+    source.mediaTime,
+    imagePath
+  );
+  if (failed) return failed;
   const depth = relative4(projectDir, fileDir).split(sep2).filter(Boolean).length;
   const stillPath = `${FREEZE_DIR.join("/")}/${fileName}`;
   return { imageSrc: `${"../".repeat(depth)}${stillPath}`, stillPath, imagePath };
@@ -5409,7 +5449,7 @@ function registerPeakRoutes(api, adapter, decode = decodePeakMap) {
 }
 
 // src/routes/fonts.ts
-import { closeSync as closeSync4, constants, fstatSync as fstatSync3, openSync as openSync4, readSync } from "fs";
+import { closeSync as closeSync3, constants, fstatSync as fstatSync3, openSync as openSync3, readSync } from "fs";
 import {
   collectFontFileEntries,
   fontDirectories,
@@ -5516,7 +5556,7 @@ function registerFontRoutes(api) {
     if (!located) return c.json({ error: "font not found" }, 404);
     let fd;
     try {
-      fd = openSync4(located.path, constants.O_RDONLY | constants.O_NOFOLLOW);
+      fd = openSync3(located.path, constants.O_RDONLY | constants.O_NOFOLLOW);
     } catch {
       return c.json({ error: "font file not accessible" }, 404);
     }
@@ -5538,7 +5578,7 @@ function registerFontRoutes(api) {
     } catch {
       return c.json({ error: "failed to read font file" }, 500);
     } finally {
-      closeSync4(fd);
+      closeSync3(fd);
     }
   });
 }
@@ -5675,7 +5715,7 @@ function registerSelectionRoutes(api, adapter) {
 // src/routes/media.ts
 import { streamSSE as streamSSE2 } from "hono/streaming";
 import { existsSync as existsSync10 } from "fs";
-import { basename as basename2, dirname as dirname6, extname, join as join15, posix } from "path";
+import { basename as basename3, dirname as dirname6, extname, join as join15, posix } from "path";
 var VIDEO_EXTENSIONS = /* @__PURE__ */ new Set([
   ".mp4",
   ".mov",
@@ -5705,7 +5745,7 @@ function containsNullByte(path) {
   return path.includes("\0");
 }
 function slugFileBase(path) {
-  const name = basename2(path, extname(path)).replace(/[^a-zA-Z0-9._-]+/g, "-").replace(/^-+|-+$/g, "");
+  const name = basename3(path, extname(path)).replace(/[^a-zA-Z0-9._-]+/g, "-").replace(/^-+|-+$/g, "");
   return name || "media";
 }
 function uniqueAssetPath(projectDir, assetPath, taken) {