@remotion/studio-server 4.0.442 → 4.0.443

package/dist/preview-server/node-path-cache.d.ts

@@ -0,0 +1,4 @@
+import type { SequenceNodePath } from '@remotion/studio-shared';
+export declare const getCachedNodePath: (fileName: string, line: number, column: number) => SequenceNodePath | undefined;
+export declare const setCachedNodePath: (fileName: string, line: number, column: number, nodePath: SequenceNodePath) => void;
+export declare const clearNodePathCache: () => void;

package/dist/preview-server/node-path-cache.js

@@ -0,0 +1,19 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.clearNodePathCache = exports.setCachedNodePath = exports.getCachedNodePath = void 0;
+const makeCacheKey = (fileName, line, column) => {
+    return `${fileName}:${line}:${column}`;
+};
+const cache = new Map();
+const getCachedNodePath = (fileName, line, column) => {
+    return cache.get(makeCacheKey(fileName, line, column));
+};
+exports.getCachedNodePath = getCachedNodePath;
+const setCachedNodePath = (fileName, line, column, nodePath) => {
+    cache.set(makeCacheKey(fileName, line, column), nodePath);
+};
+exports.setCachedNodePath = setCachedNodePath;
+const clearNodePathCache = () => {
+    cache.clear();
+};
+exports.clearNodePathCache = clearNodePathCache;

package/dist/preview-server/routes/subscribe-to-sequence-props.js

@@ -2,10 +2,11 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.subscribeToSequenceProps = void 0;
 const sequence_props_watchers_1 = require("../sequence-props-watchers");
-const subscribeToSequenceProps = ({ input: { fileName, line, keys, clientId }, remotionRoot }) => {
+const subscribeToSequenceProps = ({ input: { fileName, line, column, keys, clientId }, remotionRoot }) => {
     const result = (0, sequence_props_watchers_1.subscribeToSequencePropsWatchers)({
         fileName,
         line,
+        column,
         keys,
         remotionRoot,
         clientId,

package/dist/preview-server/sequence-props-watchers.d.ts

@@ -1,7 +1,8 @@
 import type { CanUpdateSequencePropsResponse, SequenceNodePath } from '@remotion/studio-shared';
-export declare const subscribeToSequencePropsWatchers: ({ fileName, line, keys, remotionRoot, clientId, }: {
+export declare const subscribeToSequencePropsWatchers: ({ fileName, line, column, keys, remotionRoot, clientId, }: {
     fileName: string;
     line: number;
+    column: number;
     keys: string[];
     remotionRoot: string;
     clientId: string;

package/dist/preview-server/sequence-props-watchers.js

@@ -7,29 +7,57 @@ exports.unsubscribeClientSequencePropsWatchers = exports.unsubscribeFromSequence
 const node_path_1 = __importDefault(require("node:path"));
 const file_watcher_1 = require("../file-watcher");
 const live_events_1 = require("./live-events");
+const node_path_cache_1 = require("./node-path-cache");
 const can_update_sequence_props_1 = require("./routes/can-update-sequence-props");
 const sequencePropsWatchers = {};
 const makeWatcherKey = ({ absolutePath, nodePath, }) => {
     return `${absolutePath}:${JSON.stringify(nodePath)}`;
 };
-const subscribeToSequencePropsWatchers = ({ fileName, line, keys, remotionRoot, clientId, }) => {
+const subscribeToSequencePropsWatchers = ({ fileName, line, column, keys, remotionRoot, clientId, }) => {
     var _a;
     const absolutePath = node_path_1.default.resolve(remotionRoot, fileName);
-    // Initial lookup by line+column to resolve the nodePath
-    const initialResult = (0, can_update_sequence_props_1.computeSequencePropsStatusByLine)({
-        fileName,
-        line,
-        keys,
-        remotionRoot,
-    });
+    // Try cached nodePath first (handles stale source maps after suppressed rebuilds)
+    const cachedNodePath = (0, node_path_cache_1.getCachedNodePath)(fileName, line, column);
+    let initialResult;
+    if (cachedNodePath) {
+        const cachedResult = (0, can_update_sequence_props_1.computeSequencePropsStatus)({
+            fileName,
+            nodePath: cachedNodePath,
+            keys,
+            remotionRoot,
+        });
+        if (cachedResult.canUpdate) {
+            initialResult = cachedResult;
+        }
+        else {
+            // Cached nodePath no longer valid, fall back to line-based lookup
+            initialResult = (0, can_update_sequence_props_1.computeSequencePropsStatusByLine)({
+                fileName,
+                line,
+                keys,
+                remotionRoot,
+            });
+        }
+    }
+    else {
+        initialResult = (0, can_update_sequence_props_1.computeSequencePropsStatusByLine)({
+            fileName,
+            line,
+            keys,
+            remotionRoot,
+        });
+    }
     if (!initialResult.canUpdate) {
         return initialResult;
     }
+    // Cache the resolved nodePath for future lookups with stale source maps
+    (0, node_path_cache_1.setCachedNodePath)(fileName, line, column, initialResult.nodePath);
     const { nodePath } = initialResult;
     const watcherKey = makeWatcherKey({ absolutePath, nodePath });
-    // Unwatch any existing watcher for the same key
+    // If a watcher already exists for this key, just bump the ref count
     if ((_a = sequencePropsWatchers[clientId]) === null || _a === void 0 ? void 0 : _a[watcherKey]) {
-        sequencePropsWatchers[clientId][watcherKey].unwatch();
+        sequencePropsWatchers[clientId][watcherKey].refCount++;
+        return initialResult;
     }
     const { unwatch } = (0, file_watcher_1.installFileWatcher)({
         file: absolutePath,
@@ -37,7 +65,13 @@ const subscribeToSequencePropsWatchers = ({ fileName, line, keys, remotionRoot,
             if (event.type === 'deleted') {
                 return;
             }
-            const result = (0, can_update_sequence_props_1.computeSequencePropsStatusFromContent)(event.content, nodePath, keys);
+            let result;
+            try {
+                result = (0, can_update_sequence_props_1.computeSequencePropsStatusFromContent)(event.content, nodePath, keys);
+            }
+            catch (_a) {
+                return;
+            }
             (0, live_events_1.waitForLiveEventsListener)().then((listener) => {
                 listener.sendEventToClientId(clientId, {
                     type: 'sequence-props-updated',
@@ -51,7 +85,7 @@ const subscribeToSequencePropsWatchers = ({ fileName, line, keys, remotionRoot,
     if (!sequencePropsWatchers[clientId]) {
         sequencePropsWatchers[clientId] = {};
     }
-    sequencePropsWatchers[clientId][watcherKey] = { unwatch };
+    sequencePropsWatchers[clientId][watcherKey] = { unwatch, refCount: 1 };
     return initialResult;
 };
 exports.subscribeToSequencePropsWatchers = subscribeToSequencePropsWatchers;
@@ -60,12 +94,13 @@ const unsubscribeFromSequencePropsWatchers = ({ fileName, nodePath, remotionRoot
     const watcherKey = makeWatcherKey({ absolutePath, nodePath });
     if (!sequencePropsWatchers[clientId] ||
         !sequencePropsWatchers[clientId][watcherKey]) {
-        // eslint-disable-next-line no-console
-        console.warn(`Unexpected: unsubscribe for sequence props watcher that does not exist (clientId=${clientId}, key=${watcherKey})`);
         return;
     }
-    sequencePropsWatchers[clientId][watcherKey].unwatch();
-    delete sequencePropsWatchers[clientId][watcherKey];
+    sequencePropsWatchers[clientId][watcherKey].refCount--;
+    if (sequencePropsWatchers[clientId][watcherKey].refCount <= 0) {
+        sequencePropsWatchers[clientId][watcherKey].unwatch();
+        delete sequencePropsWatchers[clientId][watcherKey];
+    }
 };
 exports.unsubscribeFromSequencePropsWatchers = unsubscribeFromSequencePropsWatchers;
 const unsubscribeClientSequencePropsWatchers = (clientId) => {

package/dist/preview-server/start-server.js

@@ -12,6 +12,7 @@ const routes_1 = require("../routes");
 const dev_middleware_1 = require("./dev-middleware");
 const hot_middleware_1 = require("./hot-middleware");
 const live_events_1 = require("./live-events");
+const node_path_cache_1 = require("./node-path-cache");
 const undo_stack_1 = require("./undo-stack");
 const watch_ignore_next_change_1 = require("./watch-ignore-next-change");
 const startServer = async (options) => {
@@ -67,6 +68,9 @@ const startServer = async (options) => {
         };
     });
     (0, hot_middleware_1.setupWebpackHmr)(compiler, options.logLevel, liveEventsServer);
+    compiler.hooks.done.tap('remotion-node-path-cache', () => {
+        (0, node_path_cache_1.clearNodePathCache)();
+    });
     const server = node_http_1.default.createServer((request, response) => {
         if (options.enableCrossSiteIsolation) {
             response.setHeader('Cross-Origin-Opener-Policy', 'same-origin');

package/package.json

@@ -3,7 +3,7 @@
 		"url": "https://github.com/remotion-dev/remotion/tree/main/packages/studio-server"
 	},
 	"name": "@remotion/studio-server",
-	"version": "4.0.442",
+	"version": "4.0.443",
 	"description": "Run a Remotion Studio with a server backend",
 	"main": "dist",
 	"sideEffects": false,
@@ -27,11 +27,11 @@
 		"@babel/parser": "7.24.1",
 		"semver": "7.5.3",
 		"prettier": "3.8.1",
-		"remotion": "4.0.442",
+		"remotion": "4.0.443",
 		"recast": "0.23.11",
-		"@remotion/bundler": "4.0.442",
-		"@remotion/renderer": "4.0.442",
-		"@remotion/studio-shared": "4.0.442",
+		"@remotion/bundler": "4.0.443",
+		"@remotion/renderer": "4.0.443",
+		"@remotion/studio-shared": "4.0.443",
 		"memfs": "3.4.3",
 		"source-map": "0.7.3",
 		"open": "^8.4.2"
@@ -41,7 +41,7 @@
 		"react": "19.2.3",
 		"@babel/types": "7.24.0",
 		"@types/semver": "^7.3.4",
-		"@remotion/eslint-config-internal": "4.0.442",
+		"@remotion/eslint-config-internal": "4.0.443",
 		"eslint": "9.19.0",
 		"@types/node": "20.12.14",
 		"@typescript/native-preview": "7.0.0-dev.20260217.1"