@remotion/studio-server 4.0.512 → 4.0.514

package/dist/preview-server/routes/can-update-sequence-props.js

@@ -33,7 +33,7 @@ var __importStar = (this && this.__importStar) || (function () {
     };
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.computeSequencePropsStatusFromFilenameByLocation = exports.computeSequencePropsStatus = exports.computeSequencePropsStatusFromContent = exports.lineColumnToNodePath = exports.findNodePathForJsxElement = exports.getStaticJsxTextContent = exports.hasJsxChildrenAttribute = exports.getStaticJsxChildrenAttribute = exports.findJsxElementNodeAtNodePath = exports.findJsxElementAtNodePath = exports.findJsxElementPathAtNodePath = exports.getNodePathForRecastPath = exports.getComputedStatus = exports.extractStaticValue = exports.isStaticValue = void 0;
+exports.computeSequencePropsStatusFromFilenameByLocation = exports.computeSequencePropsStatus = exports.computeSequencePropsStatusFromContent = exports.computeSequencePropsStatusFromAst = exports.resolveSequencePropsNodePathsFromFilename = exports.lineColumnsToNodePaths = exports.lineColumnToNodePath = exports.findNodePathForJsxElement = exports.getStaticJsxTextContent = exports.hasJsxChildrenAttribute = exports.getStaticJsxChildrenAttribute = exports.findJsxElementNodeAtNodePath = exports.findJsxElementAtNodePath = exports.findJsxElementPathAtNodePath = exports.getNodePathForRecastPath = exports.getComputedStatus = exports.extractStaticValue = exports.isStaticValue = exports.takeCachedSequencePropsStatusAst = void 0;
 const node_fs_1 = require("node:fs");
 const renderer_1 = require("@remotion/renderer");
 const studio_shared_1 = require("@remotion/studio-shared");
@@ -55,8 +55,41 @@ const can_update_effect_props_1 = require("./can-update-effect-props");
 // computation is read-only, so all subscribers can share one parsed snapshot
 // until the notification burst has finished.
 let cachedSequencePropsStatusAst = null;
+// A subsequent save can consume the last read-only snapshot if the file has
+// not changed. The save mutates the AST, so it must only be reused once.
+let reusableSequencePropsStatusAst = null;
+const getCachedSequencePropsStatusAst = (fileContents) => {
+    if ((cachedSequencePropsStatusAst === null || cachedSequencePropsStatusAst === void 0 ? void 0 : cachedSequencePropsStatusAst.fileContents) !== fileContents) {
+        const snapshot = {
+            fileContents,
+            ast: (0, parse_ast_1.parseAst)(fileContents),
+            videoConfigIdentifierValues: new Map(),
+        };
+        cachedSequencePropsStatusAst = snapshot;
+        reusableSequencePropsStatusAst = snapshot;
+        queueMicrotask(() => {
+            if (cachedSequencePropsStatusAst === snapshot) {
+                cachedSequencePropsStatusAst = null;
+            }
+        });
+    }
+    return cachedSequencePropsStatusAst;
+};
+const takeCachedSequencePropsStatusAst = (fileContents) => {
+    if ((reusableSequencePropsStatusAst === null || reusableSequencePropsStatusAst === void 0 ? void 0 : reusableSequencePropsStatusAst.fileContents) !== fileContents) {
+        return null;
+    }
+    const { ast } = reusableSequencePropsStatusAst;
+    reusableSequencePropsStatusAst = null;
+    if ((cachedSequencePropsStatusAst === null || cachedSequencePropsStatusAst === void 0 ? void 0 : cachedSequencePropsStatusAst.ast) === ast) {
+        cachedSequencePropsStatusAst = null;
+    }
+    return ast;
+};
+exports.takeCachedSequencePropsStatusAst = takeCachedSequencePropsStatusAst;
 const staticStatus = (codeValue, numericExpression) => ({
     status: 'static',
+    keyframeDisplayOffsetAdjustment: null,
     codeValue,
     ...(numericExpression === null || numericExpression.type === 'literal'
         ? {}
@@ -389,13 +422,20 @@ const getInterpolationKeyframes = (node, ast, videoConfigValues) => {
     const outputArg = callExpression.arguments[2];
     if (!frameArg ||
         frameArg.type === 'SpreadElement' ||
-        !isCurrentFrameIdentifier(frameArg, ast) ||
         !inputArg ||
         !outputArg ||
         inputArg.type !== 'ArrayExpression' ||
         outputArg.type !== 'ArrayExpression') {
         return undefined;
     }
+    const frameDisplayOffset = getCurrentFrameDisplayOffsetAdjustment({
+        node: frameArg,
+        ast,
+        videoConfigValues,
+    });
+    if (frameDisplayOffset === null) {
+        return undefined;
+    }
     if (inputArg.elements.length !== outputArg.elements.length) {
         return undefined;
     }
@@ -436,39 +476,283 @@ const getInterpolationKeyframes = (node, ast, videoConfigValues) => {
         clamping: metadata.clamping,
         posterize: metadata.posterize,
         output: metadata.output,
+        keyframeDisplayOffsetAdjustment: frameDisplayOffset.hasEnclosingElement
+            ? frameDisplayOffset.adjustment
+            : null,
     };
 };
-const isUseCurrentFrameCall = (node) => {
-    return (node.type === 'CallExpression' &&
-        node.callee.type === 'Identifier' &&
-        node.callee.name === 'useCurrentFrame' &&
-        node.arguments.length === 0);
+const getRemotionImportNames = ({ ast, importedName, }) => {
+    var _a;
+    var _b, _c;
+    const direct = new Set();
+    const namespaces = new Set();
+    for (const statement of ast.program.body) {
+        if (statement.type !== 'ImportDeclaration' ||
+            statement.source.value !== 'remotion') {
+            continue;
+        }
+        for (const specifier of (_b = statement.specifiers) !== null && _b !== void 0 ? _b : []) {
+            if (specifier.type === 'ImportSpecifier' &&
+                ((specifier.imported.type === 'Identifier' &&
+                    specifier.imported.name === importedName) ||
+                    (specifier.imported.type === 'StringLiteral' &&
+                        specifier.imported.value === importedName))) {
+                direct.add((_c = (_a = specifier.local) === null || _a === void 0 ? void 0 : _a.name) !== null && _c !== void 0 ? _c : importedName);
+            }
+            if (specifier.type === 'ImportNamespaceSpecifier') {
+                namespaces.add(specifier.local.name);
+            }
+        }
+    }
+    return { direct, namespaces };
+};
+const isUseCurrentFrameCall = (node, ast) => {
+    if (node.type !== 'CallExpression' || node.arguments.length !== 0) {
+        return false;
+    }
+    const imports = getRemotionImportNames({
+        ast,
+        importedName: 'useCurrentFrame',
+    });
+    if (node.callee.type === 'Identifier') {
+        return imports.direct.has(node.callee.name);
+    }
+    return (node.callee.type === 'MemberExpression' &&
+        !node.callee.computed &&
+        node.callee.object.type === 'Identifier' &&
+        node.callee.property.type === 'Identifier' &&
+        imports.namespaces.has(node.callee.object.name) &&
+        node.callee.property.name === 'useCurrentFrame');
+};
+const findNodePath = (ast, target) => {
+    let found = null;
+    recast.types.visit(ast, {
+        visitNode(p) {
+            if (p.node === target) {
+                found = p;
+                return false;
+            }
+            return this.traverse(p);
+        },
+    });
+    return found;
+};
+const getJsxNumericAttribute = ({ openingElement, name, defaultValue, videoConfigValues, }) => {
+    var _a, _b;
+    var _c;
+    if (openingElement.attributes.some((attribute) => attribute.type === 'JSXSpreadAttribute')) {
+        return null;
+    }
+    for (let index = openingElement.attributes.length - 1; index >= 0; index--) {
+        const attribute = openingElement.attributes[index];
+        if (attribute.type !== 'JSXAttribute' ||
+            attribute.name.type !== 'JSXIdentifier' ||
+            attribute.name.name !== name) {
+            continue;
+        }
+        if (((_a = attribute.value) === null || _a === void 0 ? void 0 : _a.type) !== 'JSXExpressionContainer' ||
+            attribute.value.expression.type === 'JSXEmptyExpression') {
+            return null;
+        }
+        return ((_c = (_b = (0, video_config_numeric_expression_1.parseVideoConfigNumericExpression)({
+            node: attribute.value.expression,
+            videoConfigValues,
+        })) === null || _b === void 0 ? void 0 : _b.value) !== null && _c !== void 0 ? _c : null);
+    }
+    return defaultValue;
+};
+const getFrameDisplayOffsetAdjustmentBetweenPaths = ({ startPath, endPath, videoConfigValues, }) => {
+    let current = startPath;
+    let hasSeenControlledElement = false;
+    let hasEnclosingElement = false;
+    let adjustment = 0;
+    while (current && current.value !== endPath.value) {
+        const currentNode = current.value;
+        if (currentNode.type === 'FunctionDeclaration' ||
+            currentNode.type === 'FunctionExpression' ||
+            currentNode.type === 'ArrowFunctionExpression') {
+            return null;
+        }
+        if (currentNode.type === 'JSXElement') {
+            if (!hasSeenControlledElement) {
+                hasSeenControlledElement = true;
+            }
+            else {
+                hasEnclosingElement = true;
+                // Sequence-backed built-ins and userland components can both shift
+                // their children, so the prop names are the semantic boundary.
+                const from = getJsxNumericAttribute({
+                    openingElement: currentNode.openingElement,
+                    name: 'from',
+                    defaultValue: 0,
+                    videoConfigValues,
+                });
+                const trimBefore = getJsxNumericAttribute({
+                    openingElement: currentNode.openingElement,
+                    name: 'trimBefore',
+                    defaultValue: 0,
+                    videoConfigValues,
+                });
+                if (from === null || trimBefore === null) {
+                    return null;
+                }
+                adjustment -= from - trimBefore;
+            }
+        }
+        current = current.parentPath;
+    }
+    return current ? { adjustment, hasEnclosingElement } : null;
 };
-const isCurrentFrameIdentifier = (node, ast) => {
+const getDefaultFrameDisplayOffsetAdjustment = ({ jsxPath, videoConfigValues, }) => {
+    var _a;
+    let functionPath = jsxPath.parentPath;
+    while (functionPath) {
+        const node = functionPath.value;
+        if (node.type === 'FunctionDeclaration' ||
+            node.type === 'FunctionExpression' ||
+            node.type === 'ArrowFunctionExpression') {
+            break;
+        }
+        functionPath = functionPath.parentPath;
+    }
+    if (!functionPath) {
+        return null;
+    }
+    const result = getFrameDisplayOffsetAdjustmentBetweenPaths({
+        startPath: jsxPath,
+        endPath: functionPath,
+        videoConfigValues,
+    });
+    return (_a = result === null || result === void 0 ? void 0 : result.adjustment) !== null && _a !== void 0 ? _a : null;
+};
+const resolveCurrentFrameExpression = ({ node, ast, seenDeclarations, }) => {
+    var _a;
     if (node.type === 'TSAsExpression') {
-        return isCurrentFrameIdentifier(node.expression, ast);
+        return resolveCurrentFrameExpression({
+            node: node.expression,
+            ast,
+            seenDeclarations,
+        });
+    }
+    if (node.type === 'BinaryExpression' &&
+        (node.operator === '+' || node.operator === '-')) {
+        const rightValue = getNumericValue(node.right);
+        if (rightValue !== null && Number.isFinite(rightValue)) {
+            const resolvedLeft = resolveCurrentFrameExpression({
+                node: node.left,
+                ast,
+                seenDeclarations,
+            });
+            if (resolvedLeft !== null) {
+                return {
+                    ...resolvedLeft,
+                    offset: resolvedLeft.offset +
+                        (node.operator === '+' ? rightValue : -rightValue),
+                };
+            }
+        }
+        if (node.operator === '+') {
+            const leftValue = getNumericValue(node.left);
+            if (leftValue !== null && Number.isFinite(leftValue)) {
+                const resolvedRight = resolveCurrentFrameExpression({
+                    node: node.right,
+                    ast,
+                    seenDeclarations,
+                });
+                if (resolvedRight !== null) {
+                    return {
+                        ...resolvedRight,
+                        offset: resolvedRight.offset + leftValue,
+                    };
+                }
+            }
+        }
+        return null;
     }
     if (node.type !== 'Identifier') {
-        return false;
+        return null;
     }
-    let hasUseCurrentFrameDeclaration = false;
-    let hasOtherDeclaration = false;
-    recast.types.visit(ast, {
+    const nodePath = findNodePath(ast, node);
+    const bindingScope = (_a = nodePath === null || nodePath === void 0 ? void 0 : nodePath.scope) === null || _a === void 0 ? void 0 : _a.lookup(node.name);
+    if (!nodePath || !bindingScope) {
+        return null;
+    }
+    let matchingDeclarations = 0;
+    let matchingDeclaration = null;
+    let initializer = null;
+    let isConstDeclaration = false;
+    recast.types.visit(bindingScope.path, {
         visitVariableDeclarator(p) {
+            var _a;
             const { id, init } = p.node;
-            if (id.type !== 'Identifier' || id.name !== node.name) {
+            if (id.type !== 'Identifier' ||
+                id.name !== node.name ||
+                ((_a = p.scope.lookup(node.name)) === null || _a === void 0 ? void 0 : _a.path.node) !== bindingScope.path.node) {
                 return this.traverse(p);
             }
-            if (init && isUseCurrentFrameCall(init)) {
-                hasUseCurrentFrameDeclaration = true;
-            }
-            else {
-                hasOtherDeclaration = true;
+            matchingDeclarations++;
+            const declaration = p.parentPath.node;
+            if (init) {
+                matchingDeclaration = p.node;
+                initializer = init;
+                isConstDeclaration =
+                    declaration.type === 'VariableDeclaration' &&
+                        declaration.kind === 'const';
             }
             return false;
         },
     });
-    return hasUseCurrentFrameDeclaration && !hasOtherDeclaration;
+    if (matchingDeclarations !== 1 ||
+        matchingDeclaration === null ||
+        initializer === null ||
+        seenDeclarations.has(matchingDeclaration)) {
+        return null;
+    }
+    if (isUseCurrentFrameCall(initializer, ast)) {
+        return { bindingScopePath: bindingScope.path, offset: 0 };
+    }
+    if (!isConstDeclaration) {
+        return null;
+    }
+    const nextSeenDeclarations = new Set(seenDeclarations);
+    nextSeenDeclarations.add(matchingDeclaration);
+    return resolveCurrentFrameExpression({
+        node: initializer,
+        ast,
+        seenDeclarations: nextSeenDeclarations,
+    });
+};
+const getCurrentFrameDisplayOffsetAdjustment = ({ node, ast, videoConfigValues, }) => {
+    if (node.type === 'TSAsExpression') {
+        return getCurrentFrameDisplayOffsetAdjustment({
+            node: node.expression,
+            ast,
+            videoConfigValues,
+        });
+    }
+    const nodePath = findNodePath(ast, node);
+    const resolved = resolveCurrentFrameExpression({
+        node,
+        ast,
+        seenDeclarations: new Set(),
+    });
+    if (!nodePath || resolved === null) {
+        return null;
+    }
+    const frameDisplayOffset = getFrameDisplayOffsetAdjustmentBetweenPaths({
+        startPath: nodePath,
+        endPath: resolved.bindingScopePath,
+        videoConfigValues,
+    });
+    if (frameDisplayOffset === null) {
+        return null;
+    }
+    return {
+        ...frameDisplayOffset,
+        // interpolate(frame + offset, [keyframe], ...) reaches the keyframe at
+        // composition frame `keyframe - offset`.
+        adjustment: frameDisplayOffset.adjustment - resolved.offset,
+    };
 };
 const getComputedStatus = (node, ast, videoConfigValues) => {
     const interpolation = getInterpolationKeyframes(node, ast, videoConfigValues);
@@ -478,6 +762,7 @@ const getComputedStatus = (node, ast, videoConfigValues) => {
     return {
         status: 'keyframed',
         interpolationFunction: interpolation.interpolationFunction,
+        keyframeDisplayOffsetAdjustment: interpolation.keyframeDisplayOffsetAdjustment,
         keyframes: interpolation.keyframes,
         easing: interpolation.easing,
         clamping: interpolation.clamping,
@@ -567,17 +852,17 @@ const findJsxElementAtNodePath = (ast, nodePath) => {
     return current ? current.value : null;
 };
 exports.findJsxElementAtNodePath = findJsxElementAtNodePath;
-const findJsxElementNodeAtNodePath = (ast, nodePath) => {
+const getJsxElementNodeFromJsxPath = (jsxPath) => {
     var _a;
-    const current = (0, exports.findJsxElementPathAtNodePath)(ast, nodePath);
-    if (!current) {
-        return null;
-    }
-    if (recast.types.namedTypes.JSXElement.check((_a = current.parentPath) === null || _a === void 0 ? void 0 : _a.value)) {
-        return current.parentPath.value;
+    if (recast.types.namedTypes.JSXElement.check((_a = jsxPath.parentPath) === null || _a === void 0 ? void 0 : _a.value)) {
+        return jsxPath.parentPath.value;
     }
     return null;
 };
+const findJsxElementNodeAtNodePath = (ast, nodePath) => {
+    const current = (0, exports.findJsxElementPathAtNodePath)(ast, nodePath);
+    return current ? getJsxElementNodeFromJsxPath(current) : null;
+};
 exports.findJsxElementNodeAtNodePath = findJsxElementNodeAtNodePath;
 const findJsxChildrenAttribute = (jsxElement) => {
     const childrenAttr = jsxElement.attributes.find((attr) => {
@@ -648,8 +933,7 @@ const findNodePathForJsxElement = (ast, target) => {
 };
 exports.findNodePathForJsxElement = findNodePathForJsxElement;
 const RECAST_TAB_WIDTH = 4;
-const sourceColumnToRecastColumn = ({ fileContents, line, column, }) => {
-    const sourceLine = fileContents.split('\n')[line - 1];
+const sourceColumnToRecastColumn = ({ sourceLine, column, }) => {
     if (sourceLine === undefined) {
         return column;
     }
@@ -671,8 +955,7 @@ const lineColumnToNodePath = (ast, targetLine, targetColumn, fileContents) => {
     const recastTargetColumn = targetColumn === undefined || fileContents === undefined
         ? targetColumn
         : sourceColumnToRecastColumn({
-            fileContents,
-            line: targetLine,
+            sourceLine: fileContents.split('\n')[targetLine - 1],
             column: targetColumn,
         });
     recast.types.visit(ast, {
@@ -694,6 +977,55 @@ const lineColumnToNodePath = (ast, targetLine, targetColumn, fileContents) => {
     return (_a = lineMatches.at(-1)) !== null && _a !== void 0 ? _a : null;
 };
 exports.lineColumnToNodePath = lineColumnToNodePath;
+const lineColumnsToNodePaths = ({ ast, targets, fileContents, }) => {
+    const sourceLines = fileContents.split('\n');
+    const targetIndicesByLine = new Map();
+    const recastTargetColumns = targets.map(({ line, column }, index) => {
+        var _a;
+        const indices = (_a = targetIndicesByLine.get(line)) !== null && _a !== void 0 ? _a : [];
+        indices.push(index);
+        targetIndicesByLine.set(line, indices);
+        return sourceColumnToRecastColumn({
+            sourceLine: sourceLines[line - 1],
+            column,
+        });
+    });
+    const lineMatches = targets.map(() => null);
+    const exactMatches = targets.map(() => null);
+    const exactMatchCounts = targets.map(() => 0);
+    recast.types.visit(ast, {
+        visitJSXOpeningElement(p) {
+            var _a, _b;
+            const { node } = p;
+            const line = (_a = node.loc) === null || _a === void 0 ? void 0 : _a.start.line;
+            const targetIndices = line ? targetIndicesByLine.get(line) : undefined;
+            if (targetIndices) {
+                const nodePath = (0, exports.getNodePathForRecastPath)(p, ast);
+                for (const index of targetIndices) {
+                    lineMatches[index] = nodePath;
+                    if (((_b = node.loc) === null || _b === void 0 ? void 0 : _b.start.column) === recastTargetColumns[index]) {
+                        exactMatches[index] = nodePath;
+                        exactMatchCounts[index]++;
+                    }
+                }
+            }
+            return this.traverse(p);
+        },
+    });
+    return targets.map((_, index) => exactMatchCounts[index] === 1 ? exactMatches[index] : lineMatches[index]);
+};
+exports.lineColumnsToNodePaths = lineColumnsToNodePaths;
+const resolveSequencePropsNodePathsFromFilename = ({ fileName, targets, remotionRoot, }) => {
+    const { absolutePath } = (0, resolve_file_inside_project_1.resolveFileInsideProject)({
+        remotionRoot,
+        fileName,
+        action: 'read',
+    });
+    const fileContents = (0, node_fs_1.readFileSync)(absolutePath, 'utf-8');
+    const { ast } = getCachedSequencePropsStatusAst(fileContents);
+    return (0, exports.lineColumnsToNodePaths)({ ast, targets, fileContents });
+};
+exports.resolveSequencePropsNodePathsFromFilename = resolveSequencePropsNodePathsFromFilename;
 const PIXEL_VALUE_REGEX = /^-?\d+(\.\d+)?px$/;
 const isSupportedTranslateValue = (value) => {
     const parts = value.split(/\s+/);
@@ -924,35 +1256,12 @@ const computeSequenceOnlyPropsRecord = ({ jsxElement, jsxElementNode, ast, keys,
     }
     return filteredProps;
 };
-const computeSequencePropsStatusFromContent = ({ fileContents, nodePath, componentIdentity, keys, assetKeys = [], effects, videoConfigValues, }) => {
+const computeSequencePropsStatusFromAstAndIdentifiers = ({ ast, nodePath, componentIdentity, keys, assetKeys, effects, videoConfigIdentifierValues, }) => {
     var _a;
-    if ((cachedSequencePropsStatusAst === null || cachedSequencePropsStatusAst === void 0 ? void 0 : cachedSequencePropsStatusAst.fileContents) !== fileContents) {
-        cachedSequencePropsStatusAst = null;
-        const snapshot = {
-            fileContents,
-            ast: (0, parse_ast_1.parseAst)(fileContents),
-            videoConfigIdentifierValues: new Map(),
-        };
-        cachedSequencePropsStatusAst = snapshot;
-        queueMicrotask(() => {
-            if (cachedSequencePropsStatusAst === snapshot) {
-                cachedSequencePropsStatusAst = null;
-            }
-        });
-    }
-    const { ast } = cachedSequencePropsStatusAst;
-    const videoConfigCacheKey = JSON.stringify(videoConfigValues);
-    let videoConfigIdentifierValues = cachedSequencePropsStatusAst.videoConfigIdentifierValues.get(videoConfigCacheKey);
-    if (videoConfigIdentifierValues === undefined) {
-        videoConfigIdentifierValues = (0, video_config_values_1.getVideoConfigIdentifierValues)({
-            ast,
-            videoConfigValues,
-        });
-        cachedSequencePropsStatusAst.videoConfigIdentifierValues.set(videoConfigCacheKey, videoConfigIdentifierValues);
-    }
-    const jsxElementNode = (0, exports.findJsxElementNodeAtNodePath)(ast, nodePath);
+    const jsxPath = (0, exports.findJsxElementPathAtNodePath)(ast, nodePath);
+    const jsxElementNode = jsxPath ? getJsxElementNodeFromJsxPath(jsxPath) : null;
     const jsxElement = (_a = jsxElementNode === null || jsxElementNode === void 0 ? void 0 : jsxElementNode.openingElement) !== null && _a !== void 0 ? _a : null;
-    if (!jsxElement || !jsxElementNode) {
+    if (!jsxPath || !jsxElement || !jsxElementNode) {
         throw new jsx_element_not_found_at_location_error_1.JsxElementNotFoundAtLocationError();
     }
     if (!(0, jsx_component_identity_1.jsxComponentIdentitiesMatch)({
@@ -975,12 +1284,79 @@ const computeSequencePropsStatusFromContent = ({ fileContents, nodePath, compone
         effects,
         videoConfigValues: videoConfigIdentifierValues,
     });
+    const defaultKeyframeDisplayOffsetAdjustment = getDefaultFrameDisplayOffsetAdjustment({
+        jsxPath,
+        videoConfigValues: videoConfigIdentifierValues,
+    });
+    const addDefaultKeyframeDisplayOffsetAdjustment = (status) => {
+        if (status.status !== 'static') {
+            return status;
+        }
+        if (defaultKeyframeDisplayOffsetAdjustment === null) {
+            return computedStatus();
+        }
+        if (defaultKeyframeDisplayOffsetAdjustment === 0) {
+            return status;
+        }
+        return {
+            ...status,
+            keyframeDisplayOffsetAdjustment: defaultKeyframeDisplayOffsetAdjustment,
+        };
+    };
     return {
         canUpdate: true,
-        props: filteredProps,
-        effects: effectsStatuses,
+        props: Object.fromEntries(Object.entries(filteredProps).map(([key, status]) => [
+            key,
+            addDefaultKeyframeDisplayOffsetAdjustment(status),
+        ])),
+        effects: effectsStatuses.map((effectStatus) => effectStatus.canUpdate
+            ? {
+                ...effectStatus,
+                props: Object.fromEntries(Object.entries(effectStatus.props).map(([key, status]) => [
+                    key,
+                    addDefaultKeyframeDisplayOffsetAdjustment(status),
+                ])),
+            }
+            : effectStatus),
     };
 };
+const computeSequencePropsStatusFromAst = ({ ast, nodePath, componentIdentity, keys, assetKeys = [], effects, videoConfigValues, }) => {
+    return computeSequencePropsStatusFromAstAndIdentifiers({
+        ast,
+        nodePath,
+        componentIdentity,
+        keys,
+        assetKeys,
+        effects,
+        videoConfigIdentifierValues: (0, video_config_values_1.getVideoConfigIdentifierValues)({
+            ast,
+            videoConfigValues,
+        }),
+    });
+};
+exports.computeSequencePropsStatusFromAst = computeSequencePropsStatusFromAst;
+const computeSequencePropsStatusFromContent = ({ fileContents, nodePath, componentIdentity, keys, assetKeys = [], effects, videoConfigValues, }) => {
+    const cachedAst = getCachedSequencePropsStatusAst(fileContents);
+    const { ast } = cachedAst;
+    const videoConfigCacheKey = JSON.stringify(videoConfigValues);
+    let videoConfigIdentifierValues = cachedAst.videoConfigIdentifierValues.get(videoConfigCacheKey);
+    if (videoConfigIdentifierValues === undefined) {
+        videoConfigIdentifierValues = (0, video_config_values_1.getVideoConfigIdentifierValues)({
+            ast,
+            videoConfigValues,
+        });
+        cachedAst.videoConfigIdentifierValues.set(videoConfigCacheKey, videoConfigIdentifierValues);
+    }
+    return computeSequencePropsStatusFromAstAndIdentifiers({
+        ast,
+        nodePath,
+        componentIdentity,
+        keys,
+        assetKeys,
+        effects,
+        videoConfigIdentifierValues,
+    });
+};
 exports.computeSequencePropsStatusFromContent = computeSequencePropsStatusFromContent;
 const computeSequencePropsStatus = ({ fileName, nodePath, componentIdentity, keys, assetKeys = [], effects, remotionRoot, videoConfigValues, }) => {
     const { absolutePath } = (0, resolve_file_inside_project_1.resolveFileInsideProject)({
@@ -1008,7 +1384,7 @@ const computeSequencePropsStatusFromFilenameByLocation = ({ fileName, line, colu
             action: 'read',
         });
         const fileContents = (0, node_fs_1.readFileSync)(absolutePath, 'utf-8');
-        const ast = (0, parse_ast_1.parseAst)(fileContents);
+        const { ast } = getCachedSequencePropsStatusAst(fileContents);
         const resolvedNodePath = (0, exports.lineColumnToNodePath)(ast, line, column, fileContents);
         if (!resolvedNodePath) {
             return {

package/dist/preview-server/routes/default-editor.js

@@ -1,11 +1,14 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getDefaultEditorInfoHandler = void 0;
+const custom_editor_1 = require("../../helpers/custom-editor");
 const editor_registry_1 = require("../../helpers/editor-registry");
 const getDefaultEditorInfoHandler = async ({ getDefaultEditor }) => {
     const installedEditors = await (0, editor_registry_1.getAvailableEditors)();
     const configuredEditor = getDefaultEditor();
-    const customEditor = configuredEditor && typeof configuredEditor === 'object'
+    const customEditor = configuredEditor &&
+        typeof configuredEditor === 'object' &&
+        (0, custom_editor_1.resolveCustomEditorExecutable)(configuredEditor)
         ? configuredEditor
         : null;
     return {

package/dist/preview-server/routes/delete-effect.js

@@ -90,7 +90,7 @@ const deleteEffectHandler = ({ input: effects, remotionRoot, logLevel }) => {
                     absolutePath: update.absolutePath,
                     line: update.logLine,
                 });
-                renderer_1.RenderInternals.Log.info({ indent: false, logLevel }, `${renderer_1.RenderInternals.chalk.blueBright(`${locationLabel}`)} ${(0, formatting_1.strikeThroughOrRemovedPrefix)(deletedEffectDescription)}`);
+                renderer_1.RenderInternals.Log.info({ indent: false, logLevel }, `${(0, source_file_write_queue_1.getCodemodTimingPrefix)(logLevel)}${renderer_1.RenderInternals.chalk.blueBright(`${locationLabel}`)} ${(0, formatting_1.strikeThroughOrRemovedPrefix)(deletedEffectDescription)}`);
                 if (!update.formatted) {
                     (0, log_update_1.warnAboutPrettierOnce)(logLevel);
                 }

package/dist/preview-server/routes/delete-jsx-node.js

@@ -87,7 +87,7 @@ const deleteJsxNodeHandler = ({ input: { nodes }, remotionRoot, logLevel }) => {
                     absolutePath: update.absolutePath,
                     line: update.logLine,
                 });
-                renderer_1.RenderInternals.Log.info({ indent: false, logLevel }, `${renderer_1.RenderInternals.chalk.blueBright(`${locationLabel}`)} Deleted ${deletedNodeDescription}`);
+                renderer_1.RenderInternals.Log.info({ indent: false, logLevel }, `${(0, source_file_write_queue_1.getCodemodTimingPrefix)(logLevel)}${renderer_1.RenderInternals.chalk.blueBright(`${locationLabel}`)} Deleted ${deletedNodeDescription}`);
                 if (!update.formatted) {
                     (0, log_update_1.warnAboutPrettierOnce)(logLevel);
                 }

package/dist/preview-server/routes/duplicate-effect.js

@@ -84,7 +84,7 @@ const duplicateEffectHandler = ({ input: effects, remotionRoot, logLevel }) => {
                     absolutePath: update.absolutePath,
                     line: update.logLine,
                 });
-                renderer_1.RenderInternals.Log.info({ indent: false, logLevel }, `${renderer_1.RenderInternals.chalk.blueBright(`${locationLabel}`)} Duplicated ${(0, formatting_1.attrName)(duplicatedEffectDescription)}`);
+                renderer_1.RenderInternals.Log.info({ indent: false, logLevel }, `${(0, source_file_write_queue_1.getCodemodTimingPrefix)(logLevel)}${renderer_1.RenderInternals.chalk.blueBright(`${locationLabel}`)} Duplicated ${(0, formatting_1.attrName)(duplicatedEffectDescription)}`);
                 if (!update.formatted) {
                     (0, log_update_1.warnAboutPrettierOnce)(logLevel);
                 }

package/dist/preview-server/routes/duplicate-jsx-node.js

@@ -56,7 +56,7 @@ const duplicateJsxNodeHandler = ({ input: { fileName, nodePath }, remotionRoot,
                 absolutePath,
                 line: logLine,
             });
-            renderer_1.RenderInternals.Log.info({ indent: false, logLevel }, `${renderer_1.RenderInternals.chalk.blueBright(`${locationLabel}`)} Duplicated ${nodeLabel}`);
+            renderer_1.RenderInternals.Log.info({ indent: false, logLevel }, `${(0, source_file_write_queue_1.getCodemodTimingPrefix)(logLevel)}${renderer_1.RenderInternals.chalk.blueBright(`${locationLabel}`)} Duplicated ${nodeLabel}`);
             if (!formatted) {
                 (0, log_update_1.warnAboutPrettierOnce)(logLevel);
             }