@umituz/react-native-bottom-sheet 1.2.4 → 1.2.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@umituz/react-native-bottom-sheet",
-  "version": "1.2.4",
+  "version": "1.2.5",
   "description": "Modern, performant bottom sheets for React Native with preset configurations, keyboard handling, and smooth animations",
   "main": "./src/index.ts",
   "types": "./src/index.ts",

package/src/presentation/components/BottomSheet.tsx

@@ -155,13 +155,7 @@ export const BottomSheet = forwardRef<BottomSheetRef, BottomSheetProps>(
   ) => {
     const tokens = useAppDesignTokens();
     const sheetRef = React.useRef<GorhomBottomSheet>(null);
-    // CRITICAL: Use centralized Reanimated ready check from animation package
-    // This must be true before GorhomBottomSheet is rendered
-    // Otherwise, internal hooks (useAnimatedLayout) will access containerLayoutState.get
-    // before Reanimated is ready, causing "containerLayoutState.get is not a function" errors
-    // 
-    // MINIMAL FIX: Don't render GorhomBottomSheet at all until Reanimated is ready
-    // This prevents internal hooks from running before Reanimated is initialized
+    // Don't render until Reanimated is ready to prevent layoutState.get errors
     const isReanimatedReady = useReanimatedReady();
 
     // Get configuration from preset or custom (must be before useImperativeHandle)
@@ -220,48 +214,31 @@ export const BottomSheet = forwardRef<BottomSheetRef, BottomSheetProps>(
     );
 
     // Expose ref methods (must be before early return to maintain hook order)
+    // No need to check isReanimatedReady here - component won't render if not ready
     React.useImperativeHandle(ref, () => ({
       snapToIndex: (index: number) => {
-        if (isReanimatedReady && sheetRef.current) {
-          sheetRef.current.snapToIndex(index);
-        }
+        sheetRef.current?.snapToIndex(index);
       },
       snapToPosition: (position: string | number) => {
-        if (isReanimatedReady && sheetRef.current) {
-          sheetRef.current.snapToPosition(position);
-        }
+        sheetRef.current?.snapToPosition(position);
       },
       expand: () => {
-        if (isReanimatedReady && sheetRef.current) {
-          sheetRef.current.expand();
-        }
+        sheetRef.current?.expand();
       },
       collapse: () => {
-        if (isReanimatedReady && sheetRef.current) {
-          sheetRef.current.collapse();
-        }
+        sheetRef.current?.collapse();
       },
       close: () => {
-        if (isReanimatedReady && sheetRef.current) {
-          sheetRef.current.close();
-        }
+        sheetRef.current?.close();
       },
-    }));
+    }), []);
 
-    // MINIMAL FIX: Don't render GorhomBottomSheet at all until Reanimated is fully ready
-    // This is the root cause fix - if we don't render GorhomBottomSheet, its internal hooks
-    // (useAnimatedLayout, useAnimatedDetents, etc.) won't run, preventing the error
-    // 
-    // GorhomBottomSheet's internal hooks access Reanimated's containerLayoutState.get during initialization
-    // If we render before Reanimated is ready, we get "containerLayoutState.get is not a function" errors
-    // 
-    // IMPORTANT: All hooks must be called before this early return to maintain hook order
+    // Don't render until Reanimated is ready
     if (!isReanimatedReady) {
       return null;
     }
 
-    // Double-check: Ensure we have a valid config before rendering
-    // This prevents errors if snapPoints are invalid
+    // Ensure valid config
     if (!config.snapPoints || config.snapPoints.length === 0) {
       return null;
     }