fix: action log for all actions, running badge logic (no badge = running)

Author: risunCode

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

@@ -314,19 +314,18 @@ class AppDetailBottomSheet : BottomSheetDialogFragment() {
                 val success = result?.isSuccess == true
 
                 // Log action - always log for all action types
-                if (appInfo != null && rollbackManager != null) {
-                    val finalLogAction = logActionName ?: actionName.uppercase().replace(" ", "_")
-                    withContext(Dispatchers.IO) {
-                        timber.log.Timber.d("Manual action: $finalLogAction, success=$success")
-                        rollbackManager!!.logAction(ActionLog(
-                            action = finalLogAction,
-                            packages = listOf(appInfo!!.packageName),
-                            success = success,
-                            message = if (success) null else "Failed"
-                        ))
+                appInfo?.let { app ->
+                    rollbackManager?.let { rm ->
+                        val finalLogAction = logActionName ?: actionName.uppercase().replace(" ", "_")
+                        withContext(Dispatchers.IO) {
+                            rm.logAction(ActionLog(
+                                action = finalLogAction,
+                                packages = listOf(app.packageName),
+                                success = success,
+                                message = if (success) null else "Failed"
+                            ))
+                        }
                     }
-                } else {
-                    timber.log.Timber.w("Cannot log: appInfo=${appInfo != null}, rm=${rollbackManager != null}")
                 }
 
                 if (success) {
@@ -369,18 +368,17 @@ class AppDetailBottomSheet : BottomSheetDialogFragment() {
                 val success = result?.isSuccess == true
 
                 // Log action
-                if (appInfo != null && rollbackManager != null) {
-                    withContext(Dispatchers.IO) {
-                        timber.log.Timber.d("Background action: $logActionName, success=$success")
-                        rollbackManager!!.logAction(ActionLog(
-                            action = logActionName,
-                            packages = listOf(appInfo!!.packageName),
-                            success = success,
-                            message = if (success) null else "Failed"
-                        ))
+                appInfo?.let { app ->
+                    rollbackManager?.let { rm ->
+                        withContext(Dispatchers.IO) {
+                            rm.logAction(ActionLog(
+                                action = logActionName,
+                                packages = listOf(app.packageName),
+                                success = success,
+                                message = if (success) null else "Failed"
+                            ))
+                        }
                     }
-                } else {
-                    timber.log.Timber.w("Cannot log bg action: appInfo=${appInfo != null}, rm=${rollbackManager != null}")
                 }
 
                 if (success) {

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

@@ -419,18 +419,15 @@ class AppListFragment : Fragment() {
             },
             onComplete = { successCount, failCount ->
                 // Log action - always log regardless of action type
-                if (rm != null) {
+                rm?.let { manager ->
                     lifecycleScope.launch(Dispatchers.IO) {
-                        timber.log.Timber.d("Batch complete: ${action.name}, success=$successCount, fail=$failCount")
-                        rm.logAction(ActionLog(
+                        manager.logAction(ActionLog(
                             action = action.name,
                             packages = packages,
                             success = failCount == 0,
                             message = if (failCount == 0) "All $successCount succeeded" else "$failCount failed"
                         ))
                     }
-                } else {
-                    timber.log.Timber.w("RollbackManager is null, cannot log action")
                 }
 
                 adapter.deselectAll()

app/src/main/java/com/appcontrolx/ui/adapter/AppListAdapter.kt

@@ -113,13 +113,13 @@ class AppListAdapter(
                 root.alpha = if (app.isEnabled) 1f else 0.6f
 
                 // Status badges
-                // Running detection: if not frozen, not stopped, not restricted = running
-                val isEffectivelyRunning = !app.isFrozen && !app.isStopped && !app.isBackgroundRestricted
-                val hasAnyBadge = app.isFrozen || isEffectivelyRunning || app.isStopped || app.isBackgroundRestricted
-                statusContainer.visibility = if (hasAnyBadge) View.VISIBLE else View.GONE
+                // New logic: No badge (not frozen, not stopped) = RUNNING
+                // RUNNING can coexist with BACKGROUND RESTRICTED
+                val isRunning = !app.isFrozen && !app.isStopped
+                statusContainer.visibility = View.VISIBLE // Always show container
 
                 tvStatusFrozen.visibility = if (app.isFrozen) View.VISIBLE else View.GONE
-                tvStatusRunning.visibility = if (isEffectivelyRunning) View.VISIBLE else View.GONE
+                tvStatusRunning.visibility = if (isRunning) View.VISIBLE else View.GONE
                 tvStatusStopped.visibility = if (app.isStopped && !app.isFrozen) View.VISIBLE else View.GONE
                 tvStatusRestricted.visibility = if (app.isBackgroundRestricted && !app.isFrozen) View.VISIBLE else View.GONE