@hyperframes/engine 0.6.119 → 0.6.120

package/src/services/parallelCoordinator.ts

@@ -1,437 +0,0 @@
-/**
- * Parallel Coordinator Service
- *
- * Coordinates parallel frame capture across multiple Puppeteer sessions.
- * Auto-detects optimal worker count based on CPU/memory.
- */
-
-import { cpus, freemem } from "os";
-import { existsSync, mkdirSync, readdirSync } from "fs";
-import { copyFile, rename } from "fs/promises";
-import { join } from "path";
-
-import {
-  createCaptureSession,
-  initializeSession,
-  closeCaptureSession,
-  captureFrame,
-  captureFrameToBuffer,
-  getCapturePerfSummary,
-  type CaptureSession,
-  type CaptureOptions,
-  type CapturePerfSummary,
-  type BeforeCaptureHook,
-} from "./frameCapture.js";
-import { DEFAULT_CONFIG, type EngineConfig } from "../config.js";
-import { assertSwiftShader } from "../utils/assertSwiftShader.js";
-import { readWebGlVendorInfoFromCanvas } from "../utils/readWebGlVendorInfoFromCanvas.js";
-import { resolveHeadlessShellPath } from "./browserManager.js";
-import { getSystemTotalMb } from "./systemMemory.js";
-
-export interface WorkerTask {
-  workerId: number;
-  startFrame: number;
-  endFrame: number;
-  outputDir: string;
-  /**
-   * Offset subtracted from the absolute frame index when naming the captured
-   * file (`frame_<i - outputFrameOffset>.{ext}`). Default 0. Distributed
-   * chunks set this to the chunk's absolute startFrame so file names land
-   * 0-indexed within the chunk's range — the encoder reads frames
-   * sequentially without an `-start_number` override. The per-frame TIME
-   * calculation still uses the absolute frame index.
-   */
-  outputFrameOffset?: number;
-}
-
-export interface WorkerResult {
-  workerId: number;
-  framesCaptured: number;
-  startFrame: number;
-  endFrame: number;
-  durationMs: number;
-  perf?: CapturePerfSummary;
-  error?: string;
-  diagnostics?: string[];
-}
-
-export interface ParallelProgress {
-  totalFrames: number;
-  capturedFrames: number;
-  activeWorkers: number;
-  workerProgress: Map<number, number>;
-}
-
-export interface WorkerSizingConfig extends Partial<
-  Pick<
-    EngineConfig,
-    "concurrency" | "coresPerWorker" | "minParallelFrames" | "largeRenderThreshold"
-  >
-> {
-  /**
-   * Relative per-frame capture cost for auto worker sizing. Values above 1
-   * represent compositions that put more CPU pressure on each Chrome worker
-   * than a plain DOM screenshot. Explicit --workers requests ignore this hint.
-   */
-  captureCostMultiplier?: number;
-}
-
-const MEMORY_PER_WORKER_MB = 256;
-const MIN_WORKERS = 1;
-const MAX_WORKER_DIAGNOSTIC_LINES = 8;
-// Hard ceiling on explicit `--workers N` requests. Above this, the cost of
-// CDP-protocol dispatch through Node's main event loop and OS scheduling
-// noise overwhelms any further parallelism. Bumped from 10 → 24 in hf#732
-// follow-up so high-core hosts (32-96+ cores) can actually surface the
-// hardware to renders that are CPU-bound on DOM capture.
-const ABSOLUTE_MAX_WORKERS = 24;
-// `auto` concurrency picks this many workers as the upper bound. Bumped
-// from a hardcoded 6 → CPU-scaled value (floor(cpuCount/8), floor at 6,
-// ceiling at 16) in hf#732 follow-up. Rationale: the prior fixed cap of 6
-// left ~90 cores idle on the validation host and forced users to pass
-// `--workers N` to opt in. Now `auto` matches what a thoughtful operator
-// would pick by hand. The /8 divisor leaves headroom for each Chrome
-// worker's SwiftShader compositor + the shader-blend thread pool, both of
-// which are themselves CPU-heavy.
-function defaultSafeMaxWorkers(): number {
-  return Math.max(6, Math.min(16, Math.floor(cpus().length / 8)));
-}
-const MIN_FRAMES_PER_WORKER = 30;
-
-export function selectWorkerDiagnostics(
-  lines: readonly string[],
-  maxLines: number = MAX_WORKER_DIAGNOSTIC_LINES,
-): string[] {
-  return lines
-    .filter((line) =>
-      /\[(FrameCapture:ERROR|Browser:ERROR|Browser:PAGEERROR|Browser:REQUESTFAILED|Browser:HTTP\d{3})\]/.test(
-        line,
-      ),
-    )
-    .slice(-maxLines);
-}
-
-function compactDiagnosticLine(line: string): string {
-  return line.replace(/\s+/g, " ").trim();
-}
-
-export function formatWorkerFailure(result: WorkerResult): string {
-  const base = `Worker ${result.workerId}: ${result.error ?? "unknown error"}`;
-  if (!result.diagnostics || result.diagnostics.length === 0) return base;
-
-  const diagnostics = result.diagnostics.map(compactDiagnosticLine).join(" | ");
-  return `${base}; diagnostics: ${diagnostics}`;
-}
-
-export function calculateOptimalWorkers(
-  totalFrames: number,
-  requested?: number,
-  config?: WorkerSizingConfig,
-): number {
-  // Resolve effective values: config overrides → DEFAULT_CONFIG fallback.
-  const effectiveMaxWorkers = (() => {
-    const concurrency = config?.concurrency ?? DEFAULT_CONFIG.concurrency;
-    if (concurrency !== "auto") {
-      return Math.max(MIN_WORKERS, Math.min(ABSOLUTE_MAX_WORKERS, Math.floor(concurrency)));
-    }
-    return defaultSafeMaxWorkers();
-  })();
-  const effectiveCoresPerWorker = config?.coresPerWorker ?? DEFAULT_CONFIG.coresPerWorker;
-  const effectiveMinParallelFrames = config?.minParallelFrames ?? DEFAULT_CONFIG.minParallelFrames;
-  const effectiveLargeRenderThreshold =
-    config?.largeRenderThreshold ?? DEFAULT_CONFIG.largeRenderThreshold;
-  const captureCostMultiplier = Math.max(1, config?.captureCostMultiplier ?? 1);
-
-  if (requested !== undefined) {
-    return Math.max(MIN_WORKERS, Math.min(effectiveMaxWorkers, requested));
-  }
-
-  if (totalFrames < MIN_FRAMES_PER_WORKER * 2) return 1;
-
-  const cpuCount = cpus().length;
-  const cpuBasedWorkers = Math.max(1, cpuCount - 2);
-
-  // Use total memory instead of free memory — macOS reports misleadingly low
-  // freemem() because it aggressively caches files in "inactive" memory that
-  // is immediately reclaimable.
-  const totalMemoryMB = getSystemTotalMb();
-  const memoryBasedWorkers = Math.max(1, Math.floor((totalMemoryMB * 0.5) / MEMORY_PER_WORKER_MB));
-
-  const frameBasedWorkers = Math.floor(totalFrames / MIN_FRAMES_PER_WORKER);
-
-  const optimal = Math.min(cpuBasedWorkers, memoryBasedWorkers, frameBasedWorkers);
-  const minWorkersForJob = totalFrames >= effectiveMinParallelFrames ? 2 : MIN_WORKERS;
-  let finalWorkers = Math.max(minWorkersForJob, Math.min(effectiveMaxWorkers, optimal));
-
-  // Adaptive scaling: cap workers for large or expensive renders to prevent
-  // CPU contention. Each Chrome process (with SwiftShader) is CPU-heavy; too
-  // many concurrent captures can starve the compositor and surface as CDP
-  // protocol timeouts. Scale proportionally to CPU count and composition cost:
-  // 8 cores → 2 workers, 16 cores → 5 workers, 32 cores → 10 workers.
-  const weightedFrames = totalFrames * captureCostMultiplier;
-  const contentionThreshold = Math.max(
-    effectiveMinParallelFrames,
-    Math.floor(effectiveLargeRenderThreshold / 3),
-  );
-  if (totalFrames >= effectiveLargeRenderThreshold || weightedFrames >= contentionThreshold) {
-    const weightedCoresPerWorker = effectiveCoresPerWorker * captureCostMultiplier;
-    const cpuScaledMax = Math.max(MIN_WORKERS, Math.floor(cpuCount / weightedCoresPerWorker));
-    if (finalWorkers > cpuScaledMax) {
-      finalWorkers = cpuScaledMax;
-    }
-  }
-
-  return finalWorkers;
-}
-
-export function distributeFrames(
-  totalFrames: number,
-  workerCount: number,
-  workDir: string,
-  rangeStart: number = 0,
-): WorkerTask[] {
-  const tasks: WorkerTask[] = [];
-  const framesPerWorker = Math.ceil(totalFrames / workerCount);
-
-  for (let i = 0; i < workerCount; i++) {
-    const startFrame = rangeStart + i * framesPerWorker;
-    const endFrame = Math.min(rangeStart + (i + 1) * framesPerWorker, rangeStart + totalFrames);
-    if (startFrame >= rangeStart + totalFrames) break;
-
-    tasks.push({
-      workerId: i,
-      startFrame,
-      endFrame,
-      outputDir: join(workDir, `worker-${i}`),
-      outputFrameOffset: rangeStart,
-    });
-  }
-
-  return tasks;
-}
-
-/**
- * Decide whether a parallel worker should run the per-worker SwiftShader
- * assertion. Gated to worker 0 only: workers within a chunk share the same
- * Chrome binary, flags, and OS/driver state, so one verification per chunk
- * is sufficient. See `heygen-com/hyperframes#955`.
- */
-export function shouldVerifyWorkerGpu(workerId: number, config?: Partial<EngineConfig>): boolean {
-  return config?.browserGpuMode === "software" && workerId === 0;
-}
-
-async function captureFrameRange(
-  session: CaptureSession,
-  task: WorkerTask,
-  captureOptions: CaptureOptions,
-  signal: AbortSignal | undefined,
-  onFrameCaptured: ((workerId: number, frameIndex: number) => void) | undefined,
-  onFrameBuffer: ((frameIndex: number, buffer: Buffer) => Promise<void>) | undefined,
-): Promise<number> {
-  let framesCaptured = 0;
-  const outputOffset = task.outputFrameOffset ?? 0;
-  for (let i = task.startFrame; i < task.endFrame; i++) {
-    if (signal?.aborted) throw new Error("Parallel worker cancelled");
-    const time = (i * captureOptions.fps.den) / captureOptions.fps.num;
-    const fileFrameIdx = i - outputOffset;
-
-    if (onFrameBuffer) {
-      const { buffer } = await captureFrameToBuffer(session, fileFrameIdx, time);
-      await onFrameBuffer(i, buffer);
-    } else {
-      await captureFrame(session, fileFrameIdx, time);
-    }
-    framesCaptured++;
-    if (onFrameCaptured) onFrameCaptured(task.workerId, i);
-  }
-  return framesCaptured;
-}
-
-async function executeWorkerTask(
-  task: WorkerTask,
-  serverUrl: string,
-  captureOptions: CaptureOptions,
-  createBeforeCaptureHook: () => BeforeCaptureHook | null,
-  signal?: AbortSignal,
-  onFrameCaptured?: (workerId: number, frameIndex: number) => void,
-  onFrameBuffer?: (frameIndex: number, buffer: Buffer) => Promise<void>,
-  config?: Partial<EngineConfig>,
-  parallel?: boolean,
-): Promise<WorkerResult> {
-  const startTime = Date.now();
-  let framesCaptured = 0;
-
-  if (!existsSync(task.outputDir)) mkdirSync(task.outputDir, { recursive: true });
-
-  let session: CaptureSession | null = null;
-  let perf: CapturePerfSummary | undefined;
-
-  // BeginFrame's compositor is process-global — multiple pages driving
-  // beginFrame in the same browser race it and crash with "Target closed".
-  // Only disable the pool when BeginFrame mode would actually be active.
-  // Must match the predicate in createCaptureSession (frameCapture.ts):
-  // Linux + headless-shell + !forceScreenshot + !supersampling.
-  const supersampling = (captureOptions.deviceScaleFactor ?? 1) > 1;
-  const needsSeparateBrowsers =
-    parallel &&
-    process.platform === "linux" &&
-    !config?.forceScreenshot &&
-    !supersampling &&
-    resolveHeadlessShellPath(config) !== undefined;
-  const workerConfig: Partial<EngineConfig> | undefined = needsSeparateBrowsers
-    ? { ...config, enableBrowserPool: false }
-    : config;
-
-  try {
-    session = await createCaptureSession(
-      serverUrl,
-      task.outputDir,
-      captureOptions,
-      createBeforeCaptureHook(),
-      workerConfig,
-    );
-    // Worker-0-only SwiftShader assertion — see `shouldVerifyWorkerGpu` and #955.
-    if (shouldVerifyWorkerGpu(task.workerId, workerConfig)) {
-      await assertSwiftShader(session.page, readWebGlVendorInfoFromCanvas);
-    }
-    await initializeSession(session);
-    framesCaptured = await captureFrameRange(
-      session,
-      task,
-      captureOptions,
-      signal,
-      onFrameCaptured,
-      onFrameBuffer,
-    );
-
-    perf = getCapturePerfSummary(session);
-    return {
-      workerId: task.workerId,
-      framesCaptured,
-      startFrame: task.startFrame,
-      endFrame: task.endFrame,
-      durationMs: Date.now() - startTime,
-      perf,
-    };
-  } catch (error) {
-    const errMsg = error instanceof Error ? error.message : String(error);
-    const diagnostics = session ? selectWorkerDiagnostics(session.browserConsoleBuffer) : [];
-    return {
-      workerId: task.workerId,
-      framesCaptured,
-      startFrame: task.startFrame,
-      endFrame: task.endFrame,
-      durationMs: Date.now() - startTime,
-      perf,
-      error: errMsg,
-      diagnostics: diagnostics.length > 0 ? diagnostics : undefined,
-    };
-  } finally {
-    if (session) await closeCaptureSession(session).catch(() => {});
-  }
-}
-
-export async function executeParallelCapture(
-  serverUrl: string,
-  workDir: string,
-  tasks: WorkerTask[],
-  captureOptions: CaptureOptions,
-  createBeforeCaptureHook: () => BeforeCaptureHook | null,
-  signal?: AbortSignal,
-  onProgress?: (progress: ParallelProgress) => void,
-  onFrameBuffer?: (frameIndex: number, buffer: Buffer) => Promise<void>,
-  config?: Partial<EngineConfig>,
-): Promise<WorkerResult[]> {
-  const totalFrames = tasks.reduce((sum, t) => sum + (t.endFrame - t.startFrame), 0);
-  const workerProgress = new Map<number, number>();
-
-  for (const task of tasks) workerProgress.set(task.workerId, 0);
-
-  const onFrameCaptured = (workerId: number, _frameIndex: number) => {
-    const current = workerProgress.get(workerId) || 0;
-    workerProgress.set(workerId, current + 1);
-
-    if (onProgress) {
-      const capturedFrames = Array.from(workerProgress.values()).reduce((a, b) => a + b, 0);
-      onProgress({
-        totalFrames,
-        capturedFrames,
-        activeWorkers: tasks.length,
-        workerProgress: new Map(workerProgress),
-      });
-    }
-  };
-
-  const parallel = tasks.length > 1;
-  const results = await Promise.all(
-    tasks.map((task) =>
-      executeWorkerTask(
-        task,
-        serverUrl,
-        captureOptions,
-        createBeforeCaptureHook,
-        signal,
-        onFrameCaptured,
-        onFrameBuffer,
-        config,
-        parallel,
-      ),
-    ),
-  );
-
-  const errors = results.filter((r) => r.error);
-  if (errors.length > 0) {
-    const errorMessages = errors.map(formatWorkerFailure).join("; ");
-    throw new Error(`[Parallel] Capture failed: ${errorMessages}`);
-  }
-
-  return results;
-}
-
-export async function mergeWorkerFrames(
-  workDir: string,
-  tasks: WorkerTask[],
-  outputDir: string,
-): Promise<number> {
-  if (!existsSync(outputDir)) mkdirSync(outputDir, { recursive: true });
-
-  let totalFrames = 0;
-  const sortedTasks = [...tasks].sort((a, b) => a.startFrame - b.startFrame);
-
-  for (const task of sortedTasks) {
-    if (!existsSync(task.outputDir)) {
-      continue;
-    }
-
-    const files = readdirSync(task.outputDir)
-      .filter((f) => f.startsWith("frame_") && (f.endsWith(".jpg") || f.endsWith(".png")))
-      .sort();
-    const copyTasks = files.map(async (file) => {
-      const sourcePath = join(task.outputDir, file);
-      const targetPath = join(outputDir, file);
-      try {
-        await rename(sourcePath, targetPath);
-      } catch {
-        await copyFile(sourcePath, targetPath);
-      }
-    });
-    await Promise.all(copyTasks);
-    totalFrames += files.length;
-  }
-
-  return totalFrames;
-}
-
-export function getSystemResources(): {
-  cpuCores: number;
-  totalMemoryMB: number;
-  freeMemoryMB: number;
-  recommendedWorkers: number;
-} {
-  return {
-    cpuCores: cpus().length,
-    totalMemoryMB: getSystemTotalMb(),
-    freeMemoryMB: Math.round(freemem() / (1024 * 1024)),
-    recommendedWorkers: calculateOptimalWorkers(1000),
-  };
-}