[j-piasecki]

Description

Discrete gestures shouldn't be sending update events, so the onUpdate method shouldn't be accepted by the type definitions.

It also moves the type definitions for events and gestures above the hook definition. I'm pretty sure it was also correct before, but I think it's cleaner when things are declared before they are used. LMK what you think.

Test plan

  const tap = useTap({
    onUpdate: () => {
      'worklet';
    },
  });

  const fling = useFling({
    onUpdate: () => {
      'worklet';
    },
  });

  const longpress = useLongPress({
    onUpdate: () => {
      'worklet';
    },
  });

  const native = useNative({
    onUpdate: () => {
      'worklet';
    },
  });

[m-bert]

Is specifying return type necessary?

[j-piasecki]

Without it, the type is inferred from useGesture, which doesn't know what kind of gesture it creates, so all are typed the same as continuous gestures. Another benefit is that gestures are now using named types.