lighthouse 9.5.0-dev.20220904 → 9.5.0-dev.20220907

package/cli/cli-flags.js

@@ -7,6 +7,7 @@
 /* eslint-disable max-len */
 
 import fs from 'fs';
+import path from 'path';
 
 import yargs from 'yargs';
 import * as yargsHelpers from 'yargs/helpers';
@@ -227,6 +228,7 @@ function getYargsParser(manualArgv) {
       },
       'output-path': {
         type: 'string',
+        coerce: coerceOutputPath,
         describe: `The file path to output the results. Use 'stdout' to write to stdout.
 If using JSON output, default is stdout.
 If using HTML or CSV output, default is a file in the working directory with a name based on the test URL and date.
@@ -408,6 +410,21 @@ function coerceOutput(values) {
   return validValues;
 }
 
+/**
+ * Verifies outputPath is something we can actually write to.
+ * @param {unknown=} value
+ * @return {string=}
+ */
+function coerceOutputPath(value) {
+  if (value === undefined) return;
+
+  if (typeof value !== 'string' || !value || !fs.existsSync(path.dirname(value))) {
+    throw new Error(`--output-path (${value}) cannot be written to`);
+  }
+
+  return value;
+}
+
 /**
  * Verifies value is a string, then coerces type to LH.Locale for convenience. However, don't
  * allowlist specific locales. Why? So we can support the user who requests 'es-MX' (unsupported)

package/cli/run.js

@@ -19,7 +19,7 @@ import * as Printer from './printer.js';
 import lighthouse, {legacyNavigation} from '../core/index.js';
 import {getLhrFilenamePrefix} from '../report/generator/file-namer.js';
 import * as assetSaver from '../core/lib/asset-saver.js';
-import URL from '../core/lib/url-shim.js';
+import UrlUtils from '../core/lib/url-utils.js';
 
 /** @typedef {Error & {code: string, friendlyMessage?: string}} ExitError */
 
@@ -232,7 +232,7 @@ async function runLighthouse(url, flags, config) {
     }
 
     const shouldGather = flags.gatherMode || flags.gatherMode === flags.auditMode;
-    const shouldUseLocalChrome = URL.isLikeLocalhost(flags.hostname);
+    const shouldUseLocalChrome = UrlUtils.isLikeLocalhost(flags.hostname);
     if (shouldGather && shouldUseLocalChrome) {
       launchedChrome = await getDebuggableChrome(flags);
       flags.port = launchedChrome.port;

package/cli/test/smokehouse/report-assert.js

@@ -259,10 +259,6 @@ function pruneExpectations(localConsole, lhr, expected, reportOptions) {
    * @param {*} obj
    */
   function failsChromeVersionCheck(obj) {
-    // LHR never actually run, so we can't know. Better to not prune
-    // things than to fail loudly when accessing lhr.environment.
-    if (!lhr.environment) return false;
-
     return !chromiumVersionCheck({
       version: getChromeVersionString(),
       min: obj._minChromiumVersion,

package/cli/test/smokehouse/smokehouse.js

@@ -195,6 +195,12 @@ async function runSmokeTest(smokeTestDefn, testOptions) {
         ...await lighthouseRunner(requestedUrl, configJson, {isDebug, useLegacyNavigation}),
         networkRequests: takeNetworkRequestUrls ? takeNetworkRequestUrls() : undefined,
       };
+
+      if (!result.lhr?.audits || !result.artifacts) {
+        // Something went really wrong and the runner didn't catch it.
+        throw new Error('lighthouse runner returned a bad result. got lhr:\n' +
+          JSON.stringify(result.lhr, null, 2));
+      }
     } catch (e) {
       // Clear the network requests so that when we retry, we don't see duplicates.
       if (takeNetworkRequestUrls) takeNetworkRequestUrls();

package/core/audits/byte-efficiency/modern-image-formats.js

@@ -9,7 +9,7 @@
 
 
 import {ByteEfficiencyAudit} from './byte-efficiency-audit.js';
-import URL from '../../lib/url-shim.js';
+import UrlUtils from '../../lib/url-utils.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 
 const UIStrings = {
@@ -106,7 +106,7 @@ class ModernImageFormats extends ByteEfficiencyAudit {
       const imageElement = imageElementsByURL.get(image.url);
 
       if (image.failed) {
-        warnings.push(`Unable to decode ${URL.getURLDisplayName(image.url)}`);
+        warnings.push(`Unable to decode ${UrlUtils.getURLDisplayName(image.url)}`);
         continue;
       }
 
@@ -123,7 +123,7 @@ class ModernImageFormats extends ByteEfficiencyAudit {
 
       if (typeof webpSize === 'undefined') {
         if (!imageElement) {
-          warnings.push(`Unable to locate resource ${URL.getURLDisplayName(image.url)}`);
+          warnings.push(`Unable to locate resource ${UrlUtils.getURLDisplayName(image.url)}`);
           continue;
         }
 
@@ -152,8 +152,8 @@ class ModernImageFormats extends ByteEfficiencyAudit {
       const wastedBytes = image.originalSize - avifSize;
       if (wastedBytes < IGNORE_THRESHOLD_IN_BYTES) continue;
 
-      const url = URL.elideDataURI(image.url);
-      const isCrossOrigin = !URL.originsMatch(pageURL, image.url);
+      const url = UrlUtils.elideDataURI(image.url);
+      const isCrossOrigin = !UrlUtils.originsMatch(pageURL, image.url);
 
       items.push({
         node: imageElement ? ByteEfficiencyAudit.makeNodeItem(imageElement.node) : undefined,

package/core/audits/byte-efficiency/offscreen-images.js

@@ -12,7 +12,7 @@
 import {ByteEfficiencyAudit} from './byte-efficiency-audit.js';
 import {NetworkRequest} from '../../lib/network-request.js';
 import {Sentry} from '../../lib/sentry.js';
-import URL from '../../lib/url-shim.js';
+import UrlUtils from '../../lib/url-utils.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 import Interactive from '../../computed/metrics/interactive.js';
 import ProcessedTrace from '../../computed/processed-trace.js';
@@ -87,7 +87,7 @@ class OffscreenImages extends ByteEfficiencyAudit {
     // If the image had its loading behavior explicitly controlled already, treat it as passed.
     if (image.loading === 'lazy' || image.loading === 'eager') return null;
 
-    const url = URL.elideDataURI(image.src);
+    const url = UrlUtils.elideDataURI(image.src);
     const totalPixels = image.displayedWidth * image.displayedHeight;
     const visiblePixels = this.computeVisiblePixels(image.clientRect, viewportDimensions);
     // Treat images with 0 area as if they're offscreen. See https://github.com/GoogleChrome/lighthouse/issues/1914

package/core/audits/byte-efficiency/uses-long-cache-ttl.js

@@ -8,7 +8,7 @@ import parseCacheControl from 'parse-cache-control';
 
 import {Audit} from '../audit.js';
 import {NetworkRequest} from '../../lib/network-request.js';
-import URL from '../../lib/url-shim.js';
+import UrlUtils from '../../lib/url-utils.js';
 import {linearInterpolation} from '../../lib/statistics.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 import NetworkRecords from '../../computed/network-records.js';
@@ -233,7 +233,7 @@ class CacheHeaders extends Audit {
         const cacheHitProbability = CacheHeaders.getCacheHitProbability(cacheLifetimeInSeconds);
         if (cacheHitProbability > IGNORE_THRESHOLD_IN_PERCENT) continue;
 
-        const url = URL.elideDataURI(record.url);
+        const url = UrlUtils.elideDataURI(record.url);
         const totalBytes = record.transferSize || 0;
         const wastedBytes = (1 - cacheHitProbability) * totalBytes;
 

package/core/audits/byte-efficiency/uses-optimized-images.js

@@ -10,7 +10,7 @@
 
 
 import {ByteEfficiencyAudit} from './byte-efficiency-audit.js';
-import URL from '../../lib/url-shim.js';
+import UrlUtils from '../../lib/url-utils.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 
 const UIStrings = {
@@ -83,7 +83,7 @@ class UsesOptimizedImages extends ByteEfficiencyAudit {
       const imageElement = imageElementsByURL.get(image.url);
 
       if (image.failed) {
-        warnings.push(`Unable to decode ${URL.getURLDisplayName(image.url)}`);
+        warnings.push(`Unable to decode ${UrlUtils.getURLDisplayName(image.url)}`);
         continue;
       } else if (/(jpeg|bmp)/.test(image.mimeType) === false) {
         continue;
@@ -94,7 +94,7 @@ class UsesOptimizedImages extends ByteEfficiencyAudit {
 
       if (typeof jpegSize === 'undefined') {
         if (!imageElement) {
-          warnings.push(`Unable to locate resource ${URL.getURLDisplayName(image.url)}`);
+          warnings.push(`Unable to locate resource ${UrlUtils.getURLDisplayName(image.url)}`);
           continue;
         }
 
@@ -111,8 +111,8 @@ class UsesOptimizedImages extends ByteEfficiencyAudit {
 
       if (image.originalSize < jpegSize + IGNORE_THRESHOLD_IN_BYTES) continue;
 
-      const url = URL.elideDataURI(image.url);
-      const isCrossOrigin = !URL.originsMatch(pageURL, image.url);
+      const url = UrlUtils.elideDataURI(image.url);
+      const isCrossOrigin = !UrlUtils.originsMatch(pageURL, image.url);
       const jpegSavings = UsesOptimizedImages.computeSavings({...image, jpegSize});
 
       items.push({

package/core/audits/byte-efficiency/uses-responsive-images-snapshot.js

@@ -13,7 +13,7 @@
 
 import {Audit} from '../audit.js';
 import * as UsesResponsiveImages from './uses-responsive-images.js';
-import URL from '../../lib/url-shim.js';
+import UrlUtils from '../../lib/url-utils.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 
 const UIStrings = {
@@ -71,7 +71,7 @@ class UsesResponsiveImagesSnapshot extends Audit {
 
       items.push({
         node: Audit.makeNodeItem(image.node),
-        url: URL.elideDataURI(image.src),
+        url: UrlUtils.elideDataURI(image.src),
         displayedDimensions: `${displayed.width}x${displayed.height}`,
         actualDimensions: `${actual.width}x${actual.height}`,
       });

package/core/audits/byte-efficiency/uses-responsive-images.js

@@ -16,7 +16,7 @@
 import {ByteEfficiencyAudit} from './byte-efficiency-audit.js';
 import {NetworkRequest} from '../../lib/network-request.js';
 import ImageRecords from '../../computed/image-records.js';
-import URL from '../../lib/url-shim.js';
+import UrlUtils from '../../lib/url-utils.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 
 const UIStrings = {
@@ -100,7 +100,7 @@ class UsesResponsiveImages extends ByteEfficiencyAudit {
     const displayed = this.getDisplayedDimensions(image, ViewportDimensions);
     const usedPixels = displayed.width * displayed.height;
 
-    const url = URL.elideDataURI(image.src);
+    const url = UrlUtils.elideDataURI(image.src);
     const actualPixels = image.naturalWidth * image.naturalHeight;
     const wastedRatio = 1 - (usedPixels / actualPixels);
     const totalBytes = NetworkRequest.getResourceSizeOnNetwork(networkRecord);

package/core/audits/byte-efficiency/uses-text-compression.js

@@ -10,7 +10,7 @@
 
 
 import {ByteEfficiencyAudit} from './byte-efficiency-audit.js';
-import URL from '../../lib/url-shim.js';
+import UrlUtils from '../../lib/url-utils.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 
 const UIStrings = {
@@ -68,7 +68,7 @@ class ResponsesAreCompressed extends ByteEfficiencyAudit {
       }
 
       // remove duplicates
-      const url = URL.elideDataURI(record.url);
+      const url = UrlUtils.elideDataURI(record.url);
       const isDuplicate = items.find(item => item.url === url &&
         item.totalBytes === record.resourceSize);
       if (isDuplicate) {

package/core/audits/dobetterweb/uses-http2.js

@@ -14,7 +14,7 @@
 
 import {Audit} from '../audit.js';
 import ThirdParty from '../../lib/third-party-web.js';
-import URL from '../../lib/url-shim.js';
+import UrlUtils from '../../lib/url-utils.js';
 import {ByteEfficiencyAudit} from '../byte-efficiency/byte-efficiency-audit.js';
 import Interactive from '../../computed/metrics/lantern-interactive.js';
 import {NetworkRequest} from '../../lib/network-request.js';
@@ -167,7 +167,7 @@ class UsesHTTP2Audit extends Audit {
     const groupedByOrigin = new Map();
     for (const record of networkRecords) {
       if (!UsesHTTP2Audit.isStaticAsset(record)) continue;
-      if (URL.isLikeLocalhost(record.parsedURL.host)) continue;
+      if (UrlUtils.isLikeLocalhost(record.parsedURL.host)) continue;
       const existing = groupedByOrigin.get(record.parsedURL.securityOrigin) || [];
       existing.push(record);
       groupedByOrigin.set(record.parsedURL.securityOrigin, existing);

package/core/audits/font-display.js

@@ -5,14 +5,15 @@
  */
 
 import {Audit} from './audit.js';
-import URL from '../lib/url-shim.js';
-const PASSING_FONT_DISPLAY_REGEX = /^(block|fallback|optional|swap)$/;
-const CSS_URL_REGEX = /url\((.*?)\)/;
-const CSS_URL_GLOBAL_REGEX = new RegExp(CSS_URL_REGEX, 'g');
+import UrlUtils from '../lib/url-utils.js';
 import * as i18n from '../lib/i18n/i18n.js';
 import {Sentry} from '../lib/sentry.js';
 import NetworkRecords from '../computed/network-records.js';
 
+const PASSING_FONT_DISPLAY_REGEX = /^(block|fallback|optional|swap)$/;
+const CSS_URL_REGEX = /url\((.*?)\)/;
+const CSS_URL_GLOBAL_REGEX = new RegExp(CSS_URL_REGEX, 'g');
+
 const UIStrings = {
   /** Title of a diagnostic audit that provides detail on if all the text on a webpage was visible while the page was loading its webfonts. This descriptive title is shown to users when the amount is acceptable and no user action is required. */
   title: 'All text remains visible during webfont loads',
@@ -98,7 +99,7 @@ class FontDisplay extends Audit {
         // Convert the relative CSS URL to an absolute URL and add it to the target set.
         for (const relativeURL of relativeURLs) {
           try {
-            const relativeRoot = URL.isValid(stylesheet.header.sourceURL) ?
+            const relativeRoot = UrlUtils.isValid(stylesheet.header.sourceURL) ?
               stylesheet.header.sourceURL : artifacts.URL.finalUrl;
             const absoluteURL = new URL(relativeURL, relativeRoot);
             targetURLSet.add(absoluteURL.href);
@@ -121,7 +122,7 @@ class FontDisplay extends Audit {
     /** @type {Map<string, number>} */
     const warningCountByOrigin = new Map();
     for (const warningUrl of warningUrls) {
-      const origin = URL.getOrigin(warningUrl);
+      const origin = UrlUtils.getOrigin(warningUrl);
       if (!origin) continue;
 
       const count = warningCountByOrigin.get(origin) || 0;

package/core/audits/image-aspect-ratio.js

@@ -12,7 +12,7 @@
 
 
 import {Audit} from './audit.js';
-import URL from '../lib/url-shim.js';
+import UrlUtils from '../lib/url-utils.js';
 import * as i18n from '../lib/i18n/i18n.js';
 
 const UIStrings = {
@@ -54,7 +54,7 @@ class ImageAspectRatio extends Audit {
    * @return {{url: string, node: LH.Audit.Details.NodeValue, displayedAspectRatio: string, actualAspectRatio: string, doRatiosMatch: boolean}}
    */
   static computeAspectRatios(image) {
-    const url = URL.elideDataURI(image.src);
+    const url = UrlUtils.elideDataURI(image.src);
     const actualAspectRatio = image.naturalDimensions.width / image.naturalDimensions.height;
     const displayedAspectRatio = image.displayedWidth / image.displayedHeight;
 
@@ -89,7 +89,7 @@ class ImageAspectRatio extends Audit {
       // - filter all svgs as they have no natural dimensions to audit
       // - filter out images that have falsy naturalWidth or naturalHeight
       return !image.isCss &&
-        URL.guessMimeType(image.src) !== 'image/svg+xml' &&
+        UrlUtils.guessMimeType(image.src) !== 'image/svg+xml' &&
         image.naturalDimensions &&
         image.naturalDimensions.height > 5 &&
         image.naturalDimensions.width > 5 &&

package/core/audits/image-size-responsive.js

@@ -11,7 +11,7 @@
 
 
 import {Audit} from './audit.js';
-import URL from '../lib/url-shim.js';
+import UrlUtils from '../lib/url-utils.js';
 import * as i18n from '../lib/i18n/i18n.js';
 
 /** @typedef {LH.Artifacts.ImageElement & Required<Pick<LH.Artifacts.ImageElement, 'naturalDimensions'>>} ImageWithNaturalDimensions */
@@ -96,7 +96,7 @@ function isCandidate(image) {
   ) {
     return false;
   }
-  if (URL.guessMimeType(image.src) === 'image/svg+xml') {
+  if (UrlUtils.guessMimeType(image.src) === 'image/svg+xml') {
     return false;
   }
   if (image.isCss) {
@@ -147,7 +147,7 @@ function getResult(image, DPR) {
   const [expectedWidth, expectedHeight] =
       expectedImageSize(image.displayedWidth, image.displayedHeight, DPR);
   return {
-    url: URL.elideDataURI(image.src),
+    url: UrlUtils.elideDataURI(image.src),
     node: Audit.makeNodeItem(image.node),
     displayedSize: `${image.displayedWidth} x ${image.displayedHeight}`,
     actualSize: `${image.naturalDimensions.width} x ${image.naturalDimensions.height}`,

package/core/audits/is-on-https.js

@@ -5,7 +5,7 @@
  */
 
 import {Audit} from './audit.js';
-import URL from '../lib/url-shim.js';
+import UrlUtils from '../lib/url-utils.js';
 import {NetworkRequest} from '../lib/network-request.js';
 import NetworkRecords from '../computed/network-records.js';
 import * as i18n from '../lib/i18n/i18n.js';
@@ -75,7 +75,7 @@ class HTTPS extends Audit {
     return NetworkRecords.request(devtoolsLogs, context).then(networkRecords => {
       const insecureURLs = networkRecords
           .filter(record => !NetworkRequest.isSecureRequest(record))
-          .map(record => URL.elideDataURI(record.url));
+          .map(record => UrlUtils.elideDataURI(record.url));
 
       /** @type {Array<{url: string, resolution?: LH.IcuMessage|string}>}  */
       const items = Array.from(new Set(insecureURLs)).map(url => ({url, resolution: undefined}));

package/core/audits/network-requests.js

@@ -5,7 +5,7 @@
  */
 
 import {Audit} from './audit.js';
-import URL from '../lib/url-shim.js';
+import UrlUtils from '../lib/url-utils.js';
 import NetworkRecords from '../computed/network-records.js';
 import MainResource from '../computed/main-resource.js';
 
@@ -62,7 +62,7 @@ class NetworkRequests extends Audit {
         undefined;
 
       return {
-        url: URL.elideDataURI(record.url),
+        url: UrlUtils.elideDataURI(record.url),
         protocol: record.protocol,
         startTime: timeToMs(record.startTime),
         endTime: timeToMs(record.endTime),

package/core/audits/seo/canonical.js

@@ -5,7 +5,7 @@
  */
 
 import {Audit} from '../audit.js';
-import URL from '../../lib/url-shim.js';
+import UrlUtils from '../../lib/url-utils.js';
 import MainResource from '../../computed/main-resource.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 
@@ -93,7 +93,7 @@ class Canonical extends Audit {
         // Links that had an hrefRaw but didn't have a valid href were invalid, flag them
         if (!link.href) invalidCanonicalLink = link;
         // Links that had a valid href but didn't have a valid hrefRaw must have been relatively resolved, flag them
-        else if (!URL.isValid(link.hrefRaw)) relativeCanonicallink = link;
+        else if (!UrlUtils.isValid(link.hrefRaw)) relativeCanonicallink = link;
         // Otherwise, it was a valid canonical URL
         else uniqueCanonicalURLs.add(link.href);
       } else if (link.rel === 'alternate') {

package/core/audits/seo/is-crawlable.js

@@ -7,7 +7,6 @@
 import robotsParser from 'robots-parser';
 
 import {Audit} from '../audit.js';
-import URL from '../../lib/url-shim.js';
 import MainResource from '../../computed/main-resource.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 

package/core/audits/seo/link-text.js

@@ -5,7 +5,7 @@
  */
 
 import {Audit} from '../audit.js';
-import URL from '../../lib/url-shim.js';
+import UrlUtils from '../../lib/url-utils.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 
 const BLOCKLIST = new Set([
@@ -121,7 +121,7 @@ class LinkText extends Audit {
           href.startsWith('mailto:') ||
           // This line prevents the audit from flagging anchor links.
           // In this case it is better to use `finalUrl` than `mainDocumentUrl`.
-          URL.equalWithExcludedFragments(link.href, artifacts.URL.finalUrl)
+          UrlUtils.equalWithExcludedFragments(link.href, artifacts.URL.finalUrl)
         ) {
           return false;
         }

package/core/audits/seo/plugins.js

@@ -5,7 +5,6 @@
  */
 
 import {Audit} from '../audit.js';
-import URL from '../../lib/url-shim.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 
 const JAVA_APPLET_TYPE = 'application/x-java-applet';

package/core/audits/seo/robots-txt.js

@@ -12,7 +12,6 @@
  */
 
 import {Audit} from '../audit.js';
-import URL from '../../lib/url-shim.js';
 import * as i18n from '../../lib/i18n/i18n.js';
 
 const HTTP_CLIENT_ERROR_CODE_LOW = 400;

package/core/audits/service-worker.js

@@ -4,7 +4,6 @@
  * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
  */
 
-import URL from '../lib/url-shim.js';
 import {Audit} from './audit.js';
 import * as i18n from '../lib/i18n/i18n.js';
 

package/core/audits/unsized-images.js

@@ -11,7 +11,7 @@
 
 import {Audit} from './audit.js';
 import * as i18n from './../lib/i18n/i18n.js';
-import URL from './../lib/url-shim.js';
+import UrlUtils from '../lib/url-utils.js';
 
 const UIStrings = {
   /** Title of a Lighthouse audit that provides detail on whether all images have explicit width and height. This descriptive title is shown to users when every image has explicit width and height */
@@ -105,9 +105,9 @@ class UnsizedImages extends Audit {
    * @return {boolean}
    */
   static isNonNetworkSvg(image) {
-    const isSvg = URL.guessMimeType(image.src) === 'image/svg+xml';
+    const isSvg = UrlUtils.guessMimeType(image.src) === 'image/svg+xml';
     const urlScheme = image.src.slice(0, image.src.indexOf(':'));
-    const isNonNetwork = URL.isNonNetworkProtocol(urlScheme);
+    const isNonNetwork = UrlUtils.isNonNetworkProtocol(urlScheme);
     return isSvg && isNonNetwork;
   }
 
@@ -139,7 +139,7 @@ class UnsizedImages extends Audit {
       if (isNotDisplayed) continue;
 
       unsizedImages.push({
-        url: URL.elideDataURI(image.src),
+        url: UrlUtils.elideDataURI(image.src),
         node: Audit.makeNodeItem(image.node),
       });
     }

package/core/audits/uses-rel-preconnect.js

@@ -6,7 +6,7 @@
 
 import {Audit} from './audit.js';
 import {ByteEfficiencyAudit} from './byte-efficiency/byte-efficiency-audit.js';
-import URL from '../lib/url-shim.js';
+import UrlUtils from '../lib/url-utils.js';
 import * as i18n from '../lib/i18n/i18n.js';
 import NetworkRecords from '../computed/network-records.js';
 import MainResource from '../computed/main-resource.js';
@@ -162,7 +162,8 @@ class UsesRelPreconnectAudit extends Audit {
       });
 
     const preconnectLinks = artifacts.LinkElements.filter(el => el.rel === 'preconnect');
-    const preconnectOrigins = new Set(preconnectLinks.map(link => URL.getOrigin(link.href || '')));
+    const preconnectOrigins =
+      new Set(preconnectLinks.map(link => UrlUtils.getOrigin(link.href || '')));
 
     /** @type {Array<{url: string, wastedMs: number}>}*/
     let results = [];

package/core/audits/uses-rel-preload.js

@@ -4,7 +4,7 @@
  * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
  */
 
-import URL from '../lib/url-shim.js';
+import UrlUtils from '../lib/url-utils.js';
 import {NetworkRequest} from '../lib/network-request.js';
 import {Audit} from './audit.js';
 import {ByteEfficiencyAudit} from './byte-efficiency/byte-efficiency-audit.js';
@@ -128,7 +128,7 @@ class UsesRelPreloadAudit extends Audit {
     // It's not a request for the main frame, it wouldn't get reused even if you did preload it.
     if (request.frameId !== mainResource.frameId) return false;
     // We survived everything else, just check that it's a first party request.
-    return URL.rootDomainsMatch(request.url, mainResource.url);
+    return UrlUtils.rootDomainsMatch(request.url, mainResource.url);
   }
 
   /**

package/core/computed/image-records.js

@@ -4,7 +4,7 @@
  * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
  */
 
-import URL from '../lib/url-shim.js';
+import UrlUtils from '../lib/url-utils.js';
 import {makeComputedArtifact} from './computed-artifact.js';
 
 class ImageRecords {
@@ -43,7 +43,7 @@ class ImageRecords {
       // Don't change the guessed mime type if no mime type was found.
       imageRecords.push({
         ...element,
-        mimeType: mimeType ? mimeType : URL.guessMimeType(element.src),
+        mimeType: mimeType ? mimeType : UrlUtils.guessMimeType(element.src),
       });
     }
 

package/core/computed/resource-summary.js

@@ -6,7 +6,6 @@
 
 import {makeComputedArtifact} from './computed-artifact.js';
 import NetworkRecords from './network-records.js';
-import URL from '../lib/url-shim.js';
 import {NetworkRequest} from '../lib/network-request.js';
 import {Budget} from '../config/budget.js';
 import {Util} from '../util.cjs';

package/core/fraggle-rock/gather/navigation-runner.js

@@ -19,7 +19,7 @@ import {initializeConfig} from '../config/config.js';
 import {getBaseArtifacts, finalizeArtifacts} from './base-artifacts.js';
 import * as format from '../../../shared/localization/format.js';
 import {LighthouseError} from '../../lib/lh-error.js';
-import URL from '../../lib/url-shim.js';
+import UrlUtils from '../../lib/url-utils.js';
 import {getPageLoadError} from '../../lib/navigation-error.js';
 import Trace from '../../gather/gatherers/trace.js';
 import DevtoolsLog from '../../gather/gatherers/devtools-log.js';
@@ -28,6 +28,7 @@ import NetworkRecords from '../../computed/network-records.js';
 /**
  * @typedef NavigationContext
  * @property {Driver} driver
+ * @property {LH.Puppeteer.Page} page
  * @property {LH.Config.FRConfig} config
  * @property {LH.Config.NavigationDefn} navigation
  * @property {LH.NavigationRequestor} requestor
@@ -218,6 +219,7 @@ async function _navigation(navigationContext) {
     url: initialUrl,
     gatherMode: /** @type {const} */ ('navigation'),
     driver: navigationContext.driver,
+    page: navigationContext.page,
     computedCache: navigationContext.computedCache,
     artifactDefinitions: navigationContext.navigation.artifacts,
     artifactState,
@@ -250,10 +252,10 @@ async function _navigation(navigationContext) {
 }
 
 /**
- * @param {{driver: Driver, config: LH.Config.FRConfig, requestor: LH.NavigationRequestor; baseArtifacts: LH.FRBaseArtifacts, computedCache: NavigationContext['computedCache']}} args
+ * @param {{driver: Driver, page: LH.Puppeteer.Page, config: LH.Config.FRConfig, requestor: LH.NavigationRequestor; baseArtifacts: LH.FRBaseArtifacts, computedCache: NavigationContext['computedCache']}} args
  * @return {Promise<{artifacts: Partial<LH.FRArtifacts & LH.FRBaseArtifacts>}>}
  */
-async function _navigations({driver, config, requestor, baseArtifacts, computedCache}) {
+async function _navigations({driver, page, config, requestor, baseArtifacts, computedCache}) {
   if (!config.navigations) throw new Error('No navigations configured');
 
   /** @type {Partial<LH.FRArtifacts & LH.FRBaseArtifacts>} */
@@ -264,6 +266,7 @@ async function _navigations({driver, config, requestor, baseArtifacts, computedC
   for (const navigation of config.navigations) {
     const navigationContext = {
       driver,
+      page,
       navigation,
       requestor,
       config,
@@ -316,7 +319,7 @@ async function navigationGather(requestor, options) {
   const artifacts = await Runner.gather(
     async () => {
       let {page} = options;
-      const normalizedRequestor = isCallback ? requestor : URL.normalizeUrl(requestor);
+      const normalizedRequestor = isCallback ? requestor : UrlUtils.normalizeUrl(requestor);
 
       // For navigation mode, we shouldn't connect to a browser in audit mode,
       // therefore we connect to the browser in the gatherFn callback.
@@ -333,7 +336,7 @@ async function navigationGather(requestor, options) {
         requestor: normalizedRequestor,
       };
       const {baseArtifacts} = await _setup(context);
-      const {artifacts} = await _navigations({...context, baseArtifacts, computedCache});
+      const {artifacts} = await _navigations({...context, page, baseArtifacts, computedCache});
       await _cleanup(context);
 
       return finalizeArtifacts(baseArtifacts, artifacts);

package/core/fraggle-rock/gather/runner-helpers.js

@@ -7,6 +7,7 @@
 /**
  * @typedef CollectPhaseArtifactOptions
  * @property {import('./driver.js').Driver} driver
+ * @property {LH.Puppeteer.Page} page
  * @property {Array<LH.Config.AnyArtifactDefn>} artifactDefinitions
  * @property {ArtifactState} artifactState
  * @property {LH.FRBaseArtifacts} baseArtifacts
@@ -66,6 +67,7 @@ const phaseToPriorPhase = {
 async function collectPhaseArtifacts(options) {
   const {
     driver,
+    page,
     artifactDefinitions,
     artifactState,
     baseArtifacts,
@@ -91,6 +93,7 @@ async function collectPhaseArtifacts(options) {
       return gatherer[phase]({
         gatherMode,
         driver,
+        page,
         baseArtifacts,
         dependencies,
         computedCache,

package/core/fraggle-rock/gather/snapshot-runner.js

@@ -17,11 +17,11 @@ import {getBaseArtifacts, finalizeArtifacts} from './base-artifacts.js';
  * @return {Promise<LH.Gatherer.FRGatherResult>}
  */
 async function snapshotGather(options) {
-  const {flags = {}} = options;
+  const {page, flags = {}} = options;
   log.setLevel(flags.logLevel || 'error');
 
   const {config} = await initializeConfig('snapshot', options.config, flags);
-  const driver = new Driver(options.page);
+  const driver = new Driver(page);
   await driver.connect();
 
   /** @type {Map<string, LH.ArbitraryEqualityMap>} */
@@ -43,6 +43,7 @@ async function snapshotGather(options) {
         phase: 'getArtifact',
         gatherMode: 'snapshot',
         driver,
+        page,
         baseArtifacts,
         artifactDefinitions,
         artifactState,

package/core/fraggle-rock/gather/timespan-runner.js

@@ -18,11 +18,11 @@ import {getBaseArtifacts, finalizeArtifacts} from './base-artifacts.js';
  * @return {Promise<{endTimespanGather(): Promise<LH.Gatherer.FRGatherResult>}>}
  */
 async function startTimespanGather(options) {
-  const {flags = {}} = options;
+  const {page, flags = {}} = options;
   log.setLevel(flags.logLevel || 'error');
 
   const {config} = await initializeConfig('timespan', options.config, flags);
-  const driver = new Driver(options.page);
+  const driver = new Driver(page);
   await driver.connect();
 
   /** @type {Map<string, LH.ArbitraryEqualityMap>} */
@@ -34,6 +34,7 @@ async function startTimespanGather(options) {
   /** @type {Omit<import('./runner-helpers.js').CollectPhaseArtifactOptions, 'phase'>} */
   const phaseOptions = {
     driver,
+    page,
     artifactDefinitions,
     artifactState,
     baseArtifacts,

package/core/gather/driver/navigation.js

@@ -10,7 +10,7 @@ import {NetworkMonitor} from './network-monitor.js';
 import {waitForFullyLoaded, waitForFrameNavigated, waitForUserToContinue} from './wait-for-condition.js'; // eslint-disable-line max-len
 import * as constants from '../../config/constants.js';
 import * as i18n from '../../lib/i18n/i18n.js';
-import URL from '../../lib/url-shim.js';
+import UrlUtils from '../../lib/url-utils.js';
 
 const UIStrings = {
   /**
@@ -130,7 +130,7 @@ async function gotoURL(driver, requestor, options) {
 
   let requestedUrl = navigationUrls.requestedUrl;
   if (typeof requestor === 'string') {
-    if (requestedUrl && !URL.equalWithExcludedFragments(requestor, requestedUrl)) {
+    if (requestedUrl && !UrlUtils.equalWithExcludedFragments(requestor, requestedUrl)) {
       log.error(
         'Navigation',
         `Provided URL (${requestor}) did not match initial navigation URL (${requestedUrl})`
@@ -169,7 +169,7 @@ function getNavigationWarnings(navigation) {
 
   if (navigation.timedOut) warnings.push(str_(UIStrings.warningTimeout));
 
-  if (!URL.equalWithExcludedFragments(requestedUrl, mainDocumentUrl)) {
+  if (!UrlUtils.equalWithExcludedFragments(requestedUrl, mainDocumentUrl)) {
     warnings.push(str_(UIStrings.warningRedirected, {
       requested: requestedUrl,
       final: mainDocumentUrl,

package/core/gather/driver/network-monitor.js

@@ -15,7 +15,7 @@ import log from 'lighthouse-logger';
 
 import {NetworkRecorder} from '../../lib/network-recorder.js';
 import {NetworkRequest} from '../../lib/network-request.js';
-import URL from '../../lib/url-shim.js';
+import UrlUtils from '../../lib/url-utils.js';
 
 /** @typedef {import('../../lib/network-recorder.js').NetworkRecorderEventMap} NetworkRecorderEventMap */
 /** @typedef {'network-2-idle'|'network-critical-idle'|'networkidle'|'networkbusy'|'network-critical-busy'|'network-2-busy'} NetworkMonitorEvent_ */
@@ -214,7 +214,7 @@ class NetworkMonitor extends NetworkMonitorEventEmitter {
     /** @type {Array<{time: number, isStart: boolean}>} */
     let timeBoundaries = [];
     requests.forEach(request => {
-      if (URL.isNonNetworkProtocol(request.protocol)) return;
+      if (UrlUtils.isNonNetworkProtocol(request.protocol)) return;
       if (request.protocol === 'ws' || request.protocol === 'wss') return;
 
       // convert the network timestamp to ms

package/core/gather/gather-runner.js

@@ -21,7 +21,7 @@ import InstallabilityErrors from './gatherers/installability-errors.js';
 import NetworkUserAgent from './gatherers/network-user-agent.js';
 import Stacks from './gatherers/stacks.js';
 import {finalizeArtifacts} from '../fraggle-rock/gather/base-artifacts.js';
-import URLShim from '../lib/url-shim.js';
+import UrlUtils from '../lib/url-utils.js';
 
 /** @typedef {import('../gather/driver.js').Driver} Driver */
 /** @typedef {import('../lib/arbitrary-equality-map.js').ArbitraryEqualityMap} ArbitraryEqualityMap */
@@ -494,7 +494,7 @@ class GatherRunner {
 
       // Hack for running benchmarkIndex extra times.
       // Add a `bidx=20` query param, eg: https://www.example.com/?bidx=50
-      const parsedUrl = URLShim.isValid(options.requestedUrl) && new URL(options.requestedUrl);
+      const parsedUrl = UrlUtils.isValid(options.requestedUrl) && new URL(options.requestedUrl);
       if (options.settings.channel === 'lr' && parsedUrl && parsedUrl.searchParams.has('bidx')) {
         const bidxRunCount = parsedUrl.searchParams.get('bidx') || 0;
         // Add the first bidx into the new set

package/core/gather/gatherers/dobetterweb/optimized-images.js

@@ -13,7 +13,7 @@
 import log from 'lighthouse-logger';
 
 import FRGatherer from '../../../fraggle-rock/gather/base-gatherer.js';
-import URL from '../../../lib/url-shim.js';
+import UrlUtils from '../../../lib/url-utils.js';
 import {NetworkRequest} from '../../../lib/network-request.js';
 import {Sentry} from '../../../lib/sentry.js';
 import NetworkRecords from '../../../computed/network-records.js';
@@ -139,7 +139,7 @@ class OptimizedImages extends FRGatherer {
         // want to tank the entire run due to a single image.
         Sentry.captureException(err, {
           tags: {gatherer: 'OptimizedImages'},
-          extra: {imageUrl: URL.elideDataURI(record.url)},
+          extra: {imageUrl: UrlUtils.elideDataURI(record.url)},
           level: 'warning',
         });
 

package/core/gather/gatherers/dobetterweb/response-compression.js

@@ -14,7 +14,7 @@ import {Buffer} from 'buffer';
 import {gzip} from 'zlib';
 
 import FRGatherer from '../../../fraggle-rock/gather/base-gatherer.js';
-import URL from '../../../lib/url-shim.js';
+import UrlUtils from '../../../lib/url-utils.js';
 import {Sentry} from '../../../lib/sentry.js';
 import {NetworkRequest} from '../../../lib/network-request.js';
 import DevtoolsLog from '../devtools-log.js';
@@ -122,7 +122,7 @@ class ResponseCompression extends FRGatherer {
       }).catch(err => {
         Sentry.captureException(err, {
           tags: {gatherer: 'ResponseCompression'},
-          extra: {url: URL.elideDataURI(record.url)},
+          extra: {url: UrlUtils.elideDataURI(record.url)},
           level: 'warning',
         });
 

package/core/gather/gatherers/link-elements.js

@@ -7,7 +7,6 @@
 import LinkHeader from 'http-link-header';
 
 import FRGatherer from '../../fraggle-rock/gather/base-gatherer.js';
-import URL from '../../lib/url-shim.js';
 import {pageFunctions} from '../../lib/page-functions.js';
 import DevtoolsLog from './devtools-log.js';
 import MainResource from '../../computed/main-resource.js';

package/core/gather/gatherers/source-maps.js

@@ -5,7 +5,6 @@
  */
 
 import FRGatherer from '../../fraggle-rock/gather/base-gatherer.js';
-import URL from '../../lib/url-shim.js';
 
 /**
  * @fileoverview Gets JavaScript source maps.

package/core/index.js

@@ -9,7 +9,7 @@ import log from 'lighthouse-logger';
 import {Runner} from './runner.js';
 import {CriConnection} from './gather/connections/cri.js';
 import {Config} from './config/config.js';
-import URL from './lib/url-shim.js';
+import UrlUtils from './lib/url-utils.js';
 import * as fraggleRock from './fraggle-rock/api.js';
 import {Driver} from './gather/driver.js';
 import {initializeConfig} from './fraggle-rock/config/config.js';
@@ -67,7 +67,7 @@ async function legacyNavigation(url, flags = {}, configJSON, userConnection) {
 
   // kick off a lighthouse run
   const artifacts = await Runner.gather(() => {
-    const requestedUrl = URL.normalizeUrl(url);
+    const requestedUrl = UrlUtils.normalizeUrl(url);
     return Runner._gatherArtifactsFromBrowser(requestedUrl, options, connection);
   }, options);
   return Runner.audit(artifacts, options);

package/core/lib/dependency-graph/simulator/network-analyzer.js

@@ -4,7 +4,7 @@
  * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
  */
 
-import URL from '../../url-shim.js';
+import UrlUtils from '../../url-utils.js';
 
 const INITIAL_CWD = 14 * 1024;
 
@@ -440,7 +440,7 @@ class NetworkAnalyzer {
     // equalWithExcludedFragments is expensive, so check that the resourceUrl starts with the request url first
     return records.find(request =>
       resourceUrl.startsWith(request.url) &&
-      URL.equalWithExcludedFragments(request.url, resourceUrl)
+      UrlUtils.equalWithExcludedFragments(request.url, resourceUrl)
     );
   }
 

package/core/lib/icons.js

@@ -4,8 +4,6 @@
  * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
  */
 
-import URL from './url-shim.js';
-
 /**
  * @param {NonNullable<LH.Artifacts.Manifest['value']>} manifest
  * @return {boolean} Does the manifest have any icons?

package/core/lib/manifest-parser.js

@@ -6,8 +6,6 @@
 
 import cssParsers from 'cssstyle/lib/parsers.js';
 
-import URL from './url-shim.js';
-
 const ALLOWED_DISPLAY_VALUES = [
   'fullscreen',
   'standalone',

package/core/lib/network-request.js

@@ -52,7 +52,7 @@
       Trace: ResourceFinish.ts
  */
 
-import URL from './url-shim.js';
+import UrlUtils from './url-utils.js';
 
 // Lightrider X-Header names for timing information.
 // See: _updateTransferSizeForLightrider and _updateTimingsForLightrider.
@@ -215,7 +215,7 @@ class NetworkRequest {
       host: url.hostname,
       securityOrigin: url.origin,
     };
-    this.isSecure = URL.isSecureScheme(this.parsedURL.scheme);
+    this.isSecure = UrlUtils.isSecureScheme(this.parsedURL.scheme);
 
     this.startTime = data.timestamp;
 
@@ -522,9 +522,9 @@ class NetworkRequest {
    */
   static isNonNetworkRequest(record) {
     // The 'protocol' field in devtools a string more like a `scheme`
-    return URL.isNonNetworkProtocol(record.protocol) ||
+    return UrlUtils.isNonNetworkProtocol(record.protocol) ||
       // But `protocol` can fail to be populated if the request fails, so fallback to scheme.
-      URL.isNonNetworkProtocol(record.parsedURL.scheme);
+      UrlUtils.isNonNetworkProtocol(record.parsedURL.scheme);
   }
 
   /**
@@ -535,9 +535,9 @@ class NetworkRequest {
    * @return {boolean}
    */
   static isSecureRequest(record) {
-    return URL.isSecureScheme(record.parsedURL.scheme) ||
-        URL.isSecureScheme(record.protocol) ||
-        URL.isLikeLocalhost(record.parsedURL.host) ||
+    return UrlUtils.isSecureScheme(record.parsedURL.scheme) ||
+        UrlUtils.isSecureScheme(record.protocol) ||
+        UrlUtils.isLikeLocalhost(record.parsedURL.host) ||
         NetworkRequest.isHstsRequest(record);
   }
 

package/core/lib/{url-shim.js → url-utils.js}

@@ -4,12 +4,8 @@
  * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
  */
 
-/**
- * URL shim so we keep our code DRY
- */
-
 import {Util} from '../util.cjs';
-import {LighthouseError} from '../lib/lh-error.js';
+import {LighthouseError} from './lh-error.js';
 
 /** @typedef {import('./network-request.js').NetworkRequest} NetworkRequest */
 
@@ -43,8 +39,7 @@ function rewriteChromeInternalUrl(url) {
   return url.replace(/^chrome:\/\/chrome\//, 'chrome://');
 }
 
-// URL is global as of node 10. https://nodejs.org/api/globals.html#globals_url
-class URLShim extends URL {
+class UrlUtils {
   /**
    * @param {string} url
    * @return {boolean}
@@ -264,11 +259,8 @@ class URLShim extends URL {
   }
 }
 
-URLShim.URL = URL;
-
-URLShim.INVALID_URL_DEBUG_STRING =
+UrlUtils.INVALID_URL_DEBUG_STRING =
     'Lighthouse was unable to determine the URL of some script executions. ' +
     'It\'s possible a Chrome extension or other eval\'d code is the source.';
 
-// TODO(esmodules): don't use default export
-export default URLShim;
+export default UrlUtils;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "lighthouse",
   "type": "module",
-  "version": "9.5.0-dev.20220904",
+  "version": "9.5.0-dev.20220907",
   "description": "Automated auditing, performance metrics, and best practices for the web.",
   "main": "./core/index.js",
   "bin": {
@@ -98,7 +98,6 @@
     "@build-tracker/cli": "^1.0.0-beta.15",
     "@esbuild-kit/esm-loader": "^2.1.1",
     "@jest/fake-timers": "^28.1.0",
-    "@puppeteer/replay": "^0.6.1",
     "@rollup/plugin-alias": "^3.1.2",
     "@rollup/plugin-commonjs": "^20.0.0",
     "@rollup/plugin-dynamic-import-vars": "^1.1.1",

package/report/test/renderer/performance-category-renderer-test.js

@@ -10,7 +10,6 @@ import jsdom from 'jsdom';
 
 import {Util} from '../../renderer/util.js';
 import {I18n} from '../../renderer/i18n.js';
-import URL from '../../../core/lib/url-shim.js';
 import {DOM} from '../../renderer/dom.js';
 import {DetailsRenderer} from '../../renderer/details-renderer.js';
 import {PerformanceCategoryRenderer} from '../../renderer/performance-category-renderer.js';

package/report/test/renderer/report-renderer-test.js

@@ -10,7 +10,6 @@ import jsdom from 'jsdom';
 import jestMock from 'jest-mock';
 
 import {Util} from '../../renderer/util.js';
-import URL from '../../../core/lib/url-shim.js';
 import {DOM} from '../../renderer/dom.js';
 import {DetailsRenderer} from '../../renderer/details-renderer.js';
 import {CategoryRenderer} from '../../renderer/category-renderer.js';

package/tsconfig.json

@@ -99,7 +99,7 @@
     "core/test/lib/tracehouse/task-summary-test.js",
     "core/test/lib/tracehouse/trace-processor-test.js",
     "core/test/lib/traces/metrics-trace-events-test.js",
-    "core/test/lib/url-shim-test.js",
+    "core/test/lib/url-utils-test.js",
     "core/test/network-records-to-devtools-log-test.js",
     "core/test/runner-test.js",
     "core/test/scoring-test.js",

package/types/gatherer.d.ts

@@ -52,6 +52,8 @@ declare module Gatherer {
     gatherMode: GatherMode;
     /** The connection to the page being analyzed. */
     driver: FRTransitionalDriver;
+    /** The Puppeteer page handle. Will be undefined in legacy navigation mode. */
+    page?: LH.Puppeteer.Page;
     /** The set of base artifacts that are always collected. */
     baseArtifacts: FRBaseArtifacts;
     /** The cached results of computed artifacts. */

package/core/fraggle-rock/replay/stringify-extension.js

@@ -1,104 +0,0 @@
-/**
- * @license Copyright 2022 The Lighthouse Authors. All Rights Reserved.
- * Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License. You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0
- * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
- */
-
-import * as PuppeteerReplay from '@puppeteer/replay';
-
-/**
- * @param {PuppeteerReplay.Schema.Step} step
- * @return {boolean}
- */
-function isNavigationStep(step) {
-  return Boolean(
-    step.type === 'navigate' ||
-    step.assertedEvents?.some(event => event.type === 'navigation')
-  );
-}
-
-class LighthouseStringifyExtension extends PuppeteerReplay.PuppeteerStringifyExtension {
-  #isProcessingTimespan = false;
-
-  /**
-   * @override
-   * @param {PuppeteerReplay.LineWriter} out
-   * @param {PuppeteerReplay.Schema.UserFlow} flow
-   */
-  async beforeAllSteps(out, flow) {
-    out.appendLine(`const fs = require('fs');`);
-
-    let isMobile = true;
-    for (const step of flow.steps) {
-      if (step.type !== 'setViewport') continue;
-      isMobile = step.isMobile;
-    }
-
-    await super.beforeAllSteps(out, flow);
-
-    const flags = {
-      screenEmulation: {
-        disabled: true,
-      },
-    };
-    out.appendLine(`const flags = ${JSON.stringify(flags)}`);
-    if (isMobile) {
-      out.appendLine(`const config = undefined;`);
-    } else {
-      // eslint-disable-next-line max-len
-      out.appendLine(`const config = (await import('lighthouse/core/config/desktop-config.js')).default;`);
-    }
-
-    out.appendLine(`const lhApi = await import('lighthouse/core/fraggle-rock/api.js');`);
-    // eslint-disable-next-line max-len
-    out.appendLine(`const lhFlow = await lhApi.startFlow(page, {name: ${JSON.stringify(flow.title)}, config, flags});`);
-  }
-
-  /**
-   * @override
-   * @param {PuppeteerReplay.LineWriter} out
-   * @param {PuppeteerReplay.Schema.Step} step
-   * @param {PuppeteerReplay.Schema.UserFlow} flow
-   */
-  async stringifyStep(out, step, flow) {
-    if (step.type === 'setViewport') {
-      await super.stringifyStep(out, step, flow);
-      return;
-    }
-
-    const isNavigation = isNavigationStep(step);
-
-    if (isNavigation) {
-      if (this.#isProcessingTimespan) {
-        out.appendLine(`await lhFlow.endTimespan();`);
-        this.#isProcessingTimespan = false;
-      }
-      out.appendLine(`await lhFlow.startNavigation();`);
-    } else if (!this.#isProcessingTimespan) {
-      out.appendLine(`await lhFlow.startTimespan();`);
-      this.#isProcessingTimespan = true;
-    }
-
-    await super.stringifyStep(out, step, flow);
-
-    if (isNavigation) {
-      out.appendLine(`await lhFlow.endNavigation();`);
-    }
-  }
-
-  /**
-   * @override
-   * @param {PuppeteerReplay.LineWriter} out
-   * @param {PuppeteerReplay.Schema.UserFlow} flow
-   */
-  async afterAllSteps(out, flow) {
-    if (this.#isProcessingTimespan) {
-      out.appendLine(`await lhFlow.endTimespan();`);
-    }
-    out.appendLine(`const lhFlowReport = await lhFlow.generateReport();`);
-    out.appendLine(`fs.writeFileSync(__dirname + '/flow.report.html', lhFlowReport)`);
-    await super.afterAllSteps(out, flow);
-  }
-}
-
-export default LighthouseStringifyExtension;