browsertime 17.0.0-beta.4 → 17.1.0

package/lib/core/engine/iteration.js

@@ -2,34 +2,34 @@ import { platform } from 'node:os';
 import webdriver from 'selenium-webdriver';
 import intel from 'intel';
 
-import SeleniumRunner from '../seleniumRunner.js';
-import preURL from '../../support/preURL.js';
-import setResourceTimingBufferSize from '../../support/setResourceTimingBufferSize.js';
-import ScreenshotManager from '../../screenshot/index.js';
-import ExtensionServer from '../../extensionserver/index.js';
-import Video from '../../video/video.js';
-import stop from '../../support/stop.js';
-import AddText from './command/addText.js';
-import Click from './command/click.js';
-import Wait from './command/wait.js';
-import Measure from './command/measure.js';
-import JavaScript from './command/javaScript.js';
-import Switch from './command/switch.js';
-import Screenshot from './command/screenshot.js';
-import Set from './command/set.js';
-import Cache from './command/cache.js';
-import Meta from './command/meta.js';
-import StopWatch from './command/stopWatch.js';
-import Select from './command/select.js';
-import Debug from './command/debug.js';
-import AndroidCommand from './command/android.js';
-import ChromeDevelopmentToolsProtocol from './command/chromeDevToolsProtocol.js';
+import { SeleniumRunner } from '../seleniumRunner.js';
+import { preURL } from '../../support/preURL.js';
+import { setResourceTimingBufferSize } from '../../support/setResourceTimingBufferSize.js';
+import { ScreenshotManager } from '../../screenshot/index.js';
+import { ExtensionServer } from '../../extensionserver/index.js';
+import { Video } from '../../video/video.js';
+import { stop } from '../../support/stop.js';
+import { AddText } from './command/addText.js';
+import { Click } from './command/click.js';
+import { Wait } from './command/wait.js';
+import { Measure } from './command/measure.js';
+import { JavaScript } from './command/javaScript.js';
+import { Switch } from './command/switch.js';
+import { Screenshot } from './command/screenshot.js';
+import { Set } from './command/set.js';
+import { Cache } from './command/cache.js';
+import { Meta } from './command/meta.js';
+import { Watch as StopWatch } from './command/stopWatch.js';
+import { Select } from './command/select.js';
+import { Debug } from './command/debug.js';
+import { AndroidCommand } from './command/android.js';
+import { ChromeDevelopmentToolsProtocol } from './command/chromeDevToolsProtocol.js';
 import {
   addConnectivity,
   removeConnectivity
 } from '../../connectivity/index.js';
 import { jsonifyVisualProgress } from '../../support/util.js';
-import flushDNS from '../../support/dns.js';
+import { flushDNS } from '../../support/dns.js';
 import {
   SingleClick,
   DoubleClick,
@@ -37,10 +37,10 @@ import {
   ContextClick,
   MouseMove
 } from './command/mouse/index.js';
-import getNumberOfRunningProcesses from '../../support/processes.js';
+import { getNumberOfRunningProcesses } from '../../support/processes.js';
 import { isAndroidConfigured, Android } from '../../android/index.js';
-import Scroll from './command/scroll.js';
-import Navigation from './command/navigation.js';
+import { Scroll } from './command/scroll.js';
+import { Navigation } from './command/navigation.js';
 const log = intel.getLogger('browsertime');
 const delay = ms => new Promise(res => setTimeout(res, ms));
 
@@ -51,7 +51,7 @@ const ANDROID_DELAY_TIME = 2000;
  * Browsertime will do through one iteration of testing a URL.
  * @class
  */
-export default class Iteration {
+export class Iteration {
   constructor(
     storageManager,
     engineDelegate,

package/lib/core/engine/run.js

@@ -1,4 +1,4 @@
-export default function (urlOrFunctions) {
+export function run(urlOrFunctions) {
   return async function (context, commands) {
     for (let urlOrFunction of urlOrFunctions) {
       await (typeof urlOrFunction === 'function'

package/lib/core/pageCompleteChecks/defaultPageCompleteCheck.js

@@ -1,4 +1,4 @@
-export default `
+export const defaultPageCompleteCheck = `
 return (function(waitTime) {
     try { 
             var end = window.performance.timing.loadEventEnd;

package/lib/core/pageCompleteChecks/pageCompleteCheckByInactivity.js

@@ -1,4 +1,4 @@
-export default `
+export const pageCompleteCheckByInactivity = `
 return (function(waitTime) {
   const timing = window.performance.timing;
   const p = window.performance;

package/lib/core/pageCompleteChecks/spaInactivity.js

@@ -1,4 +1,4 @@
-export default `
+export const spaInactivity = `
 return (function(waitTime) {
   const timing = window.performance.timing;
   const p = window.performance;

package/lib/core/seleniumRunner.js

@@ -5,10 +5,10 @@ import { Condition } from 'selenium-webdriver';
 import { isAndroidConfigured } from '../android/index.js';
 import { UrlLoadError, BrowserError, TimeoutError } from '../support/errors.js';
 import { createWebDriver } from './webdriver/index.js';
-import getViewPort from '../support/getViewPort.js';
-import defaultPageCompleteCheck from './pageCompleteChecks/defaultPageCompleteCheck.js';
-import pageCompleteCheckByInactivity from './pageCompleteChecks/pageCompleteCheckByInactivity.js';
-import spaCheck from './pageCompleteChecks/spaInactivity.js';
+import { getViewPort } from '../support/getViewPort.js';
+import { defaultPageCompleteCheck } from './pageCompleteChecks/defaultPageCompleteCheck.js';
+import { pageCompleteCheckByInactivity } from './pageCompleteChecks/pageCompleteCheckByInactivity.js';
+import { spaInactivity as spaCheck } from './pageCompleteChecks/spaInactivity.js';
 const log = intel.getLogger('browsertime');
 
 const defaults = {
@@ -49,7 +49,7 @@ async function timeout(promise, ms, errorMessage) {
  * Wrapper for Selenium.
  * @class
  */
-class SeleniumRunner {
+export class SeleniumRunner {
   constructor(baseDir, options) {
     this.options = merge({}, defaults, options);
     this.baseDir = baseDir;
@@ -296,10 +296,7 @@ class SeleniumRunner {
             `Could not load ${url} is the web page down?`,
             url
           );
-        } else if (newURI !== startURI) {
-          // We navigated to a new page, we don't need to test anymore
-          break;
-        } else {
+        } else if (newURI === startURI) {
           const waitTime = (this.options.retryWaitTime || 10_000) * (index + 1);
           totalWaitTime += waitTime;
           log.debug(
@@ -322,6 +319,9 @@ class SeleniumRunner {
               'Will check again if the browser has navigated to the page'
             );
           }
+        } else {
+          // We navigated to a new page, we don't need to test anymore
+          break;
         }
       } catch (error) {
         log.info(
@@ -667,5 +667,3 @@ class SeleniumRunner {
     return results;
   }
 }
-
-export default SeleniumRunner;

package/lib/edge/webdriver/builder.js

@@ -4,7 +4,7 @@ import get from 'lodash.get';
 import pick from 'lodash.pick';
 import isEmpty from 'lodash.isempty';
 import { logging } from 'selenium-webdriver';
-import setupChromiumOptions from '../../chrome/webdriver/setupChromiumOptions.js';
+import { setupChromiumOptions } from '../../chrome/webdriver/setupChromiumOptions.js';
 const log = intel.getLogger('browsertime.edge');
 const { pac, manual } = 'selenium-webdriver/proxy.js';
 

package/lib/extensionserver/index.js

@@ -1,5 +1,5 @@
 import { startServer, stopServer } from './httpServer.js';
-import setup from './setup.js';
+import { setup } from './setup.js';
 
 /**
  * Create a ExtensionServer. The extension server helps us with
@@ -7,7 +7,7 @@ import setup from './setup.js';
  * use a WebExtension to prepare our tests.
  * @class
  */
-class ExtensionServer {
+export class ExtensionServer {
   constructor(options) {
     this.options = options;
   }
@@ -48,5 +48,3 @@ class ExtensionServer {
     }
   }
 }
-
-export default ExtensionServer;

package/lib/extensionserver/setup.js

@@ -46,7 +46,7 @@ function generateURL(port, testUrl, options) {
   });
 }
 
-export default async function (url, browser, port, options) {
+export async function setup(url, browser, port, options) {
   const configUrl = generateURL(port, url, options);
   log.debug('Configuring browser plugin via %s', configUrl);
   await browser.loadAndWait(configUrl);

package/lib/firefox/geckoProfiler.js

@@ -6,7 +6,7 @@ import {
   desktop_sampling_interval
 } from './settings/geckoProfilerDefaults.js';
 import { isAndroidConfigured, Android } from '../android/index.js';
-import pathToFolder from '../support/pathToFolder.js';
+import { pathToFolder } from '../support/pathToFolder.js';
 import { BrowserError } from '../support/errors.js';
 const delay = ms => new Promise(res => setTimeout(res, ms));
 const log = intel.getLogger('browsertime.firefox');
@@ -32,7 +32,7 @@ async function timeout(promise, ms, errorMessage) {
     })
   ]);
 }
-class GeckoProfiler {
+export class GeckoProfiler {
   constructor(runner, storageManager, firefoxConfig, options) {
     this.runner = runner;
     this.storageManager = storageManager;
@@ -73,14 +73,38 @@ class GeckoProfiler {
     );
 
     let script;
-    if (parameterLength === 7) {
-      script = `Services.profiler.StartProfiler(${bufferSize},${interval},${featureString},
-        ${chosenFeatures.length},${threadString},${chosenThreads.length});`;
-    } else if (parameterLength === 5 || parameterLength === 6) {
-      script = `Services.profiler.StartProfiler(${bufferSize},${interval},${featureString},${threadString});`;
-    } else {
-      log.error('Unknown Gecko Profiler API');
-      throw new Error('Unknown Gecko Profiler API');
+    switch (parameterLength) {
+      case 7: {
+        // Oldest profiler API. It was deprecated in Firefox 69.
+        script = `Services.profiler.StartProfiler(${bufferSize},${interval},${featureString},
+          ${chosenFeatures.length},${threadString},${chosenThreads.length});`;
+        break;
+      }
+      case 6: {
+        // Newest profiler API. It's being used since Firefox 72.
+        // `activeBrowserId` is useful for distinguishing browser tabs in the profiler UI.
+        const activeBrowserIdScript = `
+          const win = Services.wm.getMostRecentWindow("navigator:browser");
+          return win?.gBrowser?.selectedBrowser?.browsingContext?.browserId ?? 0;
+      `;
+        const activeBrowserId = await runner.runPrivilegedScript(
+          activeBrowserIdScript,
+          'Get activeBrowserId'
+        );
+        script = `Services.profiler.StartProfiler(${bufferSize},${interval},${featureString},
+          ${threadString},${activeBrowserId});`;
+        break;
+      }
+      case 5: {
+        // The profiler API that was used between Firefox 69 and 72.
+        script = `Services.profiler.StartProfiler(${bufferSize},${interval},${featureString},
+          ${threadString});`;
+        break;
+      }
+      default: {
+        log.error('Unknown Gecko Profiler API');
+        throw new Error('Unknown Gecko Profiler API');
+      }
     }
 
     log.info(
@@ -140,16 +164,24 @@ class GeckoProfiler {
       );
 
       if (isAndroidConfigured(options)) {
+        log.info('Download profile file from Android.');
         const android = new Android(options);
         await android._downloadFile(deviceProfileFilename, destinationFilename);
       }
 
       // GZIP the profile and remove the old file
-      return storageManager.gzip(
-        destinationFilename,
-        join(profileDir, `geckoProfile-${index}.json.gz`),
-        true
+      log.info('Gzip file the profile.');
+      await timeout(
+        storageManager.gzip(
+          destinationFilename,
+          join(profileDir, `geckoProfile-${index}.json.gz`),
+          true
+        ),
+        300_000, // 5 minutes
+        'Could not gzip the profile.'
       );
+
+      log.info('Done stopping GeckoProfiler.');
     } catch (error) {
       log.error(error.message);
     }
@@ -157,5 +189,3 @@ class GeckoProfiler {
 
   addMetaData() {}
 }
-
-export default GeckoProfiler;

package/lib/firefox/getHAR.js

@@ -3,7 +3,7 @@ const log = intel.getLogger('browsertime.firefox');
 
 // Get the HAR from Firefox
 // Using https://github.com/firefox-devtools/har-export-trigger
-export default async function (runner, includeResponseBodies) {
+export async function getHAR(runner, includeResponseBodies) {
   const script = `
     const callback = arguments[arguments.length - 1];
     async function triggerExport() {

package/lib/firefox/memoryReport.js

@@ -1,6 +1,6 @@
 import path from 'node:path';
 import intel from 'intel';
-import pathToFolder from '../support/pathToFolder.js';
+import { pathToFolder } from '../support/pathToFolder.js';
 import { BrowserError } from '../support/errors.js';
 import { isAndroidConfigured, Android } from '../android/index.js';
 const delay = ms => new Promise(res => setTimeout(res, ms));

package/lib/firefox/settings/disableSafeBrowsingPreferences.js

@@ -1,4 +1,4 @@
-export default {
+export const disableSafeBrowsingPreferences = {
   'browser.safebrowsing.downloads.enabled': false,
   'browser.safebrowsing.enabled': false,
   'browser.safebrowsing.phishing.enabled': false,

package/lib/firefox/settings/disableTrackingProtectionPreferences.js

@@ -1,4 +1,4 @@
-export default {
+export const disableTrackingProtectionPreferences = {
   'privacy.trackingprotection.annotate_channels': false,
   'privacy.trackingprotection.enabled': false,
   'privacy.trackingprotection.introURL':

package/lib/firefox/settings/firefoxPreferences.js

@@ -1,4 +1,4 @@
-export default {
+export const defaultFirefoxPreferences = {
   // Common preferences file used by both unittest and perf harnesses.
   'app.update.checkInstallTime': false,
   'app.update.disabledForTesting': true,

package/lib/firefox/webdriver/builder.js

@@ -11,9 +11,9 @@ import { pac, manual } from 'selenium-webdriver/proxy.js';
 import pick from 'lodash.pick';
 import isEmpty from 'lodash.isempty';
 import get from 'lodash.get';
-import defaultFirefoxPreferences from '../settings/firefoxPreferences.js';
-import disableSafeBrowsingPreferences from '../settings/disableSafeBrowsingPreferences.js';
-import disableTrackingProtectionPreferences from '../settings/disableTrackingProtectionPreferences.js';
+import { defaultFirefoxPreferences } from '../settings/firefoxPreferences.js';
+import { disableSafeBrowsingPreferences } from '../settings/disableSafeBrowsingPreferences.js';
+import { disableTrackingProtectionPreferences } from '../settings/disableTrackingProtectionPreferences.js';
 import { toArray } from '../../support/util.js';
 import { Android, isAndroidConfigured } from '../../android/index.js';
 const log = intel.getLogger('browsertime.firefox');
@@ -90,12 +90,12 @@ export async function configureBuilder(builder, baseDir, options) {
     }
   }
 
-  if (!firefoxConfig.noDefaultPrefs) {
+  if (firefoxConfig.noDefaultPrefs) {
+    log.info('Skip setting default preferences for Firefox');
+  } else {
     for (const pref of Object.keys(defaultFirefoxPreferences)) {
       ffOptions.setPreference(pref, defaultFirefoxPreferences[pref]);
     }
-  } else {
-    log.info('Skip setting default preferences for Firefox');
   }
 
   if (firefoxConfig.disableSafeBrowsing) {
@@ -144,9 +144,9 @@ export async function configureBuilder(builder, baseDir, options) {
   }
 
   if (options.extension) {
-    const extensions = !Array.isArray(options.extension)
-      ? [options.extension]
-      : options.extension;
+    const extensions = Array.isArray(options.extension)
+      ? options.extension
+      : [options.extension];
     for (const extension of extensions) {
       ffOptions.addExtensions(extension);
     }

package/lib/firefox/webdriver/firefox.js

@@ -4,17 +4,17 @@ import { join } from 'node:path';
 import intel from 'intel';
 import get from 'lodash.get';
 import { getEmptyHAR, mergeHars } from '../../support/har/index.js';
-import pathToFolder from '../../support/pathToFolder.js';
+import { pathToFolder } from '../../support/pathToFolder.js';
 import { isAndroidConfigured, Android } from '../../android/index.js';
 import { adjustVisualProgressTimestamps } from '../../support/util.js';
 import { findFiles } from '../../support/fileUtil.js';
-import getHAR from '../getHAR.js';
-import GeckoProfiler from '../geckoProfiler.js';
+import { getHAR } from '../getHAR.js';
+import { GeckoProfiler } from '../geckoProfiler.js';
 import { MemoryReport } from '../memoryReport.js';
 import { PerfStats } from '../perfStats.js';
 const log = intel.getLogger('browsertime.firefox');
 const rename = promisify(_rename);
-class Firefox {
+export class Firefox {
   constructor(storageManager, options) {
     // Lets keep this and hope that we in the future will have HAR for FF again
     this.skipHar = options.skipHar;
@@ -73,6 +73,14 @@ class Firefox {
    * Before each URL/test runs.
    */
   async beforeEachURL(runner) {
+    await runner.runPrivilegedScript(`
+      new Promise(async function(resolve) {
+        await Services.fog.testFlushAllChildren(); // force any data that wasn't recorded yet to be immediately put in the buffers
+        Services.fog.testResetFOG(); // empty the buffers
+        resolve();
+      });
+    `);
+
     if (isAndroidConfigured(this.options) && this.options.androidPower) {
       await this.android.resetPowerUsage();
     }
@@ -146,6 +154,18 @@ class Firefox {
       );
     }
 
+    let powerusage = await runner.runPrivilegedScript(`
+      return new Promise(async function(resolve) {
+        Services.fog.initializeFOG(); // prevents timeout when collecting CPU metrics
+        await Services.fog.testFlushAllChildren(); // force data from child processes to be sent to the parent.
+        resolve(Glean.power.totalCpuTimeMs.testGetValue());
+      });
+    `);
+    if (powerusage) {
+      log.info('CPU / Power usage: ' + powerusage);
+    }
+    result.cpu = powerusage;
+
     if (this.skipHar || isAndroidConfigured(this.options)) {
       return result;
     } else {
@@ -274,5 +294,3 @@ class Firefox {
    */
   async afterBrowserStopped() {}
 }
-
-export default Firefox;

package/lib/safari/webdriver/safari.js

@@ -2,7 +2,7 @@
 import { execaCommand as command } from 'execa';
 const delay = ms => new Promise(res => setTimeout(res, ms));
 
-export default class Safari {
+export class Safari {
   constructor(storageManager, options) {
     this.options = options;
     this.safariOptions = options.safari || {};

package/lib/screenshot/defaults.js

@@ -1,4 +1,4 @@
-export default {
+export const screenshotDefaults = {
   type: 'jpg',
   png: {
     compressionLevel: 6

package/lib/screenshot/index.js

@@ -1,7 +1,7 @@
 import { join } from 'node:path';
 import merge from 'lodash.merge';
 import intel from 'intel';
-import defaultConfig from './defaults.js';
+import { screenshotDefaults } from './defaults.js';
 import {
   savePngWithoutResize,
   savePng,
@@ -9,10 +9,10 @@ import {
 } from '../support/images/index.js';
 const SCREENSHOT_DIR = 'screenshots';
 const log = intel.getLogger('browsertime.screenshot');
-class ScreenshotManager {
+export class ScreenshotManager {
   constructor(storageManager, options) {
     this.storageManager = storageManager;
-    this.config = merge({}, defaultConfig, options.screenshotParams);
+    this.config = merge({}, screenshotDefaults, options.screenshotParams);
     this.options = options;
     this.savedScreenshots = [];
   }
@@ -76,5 +76,3 @@ class ScreenshotManager {
     return this.savedScreenshots;
   }
 }
-
-export default ScreenshotManager;

package/lib/support/cli.js

@@ -13,7 +13,7 @@ import {
   threads,
   xvfbDisplay
 } from '../video/defaults.js';
-import SCREENSHOT from '../screenshot/defaults.js';
+import { screenshotDefaults } from '../screenshot/defaults.js';
 import {
   features,
   threads as _threads,
@@ -840,22 +840,22 @@ export function parseCommandLine() {
     .option('screenshotParams.type', {
       describe: 'Set the file type of the screenshot',
       choices: ['png', 'jpg'],
-      default: SCREENSHOT.type,
+      default: screenshotDefaults.type,
       group: 'Screenshot'
     })
     .option('screenshotParams.png.compressionLevel', {
       describe: 'zlib compression level',
-      default: SCREENSHOT.png.compressionLevel,
+      default: screenshotDefaults.png.compressionLevel,
       group: 'Screenshot'
     })
     .option('screenshotParams.jpg.quality', {
       describe: 'Quality of the JPEG screenshot. 1-100',
-      default: SCREENSHOT.jpg.quality,
+      default: screenshotDefaults.jpg.quality,
       group: 'Screenshot'
     })
     .option('screenshotParams.maxSize', {
       describe: 'The max size of the screenshot (width and height).',
-      default: SCREENSHOT.maxSize,
+      default: screenshotDefaults.maxSize,
       group: 'Screenshot'
     })
     .option('pageCompleteCheck', {

package/lib/support/dns.js

@@ -3,7 +3,7 @@ import intel from 'intel';
 import { isAndroidConfigured } from '../android/index.js';
 const log = intel.getLogger('browsertime.dns');
 
-export default async function (options) {
+export async function flushDNS(options) {
   if (isAndroidConfigured(options)) {
     return;
   }
@@ -25,6 +25,11 @@ export default async function (options) {
     });
   } else if (process.platform === 'linux') {
     log.info('Flush DNS cache on Linux');
+
+    // Unbuntu 22.04
+    await execa('sudo', ['resolvectl', 'flush-caches'], {
+      reject: false
+    });
     await execa('sudo', ['systemd-resolve', '--flush-caches'], {
       reject: false
     });

package/lib/support/engineUtils.js

@@ -1,6 +1,6 @@
 import { resolve, dirname, join } from 'node:path';
 import { promisify } from 'node:util';
-import { writeFile as _writeFile, access, unlink as _unlink } from 'node:fs';
+import { writeFile as _writeFile, unlink as _unlink } from 'node:fs';
 import { pathToFileURL } from 'node:url';
 import dayjs from 'dayjs';
 import intel from 'intel';
@@ -11,15 +11,6 @@ const unlink = promisify(_unlink);
 
 const AsyncFunction = Object.getPrototypeOf(async function () {}).constructor;
 
-async function exists(path) {
-  try {
-    await access(path);
-    return true;
-  } catch {
-    return false;
-  }
-}
-
 async function loadFile(script, options, throwError) {
   // if the script is an AsyncFunction, return it.
   if (script instanceof AsyncFunction) {
@@ -42,15 +33,16 @@ async function loadFile(script, options, throwError) {
     // Hack a way! Try to add a package.json file in the same folder as the
     // script file. That way, the .js file will be treated as commonjs =
     // working as before we moved Browsertime to module
+
     let createdPackageJson = false;
     try {
       const packageJson = join(dirname(resolve(script)), 'package.json');
-      if (!(await exists(packageJson))) {
+      try {
         await writeFile(packageJson, '{}');
         createdPackageJson = true;
-      } else {
+      } catch {
         log.error(
-          'Could not create package.json file, the %S file will be treated as a esmodule',
+          'Could not create package.json file, the %s file will be treated as a esmodule',
           script
         );
       }

package/lib/support/getViewPort.js

@@ -29,7 +29,7 @@ const sizeMap = {
   'Samsung Galaxy S8+': '412x846'
 };
 
-export default function (options) {
+export function getViewPort(options) {
   // you cannot set the width/height for phone so just keep the viewport undefined
   if (isAndroidConfigured(options)) {
     return;

package/lib/support/har/index.js

@@ -4,7 +4,7 @@ import pick from 'lodash.pick';
 import isEmpty from 'lodash.isempty';
 import get from 'lodash.get';
 import dayjs from 'dayjs';
-import pathToFolder from '../pathToFolder.js';
+import { pathToFolder } from '../pathToFolder.js';
 import intel from 'intel';
 const log = intel.getLogger('browsertime');
 const require = createRequire(import.meta.url);
@@ -22,7 +22,7 @@ function generateUniquePageId(baseId, existingIdMap) {
 function getDocumentRequests(entries, pageId) {
   let pageEntries = [...entries];
   if (pageId) {
-    pageEntries = [...entries.filter(entry => entry.pageref === pageId)];
+    pageEntries = entries.filter(entry => entry.pageref === pageId);
   }
 
   const requests = [];
@@ -199,11 +199,9 @@ export function getMainDocumentTimings(har) {
     let pageEntries = [...entries];
     const finalURL = getFinalURL(pageEntries, pageId);
 
-    pageEntries = [
-      ...pageEntries.filter(
-        entry => entry.pageref === pageId && entry.request.url === finalURL
-      )
-    ];
+    pageEntries = pageEntries.filter(
+      entry => entry.pageref === pageId && entry.request.url === finalURL
+    );
 
     timings.push({ url, timings: pageEntries[0].timings });
   }
@@ -219,7 +217,7 @@ export function getFullyLoaded(har) {
     const url = page._url;
 
     let pageEntries = [...entries];
-    pageEntries = [...pageEntries.filter(entry => entry.pageref === pageId)];
+    pageEntries = pageEntries.filter(entry => entry.pageref === pageId);
 
     let pageEnd = 0;
     for (let entry of pageEntries) {

package/lib/support/images/index.js

@@ -1,5 +1,5 @@
 import { join } from 'node:path';
-import pathToFolder from '../pathToFolder.js';
+import { pathToFolder } from '../pathToFolder.js';
 
 export async function savePngWithoutResize(
   name,