fix(bottomSheet): snapPoint type

[nicolascavallin]

Motivation

The motivation for making this change is to fix THIS fix when you update to the latest version of react-native-reanimated

[github-actions]

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 10 days.

[exzos28]

still need

[nicolascavallin]

@gorhom is there any chance to check it or consider it for an update?

[gorhom]

thanks @nicolascavallin for submitting this PR <3

[andrei-lt]

Thanks for the PR @nicolascavallin.

I still found some typecheck issues when directly using animatedSnapPoints from the result of useBottomSheetDynamicSnapPoints as a value for snapPoints.

I think it's because when react-native-bottom-sheet is built, it still uses reanimated v2 as a dependency.
When it generates the typescript declaration files, it generates something like this:

export declare const useBottomSheetDynamicSnapPoints: (initialSnapPoints: Array<string | number>) => {
    animatedSnapPoints: Readonly<{
        value: (string | number)[];
    }>;
...

A SharedValue object gets replaced with just { value: ... } as a definition and it doesn't match what the new SharedValue in reanimated v3 is defined as. The change happened in reanimated #3722.

TLDR is snapPoints prop expects a full shared value object, but useBottomSheetDynamicSnapPoints declared that it returns a partial object only.

Have I missed something somewhere and this is actually just fine for others?