Refactor: Use READ_MEDIA_IMAGES for Android 11+

I've switched the permission request logic to use READ_MEDIA_IMAGES
for Android 11 (API 30) and newer. This aligns with modern
Android permission practices and aims to resolve media access
issues on Android 12 by using a more standard permission
than MANAGE_EXTERNAL_STORAGE for typical media access.

Specifically:
- I modified `requiredPermissions` in MainActivity to request
  READ_MEDIA_IMAGES and READ_MEDIA_VIDEO for API 30+.
- For API 29 and older, it continues to request
  READ_EXTERNAL_STORAGE and WRITE_EXTERNAL_STORAGE.
- I reverted previous attempts to use MANAGE_EXTERNAL_STORAGE
  for Android 11/12, including removing the associated
  ActivityResultLauncher and specific dialogs.
- The existing permission rationale dialog is now used for
  all permission requests.

This change addresses your feedback to use minimal necessary
permissions and prepares for testing on Android 12 and 15.

Author: google-labs-jules[bot]

app/src/main/kotlin/com/google/ai/sample/MainActivity.kt

@@ -2,17 +2,16 @@ package com.google.ai.sample
 
 import android.Manifest
 import android.app.Activity
-import android.app.AlertDialog
+// AlertDialog, Uri, and Settings imports are removed as they were specific to the reverted MANAGE_EXTERNAL_STORAGE logic
 import android.content.BroadcastReceiver
 import android.content.Context
 import android.content.Intent
 import android.content.IntentFilter
 import android.content.SharedPreferences
 import android.content.pm.PackageManager
-import android.net.Uri
 import android.os.Build
 import android.os.Bundle
-import android.provider.Settings
+import android.os.Environment // Kept as per instruction for now
 import android.util.Log
 import android.widget.Toast
 import androidx.activity.ComponentActivity
@@ -97,6 +96,8 @@ class MainActivity : ComponentActivity() {
     private lateinit var prefs: SharedPreferences
     private var showFirstLaunchInfoDialog by mutableStateOf(false)
 
+    // manageStoragePermissionLauncher has been removed.
+
     private val trialStatusReceiver = object : BroadcastReceiver() {
         override fun onReceive(context: Context?, intent: Intent?) {
             Log.i(TAG, "trialStatusReceiver: Received broadcast: ${intent?.action}")
@@ -729,32 +730,33 @@ class MainActivity : ComponentActivity() {
 
     private fun checkAndRequestPermissions() {
         Log.d(TAG, "checkAndRequestPermissions called.")
-        val permissionsToRequest = requiredPermissions.filter {
+        // The 'requiredPermissions' field will be updated in the next plan step.
+        // This step focuses on reverting the structure of this method.
+        val permissionsNeeded = requiredPermissions.filter {
             ContextCompat.checkSelfPermission(this, it) != PackageManager.PERMISSION_GRANTED
         }.toTypedArray()
-        if (permissionsToRequest.isNotEmpty()) {
-            // Original: requestPermissionLauncher.launch(permissionsToRequest)
-            // New:
-            Log.i(TAG, "Required permissions not granted. Showing rationale dialog. Permissions: ${permissionsToRequest.joinToString()}")
-            showPermissionRationaleDialog = true
+
+        if (permissionsNeeded.isNotEmpty()) {
+            Log.i(TAG, "Permissions needed (${permissionsNeeded.joinToString()}). Showing rationale dialog.")
+            showPermissionRationaleDialog = true // This triggers PermissionRationaleDialog -> requestPermissionLauncher
         } else {
-            Log.i(TAG, "All required permissions already granted.")
-            Log.d(TAG, "checkAndRequestPermissions: Permissions granted, calling startTrialServiceIfNeeded. Current state: $currentTrialState")
-            // TODO: It seems the startTrialServiceIfNeeded() call was here in the original code when permissions were granted.
-            // Confirm if it should remain here or be handled elsewhere. For now, keeping it as per the 'else' block's original apparent structure.
-            // Based on the prompt, only the 'if (permissionsToRequest.isNotEmpty())' block's direct action changes.
+            Log.i(TAG, "All necessary permissions already granted.")
+            // Potentially call startTrialServiceIfNeeded() or other logic that runs when permissions are good.
+            // Refer to the original state of this method before MANAGE_EXTERNAL_STORAGE changes.
+            // Based on the provided MainActivity.kt, there wasn't a direct call here,
+            // but ensure the structure allows for normal app flow if permissions are granted.
         }
     }
 
-    private val requiredPermissions = if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU) {
+    private val requiredPermissions = if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.R) { // R is API Level 30 (Android 11)
         arrayOf(
             Manifest.permission.READ_MEDIA_IMAGES,
             Manifest.permission.READ_MEDIA_VIDEO
         )
-    } else {
+    } else { // Versions older than Android 11 (i.e., Android 10 and below)
         arrayOf(
             Manifest.permission.READ_EXTERNAL_STORAGE,
-            Manifest.permission.WRITE_EXTERNAL_STORAGE 
+            Manifest.permission.WRITE_EXTERNAL_STORAGE
         )
     }