@hyperframes/engine 0.8.199 → 0.8.201

package/dist/utils/runFfmpeg.js

@@ -94,11 +94,11 @@ export function formatFfmpegError(exitCode, stderr, tailLines = DEFAULT_STDERR_T
         ? `FFmpeg exited with code ${exitCode}\nffmpeg stderr (tail):\n${tail}`
         : `FFmpeg exited with code ${exitCode}`;
 }
-/** formatFfmpegError, led by the timeout when this process was killed at its deadline. */
+/** formatFfmpegError, led by the timeout when this process was killed for making no progress. */
 export function describeFfmpegFailure(result, timeoutMs) {
     const error = formatFfmpegError(result.exitCode, result.stderr);
-    if (result.terminationReason === "deadline") {
-        return (`FFmpeg timed out after ${timeoutMs} ms (ffmpegProcessTimeout; long renders can raise ` +
+    if (result.terminationReason === "inactivity") {
+        return (`FFmpeg timed out after ${timeoutMs} ms without progress (ffmpegProcessTimeout, ` +
             `FFMPEG_PROCESS_TIMEOUT_MS). ${error}`);
     }
     if (result.exitCode !== null)
@@ -147,25 +147,22 @@ const ABORTED_BEFORE_START = {
 export async function runFfmpeg(args, opts) {
     if (opts?.signal?.aborted)
         return { ...ABORTED_BEFORE_START };
-    const stallTimeout = opts?.stallTimeout;
     let framesWritten = 0;
-    const readStats = ffmpegStatsReader(({ frames }) => {
-        if (frames === undefined || frames <= framesWritten)
+    let secondsWritten = 0;
+    const readStats = ffmpegStatsReader(({ frames = 0, seconds = 0 }) => {
+        if (frames <= framesWritten && seconds <= secondsWritten)
             return;
-        framesWritten = frames;
+        framesWritten = Math.max(frames, framesWritten);
+        secondsWritten = Math.max(seconds, secondsWritten);
         managed.markActivity();
     });
     const managed = new ManagedChildProcess(spawnFfmpeg(args), {
         signal: opts?.signal,
-        ...(stallTimeout === undefined
-            ? { deadlineAtMs: Date.now() + (opts?.timeout ?? DEFAULT_TIMEOUT) }
-            : { inactivityTimeoutMs: stallTimeout }),
-        onStderr: stallTimeout === undefined
-            ? opts?.onStderr
-            : (chunk) => {
-                readStats(chunk);
-                opts?.onStderr?.(chunk);
-            },
+        inactivityTimeoutMs: opts?.timeout ?? DEFAULT_TIMEOUT,
+        onStderr: (chunk) => {
+            readStats(chunk);
+            opts?.onStderr?.(chunk);
+        },
     });
     return toResult(await managed.wait());
 }

package/dist/utils/urlDownloader.js

@@ -586,7 +586,7 @@ function checksumMismatchError(source, status, telemetry) {
 // Response protocol, streamed byte accounting, hashes, and payload validation
 // share one lifecycle so no validation can happen after publication.
 // fallow-ignore-next-line complexity
-async function fetchToPartial(url, partialPath, controller, options) {
+async function fetchToPartial(url, partialPath, controller, options, onSizedBodyBytes) {
     const { response, finalUrl } = await fetchWithValidatedRedirects(url, controller);
     const finalIdentity = safeDownloadUrlIdentity(finalUrl);
     const rangeDisposition = classifyRangeDisposition(response);
@@ -643,6 +643,8 @@ async function fetchToPartial(url, partialPath, controller, options) {
     const inspector = new Transform({
         transform(chunk, _encoding, callback) {
             const bytes = Buffer.isBuffer(chunk) ? chunk : Buffer.from(chunk);
+            if (declaredLength !== undefined)
+                onSizedBodyBytes();
             receivedBytes += bytes.length;
             sha256.update(bytes);
             md5.update(bytes);
@@ -771,16 +773,26 @@ async function runDownloadAttempt(url, localPath, timeoutMs, attempt, options, s
         controller.abort();
     };
     signal?.addEventListener("abort", onCallerAbort, { once: true });
-    const timeoutId = setTimeout(() => {
-        timedOut = true;
-        controller.abort();
-    }, timeoutMs);
+    let timeoutId;
+    const armTimeout = () => {
+        clearTimeout(timeoutId);
+        timeoutId = setTimeout(() => {
+            timedOut = true;
+            controller.abort();
+        }, timeoutMs);
+    };
+    armTimeout();
+    let sizedBodyStarted = false;
+    const extendTimeout = () => {
+        sizedBodyStarted = true;
+        armTimeout();
+    };
     const identity = safeDownloadUrlIdentity(url);
     try {
         if (callerAborted) {
             throw new UrlDownloadError("cancelled", false, "Download cancelled");
         }
-        const integrity = await fetchToPartial(url, partialPath, controller, options);
+        const integrity = await fetchToPartial(url, partialPath, controller, options, extendTimeout);
         let outcome = syncAndPublishPartial(partialPath, localPath);
         let publishedIntegrity = integrity;
         if (outcome === "race_reused") {
@@ -827,7 +839,9 @@ async function runDownloadAttempt(url, localPath, timeoutMs, attempt, options, s
             throw classified;
         }
         if (timedOut) {
-            const classified = new UrlDownloadError("timeout", true, `Download timeout after ${timeoutMs / 1000}s`);
+            const classified = new UrlDownloadError("timeout", true, sizedBodyStarted
+                ? `Download stalled: no data for ${timeoutMs / 1000}s`
+                : `Download timeout after ${timeoutMs / 1000}s`);
             emitDownloadTelemetry(options, {
                 urlFingerprint: identity.urlFingerprint,
                 initialHost: identity.host,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hyperframes/engine",
-  "version": "0.8.199",
+  "version": "0.8.201",
   "description": "Seekable web page to video rendering engine (Puppeteer + FFmpeg)",
   "repository": {
     "type": "git",
@@ -50,8 +50,8 @@
     "linkedom": "^0.18.12",
     "puppeteer": "^25.8.0",
     "puppeteer-core": "^25.8.0",
-    "@hyperframes/core": "^0.8.199",
-    "@hyperframes/parsers": "^0.8.199"
+    "@hyperframes/core": "^0.8.201",
+    "@hyperframes/parsers": "^0.8.201"
   },
   "devDependencies": {
     "@types/node": "^25.0.10",