Merge pull request #2136 from keymapperorg/develop

Version 4.1.0

Author: sds100

.cursor/rules/keymapper-add-action-checklist.mdc

@@ -0,0 +1,24 @@
+---
+description: Required checklist when adding a new Action
+alwaysApply: true
+---
+
+# Adding A New Action
+
+Before implementing, confirm whether the action is editable.
+- Branch names must follow `feature/123-branch-name` or `fix/123-branch-name`.
+
+Then follow this sequence:
+
+1. Add an ID in `ActionId`.
+2. Add a new `ActionData` sealed class variant.
+3. Map entity conversions in `ActionDataEntityMapper`.
+4. Assign category in `ActionUtils`.
+5. If editable, add it to `ActionUtils.isEditable`.
+6. Add title string in `strings.xml`.
+7. Add title and Compose icon in `ActionUtils` (ignore drawables).
+8. Add title in `ActionUiHelper`.
+9. Add execution handling in `PerformActionsUseCase`.
+10. Handle creation in `CreateActionDelegate`.
+
+Do not delete existing action code; follow existing names and place additions near similar actions.

.cursor/rules/keymapper-architecture.mdc

@@ -0,0 +1,15 @@
+---
+description: Module boundaries and architectural conventions for KeyMapper
+alwaysApply: true
+---
+
+# KeyMapper Architecture
+
+- Branch names must follow `feature/123-branch-name` or `fix/123-branch-name`.
+- Treat the codebase as Clean Architecture + MVVM with unidirectional data flow.
+- Keep dependency direction as `:app` -> `:base` -> `:common`/`:data`/`:system`/`:sysbridge`.
+- Keep domain models in `:common` and persistence entities in `:data` with explicit mappers.
+- Put business logic in use cases and inject them into ViewModels.
+- Expose UI state via `Flow`/`StateFlow`; collect from UI and send events back to ViewModels.
+- Use Hilt for DI and preserve app-specific binding overrides in app modules.
+- Use coroutines for async work and `viewModelScope` for ViewModel-owned jobs.

.cursor/rules/keymapper-build-and-tooling.mdc

@@ -0,0 +1,16 @@
+---
+description: Build, test, and tooling commands for KeyMapper
+alwaysApply: true
+---
+
+# KeyMapper Build And Tooling
+
+- Branch names must follow `feature/123-branch-name` or `fix/123-branch-name`.
+- Use these common commands:
+  - `./gradlew assembleDebug`
+  - `./gradlew test`
+  - `./gradlew connectedAndroidTest`
+  - `./gradlew ktlintCheck` or `./gradlew ktlintFormat`
+- Release builds require `KEYSTORE_PASSWORD` and `KEY_PASSWORD` environment variables.
+- Target platform constraints: min SDK 26, target SDK 36, Java 11.
+- Dependency versions are managed in `gradle/libs.versions.toml`.

.cursor/rules/keymapper-commit-messages.mdc

@@ -0,0 +1,34 @@
+---
+description: Commit message format convention for KeyMapper
+alwaysApply: true
+---
+
+# KeyMapper Commit Messages
+
+Use this format for commits:
+
+`#<issue_number> <type>: <description>`
+
+Allowed types:
+
+- `feat`
+- `fix`
+- `chore`
+- `refactor`
+- `style`
+
+Example:
+
+`#2025 feat: add button to report bug on home screen`
+
+# Branch Naming Convention
+
+- Use `feature/123-branch-name` for new features.
+- Use `fix/123-branch-name` for bug fixes.
+
+# Pull Request Changelog Requirement
+
+- When preparing a PR, update `CHANGELOG.md` for any user-facing fix/feature/change or notable
+  internal change.
+- If no changelog entry is needed (for example, docs-only or CI-only), explicitly note that in the PR
+  description.

.cursor/rules/keymapper-compose-guidelines.mdc

@@ -0,0 +1,16 @@
+---
+description: Compose coding conventions used in KeyMapper
+globs: "**/*.kt"
+alwaysApply: false
+---
+
+# KeyMapper Compose Guidelines
+
+Apply this when editing Compose code:
+
+- Branch names must follow `feature/123-branch-name` or `fix/123-branch-name`.
+- Put `Modifier` as the first optional parameter after required parameters.
+- For non-standard colors, use `LocalCustomColorsPalette` instead of `MaterialTheme.colorScheme`.
+- Check `KeyMapperIcons` before introducing icons from other sources.
+- Use `LocalUriHandler.openUriSafe` for URL launching and do not hoist URL launching logic.
+- Prefer imports; avoid fully qualified names in Compose call sites.

CHANGELOG.md

@@ -1,3 +1,26 @@
+## [4.1.0](https://github.com/sds100/KeyMapper/releases/tag/v4.1.0)
+
+#### 10 May 2026
+
+## Added
+
+- #2067 add action to select all text in the focused field.
+- #2045 add action to input on-screen keyboard enter/send button.
+- #2106 disable the keyboard auto-switching setting when manually switching the keyboard in the Key Mapper homescreen menu.
+- #1029 add action to show a toast message.
+- #2081 add getevent debug screen.
+- #2087 small segmented button text is not readable in dark mode.
+- #2077 rename "This device" and "any device" to "This Android device" and "Any input device" to prevent confusion.
+
+## Changed
+
+- Use a more compact navigation bar on the home screen.
+
+## Fixed
+
+- #2091 show an error on the "open device assistant" action when no device assistant is installed.
+- #2107 clarify the crashed accessibility service dialog text and keep only Cancel/Restart actions.
+
 ## [4.0.5](https://github.com/sds100/KeyMapper/releases/tag/v4.0.5)
 
 #### 26 February 2026

CLAUDE.md

@@ -0,0 +1,109 @@
+# KeyMapper
+
+## Project Overview
+
+Android app for custom key/gamepad remapping, macros, and on-screen buttons. Supports Android 8.0+ (minSdk 26), targets API 36. Distributed on Google Play and F-Droid (FOSS flavor).
+
+- **Package:** `io.github.sds100.keymapper`
+- **Languages:** Kotlin (primary), Rust (event device handling), C++ (system bridge JNI)
+
+## Build Commands
+
+```bash
+./gradlew assembleDebug       # debug APK
+./gradlew assembleCi          # CI build (minified)
+./gradlew assembleRelease     # release build (requires KEYSTORE_PASSWORD and KEY_PASSWORD env vars)
+./gradlew clean               # clean build outputs
+```
+
+## Test & Lint Commands
+
+```bash
+# Kotlin unit tests
+./gradlew testDebugUnitTest
+./gradlew :base:testDebugUnitTest   # single module
+
+# Kotlin lint
+./gradlew ktlintCheck         # check
+./gradlew ktlintFormat        # auto-fix
+
+# Rust (run from evdev/src/main/rust/evdev_manager)
+cargo fmt --check
+cargo test --package evdev_manager_core
+```
+
+CI runs all of the above on every PR (see `.github/workflows/pull-request.yml`).
+
+## Architecture & Modules
+
+Multi-module Gradle project using MVVM, Hilt DI, Room, StateFlow/Flow, Jetpack Compose, and Coroutines.
+
+| Module | Responsibility |
+|---|---|
+| `app/` | Entry point: `MainActivity`, `KeyMapperApp`, `MyAccessibilityService` |
+| `base/` | UI screens and ViewModels (actions, constraints, triggers, keymaps, settings) |
+| `common/` | Shared utilities and models |
+| `data/` | Room database, repositories, DataStore, migrations |
+| `system/` | Device APIs, permission management |
+| `sysbridge/` | C++ JNI via CMake for elevated system access |
+| `evdev/` | Rust event device handling + JNI bindings |
+| `api/` | AIDL public interface definitions |
+| `systemstubs/` | Mock stubs for system classes used in tests |
+
+Most business logic lives in `base/src/main/java/io/github/sds100/keymapper/`.
+
+## Code Style
+
+- **Kotlin:** ktlint with android_studio code style (configured via `.editorconfig`)
+  - No function expression bodies
+  - Trailing commas allowed
+  - Run `./gradlew ktlintFormat` before committing
+- **Rust:** `cargo fmt`
+
+## Commit Message Convention
+
+```
+#<issue_number> <type>: <description>
+```
+
+Types: `feat`, `fix`, `chore`, `refactor`, `style`
+
+Example: `#2025 feat: add button to report bug on home screen`
+
+## Branch Naming Convention
+
+- Use `feature/123-branch-name` for new features.
+- Use `fix/123-branch-name` for bug fixes.
+
+## Pull Request Requirement
+
+- Any user-facing behavior change, fix, feature, or notable internal change in a PR must include an
+  update to `CHANGELOG.md`.
+- If there is truly no changelog impact (for example, docs-only or CI-only changes), explicitly state
+  that in the PR description.
+
+## Compose Guidelines
+
+- `Modifier` is always the first parameter after required params
+- Use `LocalCustomColorsPalette` (not `MaterialTheme.colorScheme`) for non-standard colors
+- Check `KeyMapperIcons` before using other icon sources
+- Use `LocalUriHandler.openUriSafe` extension for URL launching — do not hoist URL launching logic up the call stack
+- Use import statements; never use fully qualified names in Compose code
+- Write `@Preview` composables for every screen
+
+## Adding a New Action (10-step checklist)
+
+First determine whether the action is editable. Then:
+
+1. Add a new ID to `ActionId`
+2. Create a new `ActionData` sealed class variant
+3. Map to/from entity in `ActionDataEntityMapper`
+4. Assign a category in `ActionUtils`
+5. If editable, add to `isEditable` in `ActionUtils`
+6. Add a title string in `strings.xml`
+7. Add title and Compose icon in `ActionUtils` (ignore drawables)
+8. Add title in `ActionUiHelper`
+9. Stub out execution in `PerformActionsUseCase`
+10. Handle creation in `CreateActionDelegate`
+
+Do not delete existing action code. Follow existing naming conventions exactly. Add new code near similar existing actions.

app/src/main/java/io/github/sds100/keymapper/purchasing/PurchasingManagerImpl.kt

@@ -1,34 +1,50 @@
 package io.github.sds100.keymapper.purchasing
 
-import io.github.sds100.keymapper.base.purchasing.ProductId
 import io.github.sds100.keymapper.base.purchasing.PurchasingError
 import io.github.sds100.keymapper.base.purchasing.PurchasingManager
+import io.github.sds100.keymapper.base.purchasing.RevenueCatEntitlementId
 import io.github.sds100.keymapper.common.utils.KMResult
 import io.github.sds100.keymapper.common.utils.State
 import kotlinx.coroutines.flow.Flow
 import kotlinx.coroutines.flow.MutableSharedFlow
 import kotlinx.coroutines.flow.MutableStateFlow
 
 class PurchasingManagerImpl : PurchasingManager {
-    override val onCompleteProductPurchase: MutableSharedFlow<ProductId> = MutableSharedFlow()
-    override val purchases: Flow<State<KMResult<Set<ProductId>>>> =
+    override val onCompleteProductPurchase: MutableSharedFlow<RevenueCatEntitlementId> =
+        MutableSharedFlow()
+    override val entitlements: Flow<State<KMResult<Set<RevenueCatEntitlementId>>>> =
         MutableStateFlow(State.Data(PurchasingError.PurchasingNotImplemented))
 
-    override suspend fun launchPurchasingFlow(product: ProductId): KMResult<Unit> {
+    override suspend fun launchPurchasingFlow(
+        packageId: String,
+        verifyEntitlements: Array<RevenueCatEntitlementId>,
+    ): KMResult<Unit> {
         return PurchasingError.PurchasingNotImplemented
     }
 
-    override suspend fun getProductPrice(product: ProductId): KMResult<String> {
+    override suspend fun isPackagePurchased(packageId: String): KMResult<Boolean> {
         return PurchasingError.PurchasingNotImplemented
     }
 
-    override suspend fun isPurchased(product: ProductId): KMResult<Boolean> {
+    override suspend fun getNonSubscriptionPurchaseCount(packageId: String): KMResult<Int> {
         return PurchasingError.PurchasingNotImplemented
     }
 
-    override suspend fun getMetadata(): KMResult<Map<String, Any>> {
+    override suspend fun getPackagePrice(packageId: String): KMResult<String> {
+        return PurchasingError.PurchasingNotImplemented
+    }
+
+    override suspend fun hasEntitlement(entitlement: RevenueCatEntitlementId): KMResult<Boolean> {
+        return PurchasingError.PurchasingNotImplemented
+    }
+
+    override suspend fun getCurrentOfferingId(): KMResult<String?> {
         return PurchasingError.PurchasingNotImplemented
     }
 
     override fun refresh() {}
+
+    override fun trackCustomPaywallImpression(paywallIdentifier: String) {
+        // Purchasing is not available in FOSS.
+    }
 }

app/src/main/java/io/github/sds100/keymapper/purchasing/RevenueCatPackageId.kt

@@ -0,0 +1,8 @@
+package io.github.sds100.keymapper.purchasing
+
+/**
+ * Placeholder for RevenueCat package identifiers.
+ *
+ * Purchasing is not available in the FOSS build.
+ */
+enum class RevenueCatPackageId