[Bug]: NewArch: Exception in setHandledMapChangedEvents when loading the map in RN 0.74.2

[kubabutkiewicz]

Mapbox Implementation

Mapbox

Mapbox Version

default

React Native Version

0.74.2

Platform

Android

@rnmapbox/maps version

10.1.23

Standalone component to reproduce

// I couldn't reproduce it in example app
export default () => {}

Observed behavior and steps to reproduce

We are working in Expensify on upgrading RN version to 0.74.2 and we noticed the crash caused by rnmapbox. When we navigating to screen when we are displaying map we are getting this error Error: Exception in HostFunction: no non-static method "Lcom/rnmapbox/rnmbx/components/mapview/NativeMapViewModule;.setHandledMapChangedEvents(Ljava/lang/Double;Lcom/facebook/react/bridge/ReadableArray;Lcom/facebook/react/bridge/Promise;)V" . More context and logs are here.

Expected behavior

No response

Notes / preliminary analysis

No response

Additional links and references

rnmapbox-bug.mov

No response

[mfazekas]

Should be fixed by #3527, there might be further similar issues

[mfazekas]

See facebook/react-native#45024
facebook/react-native#44963

As expected there other issues as well.

#3511 (comment)

Could be worked around with patch-package, see mfazekas@dbcf2b1 for example

[roryabraham]

Looks like the upstream issue may have been fixed in facebook/react-native@8a6b88e, though it doesn't look like that has been included in a release yet

[mfazekas]

@roryabraham so it's a bit complicated. So the issue is that in RN 0.73 what was Int32 in typescript Native codespec in was generated as Double.
In RN 0.74 this was fixed so Int32 in typescript, generated Long etc on native side. But this was not 100% change and runtime was looking for Double on android side. The plan was to fix that, but because of backward compatibility it was decided to go back 0.73 where Int32 is Double. This is something I see even in RN 0.75.rc*. Not sure about 0.74.3, will check.

I guess RN 0.76 will go with Int32 -> Long.

[mfazekas]

So it looks like codegen is fixed (aka reverted back to java Double) in RN 0.74.3 - see #3570.