Skip to content

fix(core): correct WAAPI rediscovery seek baselines and preview reuse…#1069

Merged
miguel-heygen merged 2 commits into
heygen-com:mainfrom
func25:fix/preview-reuse-and-waapi-seek-baselines
May 25, 2026
Merged

fix(core): correct WAAPI rediscovery seek baselines and preview reuse…#1069
miguel-heygen merged 2 commits into
heygen-com:mainfrom
func25:fix/preview-reuse-and-waapi-seek-baselines

Conversation

@func25
Copy link
Copy Markdown
Contributor

@func25 func25 commented May 25, 2026

… invalidation

What

Fix two related preview/runtime issues:

  • correct WAAPI seek baselines for animations that appear after the composition has already advanced, including rediscovery paths
  • stop reusing same-project preview servers when the embedded preview server build is stale

Why

Some WAAPI animations created mid-seek could inherit absolute composition time and collapse into the wrong visual state. In rediscovery paths, newly seen animations could also be double-counted after time had already advanced.

Preview server reuse only invalidated on project identity, and later only on runtime script identity. That was too narrow and could still reuse a stale same-project preview server even when other preview-server behavior had changed.

How

  • track WAAPI baselines relative to the current seek/discovery point instead of always anchoring at zero
  • add regression coverage for:
    • newly discovered WAAPI animations
    • rebased inherited absolute time
    • rediscovery after time has already advanced
    • replace preview reuse comparison based on runtime-only identity with a broader server build signature

Test plan

How was this tested?

  • Unit tests added/updated
  • Manual testing performed
  • Documentation updated (if applicable)

@miguel-heygen miguel-heygen merged commit 8b776cb into heygen-com:main May 25, 2026
45 checks passed
@func25 func25 deleted the fix/preview-reuse-and-waapi-seek-baselines branch May 25, 2026 13:35
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.

2 participants