Skip to content

Commit ba2974a

Browse files
sds100sds100
committed
Merge branch 'refs/heads/develop'
2 parents a070cad + ced007e commit ba2974a

File tree

12 files changed

+538
-80
lines changed

12 files changed

+538
-80
lines changed

‎CHANGELOG.md‎

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,13 @@
1+
## [2.7.1](https://github.com/sds100/KeyMapper/releases/tag/v2.7.1)
2+
3+
#### 13 December 2024
4+
5+
## Bug fixes
6+
7+
- #1360 complete the documentation for advanced triggers at docs.keymapper.club
8+
- #1364 key event actions no longer crash when using Shizuku
9+
- #1362 backing up and restoring key maps works again
10+
111
## [2.7.0](https://github.com/sds100/KeyMapper/releases/tag/v2.7.0)
212

313
#### 8 December 2024

‎app/proguard-rules.pro‎

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,9 +64,17 @@
6464
-keep class androidx.recyclerview.** { *; }
6565
-keep interface androidx.recyclerview.** { *; }
6666

67+
# Keep all the AIDL classes because they must not be ofuscated for the bindings to work.
68+
-keep class android.hardware.input.IInputManager { *; }
6769
-keep class android.hardware.input.IInputManager$Stub { *; }
70+
-keep class android.content.pm.IPackageManager { *; }
6871
-keep class android.content.pm.IPackageManager$Stub { *; }
72+
-keep class android.permission.IPermissionManager { *; }
6973
-keep class android.permission.IPermissionManager$Stub { *; }
74+
-keep class io.github.sds100.keymapper.api.IKeyEventRelayService { *; }
75+
-keep class io.github.sds100.keymapper.api.IKeyEventRelayService$Stub { *; }
76+
-keep class io.github.sds100.keymapper.api.IKeyEventRelayServiceCallback { *; }
77+
-keep class io.github.sds100.keymapper.api.IKeyEventRelayServiceCallback$Stub { *; }
7078

7179
-keepattributes *Annotation*, InnerClasses
7280
-dontnote kotlinx.serialization.AnnotationsKt # core serialization annotations

‎app/src/main/java/io/github/sds100/keymapper/backup/BackupManager.kt‎

Lines changed: 42 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -113,10 +113,12 @@ class BackupManagerImpl(
113113
.registerTypeAdapter(FingerprintMapEntity.DESERIALIZER)
114114
.registerTypeAdapter(KeyMapEntity.DESERIALIZER)
115115
.registerTypeAdapter(TriggerEntity.DESERIALIZER)
116+
.registerTypeAdapter(TriggerKeyEntity.SERIALIZER)
116117
.registerTypeAdapter(TriggerKeyEntity.DESERIALIZER)
117118
.registerTypeAdapter(ActionEntity.DESERIALIZER)
118119
.registerTypeAdapter(Extra.DESERIALIZER)
119-
.registerTypeAdapter(ConstraintEntity.DESERIALIZER).create()
120+
.registerTypeAdapter(ConstraintEntity.DESERIALIZER)
121+
.create()
120122
}
121123

122124
private val backupAutomatically: Flow<Boolean> = preferenceRepository
@@ -426,7 +428,7 @@ class BackupManagerImpl(
426428
} catch (e: NoSuchElementException) {
427429
return Error.CorruptJsonFile(e.message ?: "")
428430
} catch (e: Exception) {
429-
e.printStackTrace()
431+
Timber.e(e)
430432

431433
if (throwExceptions) {
432434
throw e
@@ -448,45 +450,45 @@ class BackupManagerImpl(
448450
// delete the contents of the file
449451
output.clear()
450452

451-
val json = gson.toJson(
452-
BackupModel(
453-
AppDatabase.DATABASE_VERSION,
454-
Constants.VERSION_CODE,
455-
keyMapList,
456-
fingerprintMaps,
457-
defaultLongPressDelay =
458-
preferenceRepository
459-
.get(Keys.defaultLongPressDelay)
460-
.first()
461-
.takeIf { it != PreferenceDefaults.LONG_PRESS_DELAY },
462-
defaultDoublePressDelay =
463-
preferenceRepository
464-
.get(Keys.defaultDoublePressDelay)
465-
.first()
466-
.takeIf { it != PreferenceDefaults.DOUBLE_PRESS_DELAY },
467-
defaultRepeatDelay =
468-
preferenceRepository
469-
.get(Keys.defaultRepeatDelay)
470-
.first()
471-
.takeIf { it != PreferenceDefaults.REPEAT_DELAY },
472-
defaultRepeatRate =
473-
preferenceRepository
474-
.get(Keys.defaultRepeatRate)
475-
.first()
476-
.takeIf { it != PreferenceDefaults.REPEAT_RATE },
477-
defaultSequenceTriggerTimeout =
478-
preferenceRepository
479-
.get(Keys.defaultSequenceTriggerTimeout)
480-
.first()
481-
.takeIf { it != PreferenceDefaults.SEQUENCE_TRIGGER_TIMEOUT },
482-
defaultVibrationDuration =
483-
preferenceRepository
484-
.get(Keys.defaultVibrateDuration)
485-
.first()
486-
.takeIf { it != PreferenceDefaults.VIBRATION_DURATION },
487-
),
453+
val backupModel = BackupModel(
454+
AppDatabase.DATABASE_VERSION,
455+
Constants.VERSION_CODE,
456+
keyMapList,
457+
fingerprintMaps,
458+
defaultLongPressDelay =
459+
preferenceRepository
460+
.get(Keys.defaultLongPressDelay)
461+
.first()
462+
.takeIf { it != PreferenceDefaults.LONG_PRESS_DELAY },
463+
defaultDoublePressDelay =
464+
preferenceRepository
465+
.get(Keys.defaultDoublePressDelay)
466+
.first()
467+
.takeIf { it != PreferenceDefaults.DOUBLE_PRESS_DELAY },
468+
defaultRepeatDelay =
469+
preferenceRepository
470+
.get(Keys.defaultRepeatDelay)
471+
.first()
472+
.takeIf { it != PreferenceDefaults.REPEAT_DELAY },
473+
defaultRepeatRate =
474+
preferenceRepository
475+
.get(Keys.defaultRepeatRate)
476+
.first()
477+
.takeIf { it != PreferenceDefaults.REPEAT_RATE },
478+
defaultSequenceTriggerTimeout =
479+
preferenceRepository
480+
.get(Keys.defaultSequenceTriggerTimeout)
481+
.first()
482+
.takeIf { it != PreferenceDefaults.SEQUENCE_TRIGGER_TIMEOUT },
483+
defaultVibrationDuration =
484+
preferenceRepository
485+
.get(Keys.defaultVibrateDuration)
486+
.first()
487+
.takeIf { it != PreferenceDefaults.VIBRATION_DURATION },
488488
)
489489

490+
val json = gson.toJson(backupModel)
491+
490492
val backupUid = uuidGenerator.random()
491493

492494
tempBackupDir = fileAdapter.getPrivateFile("$TEMP_BACKUP_ROOT_DIR/$backupUid")
@@ -549,7 +551,7 @@ class BackupManagerImpl(
549551
val appVersion: Int,
550552

551553
@SerializedName(NAME_KEYMAP_LIST)
552-
val keymapList: List<KeyMapEntity>? = null,
554+
val keyMapList: List<KeyMapEntity>? = null,
553555

554556
@SerializedName(NAME_FINGERPRINT_MAP_LIST)
555557
val fingerprintMapList: List<FingerprintMapEntity>?,

‎app/src/main/java/io/github/sds100/keymapper/data/db/typeconverter/TriggerTypeConverter.kt‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ class TriggerTypeConverter {
1818
fun toTrigger(json: String): TriggerEntity {
1919
val gson = GsonBuilder()
2020
.registerTypeAdapter(TriggerEntity.DESERIALIZER)
21+
.registerTypeAdapter(TriggerKeyEntity.SERIALIZER)
2122
.registerTypeAdapter(TriggerKeyEntity.DESERIALIZER)
2223
.registerTypeAdapter(Extra.DESERIALIZER).create()
2324

‎app/src/main/java/io/github/sds100/keymapper/data/entities/TriggerKeyEntity.kt‎

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,12 @@ import com.github.salomonbrys.kotson.byNullableInt
66
import com.github.salomonbrys.kotson.byNullableString
77
import com.github.salomonbrys.kotson.byString
88
import com.github.salomonbrys.kotson.jsonDeserializer
9+
import com.github.salomonbrys.kotson.jsonSerializer
910
import com.github.salomonbrys.kotson.obj
11+
import com.google.gson.Gson
1012
import com.google.gson.JsonDeserializer
1113
import com.google.gson.JsonElement
14+
import com.google.gson.JsonSerializer
1215
import java.util.UUID
1316

1417
sealed class TriggerKeyEntity : Parcelable {
@@ -26,6 +29,17 @@ sealed class TriggerKeyEntity : Parcelable {
2629
const val LONG_PRESS = 1
2730
const val DOUBLE_PRESS = 2
2831

32+
/**
33+
* This is required because they are subclasses and Gson doesn't know about their
34+
* fields otherwise.
35+
*/
36+
val SERIALIZER: JsonSerializer<TriggerKeyEntity> = jsonSerializer { (key) ->
37+
when (key) {
38+
is AssistantTriggerKeyEntity -> Gson().toJsonTree(key)
39+
is KeyCodeTriggerKeyEntity -> Gson().toJsonTree(key)
40+
}
41+
}
42+
2943
val DESERIALIZER: JsonDeserializer<TriggerKeyEntity> =
3044
jsonDeserializer { (json, _, _) ->
3145
// nullable because this property was added after backup and restore was released.

‎app/version.properties‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
1-
VERSION_NAME=2.7.0
2-
VERSION_CODE=67
1+
VERSION_NAME=2.7.1
2+
VERSION_CODE=68
33
VERSION_NUM=0

‎docs/contributing/codebase.md‎

Lines changed: 35 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,35 @@
1+
# Structure
2+
3+
This app follows something inspired from Clean Architecture and package-by-feature.
4+
5+
## Architecture
6+
7+
All data structures that are persisted are passed around as one of two objects:
8+
9+
1. **Non-Entity**. This models the data in a way that makes the code more readable and doing the business logic easier. There are no rules for how these need to be named. They should be named what they are. E.g KeyMap, Action, Constraint.
10+
11+
2. **Entity**. This models how the data should be stored. The class name has an ...Entity suffix. E.g KeyMapEntity. The data is more optimised for storing and the code required to get the data from these models isn't very concise or elegant. The developer took some strange decisions in the first versions of this app. 😆
12+
13+
Every screen in the app has a view model and the view model interacts with one or multiple *use cases* (more below). The view model converts data that needs to be shown to the user into something that can populate the user interface. For example, the data values in the Action object isn't very useful to the user so this needs to be converted into strings and images that do mean something to the user. All the view models have a ResourceProvider dependency which is how they get strings, Drawables and colours from the resources without having to use a Context. This isn't a problem for configuration changes (e.g locale change) because the activity is recreated, which means all the resources are re-fetched in the view model.
14+
15+
The use cases contains all the business logic in the app. A *use case* interacts with the adapters and repositories mentioned below. A use case is made for everything that can be done in the app. E.g configuring a key map, displaying a mapping, configuring settings, onboarding the user. Most use cases correspond to something that *the user can do* in the app but some do not because they contain complicated code that is used in multiple use cases. E.g the GetActionErrorUseCase which determines if an action can be performed successfully.
16+
17+
Adapters and repositories contain all interactions with the Android framework (except UI stuff). This is so that tests can be more easily written and executed. Android often changes what apps are allowed to do and how so abstracting these interactions away means the code only needs to be changed in a single place. This means that the only place that a Context object is used is in Services, Activities, Fragments and the adapters.
18+
19+
## Package by feature
20+
21+
Every package contains files related to each other. For example, everything (view models, fragments, use cases) to do with constraints is stored in one package.
22+
The only package which isn't a feature is the `data` package because it is useful to have some of the things in there together, e.g the migrations.
23+
The `system` package bundles all the packages which are related to the Android framework because there are so many.
24+
25+
![contributing-app-structure](../images/contributing-app-structure.png)
26+
27+
# Key event detection and input
28+
29+
The diagram below shows how key events are passed around Key Mapper on Android 14+. This change was required because in Android 14 Android restricted the rate at which intents can be broadcast to once per second when an app is backgrounded. This is too slow for repeating key event actions in Key Mapper. Key Mapper still uses broadcast receivers to send key events between the accessibility service and input method on older Android versions to reduce the chance of breaking everyone's key maps. As shown in the diagram this is a bit complicated and potentially over-engineered but it must be two-way and future proof to any further restrictions. Using manifest-defined broadcast receivers, that aren't rate limited isn't an elegant solution because one has to pass messages between these broadcast receivers and the services through some 3rd class. Binder is lower latency than using intents and is synchronous whereas broadcast receivers are asynchronous. Apps are not allowed to bind to accessibility services so a new "relay" service needed to be made to link the accessibility and input method services.
30+
31+
The accessibility service is where triggers are detected by listening to the key events that Android system sends it. Key event and text actions send their key events from the accessibility service to the relay service, which then forwards it to the input method.
32+
33+
The code for input methods to talk to the Key Event relay service can be found in KeyEventRelayServiceWrapper. Key events are listened to from the input method service when Android blocks key events to the accessibility service during phone calls.
34+
35+
![key-event-relay-service](../images/key-event-relay-service.svg)

docs/contributing.md renamed to docs/contributing/introduction.md

Lines changed: 1 addition & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ You can get the apks for the pre-release versions in 2 ways:
1818

1919
### How can I help?
2020
- Test and experiment new features. All features and bug-fixes that are being worked on for a release can be found on the Projects page [here](https://github.com/keymapperorg/KeyMapper/projects).
21-
- If you find any bugs or crashes then report them by following the guide [here](report-issues.md).
21+
- If you find any bugs or crashes then report them by following the guide [here](../report-issues.md).
2222

2323
## Contributing code
2424

@@ -46,32 +46,6 @@ There are also 4 build types, which have different optimizations and package nam
4646
- **debug_release** = This is a debug build type that does not include a package name suffix so that it is possible to test how the production app will look. It is the only way to get the Google Play Billing library functioning because it will break if the package name isn't the same as on the Play store.
4747
- **ci** = This is used for alpha builds to the community in Discord. It includes some optimizations to ensure it is performant but doesn't obfuscate the code so it is possible to understand logs and bug reports. It has a `.ci` package name suffix.
4848

49-
### Introduction to the structure
50-
51-
This app follows something inspired from Clean Architecture and package-by-feature.
52-
53-
#### Architecture
54-
55-
All data structures that are persisted are passed around as one of two objects:
56-
57-
1. **Non-Entity**. This models the data in a way that makes the code more readable and doing the business logic easier. There are no rules for how these need to be named. They should be named what they are. E.g KeyMap, Action, Constraint.
58-
59-
2. **Entity**. This models how the data should be stored. The class name has an ...Entity suffix. E.g KeyMapEntity. The data is more optimised for storing and the code required to get the data from these models isn't very concise or elegant. The developer took some strange decisions in the first versions of this app. 😆
60-
61-
Every screen in the app has a view model and the view model interacts with one or multiple *use cases* (more below). The view model converts data that needs to be shown to the user into something that can populate the user interface. For example, the data values in the Action object isn't very useful to the user so this needs to be converted into strings and images that do mean something to the user. All the view models have a ResourceProvider dependency which is how they get strings, Drawables and colours from the resources without having to use a Context. This isn't a problem for configuration changes (e.g locale change) because the activity is recreated, which means all the resources are re-fetched in the view model.
62-
63-
The use cases contains all the business logic in the app. A *use case* interacts with the adapters and repositories mentioned below. A use case is made for everything that can be done in the app. E.g configuring a key map, displaying a mapping, configuring settings, onboarding the user. Most use cases correspond to something that *the user can do* in the app but some do not because they contain complicated code that is used in multiple use cases. E.g the GetActionErrorUseCase which determines if an action can be performed successfully.
64-
65-
Adapters and repositories contain all interactions with the Android framework (except UI stuff). This is so that tests can be more easily written and executed. Android often changes what apps are allowed to do and how so abstracting these interactions away means the code only needs to be changed in a single place. This means that the only place that a Context object is used is in Services, Activities, Fragments and the adapters.
66-
67-
#### Package by feature
68-
69-
Every package contains files related to each other. For example, everything (view models, fragments, use cases) to do with constraints is stored in one package.
70-
The only package which isn't a feature is the `data` package because it is useful to have some of the things in there together, e.g the migrations.
71-
The `system` package bundles all the packages which are related to the Android framework because there are so many.
72-
73-
![contributing-app-structure](images/contributing-app-structure.png)
74-
7549
### Branches 🌴
7650

7751
- master: Everything in the latest stable release.

0 commit comments

Comments
 (0)