From 3526e5fd1285245fe5d08d24f658f0c61170c42a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?V=C3=ADctor=20Falc=C3=B3n?= Date: Sun, 10 May 2026 17:45:47 +0100 Subject: [PATCH] Fix service worker registration rejection (#376) ## Sentry - Fixes PHP-LARAVEL-1Q - https://whisper-money.sentry.io/issues/PHP-LARAVEL-1Q ## Root cause - Service worker registration promise rejected in production and had no rejection handler. - Browser reported unhandled promise rejection: Error: Rejected. ## Fix - Add a catch handler to service worker registration so unsupported/intercepted registration failures do not become unhandled global errors. - Add regression coverage asserting registration has rejection handling. ## Verification - vendor/bin/pint --dirty --format agent - php artisan test --compact tests/Feature/PwaTest.php --- resources/views/app.blade.php | 2 +- tests/Feature/PwaTest.php | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/resources/views/app.blade.php b/resources/views/app.blade.php index 6a64a714..3500dd8a 100644 --- a/resources/views/app.blade.php +++ b/resources/views/app.blade.php @@ -66,7 +66,7 @@ diff --git a/tests/Feature/PwaTest.php b/tests/Feature/PwaTest.php index edea1b5f..67a2eee9 100644 --- a/tests/Feature/PwaTest.php +++ b/tests/Feature/PwaTest.php @@ -20,5 +20,6 @@ test('app template includes pwa meta tags and service worker registration', func ->assertSee('content="default"', false) ->assertDontSee('black-translucent', false) ->assertSee('serviceWorker', false) + ->assertSee("navigator.serviceWorker.register('/sw.js', { scope: '/' }).catch", false) ->assertSee('viewport-fit=cover', false); });