remotion 4.0.536 → 4.0.537

package/dist/cjs/with-interactivity-schema.js

@@ -171,8 +171,6 @@ const withInteractivitySchema = ({ Component, componentName, componentIdentity =
         // eslint-disable-next-line react-hooks/rules-of-hooks
         const nodePathMapping = (0, react_1.useContext)(sequence_node_path_js_1.OverrideIdsToNodePathsGettersContext);
         // eslint-disable-next-line react-hooks/rules-of-hooks
-        const frame = (0, use_current_frame_js_1.useCurrentFrame)();
-        // eslint-disable-next-line react-hooks/rules-of-hooks
         const videoConfig = (0, use_unsafe_video_config_js_1.useUnsafeVideoConfig)();
         const durationInFrames = videoConfig === null || videoConfig === void 0 ? void 0 : videoConfig.durationInFrames;
         const fps = videoConfig === null || videoConfig === void 0 ? void 0 : videoConfig.fps;
@@ -229,6 +227,22 @@ const withInteractivitySchema = ({ Component, componentName, componentIdentity =
             : ((_a = nodePathMapping.overrideIdToNodePathMappings[overrideId]) !== null && _a !== void 0 ? _a : null);
         // eslint-disable-next-line react-hooks/rules-of-hooks
         const dragOverrides = (0, SequenceManager_js_1.useDragOverridesForNodePath)(nodePath);
+        // eslint-disable-next-line react-hooks/rules-of-hooks
+        const propStatus = (0, react_1.useMemo)(() => nodePath === null
+            ? undefined
+            : (0, use_memoized_effects_js_1.getPropStatusesCtx)(propStatuses, {
+                ...nodePath,
+                videoConfigValues,
+            }), [nodePath, propStatuses, videoConfigValues]);
+        // Runtime props already update through their owner. Only overrides with
+        // keyframes need this wrapper to subscribe to the advancing clock.
+        const needsFrame = flatKeys.some((key) => {
+            var _a, _b;
+            return ((_a = propStatus === null || propStatus === void 0 ? void 0 : propStatus[key]) === null || _a === void 0 ? void 0 : _a.status) === 'keyframed' ||
+                ((_b = dragOverrides[key]) === null || _b === void 0 ? void 0 : _b.type) === 'keyframed';
+        });
+        // eslint-disable-next-line react-hooks/rules-of-hooks
+        const frame = (0, use_current_frame_js_1.useCurrentFrameWithOptionalSubscription)(needsFrame);
         // Read the runtime values for every flat key from the JSX props,
         // memoized on the leaf values so the object reference is stable
         // when nothing changed — otherwise downstream `useMemo`s churn and
@@ -284,22 +298,10 @@ const withInteractivitySchema = ({ Component, componentName, componentIdentity =
                 schema: schemaWithSequenceName,
                 currentValue: currentRuntimeValueDotNotation,
                 overrideValues: dragOverrides,
-                propStatus: nodePath === null
-                    ? undefined
-                    : (0, use_memoized_effects_js_1.getPropStatusesCtx)(propStatuses, {
-                        ...nodePath,
-                        videoConfigValues,
-                    }),
+                propStatus,
                 frame,
             });
-        }, [
-            currentRuntimeValueDotNotation,
-            dragOverrides,
-            nodePath,
-            propStatuses,
-            videoConfigValues,
-            frame,
-        ]);
+        }, [currentRuntimeValueDotNotation, dragOverrides, propStatus, frame]);
         // 4. Eliminate values forbidden by the resolved discriminated union.
         const activeKeys = (0, exports.selectActiveKeys)(schemaWithSequenceName, valuesDotNotation);
         // 5. Apply the active values back onto the props.