Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Greptile Summary

Fixes a startup crash on pre-API 31 devices (e.g., Samsung Galaxy S8 on API 26-28) caused by class verification of RNodeCompanionService, which extends CompanionDeviceService — a class that only exists on API 31+. The fix uses Android's resource-qualified android:enabled attribute pattern to disable the service component by default (values/enable_companion_service = false) and enable it only on API 31+ (values-v31/enable_companion_service = true).

• The manifest now references @bool/enable_companion_service for the service's android:enabled attribute
• Two new resource files provide the API-level-qualified boolean values
• No behavior change on API 31+ devices; existing registerExistingCompanionDevices() in ColumbaApplication is already guarded by API 33+ checks
• This is a well-established Android pattern for preventing class verification of manifest-declared components on older API levels

Confidence Score: 5/5

• This PR is safe to merge — it uses a standard Android pattern with no risk to API 31+ behavior.
• The change is minimal, uses a well-established Android resource-qualification pattern, and all existing code paths that interact with RNodeCompanionService are already guarded by appropriate API level checks (API 33+ in ColumbaApplication, @RequiresApi(S) on the service class itself). No logic changes, no new code paths, no risk of regression.
• No files require special attention.

Important Files Changed

Flowchart

flowchart TD
    A[App Startup] --> B{Device API Level?}
    B -->|API < 31| C["values/companion_service.xml\nenable_companion_service = false"]
    B -->|API >= 31| D["values-v31/companion_service.xml\nenable_companion_service = true"]
    C --> E["RNodeCompanionService\nandroid:enabled = false"]
    D --> F["RNodeCompanionService\nandroid:enabled = true"]
    E --> G["Service skipped by system\nNo class verification crash"]
    F --> H["Service available for\nCompanion Device binding"]
    H --> I["System binds service when\nassociated RNode appears"]

_{Last reviewed commit: 5ac5afb}

@brothercorvo this should fix your samsung S8 issue