react-native-bug-reporter 1.0.2 → 1.0.4

package/android/src/main/AndroidManifest.xml

@@ -3,9 +3,14 @@
   <!-- Official screenshot detection on Android 14+ (no runtime prompt). -->
   <uses-permission android:name="android.permission.DETECT_SCREEN_CAPTURE" />
 
-  <!-- MediaStore fallback for screenshot detection on Android 13 and below. -->
+  <!-- MediaStore fallback for screenshot detection on Android 13. -->
   <uses-permission
       android:name="android.permission.READ_MEDIA_IMAGES"
       android:maxSdkVersion="33" />
 
+  <!-- MediaStore fallback for Android 12L and below (older devices). -->
+  <uses-permission
+      android:name="android.permission.READ_EXTERNAL_STORAGE"
+      android:maxSdkVersion="32" />
+
 </manifest>

package/android/src/main/java/com/bugreporter/ScreenCaptureCallbackApi34.kt

@@ -0,0 +1,34 @@
+package com.bugreporter
+
+import android.app.Activity
+import android.os.Handler
+import java.util.concurrent.Executor
+
+/**
+ * Isolated wrapper around the API 34+ `Activity.ScreenCaptureCallback`.
+ *
+ * IMPORTANT: this class is the ONLY place that references
+ * `Activity.ScreenCaptureCallback`. The main module must never reference that
+ * type directly — otherwise, on Android 13 and below where the class does not
+ * exist, the runtime's reflection over the module's methods throws
+ * `NoClassDefFoundError` and the whole native module fails to load.
+ *
+ * This class is only ever instantiated inside an `SDK_INT >= 34` guard, so it
+ * (and the missing framework class) is never loaded on older devices.
+ */
+class ScreenCaptureCallbackApi34(
+  private val activity: Activity,
+  private val handler: Handler,
+  private val onCapture: () -> Unit,
+) {
+  private val callback = Activity.ScreenCaptureCallback { onCapture() }
+
+  fun register() {
+    val executor = Executor { command -> handler.post(command) }
+    activity.registerScreenCaptureCallback(executor, callback)
+  }
+
+  fun unregister() {
+    activity.unregisterScreenCaptureCallback(callback)
+  }
+}

package/android/src/main/java/com/bugreporter/ScreenshotDetectorModule.kt

@@ -1,6 +1,8 @@
 package com.bugreporter
 
+import android.Manifest
 import android.app.Activity
+import android.content.pm.PackageManager
 import android.database.ContentObserver
 import android.graphics.Bitmap
 import android.net.Uri
@@ -18,7 +20,6 @@ import com.facebook.react.bridge.WritableMap
 import com.facebook.react.modules.core.DeviceEventManagerModule
 import java.io.File
 import java.io.FileOutputStream
-import java.util.concurrent.Executor
 
 /**
  * Native screenshot detector for Android.
@@ -38,7 +39,8 @@ class ScreenshotDetectorModule(private val reactContext: ReactApplicationContext
 
   private val mainHandler = Handler(Looper.getMainLooper())
   private var contentObserver: ContentObserver? = null
-  private var screenCaptureCallback: Any? = null
+  // Typed as Any? so this class never references the API-34 ScreenCaptureCallback.
+  private var screenCapture: ScreenCaptureCallbackApi34? = null
   private var isListening = false
 
   override fun getName(): String = NAME
@@ -50,10 +52,32 @@ class ScreenshotDetectorModule(private val reactContext: ReactApplicationContext
     if (Build.VERSION.SDK_INT >= 34) {
       registerScreenCaptureCallback()
     } else {
+      // Older devices detect screenshots via MediaStore, which needs the media
+      // read permission granted at runtime.
+      requestStoragePermissionIfNeeded()
       registerContentObserver()
     }
   }
 
+  /**
+   * Requests the media-read permission on Android 13- so the MediaStore
+   * ContentObserver reliably receives screenshot notifications. No-op on
+   * Android 14+ (the official ScreenCaptureCallback needs no permission).
+   */
+  private fun requestStoragePermissionIfNeeded() {
+    val activity = reactContext.currentActivity ?: return
+    val perm =
+      if (Build.VERSION.SDK_INT >= 33) Manifest.permission.READ_MEDIA_IMAGES
+      else Manifest.permission.READ_EXTERNAL_STORAGE
+    try {
+      if (activity.checkSelfPermission(perm) != PackageManager.PERMISSION_GRANTED) {
+        activity.requestPermissions(arrayOf(perm), PERMISSION_REQUEST_CODE)
+      }
+    } catch (_: Exception) {
+      // Ignore — observer is still registered; some OEMs notify without it.
+    }
+  }
+
   @ReactMethod
   fun stop() {
     isListening = false
@@ -78,13 +102,14 @@ class ScreenshotDetectorModule(private val reactContext: ReactApplicationContext
   private fun registerScreenCaptureCallback() {
     val activity = reactContext.currentActivity ?: return
     mainHandler.post {
-      val executor = Executor { command -> mainHandler.post(command) }
-      val callback = Activity.ScreenCaptureCallback {
-        onScreenshotDetected()
-      }
-      screenCaptureCallback = callback
       try {
-        activity.registerScreenCaptureCallback(executor, callback)
+        // All ScreenCaptureCallback usage is isolated in this helper so the
+        // missing class never loads on older Android.
+        val helper = ScreenCaptureCallbackApi34(activity, mainHandler) {
+          onScreenshotDetected()
+        }
+        helper.register()
+        screenCapture = helper
       } catch (e: Exception) {
         // Fall back to the content observer if the callback can't be registered.
         registerContentObserver()
@@ -93,14 +118,13 @@ class ScreenshotDetectorModule(private val reactContext: ReactApplicationContext
   }
 
   private fun unregisterScreenCaptureCallback() {
-    val activity = reactContext.currentActivity ?: return
-    val callback = screenCaptureCallback as? Activity.ScreenCaptureCallback ?: return
+    val helper = screenCapture ?: return
     mainHandler.post {
       try {
-        activity.unregisterScreenCaptureCallback(callback)
+        helper.unregister()
       } catch (_: Exception) {
       }
-      screenCaptureCallback = null
+      screenCapture = null
     }
   }
 
@@ -223,5 +247,6 @@ class ScreenshotDetectorModule(private val reactContext: ReactApplicationContext
 
   companion object {
     const val NAME = "RNBugReporterScreenshot"
+    private const val PERMISSION_REQUEST_CODE = 7322
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-native-bug-reporter",
-  "version": "1.0.2",
+  "version": "1.0.4",
   "description": "Universal React Native bug reporter: auto-detect screenshots, annotate, collect device/app/user context, and ship to Supabase (Postgres + Storage) with Edge Function email.",
   "main": "lib/commonjs/index.js",
   "module": "lib/module/index.js",