useStableCallback causing troubles in New Architecture

[pavel-krasnov]

Hi, I am developing a New Architecture app and trying to use this library. I noticed that when I use the BottomSheetScrollView component, the onContentSizeChange callback is not called. A short investigation led me to how this callback is passed to the underlying React Native ScrollView component. It is done using the useStableCallback hook, which is intended to not cause rerenders when the callback reference changes. Also I noticed the comment which was left two years after @gorhom had implemented the hook in this library saying that you probably don't need to use useEffect to update the callback reference. This appers to be so true - looks like by the time the useEffect is called, the New Architecture version of the onContentSizeChange has been already called and thus the callback, which was undefined in the time of calling, couldn't be called. I replaced useEffect with simply settings the ref value in the hook body and everything started to work.

So I would like to ask you for an opinion if such a change can be introduced to the library any time soon.