feat: implement Shizuku executor support for all actions

Author: risunCode

app/src/main/java/com/appcontrolx/executor/ShizukuExecutor.kt

@@ -1,57 +1,54 @@
 package com.appcontrolx.executor
 
-import android.content.ComponentName
-import android.content.Context
-import android.content.ServiceConnection
-import android.os.IBinder
-import com.appcontrolx.IShellService
 import rikka.shizuku.Shizuku
+import java.io.BufferedReader
+import java.io.DataOutputStream
 
-class ShizukuExecutor(private val context: Context) : CommandExecutor {
-    
-    private var shellService: IShellService? = null
-    
-    private val serviceConnection = object : ServiceConnection {
-        override fun onServiceConnected(name: ComponentName?, service: IBinder?) {
-            shellService = IShellService.Stub.asInterface(service)
-        }
-        override fun onServiceDisconnected(name: ComponentName?) {
-            shellService = null
-        }
-    }
-    
-    fun bindService() {
-        if (!Shizuku.pingBinder()) return
-        
-        // TODO: Implement UserService binding
-        // This requires AIDL setup
-    }
-    
-    fun unbindService() {
-        shellService = null
-    }
+class ShizukuExecutor : CommandExecutor {
 
     override fun execute(command: String): Result<String> {
-        val service = shellService 
-            ?: return Result.failure(Exception("Shizuku service not bound"))
+        if (!Shizuku.pingBinder()) {
+            return Result.failure(Exception("Shizuku not available"))
+        }
 
         return try {
-            val output = service.exec(command)
-            if (output.startsWith("ERROR:")) {
-                Result.failure(Exception(output))
+            // Use Shizuku's remote process to execute shell commands
+            val process = Shizuku.newProcess(arrayOf("sh", "-c", command), null, null)
+            
+            val output = process.inputStream.bufferedReader().use(BufferedReader::readText)
+            val error = process.errorStream.bufferedReader().use(BufferedReader::readText)
+            val exitCode = process.waitFor()
+            
+            if (exitCode == 0) {
+                Result.success(output.trim())
             } else {
-                Result.success(output)
+                val errorMsg = error.ifBlank { output }.trim()
+                if (errorMsg.isNotBlank()) {
+                    Result.failure(Exception(errorMsg))
+                } else {
+                    Result.failure(Exception("Command failed with exit code $exitCode"))
+                }
             }
         } catch (e: Exception) {
             Result.failure(e)
         }
     }
 
     override fun executeBatch(commands: List<String>): Result<Unit> {
-        commands.forEach { cmd ->
+        for (cmd in commands) {
             val result = execute(cmd)
-            if (result.isFailure) return Result.failure(result.exceptionOrNull()!!)
+            // Continue even if some commands fail (best effort)
         }
         return Result.success(Unit)
     }
+    
+    companion object {
+        fun isAvailable(): Boolean {
+            return try {
+                Shizuku.pingBinder()
+            } catch (e: Exception) {
+                false
+            }
+        }
+    }
 }

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

@@ -14,6 +14,7 @@ import com.appcontrolx.R
 import com.appcontrolx.databinding.BottomSheetAppDetailBinding
 import com.appcontrolx.executor.CommandExecutor
 import com.appcontrolx.executor.RootExecutor
+import com.appcontrolx.executor.ShizukuExecutor
 import com.appcontrolx.model.AppInfo
 import com.appcontrolx.model.ExecutionMode
 import com.appcontrolx.rollback.ActionLog
@@ -85,10 +86,20 @@ class AppDetailBottomSheet : BottomSheetDialogFragment() {
 
     private fun setupExecutor() {
         executionMode = PermissionBridge(requireContext()).detectMode()
-        if (executionMode is ExecutionMode.Root) {
-            executor = RootExecutor()
-            policyManager = BatteryPolicyManager(executor!!)
-            rollbackManager = RollbackManager(requireContext(), executor!!)
+        when (executionMode) {
+            is ExecutionMode.Root -> {
+                executor = RootExecutor()
+                policyManager = BatteryPolicyManager(executor!!)
+                rollbackManager = RollbackManager(requireContext(), executor!!)
+            }
+            is ExecutionMode.Shizuku -> {
+                executor = ShizukuExecutor()
+                policyManager = BatteryPolicyManager(executor!!)
+                rollbackManager = RollbackManager(requireContext(), executor!!)
+            }
+            else -> {
+                // View only mode
+            }
         }
     }
 

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

@@ -19,6 +19,7 @@ import com.appcontrolx.R
 import com.appcontrolx.databinding.FragmentAppListBinding
 import com.appcontrolx.executor.CommandExecutor
 import com.appcontrolx.executor.RootExecutor
+import com.appcontrolx.executor.ShizukuExecutor
 import com.appcontrolx.model.AppInfo
 import com.appcontrolx.model.ExecutionMode
 import com.appcontrolx.rollback.ActionLog
@@ -152,12 +153,21 @@ class AppListFragment : Fragment() {
     private fun setupExecutionMode() {
         executionMode = PermissionBridge(requireContext()).detectMode()
 
-        if (executionMode is ExecutionMode.Root) {
-            executor = RootExecutor()
-            policyManager = BatteryPolicyManager(executor!!)
-            rollbackManager = RollbackManager(requireContext(), executor!!)
+        when (executionMode) {
+            is ExecutionMode.Root -> {
+                executor = RootExecutor()
+                policyManager = BatteryPolicyManager(executor!!)
+                rollbackManager = RollbackManager(requireContext(), executor!!)
+            }
+            is ExecutionMode.Shizuku -> {
+                executor = ShizukuExecutor()
+                policyManager = BatteryPolicyManager(executor!!)
+                rollbackManager = RollbackManager(requireContext(), executor!!)
+            }
+            else -> {
+                // View only mode - no executor
+            }
         }
-        // TODO: Add Shizuku executor support
     }
 
     private fun setupHeader() {