react-native-rectangle-doc-scanner 3.131.0 → 3.132.0

package/android/src/main/kotlin/com/reactnativerectangledocscanner/DocumentScannerModule.kt

@@ -43,18 +43,11 @@ class DocumentScannerModule(reactContext: ReactApplicationContext) :
                     val view = uiManager.resolveView(tag)
 
                     if (view is DocumentScannerView) {
-                        Log.d(TAG, "Found DocumentScannerView, triggering capture")
-
-                        // Store promise to be resolved when capture completes
-                        // For simplicity, we'll trigger the capture which will emit the event
-                        view.capture()
-
-                        // Note: In the current implementation, we use events (onPictureTaken)
-                        // iOS also uses events for the main flow, but has a promise-based method too
-                        // For consistency with the event-based approach, resolve immediately
-                        promise.resolve(Arguments.createMap().apply {
-                            putString("status", "capturing")
-                        })
+                        Log.d(TAG, "Found DocumentScannerView, triggering capture with promise")
+
+                        // Pass promise to view so it can be resolved when capture completes
+                        // This matches iOS behavior where promise is resolved with actual image data
+                        view.captureWithPromise(promise)
                     } else {
                         Log.e(TAG, "View with tag $tag is not DocumentScannerView: ${view?.javaClass?.simpleName}")
                         promise.reject("INVALID_VIEW", "View is not a DocumentScannerView")

package/android/src/main/kotlin/com/reactnativerectangledocscanner/DocumentScannerView.kt

@@ -153,13 +153,22 @@ class DocumentScannerView(context: ThemedReactContext) : FrameLayout(context) {
     }
 
     fun capture() {
+        captureWithPromise(null)
+    }
+
+    /**
+     * Capture image with promise support (matches iOS captureWithResolver:rejecter:)
+     * @param promise Optional promise to resolve with capture result
+     */
+    fun captureWithPromise(promise: com.facebook.react.bridge.Promise?) {
         if (isCapturing) {
             Log.d(TAG, "Already capturing, ignoring request")
+            promise?.reject("CAPTURE_IN_PROGRESS", "Capture already in progress")
             return
         }
 
         isCapturing = true
-        Log.d(TAG, "Capture initiated")
+        Log.d(TAG, "Capture initiated with promise: ${promise != null}")
 
         val outputDir = if (saveInAppDocument) {
             context.filesDir
@@ -171,18 +180,23 @@ class DocumentScannerView(context: ThemedReactContext) : FrameLayout(context) {
             outputDirectory = outputDir,
             onImageCaptured = { file ->
                 scope.launch {
-                    processAndEmitImage(file)
+                    processAndEmitImage(file, promise)
                 }
             },
             onError = { exception ->
                 Log.e(TAG, "Capture failed", exception)
                 isCapturing = false
+
+                // Reject promise if provided
+                promise?.reject("CAPTURE_FAILED", "Failed to capture image", exception)
+
+                // Also send event for backwards compatibility
                 sendErrorEvent("capture_failed")
             }
         )
     }
 
-    private suspend fun processAndEmitImage(imageFile: File) = withContext(Dispatchers.IO) {
+    private suspend fun processAndEmitImage(imageFile: File, promise: com.facebook.react.bridge.Promise? = null) = withContext(Dispatchers.IO) {
         try {
             // Detect rectangle in captured image
             val bitmap = BitmapFactory.decodeFile(imageFile.absolutePath)
@@ -229,7 +243,14 @@ class DocumentScannerView(context: ThemedReactContext) : FrameLayout(context) {
             }
 
             withContext(Dispatchers.Main) {
+                Log.d(TAG, "Processing completed, resolving promise: ${promise != null}")
+
+                // Resolve promise first (if provided) - matches iOS behavior
+                promise?.resolve(result)
+
+                // Then send event for backwards compatibility
                 sendPictureTakenEvent(result)
+
                 isCapturing = false
 
                 if (!captureMultiple) {
@@ -239,6 +260,10 @@ class DocumentScannerView(context: ThemedReactContext) : FrameLayout(context) {
         } catch (e: Exception) {
             Log.e(TAG, "Failed to process image", e)
             withContext(Dispatchers.Main) {
+                // Reject promise if provided
+                promise?.reject("PROCESSING_FAILED", "Failed to process image: ${e.message}", e)
+
+                // Also send error event for backwards compatibility
                 sendErrorEvent("processing_failed")
                 isCapturing = false
             }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-native-rectangle-doc-scanner",
-  "version": "3.131.0",
+  "version": "3.132.0",
   "description": "Native-backed document scanner for React Native with customizable overlays.",
   "license": "MIT",
   "main": "dist/index.js",