lighthouse 10.4.0-dev.20230713 → 10.4.0-dev.20230715

package/core/gather/gatherers/network-user-agent.js

@@ -4,11 +4,11 @@
  * 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 FRGatherer from '../base-gatherer.js';
+import BaseGatherer from '../base-gatherer.js';
 import DevtoolsLogGatherer from './devtools-log.js';
 
-/** @implements {LH.Gatherer.FRGathererInstance<'DevtoolsLog'>} */
-class NetworkUserAgent extends FRGatherer {
+/** @implements {LH.Gatherer.GathererInstance<'DevtoolsLog'>} */
+class NetworkUserAgent extends BaseGatherer {
   /** @type {LH.Gatherer.GathererMeta<'DevtoolsLog'>} */
   meta = {
     supportedModes: ['timespan', 'navigation'],
@@ -30,7 +30,7 @@ class NetworkUserAgent extends FRGatherer {
   }
 
   /**
-   * @param {LH.Gatherer.FRTransitionalContext<'DevtoolsLog'>} context
+   * @param {LH.Gatherer.Context<'DevtoolsLog'>} context
    * @return {Promise<LH.Artifacts['NetworkUserAgent']>}
    */
   async getArtifact(context) {

package/core/gather/gatherers/script-elements.d.ts

@@ -2,25 +2,20 @@ export default ScriptElements;
 /**
  * @fileoverview Gets JavaScript file contents.
  */
-declare class ScriptElements extends FRGatherer {
+declare class ScriptElements extends BaseGatherer {
     /** @type {LH.Gatherer.GathererMeta<'DevtoolsLog'>} */
     meta: LH.Gatherer.GathererMeta<'DevtoolsLog'>;
     /**
-     * @param {LH.Gatherer.FRTransitionalContext} context
+     * @param {LH.Gatherer.Context} context
      * @param {LH.Artifacts.NetworkRequest[]} networkRecords
      * @return {Promise<LH.Artifacts['ScriptElements']>}
      */
-    _getArtifact(context: LH.Gatherer.FRTransitionalContext, networkRecords: LH.Artifacts.NetworkRequest[]): Promise<LH.Artifacts['ScriptElements']>;
+    _getArtifact(context: LH.Gatherer.Context, networkRecords: LH.Artifacts.NetworkRequest[]): Promise<LH.Artifacts['ScriptElements']>;
     /**
-     * @param {LH.Gatherer.FRTransitionalContext<'DevtoolsLog'>} context
+     * @param {LH.Gatherer.Context<'DevtoolsLog'>} context
      */
-    getArtifact(context: LH.Gatherer.FRTransitionalContext<'DevtoolsLog'>): Promise<import("../../index.js").Artifacts.ScriptElement[]>;
-    /**
-     * @param {LH.Gatherer.PassContext} passContext
-     * @param {LH.Gatherer.LoadData} loadData
-     */
-    afterPass(passContext: LH.Gatherer.PassContext, loadData: LH.Gatherer.LoadData): Promise<import("../../index.js").Artifacts.ScriptElement[]>;
+    getArtifact(context: LH.Gatherer.Context<'DevtoolsLog'>): Promise<import("../../index.js").Artifacts.ScriptElement[]>;
 }
-import FRGatherer from '../base-gatherer.js';
+import BaseGatherer from '../base-gatherer.js';
 import { NetworkRequest } from '../../lib/network-request.js';
 //# sourceMappingURL=script-elements.d.ts.map

package/core/gather/gatherers/script-elements.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 FRGatherer from '../base-gatherer.js';
+import BaseGatherer from '../base-gatherer.js';
 import {NetworkRecords} from '../../computed/network-records.js';
 import {NetworkRequest} from '../../lib/network-request.js';
 import {pageFunctions} from '../../lib/page-functions.js';
@@ -39,7 +39,7 @@ function collectAllScriptElements() {
 /**
  * @fileoverview Gets JavaScript file contents.
  */
-class ScriptElements extends FRGatherer {
+class ScriptElements extends BaseGatherer {
   /** @type {LH.Gatherer.GathererMeta<'DevtoolsLog'>} */
   meta = {
     supportedModes: ['timespan', 'navigation'],
@@ -47,7 +47,7 @@ class ScriptElements extends FRGatherer {
   };
 
   /**
-   * @param {LH.Gatherer.FRTransitionalContext} context
+   * @param {LH.Gatherer.Context} context
    * @param {LH.Artifacts.NetworkRequest[]} networkRecords
    * @return {Promise<LH.Artifacts['ScriptElements']>}
    */
@@ -88,22 +88,13 @@ class ScriptElements extends FRGatherer {
   }
 
   /**
-   * @param {LH.Gatherer.FRTransitionalContext<'DevtoolsLog'>} context
+   * @param {LH.Gatherer.Context<'DevtoolsLog'>} context
    */
   async getArtifact(context) {
     const devtoolsLog = context.dependencies.DevtoolsLog;
     const networkRecords = await NetworkRecords.request(devtoolsLog, context);
     return this._getArtifact(context, networkRecords);
   }
-
-  /**
-   * @param {LH.Gatherer.PassContext} passContext
-   * @param {LH.Gatherer.LoadData} loadData
-   */
-  async afterPass(passContext, loadData) {
-    const networkRecords = loadData.networkRecords;
-    return this._getArtifact({...passContext, dependencies: {}}, networkRecords);
-  }
 }
 
 export default ScriptElements;

package/core/gather/gatherers/scripts.d.ts

@@ -2,7 +2,7 @@ export default Scripts;
 /**
  * @fileoverview Gets JavaScript file contents.
  */
-declare class Scripts extends FRGatherer {
+declare class Scripts extends BaseGatherer {
     /** @type {LH.Crdp.Debugger.ScriptParsedEvent[]} */
     _scriptParsedEvents: LH.Crdp.Debugger.ScriptParsedEvent[];
     /** @type {Array<string | undefined>} */
@@ -12,14 +12,14 @@ declare class Scripts extends FRGatherer {
      */
     onScriptParsed(params: LH.Crdp.Debugger.ScriptParsedEvent): void;
     /**
-     * @param {LH.Gatherer.FRTransitionalContext} context
+     * @param {LH.Gatherer.Context} context
      */
-    startInstrumentation(context: LH.Gatherer.FRTransitionalContext): Promise<void>;
+    startInstrumentation(context: LH.Gatherer.Context): Promise<void>;
     /**
-     * @param {LH.Gatherer.FRTransitionalContext} context
+     * @param {LH.Gatherer.Context} context
      */
-    stopInstrumentation(context: LH.Gatherer.FRTransitionalContext): Promise<void>;
+    stopInstrumentation(context: LH.Gatherer.Context): Promise<void>;
     getArtifact(): Promise<import("../../index.js").Artifacts.Script[]>;
 }
-import FRGatherer from '../base-gatherer.js';
+import BaseGatherer from '../base-gatherer.js';
 //# sourceMappingURL=scripts.d.ts.map

package/core/gather/gatherers/scripts.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 FRGatherer from '../base-gatherer.js';
+import BaseGatherer from '../base-gatherer.js';
 
 /**
  * @template T, U
@@ -47,7 +47,7 @@ function isLighthouseRuntimeEvaluateScript(script) {
 /**
  * @fileoverview Gets JavaScript file contents.
  */
-class Scripts extends FRGatherer {
+class Scripts extends BaseGatherer {
   /** @type {LH.Gatherer.GathererMeta} */
   meta = {
     supportedModes: ['timespan', 'navigation'],
@@ -74,7 +74,7 @@ class Scripts extends FRGatherer {
   }
 
   /**
-   * @param {LH.Gatherer.FRTransitionalContext} context
+   * @param {LH.Gatherer.Context} context
    */
   async startInstrumentation(context) {
     const session = context.driver.defaultSession;
@@ -83,7 +83,7 @@ class Scripts extends FRGatherer {
   }
 
   /**
-   * @param {LH.Gatherer.FRTransitionalContext} context
+   * @param {LH.Gatherer.Context} context
    */
   async stopInstrumentation(context) {
     const session = context.driver.defaultSession;

package/core/gather/gatherers/seo/embedded-content.d.ts

@@ -1,10 +1,10 @@
 export default EmbeddedContent;
-declare class EmbeddedContent extends FRGatherer {
+declare class EmbeddedContent extends BaseGatherer {
     /**
-     * @param {LH.Gatherer.FRTransitionalContext} passContext
+     * @param {LH.Gatherer.Context} passContext
      * @return {Promise<LH.Artifacts['EmbeddedContent']>}
      */
-    getArtifact(passContext: LH.Gatherer.FRTransitionalContext): Promise<LH.Artifacts['EmbeddedContent']>;
+    getArtifact(passContext: LH.Gatherer.Context): Promise<LH.Artifacts['EmbeddedContent']>;
 }
-import FRGatherer from '../../base-gatherer.js';
+import BaseGatherer from '../../base-gatherer.js';
 //# sourceMappingURL=embedded-content.d.ts.map

package/core/gather/gatherers/seo/embedded-content.js

@@ -6,7 +6,7 @@
 
 /* globals getElementsInDocument getNodeDetails */
 
-import FRGatherer from '../../base-gatherer.js';
+import BaseGatherer from '../../base-gatherer.js';
 import {pageFunctions} from '../../../lib/page-functions.js';
 
 /**
@@ -39,14 +39,14 @@ function getEmbeddedContent() {
     }));
 }
 
-class EmbeddedContent extends FRGatherer {
+class EmbeddedContent extends BaseGatherer {
   /** @type {LH.Gatherer.GathererMeta} */
   meta = {
     supportedModes: ['snapshot', 'navigation'],
   };
 
   /**
-   * @param {LH.Gatherer.FRTransitionalContext} passContext
+   * @param {LH.Gatherer.Context} passContext
    * @return {Promise<LH.Artifacts['EmbeddedContent']>}
    */
   getArtifact(passContext) {

package/core/gather/gatherers/seo/font-size.d.ts

@@ -4,12 +4,12 @@ export type BackendIdsToFontData = Map<number, {
     fontSize: number;
     textLength: number;
 }>;
-declare class FontSize extends FRGatherer {
+declare class FontSize extends BaseGatherer {
     /**
-     * @param {LH.Gatherer.FRProtocolSession} session
+     * @param {LH.Gatherer.ProtocolSession} session
      * @param {Array<NodeFontData>} failingNodes
      */
-    static fetchFailingNodeSourceRules(session: LH.Gatherer.FRProtocolSession, failingNodes: Array<NodeFontData>): Promise<{
+    static fetchFailingNodeSourceRules(session: LH.Gatherer.ProtocolSession, failingNodes: Array<NodeFontData>): Promise<{
         analyzedFailingNodesData: {
             nodeId: number;
             fontSize: number;
@@ -104,10 +104,10 @@ declare class FontSize extends FRGatherer {
         }[];
     };
     /**
-     * @param {LH.Gatherer.FRTransitionalContext} passContext
+     * @param {LH.Gatherer.Context} passContext
      * @return {Promise<LH.Artifacts.FontSize>} font-size analysis
      */
-    getArtifact(passContext: LH.Gatherer.FRTransitionalContext): Promise<LH.Artifacts.FontSize>;
+    getArtifact(passContext: LH.Gatherer.Context): Promise<LH.Artifacts.FontSize>;
 }
 /**
  * Returns the governing/winning CSS font-size rule for the set of styles given.
@@ -126,5 +126,5 @@ export function getEffectiveFontRule({ attributesStyle, inlineStyle, matchedCSSR
  * @return {NodeFontData['cssRule']|undefined}
  */
 export function findMostSpecificMatchedCSSRule(matchedCSSRules: import("devtools-protocol").Protocol.CSS.RuleMatch[] | undefined, isDeclarationOfInterest: (arg0: LH.Crdp.CSS.CSSStyle) => boolean | string | undefined): NodeFontData['cssRule'] | undefined;
-import FRGatherer from '../../base-gatherer.js';
+import BaseGatherer from '../../base-gatherer.js';
 //# sourceMappingURL=font-size.d.ts.map

package/core/gather/gatherers/seo/font-size.js

@@ -15,7 +15,7 @@
  * This gatherer collects stylesheet metadata by itself, instead of relying on the styles gatherer which is slow (because it parses the stylesheet content).
  */
 
-import FRGatherer from '../../base-gatherer.js';
+import BaseGatherer from '../../base-gatherer.js';
 
 const FONT_SIZE_PROPERTY_NAME = 'font-size';
 const MINIMAL_LEGIBLE_FONT_SIZE_PX = 12;
@@ -116,7 +116,7 @@ function getTextLength(text) {
 }
 
 /**
- * @param {LH.Gatherer.FRProtocolSession} session
+ * @param {LH.Gatherer.ProtocolSession} session
  * @param {number} nodeId text node
  * @return {Promise<NodeFontData['cssRule']|undefined>}
  */
@@ -138,14 +138,14 @@ async function fetchSourceRule(session, nodeId) {
   };
 }
 
-class FontSize extends FRGatherer {
+class FontSize extends BaseGatherer {
   /** @type {LH.Gatherer.GathererMeta} */
   meta = {
     supportedModes: ['snapshot', 'navigation'],
   };
 
   /**
-   * @param {LH.Gatherer.FRProtocolSession} session
+   * @param {LH.Gatherer.ProtocolSession} session
    * @param {Array<NodeFontData>} failingNodes
    */
   static async fetchFailingNodeSourceRules(session, failingNodes) {
@@ -274,7 +274,7 @@ class FontSize extends FRGatherer {
   }
 
   /**
-   * @param {LH.Gatherer.FRTransitionalContext} passContext
+   * @param {LH.Gatherer.Context} passContext
    * @return {Promise<LH.Artifacts.FontSize>} font-size analysis
    */
   async getArtifact(passContext) {

package/core/gather/gatherers/seo/robots-txt.d.ts

@@ -1,10 +1,10 @@
 export default RobotsTxt;
-declare class RobotsTxt extends FRGatherer {
+declare class RobotsTxt extends BaseGatherer {
     /**
-     * @param {LH.Gatherer.FRTransitionalContext} passContext
+     * @param {LH.Gatherer.Context} passContext
      * @return {Promise<LH.Artifacts['RobotsTxt']>}
      */
-    getArtifact(passContext: LH.Gatherer.FRTransitionalContext): Promise<LH.Artifacts['RobotsTxt']>;
+    getArtifact(passContext: LH.Gatherer.Context): Promise<LH.Artifacts['RobotsTxt']>;
 }
-import FRGatherer from '../../base-gatherer.js';
+import BaseGatherer from '../../base-gatherer.js';
 //# sourceMappingURL=robots-txt.d.ts.map

package/core/gather/gatherers/seo/robots-txt.js

@@ -4,16 +4,16 @@
  * 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 FRGatherer from '../../base-gatherer.js';
+import BaseGatherer from '../../base-gatherer.js';
 
-class RobotsTxt extends FRGatherer {
+class RobotsTxt extends BaseGatherer {
   /** @type {LH.Gatherer.GathererMeta} */
   meta = {
     supportedModes: ['snapshot', 'navigation'],
   };
 
   /**
-   * @param {LH.Gatherer.FRTransitionalContext} passContext
+   * @param {LH.Gatherer.Context} passContext
    * @return {Promise<LH.Artifacts['RobotsTxt']>}
    */
   async getArtifact(passContext) {

package/core/gather/gatherers/seo/tap-targets.d.ts

@@ -1,21 +1,21 @@
 export default TapTargets;
-declare class TapTargets extends FRGatherer {
+declare class TapTargets extends BaseGatherer {
     /**
-     * @param {LH.Gatherer.FRProtocolSession} session
+     * @param {LH.Gatherer.ProtocolSession} session
      * @param {string} className
      * @return {Promise<string>}
      */
-    addStyleRule(session: LH.Gatherer.FRProtocolSession, className: string): Promise<string>;
+    addStyleRule(session: LH.Gatherer.ProtocolSession, className: string): Promise<string>;
     /**
-     * @param {LH.Gatherer.FRProtocolSession} session
+     * @param {LH.Gatherer.ProtocolSession} session
      * @param {string} styleSheetId
      */
-    removeStyleRule(session: LH.Gatherer.FRProtocolSession, styleSheetId: string): Promise<void>;
+    removeStyleRule(session: LH.Gatherer.ProtocolSession, styleSheetId: string): Promise<void>;
     /**
-     * @param {LH.Gatherer.FRTransitionalContext} passContext
+     * @param {LH.Gatherer.Context} passContext
      * @return {Promise<LH.Artifacts.TapTarget[]>} All visible tap targets with their positions and sizes
      */
-    getArtifact(passContext: LH.Gatherer.FRTransitionalContext): Promise<LH.Artifacts.TapTarget[]>;
+    getArtifact(passContext: LH.Gatherer.Context): Promise<LH.Artifacts.TapTarget[]>;
 }
-import FRGatherer from '../../base-gatherer.js';
+import BaseGatherer from '../../base-gatherer.js';
 //# sourceMappingURL=tap-targets.d.ts.map

package/core/gather/gatherers/seo/tap-targets.js

@@ -6,7 +6,7 @@
 
 /* global document, window, getComputedStyle, getElementsInDocument, Node, getNodeDetails, getRectCenterPoint */
 
-import FRGatherer from '../../base-gatherer.js';
+import BaseGatherer from '../../base-gatherer.js';
 import {pageFunctions} from '../../../lib/page-functions.js';
 import * as RectHelpers from '../../../lib/rect-helpers.js';
 
@@ -301,7 +301,7 @@ function gatherTapTargetsAndResetScroll(tapTargetsSelector, className) {
 }
 /* c8 ignore stop */
 
-class TapTargets extends FRGatherer {
+class TapTargets extends BaseGatherer {
   constructor() {
     super();
     /**
@@ -315,7 +315,7 @@ class TapTargets extends FRGatherer {
   }
 
   /**
-   * @param {LH.Gatherer.FRProtocolSession} session
+   * @param {LH.Gatherer.ProtocolSession} session
    * @param {string} className
    * @return {Promise<string>}
    */
@@ -333,7 +333,7 @@ class TapTargets extends FRGatherer {
   }
 
   /**
-   * @param {LH.Gatherer.FRProtocolSession} session
+   * @param {LH.Gatherer.ProtocolSession} session
    * @param {string} styleSheetId
    */
   async removeStyleRule(session, styleSheetId) {
@@ -344,7 +344,7 @@ class TapTargets extends FRGatherer {
   }
 
   /**
-   * @param {LH.Gatherer.FRTransitionalContext} passContext
+   * @param {LH.Gatherer.Context} passContext
    * @return {Promise<LH.Artifacts.TapTarget[]>} All visible tap targets with their positions and sizes
    */
   async getArtifact(passContext) {

package/core/gather/gatherers/service-worker.d.ts

@@ -1,16 +1,10 @@
 export default ServiceWorker;
-declare class ServiceWorker extends FRGatherer {
+declare class ServiceWorker extends BaseGatherer {
     /**
-     * @param {LH.Gatherer.PassContext} passContext
+     * @param {LH.Gatherer.Context} context
      * @return {Promise<LH.Artifacts['ServiceWorker']>}
      */
-    beforePass(passContext: LH.Gatherer.PassContext): Promise<LH.Artifacts['ServiceWorker']>;
-    afterPass(): Promise<void>;
-    /**
-     * @param {LH.Gatherer.FRTransitionalContext} context
-     * @return {Promise<LH.Artifacts['ServiceWorker']>}
-     */
-    getArtifact(context: LH.Gatherer.FRTransitionalContext): Promise<LH.Artifacts['ServiceWorker']>;
+    getArtifact(context: LH.Gatherer.Context): Promise<LH.Artifacts['ServiceWorker']>;
 }
-import FRGatherer from '../base-gatherer.js';
+import BaseGatherer from '../base-gatherer.js';
 //# sourceMappingURL=service-worker.d.ts.map

package/core/gather/gatherers/service-worker.js

@@ -4,29 +4,17 @@
  * 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 FRGatherer from '../base-gatherer.js';
+import BaseGatherer from '../base-gatherer.js';
 import * as serviceWorkers from '../driver/service-workers.js';
 
-class ServiceWorker extends FRGatherer {
+class ServiceWorker extends BaseGatherer {
   /** @type {LH.Gatherer.GathererMeta} */
   meta = {
     supportedModes: ['navigation'],
   };
 
   /**
-   * @param {LH.Gatherer.PassContext} passContext
-   * @return {Promise<LH.Artifacts['ServiceWorker']>}
-   */
-  async beforePass(passContext) {
-    return this.getArtifact({...passContext, dependencies: {}});
-  }
-
-  // This gatherer is run in a separate pass for legacy mode.
-  // Legacy compat code is in `beforePass`.
-  async afterPass() { }
-
-  /**
-   * @param {LH.Gatherer.FRTransitionalContext} context
+   * @param {LH.Gatherer.Context} context
    * @return {Promise<LH.Artifacts['ServiceWorker']>}
    */
   async getArtifact(context) {

package/core/gather/gatherers/source-maps.d.ts

@@ -2,7 +2,7 @@ export default SourceMaps;
 /**
  * @fileoverview Gets JavaScript source maps.
  */
-declare class SourceMaps extends FRGatherer {
+declare class SourceMaps extends BaseGatherer {
     /** @type {LH.Crdp.Debugger.ScriptParsedEvent[]} */
     _scriptParsedEvents: LH.Crdp.Debugger.ScriptParsedEvent[];
     /**
@@ -10,11 +10,11 @@ declare class SourceMaps extends FRGatherer {
      */
     onScriptParsed(event: LH.Crdp.Debugger.ScriptParsedEvent): void;
     /**
-     * @param {LH.Gatherer.FRTransitionalDriver} driver
+     * @param {LH.Gatherer.Driver} driver
      * @param {string} sourceMapUrl
      * @return {Promise<LH.Artifacts.RawSourceMap>}
      */
-    fetchSourceMap(driver: LH.Gatherer.FRTransitionalDriver, sourceMapUrl: string): Promise<LH.Artifacts.RawSourceMap>;
+    fetchSourceMap(driver: LH.Gatherer.Driver, sourceMapUrl: string): Promise<LH.Artifacts.RawSourceMap>;
     /**
      * @param {string} sourceMapURL
      * @return {LH.Artifacts.RawSourceMap}
@@ -27,24 +27,24 @@ declare class SourceMaps extends FRGatherer {
      */
     _resolveUrl(url: string, base: string): string | undefined;
     /**
-     * @param {LH.Gatherer.FRTransitionalDriver} driver
+     * @param {LH.Gatherer.Driver} driver
      * @param {LH.Crdp.Debugger.ScriptParsedEvent} event
      * @return {Promise<LH.Artifacts.SourceMap>}
      */
-    _retrieveMapFromScriptParsedEvent(driver: LH.Gatherer.FRTransitionalDriver, event: LH.Crdp.Debugger.ScriptParsedEvent): Promise<LH.Artifacts.SourceMap>;
+    _retrieveMapFromScriptParsedEvent(driver: LH.Gatherer.Driver, event: LH.Crdp.Debugger.ScriptParsedEvent): Promise<LH.Artifacts.SourceMap>;
     /**
-     * @param {LH.Gatherer.FRTransitionalContext} context
+     * @param {LH.Gatherer.Context} context
      */
-    startSensitiveInstrumentation(context: LH.Gatherer.FRTransitionalContext): Promise<void>;
+    startSensitiveInstrumentation(context: LH.Gatherer.Context): Promise<void>;
     /**
-     * @param {LH.Gatherer.FRTransitionalContext} context
+     * @param {LH.Gatherer.Context} context
      */
-    stopSensitiveInstrumentation(context: LH.Gatherer.FRTransitionalContext): Promise<void>;
+    stopSensitiveInstrumentation(context: LH.Gatherer.Context): Promise<void>;
     /**
-     * @param {LH.Gatherer.FRTransitionalContext} context
+     * @param {LH.Gatherer.Context} context
      * @return {Promise<LH.Artifacts['SourceMaps']>}
      */
-    getArtifact(context: LH.Gatherer.FRTransitionalContext): Promise<LH.Artifacts['SourceMaps']>;
+    getArtifact(context: LH.Gatherer.Context): Promise<LH.Artifacts['SourceMaps']>;
 }
-import FRGatherer from '../base-gatherer.js';
+import BaseGatherer from '../base-gatherer.js';
 //# sourceMappingURL=source-maps.d.ts.map

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

@@ -5,12 +5,12 @@
  */
 
 import SDK from '../../lib/cdt/SDK.js';
-import FRGatherer from '../base-gatherer.js';
+import BaseGatherer from '../base-gatherer.js';
 
 /**
  * @fileoverview Gets JavaScript source maps.
  */
-class SourceMaps extends FRGatherer {
+class SourceMaps extends BaseGatherer {
   /** @type {LH.Gatherer.GathererMeta} */
   meta = {
     supportedModes: ['timespan', 'navigation'],
@@ -24,7 +24,7 @@ class SourceMaps extends FRGatherer {
   }
 
   /**
-   * @param {LH.Gatherer.FRTransitionalDriver} driver
+   * @param {LH.Gatherer.Driver} driver
    * @param {string} sourceMapUrl
    * @return {Promise<LH.Artifacts.RawSourceMap>}
    */
@@ -68,7 +68,7 @@ class SourceMaps extends FRGatherer {
   }
 
   /**
-   * @param {LH.Gatherer.FRTransitionalDriver} driver
+   * @param {LH.Gatherer.Driver} driver
    * @param {LH.Crdp.Debugger.ScriptParsedEvent} event
    * @return {Promise<LH.Artifacts.SourceMap>}
    */
@@ -125,7 +125,7 @@ class SourceMaps extends FRGatherer {
   }
 
   /**
-   * @param {LH.Gatherer.FRTransitionalContext} context
+   * @param {LH.Gatherer.Context} context
    */
   async startSensitiveInstrumentation(context) {
     const session = context.driver.defaultSession;
@@ -134,7 +134,7 @@ class SourceMaps extends FRGatherer {
   }
 
   /**
-   * @param {LH.Gatherer.FRTransitionalContext} context
+   * @param {LH.Gatherer.Context} context
    */
   async stopSensitiveInstrumentation(context) {
     const session = context.driver.defaultSession;
@@ -143,7 +143,7 @@ class SourceMaps extends FRGatherer {
   }
 
   /**
-   * @param {LH.Gatherer.FRTransitionalContext} context
+   * @param {LH.Gatherer.Context} context
    * @return {Promise<LH.Artifacts['SourceMaps']>}
    */
   async getArtifact(context) {

package/core/gather/gatherers/stacks.d.ts

@@ -21,18 +21,18 @@ export type JSLibrary = {
     version: string | number | null;
     npm: string | null;
 };
-/** @implements {LH.Gatherer.FRGathererInstance} */
-declare class Stacks extends FRGatherer implements LH.Gatherer.FRGathererInstance {
+/** @implements {LH.Gatherer.GathererInstance} */
+declare class Stacks extends BaseGatherer implements LH.Gatherer.GathererInstance {
     /**
-     * @param {LH.Gatherer.FRTransitionalDriver['executionContext']} executionContext
+     * @param {LH.Gatherer.Driver['executionContext']} executionContext
      * @return {Promise<LH.Artifacts['Stacks']>}
      */
-    static collectStacks(executionContext: LH.Gatherer.FRTransitionalDriver['executionContext']): Promise<LH.Artifacts['Stacks']>;
+    static collectStacks(executionContext: LH.Gatherer.Driver['executionContext']): Promise<LH.Artifacts['Stacks']>;
     /**
-     * @param {LH.Gatherer.FRTransitionalContext} context
+     * @param {LH.Gatherer.Context} context
      * @return {Promise<LH.Artifacts['Stacks']>}
      */
-    getArtifact(context: LH.Gatherer.FRTransitionalContext): Promise<LH.Artifacts['Stacks']>;
+    getArtifact(context: LH.Gatherer.Context): Promise<LH.Artifacts['Stacks']>;
 }
-import FRGatherer from '../base-gatherer.js';
+import BaseGatherer from '../base-gatherer.js';
 //# sourceMappingURL=stacks.d.ts.map

package/core/gather/gatherers/stacks.js

@@ -16,10 +16,10 @@ import {createRequire} from 'module';
 
 import log from 'lighthouse-logger';
 
-import FRGatherer from '../base-gatherer.js';
+import BaseGatherer from '../base-gatherer.js';
 
 
-// This is removed by rollup, because the only usage is to resolve a module path
+// This is removed by esbuild (if minified), because the only usage is to resolve a module path
 // but that is replaced by the inline-fs plugin, leaving `require` unused.
 const require = /* #__PURE__ */ createRequire(import.meta.url);
 
@@ -85,8 +85,8 @@ async function detectLibraries() {
 /* c8 ignore stop */
 
 
-/** @implements {LH.Gatherer.FRGathererInstance} */
-class Stacks extends FRGatherer {
+/** @implements {LH.Gatherer.GathererInstance} */
+class Stacks extends BaseGatherer {
   constructor() {
     super();
 
@@ -97,7 +97,7 @@ class Stacks extends FRGatherer {
   }
 
   /**
-   * @param {LH.Gatherer.FRTransitionalDriver['executionContext']} executionContext
+   * @param {LH.Gatherer.Driver['executionContext']} executionContext
    * @return {Promise<LH.Artifacts['Stacks']>}
    */
   static async collectStacks(executionContext) {
@@ -122,7 +122,7 @@ class Stacks extends FRGatherer {
   }
 
   /**
-   * @param {LH.Gatherer.FRTransitionalContext} context
+   * @param {LH.Gatherer.Context} context
    * @return {Promise<LH.Artifacts['Stacks']>}
    */
   async getArtifact(context) {