chore(demo): remove E2E masking utility

Author: fadi-george

.github/actions/setup-demo/action.yml

@@ -34,7 +34,6 @@ runs:
         echo "VITE_ONESIGNAL_APP_ID=${{ inputs.onesignal-app-id }}" > .env
         echo "VITE_ONESIGNAL_API_KEY=${{ inputs.onesignal-api-key }}" >> .env
         echo "VITE_ONESIGNAL_ANDROID_CHANNEL_ID=7ec2ece9-c538-4656-9516-1316f48a005c" >> .env
-        echo "VITE_E2E_MODE=true" >> .env
 
     - name: Install and set up demo
       shell: bash

examples/build.md

@@ -274,10 +274,8 @@ examples/
     │   │       ├── CustomEventsSection.tsx
     │   │       ├── LocationSection.tsx
     │   │       └── LiveActivitySection.tsx
-    │   ├── theme/
-    │   │   └── variables.css
-    │   └── utils/
-    │       └── maskValue.ts        # E2E_MODE bullet masking helper
+    │   └── theme/
+    │       └── variables.css
     ├── android/                    # Capacitor Android project
     └── ios/
         └── App/

examples/demo/.env.example

@@ -1,4 +1,3 @@
 VITE_ONESIGNAL_APP_ID=your_onesignal_app_id
 VITE_ONESIGNAL_API_KEY=your_rest_api_key
-VITE_ONESIGNAL_ANDROID_CHANNEL_ID=
-VITE_E2E_MODE=false
+VITE_ONESIGNAL_ANDROID_CHANNEL_ID=

examples/demo/README.md

@@ -31,7 +31,6 @@ cp .env.example .env
 VITE_ONESIGNAL_APP_ID=<your_app_id>
 VITE_ONESIGNAL_API_KEY=<your_rest_api_key>
 VITE_ONESIGNAL_ANDROID_CHANNEL_ID=
-VITE_E2E_MODE=false
 ```
 
 ---

examples/demo/src/components/sections/AppSection.tsx

@@ -1,7 +1,6 @@
 import { IonToggle } from '@ionic/react';
 import type { FC } from 'react';
 
-import { maskValue } from '../../utils/maskValue';
 import SectionCard from '../SectionCard';
 
 interface AppSectionProps {
@@ -24,7 +23,7 @@ const AppSection: FC<AppSectionProps> = ({
       <div className="kv-row">
         <span>App ID</span>
         <span className="id-value" data-testid="app_id_value">
-          {maskValue(appId)}
+          {appId}
         </span>
       </div>
     </div>

examples/demo/src/components/sections/PushSection.tsx

@@ -1,7 +1,6 @@
 import { IonToggle } from '@ionic/react';
 import type { FC } from 'react';
 
-import { maskValue } from '../../utils/maskValue';
 import ActionButton from '../ActionButton';
 import SectionCard from '../SectionCard';
 
@@ -27,7 +26,7 @@ const PushSection: FC<PushSectionProps> = ({
       <div className="kv-row">
         <span>Push ID</span>
         <span className="id-value" data-testid="push_id_value">
-          {maskValue(pushSubscriptionId ?? '—')}
+          {pushSubscriptionId ?? '—'}
         </span>
       </div>
       <div className="divider" />

examples/demo/src/hooks/useOneSignal.ts

@@ -59,8 +59,6 @@ function initOneSignal(): void {
   if (storedExternalUserId) {
     void OneSignal.login(storedExternalUserId);
   }
-
-  console.log(`OneSignal initialized with app ID: ${RESOLVED_APP_ID}`);
 }
 initOneSignal();
 
@@ -91,6 +89,7 @@ export type UseOneSignalReturn = {
   consentRequired: boolean;
   privacyConsentGiven: boolean;
   externalUserId: string | undefined;
+  oneSignalId: string | undefined;
   pushSubscriptionId: string | undefined;
   isPushEnabled: boolean;
   hasNotificationPermission: boolean;
@@ -151,6 +150,7 @@ export function useOneSignal(): UseOneSignalReturn {
     preferences.getConsentGiven(),
   );
   const [externalUserId, setExternalUserId] = useState<string | undefined>(undefined);
+  const [oneSignalId, setOneSignalId] = useState<string | undefined>(undefined);
   const [pushSubscriptionId, setPushSubscriptionId] = useState<string | undefined>(undefined);
   const [isPushEnabled, setIsPushEnabled] = useState(false);
   const [hasNotificationPermission, setHasNotificationPermission] = useState(false);
@@ -208,22 +208,6 @@ export function useOneSignal(): UseOneSignalReturn {
 
     const handleNotificationClick = (e: NotificationClickEvent) => {
       console.log(`Notification click: ${e.notification.title ?? ''}`);
-      // Persist to localStorage so cold-start clicks are still inspectable
-      // after the Safari Web Inspector reattaches to the WKWebView.
-      try {
-        const existing = JSON.parse(localStorage.getItem('lastNotificationClicks') ?? '[]');
-        existing.push({
-          notificationId: e.notification.notificationId,
-          title: e.notification.title ?? null,
-          body: e.notification.body ?? null,
-          actionId: e.result.actionId ?? null,
-          url: e.result.url ?? null,
-          receivedAt: new Date().toISOString(),
-        });
-        localStorage.setItem('lastNotificationClicks', JSON.stringify(existing.slice(-20)));
-      } catch (err) {
-        console.warn('Failed to persist notification click to localStorage', err);
-      }
     };
 
     const handleForegroundWillDisplay = (e: NotificationWillDisplayEvent) => {
@@ -257,6 +241,8 @@ export function useOneSignal(): UseOneSignalReturn {
         `User changed: onesignalId=${nextOnesignalId ?? 'null'}, externalId=${event.current.externalId ?? 'null'}`,
       );
 
+      setOneSignalId(nextOnesignalId ?? undefined);
+
       if (nextOnesignalId === null) return;
       void fetchUserDataFromApi();
     };
@@ -283,6 +269,7 @@ export function useOneSignal(): UseOneSignalReturn {
       if (cancelled) return;
 
       setExternalUserId(externalId ?? preferences.getExternalUserId() ?? undefined);
+      setOneSignalId(initialOnesignalId ?? undefined);
       setPushSubscriptionId(pushId ?? undefined);
       setIsPushEnabled(pushOptedIn);
       setHasNotificationPermission(hasPerm);
@@ -550,6 +537,7 @@ export function useOneSignal(): UseOneSignalReturn {
     consentRequired,
     privacyConsentGiven,
     externalUserId,
+    oneSignalId,
     pushSubscriptionId,
     isPushEnabled,
     hasNotificationPermission,

examples/demo/src/pages/HomeScreen.css

@@ -150,6 +150,8 @@
 .kv-card .kv-row .id-value {
   font-size: var(--font-size-body-small);
   font-family: monospace;
+  user-select: all;
+  -webkit-user-select: all;
 }
 
 .divider {

examples/demo/src/services/OneSignalApiService.ts

@@ -82,7 +82,8 @@ class OneSignalApiService {
       ...extra,
     };
 
-    const maxAttempts = 3;
+    const maxAttempts = 5;
+    const backoffMs = (n: number) => 2_000 * 2 ** (n - 1);
 
     // Retry on `invalid_player_ids` to absorb the brief race where the
     // subscription has been created locally but is not yet visible to the
@@ -106,7 +107,7 @@ class OneSignalApiService {
         const invalidIds = response.data?.errors?.invalid_player_ids;
         if (Array.isArray(invalidIds) && invalidIds.length > 0) {
           if (attempt < maxAttempts) {
-            await new Promise((resolve) => setTimeout(resolve, 3_000 * attempt));
+            await new Promise((resolve) => setTimeout(resolve, backoffMs(attempt)));
             continue;
           }
           console.error(