OvershootClamping does not work since v1.5.0

Hey everyone. Thanks for a great library.

I’m trying to implement an animation that runs a spring with overshootClamping. The code was working fine until I upgraded Reanimated to the newest version. Below is the code in question:

function runSpring(
  clock: Animated.Clock,
  animationState: Animated.PhysicsAnimationState,
): Animated.Node<number> {
  const config = {
    ...SpringUtils.makeDefaultConfig(),
    overshootClamping: true,
  };

  return block([
    cond(not(clockRunning(clock)), [
      set(animationState.finished, 0),
      startClock(clock),
    ]),
    spring(clock, animationState, config),
    cond(animationState.finished, stopClock(clock)),
    animationState.position,
  ]);
}

There seems to have been some regression in v1.5.0, because the code above no longer works properly. The animation does not apply the overshootClamping: true to the configuration.

Hopefully this can be fixed in future versions, as this issue prevents me from upgrading past version 1.4.0

[jakub-gonet]

I managed to reproduce it on a snappable example in REA Example app.

After bisecting, it seems #468 (2dc3df7) introduced this regression.

@felixakiragreen, @Sakarisson

A possible workaround is to change export default (...) at the end of src/animation/spring.js to export default spring;

[jakub-gonet]

You can use patch-package library to do it, here’s a quick blog post about it