Greptile Summary

This PR fixes a bug where notifications for the active conversation were incorrectly suppressed even when the screen was locked or the app was backgrounded, by adding a ProcessLifecycleOwner foreground check that gates the existing suppression logic. The lifecycle-process dependency is added, and an @VisibleForTesting-annotated isAppInForeground lambda is introduced on NotificationHelper to keep the new behavior unit-testable via Robolectric.

Key changes:

• NotificationHelper: Suppression now requires both isAppInForeground() and the active-conversation hash match (line 166), correctly sending notifications when the screen is off or the app is in the background.
• app/build.gradle.kts: Adds libs.lifecycle.process to provide ProcessLifecycleOwner.
• NotificationHelperTest: Updated the existing suppression test and added a new test for the backgrounded case; however, two pre-existing "not suppressed" tests do not override isAppInForeground, causing them to pass trivially in Robolectric (where the process lifecycle never reaches STARTED) rather than validating the intended conversation-hash logic.

Confidence Score: 3/5

• Core logic change is sound, but a pre-existing threshold concern and two under-specified tests leave minor gaps before merging.
• The production fix is minimal and well-scoped. The STARTED threshold (vs RESUMED) concern noted in a previous thread remains unresolved, leaving a brief window where notifications can still be suppressed after the screen locks. Additionally, two pre-existing Robolectric tests now pass for the wrong reason, reducing confidence in test coverage of the foreground-but-wrong-conversation path.
• NotificationHelperTest.kt — pre-existing suppression tests need isAppInForeground = { true } to be meaningful after this refactor.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[notifyMessageReceived called] --> B{Master toggle on?}
    B -- No --> Z[return — no notification]
    B -- Yes --> C{Message notifications enabled?}
    C -- No --> Z
    C -- Yes --> D{Has notification permission?}
    D -- No --> Z
    D -- Yes --> E{isAppInForeground\nAND\nactiveConversation == hash?}
    E -- Yes → suppress --> Z
    E -- No → allow --> F[Build & post notification]

_{Last reviewed commit: 7c4713c}

@greptileai