Merge pull request #778 from heygen-com/fix/ci-format-and-windows-esbuild

fix(ci): unbreak format check on main + skip cli build:fonts when present

Author: jrusso1020

.github/workflows/windows-render.yml

@@ -150,7 +150,16 @@ jobs:
 
       - name: Install dependencies
         shell: pwsh
-        run: bun install --frozen-lockfile
+        # Force the pre-1.3 "hoisted" linker on Windows. Bun 1.3.x's default
+        # "isolated" linker creates nested workspace junctions that don't
+        # materialize reliably on Windows GHA runners — manifests as
+        # `Cannot find package 'esbuild'` walking up from
+        # `packages/producer/build.mjs` and `EPERM stat` on
+        # `packages/producer/node_modules/@fontsource/*` from generators.
+        # See oven-sh/bun#23615, #18354, #10146. Linux CI keeps isolated;
+        # the lockfile is linker-agnostic so this flag is safe with
+        # --frozen-lockfile.
+        run: bun install --frozen-lockfile --linker=hoisted
 
       - name: Build all packages
         shell: pwsh
@@ -371,7 +380,16 @@ jobs:
 
       - name: Install dependencies
         shell: pwsh
-        run: bun install --frozen-lockfile
+        # Force the pre-1.3 "hoisted" linker on Windows. Bun 1.3.x's default
+        # "isolated" linker creates nested workspace junctions that don't
+        # materialize reliably on Windows GHA runners — manifests as
+        # `Cannot find package 'esbuild'` walking up from
+        # `packages/producer/build.mjs` and `EPERM stat` on
+        # `packages/producer/node_modules/@fontsource/*` from generators.
+        # See oven-sh/bun#23615, #18354, #10146. Linux CI keeps isolated;
+        # the lockfile is linker-agnostic so this flag is safe with
+        # --frozen-lockfile.
+        run: bun install --frozen-lockfile --linker=hoisted
 
       - name: Build
         shell: pwsh

bun.lock

@@ -18,7 +18,7 @@
     "test": "vitest run",
     "dev": "tsx src/cli.ts",
     "build": "bun run build:fonts && tsup && bun run build:runtime && bun run build:copy",
-    "build:fonts": "cd ../producer && tsx scripts/generate-font-data.ts",
+    "build:fonts": "node -e \"const fs=require('fs');if(!fs.existsSync('../producer/src/services/fontData.generated.ts')){const {execSync}=require('child_process');execSync('cd ../producer && tsx scripts/generate-font-data.ts',{stdio:'inherit'})}else{console.log('[build:fonts] skipped — fontData.generated.ts already exists')}\"",
     "build:runtime": "tsx scripts/build-runtime.ts",
     "build:copy": "node scripts/build-copy.mjs",
     "typecheck": "tsc --noEmit"
@@ -29,7 +29,7 @@
     "adm-zip": "^0.5.16",
     "citty": "^0.2.1",
     "compare-versions": "^6.1.1",
-    "esbuild": "^0.25.0",
+    "esbuild": "^0.25.12",
     "giget": "^3.2.0",
     "hono": "^4.0.0",
     "mime-types": "^3.0.2",
@@ -48,7 +48,7 @@
     "@hyperframes/studio": "workspace:*",
     "@types/adm-zip": "^0.5.7",
     "@types/mime-types": "^3.0.1",
-    "@types/node": "^22.0.0",
+    "@types/node": "^25.0.10",
     "linkedom": "^0.18.12",
     "picocolors": "^1.1.1",
     "tsup": "^8.0.0",

packages/cli/package.json

@@ -296,7 +296,7 @@ function patchVideoSrc(
 ): void {
   const htmlFiles = readdirSync(dir, { withFileTypes: true, recursive: true })
     .filter((e) => e.isFile() && e.name.endsWith(".html"))
-    .map((e) => join(e.parentPath ?? e.path, e.name));
+    .map((e) => join(e.parentPath, e.name));
 
   for (const file of htmlFiles) {
     let content = readFileSync(file, "utf-8");

packages/cli/src/commands/init.ts

@@ -332,7 +332,7 @@ export function patchCaptionHtml(dir: string, words: Word[]): void {
   try {
     htmlFiles = readdirSync(dir, { withFileTypes: true, recursive: true })
       .filter((e) => e.isFile() && e.name.endsWith(".html"))
-      .map((e) => join(e.parentPath ?? e.path, e.name));
+      .map((e) => join(e.parentPath, e.name));
   } catch {
     return;
   }

packages/cli/src/whisper/normalize.ts

@@ -62,8 +62,7 @@
     "./schemas/registry-item.json": "./schemas/registry-item.json"
   },
   "publishConfig": {
-    "main": "./dist/index.js",
-    "types": "./dist/index.d.ts",
+    "access": "public",
     "exports": {
       ".": {
         "import": "./dist/index.js",
@@ -109,7 +108,8 @@
       "./schemas/registry.json": "./schemas/registry.json",
       "./schemas/registry-item.json": "./schemas/registry-item.json"
     },
-    "access": "public"
+    "main": "./dist/index.js",
+    "types": "./dist/index.d.ts"
   },
   "scripts": {
     "build": "bun run build:hyperframes-runtime && tsc",
@@ -139,7 +139,7 @@
   },
   "devDependencies": {
     "@types/jsdom": "^28.0.0",
-    "@types/node": "^24.10.13",
+    "@types/node": "^25.0.10",
     "@vitest/coverage-v8": "^3.2.4",
     "jsdom": "^29.0.0",
     "tsx": "^4.21.0",

packages/core/package.json

@@ -306,8 +306,21 @@ export function registerFileRoutes(api: Hono, adapter: StudioApiAdapter): void {
       const skipped: string[] = [];
       const invalid: Array<{ name: string; reason: string }> = [];
 
-      for (const [, value] of formData.entries()) {
-        if (!(value instanceof File)) continue;
+      // @types/node v25 narrows the ambient `FormData.entries()` to
+      // `[string, string]` in workspaces where another dep declares an
+      // `onmessage` global (it trips the worker branch of v25's conditional
+      // File type). At runtime the value is still `File | string` — cast the
+      // iterator so the rest of this block keeps type-checking on every
+      // bun-install layout (hoisted on Windows surfaces this; isolated on
+      // Linux happens to keep v24 in scope).
+      type FileLike = {
+        readonly name: string;
+        readonly size: number;
+        arrayBuffer(): Promise<ArrayBuffer>;
+      };
+      const entries = formData.entries() as unknown as Iterable<[string, FileLike | string]>;
+      for (const [, value] of entries) {
+        if (typeof value === "string") continue;
 
         // Strip path separators — browsers may include directory components
         const name = value.name.split("/").pop()?.split("\\").pop() ?? "";

packages/core/src/studio-api/routes/files.ts

@@ -27,7 +27,7 @@
     "puppeteer-core": "^24.39.1"
   },
   "devDependencies": {
-    "@types/node": "^22.10.1",
+    "@types/node": "^25.0.10",
     "@webgpu/types": "^0.1.69",
     "typescript": "^5.7.2",
     "vitest": "^3.2.4"

packages/engine/package.json

@@ -75,10 +75,10 @@
     "@fontsource/poppins": "^5.2.7",
     "@fontsource/roboto": "^5.2.10",
     "@fontsource/source-code-pro": "^5.2.7",
-    "@types/node": "^22.10.1",
+    "@types/node": "^25.0.10",
     "@webgpu/types": "^0.1.69",
-    "esbuild": "^0.27.2",
-    "tsx": "^4.7.0",
+    "esbuild": "^0.25.12",
+    "tsx": "^4.21.0",
     "typescript": "^5.7.2"
   },
   "engines": {

packages/producer/package.json

@@ -0,0 +1,3 @@
+version https://git-lfs.github.com/spec/v1
+oid sha256:4f419db5ff8a944712be32eb03215e0111a539d7c15da1feffd5cc6e85d6de03
+size 159786