sitespeed.io 37.1.0 → 37.1.1

package/CHANGELOG.md

@@ -1,6 +1,10 @@
 
 # CHANGELOG - sitespeed.io  (we use [semantic versioning](https://semver.org))
 
+## 37.1.1 - 2025-03-10
+### Fixed
+* Multiple fixes to better handle if a user journey scriptimng fails [#4474](https://github.com/sitespeedio/sitespeed.io/pull/4474) and [#4475](https://github.com/sitespeedio/sitespeed.io/pull/4475).
+
 ## 37.1.0 - 2025-03-08
 ### Fixed
 * Simplified logic for uploading files to S3 since we have reports that it do not work 100% [#4472](https://github.com/sitespeedio/sitespeed.io/pull/4472).

package/lib/core/resultsStorage/pathToFolder.js

@@ -17,7 +17,12 @@ export function pathToFolder(url, options, alias) {
 
   const pathSegments = [];
   const urlSegments = [];
-  pathSegments.push('pages', parsedUrl.hostname.split('.').join('_'));
+
+  // If a measurements fail and we use a script file and no URL
+  // has been tested, we don't have a hostname
+  if (parsedUrl.hostname) {
+    pathSegments.push('pages', parsedUrl.hostname.split('.').join('_'));
+  }
 
   if (options.urlMetaData && options.urlMetaData[url]) {
     pathSegments.push(options.urlMetaData[url]);

package/lib/plugins/browsertime/index.js

@@ -185,7 +185,7 @@ export default class BrowsertimePlugin extends SitespeedioPlugin {
           // https://github.com/sitespeedio/sitespeed.io/issues/2341
           for (const element of result) {
             // Browsertime supports alias for URLS in a script
-            const alias = element.info.alias;
+            const alias = element.info?.alias;
             if (alias) {
               if (this.scriptOrMultiple) {
                 url = element.info.url;
@@ -207,14 +207,32 @@ export default class BrowsertimePlugin extends SitespeedioPlugin {
             resultIndex++
           ) {
             axeAggregatorPerURL = new AxeAggregator(this.options);
+            // Send errors from Browsertime as early as possible
+            // Check for errors. Browsertime errors is an array of all iterations
+            // [[],[],[]] where one iteration can have multiple errors
+            for (let errorsForOneIteration of result[resultIndex].errors) {
+              if (Array.isArray(errorsForOneIteration)) {
+                for (let error of errorsForOneIteration) {
+                  super.sendMessage('error', error, _merge({ url }));
+                }
+              } else {
+                super.sendMessage(
+                  'error',
+                  `${errorsForOneIteration} ${result[resultIndex].failureMessages}`,
+                  _merge({ url })
+                );
+              }
+            }
             // If we use scripts or multiple, use the URL from the tested page
             // so that we can handle click on links etc
             // see https://github.com/sitespeedio/sitespeed.io/issues/2260
             // we could change the plugins but since they do not work with
             // multiple/scripting lets do it like this for now
             if (this.scriptOrMultiple) {
-              url = result[resultIndex].info.url;
-              group = parse(url).hostname;
+              url = result[resultIndex].info?.url;
+              if (url) {
+                group = parse(url).hostname;
+              }
             }
             let runIndex = 0;
             for (let browserScriptsData of result[resultIndex].browserScripts) {
@@ -597,14 +615,6 @@ export default class BrowsertimePlugin extends SitespeedioPlugin {
                 group
               });
             }
-
-            // Check for errors. Browsertime errors is an array of all iterations
-            // [[],[],[]] where one iteration can have multiple errors
-            for (let errorsForOneIteration of result[resultIndex].errors) {
-              for (let error of errorsForOneIteration) {
-                super.sendMessage('error', error, _merge({ url }));
-              }
-            }
           }
           break;
         } catch (error) {

package/npm-shrinkwrap.json

@@ -1,12 +1,12 @@
 {
   "name": "sitespeed.io",
-  "version": "37.1.0",
+  "version": "37.1.1",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "sitespeed.io",
-      "version": "37.1.0",
+      "version": "37.1.1",
       "hasInstallScript": true,
       "license": "MIT",
       "dependencies": {

package/package.json

@@ -5,7 +5,7 @@
     "sitespeed.io": "./bin/sitespeed.js",
     "sitespeed.io-wpr": "./bin/browsertimeWebPageReplay.js"
   },
-  "version": "37.1.0",
+  "version": "37.1.1",
   "description": "sitespeed.io is an open-source tool for comprehensive web performance analysis, enabling you to test, monitor, and optimize your website’s speed using real browsers in various environments.",
   "keywords": [
     "performance",