From d71663ccf115ecdabbac64b0f2c71298aca3ece1 Mon Sep 17 00:00:00 2001 From: Miguel Angel Simon Sierra Date: Mon, 21 Sep 2026 08:29:27 -0400 Subject: [PATCH] fix(studio): remove stale timeline model inputs --- packages/studio/src/player/components/TimelineLanes.test.tsx | 1 - packages/studio/src/player/components/TimelineTrackHeader.tsx | 4 ++++ .../studio/src/player/components/useTimelineLogicalFocus.ts | 3 --- .../src/player/components/useTimelineLogicalRows.test.tsx | 2 -- .../studio/src/player/components/useTimelineProviderState.tsx | 2 -- 5 files changed, 4 insertions(+), 8 deletions(-) diff --git a/packages/studio/src/player/components/TimelineLanes.test.tsx b/packages/studio/src/player/components/TimelineLanes.test.tsx index b5660d3257..10eb2d7505 100644 --- a/packages/studio/src/player/components/TimelineLanes.test.tsx +++ b/packages/studio/src/player/components/TimelineLanes.test.tsx @@ -121,7 +121,6 @@ function renderLanes(options: RenderLanesOptions = {}): { expandedLaneOwnerIds: new Set(), groups: [], trackGroupOf: new Map(), - gsapAnimations, })} clipIndex={createTimelineClipIndex(tracks)} renderTimeRange={{ start: 0, end: Number.POSITIVE_INFINITY }} diff --git a/packages/studio/src/player/components/TimelineTrackHeader.tsx b/packages/studio/src/player/components/TimelineTrackHeader.tsx index 8697449df4..90ed0f105d 100644 --- a/packages/studio/src/player/components/TimelineTrackHeader.tsx +++ b/packages/studio/src/player/components/TimelineTrackHeader.tsx @@ -53,6 +53,8 @@ interface TimelineTrackHeaderProps { * clip and one by the row. Minted by TimelineLanes, the one place that sees * every subtree. */ lanesId: string; + /** @deprecated Keyframe property lanes were removed; retained only for stale callers during migration. */ + animations?: readonly unknown[]; contentOrigin: number; /** The track's active keyframe clip (selected, else primary) — the one whose * disclosure + property rows this header shows, whether expanded or not. */ @@ -71,6 +73,8 @@ interface TimelineTrackHeaderProps { rovingTargetId?: string | null; theme: TimelineTheme; onToggleClipExpanded: () => void; + /** @deprecated Keyframe property controls were removed. */ + onTogglePropertyGroupKeyframe?: TimelineEditCallbacks["onTogglePropertyGroupKeyframe"]; onToggleTrackHidden: TimelineEditCallbacks["onToggleTrackHidden"]; /** Drop one envelope. Absent while the lanes are read-only, which is what * hides the control rather than offering a button that cannot act. */ diff --git a/packages/studio/src/player/components/useTimelineLogicalFocus.ts b/packages/studio/src/player/components/useTimelineLogicalFocus.ts index e157185205..d5190e89de 100644 --- a/packages/studio/src/player/components/useTimelineLogicalFocus.ts +++ b/packages/studio/src/player/components/useTimelineLogicalFocus.ts @@ -1,5 +1,4 @@ import type { RefObject } from "react"; -import type { GsapAnimation } from "@hyperframes/core/gsap-parser"; import type { TimelineElement } from "../store/playerStore"; import type { TimelineRowGeometry } from "./timelineLayout"; import type { TimelineScrollViewportSnapshot } from "./useTimelineScrollViewport"; @@ -18,7 +17,6 @@ interface TimelineLogicalFocusInput { selectedElementIds: ReadonlySet; groups: readonly TimelineTrackGroupInfo[]; trackGroupOf: ReadonlyMap; - gsapAnimations: ReadonlyMap; elements: readonly TimelineElement[]; pixelsPerSecond: number; contentOrigin: number; @@ -47,7 +45,6 @@ export function useTimelineLogicalFocus(input: TimelineLogicalFocusInput) { expandedLaneOwnerIds, groups: input.groups, trackGroupOf: input.trackGroupOf, - gsapAnimations: input.gsapAnimations, }); const focus = useTimelineFocusCoordinator({ scrollRef: input.scrollRef, diff --git a/packages/studio/src/player/components/useTimelineLogicalRows.test.tsx b/packages/studio/src/player/components/useTimelineLogicalRows.test.tsx index cb57440ffb..86071476d5 100644 --- a/packages/studio/src/player/components/useTimelineLogicalRows.test.tsx +++ b/packages/studio/src/player/components/useTimelineLogicalRows.test.tsx @@ -26,7 +26,6 @@ const collapsedGroupIds = new Set(); const expandedLaneOwnerIds = new Set(); const groups: never[] = []; const trackGroupOf = new Map(); -const gsapAnimations = new Map(); function Harness({ snapshots, @@ -46,7 +45,6 @@ function Harness({ expandedLaneOwnerIds, groups, trackGroupOf, - gsapAnimations, }); snapshots.push(logicalRows); return null; diff --git a/packages/studio/src/player/components/useTimelineProviderState.tsx b/packages/studio/src/player/components/useTimelineProviderState.tsx index 1ce28225e3..f89c61e426 100644 --- a/packages/studio/src/player/components/useTimelineProviderState.tsx +++ b/packages/studio/src/player/components/useTimelineProviderState.tsx @@ -147,7 +147,6 @@ export function useTimelineProviderState({ trackGroupOf, } = useTimelineTrackLayout( timelineElements, - gsapAnimations, selectedElementId, selectedElementIds, ); @@ -259,7 +258,6 @@ export function useTimelineProviderState({ selectedElementIds, groups, trackGroupOf, - gsapAnimations, elements: timelineElements, pixelsPerSecond: pps, contentOrigin,