Skip to content

@remotion/studio-server: Normalize legacy keyframed sequence status#7764

Merged
JonnyBurger merged 3 commits into
mainfrom
jonnyburger/fix-keyframed-server
May 28, 2026
Merged

@remotion/studio-server: Normalize legacy keyframed sequence status#7764
JonnyBurger merged 3 commits into
mainfrom
jonnyburger/fix-keyframed-server

Conversation

@JonnyBurger
Copy link
Copy Markdown
Member

Summary

  • normalize legacy sequence prop status returned by /api/save-sequence-props
  • map malformed computed statuses with keyframes to keyframed
  • provide default easing/clamping fallback for normalized keyframed payloads

Testing

  • bunx oxfmt src --write (packages/studio-server)
  • bun run build
  • bun run formatting

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 28, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
bugs Ready Ready Preview, Comment May 28, 2026 6:19pm
remotion Error Error May 28, 2026 6:19pm

Request Review

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@pullfrog pullfrog Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ No new issues found.

Reviewed changes — adds server-side normalization to convert legacy computed sequence prop statuses that carry keyframe data into properly-typed keyframed statuses with sensible defaults (linear easing, extend clamping) before returning from /api/save-sequence-props.

  • Add normalizePropStatus helper in save-sequence-props.ts — detects malformed computed statuses containing a keyframes array, validates element shape, and rewrites them to keyframed.
  • Wire normalization into response — iterates over all props returned by computeSequencePropsOnlyStatus and applies normalizePropStatus before sending the response.

Pullfrog  | View workflow run | Using Kimi K2𝕏

Copy link
Copy Markdown
Contributor

@pullfrog pullfrog Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ No new issues found.

Reviewed changes — adds server-side normalization to convert legacy computed sequence prop statuses that carry keyframe data into properly-typed keyframed statuses with sensible defaults (linear easing, extend clamping) before returning from /api/save-sequence-props.

  • Add normalizePropStatus helper in save-sequence-props.ts — detects malformed computed statuses containing a keyframes array, validates element shape, and rewrites them to keyframed.
  • Wire normalization into response — iterates over all props returned by computeSequencePropsOnlyStatus and applies normalizePropStatus before sending the response.

Pullfrog  | View workflow run | Using Kimi K2𝕏

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@JonnyBurger JonnyBurger merged commit a9b9a38 into main May 28, 2026
13 of 15 checks passed
@JonnyBurger JonnyBurger deleted the jonnyburger/fix-keyframed-server branch May 28, 2026 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant