fix: prevent crash when trigger is called after dispose

[mfazekas]

Calling trigger() from a stale closure (e.g. a Reanimated callback that fires after navigation/unmount) would crash because the native property was already disposed.

useDisposableMemo now accepts an optional liveRef parameter — set on create, cleared on dispose. useRiveTrigger uses this to safely no-op with a console.warn instead of crashing.

Reproducer: mount RiveView with trigger → unmount → call trigger from async callback.

[Copilot]

Pull request overview

This PR addresses a crash caused by calling trigger() from stale closures after a RiveView unmounts/cleans up its native trigger property, by tracking whether the underlying native property is still “live” and safely no-op’ing when it isn’t.

Changes:

• Add optional liveRef tracking to useDisposableMemo (set on create, cleared on dispose).
• Thread liveRef through useRiveProperty options so hooks can observe property liveness.
• Update useRiveTrigger to call trigger() via liveRef and warn/no-op instead of crashing after disposal.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File	Description
src/hooks/useRiveTrigger.ts	Switches trigger execution to a liveness-tracked ref and adds a warning/no-op path.
src/hooks/useRiveProperty.ts	Extends property options to accept a liveRef and forwards it into disposable memoization.
src/hooks/useDisposableMemo.ts	Adds liveRef support and clears/sets it alongside disposal/creation lifecycle.

Comments suppressed due to low confidence (1)

src/hooks/useDisposableMemo.ts:70

• liveRef is only synchronized inside the deps-change/initialization branch. If a caller starts passing a liveRef after the initial render (or swaps to a different ref) while deps remain equal, the new ref will never be populated with the current value. Consider handling liveRef changes by immediately syncing liveRef.current to the current memoized value (and clearing any previous liveRef) even when deps are unchanged.

  if (
    ref.current.deps === UNINITIALIZED ||
    !depsEqual(ref.current.deps, deps)
  ) {

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[HayesGordon]

LGTM. Thanks for the quick fix