@remotion/serverless 4.0.516 → 4.0.517

package/dist/artifact-registry.d.ts

@@ -0,0 +1,23 @@
+import type { EmittedArtifact } from '@remotion/renderer';
+type ArtifactRegistration = {
+    chunk: number;
+    frame: number;
+    attempt: number;
+    filename: string;
+};
+export type ArtifactRegistrationResult = {
+    type: 'accepted';
+} | {
+    type: 'retry-replay';
+} | {
+    type: 'conflict';
+};
+export type OnArtifactFromRenderer = (params: {
+    artifact: EmittedArtifact;
+    chunk: number;
+    attempt: number;
+}) => ArtifactRegistrationResult;
+export declare const makeArtifactRegistry: () => {
+    registerArtifact: (artifact: ArtifactRegistration) => ArtifactRegistrationResult;
+};
+export {};

package/dist/artifact-registry.js

@@ -0,0 +1,31 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.makeArtifactRegistry = void 0;
+const makeArtifactRegistry = () => {
+    const artifactsByFilename = new Map();
+    const artifactsByAttempt = new Set();
+    const registerArtifact = (artifact) => {
+        const attemptKey = JSON.stringify([
+            artifact.chunk,
+            artifact.attempt,
+            artifact.filename,
+        ]);
+        if (artifactsByAttempt.has(attemptKey)) {
+            return { type: 'conflict' };
+        }
+        artifactsByAttempt.add(attemptKey);
+        const existingArtifact = artifactsByFilename.get(artifact.filename);
+        if (!existingArtifact) {
+            artifactsByFilename.set(artifact.filename, artifact);
+            return { type: 'accepted' };
+        }
+        if (existingArtifact.chunk === artifact.chunk &&
+            existingArtifact.frame === artifact.frame &&
+            artifact.attempt > existingArtifact.attempt) {
+            return { type: 'retry-replay' };
+        }
+        return { type: 'conflict' };
+    };
+    return { registerArtifact };
+};
+exports.makeArtifactRegistry = makeArtifactRegistry;

package/dist/handlers/launch.js

@@ -6,6 +6,7 @@ const path_1 = require("path");
 const renderer_1 = require("@remotion/renderer");
 const serverless_client_1 = require("@remotion/serverless-client");
 const serverless_client_2 = require("@remotion/serverless-client");
+const artifact_registry_1 = require("../artifact-registry");
 const cleanup_props_1 = require("../cleanup-props");
 const find_output_file_in_bucket_1 = require("../find-output-file-in-bucket");
 const get_tmp_dir_1 = require("../get-tmp-dir");
@@ -298,12 +299,17 @@ const innerLaunchHandler = async ({ params, options, overallProgress, registerCl
     }
     (0, fs_1.mkdirSync)(outdir);
     const files = [];
-    const onArtifact = (artifact) => {
+    const artifactRegistry = (0, artifact_registry_1.makeArtifactRegistry)();
+    const onArtifact = ({ artifact, chunk, attempt }) => {
         var _a;
-        if (overallProgress
-            .getReceivedArtifacts()
-            .find((a) => a.filename === artifact.filename)) {
-            return { alreadyExisted: true };
+        const artifactRegistration = artifactRegistry.registerArtifact({
+            chunk,
+            frame: artifact.frame,
+            attempt,
+            filename: artifact.filename,
+        });
+        if (artifactRegistration.type !== 'accepted') {
+            return artifactRegistration;
         }
         const region = insideFunctionSpecifics.getCurrentRegionInFunction();
         const storageKey = (0, serverless_client_2.artifactName)(renderMetadata.renderId, artifact.filename);
@@ -340,16 +346,16 @@ const innerLaunchHandler = async ({ params, options, overallProgress, registerCl
                 stack: err.stack,
                 tmpDir: null,
                 frame: artifact.frame,
-                chunk: null,
+                chunk,
                 isFatal: false,
-                attempt: 1,
+                attempt,
                 willRetry: false,
                 totalAttempts: 1,
             });
             overallProgress.upload('artifactWriteError');
             renderer_1.RenderInternals.Log.error({ indent: false, logLevel: params.logLevel }, 'Failed to write artifact to S3', err);
         });
-        return { alreadyExisted: false };
+        return artifactRegistration;
     };
     await Promise.all(lambdaPayloads.map(async (payload) => {
         await (0, stream_renderer_1.renderRendererFunctionWithRetry)({

package/dist/stream-renderer.d.ts

@@ -1,5 +1,5 @@
-import type { EmittedArtifact } from '@remotion/renderer';
 import type { CloudProvider, ProviderSpecifics, ServerlessPayload } from '@remotion/serverless-client';
+import type { OnArtifactFromRenderer } from './artifact-registry';
 import type { OverallProgressHelper } from './overall-render-progress';
 import type { InsideFunctionSpecifics } from './provider-implementation';
 export declare const renderRendererFunctionWithRetry: <Provider extends CloudProvider<string, Record<string, unknown>, Record<string, unknown>, string, object>>({ payload, files, functionName, outdir, overallProgress, logLevel, onArtifact, providerSpecifics, insideFunctionSpecifics, requestHandler, expectedBucketOwner, }: {
@@ -9,9 +9,7 @@ export declare const renderRendererFunctionWithRetry: <Provider extends CloudPro
     overallProgress: OverallProgressHelper<Provider>;
     files: string[];
     logLevel: "error" | "info" | "trace" | "verbose" | "warn";
-    onArtifact: (asset: EmittedArtifact) => {
-        alreadyExisted: boolean;
-    };
+    onArtifact: OnArtifactFromRenderer;
     providerSpecifics: ProviderSpecifics<Provider>;
     insideFunctionSpecifics: InsideFunctionSpecifics<Provider>;
     requestHandler: Provider["requestHandler"] | null;

package/dist/stream-renderer.js

@@ -76,14 +76,21 @@ const streamRenderer = ({ payload, functionName, outdir, overallProgress, files,
             if (message.type === 'artifact-emitted') {
                 const artifact = (0, serverless_client_1.deserializeArtifact)(message.payload.artifact);
                 renderer_1.RenderInternals.Log.info({ indent: false, logLevel }, `Received artifact on frame ${message.payload.artifact.frame}:`, artifact.filename, artifact.content.length + 'bytes.');
-                const { alreadyExisted } = onArtifact(artifact);
-                if (alreadyExisted) {
+                const artifactRegistration = onArtifact({
+                    artifact,
+                    chunk: payload.chunk,
+                    attempt: payload.attempt,
+                });
+                if (artifactRegistration.type === 'conflict') {
                     return resolve({
                         type: 'error',
                         error: `Chunk ${payload.chunk} emitted an asset filename ${message.payload.artifact.filename} at frame ${message.payload.artifact.frame} but there is already another artifact with the same name. https://remotion.dev/docs/artifacts`,
                         shouldRetry: false,
                     });
                 }
+                if (artifactRegistration.type === 'retry-replay') {
+                    renderer_1.RenderInternals.Log.info({ indent: false, logLevel }, `Ignoring artifact ${artifact.filename} replayed by attempt ${payload.attempt} of chunk ${payload.chunk}`);
+                }
                 return;
             }
             if (message.type === 'error-occurred') {
@@ -309,14 +316,21 @@ const s3Renderer = async ({ payload, functionName, outdir, overallProgress, file
                     downloadedKeys.push(key);
                 }
                 for (const emittedArtifact of emittedArtifacts) {
-                    const { alreadyExisted } = onArtifact(emittedArtifact);
-                    if (alreadyExisted) {
+                    const artifactRegistration = onArtifact({
+                        artifact: emittedArtifact,
+                        chunk: payload.chunk,
+                        attempt: payload.attempt,
+                    });
+                    if (artifactRegistration.type === 'conflict') {
                         return {
                             type: 'error',
                             error: `Chunk ${payload.chunk} emitted an asset filename ${emittedArtifact.filename} at frame ${emittedArtifact.frame} but there is already another artifact with the same name. https://remotion.dev/docs/artifacts`,
                             shouldRetry: false,
                         };
                     }
+                    if (artifactRegistration.type === 'retry-replay') {
+                        renderer_1.RenderInternals.Log.info({ indent: false, logLevel }, `Ignoring artifact ${emittedArtifact.filename} replayed by attempt ${payload.attempt} of chunk ${payload.chunk}`);
+                    }
                 }
                 files.push(...downloadedFiles);
                 overallProgress.addChunkCompleted(payload.chunk, status.startedAt, status.completedAt);

package/package.json

@@ -3,7 +3,7 @@
 		"url": "https://github.com/remotion-dev/remotion/tree/main/packages/serverless"
 	},
 	"name": "@remotion/serverless",
-	"version": "4.0.516",
+	"version": "4.0.517",
 	"description": "A runtime for distributed rendering",
 	"main": "dist",
 	"scripts": {
@@ -23,13 +23,13 @@
 		"access": "public"
 	},
 	"dependencies": {
-		"@remotion/renderer": "4.0.516",
-		"@remotion/bundler": "4.0.516",
-		"@remotion/licensing": "4.0.516",
-		"@remotion/serverless-client": "4.0.516"
+		"@remotion/renderer": "4.0.517",
+		"@remotion/bundler": "4.0.517",
+		"@remotion/licensing": "4.0.517",
+		"@remotion/serverless-client": "4.0.517"
 	},
 	"devDependencies": {
-		"@remotion/eslint-config-internal": "4.0.516",
+		"@remotion/eslint-config-internal": "4.0.517",
 		"eslint": "9.19.0",
 		"@typescript/native-preview": "7.0.0-dev.20260217.1"
 	},