@hyperframes/studio 0.7.70 → 0.7.72

package/src/components/editor/propertyPanelFlatOverlaysSection.tsx

@@ -0,0 +1,108 @@
+import { useState } from "react";
+import type { RegistryItem } from "@hyperframes/core/registry";
+import { useBlockCatalog } from "../../hooks/useBlockCatalog";
+import { Film, Plus } from "../../icons/SystemIcons";
+import { useTrackDesignInput } from "../../contexts/DesignPanelInputContext";
+import type { DomEditSelection } from "./domEditing";
+import { FLAT_PREVIEW_GRID } from "./propertyPanelFlatPrimitives";
+import type { ElementTiming } from "./propertyPanelFlatTimingDerivation";
+
+export const MEDIA_TREATMENT_OVERLAY_TAG = "media-treatment-overlay";
+
+export function filterMediaTreatmentOverlays(items: readonly RegistryItem[]): RegistryItem[] {
+  return items.filter(
+    (item) =>
+      item.type === "hyperframes:block" &&
+      item.tags?.includes(MEDIA_TREATMENT_OVERLAY_TAG) === true,
+  );
+}
+
+export function deriveMediaOverlayPlacement(
+  element: Pick<DomEditSelection, "dataAttributes" | "sourceFile">,
+  timing: Pick<ElementTiming, "start" | "duration">,
+) {
+  const authoredTrack = Number.parseInt(element.dataAttributes["track-index"] ?? "", 10);
+  return {
+    start: timing.start,
+    ...(timing.duration > 0 ? { duration: timing.duration } : {}),
+    ...(Number.isFinite(authoredTrack) ? { track: authoredTrack + 1 } : {}),
+    compositionPath: element.sourceFile,
+  };
+}
+
+export function FlatOverlaysSection({
+  onAddOverlay,
+}: {
+  onAddOverlay: (name: string) => Promise<void>;
+}) {
+  const track = useTrackDesignInput();
+  const { blocks, loading, error } = useBlockCatalog();
+  const overlays = filterMediaTreatmentOverlays(blocks);
+  const [adding, setAdding] = useState<string | null>(null);
+  const [previewing, setPreviewing] = useState<string | null>(null);
+
+  if (loading) {
+    return <div className="py-4 text-center text-[10px] text-panel-text-4">Loading overlays…</div>;
+  }
+  if (error) {
+    return <div className="py-4 text-center text-[10px] text-red-300">{error}</div>;
+  }
+
+  const busy = adding !== null;
+  return (
+    <div data-flat-overlays="true" className={FLAT_PREVIEW_GRID}>
+      {overlays.map((overlay) => (
+        <button
+          key={overlay.name}
+          type="button"
+          data-flat-overlay={overlay.name}
+          aria-label={`Add ${overlay.title}`}
+          disabled={busy}
+          title={overlay.description}
+          onPointerEnter={() => setPreviewing(overlay.name)}
+          onPointerLeave={() => setPreviewing(null)}
+          onFocus={() => setPreviewing(overlay.name)}
+          onBlur={() => setPreviewing(null)}
+          onClick={() => {
+            track("button", `Add ${overlay.title}`);
+            setAdding(overlay.name);
+            void onAddOverlay(overlay.name).finally(() => setAdding(null));
+          }}
+          className="group min-w-0 overflow-hidden border border-panel-hairline bg-panel-bg-soft text-left transition-colors hover:border-panel-border-input hover:bg-panel-bg disabled:cursor-wait disabled:opacity-50"
+        >
+          <span className="relative flex aspect-video w-full items-center justify-center overflow-hidden bg-black/20">
+            {previewing === overlay.name && overlay.preview?.video ? (
+              <video
+                src={overlay.preview.video}
+                poster={overlay.preview.poster}
+                autoPlay
+                muted
+                loop
+                playsInline
+                className="block h-full w-full object-cover"
+              />
+            ) : overlay.preview?.poster ? (
+              <img
+                data-flat-overlay-preview={overlay.name}
+                src={overlay.preview.poster}
+                alt=""
+                draggable={false}
+                loading="lazy"
+                className="block h-full w-full object-cover"
+              />
+            ) : (
+              <Film size={15} className="text-panel-text-4" />
+            )}
+            <Plus
+              size={12}
+              className="absolute right-1.5 top-1.5 text-white opacity-70 drop-shadow group-hover:text-panel-accent group-hover:opacity-100"
+            />
+          </span>
+          <span className="block truncate px-2 py-1.5 text-[10px] text-panel-text-2">
+            {adding === overlay.name ? "Adding…" : overlay.title}
+          </span>
+        </button>
+      ))}
+    </div>
+  );
+}

package/src/components/editor/propertyPanelFlatPrimitives.test.tsx

@@ -1159,20 +1159,20 @@ describe("FlatSelectRow — label/value options", () => {
     const { host, root } = renderInto(
       <FlatSelectRow
         label="Preset"
-        value="natural-lift"
+        value="clean-studio"
         options={[
           { value: "neutral", label: "Neutral" },
-          { value: "natural-lift", label: "Natural Lift" },
-          { value: "fresh-pop", label: "Fresh Pop" },
+          { value: "clean-studio", label: "Clean Studio" },
+          { value: "bright-pop", label: "Bright Pop" },
         ]}
         tier="explicitCustom"
         onChange={vi.fn()}
       />,
     );
     const select = host.querySelector("select");
-    expect(select?.value).toBe("natural-lift");
+    expect(select?.value).toBe("clean-studio");
     const options = Array.from(host.querySelectorAll("option")).map((o) => o.textContent);
-    expect(options).toEqual(["Neutral", "Natural Lift", "Fresh Pop"]);
+    expect(options).toEqual(["Neutral", "Clean Studio", "Bright Pop"]);
     act(() => root.unmount());
   });
 

package/src/components/editor/propertyPanelFlatPrimitives.tsx

@@ -8,6 +8,8 @@ import {
   type PropertyValueTier,
 } from "./propertyPanelValueTier";
 
+export const FLAT_PREVIEW_GRID = "grid grid-cols-[repeat(auto-fill,minmax(120px,1fr))] gap-1";
+
 /* ------------------------------------------------------------------ */
 /*  FlatRow — single-column label/value property row                   */
 /* ------------------------------------------------------------------ */

package/src/components/editor/propertyPanelPresetPreview.ts

@@ -0,0 +1,36 @@
+import type { NormalizedHfColorGrading } from "@hyperframes/core/color-grading";
+import type { ColorGradingPreviewOptions } from "./useColorGradingController";
+
+export function presetPreviewHandlers({
+  id,
+  label,
+  resolve,
+  onPreview,
+  onCommit,
+  onTrack,
+}: {
+  id: string;
+  label: string;
+  resolve: () => NormalizedHfColorGrading;
+  onPreview: (
+    grading: NormalizedHfColorGrading | null,
+    options?: ColorGradingPreviewOptions,
+  ) => void;
+  onCommit: (grading: NormalizedHfColorGrading) => void;
+  onTrack: (label: string) => void;
+}) {
+  return {
+    title: `Preview ${label}`,
+    onPointerEnter: () =>
+      onPreview(resolve(), {
+        animatedPreview: { kind: "presets" as const, id },
+      }),
+    onPointerLeave: () => onPreview(null),
+    onFocus: () => onPreview(resolve()),
+    onBlur: () => onPreview(null),
+    onClick: () => {
+      onTrack(label);
+      onCommit(resolve());
+    },
+  };
+}

package/src/components/editor/propertyPanelTypes.ts

@@ -19,6 +19,18 @@ export interface BackgroundRemovalResult {
   provider?: string;
 }
 
+export interface MediaOverlayPlacement {
+  start: number;
+  duration?: number;
+  track?: number;
+  compositionPath?: string;
+}
+
+export type AddMediaOverlayHandler = (
+  blockName: string,
+  placement: MediaOverlayPlacement,
+) => Promise<void>;
+
 export interface PropertyPanelProps {
   projectId: string;
   projectDir: string | null;
@@ -69,6 +81,7 @@ export interface PropertyPanelProps {
   onAskAgent: () => void;
   onToggleElementHidden?: (elementKey: string, hidden: boolean) => void | Promise<void>;
   onImportAssets?: (files: FileList, dir?: string) => Promise<string[]>;
+  onAddMediaOverlay?: AddMediaOverlayHandler;
   fontAssets?: ImportedFontAsset[];
   onImportFonts?: (files: FileList | File[]) => Promise<ImportedFontAsset[]>;
   previewIframeRef?: RefObject<HTMLIFrameElement | null>;

package/src/components/editor/useColorGradingController.test.ts

@@ -7,15 +7,15 @@ import { normalizeHfColorGrading } from "@hyperframes/core/color-grading";
 import { useColorGradingController } from "./useColorGradingController";
 import type { DomEditSelection } from "./domEditing";
 
-function freshPopGrading() {
-  const next = normalizeHfColorGrading({ preset: "fresh-pop", intensity: 1 });
-  if (!next) throw new Error("expected fresh-pop preset to normalize");
+function brightPopGrading() {
+  const next = normalizeHfColorGrading({ preset: "bright-pop", intensity: 1 });
+  if (!next) throw new Error("expected bright-pop preset to normalize");
   return next;
 }
 
-function naturalLiftGrading() {
-  const next = normalizeHfColorGrading({ preset: "natural-lift", intensity: 1 });
-  if (!next) throw new Error("expected natural-lift preset to normalize");
+function cleanStudioGrading() {
+  const next = normalizeHfColorGrading({ preset: "clean-studio", intensity: 1 });
+  if (!next) throw new Error("expected clean-studio preset to normalize");
   return next;
 }
 
@@ -60,14 +60,17 @@ function HookHost({
   onState,
   onSetAttributeLive,
   element,
+  previewIframeRef,
 }: {
   onState: (state: ReturnType<typeof useColorGradingController>) => void;
   onSetAttributeLive: (attr: string, value: string | null) => void;
   element: DomEditSelection;
+  previewIframeRef?: React.RefObject<HTMLIFrameElement | null>;
 }) {
   const state = useColorGradingController({
     projectId: "proj",
     element,
+    previewIframeRef,
     onSetAttributeLive,
   });
   onState(state);
@@ -77,6 +80,7 @@ function HookHost({
 function renderHook(
   onSetAttributeLive: (attr: string, value: string | null) => void,
   initialElement: DomEditSelection = makeElement(),
+  previewIframeRef?: React.RefObject<HTMLIFrameElement | null>,
 ) {
   const host = document.createElement("div");
   document.body.append(host);
@@ -89,6 +93,7 @@ function renderHook(
           onState: (s: ReturnType<typeof useColorGradingController>) => (latest = s),
           onSetAttributeLive,
           element,
+          previewIframeRef,
         }),
       );
     });
@@ -107,6 +112,43 @@ function renderHook(
   };
 }
 
+type PreviewWindow = Window & {
+  __hf?: {
+    colorGrading?: {
+      renderPreviews?: ReturnType<typeof vi.fn>;
+      startPreviewPlayback?: ReturnType<typeof vi.fn>;
+    };
+  };
+  __player?: { play: ReturnType<typeof vi.fn> };
+};
+
+function createPreviewFrame() {
+  const iframe = document.body.appendChild(document.createElement("iframe"));
+  const contentWindow = iframe.contentWindow as PreviewWindow | null;
+  if (!contentWindow) throw new Error("expected iframe contentWindow");
+  return { contentWindow, iframe };
+}
+
+async function flushPreviewRequest() {
+  act(() => vi.advanceTimersByTime(0));
+  await act(async () => {
+    await Promise.resolve();
+    await Promise.resolve();
+  });
+}
+
+function colorGradingMessages(calls: ReadonlyArray<ReadonlyArray<unknown>>) {
+  return calls
+    .map(([message]) => message)
+    .filter(
+      (message): message is { action: string; grading: unknown } =>
+        typeof message === "object" &&
+        message !== null &&
+        "action" in message &&
+        message.action === "set-color-grading",
+    );
+}
+
 describe("useColorGradingController", () => {
   it("starts with the neutral (inactive) grading and idle compare state", () => {
     const { root, getState } = renderHook(vi.fn());
@@ -115,14 +157,85 @@ describe("useColorGradingController", () => {
     act(() => root.unmount());
   });
 
+  it("requests one exact preset batch from the selected media runtime", async () => {
+    vi.useFakeTimers();
+    const devicePixelRatio = vi.spyOn(window, "devicePixelRatio", "get").mockReturnValue(2);
+    const { contentWindow, iframe } = createPreviewFrame();
+    const renderPreviews = vi.fn().mockResolvedValue({
+      width: 160,
+      height: 90,
+      images: [{ id: "bright-pop", dataUrl: "data:image/png;base64,bright" }],
+    });
+    contentWindow.__hf = { colorGrading: { renderPreviews } };
+    const { root, getState } = renderHook(vi.fn(), makeElement(), { current: iframe });
+
+    act(() => getState().requestPresetPreviews());
+    await flushPreviewRequest();
+
+    expect(renderPreviews).toHaveBeenCalledTimes(1);
+    expect(renderPreviews.mock.calls[0]?.[1]).toHaveLength(18);
+    expect(renderPreviews.mock.calls[0]?.[2]).toEqual({ maxDimension: 320 });
+    expect(getState().presetPreviews).toEqual({
+      status: "ready",
+      images: { "bright-pop": "data:image/png;base64,bright" },
+      width: 160,
+      height: 90,
+    });
+    act(() => root.unmount());
+    devicePixelRatio.mockRestore();
+    vi.useRealTimers();
+  });
+
+  it("requests exact effect families and retains earlier family images", async () => {
+    vi.useFakeTimers();
+    const { contentWindow, iframe } = createPreviewFrame();
+    const renderPreviews = vi
+      .fn()
+      .mockImplementation(async (_target: unknown, candidates: Array<{ id: string }>) => ({
+        width: 160,
+        height: 90,
+        images: candidates.map(({ id }) => ({ id, dataUrl: `data:image/png;base64,${id}` })),
+      }));
+    contentWindow.__hf = { colorGrading: { renderPreviews } };
+    const { root, getState } = renderHook(vi.fn(), makeElement(), { current: iframe });
+
+    act(() => getState().requestEffectPreviews(["blur", "pixelate", "bloom"]));
+    await flushPreviewRequest();
+
+    expect(renderPreviews).toHaveBeenCalledTimes(1);
+    expect(renderPreviews.mock.calls[0]?.[1].map(({ id }: { id: string }) => id)).toEqual([
+      "blur",
+      "pixelate",
+      "bloom",
+    ]);
+
+    act(() => getState().requestEffectPreviews(["kuwahara"]));
+    await flushPreviewRequest();
+
+    expect(renderPreviews).toHaveBeenCalledTimes(2);
+    expect(getState().effectPreviews).toEqual({
+      status: "ready",
+      images: {
+        blur: "data:image/png;base64,blur",
+        pixelate: "data:image/png;base64,pixelate",
+        bloom: "data:image/png;base64,bloom",
+        kuwahara: "data:image/png;base64,kuwahara",
+      },
+      width: 160,
+      height: 90,
+    });
+    act(() => root.unmount());
+    vi.useRealTimers();
+  });
+
   it("commitColorGrading updates grading state synchronously and schedules a debounced persist", async () => {
     vi.useFakeTimers();
     const onSetAttributeLive = vi.fn();
     const { root, getState } = renderHook(onSetAttributeLive);
     act(() => {
-      getState().commitColorGrading(freshPopGrading());
+      getState().commitColorGrading(brightPopGrading());
     });
-    expect(getState().grading.preset).toBe("fresh-pop");
+    expect(getState().grading.preset).toBe("bright-pop");
     expect(onSetAttributeLive).not.toHaveBeenCalled();
     act(() => {
       vi.advanceTimersByTime(400);
@@ -130,7 +243,87 @@ describe("useColorGradingController", () => {
     expect(onSetAttributeLive).toHaveBeenCalledTimes(1);
     const [attr, value] = onSetAttributeLive.mock.calls[0] as [string, string];
     expect(attr).toBe("color-grading");
-    expect(value).toContain("fresh-pop");
+    expect(value).toContain("bright-pop");
+    act(() => root.unmount());
+    vi.useRealTimers();
+  });
+
+  it("previews through the runtime only and restores the committed grade without persisting", () => {
+    vi.useFakeTimers();
+    const onSetAttributeLive = vi.fn();
+    const { contentWindow, iframe } = createPreviewFrame();
+    const postMessage = vi.spyOn(contentWindow, "postMessage");
+    const { root, getState } = renderHook(onSetAttributeLive, makeElement(), {
+      current: iframe,
+    });
+
+    act(() => getState().previewColorGrading(brightPopGrading()));
+    act(() => getState().previewColorGrading(null));
+    const gradingMessages = colorGradingMessages(postMessage.mock.calls);
+    expect(gradingMessages).toHaveLength(2);
+    expect(gradingMessages[0]?.grading).toMatchObject({ preset: "bright-pop" });
+    expect(gradingMessages[1]?.grading).toBeNull();
+    expect(getState().grading.preset).toBe("neutral");
+    act(() => vi.advanceTimersByTime(500));
+    expect(onSetAttributeLive).not.toHaveBeenCalled();
+
+    act(() => root.unmount());
+    vi.useRealTimers();
+  });
+
+  it("animates only the selected video card without starting the project player", async () => {
+    vi.useFakeTimers();
+    const { contentWindow, iframe } = createPreviewFrame();
+    const stopPlayback = vi.fn();
+    const renderPreviews = vi.fn().mockResolvedValue({
+      width: 320,
+      height: 180,
+      images: [{ id: "bright-pop", dataUrl: "data:image/png;base64,animated" }],
+    });
+    const startPreviewPlayback = vi.fn(() => stopPlayback);
+    contentWindow.__hf = { colorGrading: { renderPreviews, startPreviewPlayback } };
+    contentWindow.__player = { play: vi.fn() };
+    const { root, getState } = renderHook(vi.fn(), makeElement(), { current: iframe });
+
+    act(() =>
+      getState().previewColorGrading(brightPopGrading(), {
+        animatedPreview: { kind: "presets", id: "bright-pop" },
+      }),
+    );
+    await act(async () => {
+      await Promise.resolve();
+      await Promise.resolve();
+    });
+    expect(startPreviewPlayback).toHaveBeenCalledTimes(1);
+    expect(renderPreviews).toHaveBeenCalledWith(
+      expect.anything(),
+      [{ id: "bright-pop", grading: expect.objectContaining({ preset: "bright-pop" }) }],
+      { maxDimension: 160, useMediaTime: true },
+    );
+    expect(contentWindow.__player.play).not.toHaveBeenCalled();
+    expect(getState().presetPreviews.images["bright-pop"]).toBe("data:image/png;base64,animated");
+
+    act(() => getState().previewColorGrading(null));
+    expect(stopPlayback).toHaveBeenCalledTimes(1);
+
+    act(() => root.unmount());
+    vi.useRealTimers();
+  });
+
+  it("restores a just-committed look even before React renders the new state", () => {
+    vi.useFakeTimers();
+    const { contentWindow, iframe } = createPreviewFrame();
+    const postMessage = vi.spyOn(contentWindow, "postMessage");
+    const { root, getState } = renderHook(vi.fn(), makeElement(), { current: iframe });
+    const brightPop = brightPopGrading();
+
+    act(() => {
+      getState().commitColorGrading(brightPop);
+      getState().previewColorGrading(null);
+    });
+    const gradingMessages = colorGradingMessages(postMessage.mock.calls);
+    expect(gradingMessages.at(-1)?.grading).toMatchObject({ preset: "bright-pop" });
+
     act(() => root.unmount());
     vi.useRealTimers();
   });
@@ -149,9 +342,9 @@ describe("useColorGradingController", () => {
     );
     const { root, getState } = renderHook(onSetAttributeLive);
     act(() => {
-      getState().commitColorGrading(freshPopGrading());
+      getState().commitColorGrading(brightPopGrading());
     });
-    expect(getState().grading.preset).toBe("fresh-pop");
+    expect(getState().grading.preset).toBe("bright-pop");
     act(() => {
       vi.advanceTimersByTime(400);
     });
@@ -170,9 +363,9 @@ describe("useColorGradingController", () => {
     const onSetAttributeLive = vi.fn().mockRejectedValue(new Error("disk full"));
     const { root, getState } = renderHook(onSetAttributeLive);
     act(() => {
-      getState().commitColorGrading(freshPopGrading());
+      getState().commitColorGrading(brightPopGrading());
     });
-    expect(getState().grading.preset).toBe("fresh-pop");
+    expect(getState().grading.preset).toBe("bright-pop");
     act(() => {
       vi.advanceTimersByTime(400);
     });
@@ -182,7 +375,7 @@ describe("useColorGradingController", () => {
       await Promise.resolve();
     });
     // Reverted to "neutral" (the last confirmed-good value, from before this
-    // commit) instead of permanently showing "fresh-pop" as if it had saved.
+    // commit) instead of permanently showing "bright-pop" as if it had saved.
     expect(getState().grading.preset).toBe("neutral");
     expect(getState().runtimeStatus.state).toBe("unavailable");
     act(() => root.unmount());
@@ -206,7 +399,7 @@ describe("useColorGradingController", () => {
       makeElement({ id: "s1-bg" }),
     );
     act(() => {
-      getState().commitColorGrading(freshPopGrading());
+      getState().commitColorGrading(brightPopGrading());
     });
     // Let the debounce fire while still on s1-bg — the persist call is now
     // genuinely in flight (its promise won't settle until resolveA() below).
@@ -243,7 +436,7 @@ describe("useColorGradingController", () => {
     let capturedOnSettledA: ((ok: boolean) => void) | undefined;
     const onSetAttributeLive = vi
       .fn()
-      // Edit A (fresh-pop): captures its onSettled and never resolves until
+      // Edit A (bright-pop): captures its onSettled and never resolves until
       // resolveA() is called below — simulates a slow persist.
       .mockImplementationOnce(
         (_attr: string, _value: string | null, onSettled?: (ok: boolean) => void) => {
@@ -253,7 +446,7 @@ describe("useColorGradingController", () => {
           });
         },
       )
-      // Edit B (natural-lift): settles immediately and successfully.
+      // Edit B (clean-studio): settles immediately and successfully.
       .mockImplementationOnce(
         (_attr: string, _value: string | null, onSettled?: (ok: boolean) => void) => {
           onSettled?.(true);
@@ -263,7 +456,7 @@ describe("useColorGradingController", () => {
     const { root, getState } = renderHook(onSetAttributeLive);
 
     act(() => {
-      getState().commitColorGrading(freshPopGrading());
+      getState().commitColorGrading(brightPopGrading());
     });
     act(() => {
       vi.advanceTimersByTime(400);
@@ -272,13 +465,13 @@ describe("useColorGradingController", () => {
 
     // B commits on the SAME element before A's persist has settled.
     act(() => {
-      getState().commitColorGrading(naturalLiftGrading());
+      getState().commitColorGrading(cleanStudioGrading());
     });
     act(() => {
       vi.advanceTimersByTime(400);
     });
     expect(onSetAttributeLive).toHaveBeenCalledTimes(2); // B's persist has already settled (mock resolves sync)
-    expect(getState().grading.preset).toBe("natural-lift");
+    expect(getState().grading.preset).toBe("clean-studio");
 
     // NOW A's stale persist finally settles as a FAILURE — must not revert
     // `grading` (which now correctly shows B's newer edit) back to the
@@ -292,20 +485,32 @@ describe("useColorGradingController", () => {
       await Promise.resolve();
       await Promise.resolve();
     });
-    expect(getState().grading.preset).toBe("natural-lift");
+    expect(getState().grading.preset).toBe("clean-studio");
     act(() => root.unmount());
     vi.useRealTimers();
   });
 
-  it("resetGrading returns to the neutral preset", () => {
+  it("resetGrading resets Grade fields without clearing Effects or Palette", () => {
     const { root, getState } = renderHook(vi.fn());
+    const grading = normalizeHfColorGrading({
+      preset: "bright-pop",
+      lut: { src: "assets/luts/custom.cube", intensity: 0.6 },
+      effects: { pixelate: 0.5 },
+      palette: ["#112233", "#ffffff"],
+    });
+    if (!grading) throw new Error("expected grading to normalize");
     act(() => {
-      getState().commitColorGrading(freshPopGrading());
+      getState().commitColorGrading(grading);
     });
     act(() => {
       getState().resetGrading();
     });
-    expect(getState().grading.preset).toBe("neutral");
+    expect(getState().grading).toMatchObject({
+      preset: "neutral",
+      lut: null,
+      effects: { pixelate: 0.5 },
+      palette: ["#112233", "#ffffff"],
+    });
     act(() => root.unmount());
   });
 
@@ -315,9 +520,9 @@ describe("useColorGradingController", () => {
       makeElement({ id: "s1-bg" }),
     );
     act(() => {
-      getState().commitColorGrading(freshPopGrading());
+      getState().commitColorGrading(brightPopGrading());
     });
-    expect(getState().grading.preset).toBe("fresh-pop");
+    expect(getState().grading.preset).toBe("bright-pop");
     // A different element, with no persisted grading of its own — without a
     // reset, this hook (unlike the legacy component it was extracted from,
     // which remounts via a `key={selectionIdentityKey}`) would keep showing
@@ -337,9 +542,9 @@ describe("useColorGradingController", () => {
       makeElement({ id: "bg", sourceFile: "index.html" }),
     );
     act(() => {
-      getState().commitColorGrading(freshPopGrading());
+      getState().commitColorGrading(brightPopGrading());
     });
-    expect(getState().grading.preset).toBe("fresh-pop");
+    expect(getState().grading.preset).toBe("bright-pop");
     rerenderWithElement(makeElement({ id: "bg", sourceFile: "sub-comp.html" }));
     expect(getState().grading.preset).toBe("neutral");
     act(() => root.unmount());
@@ -353,7 +558,7 @@ describe("useColorGradingController", () => {
       makeElement({ id: "s1-bg" }),
     );
     act(() => {
-      getState().commitColorGrading(freshPopGrading());
+      getState().commitColorGrading(brightPopGrading());
     });
     // Switch selection before the 350ms debounce fires — the in-flight edit
     // must be written immediately (targeting the OUTGOING element's own
@@ -366,7 +571,7 @@ describe("useColorGradingController", () => {
     expect(onSetAttributeLive).toHaveBeenCalledTimes(1);
     const [attr, value] = onSetAttributeLive.mock.calls[0] as [string, string];
     expect(attr).toBe("color-grading");
-    expect(value).toContain("fresh-pop");
+    expect(value).toContain("bright-pop");
     // And it must not ALSO fire again once the (now-cleared) original timer
     // window would have elapsed.
     act(() => {