react-native-pdf-jsi 3.3.1 → 3.4.2

package/README.md

@@ -8,7 +8,7 @@
 
 ---
 
-## ⚡ Performance Benchmarks (v3.3.0)
+## ⚡ Performance Benchmarks (v3.4.0)
 
 **World-class performance proven with real-world testing:**
 
@@ -87,6 +87,27 @@ Starting November 1, 2025, Google Play will require apps to support 16KB page si
 - ✅ **Google Play Approved** - Meets all current and future requirements  
 - ✅ **Drop-in Replacement** - Easy migration from existing libraries
 
+## 🎉 Version 3.4.0 - Enhanced Navigation & File Path Handling!
+
+**Major improvements to navigation reliability and file path tracking for better bookmarking, export, and PDF operations!**
+
+### 🚀 **What's New in v3.4.0:**
+
+#### ✨ **New Features**
+- **🎯 Enhanced Navigation** - Immediate page navigation support in `setPage()` method. PDF now jumps to the specified page instantly when navigation is triggered programmatically
+- **📁 Improved File Path Handling** - Added `downloadedFilePath` instance variable for reliable path tracking during PDF loading. Paths are available immediately, even before React state updates
+- **🔍 New `getPath()` Method** - Public method to retrieve the current PDF file path. Returns the most reliable path source (instance variable > state path)
+- **📡 Enhanced Path Extraction** - Improved `onLoadComplete` callback to receive file path directly from native module, ensuring reliable path availability for bookmarking, export, and PDF operations
+
+#### 🐛 **Bug Fixes**
+- **🧭 Navigation Reliability** - Fixed issue where programmatic page navigation would not scroll to the target page. Native `setPage()` method now calls `jumpTo()` immediately when page changes
+- **📄 File Path Availability** - Fixed issue where `pdfFilePath` was sometimes empty in `onLoadComplete`, causing failures in export and PDF operations. Path is now reliably extracted from multiple sources with proper fallbacks
+- **🔧 Native Event Handling** - Enhanced `loadComplete` event message format to include file path, improving reliability of path extraction in JavaScript
+
+#### 🔄 **Technical Improvements**
+- **Native Message Format** - Updated `loadComplete` event format from `loadComplete|pages|width|height|tableContents` to `loadComplete|pages|width|height|path|tableContents` with automatic detection of old/new formats for backward compatibility
+- **Path Tracking** - File paths are now stored in an instance variable (`downloadedFilePath`) immediately upon file preparation/download completion, in addition to React state, ensuring synchronous access
+
 ## 🎉 Version 3.0.0 - Major Release with Complete Feature Sync!
 
 **Complete synchronization of all features from development package with enhanced Android capabilities!**
@@ -432,13 +453,13 @@ Register the JSI package in your React Native application:
 
 ```java
 // MainApplication.java
-import org.wonday.pdf.RNPDFPackage;
+import org.wonday.pdf.RNPDFJSIPackage;
 
 @Override
 protected List<ReactPackage> getPackages() {
     return Arrays.<ReactPackage>asList(
         new MainReactPackage(),
-        new RNPDFPackage() // This includes JSI modules
+        new RNPDFJSIPackage() // This includes JSI modules
     );
 }
 ```
@@ -1028,7 +1049,7 @@ private static final String FOLDER_NAME = "YourAppName"; // Change this
 -keep class org.wonday.pdf.PDFJSIManager { *; }
 -keep class org.wonday.pdf.PDFJSIModule { *; }
 -keep class org.wonday.pdf.EnhancedPdfJSIBridge { *; }
--keep class org.wonday.pdf.RNPDFPackage { *; }
+-keep class org.wonday.pdf.RNPDFJSIPackage { *; }
 -keep class org.wonday.pdf.PdfManager { *; }
 -keep class org.wonday.pdf.PDFNativeCacheManager { *; }
 -keep class org.wonday.pdf.PdfView { *; }
@@ -1252,7 +1273,25 @@ const Pdf = PdfModule.default;
 
 ## 📝 Changelog
 
-### v3.3.0 (2025-11-12) - Latest ✅ STREAMING BASE64 & CACHE MANAGER
+### v3.4.0 (2025-11-29) - Latest ✅ ENHANCED NAVIGATION & FILE PATH HANDLING
+
+#### ✨ **New Features**
+- **Enhanced Navigation** - Added immediate page navigation support in `setPage()` method. PDF now jumps to the specified page instantly when navigation is triggered programmatically
+- **Improved File Path Handling** - Added `downloadedFilePath` instance variable for reliable path tracking during PDF loading
+- **New `getPath()` Method** - Added public method to retrieve the current PDF file path with multiple fallback sources
+- **Enhanced Path Extraction** - Improved `onLoadComplete` callback to receive file path directly from native module
+
+#### 🐛 **Bug Fixes**
+- **Navigation Reliability** - Fixed programmatic page navigation not scrolling to target page
+- **File Path Availability** - Fixed empty `pdfFilePath` in `onLoadComplete` causing export/operation failures
+- **Native Event Handling** - Enhanced `loadComplete` event message format to include file path
+
+#### 🔄 **Technical Changes**
+- Native `loadComplete` event now includes file path for improved reliability
+- Backward compatible with old message format (auto-detects)
+- Path stored in instance variable immediately upon file preparation
+
+### v3.3.0 (2025-11-12) - STREAMING BASE64 & CACHE MANAGER
 
 #### 🚀 **Major Features**
 
@@ -1969,7 +2008,7 @@ For bug reports, include:
 
 *Transform your PDF viewing experience with enterprise-grade performance and reliability.*
 
-**v2.2.7 - iOS Codegen Fix & New Architecture Support**  
+**v3.4.0 - Enhanced Navigation & File Path Handling**  
 **Copyright (c) 2025-present, Punith M (punithm300@gmail.com). Enhanced PDF JSI Integration. All rights reserved.**
 
 *Original work Copyright (c) 2017-present, Wonday (@wonday.org). All rights reserved.*

package/android/src/main/java/org/wonday/pdf/FileManager.java

@@ -12,6 +12,8 @@ import com.facebook.react.bridge.ReactMethod;
 import com.facebook.react.bridge.WritableMap;
 import com.facebook.react.bridge.Arguments;
 
+import java.io.File;
+
 /**
  * FileManager - Native module for file operations like opening folders
  */

package/android/src/main/java/org/wonday/pdf/MemoryMappedCache.java

@@ -31,7 +31,7 @@ public class MemoryMappedCache {
     // Statistics
     private int totalMaps = 0;
     private int totalUnmaps = 0;
-    private long totalBytesM apped = 0;
+    private long totalBytesMapped = 0;
     
     /**
      * Memory-map a PDF file for zero-copy access

package/android/src/main/java/org/wonday/pdf/PDFExporter.java

@@ -31,13 +31,11 @@ import android.graphics.pdf.PdfDocument;
 
 public class PDFExporter extends ReactContextBaseJavaModule {
     private static final String TAG = "PDFExporter";
-    private LicenseVerifier licenseVerifier;
     // OPTIMIZATION: Bitmap pool for 90% reduction in bitmap allocations
     private final BitmapPool bitmapPool = new BitmapPool();
 
     public PDFExporter(ReactApplicationContext reactContext) {
         super(reactContext);
-        licenseVerifier = new LicenseVerifier(reactContext);
     }
 
     @Override
@@ -69,11 +67,6 @@ public class PDFExporter extends ReactContextBaseJavaModule {
     @ReactMethod
     public void exportToImages(String filePath, ReadableMap options, Promise promise) {
         try {
-            if (!licenseVerifier.isProActive()) {
-                promise.reject("LICENSE_REQUIRED", "Export to Images requires a Pro license");
-                return;
-            }
-
             if (filePath == null || filePath.isEmpty()) {
                 promise.reject("INVALID_PATH", "File path is required");
                 return;
@@ -113,15 +106,6 @@ public class PDFExporter extends ReactContextBaseJavaModule {
             double scale = options.hasKey("scale") ? options.getDouble("scale") : 2.0;
             
             Log.i(TAG, "🖼️ [EXPORT] exportPageToImage - START - page: " + pageIndex + ", format: " + format + ", quality: " + quality + ", scale: " + scale);
-            
-            boolean licenseActive = licenseVerifier.isProActive();
-            Log.i(TAG, "🔑 [LICENSE] isProActive: " + licenseActive);
-            
-            if (!licenseActive) {
-                Log.e(TAG, "❌ [EXPORT] exportPageToImage - FAILED - License required");
-                promise.reject("LICENSE_REQUIRED", "Export to Images requires a Pro license");
-                return;
-            }
 
             if (filePath == null || filePath.isEmpty()) {
                 Log.e(TAG, "❌ [EXPORT] exportPageToImage - FAILED - Invalid path");
@@ -339,12 +323,6 @@ public class PDFExporter extends ReactContextBaseJavaModule {
     @ReactMethod
     public void mergePDFs(ReadableArray filePaths, String outputPath, Promise promise) {
         try {
-            // Check Pro license
-            if (!licenseVerifier.isProActive()) {
-                promise.reject("LICENSE_REQUIRED", "PDF Operations requires a Pro license");
-                return;
-            }
-
             if (filePaths == null || filePaths.size() < 2) {
                 promise.reject("INVALID_INPUT", "At least 2 PDF files are required for merging");
                 return;
@@ -478,16 +456,6 @@ public class PDFExporter extends ReactContextBaseJavaModule {
             
             // Original code continues...
             Log.i(TAG, "✂️ [SPLIT] splitPDF - START - file: " + filePath + ", ranges: " + pageRanges.size());
-            
-            // Check Pro license
-            boolean licenseActive = licenseVerifier.isProActive();
-            Log.i(TAG, "🔑 [LICENSE] isProActive: " + licenseActive);
-            
-            if (!licenseActive) {
-                Log.e(TAG, "❌ [SPLIT] License required");
-                promise.reject("LICENSE_REQUIRED", "PDF Operations requires a Pro license");
-                return;
-            }
 
             if (filePath == null || filePath.isEmpty()) {
                 Log.e(TAG, "❌ [SPLIT] Invalid path");
@@ -607,16 +575,6 @@ public class PDFExporter extends ReactContextBaseJavaModule {
     public void extractPages(String filePath, ReadableArray pageNumbers, String outputPath, Promise promise) {
         try {
             Log.i(TAG, "✂️ [EXTRACT] extractPages - START - file: " + filePath + ", pages: " + pageNumbers.size());
-            
-            // Check Pro license
-            boolean licenseActive = licenseVerifier.isProActive();
-            Log.i(TAG, "🔑 [LICENSE] isProActive: " + licenseActive);
-            
-            if (!licenseActive) {
-                Log.e(TAG, "❌ [EXTRACT] License required");
-                promise.reject("LICENSE_REQUIRED", "PDF Operations requires a Pro license");
-                return;
-            }
 
             if (filePath == null || filePath.isEmpty()) {
                 Log.e(TAG, "❌ [EXTRACT] Invalid path");
@@ -717,12 +675,6 @@ public class PDFExporter extends ReactContextBaseJavaModule {
     @ReactMethod
     public void rotatePage(String filePath, int pageNumber, int degrees, Promise promise) {
         try {
-            // Check Pro license
-            if (!licenseVerifier.isProActive()) {
-                promise.reject("LICENSE_REQUIRED", "PDF Operations requires a Pro license");
-                return;
-            }
-
             if (filePath == null || filePath.isEmpty()) {
                 promise.reject("INVALID_PATH", "File path is required");
                 return;
@@ -740,12 +692,6 @@ public class PDFExporter extends ReactContextBaseJavaModule {
     @ReactMethod
     public void deletePage(String filePath, int pageNumber, Promise promise) {
         try {
-            // Check Pro license
-            if (!licenseVerifier.isProActive()) {
-                promise.reject("LICENSE_REQUIRED", "PDF Operations requires a Pro license");
-                return;
-            }
-
             if (filePath == null || filePath.isEmpty()) {
                 promise.reject("INVALID_PATH", "File path is required");
                 return;

package/android/src/main/java/org/wonday/pdf/PdfManager.java

@@ -144,6 +144,11 @@ public class PdfManager extends SimpleViewManager<PdfView> implements RNPDFPdfVi
         // NOOP on Android
     }
 
+    @Override
+    public void setEnableMomentum(PdfView view, boolean value) {
+        // NOOP on Android - momentum scrolling is handled automatically by Android's ScrollView
+    }
+
     @ReactProp(name = "fitPolicy")
     public void setFitPolicy(PdfView pdfView, int fitPolicy) {
         pdfView.setFitPolicy(fitPolicy);

package/android/src/main/java/org/wonday/pdf/PdfView.java

@@ -154,7 +154,9 @@ public class PdfView extends PDFView implements OnPageChangeListener,OnLoadCompl
 
         //create a new json Object for the TableOfContents
         Gson gson = new Gson();
-        event.putString("message", "loadComplete|"+numberOfPages+"|"+width+"|"+height+"|"+gson.toJson(this.getTableOfContents()));
+        // Include path in loadComplete message for reliable access in JS
+        String pathValue = this.path != null ? this.path : "";
+        event.putString("message", "loadComplete|"+numberOfPages+"|"+width+"|"+height+"|"+pathValue+"|"+gson.toJson(this.getTableOfContents()));
 
         ThemedReactContext context = (ThemedReactContext) getContext();
         EventDispatcher dispatcher = UIManagerHelper.getEventDispatcherForReactTag(context, getId());
@@ -369,7 +371,15 @@ public class PdfView extends PDFView implements OnPageChangeListener,OnLoadCompl
 
     // page start from 1
     public void setPage(int page) {
-        this.page = page>1?page:1;
+        int newPage = page>1?page:1;
+        int oldPage = this.page;
+        this.page = newPage;
+        
+        // If page changed and PDF is already loaded, jump to the new page immediately
+        if (newPage != oldPage && !needsReload && this.path != null && !this.isRecycled()) {
+            showLog(format("setPage: Jumping to page %d (from %d)", newPage, oldPage));
+            this.jumpTo(newPage - 1, false);
+        }
     }
 
     public void setScale(float scale) {

package/android/src/main/java/org/wonday/pdf/RNPDFJSIPackage.java

@@ -0,0 +1,50 @@
+/**
+ * Copyright (c) 2017-present, Wonday (@wonday.org)
+ * All rights reserved.
+ *
+ * This source code is licensed under the MIT-style license found in the
+ * LICENSE file in the root directory of this source tree.
+ */
+
+package org.wonday.pdf;
+
+import java.util.Collections;
+import java.util.List;
+import java.util.ArrayList;
+
+import com.facebook.react.ReactPackage;
+import com.facebook.react.bridge.JavaScriptModule;
+import com.facebook.react.bridge.NativeModule;
+import com.facebook.react.bridge.ReactApplicationContext;
+import com.facebook.react.uimanager.ViewManager;
+
+public class RNPDFJSIPackage implements ReactPackage {
+
+    @Override
+    public List<NativeModule> createNativeModules(ReactApplicationContext reactContext) {
+        List<NativeModule> modules = new ArrayList<>();
+        // Add JSI modules for enhanced PDF performance
+        modules.add(new PDFJSIManager(reactContext));
+        modules.add(new EnhancedPdfJSIBridge(reactContext));
+        
+        // Add advanced feature modules
+        modules.add(new PDFExporter(reactContext));
+        modules.add(new FileDownloader(reactContext));
+        modules.add(new FileManager(reactContext));
+        
+        return modules;
+    }
+
+    // Deprecated as of RN 0.47.0
+    public List<Class<? extends JavaScriptModule>> createJSModules() {
+        return Collections.emptyList();
+    }
+
+    @Override
+    public List<ViewManager> createViewManagers(ReactApplicationContext reactContext) {
+        List<ViewManager> modules = new ArrayList<>();
+        modules.add(new PdfManager(reactContext));
+        return modules;
+    }
+}
+

package/android/src/main/java/org/wonday/pdf/RNPDFPackage.java

@@ -31,7 +31,6 @@ public class RNPDFPackage implements ReactPackage {
         modules.add(new PDFExporter(reactContext));
         modules.add(new FileDownloader(reactContext));
         modules.add(new FileManager(reactContext));
-        modules.add(new LicenseVerifier(reactContext));
         
         return modules;
     }

package/fabric/RNPDFPdfNativeComponent.js

@@ -4,7 +4,11 @@
  */
  'use strict';
 
+ // Note: Deep imports are required for codegen components
+ // Suppressing deprecation warnings as these are the correct imports for codegen
+ // @ts-ignore - React Native codegen requires these deep imports
  import codegenNativeComponent from 'react-native/Libraries/Utilities/codegenNativeComponent';
+ // @ts-ignore - React Native codegen requires these deep imports
  import codegenNativeCommands from 'react-native/Libraries/Utilities/codegenNativeCommands';
  
  type ChangeEvent = $ReadOnly<{|
@@ -24,9 +28,10 @@
    enableAnnotationRendering: ?boolean,
    showsHorizontalScrollIndicator: ?boolean,
    showsVerticalScrollIndicator: ?boolean,
-   scrollEnabled: ?boolean,
-   enableAntialiasing: ?boolean,
-   enableDoubleTapZoom: ?boolean,
+  scrollEnabled: ?boolean,
+  enableMomentum: ?boolean,
+  enableAntialiasing: ?boolean,
+  enableDoubleTapZoom: ?boolean,
    fitPolicy: ?Int32,
    spacing: ?Int32,
    password: ?string,

package/index.js

@@ -20,9 +20,9 @@ import {
     Text,
     requireNativeComponent
 } from 'react-native';
-import PdfViewNativeComponent, {
-    Commands as PdfViewCommands,
-  } from './fabric/RNPDFPdfNativeComponent';
+// Codegen component variables - will be loaded lazily to prevent hooks errors
+let PdfViewNativeComponent = null;
+let PdfViewCommands = null;
 import ReactNativeBlobUtil from 'react-native-blob-util'
 import {ViewPropTypes} from 'deprecated-react-native-prop-types';
 const SHA1 = require('crypto-js/sha1');
@@ -94,7 +94,7 @@ export default class Pdf extends Component {
         scrollEnabled: true,
         enablePaging: false,
         enableRTL: false,
-        trustAllCerts: true,
+        trustAllCerts: false,
         usePDFKit: true,
         singlePage: false,
         onLoadProgress: (percent) => {
@@ -123,6 +123,10 @@ export default class Pdf extends Component {
             jsiAvailable: false,
         };
 
+        // Store downloaded file path in instance variable for immediate access
+        // This ensures path is available when onLoadComplete fires, even before state updates
+        this.downloadedFilePath = '';
+        
         this.lastRNBFTask = null;
         this.pdfJSI = PDFJSI;
         this.initializeJSI();
@@ -184,6 +188,7 @@ export default class Pdf extends Component {
 
         let uri = source.uri || '';
         // first set to initial state
+        this.downloadedFilePath = ''; // Reset instance variable
         if (this._mounted) {
             this.setState({isDownloaded: false, path: '', progress: 0});
         }
@@ -195,6 +200,8 @@ export default class Pdf extends Component {
                 .stat(cacheFile)
                 .then(stats => {
                     if (!Boolean(source.expiration) || (source.expiration * 1000 + stats.lastModified) > (new Date().getTime())) {
+                        // Store in instance variable immediately for onLoadComplete callback
+                        this.downloadedFilePath = cacheFile;
                         if (this._mounted) {
                             this.setState({path: cacheFile, isDownloaded: true});
                         }
@@ -234,6 +241,8 @@ export default class Pdf extends Component {
                     ReactNativeBlobUtil.fs
                         .cp(uri, cacheFile)
                         .then(() => {
+                            // Store in instance variable immediately for onLoadComplete callback
+                            this.downloadedFilePath = cacheFile;
                             if (this._mounted) {
                                 this.setState({path: cacheFile, isDownloaded: true, progress: 1});
                             }
@@ -247,6 +256,8 @@ export default class Pdf extends Component {
                     ReactNativeBlobUtil.fs
                         .writeFile(cacheFile, data, 'base64')
                         .then(() => {
+                            // Store in instance variable immediately for onLoadComplete callback
+                            this.downloadedFilePath = cacheFile;
                             if (this._mounted) {
                                 this.setState({path: cacheFile, isDownloaded: true, progress: 1});
                             }
@@ -256,9 +267,13 @@ export default class Pdf extends Component {
                             this._onError(error)
                         });
                 } else {
+                    // Local file path
+                    const localPath = decodeURIComponent(uri.replace(/file:\/\//i, ''));
+                    // Store in instance variable immediately for onLoadComplete callback
+                    this.downloadedFilePath = localPath;
                     if (this._mounted) {
                        this.setState({
-                            path: decodeURIComponent(uri.replace(/file:\/\//i, '')),
+                            path: localPath,
                             isDownloaded: true,
                         });
                     }
@@ -284,11 +299,32 @@ export default class Pdf extends Component {
         const tempCacheFile = cacheFile + '.tmp';
         this._unlinkFile(tempCacheFile);
 
-        this.lastRNBFTask = ReactNativeBlobUtil.config({
-            // response data will be saved to this path if it has access right.
+        // Ensure cache directory exists before downloading
+        const cacheDir = ReactNativeBlobUtil.fs.dirs.CacheDir;
+        try {
+            const dirExists = await ReactNativeBlobUtil.fs.exists(cacheDir);
+            if (!dirExists) {
+                await ReactNativeBlobUtil.fs.mkdir(cacheDir);
+            }
+        } catch (error) {
+            console.warn('Failed to ensure cache directory exists:', error);
+            // Continue anyway - ReactNativeBlobUtil might handle it
+        }
+
+        // Build config object - conditionally include trusty based on URL protocol and trustAllCerts
+        const isHttps = source.uri && source.uri.startsWith('https://');
+        const config = {
             path: tempCacheFile,
-            trusty: this.props.trustAllCerts,
-        })
+        };
+        
+        // Only include trusty option for HTTPS URLs and only if trustAllCerts is explicitly true
+        // For HTTP URLs, never include trusty option to avoid trust manager errors
+        if (isHttps && this.props.trustAllCerts === true) {
+            config.trusty = true;
+        }
+        // For HTTP or when trustAllCerts is false, omit trusty option entirely
+
+        this.lastRNBFTask = ReactNativeBlobUtil.config(config)
             .fetch(
                 source.method ? source.method : 'GET',
                 source.uri,
@@ -336,6 +372,9 @@ export default class Pdf extends Component {
                 ReactNativeBlobUtil.fs
                     .cp(tempCacheFile, cacheFile)
                     .then(() => {
+                        // Store in instance variable immediately for onLoadComplete callback
+                        // This ensures path is available even if state hasn't updated yet
+                        this.downloadedFilePath = cacheFile;
                         if (this._mounted) {
                             this.setState({path: cacheFile, isDownloaded: true, progress: 1});
                         }
@@ -367,6 +406,12 @@ export default class Pdf extends Component {
         }
     };
 
+    // Public method to get the current PDF file path
+    getPath() {
+        // Return instance variable first (most reliable), then state path
+        return this.downloadedFilePath || this.state.path || '';
+    }
+
     setPage( pageNumber ) {
         if ( (pageNumber === null) || (isNaN(pageNumber)) ) {
             throw new Error('Specified pageNumber is not a number');
@@ -385,6 +430,16 @@ export default class Pdf extends Component {
         
         if (!!global?.nativeFabricUIManager ) {
             if (this._root) {
+                // Lazy load PdfViewCommands if not already loaded
+                if (!PdfViewCommands) {
+                    try {
+                        const codegenModule = require('./fabric/RNPDFPdfNativeComponent');
+                        PdfViewCommands = codegenModule.Commands;
+                    } catch (error) {
+                        console.warn('PdfViewCommands not available:', error);
+                        return;
+                    }
+                }
                 PdfViewCommands.setNativePage(
                     this._root,
                     pageNumber,
@@ -494,22 +549,79 @@ export default class Pdf extends Component {
         let message = event.nativeEvent.message.split('|');
         //__DEV__ && console.log("onChange: " + message);
         if (message.length > 0) {
-            if (message.length > 5) {
-                message[4] = message.splice(4).join('|');
-            }
             if (message[0] === 'loadComplete') {
                 let tableContents;
-                try {
-                    tableContents = message[4]&&JSON.parse(message[4]);
-                } catch(e) {
-                    tableContents = message[4];
+                let filePath;
+                
+                // Handle both old format (without path) and new format (with path)
+                // Old format: loadComplete|pages|width|height|tableContents
+                // New format: loadComplete|pages|width|height|path|tableContents
+                
+                // First, check if we have the new format (6+ parts before splice)
+                const originalLength = message.length;
+                const hasPath = originalLength >= 6;
+                
+                if (hasPath) {
+                    // New format: extract path from message[4], rest is tableContents
+                    filePath = message[4] || '';
+                    // Join everything after path (index 5+) as tableContents JSON
+                    const tableContentsStr = message.slice(5).join('|');
+                    try {
+                        tableContents = tableContentsStr && JSON.parse(tableContentsStr);
+                    } catch(e) {
+                        tableContents = tableContentsStr;
+                    }
+                } else {
+                    // Old format: no path, everything after height (index 4+) is tableContents
+                    filePath = this.downloadedFilePath || this.state.path || '';
+                    // Handle old splice logic for tableContents that might contain |
+                    if (originalLength > 5) {
+                        message[4] = message.splice(4).join('|');
+                    }
+                    try {
+                        tableContents = message[4] && JSON.parse(message[4]);
+                    } catch(e) {
+                        tableContents = message[4];
+                    }
                 }
-                this.props.onLoadComplete && this.props.onLoadComplete(Number(message[1]), this.state.path, {
-                    width: Number(message[2]),
-                    height: Number(message[3]),
-                },
-                tableContents
-                );
+                
+                // Final fallback: use instance variable or state if path from native is empty
+                if (!filePath || filePath.trim() === '') {
+                    filePath = this.downloadedFilePath || this.state.path || '';
+                }
+                
+                // Log path extraction for debugging
+                if (__DEV__) {
+                    console.log('📁 [Pdf] loadComplete - Path extraction:', {
+                        originalMessageLength: originalLength,
+                        hasPathInMessage: hasPath,
+                        fromNativeMessage: hasPath ? (message[4] || 'empty') : 'not in message',
+                        fromInstance: this.downloadedFilePath || 'empty',
+                        fromState: this.state.path || 'empty',
+                        final: filePath || 'empty',
+                    });
+                }
+                
+                // Always call onLoadComplete callback
+                if (this.props.onLoadComplete) {
+                    console.log('📁 [Pdf] Calling onLoadComplete callback with:', {
+                        pages: Number(message[1]),
+                        path: filePath,
+                        width: Number(message[2]),
+                        height: Number(message[3]),
+                    });
+                    this.props.onLoadComplete(Number(message[1]), filePath, {
+                        width: Number(message[2]),
+                        height: Number(message[3]),
+                    },
+                    tableContents
+                    );
+                } else {
+                    console.warn('⚠️ [Pdf] onLoadComplete callback not provided');
+                }
+            } else if (message.length > 5) {
+                // Only apply splice logic for non-loadComplete messages
+                message[4] = message.splice(4).join('|');
             } else if (message[0] === 'pageChanged') {
                 this.props.onPageChanged && this.props.onPageChanged(Number(message[1]), Number(message[2]));
             } else if (message[0] === 'error') {
@@ -583,7 +695,20 @@ export default class Pdf extends Component {
 }
 
 if (Platform.OS === "android" || Platform.OS === "ios") {
-    var PdfCustom = PdfViewNativeComponent;
+    // Load codegen component immediately - it should work after React is initialized
+    try {
+        const codegenModule = require('./fabric/RNPDFPdfNativeComponent');
+        const CodegenComponent = codegenModule.default;
+        PdfViewNativeComponent = CodegenComponent;
+        PdfViewCommands = codegenModule.Commands;
+        var PdfCustom = CodegenComponent;
+    } catch (error) {
+        console.warn('Failed to load codegen component, using fallback:', error);
+        // Use the correct native component name for Android/iOS
+        var PdfCustom = requireNativeComponent('RNPDFPdfView', Pdf, {
+            nativeOnly: {path: true, onChange: true},
+        });
+    }
 }  else if (Platform.OS === "windows") {
     var PdfCustom = requireNativeComponent('RCTPdf', Pdf, {
         nativeOnly: {path: true, onChange: true},

package/ios/RNPDFPdf/RNPDFPdfView.h

@@ -19,6 +19,24 @@
 
 #ifdef RCT_NEW_ARCH_ENABLED
 #import <React/RCTViewComponentView.h>
+#import <React/RCTComponentViewProtocol.h>
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+
+// Forward declaration for codegen - ensures RNPDFPdfViewCls() is visible to RCTThirdPartyComponentsProvider
+// Using extern "C" for proper C linkage, matching React Native's pattern
+Class<RCTComponentViewProtocol> RNPDFPdfViewCls(void);
+
+// Alias function based on codegen name "rnpdf" - ensures codegen can find the function
+// even if it uses the codegen name instead of componentProvider name
+Class<RCTComponentViewProtocol> rnpdfCls(void);
+
+#ifdef __cplusplus
+}
+#endif
+
 #endif
 
 @class RCTEventDispatcher;
@@ -29,7 +47,7 @@ RCTViewComponentView
 #else
 UIView
 #endif
-<UIGestureRecognizerDelegate>
+<UIGestureRecognizerDelegate, UIScrollViewDelegate>
 - (instancetype)initWithBridge:(RCTBridge *)bridge;
 
 @property(nonatomic, strong) NSString *path;