@finos_sdk/sdk-ekyc 1.4.9 → 1.5.0

package/android/src/main/java/finos/sdk/ekyc/EKYCModule.kt

@@ -92,97 +92,154 @@ class EKYCModule(reactContext: ReactApplicationContext) : ReactContextBaseJavaMo
     private var exitSheetResolvedViaButton = false
 
     private fun getTrueCurrentActivity(): Activity? {
-        // Try React Native's currentActivity first
+        // Try React Native's currentActivity first as a baseline
         val rnActivity = currentActivity
-        if (rnActivity != null && rnActivity.javaClass.simpleName != "MainActivity") {
+        
+        Log.d(TAG, "🔍 getTrueCurrentActivity check: rnActivity=${rnActivity?.javaClass?.simpleName}")
+
+        // If rnActivity is already an SDK activity, we can likely trust it
+        if (rnActivity != null && rnActivity.javaClass.name.startsWith("finos.sdk.")) {
+            Log.d(TAG, "✅ Current activity is already an SDK activity: ${rnActivity.javaClass.simpleName}")
             return rnActivity
         }
 
-        // If RN gives us MainActivity or null, but we know an SDK is showing, 
-        // we must find the actual top activity via reflection or ActivityThread
+        // Use reflection to find the absolute top resumed activity.
+        // This is crucial when the SDK is running in a native Activity on top of the RN Activity,
+        // and currentActivity might still point to the background Activity.
+        try {
+            val topActivity = findTopActivityViaReflection()
+            if (topActivity != null) {
+                Log.d(TAG, "✅ Found top activity via reflection: ${topActivity.javaClass.simpleName}")
+                return topActivity
+            }
+        } catch (e: Exception) {
+            Log.e(TAG, "Failed to find top activity via reflection", e)
+        }
+        
+        Log.d(TAG, "⚠️ Fallback to rnActivity: ${rnActivity?.javaClass?.simpleName}")
+        return rnActivity
+    }
+
+    /**
+     * Finds the top-most resumed activity using ActivityThread's internal records.
+     */
+    private fun findTopActivityViaReflection(): Activity? {
         try {
             val activityThreadClass = Class.forName("android.app.ActivityThread")
-            val activityThread = activityThreadClass.getMethod("currentActivityThread").invoke(null)
+            val activityThread = activityThreadClass.getMethod("currentActivityThread").invoke(null) ?: return null
             val activitiesField = activityThreadClass.getDeclaredField("mActivities")
             activitiesField.isAccessible = true
             
-            val activities = activitiesField.get(activityThread) as Map<Any, Any>
-            Log.d(TAG, "🔍 Scanning ${activities.size} activities...")
-            
-            var topActivity: Activity? = null
+            val activities = activitiesField.get(activityThread) as? Map<*, *> ?: return null
+            Log.d(TAG, "🔍 Scanning ${activities.size} activities via reflection...")
             
+            var bestCandidate: Activity? = null
+            var highestPriority = -1
+
             for (activityRecord in activities.values) {
-                val activityRecordClass = activityRecord.javaClass
-                
-                val activityField = activityRecordClass.getDeclaredField("activity")
-                activityField.isAccessible = true
-                val activity = activityField.get(activityRecord) as Activity
-                
-                val pausedField = activityRecordClass.getDeclaredField("paused")
-                pausedField.isAccessible = true
-                val isPaused = pausedField.get(activityRecord) as Boolean
-                
-                val stoppedField = activityRecordClass.getDeclaredField("stopped")
-                stoppedField.isAccessible = true
-                val isStopped = stoppedField.get(activityRecord) as Boolean
-                
-                Log.d(TAG, "   - Found: ${activity.javaClass.simpleName} (Paused: $isPaused, Stopped: $isStopped, Finishing: ${activity.isFinishing})")
-                
-                if (!activity.isFinishing && !isStopped) {
-                    topActivity = activity
-                    if (!isPaused) break // Found the resumed one
+                if (activityRecord == null) continue
+                try {
+                    val activityRecordClass = activityRecord.javaClass
+                    val activityField = activityRecordClass.getDeclaredField("activity")
+                    activityField.isAccessible = true
+                    val activity = activityField.get(activityRecord) as? Activity ?: continue
+
+                    if (activity.isFinishing) continue
+
+                    val pausedField = activityRecordClass.getDeclaredField("paused")
+                    pausedField.isAccessible = true
+                    val isPaused = pausedField.get(activityRecord) as Boolean
+
+                    val stoppedField = activityRecordClass.getDeclaredField("stopped")
+                    stoppedField.isAccessible = true
+                    val isStopped = stoppedField.get(activityRecord) as Boolean
+
+                    if (isStopped) continue
+
+                    // Identity check: prioritize known SDK activity patterns
+                    val className = activity.javaClass.name
+                    val isSDK = className.startsWith("finos.sdk.") || 
+                               className.contains("EKYC", ignoreCase = true) ||
+                               className.contains("Liveness", ignoreCase = true) ||
+                               className.contains("OCR", ignoreCase = true) ||
+                               className.contains("NFC", ignoreCase = true)
+
+                    // Priority scoring:
+                    // 3: Resumed SDK Activity
+                    // 2: Resumed App Activity
+                    // 1: Paused SDK Activity
+                    // 0: Paused App Activity
+                    val currentPriority = (if (isSDK) 1 else 0) + (if (!isPaused) 2 else 0)
+
+                    Log.v(TAG, "   [Audit] ${activity.javaClass.simpleName} -> Prio: $currentPriority (SDK=$isSDK, Paused=$isPaused)")
+
+                    if (currentPriority > highestPriority) {
+                        highestPriority = currentPriority
+                        bestCandidate = activity
+                    }
+
+                    // Optimization: Found the absolute best target
+                    if (highestPriority == 3) break
+
+                } catch (e: Exception) {
+                    // Silently continue for individual activity record failures
                 }
             }
-            return topActivity ?: rnActivity
+            
+            if (bestCandidate != null) {
+                Log.d(TAG, "✅ Best candidate found: ${bestCandidate.javaClass.simpleName} (Prio: $highestPriority)")
+            }
+            
+            return bestCandidate
         } catch (e: Exception) {
-            Log.e(TAG, "Error finding true current activity", e)
+            Log.e(TAG, "❌ Critical error in findTopActivityViaReflection", e)
+            return null
         }
-        
-        return rnActivity
     }
 
     @ReactMethod
     fun showRNExitSheet(bundleName: String, initialProps: ReadableMap, promise: Promise) {
-        val activity = getTrueCurrentActivity() ?: run {
-            Log.e(TAG, "❌ showRNExitSheet failed: No visible activity found")
-            promise.reject("NO_ACTIVITY", "No visible activity found")
-            return
-        }
-
-        if (activity.isFinishing || activity.isDestroyed) {
-            Log.e(TAG, "❌ showRNExitSheet failed: Activity is finishing/destroyed")
-            promise.reject("ACTIVITY_INVALID", "Activity is finishing or destroyed")
-            return
-        }
+        // Fix: Move activity detection to UI thread to avoid race conditions and ensure correct window attachment
+        Handler(Looper.getMainLooper()).post {
+            val activity = getTrueCurrentActivity() ?: run {
+                Log.e(TAG, "❌ showRNExitSheet failed: No visible activity found")
+                promise.reject("NO_ACTIVITY", "No visible activity found")
+                return@post
+            }
 
-        // Guard chống double-resolve/reject promise (vì có nhiều entry point: sync error, async listener, exception)
-        val promiseSettled = java.util.concurrent.atomic.AtomicBoolean(false)
-        fun safeResolve(value: Any?) {
-            if (promiseSettled.compareAndSet(false, true)) promise.resolve(value)
-        }
-        fun safeReject(code: String, msg: String, e: Throwable? = null) {
-            if (promiseSettled.compareAndSet(false, true)) {
-                if (e != null) promise.reject(code, msg, e) else promise.reject(code, msg)
+            if (activity.isFinishing || (android.os.Build.VERSION.SDK_INT >= 17 && activity.isDestroyed)) {
+                Log.e(TAG, "❌ showRNExitSheet failed: Activity is finishing or destroyed")
+                promise.reject("ACTIVITY_INVALID", "Activity is finishing or destroyed")
+                return@post
             }
-        }
 
-        activity.runOnUiThread {
+            // Guard chống double-resolve/reject promise
+            val promiseSettled = java.util.concurrent.atomic.AtomicBoolean(false)
+            fun safeResolve(value: Any?) {
+                if (promiseSettled.compareAndSet(false, true)) promise.resolve(value)
+            }
+            fun safeReject(code: String, msg: String, e: Throwable? = null) {
+                if (promiseSettled.compareAndSet(false, true)) {
+                    if (e != null) promise.reject(code, msg, e) else promise.reject(code, msg)
+                }
+            }
             try {
                 val activityName = activity.javaClass.simpleName
                 Log.d(TAG, "▶️ showRNExitSheet: Top Activity=$activityName, bundle=$bundleName")
 
                 val reactApplication = activity.application as? ReactApplication
                 if (reactApplication == null) {
-                    Log.e(TAG, "❌ Application does not implement ReactApplication")
+                    Log.e(TAG, "❌ showRNExitSheet failed: Application class (${activity.application.javaClass.name}) does not implement ReactApplication. Please ensure your Application class implements ReactApplication or provide a way for the SDK to access the ReactInstanceManager.")
                     safeReject("NO_REACT_APP", "Application does not implement ReactApplication")
-                    return@runOnUiThread
+                    return@post
                 }
                 val reactInstanceManager = reactApplication.reactNativeHost.reactInstanceManager
 
                 if (reactInstanceManager.currentReactContext != null) {
+                    Log.d(TAG, "✅ ReactContext is ready, presenting sheet...")
                     presentRNBottomSheet(activity, reactInstanceManager, bundleName, initialProps,
                         ::safeResolve, ::safeReject)
-                    return@runOnUiThread
+                    return@post
                 }
 
                 // Context chưa sẵn sàng: chờ listener
@@ -212,7 +269,7 @@ class EKYCModule(reactContext: ReactApplicationContext) : ReactContextBaseJavaMo
                     Log.d(TAG, "✅ ReactContext init xong giữa lúc add listener — present luôn")
                     presentRNBottomSheet(activity, reactInstanceManager, bundleName, initialProps,
                         ::safeResolve, ::safeReject)
-                    return@runOnUiThread
+                    return@post
                 }
 
                 // Chỉ trigger background create nếu chưa start (tránh IllegalStateException)
@@ -250,12 +307,17 @@ class EKYCModule(reactContext: ReactApplicationContext) : ReactContextBaseJavaMo
         safeReject: (String, String, Throwable?) -> Unit
     ) {
         try {
+            val minHeightPx = (400 * activity.resources.displayMetrics.density).toInt()
             val container = LinearLayout(activity).apply {
                 orientation = LinearLayout.VERTICAL
                 setBackgroundColor(Color.TRANSPARENT)
+                minimumHeight = minHeightPx
             }
 
             val rootView = ReactRootView(activity)
+            // Hardening: Set a minimum height to ensure the sheet is visible even before RN finishes layout
+            rootView.minimumHeight = minHeightPx
+            
             rootView.layoutParams = LinearLayout.LayoutParams(
                 LinearLayout.LayoutParams.MATCH_PARENT,
                 LinearLayout.LayoutParams.WRAP_CONTENT
@@ -328,7 +390,7 @@ class EKYCModule(reactContext: ReactApplicationContext) : ReactContextBaseJavaMo
 
             bottomSheetDialog.show()
 
-            Log.d(TAG, "✅ showRNExitSheet: BottomSheetDialog shown (wrap content), bundle=$bundleName")
+            Log.d(TAG, "✅ showRNExitSheet: BottomSheetDialog shown on ${activity.javaClass.simpleName}, bundle=$bundleName")
             safeResolve(true)
         } catch (e: Exception) {
             Log.e(TAG, "❌ Exception in presentRNBottomSheet", e)
@@ -903,12 +965,7 @@ class EKYCModule(reactContext: ReactApplicationContext) : ReactContextBaseJavaMo
                     isActiveLiveness = isActiveLiveness ?: false,
                     isActiveLivenessColor = isActiveLivenessColor ?: false,
                     isShowCameraFont = isShowCameraFont ?: true,
-                    // Custom actions từ checkboxes (nếu có chọn)
-                    // Nếu có actions được chọn → sử dụng customActions
-                    // Nếu không có actions nào được chọn → sử dụng random actions với activeActionCount
                     customActions = customActions,
-                    // Number of random actions (1-10), only used when customActions = null
-                    // activeActionCount = 2 → 2 random actions + STRAIGHT
                     activeActionCount = activeActionCount ?: 2,
                     forceCaptureTimeout = (forceCaptureTimeout ?: 0.0).toLong(),
                     selfieImage = imageFile,
@@ -2645,15 +2702,16 @@ class EKYCModule(reactContext: ReactApplicationContext) : ReactContextBaseJavaMo
 
     @ReactMethod
     fun setExitSheetHeight(heightDp: Float) {
-        val activity = getTrueCurrentActivity() ?: return
-        val heightPx = (heightDp * activity.resources.displayMetrics.density).toInt()
-        Log.d(TAG, "▶️ setExitSheetHeight: ${heightDp}dp → ${heightPx}px")
-        activity.runOnUiThread {
-            val sheet = currentExitSheet ?: return@runOnUiThread
-            if (!sheet.isShowing) return@runOnUiThread
+        Handler(Looper.getMainLooper()).post {
+            val activity = getTrueCurrentActivity() ?: return@post
+            val heightPx = (heightDp * activity.resources.displayMetrics.density).toInt()
+            Log.d(TAG, "▶️ setExitSheetHeight: ${heightDp}dp → ${heightPx}px")
+            
+            val sheet = currentExitSheet ?: return@post
+            if (!sheet.isShowing) return@post
             val bs = sheet.findViewById<View>(com.google.android.material.R.id.design_bottom_sheet)
-                ?: return@runOnUiThread
-            val lp = bs.layoutParams ?: return@runOnUiThread
+                ?: return@post
+            val lp = bs.layoutParams ?: return@post
             lp.height = heightPx
             bs.layoutParams = lp
             val behavior = com.google.android.material.bottomsheet.BottomSheetBehavior.from(bs)

package/dist/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@finos_sdk/sdk-ekyc",
-  "version": "1.4.9",
+  "version": "1.5.0",
   "description": "React Native SDK for eKYC - Vietnamese CCCD NFC reading, OCR, Liveness detection, Face matching, and C06, eSign, SmsOTP residence verification",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/dist/src/modules/FinosEKYCModule.js

@@ -826,7 +826,11 @@ const isMethod = (prop) => {
         prop === 'checkC06' ||
         prop === 'startOcr' ||
         prop === 'startLiveness' ||
-        prop === 'startFaceCompare';
+        prop === 'startFaceCompare' ||
+        prop === 'registerExitHandler' ||
+        prop === 'resolveExit' ||
+        prop === 'showRNExitSheet' ||
+        prop === 'setExitSheetComponent';
 };
 // Create a comprehensive stub object with all methods to prevent undefined errors
 const createFinosEKYCStub = () => {
@@ -856,6 +860,7 @@ const createFinosEKYCStub = () => {
         'startEkycUI', 'sendOtp', 'verifyOtp', 'resendOtp', 'initializeESign', 'openSessionId',
         'registerDevice', 'listCerts', 'verifyCert', 'listSignRequest', 'confirmSign',
         'registerRemoteSigning', 'signPdf', 'sendConfirmationDocument',
+        'registerExitHandler', 'resolveExit', 'showRNExitSheet', 'setExitSheetComponent',
         'onResume', 'onPause', 'isSDKReady', 'getSDKInfo'
     ];
     otherMethods.forEach(method => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@finos_sdk/sdk-ekyc",
-  "version": "1.4.9",
+  "version": "1.5.0",
   "description": "React Native SDK for eKYC - Vietnamese CCCD NFC reading, OCR, Liveness detection, Face matching, and C06, eSign, SmsOTP residence verification",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/modules/FinosEKYCModule.ts

@@ -1001,7 +1001,11 @@ const isMethod = (prop: string | symbol): boolean => {
         prop === 'checkC06' ||
         prop === 'startOcr' ||
         prop === 'startLiveness' ||
-        prop === 'startFaceCompare';
+        prop === 'startFaceCompare' ||
+        prop === 'registerExitHandler' ||
+        prop === 'resolveExit' ||
+        prop === 'showRNExitSheet' ||
+        prop === 'setExitSheetComponent';
 };
 
 // Create a comprehensive stub object with all methods to prevent undefined errors
@@ -1035,6 +1039,7 @@ const createFinosEKYCStub = (): FinosEKYCModule => {
         'startEkycUI', 'sendOtp', 'verifyOtp', 'resendOtp', 'initializeESign', 'openSessionId',
         'registerDevice', 'listCerts', 'verifyCert', 'listSignRequest', 'confirmSign',
         'registerRemoteSigning', 'signPdf', 'sendConfirmationDocument',
+        'registerExitHandler', 'resolveExit', 'showRNExitSheet', 'setExitSheetComponent',
         'onResume', 'onPause', 'isSDKReady', 'getSDKInfo'
     ];