Fix onUpdate crash (#3636)

m-bert · web-flow · commit ff64e3f72eda · 2025-07-25T11:13:35.000+02:00
## Description #3630 introduced automatic support for `Animated.Event` in `onUpdate` callback. However, I've missed the fact that this callback may not be defined. This PR fixes this crash. ## Test plan Tested on current basic-example without specifying `onUpdate`
diff --git a/packages/react-native-gesture-handler/src/v3/hooks/utils.ts b/packages/react-native-gesture-handler/src/v3/hooks/utils.ts
@@ -89,11 +89,11 @@ export function isEventForHandlerWithTag(
 }
 
 export function isAnimatedEvent(
-  callback: ((event: any) => void) | AnimatedEvent
+  callback: ((event: any) => void) | AnimatedEvent | undefined
 ): callback is AnimatedEvent {
   'worklet';
 
-  return '_argMapping' in callback;
+  return !!callback && '_argMapping' in callback;
 }
 
 export function checkMappingForChangeProperties(obj: Animated.Mapping) {

Original file line number	Diff line number	Diff line change
`@@ -89,11 +89,11 @@ export function isEventForHandlerWithTag(`
`89`	`89`	`}`
`90`	`90`
`91`	`91`	`export function isAnimatedEvent(`
`92`		`- callback: ((event: any) => void) \| AnimatedEvent`
	`92`	`+ callback: ((event: any) => void) \| AnimatedEvent \| undefined`
`93`	`93`	`): callback is AnimatedEvent {`
`94`	`94`	`'worklet';`
`95`	`95`
`96`		`- return '_argMapping' in callback;`
	`96`	`+ return !!callback && '_argMapping' in callback;`
`97`	`97`	`}`
`98`	`98`
`99`	`99`	`export function checkMappingForChangeProperties(obj: Animated.Mapping) {`