chore: improve web load robustness by detecting boot crashes and reload after cache reset

Author: abose

src/index.html

@@ -349,7 +349,7 @@
         Note that this cannot be moved to a separate file and should be in index.html due to above reason.
         This is a nuke cache option. Though mostly safe, Use it wisely to prevent slow startup when resetting.
         */
-        async function _resetCacheIfNeeded() {
+        async function _resetCacheIfNeeded(force) {
             if(Phoenix.browser.isTauri || Phoenix.isTestWindow) {
                 // no web cache in desktop builds ot test windows
                 return;
@@ -416,7 +416,7 @@
                 console.log("No cache detected, ignore if first boot or safari ITP.");
                 await _writeFileSafe(WEB_CACHE_FILE_PATH, PHOENIX_APP_CACHE_VERSION);
                 // on next reload, the new cache will be active and populated by service worker
-            } else if(currentCacheName !== PHOENIX_APP_CACHE_VERSION) {
+            } else if(currentCacheName && (currentCacheName !== PHOENIX_APP_CACHE_VERSION || force)) {
                 // This is an upgrade we could in the future use the cacheManifest.json to increase update speed
                 // by only selectively updating the changed files.
                 console.log("Old version cache detected", currentCacheName, "purging and setting", PHOENIX_APP_CACHE_VERSION);

src/main.js

@@ -151,16 +151,26 @@ define(function (require) {
     require(["utils/Metrics", "utils/Compatibility", "utils/EventDispatcher"], function () {
         window.Metrics = require("utils/Metrics");
         // Load the brackets module. This is a self-running module that loads and runs the entire application.
-        try{
-            require(["brackets"]);
-        } catch (err) {
-            // try a cache reset
-            window._resetCacheIfNeeded && window._resetCacheIfNeeded();
+        require(["brackets"], ()=>{}, (err)=>{
             // metrics api might not be available here as we were seeing no metrics raised. Only bugsnag there.
             window.logger && window.logger.reportError(err,
                 'Critical error when loading brackets. Trying to reload again.');
-            // wait for 3 seconds for bugsnag to send report.
-            setTimeout(window.location.reload, 3000);
-        }
+            alert("Oops! Something went wrong. Trying to restart app...");
+            // try a cache reset
+            if(window._resetCacheIfNeeded){
+                window._resetCacheIfNeeded(true)
+                    .finally(()=>{
+                        // wait for 3 seconds for bugsnag to send report.
+                        setTimeout(()=>{
+                            location.reload();
+                        }, 3000);
+                    });
+            } else {
+                // wait for 3 seconds for bugsnag to send report.
+                setTimeout(()=>{
+                    location.reload();
+                }, 3000);
+            }
+        });
     });
 });