Merge pull request #992 from codemonkey85/dev

Retry dotnetjs boot resource; add reload hint for network startup errors

Author: codemonkey85

Pkmds.Web/wwwroot/index.html

@@ -185,11 +185,19 @@
 
         Blazor.start({
             loadBootResource: function (type, name, defaultUri, integrity) {
-                // The 'dotnetjs' module is imported as a script and must be
-                // returned as a URL string, not a Response — leave it to the
-                // default loader.
                 if (type === 'dotnetjs') {
-                    return undefined;
+                    // dotnetjs must be returned as a URL string (or Promise<string>),
+                    // not a Response. Pre-fetch with retry (and SRI when integrity is
+                    // supplied) so a transient failure gets multiple attempts; reject
+                    // explicitly on non-OK so bootstrap fails with a clear error rather
+                    // than silently returning the URL and letting the script import fail
+                    // generically later (issue #991).
+                    return retryFetchBootResource(defaultUri, integrity).then(function (response) {
+                        if (!response.ok) {
+                            throw new TypeError('dotnetjs load failed: HTTP ' + response.status);
+                        }
+                        return defaultUri;
+                    });
                 }
                 return retryFetchBootResource(defaultUri, integrity);
             }

Pkmds.Web/wwwroot/js/error-reporting.js

@@ -82,7 +82,21 @@
             ui.insertBefore(details, msgEl.nextSibling);
         }
 
-        // Button row.
+        // For network-layer startup failures, add a reload hint before the
+        // button row so users know what to try before filing a bug.
+        var isNetworkError = (
+            message === 'Load failed' ||
+            message.indexOf('Failed to fetch') !== -1 ||
+            message.indexOf('NetworkError') !== -1
+        );
+        if (isNetworkError) {
+            var hint = document.createElement('p');
+            hint.style.cssText = 'margin: 0.4rem 0 0; font-size: 0.8rem; color: #555;';
+            hint.textContent = 'This is usually a temporary network problem or an app update in progress — reloading typically fixes it.';
+            ui.insertBefore(hint, (stack ? details : msgEl).nextSibling);
+        }
+
+        // Button row — anchor after hint when present (var-hoisted as undefined otherwise).
         var row = document.createElement('div');
         row.style.cssText = 'margin-top: 0.5rem; display: flex; gap: 0.5rem; flex-wrap: wrap; align-items: center;';
 
@@ -120,7 +134,7 @@
         row.appendChild(copyBtn);
         row.appendChild(reportLink);
 
-        var insertAfter = stack ? details : msgEl;
+        var insertAfter = hint || (stack ? details : msgEl);
         ui.insertBefore(row, insertAfter.nextSibling);
     }