laantungir/Amber
1
0
Fork 0
mirror of https://github.com/greenart7c3/Amber.git synced 2026-10-05 19:08:23 +00:00
Code Issues Packages Projects Releases Wiki Activity

Improve BootReceiver: fix double start, dedup branches, use Flow.first

Browse Source 
- Remove ACTION_PACKAGE_REPLACED handler and manifest entry; MY_PACKAGE_REPLACED
  covers all supported API levels, so both firing caused startService() to be
  called twice on pre-S devices
- Merge identical ACTION_MY_PACKAGE_REPLACED and ACTION_BOOT_COMPLETED branches
- Replace busy-poll loop with isStartingAppState.first { !it } for efficient suspension

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
greenart7c3
2026-02-28 14:40:23 -03:00
co-authored by Claude Sonnet 4.6
parent 7d910ea57d
commit ee9d6e5ab9
2 changed files with 4 additions and 27 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/src/main/AndroidManifest.xml
-1
View File
@@ -46,7 +46,6 @@
<intent-filter>
<action android:name="android.intent.action.BOOT_COMPLETED" />
<action android:name="android.intent.action.MY_PACKAGE_REPLACED" />
<action android:name="android.intent.action.PACKAGE_REPLACED" />
<category android:name="android.intent.category.DEFAULT" />
</intent-filter>
app/src/main/java/com/greenart7c3/nostrsigner/service/BootReceiver.kt
+4 -26
View File
@@ -3,11 +3,10 @@ package com.greenart7c3.nostrsigner.service
import android.content.BroadcastReceiver
import android.content.Context
import android.content.Intent
import android.os.Build
import android.util.Log
import com.greenart7c3.nostrsigner.Amber
import com.greenart7c3.nostrsigner.BuildFlavorChecker
import kotlinx.coroutines.delay
import kotlinx.coroutines.flow.first
import kotlinx.coroutines.launch
class BootReceiver : BroadcastReceiver() {
@@ -16,32 +15,11 @@ class BootReceiver : BroadcastReceiver() {
Log.d(Amber.TAG, "Received intent: ${intent.action}")
when (intent.action) {
Intent.ACTION_PACKAGE_REPLACED -> {
Log.d(Amber.TAG, "Received ACTION_PACKAGE_REPLACED")
if (intent.dataString?.contains("com.greenart7c3.nostrsigner") == true && Build.VERSION.SDK_INT < Build.VERSION_CODES.S) {
Amber.instance.applicationIOScope.launch {
while (Amber.instance.isStartingAppState.value) {
delay(1000)
}
Amber.instance.startService()
}
}
}
Intent.ACTION_MY_PACKAGE_REPLACED -> {
Log.d(Amber.TAG, "Received ACTION_MY_PACKAGE_REPLACED")
Amber.instance.applicationIOScope.launch {
while (Amber.instance.isStartingAppState.value) {
delay(1000)
}
Amber.instance.startService()
}
}
Intent.ACTION_MY_PACKAGE_REPLACED,
Intent.ACTION_BOOT_COMPLETED -> {
Log.d(Amber.TAG, "Received ACTION_BOOT_COMPLETED")
Log.d(Amber.TAG, "Received ${intent.action}")
Amber.instance.applicationIOScope.launch {
while (Amber.instance.isStartingAppState.value) {
delay(1000)
}
Amber.instance.isStartingAppState.first { !it }
Amber.instance.startService()
}
}