@hyperframes/studio 0.8.35 → 0.8.36

package/src/hooks/useExternalFileChangeCoordinator.test.tsx

@@ -322,4 +322,58 @@ describe("external file change coordinator", () => {
     expect(reloadPreview).toHaveBeenCalledTimes(2);
     expect(onAcceptedPersistedFileChange).toHaveBeenCalledTimes(2);
   });
+
+  // `hyperframes preview` serves file-change over SSE, where the delivery is a
+  // MessageEvent whose `data` is a JSON STRING. Driven through the test adapter
+  // because vitest defines `import.meta.hot`, so the EventSource rung is
+  // unreachable here, which is exactly why decoding is shared by all rungs.
+  describe("SSE-shaped deliveries", () => {
+    const sseDelivery = (payload: unknown) =>
+      new MessageEvent("file-change", { data: JSON.stringify(payload) });
+
+    it("suppresses every reload for Studio's own write", async () => {
+      const drainPendingChanges = vi.fn(async () => ({ status: "clean" as const }));
+      const reloadPreview = vi.fn();
+      const onAcceptedPersistedFileChange = vi.fn();
+      await mountCoordinator({ drainPendingChanges, reloadPreview, onAcceptedPersistedFileChange });
+      markStudioWriteToken("studio-write-1");
+
+      await act(async () =>
+        handler?.(sseDelivery({ path: "index.html", version: "v2", writeToken: "studio-write-1" })),
+      );
+
+      expect(drainPendingChanges).not.toHaveBeenCalled();
+      expect(reloadPreview).not.toHaveBeenCalled();
+      expect(onAcceptedPersistedFileChange).toHaveBeenCalledWith("index.html");
+    });
+
+    it("reloads once when one watcher event reaches two subscribers", async () => {
+      const reloadPreview = vi.fn();
+      await mountCoordinator({ reloadPreview });
+
+      const external = { path: "index.html", version: "v2" };
+      await act(async () => handler?.(sseDelivery(external)));
+      await act(async () => handler?.(sseDelivery(external)));
+
+      expect(reloadPreview).toHaveBeenCalledOnce();
+    });
+
+    it("still reloads for a genuinely external write", async () => {
+      const reloadPreview = vi.fn();
+      await mountCoordinator({ reloadPreview });
+
+      await act(async () => handler?.(sseDelivery({ path: "index.html", version: "v9" })));
+
+      expect(reloadPreview).toHaveBeenCalledOnce();
+    });
+
+    it("drops an unparseable delivery instead of throwing", async () => {
+      const reloadPreview = vi.fn();
+      await mountCoordinator({ reloadPreview });
+
+      await act(async () => handler?.(new MessageEvent("file-change", { data: "not json" })));
+
+      expect(reloadPreview).not.toHaveBeenCalled();
+    });
+  });
 });

package/src/hooks/useExternalFileChangeCoordinator.ts

@@ -1,5 +1,5 @@
 import { useCallback, useEffect, useRef, useState, type MutableRefObject } from "react";
-import { readStudioFileChangePath } from "../components/editor/manualEdits";
+import { readFileChangeField, readStudioFileChangePath } from "../components/editor/manualEdits";
 import { StudioFileConflictError } from "../utils/studioSaveDiagnostics";
 import type { ExternalConflictSnapshot } from "../utils/externalConflictStorage";
 import { isSelfWriteEcho } from "./sdkSelfWriteRegistry";
@@ -81,25 +81,39 @@ function testHotAdapter(): HotTestAdapter | null {
     : null;
 }
 
-function readFileChangeContent(payload: unknown): string | null {
-  if (!payload || typeof payload !== "object") return null;
-  const record = payload as Record<string, unknown>;
-  if (typeof record.content === "string") return record.content;
-  return "data" in record ? readFileChangeContent(record.data) : null;
-}
+const readFileChangeContent = (payload: unknown) => readFileChangeField(payload, "content");
+const readFileChangeVersion = (payload: unknown) => readFileChangeField(payload, "version");
+const readFileChangeWriteToken = (payload: unknown) => readFileChangeField(payload, "writeToken");
 
-function readFileChangeVersion(payload: unknown): string | null {
-  if (!payload || typeof payload !== "object") return null;
-  const record = payload as Record<string, unknown>;
-  if (typeof record.version === "string") return record.version;
-  return "data" in record ? readFileChangeVersion(record.data) : null;
+/**
+ * Decode one delivery into the payload shape every reader above assumes.
+ *
+ * SSE delivers a `MessageEvent` whose `data` is a JSON string; the hot-reload
+ * transports deliver the payload object. This is the only place that difference
+ * exists: a reader handed an undecoded envelope reports every field as absent,
+ * which is indistinguishable from a field that is genuinely unset.
+ */
+function decodeFileChange(delivery: unknown): unknown {
+  // Structural, not `instanceof`: a polyfilled or cross-realm event must still decode.
+  const data = (delivery as { data?: unknown } | null)?.data;
+  if (typeof data !== "string") return delivery;
+  try {
+    return JSON.parse(data);
+  } catch {
+    logReload("file-change", { path: null, why: "unparseable payload" });
+    return null;
+  }
 }
 
-function readFileChangeWriteToken(payload: unknown): string | null {
-  if (!payload || typeof payload !== "object") return null;
-  const record = payload as Record<string, unknown>;
-  if (typeof record.writeToken === "string") return record.writeToken;
-  return "data" in record ? readFileChangeWriteToken(record.data) : null;
+/**
+ * The production transport. vitest defines `import.meta.hot`, so the selection
+ * below never reaches this rung under test; the rung itself is exported so a test
+ * can drive a real `MessageEvent` through the listener it registers.
+ */
+export function sseFileChangeChannel(onDelivery: (delivery: unknown) => void): () => void {
+  const eventSource = new EventSource("/api/events");
+  eventSource.addEventListener("file-change", onDelivery);
+  return () => eventSource.close();
 }
 
 function eventIdentity(path: string, payload: unknown): string | null {
@@ -331,7 +345,10 @@ export function useExternalFileChangeCoordinator({
     // fallow-ignore-next-line complexity
     (payload: unknown) => {
       const path = readStudioFileChangePath(payload);
-      if (!path || !projectId) return;
+      if (!path || !projectId) {
+        logReload("file-change", { path: null, why: path ? "no project" : "no path in payload" });
+        return;
+      }
       pendingTimelineEditPathRef.current.delete(path);
 
       const content = readFileChangeContent(payload);
@@ -362,7 +379,8 @@ export function useExternalFileChangeCoordinator({
   );
 
   useEffect(() => {
-    const handler = (payload?: unknown) => processChange(payload);
+    // One decoder for all three transports; the rungs only choose the channel.
+    const handler = (delivery?: unknown) => processChange(decodeFileChange(delivery));
     const adapter = testHotAdapter();
     if (adapter) {
       adapter.on("hf:file-change", handler);
@@ -372,9 +390,7 @@ export function useExternalFileChangeCoordinator({
       import.meta.hot.on("hf:file-change", handler);
       return () => import.meta.hot?.off?.("hf:file-change", handler);
     }
-    const eventSource = new EventSource("/api/events");
-    eventSource.addEventListener("file-change", handler);
-    return () => eventSource.close();
+    return sseFileChangeChannel(handler);
   }, [processChange]);
 
   const retry = useCallback(async () => {

package/src/hooks/useFileManager.projectOwnership.test.tsx

@@ -37,7 +37,6 @@ function useTestFileManager(projectId: string) {
     projectId,
     showToast: vi.fn(),
     recordEdit: vi.fn(async () => {}),
-    domEditSaveTimestampRef: { current: 0 },
     setRefreshKey: vi.fn(),
   });
 }

package/src/hooks/useFileManager.ts

@@ -27,7 +27,6 @@ interface UseFileManagerOptions {
   projectId: string | null;
   showToast: (message: string, tone?: "error" | "info") => void;
   recordEdit: (input: RecordEditInput) => Promise<void>;
-  domEditSaveTimestampRef: React.MutableRefObject<number>;
   setRefreshKey: React.Dispatch<React.SetStateAction<number>>;
 }
 
@@ -37,7 +36,6 @@ export function useFileManager({
   projectId,
   showToast,
   recordEdit,
-  domEditSaveTimestampRef,
   setRefreshKey,
 }: UseFileManagerOptions) {
   // ── Shared refs ──
@@ -198,7 +196,6 @@ export function useFileManager({
     readProjectFile,
     writeProjectFile,
     recordEdit,
-    domEditSaveTimestampRef,
     setRefreshKey,
     showToast,
   });

package/src/hooks/useGroupCommits.ts

@@ -64,7 +64,6 @@ async function commitStructuralMutation(
     UseGroupCommitsParams,
     | "writeProjectFile"
     | "editHistory"
-    | "domEditSaveTimestampRef"
     | "clearDomSelection"
     | "forceReloadSdkSession"
     | "reloadPreview"
@@ -72,7 +71,6 @@ async function commitStructuralMutation(
 ): Promise<{ content?: string; groupId?: string }> {
   const originalContent = await readProjectFileContent(pid, targetPath);
 
-  deps.domEditSaveTimestampRef.current = Date.now();
   const mutateResponse = await fetch(
     buildProjectApiPath(pid, `/file-mutations/${route}/${encodeURIComponent(targetPath)}`),
     {

package/src/hooks/useGsapScriptCommits.test.tsx

@@ -379,7 +379,6 @@ function renderCommitHook(
       activeCompPath: "index.html",
       previewIframeRef: { current: FAKE_IFRAME },
       editHistory: { recordEdit },
-      domEditSaveTimestampRef: { current: 0 },
       reloadPreview,
       onCacheInvalidate,
       onFileContentChanged,
@@ -669,7 +668,6 @@ describe("runCommit — instantPatch wiring", () => {
       writeProjectFile,
       readProjectFile: vi.fn(async () => "BEFORE"),
       reloadPreview: deps.reloadPreview,
-      domEditSaveTimestampRef: { current: 0 },
     });
     await sdkWriteStarted;
 

package/src/hooks/useGsapScriptCommits.ts

@@ -313,7 +313,7 @@ function instantPatchesFor(
 
 // oxfmt-ignore
 // fallow-ignore-next-line complexity
-export function useGsapScriptCommits({ projectIdRef, activeCompPath, previewIframeRef, editHistory, domEditSaveTimestampRef, reloadPreview, onCacheInvalidate, onFileContentChanged, showToast, sdkSession, publishSdkSession, writeProjectFile, forceReloadSdkSession }: GsapScriptCommitsParams) {
+export function useGsapScriptCommits({ projectIdRef, activeCompPath, previewIframeRef, editHistory, reloadPreview, onCacheInvalidate, onFileContentChanged, showToast, sdkSession, publishSdkSession, writeProjectFile, forceReloadSdkSession }: GsapScriptCommitsParams) {
   const activeProjectId = projectIdRef.current;
   const activeCompPathRef = useRef(activeCompPath);
   activeCompPathRef.current = activeCompPath;
@@ -348,7 +348,6 @@ export function useGsapScriptCommits({ projectIdRef, activeCompPath, previewIfra
       }
       return;
     }
-    if (previewIsActive) domEditSaveTimestampRef.current = Date.now();
     await recordMutationEdit(targetPath, result, options);
     // The durable mutation belongs to the project captured when it was queued.
     // A later project must never receive its file state or preview refresh.
@@ -365,7 +364,7 @@ export function useGsapScriptCommits({ projectIdRef, activeCompPath, previewIfra
       reloadPreview,
       onCacheInvalidate,
     });
-  }, [projectIdRef, previewIframeRef, domEditSaveTimestampRef, reloadPreview, onCacheInvalidate, onFileContentChanged, forceReloadSdkSession, recordMutationEdit]);
+  }, [projectIdRef, previewIframeRef, reloadPreview, onCacheInvalidate, onFileContentChanged, forceReloadSdkSession, recordMutationEdit]);
 
   const runCommit = useCallback(async (pid: string, compositionPath: string | null, targetPath: string, selection: DomEditSelection, mutation: Record<string, unknown>, options: CommitMutationOptions) => {
     const result = await runMutationRequest([mutation], options, showToast, () =>
@@ -494,7 +493,6 @@ export function useGsapScriptCommits({ projectIdRef, activeCompPath, previewIfra
             editHistory: { recordEdit: editHistory.recordEdit },
             writeProjectFile,
             reloadPreview,
-            domEditSaveTimestampRef,
             refresh: sdkRefresh,
             compositionPath: activeCompPath,
             readProjectFile: readProjectFileContent,
@@ -505,7 +503,6 @@ export function useGsapScriptCommits({ projectIdRef, activeCompPath, previewIfra
       editHistory.recordEdit,
       writeProjectFile,
       reloadPreview,
-      domEditSaveTimestampRef,
       sdkRefresh,
       activeCompPath,
       readProjectFileContent,

package/src/hooks/usePreviewInteraction.test.ts

@@ -224,12 +224,17 @@ describe("usePreviewInteraction", () => {
     cleanup();
   });
 
-  it("resumes playback when a click resolves to nothing (dead-zone / deselect)", async () => {
-    usePlayerStore.setState({ isPlaying: true });
+  // The deselect resume asks the PLAYER to play; it must never move the store's
+  // flag on its own. The runtime really was paused (playerPause below), so a bare
+  // setIsPlaying(true) leaves the button reading "playing" over a stopped runtime
+  // with nothing able to reconcile the two.
+  it("requests a real resume when a click resolves to nothing (dead-zone / deselect)", async () => {
+    usePlayerStore.setState({ isPlaying: true, playbackRequest: null });
+    const playerPause = vi.fn();
     const applyDomSelection = vi.fn();
     const resolveDomSelectionFromPreviewPoint = vi.fn(async () => null);
     const { canvas, cleanup } = renderHarness({
-      previewIframe: createPreviewIframe(vi.fn()),
+      previewIframe: createPreviewIframe(playerPause),
       resolveDomSelectionFromPreviewPoint,
       applyDomSelection,
     });
@@ -237,12 +242,15 @@ describe("usePreviewInteraction", () => {
     await dispatchMouseDown(canvas, {});
 
     expect(applyDomSelection).toHaveBeenCalledWith(null, { revealPanel: false });
-    expect(usePlayerStore.getState().isPlaying).toBe(true);
+    expect(playerPause).toHaveBeenCalled();
+    expect(usePlayerStore.getState().playbackRequest).toMatchObject({ playing: true });
+    // Only the player's own play() may raise this, once the runtime is running.
+    expect(usePlayerStore.getState().isPlaying).toBe(false);
     cleanup();
   });
 
   it("does not resume playback on deselect when it was already paused", async () => {
-    usePlayerStore.setState({ isPlaying: false });
+    usePlayerStore.setState({ isPlaying: false, playbackRequest: null });
     const applyDomSelection = vi.fn();
     const resolveDomSelectionFromPreviewPoint = vi.fn(async () => null);
     const { canvas, cleanup } = renderHarness({
@@ -255,6 +263,7 @@ describe("usePreviewInteraction", () => {
 
     expect(applyDomSelection).toHaveBeenCalledWith(null, { revealPanel: false });
     expect(usePlayerStore.getState().isPlaying).toBe(false);
+    expect(usePlayerStore.getState().playbackRequest).toBeNull();
     cleanup();
   });
 });

package/src/hooks/usePreviewInteraction.ts

@@ -101,10 +101,10 @@ export function usePreviewInteraction({
       const wasPlaying = usePlayerStore.getState().isPlaying;
       pausePreviewPlayback();
       // A click that resolves to nothing (dead-zone / deselect) shouldn't leave
-      // playback paused — pausing before sampling only exists to keep the hit
-      // target stable while resolving; resume if nothing was selected.
+      // playback paused; the pause only keeps the hit target stable while resolving.
+      // Resume through requestPlayback so adapter, rAF loop and flag move together.
       const resumeIfNothingSelected = () => {
-        if (wasPlaying) usePlayerStore.getState().setIsPlaying(true);
+        if (wasPlaying) usePlayerStore.getState().requestPlayback(true);
       };
 
       // Double-click a group → drill into it and select the child under the

package/src/hooks/useProjectCompositionVariables.ts

@@ -1,4 +1,4 @@
-import { useCallback, useEffect, useState, type MutableRefObject } from "react";
+import { useCallback, useEffect, useState } from "react";
 import { openComposition, type Composition, type CompositionVariable } from "@hyperframes/sdk";
 import { persistSdkSerialize } from "../utils/sdkCutover";
 import type { EditHistoryKind } from "../utils/editHistory";
@@ -87,7 +87,6 @@ interface EditVariablesDeps {
   writeProjectFile: (path: string, content: string) => Promise<void>;
   recordEdit: RecordEditFn;
   reloadPreview: () => void;
-  domEditSaveTimestampRef: MutableRefObject<number>;
 }
 
 /**
@@ -98,8 +97,7 @@ interface EditVariablesDeps {
  * but keyed on `path` rather than a live session.
  */
 export function useEditVariablesInFile(deps: EditVariablesDeps) {
-  const { readProjectFile, writeProjectFile, recordEdit, reloadPreview, domEditSaveTimestampRef } =
-    deps;
+  const { readProjectFile, writeProjectFile, recordEdit, reloadPreview } = deps;
   return useCallback(
     async (path: string, label: string, mutate: (session: Composition) => void): Promise<void> => {
       const originalContent = await readProjectFile(path);
@@ -119,13 +117,12 @@ export function useEditVariablesInFile(deps: EditVariablesDeps) {
           editHistory: { recordEdit },
           writeProjectFile,
           reloadPreview,
-          domEditSaveTimestampRef,
           compositionPath: path,
           readProjectFile,
         },
         { label },
       );
     },
-    [readProjectFile, writeProjectFile, recordEdit, reloadPreview, domEditSaveTimestampRef],
+    [readProjectFile, writeProjectFile, recordEdit, reloadPreview],
   );
 }

package/src/hooks/useRazorSplit.history.test.tsx

@@ -117,7 +117,6 @@ function mountRazorSplit(opts: { gsap?: boolean; previewStamp?: boolean } = {}):
         disk[path] = content;
       },
       recordEdit: store.recordEdit,
-      domEditSaveTimestampRef: { current: 0 },
       reloadPreview: () => {
         if (!opts.previewStamp) return;
         const stamped = ensureHfIds(disk["index.html"]);
@@ -258,7 +257,6 @@ function mountRazorSplitAll(failOnSplit?: number): SplitAllHarness {
         disk[path] = content;
       },
       recordEdit: store.recordEdit,
-      domEditSaveTimestampRef: { current: 0 },
       reloadPreview: () => {},
     });
     splitAllRef.current = handleRazorSplitAll;

package/src/hooks/useRazorSplit.test.ts

@@ -88,7 +88,6 @@ function mountRazorSplit(): Harness {
         disk[path] = content;
       },
       recordEdit: async () => {},
-      domEditSaveTimestampRef: { current: 0 },
       reloadPreview: () => {},
     });
     singleRef.current = handleRazorSplit;
@@ -211,7 +210,6 @@ function mountRazorSplitWithHistory(): UndoHarness {
         disk[path] = content;
       },
       recordEdit: (input) => store.recordEdit(input),
-      domEditSaveTimestampRef: { current: 0 },
       reloadPreview: () => {},
       forceReloadSdkSession,
     });

package/src/hooks/useRazorSplit.test.tsx

@@ -66,7 +66,6 @@ describe("useRazorSplit mutation versions", () => {
         writeProjectFile,
         observeProjectFileVersion,
         recordEdit,
-        domEditSaveTimestampRef: { current: 0 },
         reloadPreview: vi.fn(),
       }).handleRazorSplit;
       return null;

package/src/hooks/useRazorSplit.ts

@@ -15,7 +15,6 @@ interface UseRazorSplitOptions {
   writeProjectFile: (path: string, content: string, expectedContent?: string) => Promise<void>;
   observeProjectFileVersion?: (path: string, version: string | null) => void;
   recordEdit: (input: RecordEditInput) => Promise<void>;
-  domEditSaveTimestampRef: React.MutableRefObject<number>;
   reloadPreview: () => void;
   forceReloadSdkSession?: () => void;
   isRecordingRef?: React.RefObject<boolean>;
@@ -28,7 +27,6 @@ export function useRazorSplit({
   writeProjectFile,
   observeProjectFileVersion,
   recordEdit,
-  domEditSaveTimestampRef,
   reloadPreview,
   forceReloadSdkSession,
   isRecordingRef,
@@ -62,9 +60,6 @@ export function useRazorSplit({
           ? "Split timeline clip"
           : `Split ${requestedCount} clips at ${splitTime.toFixed(2)}s`;
 
-      // Server writes arrive through the watcher before React can refresh. Keep
-      // the existing short self-write window active for this owned transaction.
-      domEditSaveTimestampRef.current = Date.now();
       const result = await runAtomicCutTransaction({
         projectId: pid,
         intents,
@@ -91,7 +86,6 @@ export function useRazorSplit({
     },
     [
       activeCompPath,
-      domEditSaveTimestampRef,
       observeProjectFileVersion,
       recordEdit,
       showToast,

package/src/hooks/useSdkSession.ts

@@ -1,7 +1,6 @@
 import { useState, useEffect, useCallback, useRef } from "react";
 import { openComposition } from "@hyperframes/sdk";
 import type { Composition } from "@hyperframes/sdk";
-import { readStudioFileChangePath } from "../components/editor/manualEdits";
 import { isSelfWriteEcho } from "./sdkSelfWriteRegistry";
 import { trackStudioEvent } from "../utils/studioTelemetry";
 import type { PublishSdkSession } from "../utils/sdkCutover";
@@ -29,15 +28,6 @@ async function readProjectFileOptional(
   return typeof data.content === "string" ? data.content : undefined;
 }
 
-/**
- * True when an external file-change payload targets the active composition and
- * the SDK session must be re-opened to pick up the new content.
- */
-export function shouldReloadSdkSession(payload: unknown, activeCompPath: string | null): boolean {
-  if (!activeCompPath) return false;
-  return readStudioFileChangePath(payload) === activeCompPath;
-}
-
 /**
  * Stage 7 Step 3a — SDK session wired to the active composition.
  *
@@ -73,7 +63,9 @@ export interface SdkSessionHandle {
   /**
    * Force a session reload immediately, bypassing the self-write suppress
    * window. Call after undo/redo writes the active composition file so the
-   * SDK in-memory document reflects the reverted content.
+   * SDK in-memory document reflects the reverted content. Without it the
+   * window swallows the file-change and the session stays stale; the write
+   * side of that path is covered by usePersistentEditHistory.test.ts.
    */
   forceReload: () => void;
 }

package/src/hooks/useSlideshowPersist.ts

@@ -1,4 +1,4 @@
-import { useCallback, type MutableRefObject } from "react";
+import { useCallback } from "react";
 import type { Composition } from "@hyperframes/sdk";
 import type { SlideshowManifest } from "@hyperframes/core/slideshow";
 import type { EditHistoryKind } from "../utils/editHistory";
@@ -16,7 +16,6 @@ export interface UseSlideshowPersistParams {
     files: Record<string, { before: string; after: string }>;
   }) => Promise<void>;
   reloadPreview: () => void;
-  domEditSaveTimestampRef: MutableRefObject<number>;
   /** Publish a fully persisted candidate SDK session. */
   publishSdkSession?: PublishSdkSession;
   /**
@@ -35,7 +34,6 @@ export function useSlideshowPersist({
   writeProjectFile,
   recordEdit,
   reloadPreview,
-  domEditSaveTimestampRef,
   publishSdkSession,
   coalesceKey,
 }: UseSlideshowPersistParams): (manifest: SlideshowManifest) => Promise<void> {
@@ -52,7 +50,6 @@ export function useSlideshowPersist({
           editHistory: { recordEdit },
           writeProjectFile,
           reloadPreview,
-          domEditSaveTimestampRef,
           readProjectFile,
           publishSession: publishSdkSession,
         },
@@ -66,7 +63,6 @@ export function useSlideshowPersist({
       writeProjectFile,
       recordEdit,
       reloadPreview,
-      domEditSaveTimestampRef,
       publishSdkSession,
       coalesceKey,
     ],

package/src/hooks/useTimelineAssetDropOps.ts

@@ -29,7 +29,6 @@ interface UseTimelineAssetDropOpsOptions {
   showToast: (message: string, tone?: "error" | "info") => void;
   writeProjectFile: (path: string, content: string, expectedContent?: string) => Promise<void>;
   recordEdit: (input: RecordEditInput) => Promise<void>;
-  domEditSaveTimestampRef: MutableRefObject<number>;
   reloadPreview: () => void;
   uploadProjectFiles: (files: Iterable<File>, dir?: string) => Promise<string[]>;
   isRecordingRef?: RefObject<boolean>;
@@ -44,7 +43,6 @@ export function useTimelineAssetDropOps({
   showToast,
   writeProjectFile,
   recordEdit,
-  domEditSaveTimestampRef,
   reloadPreview,
   uploadProjectFiles,
   isRecordingRef,
@@ -109,7 +107,6 @@ export function useTimelineAssetDropOps({
           }),
         );
 
-        domEditSaveTimestampRef.current = Date.now();
         await saveProjectFilesWithHistory({
           projectId: pid,
           label: "Add timeline asset",
@@ -135,7 +132,6 @@ export function useTimelineAssetDropOps({
       showToast,
       timelineElements,
       writeProjectFile,
-      domEditSaveTimestampRef,
       reloadPreview,
       isRecordingRef,
       forceReloadSdkSession,
@@ -198,7 +194,6 @@ export function useTimelineAssetDropOps({
           resync: forceReloadSdkSession,
           refresh: reloadPreview,
         });
-        domEditSaveTimestampRef.current = Date.now();
         showToast("Composition added to the timeline.", "info");
       } catch (error) {
         showToast(
@@ -209,7 +204,6 @@ export function useTimelineAssetDropOps({
     },
     [
       activeCompPath,
-      domEditSaveTimestampRef,
       forceReloadSdkSession,
       isRecordingRef,
       observeProjectFileVersion,

package/src/hooks/useTimelineDeleteOps.ts

@@ -20,7 +20,6 @@ interface UseTimelineDeleteOpsOptions {
   showToast: (message: string, tone?: "error" | "info") => void;
   writeProjectFile: (path: string, content: string, expectedContent?: string) => Promise<void>;
   recordEdit: (input: RecordEditInput) => Promise<void>;
-  domEditSaveTimestampRef: MutableRefObject<number>;
   reloadPreview: () => void;
   isRecordingRef?: MutableRefObject<boolean>;
   forceReloadSdkSession?: () => void;
@@ -34,7 +33,6 @@ export function useTimelineDeleteOps({
   showToast,
   writeProjectFile,
   recordEdit,
-  domEditSaveTimestampRef,
   reloadPreview,
   isRecordingRef,
   forceReloadSdkSession,
@@ -111,7 +109,6 @@ export function useTimelineDeleteOps({
           usePlayerStore.getState().setDuration(deleteContentEnd);
         }
 
-        domEditSaveTimestampRef.current = Date.now();
         try {
           await saveProjectFilesWithHistory({
             projectId: pid,
@@ -154,7 +151,6 @@ export function useTimelineDeleteOps({
       showToast,
       timelineElements,
       writeProjectFile,
-      domEditSaveTimestampRef,
       reloadPreview,
       isRecordingRef,
       forceReloadSdkSession,

package/src/hooks/useTimelineEditing.test.tsx

@@ -147,7 +147,6 @@ function renderTimelineEditingHook(input: {
       showToast: input.showToast ?? vi.fn(),
       writeProjectFile: input.writeProjectFile ?? vi.fn(),
       recordEdit: input.recordEdit ?? vi.fn(),
-      domEditSaveTimestampRef: { current: 0 },
       reloadPreview: input.reloadPreview ?? vi.fn(),
       previewIframeRef: { current: input.iframe },
       pendingTimelineEditPathRef: { current: new Set<string>() },
@@ -201,7 +200,6 @@ function renderTimelineEditingHookWithLifecycle(input: {
       activeCompPath: "index.html",
       showToast: vi.fn(),
       writeProjectFile: vi.fn(),
-      domEditSaveTimestampRef: { current: 0 },
       editHistory: { recordEdit: vi.fn() },
       projectIdRef: { current: "p1" },
       reloadPreview: vi.fn(),
@@ -217,7 +215,6 @@ function renderTimelineEditingHookWithLifecycle(input: {
       showToast: vi.fn(),
       writeProjectFile: vi.fn(),
       recordEdit: vi.fn(),
-      domEditSaveTimestampRef: { current: 0 },
       reloadPreview: vi.fn(),
       previewIframeRef: { current: input.iframe },
       pendingTimelineEditPathRef: { current: new Set<string>() },