Merge pull request #938 from heygen-com/chore/fallow-config-and-cleanup

chore: add fallow config and fix high-signal findings

Author: jrusso1020

.fallowrc.jsonc

@@ -0,0 +1,92 @@
+{
+  "$schema": "https://raw.githubusercontent.com/fallow-rs/fallow/main/config-schema.json",
+  "entry": [
+    "packages/producer/src/**/*.test.ts",
+    "packages/aws-lambda/src/**/*.test.ts",
+    "packages/producer/src/regression-harness.ts",
+    "packages/producer/src/regression-harness-distributed.test.ts",
+    "packages/producer/src/regression-harness-lambda-local.ts",
+    "packages/producer/src/transparency-test.ts",
+    "packages/producer/src/parity-harness.ts",
+    "packages/producer/src/parity-fixtures.ts",
+    "packages/producer/src/perf-gate.ts",
+    "packages/producer/src/runtime-conformance.ts",
+    "packages/producer/src/benchmark.ts",
+    "packages/producer/scripts/generate-font-data.ts",
+    "packages/cli/scripts/generate-font-data.ts",
+    "packages/engine/scripts/test-fitTextFontSize-browser.ts",
+    "packages/aws-lambda/scripts/*.ts",
+    // Built as standalone IIFE for the browser-side sandbox runtime;
+    // referenced by file path (not import) in build-hyperframes-runtime-artifact.ts.
+    "packages/core/src/runtime/entry.ts",
+    // Worker entry points loaded dynamically by their *Pool.ts companions.
+    "packages/producer/src/services/pngDecodeBlitWorker.ts",
+    "packages/producer/src/services/shaderTransitionWorker.ts",
+    "scripts/*.{ts,mjs,js}",
+    "scripts/*/run.mjs",
+  ],
+  "ignorePatterns": [
+    "docs/**",
+    "packages/producer/tests/**",
+    "packages/player/tests/**",
+    "packages/engine/tests/**",
+    "skills/**/test-corpus/**",
+    "skills/**/scripts/**",
+    "registry/**",
+    "examples/**",
+    ".github/workflows/fixtures/**",
+  ],
+  "ignoreExports": [
+    // CLI command files: every command exports a const `examples` per the
+    // convention documented in CLAUDE.md. This is a namespace barrel, not a
+    // collision.
+    { "file": "packages/cli/src/commands/*.ts", "exports": ["examples"] },
+    // Independent ML model managers each declare their own DEFAULT_MODEL /
+    // MODELS_DIR / ensureModel for their model namespace.
+    {
+      "file": "packages/cli/src/{background-removal,tts,whisper}/manager.ts",
+      "exports": ["DEFAULT_MODEL", "MODELS_DIR", "ensureModel"],
+    },
+    // `isPathInside` is documented as exported-for-tests only in fileServer.ts;
+    // it has different semantics (symlink resolution) from utils/paths.ts.
+    {
+      "file": "packages/producer/src/services/fileServer.ts",
+      "exports": ["isPathInside"],
+    },
+  ],
+  "ignoreDependencies": [
+    // Runtime/dynamic deps not visible to static analysis: tsup `external`,
+    // dynamic require() resolution, peer/static-file consumption in tests,
+    // and bun-hoisted workspace devDeps (e.g. happy-dom in root package.json
+    // resolves for every workspace, so workspaces don't redeclare it).
+    "puppeteer",
+    "puppeteer-core",
+    "esbuild",
+    "giget",
+    "gsap",
+    "happy-dom",
+    "ffmpeg-static",
+    "ffprobe-static",
+    "@hyperframes/core",
+    "@hyperframes/studio",
+    "@hyperframes/producer",
+    "@fontsource/archivo-black",
+    "@fontsource/eb-garamond",
+    "@fontsource/ibm-plex-mono",
+    "@fontsource/inter",
+    "@fontsource/jetbrains-mono",
+    "@fontsource/league-gothic",
+    "@fontsource/montserrat",
+    "@fontsource/nunito",
+    "@fontsource/oswald",
+    "@fontsource/outfit",
+    "@fontsource/space-mono",
+    "@fontsource/lato",
+    "@fontsource/noto-sans-jp",
+    "@fontsource/open-sans",
+    "@fontsource/playfair-display",
+    "@fontsource/poppins",
+    "@fontsource/roboto",
+    "@fontsource/source-code-pro",
+  ],
+}

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": "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:fonts": "node scripts/build-fonts.mjs",
     "build:runtime": "tsx scripts/build-runtime.ts",
     "build:copy": "node scripts/build-copy.mjs",
     "typecheck": "tsc --noEmit"
@@ -32,7 +32,6 @@
     "esbuild": "^0.25.12",
     "giget": "^3.2.0",
     "hono": "^4.0.0",
-    "mime-types": "^3.0.2",
     "onnxruntime-node": "^1.20.0",
     "open": "^10.0.0",
     "postcss": "^8.5.8",

packages/cli/package.json

@@ -0,0 +1,14 @@
+#!/usr/bin/env node
+import { existsSync } from "node:fs";
+import { execSync } from "node:child_process";
+
+const target = "../producer/src/services/fontData.generated.ts";
+
+if (existsSync(target)) {
+  console.log("[build:fonts] skipped — fontData.generated.ts already exists");
+  process.exit(0);
+}
+
+execSync("cd ../producer && tsx scripts/generate-font-data.ts", {
+  stdio: "inherit",
+});

packages/cli/scripts/build-fonts.mjs

@@ -47,7 +47,6 @@ var __dirname = __hf_dirname(__filename);`,
     "hono",
     "hono/*",
     "@hono/node-server",
-    "mime-types",
     "adm-zip",
     "esbuild",
     "giget",

packages/cli/src/utils/mime.ts

@@ -1,4 +1,4 @@
-import type { Keyframe, KeyframeProperties } from "../core.types";
+import type { Keyframe, KeyframeProperties, ValidationResult } from "../core.types";
 
 export type GsapMethod = "set" | "to" | "from" | "fromTo";
 
@@ -344,12 +344,6 @@ export function getAnimationsForElement(
   return animations.filter((a) => a.targetSelector === selector);
 }
 
-export interface ValidationResult {
-  valid: boolean;
-  errors: string[];
-  warnings: string[];
-}
-
 const FORBIDDEN_GSAP_PATTERNS: Array<{ pattern: RegExp; message: string }> = [
   { pattern: /\.call\s*\(/, message: "call() method not allowed" },
   {

packages/cli/tsup.config.ts

@@ -17,7 +17,7 @@ import {
   gsapAnimationsToKeyframes,
   getAnimationsForElement,
 } from "./gsapParser";
-import type { ValidationResult } from "./gsapParser";
+import type { ValidationResult } from "../core.types";
 
 const MEDIA_TYPES = new Set<string>(["video", "image", "audio"]);
 

packages/core/src/parsers/gsapParser.ts

@@ -32,7 +32,7 @@
   },
   "scripts": {
     "build": "bun run build:fonts && bun run --cwd ../.. build:hyperframes-runtime:modular && node build.mjs",
-    "build:fonts": "node -e \"const fs=require('fs');if(!fs.existsSync('src/services/fontData.generated.ts')){const {execSync}=require('child_process');execSync('node --experimental-strip-types scripts/generate-font-data.ts',{stdio:'inherit'})}else{console.log('[build:fonts] skipped — fontData.generated.ts already exists')}\"",
+    "build:fonts": "node scripts/build-fonts.mjs",
     "typecheck": "tsc --noEmit",
     "parity:check": "tsx src/parity-harness.ts",
     "parity:fixtures": "tsx src/parity-fixtures.ts",

packages/core/src/parsers/htmlParser.ts

@@ -0,0 +1,14 @@
+#!/usr/bin/env node
+import { existsSync } from "node:fs";
+import { execSync } from "node:child_process";
+
+const target = "src/services/fontData.generated.ts";
+
+if (existsSync(target)) {
+  console.log("[build:fonts] skipped — fontData.generated.ts already exists");
+  process.exit(0);
+}
+
+execSync("node --experimental-strip-types scripts/generate-font-data.ts", {
+  stdio: "inherit",
+});