From 8ecaeef221dc809297cdc524c41f7e1d4fa38f74 Mon Sep 17 00:00:00 2001
From: Vance Ingalls <vance@heygen.com>
Date: Fri, 12 Jun 2026 14:01:06 -0700
Subject: [PATCH] refactor(core): swap studio-api read path from recast to
 acorn parser (T6e)

---
 packages/core/src/studio-api/routes/files.ts | 20 ++++++++++++--------
 1 file changed, 12 insertions(+), 8 deletions(-)

diff --git a/packages/core/src/studio-api/routes/files.ts b/packages/core/src/studio-api/routes/files.ts
index 0f5e2a0fc8..2a471b89df 100644
--- a/packages/core/src/studio-api/routes/files.ts
+++ b/packages/core/src/studio-api/routes/files.ts
@@ -24,6 +24,7 @@ import {
   type UnsafeMutationValue,
 } from "../helpers/finiteMutation.js";
 import type { GsapAnimation } from "../../parsers/gsapSerialize.js";
+import { parseGsapScriptAcorn } from "../../parsers/gsapParserAcorn.js";
 import {
   removeElementFromHtml,
   patchElementInHtml,
@@ -316,7 +317,13 @@ function bakeVisibilityOnDelete(document: Document, anim: GsapAnimation): void {
   }
 }
 
-/** Lazy-load gsapParser to avoid pulling recast into every file-route import. */
+/**
+ * Lazy-load gsapParser for write ops (recast-backed) that are not yet ported to
+ * the acorn writer. The read path (`parseGsapScript`) has been replaced by the
+ * browser-safe `parseGsapScriptAcorn` — this loader is only needed for the write
+ * ops that remain: convertToKeyframesInScript, removeAllKeyframesFromScript,
+ * materializeKeyframesInScript, unrollDynamicAnimations, setArcPathInScript, etc.
+ */
 async function loadGsapParser() {
   return import("../../parsers/gsapParser.js");
 }
@@ -479,7 +486,6 @@ async function executeGsapMutation(
 ): Promise<GsapMutationResult | Response> {
   const parser = await loadGsapParser();
   const {
-    parseGsapScript,
     updateAnimationInScript,
     addAnimationToScript,
     removeAnimationFromScript,
@@ -502,7 +508,7 @@ async function executeGsapMutation(
     scriptText: string,
     animationId: string,
   ): { anim: GsapAnimation } | { err: Response } {
-    const parsed = parseGsapScript(scriptText);
+    const parsed = parseGsapScriptAcorn(scriptText);
     const anim = parsed.animations.find((a) => a.id === animationId);
     if (!anim) return { err: respond({ error: "animation not found" }, 404) };
     return { anim };
@@ -565,7 +571,7 @@ async function executeGsapMutation(
       return removeAnimationFromScript(block.scriptText, body.animationId);
     }
     case "delete-all-for-selector": {
-      const parsed = parseGsapScript(block.scriptText);
+      const parsed = parseGsapScriptAcorn(block.scriptText);
       const matching = parsed.animations.filter((a) => a.targetSelector === body.targetSelector);
       if (matching.length === 0) return block.scriptText;
       stripStudioEditsFromTarget(block.document, body.targetSelector);
@@ -1120,8 +1126,7 @@ export function registerFileRoutes(api: Hono, adapter: StudioApiAdapter): void {
       });
     }
 
-    const { parseGsapScript } = await loadGsapParser();
-    const parsed = parseGsapScript(block.scriptText);
+    const parsed = parseGsapScriptAcorn(block.scriptText);
     return c.json(parsed);
   });
 
@@ -1186,8 +1191,7 @@ export function registerFileRoutes(api: Hono, adapter: StudioApiAdapter): void {
       writeFileSync(res.absPath, newHtml, "utf-8");
     }
 
-    const { parseGsapScript } = await loadGsapParser();
-    const freshParsed = parseGsapScript(newScript);
+    const freshParsed = parseGsapScriptAcorn(newScript);
     const responsePayload: Record<string, unknown> = {
       ok: true,
       changed,