From 85d7858a0ef4b4f3ee60eb2bd5621e4899195935 Mon Sep 17 00:00:00 2001
From: Adam Soltys <adam@coinos.io>
Date: Sat, 12 Sep 2026 15:47:56 -0700
Subject: The wake had to be a job, not a foreground service — Android says so.
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Tested on a real phone, and the first attempt didn't work at all:

  Background started FGS: Disallowed [... code:DENIED ...]
  startForegroundService() not allowed due to mAllowStartForeground false

Android 12 forbids starting a foreground service from a broadcast receiver.
An expedited job is the sanctioned replacement — it runs promptly from the
background and needs no notification of its own to apologise for existing —
so WakeService is now WakeJob, and the notification is decided by what
arrived rather than by whether we could wake: a payment or a message is news
either way, a wallet-connect request the wallet answers by itself is not.

Two more things the phone found. A job with a connectivity constraint needs
ACCESS_NETWORK_STATE, or schedule() throws and nothing runs at all. And a
wake with nothing to answer used to hold a WebView open for the full thirty
seconds — a locked wallet, no wallet on the device, or no NWC connections all
end the job immediately now.

Also a bug worth naming: ctx carried `wakeDone` by shorthand, which captured
the binding while it was still undefined (ctx is built before it's defined).
The page could never say it had finished, so every wake ran its full timeout.
It reads through a closure now, like its neighbour.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019mrt228Upn9nNCwGixzSLx
---
 android/app/src/webview/AndroidManifest.xml | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

(limited to 'android/app/src/webview/AndroidManifest.xml')

diff --git a/android/app/src/webview/AndroidManifest.xml b/android/app/src/webview/AndroidManifest.xml
index e88217b..0942377 100644
--- a/android/app/src/webview/AndroidManifest.xml
+++ b/android/app/src/webview/AndroidManifest.xml
@@ -3,11 +3,17 @@
      See MainActivity for why that trade exists. -->
 <manifest xmlns:android="http://schemas.android.com/apk/res/android">
   <uses-permission android:name="android.permission.INTERNET" />
+  <!-- the wake job asks to run when there's a network; the scheduler won't
+       take that constraint without this -->
+  <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />
   <uses-permission android:name="android.permission.CAMERA" />
-  <uses-permission android:name="android.permission.FOREGROUND_SERVICE" />
-  <uses-permission android:name="android.permission.FOREGROUND_SERVICE_DATA_SYNC" />
   <uses-feature android:name="android.hardware.camera" android:required="false" />
-  <application android:usesCleartextTraffic="false">
+  <application
+      android:label="@string/app_name"
+      android:icon="@mipmap/ic_launcher"
+      android:theme="@style/AppTheme"
+      android:supportsRtl="true"
+      android:usesCleartextTraffic="false">
     <activity
         android:name="io.coinos.app.MainActivity"
         android:exported="true"
@@ -43,8 +49,8 @@
     </activity>
     <!-- the off-screen wallet a push wakes -->
     <service
-        android:name="io.coinos.app.WakeService"
+        android:name="io.coinos.app.WakeJob"
         android:exported="false"
-        android:foregroundServiceType="dataSync" />
+        android:permission="android.permission.BIND_JOB_SERVICE" />
   </application>
 </manifest>
-- 
cgit v1.2.3