Fix action logs not being recorded - BatchProgressBottomSheet now returns success/fail counts in onComplete - AppListFragment logs actions after batch completes - Snapshot saved before action for potential rollback

Author: risunCode

app/src/main/java/com/appcontrolx/ui/AppListFragment.kt

@@ -370,7 +370,17 @@ class AppListFragment : Fragment() {
                     }
                 }
             },
-            onComplete = {
+            onComplete = { successCount, failCount ->
+                // Log action
+                lifecycleScope.launch(Dispatchers.IO) {
+                    rm?.logAction(ActionLog(
+                        action = action.name,
+                        packages = packages,
+                        success = failCount == 0,
+                        message = if (failCount == 0) null else "$failCount failed"
+                    ))
+                }
+                
                 adapter.deselectAll()
                 clearCache()
                 loadApps(forceRefresh = true)
@@ -380,7 +390,7 @@ class AppListFragment : Fragment() {
         // Show immediately without waiting
         bottomSheet.show(childFragmentManager, BatchProgressBottomSheet.TAG)
 
-        // Save snapshot in background (non-blocking)
+        // Save snapshot before action (for rollback)
         lifecycleScope.launch(Dispatchers.IO) {
             rm?.saveSnapshot(packages)
         }

app/src/main/java/com/appcontrolx/ui/BatchProgressBottomSheet.kt

@@ -29,7 +29,7 @@ class BatchProgressBottomSheet : BottomSheetDialogFragment() {
     private var packageNames = listOf<String>()
     private var showCacheSize = false
     private var onExecute: (suspend (String) -> Result<Unit>)? = null
-    private var onComplete: (() -> Unit)? = null
+    private var onComplete: ((successCount: Int, failCount: Int) -> Unit)? = null
 
     private val adapter = BatchAppAdapter()
     private var executionJob: Job? = null
@@ -43,7 +43,7 @@ class BatchProgressBottomSheet : BottomSheetDialogFragment() {
             appNames: List<String>,
             packageNames: List<String>,
             onExecute: suspend (String) -> Result<Unit>,
-            onComplete: () -> Unit
+            onComplete: (successCount: Int, failCount: Int) -> Unit
         ): BatchProgressBottomSheet {
             return BatchProgressBottomSheet().apply {
                 this.actionName = actionName
@@ -186,8 +186,11 @@ class BatchProgressBottomSheet : BottomSheetDialogFragment() {
             getString(R.string.batch_partial_success, successCount, failCount)
         }
         b.btnCancel.text = getString(R.string.btn_close)
+        
+        val finalSuccess = successCount
+        val finalFail = failCount
         b.btnCancel.setOnClickListener {
-            onComplete?.invoke()
+            onComplete?.invoke(finalSuccess, finalFail)
             dismiss()
         }
     }