browsertime 22.5.2 → 22.5.4

package/CHANGELOG.md

@@ -1,5 +1,16 @@
 # Browsertime changelog (we do [semantic versioning](https://semver.org))
 
+## 22.5.4 - 2024-07-06
+### Fixed
+* Update to Chrome-HAR 0.13.3 [#2148](https://github.com/sitespeedio/browsertime/pull/2148).
+* Update Chrome remore interface [#2149](https://github.com/sitespeedio/browsertime/pull/2149).
+* Update execa, dayjs and get-port [#2153](https://github.com/sitespeedio/browsertime/pull/2153).
+* Update developer dependencies [#2154](https://github.com/sitespeedio/browsertime/pull/2154), [#2152](https://github.com/sitespeedio/browsertime/pull/2152), [#2151](https://github.com/sitespeedio/browsertime/pull/2151) and [#2150](https://github.com/sitespeedio/browsertime/pull/2150).
+
+## 22.5.3 - 2024-06-25
+### Fixed
+* Another fix to make IntelliSense/Code completion work again [#2147](https://github.com/sitespeedio/browsertime/pull/2147).
+
 ## 22.5.2 - 2024-06-24
 ### Fixed
 * Updated Selenium dependencies [#2146](https://github.com/sitespeedio/browsertime/pull/2146).

package/bin/browsertime.js

@@ -4,7 +4,7 @@ import get from 'lodash.get';
 import set from 'lodash.set';
 import intel from 'intel';
 import { existsSync, mkdirSync } from 'node:fs';
-import { resolve, relative } from 'node:path';
+import path from 'node:path';
 import { Engine } from '../lib/core/engine/index.js';
 import {
   findAndParseScripts,
@@ -25,7 +25,7 @@ async function parseUserScripts(scripts) {
   if (!Array.isArray(scripts)) scripts = [scripts];
   const results = {};
   for (const script of scripts) {
-    const code = await findAndParseScripts(resolve(script), 'custom');
+    const code = await findAndParseScripts(path.resolve(script), 'custom');
     merge(results, code);
   }
   return results;
@@ -125,7 +125,10 @@ async function run(urls, options) {
 
       await Promise.all(saveOperations);
 
-      const resultDirectory = relative(process.cwd(), storageManager.directory);
+      const resultDirectory = path.relative(
+        process.cwd(),
+        storageManager.directory
+      );
 
       // check for errors
       // If we have set the exit code in scripts, respect that

package/lib/android/index.js

@@ -1,6 +1,6 @@
 import { promisify } from 'node:util';
 import { mkdir as _mkdir, createWriteStream } from 'node:fs';
-import { join } from 'node:path';
+import path from 'node:path';
 import { EOL as endOfLine } from 'node:os';
 import { execa } from 'execa';
 import intel from 'intel';
@@ -105,7 +105,7 @@ export class Android {
 
     for (const file of files) {
       const fullSourcePath = `${sourcePath}/${file.name}`;
-      const fullDestinationPath = join(destinationPath, file.name);
+      const fullDestinationPath = path.join(destinationPath, file.name);
 
       if (file.isFile()) {
         await this._downloadFile(fullSourcePath, fullDestinationPath);
@@ -475,7 +475,7 @@ export class Android {
 
   async getUsbPowerUsageProfile(index, url, result, options, storageManager) {
     let profileData = await usbPowerProfiler.profileFromData();
-    let destinationFilename = join(
+    let destinationFilename = path.join(
       await pathToFolder(url, options),
       `powerProfile-${index}.json`
     );

package/lib/chrome/networkManager.js

@@ -31,7 +31,7 @@ export class NetworkManager {
 
   async waitForNetworkIdle() {
     const startTime = Date.now();
-    // eslint-disable-next-line no-constant-condition
+
     while (true) {
       const now = Date.now();
       const sinceLastResponseRequest =

package/lib/chrome/webdriver/chromium.js

@@ -1,6 +1,6 @@
 import { promisify } from 'node:util';
 import { unlink as _unlink, rm as _rm } from 'node:fs';
-import { join } from 'node:path';
+import path from 'node:path';
 import { logging } from 'selenium-webdriver';
 import intel from 'intel';
 import usbPowerProfiler from 'usb-power-profiling/usb-power-profiling.js';
@@ -261,12 +261,12 @@ export class Chromium {
 
     if (this.chrome.collectNetLog && !this.chrome.android) {
       await this.storageManager.createSubDataDir(
-        join(pathToFolder(result.url, this.options))
+        path.join(pathToFolder(result.url, this.options))
       );
 
       await this.storageManager.gzip(
         `${this.baseDir}/chromeNetlog.json`,
-        join(
+        path.join(
           this.baseDir,
           pathToFolder(result.url, this.options),
           `chromeNetlog-${index}.json.gz`
@@ -350,13 +350,13 @@ export class Chromium {
 
     if (this.chrome.collectNetLog && this.chrome.android) {
       // THIS needs to be unique per page
-      const filename = join(
+      const filename = path.join(
         this.baseDir,
         pathToFolder(result.url, this.options),
         `chromeNetlog-${index}.json`
       );
 
-      const gzFilename = join(
+      const gzFilename = path.join(
         this.baseDir,
         pathToFolder(result.url, this.options),
         `chromeNetlog-${index}.json.gz`

package/lib/core/engine/collector.js

@@ -1,4 +1,4 @@
-import { join } from 'node:path';
+import path from 'node:path';
 import { createRequire } from 'node:module';
 import get from 'lodash.get';
 import intel from 'intel';
@@ -482,7 +482,7 @@ export class Collector {
       for (let filename of Object.keys(data.extraJson)) {
         extraWork.push(
           this.storageManager.writeJson(
-            join(pathToFolder(url, this.options), filename),
+            path.join(pathToFolder(url, this.options), filename),
             data.extraJson[filename],
             true
           )

package/lib/core/engine/command/measure.js

@@ -1,4 +1,4 @@
-import { join } from 'node:path';
+import path from 'node:path';
 import intel from 'intel';
 import get from 'lodash.get';
 import merge from 'lodash.merge';
@@ -429,13 +429,14 @@ export class Measure {
 
     // We have the URL, create the data dir
     await this.storageManager.createSubDataDir(
-      join(pathToFolder(url, this.options))
+      path.join(pathToFolder(url, this.options))
     );
     await this._stopVideo(url);
 
     const alias = this.options.urlMetaData
       ? this.options.urlMetaData[url]
-      : undefined || this.result[this.numberOfMeasuredPages].alias;
+      : // eslint-disable-next-line no-constant-binary-expression
+        undefined || this.result[this.numberOfMeasuredPages].alias;
     const res = await this.engineDelegate.afterPageCompleteCheck(
       this.browser,
       this.index,

package/lib/core/engine/index.js

@@ -224,7 +224,7 @@ export class Engine {
       }
     }
 
-    return Promise.all([this.myXVFB.start()]);
+    return this.myXVFB.start();
   }
 
   async runByScript(
@@ -518,6 +518,6 @@ export class Engine {
       await this.gnirehtet.stop();
     }
 
-    return Promise.all([this.myXVFB.stop()]);
+    return this.myXVFB.stop();
   }
 }

package/lib/firefox/geckoProfiler.js

@@ -1,4 +1,4 @@
-import { join } from 'node:path';
+import path from 'node:path';
 import get from 'lodash.get';
 import intel from 'intel';
 import { geckoProfilerDefaults } from './settings/geckoProfilerDefaults.js';
@@ -156,9 +156,12 @@ export class GeckoProfiler {
     const options = this.options;
 
     let profileDir = await storageManager.createSubDataDir(
-      join(pathToFolder(url, options))
+      path.join(pathToFolder(url, options))
+    );
+    let destinationFilename = path.join(
+      profileDir,
+      `geckoProfile-${index}.json`
     );
-    let destinationFilename = join(profileDir, `geckoProfile-${index}.json`);
 
     let deviceProfileFilename = destinationFilename;
     if (isAndroidConfigured(options)) {
@@ -214,7 +217,7 @@ export class GeckoProfiler {
           const profile = JSON.parse(
             await storageManager.readData(
               `geckoProfile-${index}.json`,
-              join(pathToFolder(url, options))
+              path.join(pathToFolder(url, options))
             )
           );
           let power = 0;
@@ -237,7 +240,11 @@ export class GeckoProfiler {
         ? `geckoProfile-${index}-extra.json.gz`
         : `geckoProfile-${index}.json.gz`;
       await timeout(
-        storageManager.gzip(destinationFilename, join(profileDir, name), true),
+        storageManager.gzip(
+          destinationFilename,
+          path.join(profileDir, name),
+          true
+        ),
         300_000, // 5 minutes
         'Could not gzip the profile.'
       );

package/lib/firefox/networkManager.js

@@ -53,7 +53,7 @@ export class NetworkManager {
     });
 
     const startTime = Date.now();
-    // eslint-disable-next-line no-constant-condition
+
     while (true) {
       const now = Date.now();
       const sinceLastResponseRequest =

package/lib/firefox/webdriver/firefox.js

@@ -1,6 +1,6 @@
 import { rename as _rename } from 'node:fs';
 import { promisify } from 'node:util';
-import { join } from 'node:path';
+import path from 'node:path';
 import intel from 'intel';
 import get from 'lodash.get';
 import usbPowerProfiler from 'usb-power-profiling/usb-power-profiling.js';
@@ -312,7 +312,7 @@ export class Firefox {
         for (const file of files) {
           await rename(
             `${this.baseDir}/${file}`,
-            join(
+            path.join(
               this.baseDir,
               pathToFolder(result.url, this.options),
               `${file}-${index}.txt`
@@ -358,7 +358,7 @@ export class Firefox {
           geckoProfile.meta.visualMetrics = result.visualMetrics;
 
           await this.storageManager.writeJson(
-            join(profileSubdir, `geckoProfile-${index}.json`),
+            path.join(profileSubdir, `geckoProfile-${index}.json`),
             geckoProfile,
             true
           );

package/lib/screenshot/index.js

@@ -1,4 +1,4 @@
-import { join } from 'node:path';
+import path from 'node:path';
 import merge from 'lodash.merge';
 import intel from 'intel';
 import { screenshotDefaults } from './defaults.js';
@@ -36,7 +36,7 @@ export class ScreenshotManager {
         data,
         url,
         this.storageManager,
-        join(SCREENSHOT_DIR, `${index}`),
+        path.join(SCREENSHOT_DIR, `${index}`),
         this.options
       );
       this.savedScreenshots.push(
@@ -50,7 +50,7 @@ export class ScreenshotManager {
         url,
         this.storageManager,
         this.config,
-        join(SCREENSHOT_DIR, `${index}`),
+        path.join(SCREENSHOT_DIR, `${index}`),
         this.options
       );
       this.savedScreenshots.push(
@@ -63,7 +63,7 @@ export class ScreenshotManager {
         url,
         this.storageManager,
         this.config,
-        join(SCREENSHOT_DIR, `${index}`),
+        path.join(SCREENSHOT_DIR, `${index}`),
         this.options
       );
       this.savedScreenshots.push(

package/lib/support/engineUtils.js

@@ -1,4 +1,4 @@
-import { resolve, dirname, join } from 'node:path';
+import path from 'node:path';
 import { promisify } from 'node:util';
 import {
   writeFile as _writeFile,
@@ -32,7 +32,7 @@ async function loadFile(script, options, throwError) {
     script.endsWith('.mjs') ||
     script.endsWith('.cjs')
   ) {
-    let myFunction = await import(pathToFileURL(resolve(script)));
+    let myFunction = await import(pathToFileURL(path.resolve(script)));
     return myFunction.default ?? myFunction;
   } else {
     // Hack a way! Try to add a package.json file in the same folder as the
@@ -43,7 +43,10 @@ async function loadFile(script, options, throwError) {
     );
     let createdPackageJson = false;
     try {
-      const packageJson = join(dirname(resolve(script)), 'package.json');
+      const packageJson = path.join(
+        path.dirname(path.resolve(script)),
+        'package.json'
+      );
       try {
         await writeFile(packageJson, '{}');
         createdPackageJson = true;
@@ -53,7 +56,7 @@ async function loadFile(script, options, throwError) {
           script
         );
       }
-      const myFunction = await import(resolve(script));
+      const myFunction = await import(path.resolve(script));
 
       try {
         if (createdPackageJson) {
@@ -89,7 +92,7 @@ export async function loadPrePostScripts(scripts, options) {
 
 export async function loadPageCompleteScript(script) {
   if (script && script.endsWith('js')) {
-    return readFile(resolve(script), 'utf8');
+    return readFile(path.resolve(script), 'utf8');
   }
   return script;
 }

package/lib/support/fileUtil.js

@@ -1,4 +1,4 @@
-import { join } from 'node:path';
+import path from 'node:path';
 import { promisify } from 'node:util';
 import {
   readdir as _readdir,
@@ -50,7 +50,7 @@ export async function removeDirAndFiles(dirName) {
       await Promise.all(
         files.map(async file => {
           try {
-            const p = join(dir, file);
+            const p = path.join(dir, file);
             const stat = await lstat(p);
             await (stat.isDirectory() ? removeDir(p) : unlink(p));
           } catch (error) {
@@ -68,7 +68,7 @@ export async function removeDirAndFiles(dirName) {
 export async function removeByType(dir, type) {
   const fileNames = await readdir(dir);
   const filePaths = fileNames
-    .map(fileName => join(dir, fileName))
+    .map(fileName => path.join(dir, fileName))
     .filter(element => onlyFiles(element))
     .filter(onlyWithExtension('.' + type));
 

package/lib/support/filters.js

@@ -1,13 +1,13 @@
 import { promisify } from 'node:util';
 import { stat as _stat } from 'node:fs';
-import { extname } from 'node:path';
+import path from 'node:path';
 const stat = promisify(_stat);
 
 /**
  * Filters to use with Array.prototype.filter, e.g. ['/a/path', '/another/path'].filter(onlyFiles)
  */
 export function onlyWithExtension(extension) {
-  return filepath => extname(filepath) === extension;
+  return filepath => path.extname(filepath) === extension;
 }
 export async function onlyFiles(filepath) {
   const stats = await stat(filepath);

package/lib/support/images/index.js

@@ -1,4 +1,4 @@
-import { join } from 'node:path';
+import path from 'node:path';
 import { pathToFolder } from '../pathToFolder.js';
 
 export async function savePngWithoutResize(
@@ -12,7 +12,7 @@ export async function savePngWithoutResize(
   return storageManager.writeData(
     `${name}.png`,
     data,
-    join(pathToFolder(url, options), dir)
+    path.join(pathToFolder(url, options), dir)
   );
 }
 export async function savePng(
@@ -41,7 +41,7 @@ export async function savePng(
     return storageManager.writeData(
       `${name}.png`,
       buffer,
-      join(pathToFolder(url, options), dir)
+      path.join(pathToFolder(url, options), dir)
     );
   }
 }
@@ -70,7 +70,7 @@ export async function saveJpg(
     return storageManager.writeData(
       `${name}.jpg`,
       buffer,
-      join(pathToFolder(url, options), dir)
+      path.join(pathToFolder(url, options), dir)
     );
   }
 }

package/lib/support/storageManager.js

@@ -1,4 +1,4 @@
-import { resolve as _resolve, join } from 'node:path';
+import path from 'node:path';
 import { createHash } from 'node:crypto';
 import { gunzip as _gunzip, gzip as _gzip, createGzip } from 'node:zlib';
 import { parse } from 'node:url';
@@ -52,8 +52,8 @@ async function streamToString(stream) {
 export class StorageManager {
   constructor(url, { resultDir, prettyPrint = false } = {}) {
     this.baseDir = resultDir
-      ? _resolve(resultDir)
-      : _resolve(defaultDir, pathNameFromUrl(url), timestamp);
+      ? path.resolve(resultDir)
+      : path.resolve(defaultDir, pathNameFromUrl(url), timestamp);
     this.jsonIndentation = prettyPrint ? 2 : 0;
   }
 
@@ -63,13 +63,13 @@ export class StorageManager {
   }
 
   async createSubDataDir(...name) {
-    const dir = join(this.baseDir, ...name);
+    const dir = path.join(this.baseDir, ...name);
     await mkdir(dir, { recursive: true });
     return dir;
   }
 
   async rm(filename) {
-    return unlink(join(this.baseDir, filename));
+    return unlink(path.join(this.baseDir, filename));
   }
 
   async writeData(filename, data, subdir) {
@@ -77,7 +77,7 @@ export class StorageManager {
     dirPath = await (subdir
       ? this.createSubDataDir(subdir)
       : this.createDataDir());
-    const fullPath = join(dirPath, filename);
+    const fullPath = path.join(dirPath, filename);
     return writeFile(fullPath, data).then(() => {
       return fullPath;
     });
@@ -96,8 +96,8 @@ export class StorageManager {
   async readData(filename, subdir) {
     let filepath;
     filepath = subdir
-      ? join(this.baseDir, subdir, filename)
-      : join(this.baseDir, filename);
+      ? path.join(this.baseDir, subdir, filename)
+      : path.join(this.baseDir, filename);
 
     if (filename.endsWith('.gz')) {
       const readStream = createReadStream(filepath);

package/lib/support/tcpdump.js

@@ -1,4 +1,4 @@
-import { join } from 'node:path';
+import path from 'node:path';
 import { execa } from 'execa';
 import { pathToFolder } from './pathToFolder.js';
 import { rename } from './fileUtil.js';
@@ -9,7 +9,7 @@ export class TCPDump {
     this.options = options;
   }
   async start(iteration) {
-    const captureFile = join(this.baseDir, iteration + '.pcap');
+    const captureFile = path.join(this.baseDir, iteration + '.pcap');
     const parameters = [
       'tcpdump',
       '-i',
@@ -36,8 +36,8 @@ export class TCPDump {
   }
 
   async mv(url, iteration) {
-    const oldLocation = join(this.baseDir, iteration + '.pcap');
-    const newLocation = join(
+    const oldLocation = path.join(this.baseDir, iteration + '.pcap');
+    const newLocation = path.join(
       this.baseDir,
       pathToFolder(url, this.options),
       iteration + '.pcap'

package/lib/video/postprocessing/finetune/index.js

@@ -1,4 +1,4 @@
-import { join } from 'node:path';
+import path from 'node:path';
 import {
   rename as _rename,
   copyFile as _copyFile,
@@ -26,10 +26,10 @@ export async function finetuneVideo(
   options
 ) {
   const newStart = videoMetrics.videoRecordingStart / 1000;
-  let temporaryFile = join(videoDir, 'tmp.mp4');
+  let temporaryFile = path.join(videoDir, 'tmp.mp4');
 
   if (get(options, 'videoParams.keepOriginalVideo', false)) {
-    const originalFile = join(videoDir, index + '-original.mp4');
+    const originalFile = path.join(videoDir, index + '-original.mp4');
     await copyFile(videoPath, originalFile);
   }
 
@@ -59,19 +59,19 @@ export async function finetuneVideo(
     isAndroidConfigured(options) &&
     get(options, 'videoParams.convert', _convert)
   ) {
-    const temporaryFile2 = join(videoDir, 'tmp-60fps.mp4');
+    const temporaryFile2 = path.join(videoDir, 'tmp-60fps.mp4');
     await convert(temporaryFile, temporaryFile2, 60);
     await unlink(temporaryFile);
     await rename(temporaryFile2, temporaryFile);
   } else if (options.safari && options.safari.useSimulator) {
-    const temporaryFile2 = join(videoDir, 'tmp-60fps.mp4');
+    const temporaryFile2 = path.join(videoDir, 'tmp-60fps.mp4');
     await convert(temporaryFile, temporaryFile2, 60);
     await unlink(temporaryFile);
     await rename(temporaryFile2, temporaryFile);
   }
 
   if (options.videoParams.addTimer) {
-    const temporaryFile2 = join(videoDir, 'tmp2.mp4');
+    const temporaryFile2 = path.join(videoDir, 'tmp2.mp4');
     await addTextToVideo(
       temporaryFile,
       temporaryFile2,

package/lib/video/postprocessing/visualmetrics/getVideoMetrics.js

@@ -1,4 +1,4 @@
-import { join } from 'node:path';
+import path from 'node:path';
 import intel from 'intel';
 import { run } from './visualMetrics.js';
 import { extraMetrics } from './extraMetrics.js';
@@ -25,7 +25,7 @@ export async function getVideoMetrics(
       true
     );
   }
-  const elementsFile = join(
+  const elementsFile = path.join(
     storageManager.directory,
     index + '-visualElements.json.gz'
   );

package/lib/video/postprocessing/visualmetrics/visualMetrics.js

@@ -1,5 +1,4 @@
 import { fileURLToPath } from 'node:url';
-import { join } from 'node:path';
 import path from 'node:path';
 import { execa } from 'execa';
 import intel from 'intel';
@@ -8,7 +7,7 @@ import { readFile, removeFile, copyFile } from '../../../support/fileUtil.js';
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 const log = intel.getLogger('browsertime.video');
 
-const SCRIPT_PATH = join(
+const SCRIPT_PATH = path.join(
   __dirname,
   '..',
   '..',
@@ -18,7 +17,7 @@ const SCRIPT_PATH = join(
   'visualmetrics.py'
 );
 
-const PORTABLE_SCRIPT_PATH = join(
+const PORTABLE_SCRIPT_PATH = path.join(
   __dirname,
   '..',
   '..',
@@ -120,7 +119,10 @@ export async function run(
     }
   }
 
-  const visualMetricsLogFile = join(videoDir, index + '.visualmetrics.log');
+  const visualMetricsLogFile = path.join(
+    videoDir,
+    index + '.visualmetrics.log'
+  );
 
   scriptArguments.push('--logfile', visualMetricsLogFile);
   if (options.verbose >= 1) {

package/lib/video/screenRecording/firefox/firefoxWindowRecorder.js

@@ -6,7 +6,7 @@ import {
   createWriteStream,
   copyFileSync
 } from 'node:fs';
-import { join } from 'node:path';
+import path from 'node:path';
 import intel from 'intel';
 import { execa } from 'execa';
 import { removeDirAndFiles } from '../../../support/fileUtil.js';
@@ -41,7 +41,7 @@ function findRecordingDirectory(baseDir) {
 
   for (const file of readdirSync(baseDir)) {
     if (file.startsWith('windowrecording-')) {
-      let fullPath = join(baseDir, file);
+      let fullPath = path.join(baseDir, file);
       let mtime = statSync(fullPath).mtime;
       if (mtime > closest_mtime) {
         closest_mtime = mtime;
@@ -79,17 +79,19 @@ function findTimeToFirstFrame(recordingDir) {
 
 function writeFrameDurationsToFile(directoryName, imageFiles) {
   return new Promise((resolve, reject) => {
-    const stream = createWriteStream(join(directoryName, 'durations.txt'));
+    const stream = createWriteStream(path.join(directoryName, 'durations.txt'));
     stream.once('open', function () {
       stream.write(
-        "file '" + join(directoryName, imageFiles[0].filename) + "'\n"
+        "file '" + path.join(directoryName, imageFiles[0].filename) + "'\n"
       );
       for (let index = 1; index < imageFiles.length; index++) {
         let duration =
           (imageFiles[index].offset - imageFiles[index - 1].offset) / 1000;
         stream.write('duration ' + duration.toString() + '\n');
         stream.write(
-          "file '" + join(directoryName, imageFiles[index].filename) + "'\n"
+          "file '" +
+            path.join(directoryName, imageFiles[index].filename) +
+            "'\n"
         );
       }
       stream.end();
@@ -114,8 +116,8 @@ async function generateVideo(destination, recordingDirectoryName) {
     let newFilename = 'frame' + frameno + '.png';
     let offset = fields[2].split('.')[0];
     copyFileSync(
-      join(recordingDirectoryName, file),
-      join(recordingDirectoryName, newFilename)
+      path.join(recordingDirectoryName, file),
+      path.join(recordingDirectoryName, newFilename)
     );
     imageFiles.push({ filename: newFilename, offset: offset });
   }
@@ -133,7 +135,7 @@ async function generateVideo(destination, recordingDirectoryName) {
     '-safe',
     '0',
     '-i',
-    join(recordingDirectoryName, 'durations.txt'),
+    path.join(recordingDirectoryName, 'durations.txt'),
     '-vf',
     'pad=ceil(iw/2)*2:ceil(ih/2)*2',
     '-vsync',