@hyperframes/studio-server 0.8.116 → 0.8.117

package/dist/index.js

@@ -4890,7 +4890,7 @@ function registerWaveformRoutes(api, adapter) {
 
 // src/routes/freezeFrame.ts
 import { execFile } from "child_process";
-import { existsSync as existsSync8, readFileSync as readFileSync11, statSync as statSync8 } from "fs";
+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 { findFfBinary as findFfBinary3 } from "@hyperframes/parsers/ff-binaries";
 import { replaceFileAtomically as replaceFileAtomically3 } from "@hyperframes/core/atomic-file";
@@ -4918,7 +4918,7 @@ function freezeFrameMediaTime(input) {
   return input.mediaStart + sourceTimeAt(rate, Math.max(0, input.playhead - input.clipStart));
 }
 function freezeExtractArgs(src, mediaTime, output) {
-  return ["-n", "-ss", String(round3(mediaTime)), "-i", src, "-frames:v", "1", output];
+  return ["-y", "-ss", String(round3(mediaTime)), "-i", src, "-frames:v", "1", output];
 }
 var randomStillToken = () => randomBytes(4).toString("hex");
 function freezeStillFileName(clipId, playhead, token = randomStillToken()) {
@@ -5091,6 +5091,15 @@ function readExpected(absPath, expectedVersion) {
   }
   return fileContentVersion(content) === expectedVersion ? { content } : { error: "file conflict", status: 409 };
 }
+function claimStillNameBeforeExtracting(path) {
+  try {
+    closeSync3(openSync3(path, "wx"));
+    return true;
+  } catch (error) {
+    if (error.code === "EEXIST") return false;
+    throw error;
+  }
+}
 async function extractStill(projectDir, absPath, source, playhead, tools) {
   const fileDir = dirname5(absPath);
   const mediaPath = pinWithinProject(projectDir, relative4(projectDir, join13(fileDir, source.src)));
@@ -5102,17 +5111,18 @@ async function extractStill(projectDir, absPath, source, playhead, tools) {
   if (!imagePath || dirname5(imagePath) !== freezeDir) {
     return { error: `forbidden freeze path: ${fileName}`, status: 403 };
   }
-  if (existsSync8(imagePath))
+  if (!claimStillNameBeforeExtracting(imagePath))
     return { error: `freeze still already exists: ${fileName}`, status: 409 };
   const extracted = await tools.extract(freezeExtractArgs(mediaPath, source.mediaTime, imagePath));
-  if (!extracted.ok) {
-    return {
-      error: `Could not extract the frame: ${extracted.error ?? "ffmpeg failed"}`,
-      status: 500
-    };
+  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 depth = relative4(projectDir, fileDir).split(sep2).filter(Boolean).length;
-  return { imageSrc: `${"../".repeat(depth)}${FREEZE_DIR.join("/")}/${fileName}` };
+  const stillPath = `${FREEZE_DIR.join("/")}/${fileName}`;
+  return { imageSrc: `${"../".repeat(depth)}${stillPath}`, stillPath, imagePath };
 }
 function writeFolded(projectDir, absPath, path, before, after, options) {
   if (readFileSync11(absPath, "utf-8") !== before) {
@@ -5146,17 +5156,21 @@ function registerFreezeFrameRoutes(api, adapter, extract = ffmpegExtractor, stil
       stillToken
     });
     if ("error" in still) return c.json({ error: still.error }, still.status);
-    const { imageSrc } = still;
+    const { imageSrc, stillPath, imagePath } = still;
     const folded = applyFreezeFrameToHtml(before, {
       target: body.target,
       playhead: body.playhead,
       imageSrc
     });
-    if (!folded) return c.json({ error: "Freeze target was not found in the file" }, 400);
+    const failAndRemoveStill = (error, status) => {
+      rmSync5(imagePath, { force: true });
+      return c.json({ error }, status);
+    };
+    if (!folded) return failAndRemoveStill("Freeze target was not found in the file", 400);
     const written = writeFolded(project.dir, absPath, body.path, before, folded.html, {
       token: body.transactionToken ?? c.req.header("X-Hyperframes-Write-Token")
     });
-    if ("error" in written) return c.json({ error: written.error }, written.status);
+    if ("error" in written) return failAndRemoveStill(written.error, written.status);
     const { version, writeToken, backupPath } = written;
     return c.json({
       ok: true,
@@ -5167,13 +5181,14 @@ function registerFreezeFrameRoutes(api, adapter, extract = ffmpegExtractor, stil
       writeToken,
       backupPath,
       freezeId: folded.freezeId,
-      imageSrc
+      imageSrc,
+      stillPath
     });
   });
 }
 
 // src/routes/loudness.ts
-import { existsSync as existsSync9 } from "fs";
+import { existsSync as existsSync8 } from "fs";
 var defaultMeasure = (file, window) => measureAudio(requiredFfmpeg(), file, window);
 var InvalidField = class extends Error {
 };
@@ -5224,7 +5239,7 @@ function registerLoudnessRoutes(api, adapter, measure = defaultMeasure) {
     } catch (error) {
       return c.json({ error: error instanceof Error ? error.message : String(error) }, 400);
     }
-    if (!existsSync9(file)) return c.json({ error: "file not found" }, 404);
+    if (!existsSync8(file)) return c.json({ error: "file not found" }, 404);
     try {
       const measurement = await measure(file, request.window);
       const plan = absoluteLoudnessPlan(
@@ -5239,7 +5254,7 @@ function registerLoudnessRoutes(api, adapter, measure = defaultMeasure) {
 }
 
 // src/routes/peaks.ts
-import { existsSync as existsSync10, readFileSync as readFileSync12, statSync as statSync9 } from "fs";
+import { existsSync as existsSync9, readFileSync as readFileSync12, statSync as statSync9 } from "fs";
 import { join as join14, resolve as resolve6 } from "path";
 import { isWithinProjectRoot as isWithinProjectRoot3 } from "@hyperframes/parsers/asset-resolution";
 
@@ -5357,7 +5372,7 @@ function decodeOnce(cachePath, mediaPath, decode) {
 }
 function readCachedBins(cacheDir, cachePath) {
   try {
-    if (!isWaveformCacheDirectory(cacheDir) || !existsSync10(cachePath)) return null;
+    if (!isWaveformCacheDirectory(cacheDir) || !existsSync9(cachePath)) return null;
     const parsed = JSON.parse(readFileSync12(cachePath, "utf-8"));
     return Array.isArray(parsed) && parsed.every((n) => typeof n === "number") ? parsed : null;
   } catch {
@@ -5393,7 +5408,7 @@ function registerPeakRoutes(api, adapter, decode = decodePeakMap) {
 }
 
 // src/routes/fonts.ts
-import { closeSync as closeSync3, constants, fstatSync as fstatSync3, openSync as openSync3, readSync } from "fs";
+import { closeSync as closeSync4, constants, fstatSync as fstatSync3, openSync as openSync4, readSync } from "fs";
 import {
   collectFontFileEntries,
   fontDirectories,
@@ -5500,7 +5515,7 @@ function registerFontRoutes(api) {
     if (!located) return c.json({ error: "font not found" }, 404);
     let fd;
     try {
-      fd = openSync3(located.path, constants.O_RDONLY | constants.O_NOFOLLOW);
+      fd = openSync4(located.path, constants.O_RDONLY | constants.O_NOFOLLOW);
     } catch {
       return c.json({ error: "font file not accessible" }, 404);
     }
@@ -5522,7 +5537,7 @@ function registerFontRoutes(api) {
     } catch {
       return c.json({ error: "failed to read font file" }, 500);
     } finally {
-      closeSync3(fd);
+      closeSync4(fd);
     }
   });
 }
@@ -5658,7 +5673,7 @@ function registerSelectionRoutes(api, adapter) {
 
 // src/routes/media.ts
 import { streamSSE as streamSSE2 } from "hono/streaming";
-import { existsSync as existsSync11 } from "fs";
+import { existsSync as existsSync10 } from "fs";
 import { basename as basename2, dirname as dirname6, extname, join as join15, posix } from "path";
 var VIDEO_EXTENSIONS = /* @__PURE__ */ new Set([
   ".mp4",
@@ -5696,7 +5711,7 @@ function uniqueAssetPath(projectDir, assetPath, taken) {
   const ext = extname(assetPath);
   const withoutExt = assetPath.slice(0, -ext.length);
   let candidate = assetPath;
-  for (let index = 2; taken.has(candidate) || existsSync11(join15(projectDir, candidate)); index++) {
+  for (let index = 2; taken.has(candidate) || existsSync10(join15(projectDir, candidate)); index++) {
     candidate = `${withoutExt}-${index}${ext}`;
   }
   return candidate;
@@ -5757,7 +5772,7 @@ function registerMediaRoutes(api, adapter, options = {}) {
     }
     const filePath = resolveWithinProject(project.dir, assetPath);
     if (!filePath) return c.json({ error: "forbidden" }, 403);
-    if (!existsSync11(filePath)) return c.json({ error: "media not found" }, 404);
+    if (!existsSync10(filePath)) return c.json({ error: "media not found" }, 404);
     return c.json({ path: assetPath, metadata: await readMediaMetadata(filePath) });
   });
   api.post(
@@ -5779,7 +5794,7 @@ function registerMediaRoutes(api, adapter, options = {}) {
       }
       const inputPath = resolveWithinProject(project.dir, inputAssetPath);
       if (!inputPath) return c.json({ error: "forbidden" }, 403);
-      if (!existsSync11(inputPath)) return c.json({ error: "input media not found" }, 404);
+      if (!existsSync10(inputPath)) return c.json({ error: "input media not found" }, 404);
       const inputIsVideo = isVideoPath(inputAssetPath);
       const inputIsImage = isImagePath(inputAssetPath);
       if (!inputIsVideo && !inputIsImage) {
@@ -5873,12 +5888,12 @@ function registerMediaRoutes(api, adapter, options = {}) {
 }
 
 // src/routes/globalAssets.ts
-import { existsSync as existsSync12, readFileSync as readFileSync13 } from "fs";
+import { existsSync as existsSync11, readFileSync as readFileSync13 } from "fs";
 import { homedir } from "os";
 import { join as join16 } from "path";
 function readGlobalAssets(home = homedir()) {
   const manifestPath = join16(home, ".media", "manifest.jsonl");
-  if (!existsSync12(manifestPath)) return [];
+  if (!existsSync11(manifestPath)) return [];
   const out = [];
   for (const line of readFileSync13(manifestPath, "utf8").split("\n")) {
     if (!line.trim()) continue;
@@ -6132,7 +6147,7 @@ function numericAttribute(element, name) {
 
 // src/helpers/previewDocumentStore.ts
 import { createHash as createHash6 } from "crypto";
-import { readFileSync as readFileSync14, readdirSync as readdirSync5, rmSync as rmSync5 } from "fs";
+import { readFileSync as readFileSync14, readdirSync as readdirSync5, rmSync as rmSync6 } from "fs";
 import { join as join17 } from "path";
 import { replaceFileAtomically as replaceFileAtomically4 } from "@hyperframes/core/atomic-file";
 function createPreviewDocumentStore(projectDir, salt) {
@@ -6153,7 +6168,7 @@ ${key}`).digest("hex").slice(0, 32)}.html`);
         const file = fileFor(key);
         replaceFileAtomically4(file, html, 420);
         for (const name of readdirSync5(dir)) {
-          if (join17(dir, name) !== file) rmSync5(join17(dir, name), { force: true });
+          if (join17(dir, name) !== file) rmSync6(join17(dir, name), { force: true });
         }
       } catch {
       }