@hyperframes/studio-server 0.8.40 → 0.8.42

package/dist/index.js

@@ -1150,7 +1150,7 @@ function writeFileWithReceipt(c, filePath, absPath, html) {
 }
 function writeMutationResult(c, projectDir, filePath, absPath, html) {
   const backup = snapshotBeforeWrite(projectDir, absPath);
-  if (backup.error) console.warn(`Failed to create backup for ${filePath}: ${backup.error}`);
+  if (backup.error) return c.json({ error: `backup failed: ${backup.error}` }, 500);
   const { version } = writeFileWithReceipt(c, filePath, absPath, html);
   return { backupPath: backupPathForResponse(projectDir, backup.backupPath), version };
 }
@@ -1158,7 +1158,9 @@ function writeIfChanged(c, projectDir, filePath, absPath, original, next) {
   if (next === original) {
     return c.json({ ok: true, changed: false, content: original, path: filePath });
   }
-  const { backupPath } = writeMutationResult(c, projectDir, filePath, absPath, next);
+  const mutationResult = writeMutationResult(c, projectDir, filePath, absPath, next);
+  if (mutationResult instanceof Response) return mutationResult;
+  const { backupPath } = mutationResult;
   return c.json({
     ok: true,
     changed: true,
@@ -1573,13 +1575,15 @@ async function applyGsapMutations(c, res, mutations) {
     return c.json({ error: "file changed during GSAP mutation", conflict: true }, 409);
   }
   if (changed) {
-    backupPath = writeMutationResult(
+    const mutationResult = writeMutationResult(
       c,
       res.project.dir,
       res.filePath,
       res.absPath,
       newHtml
-    ).backupPath;
+    );
+    if (mutationResult instanceof Response) return mutationResult;
+    backupPath = mutationResult.backupPath;
   }
   const responsePayload = {
     ok: true,
@@ -2476,8 +2480,7 @@ function registerFileRoutes(api, adapter) {
           );
         }
         backup = snapshotBeforeWrite(res.project.dir, res.absPath);
-        if (backup.error)
-          console.warn(`Failed to create backup for ${res.filePath}: ${backup.error}`);
+        if (backup.error) return c.json({ error: `backup failed: ${backup.error}` }, 500);
         ftruncateSync(fd, 0);
         writeSync(fd, body, 0, body.length, 0);
       } finally {
@@ -2516,7 +2519,7 @@ function registerFileRoutes(api, adapter) {
     if ("error" in res) return res.error;
     const stat = statSync2(res.absPath);
     const backup = snapshotBeforeWrite(res.project.dir, res.absPath);
-    if (backup.error) console.warn(`Failed to create backup for ${res.filePath}: ${backup.error}`);
+    if (backup.error) return c.json({ error: `backup failed: ${backup.error}` }, 500);
     if (stat.isDirectory()) {
       rmSync3(res.absPath, { recursive: true });
     } else {
@@ -2786,13 +2789,15 @@ function registerFileRoutes(api, adapter) {
         version: version2
       });
     }
-    const { version, backupPath } = writeMutationResult(
+    const mutationResult = writeMutationResult(
       c,
       ctx.project.dir,
       ctx.filePath,
       ctx.absPath,
       result.html
     );
+    if (mutationResult instanceof Response) return mutationResult;
+    const { version, backupPath } = mutationResult;
     c.header("ETag", version);
     return c.json({
       ok: true,
@@ -2839,13 +2844,15 @@ function registerFileRoutes(api, adapter) {
         version: version2
       });
     }
-    const { backupPath, version } = writeMutationResult(
+    const mutationResult = writeMutationResult(
       c,
       ctx.project.dir,
       ctx.filePath,
       ctx.absPath,
       patched
     );
+    if (mutationResult instanceof Response) return mutationResult;
+    const { backupPath, version } = mutationResult;
     c.header("ETag", version);
     return c.json({
       ok: true,
@@ -2940,13 +2947,15 @@ function registerFileRoutes(api, adapter) {
         result.error === "grouped elements must share a single parent" ? 422 : 400
       );
     }
-    const { backupPath } = writeMutationResult(
+    const mutationResult = writeMutationResult(
       c,
       ctx.project.dir,
       ctx.filePath,
       ctx.absPath,
       result.html
     );
+    if (mutationResult instanceof Response) return mutationResult;
+    const { backupPath } = mutationResult;
     return c.json({
       ok: true,
       changed: true,
@@ -3692,7 +3701,8 @@ ${runtimeTag}`;
     if (!compFile || !existsSync6(compFile) || !statSync3(compFile).isFile()) {
       return c.text("not found", 404);
     }
-    const etag = `"comp:v2:${compPath}:${signature}${variablesEtagSalt(vars.raw)}"`;
+    const compPathHash = createHash3("sha1").update(compPath).digest("hex");
+    const etag = `"comp:v2:${compPathHash}:${signature}${variablesEtagSalt(vars.raw)}"`;
     const ifNoneMatch = c.req.header("If-None-Match");
     if (ifNoneMatch === etag) {
       return new Response(null, {
@@ -3878,6 +3888,14 @@ import { join as join10 } from "path";
 import { VALID_CANVAS_RESOLUTIONS } from "@hyperframes/parsers";
 import { formatRenderOutputTimestamp, parseFps } from "@hyperframes/core";
 var VALID_RESOLUTIONS = new Set(VALID_CANVAS_RESOLUTIONS);
+function contentDispositionHeader(disposition, filename) {
+  const fallback = filename.replace(/[^\x20-\x7e]|["\\]/g, "_");
+  const encoded = encodeURIComponent(filename).replace(
+    /[!'()*]/g,
+    (character) => `%${character.charCodeAt(0).toString(16).toUpperCase()}`
+  );
+  return `${disposition}; filename="${fallback}"; filename*=UTF-8''${encoded}`;
+}
 function registerRenderRoutes(api, adapter) {
   const renderJobs = /* @__PURE__ */ new Map();
   const TTL_MS = 3e5;
@@ -4007,7 +4025,7 @@ function registerRenderRoutes(api, adapter) {
     return new Response(content, {
       headers: {
         "Content-Type": contentType,
-        "Content-Disposition": `inline; filename="${filename}"`,
+        "Content-Disposition": contentDispositionHeader("inline", filename),
         "Accept-Ranges": "bytes",
         "Content-Length": String(content.length)
       }
@@ -4025,7 +4043,7 @@ function registerRenderRoutes(api, adapter) {
     return new Response(content, {
       headers: {
         "Content-Type": contentType,
-        "Content-Disposition": `attachment; filename="${filename}"`
+        "Content-Disposition": contentDispositionHeader("attachment", filename)
       }
     });
   });
@@ -4058,7 +4076,7 @@ function registerRenderRoutes(api, adapter) {
     return new Response(content, {
       headers: {
         "Content-Type": contentType,
-        "Content-Disposition": `inline; filename="${filename}"`,
+        "Content-Disposition": contentDispositionHeader("inline", filename),
         "Accept-Ranges": "bytes",
         "Content-Length": String(content.length)
       }