lighthouse 12.4.0-dev.20250319 → 12.4.0-dev.20250321

package/core/audits/audit.js

@@ -492,7 +492,8 @@ class Audit {
     const devtoolsLog = artifacts.devtoolsLogs[Audit.DEFAULT_PASS];
     const gatherContext = artifacts.GatherContext;
     const {URL, SourceMaps} = artifacts;
-    return {trace, devtoolsLog, gatherContext, settings: context.settings, URL, SourceMaps};
+    // eslint-disable-next-line max-len
+    return {trace, devtoolsLog, gatherContext, settings: context.settings, URL, SourceMaps, simulator: null};
   }
 }
 

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

@@ -199,7 +199,8 @@ class OffscreenImages extends ByteEfficiencyAudit {
     const unfilteredResults = Array.from(resultsMap.values());
     // get the interactive time or fallback to getting the end of trace time
     try {
-      const metricComputationData = {trace, devtoolsLog, gatherContext, settings, URL, SourceMaps};
+      const metricComputationData =
+        {trace, devtoolsLog, gatherContext, settings, URL, SourceMaps, simulator: null};
       const interactive = await Interactive.request(metricComputationData, context);
 
       // use interactive to generate items

package/core/audits/byte-efficiency/unused-javascript.js

@@ -94,7 +94,7 @@ class UnusedJavaScript extends ByteEfficiencyAudit {
       const script = artifacts.Scripts.find(s => s.scriptId === scriptId);
       if (!script) continue; // This should never happen.
 
-      const bundle = bundles.find(b => b.script.scriptId === scriptId);
+      const bundle = bundles.find(b => b.script.scriptId === scriptId) ?? null;
       const unusedJsSummary =
         await UnusedJavascriptSummary.request({scriptId, scriptCoverage, bundle}, context);
       if (unusedJsSummary.wastedBytes === 0 || unusedJsSummary.totalBytes === 0) continue;

package/core/audits/insights/use-cache-insight.js

@@ -1,5 +1,3 @@
-/* eslint-disable no-unused-vars */ // TODO: remove once implemented.
-
 /**
  * @license
  * Copyright 2025 Google LLC
@@ -10,7 +8,7 @@ import {UIStrings} from '@paulirish/trace_engine/models/trace/insights/UseCache.
 
 import {Audit} from '../audit.js';
 import * as i18n from '../../lib/i18n/i18n.js';
-import {adaptInsightToAuditProduct, makeNodeItemForNodeId} from './insight-audit.js';
+import {adaptInsightToAuditProduct} from './insight-audit.js';
 
 // eslint-disable-next-line max-len
 const str_ = i18n.createIcuMessageFn('node_modules/@paulirish/trace_engine/models/trace/insights/UseCache.js', UIStrings);
@@ -25,8 +23,9 @@ class UseCacheInsight extends Audit {
       title: str_(UIStrings.title),
       failureTitle: str_(UIStrings.title),
       description: str_(UIStrings.description),
-      guidanceLevel: 3, // TODO: confirm/change.
-      requiredArtifacts: ['traces', 'TraceElements'],
+      guidanceLevel: 3,
+      requiredArtifacts: ['traces', 'SourceMaps'],
+      replacesAudits: ['uses-long-cache-ttl'],
     };
   }
 
@@ -36,17 +35,25 @@ class UseCacheInsight extends Audit {
    * @return {Promise<LH.Audit.Product>}
    */
   static async audit(artifacts, context) {
-    // TODO: implement.
     return adaptInsightToAuditProduct(artifacts, context, 'UseCache', (insight) => {
       /** @type {LH.Audit.Details.Table['headings']} */
       const headings = [
         /* eslint-disable max-len */
+        {key: 'url', valueType: 'url', label: str_(UIStrings.requestColumn)},
+        {key: 'cacheLifetimeMs', valueType: 'ms', label: str_(UIStrings.cacheTTL), displayUnit: 'duration'},
+        {key: 'totalBytes', valueType: 'bytes', label: str_(i18n.UIStrings.columnTransferSize), displayUnit: 'kb', granularity: 1},
         /* eslint-enable max-len */
       ];
       /** @type {LH.Audit.Details.Table['items']} */
-      const items = [
-      ];
-      return Audit.makeTableDetails(headings, items);
+      const items = insight.requests.map(request => ({
+        url: request.request.args.data.url,
+        cacheLifetimeMs: request.ttl * 1000,
+        totalBytes: request.request.args.data.encodedDataLength,
+      }));
+      return Audit.makeTableDetails(headings, items, {
+        sortedBy: ['totalBytes'],
+        skipSumming: ['cacheLifetimeMs'],
+      });
     });
   }
 }

package/core/audits/largest-contentful-paint-element.js

@@ -123,8 +123,11 @@ class LargestContentfulPaintElement extends Audit {
     const trace = artifacts.traces[Audit.DEFAULT_PASS];
     const devtoolsLog = artifacts.devtoolsLogs[Audit.DEFAULT_PASS];
     const gatherContext = artifacts.GatherContext;
-    const metricComputationData = {trace, devtoolsLog, gatherContext,
-      settings: context.settings, URL: artifacts.URL, SourceMaps: artifacts.SourceMaps};
+    const metricComputationData = {
+      trace, devtoolsLog, gatherContext,
+      settings: context.settings, URL: artifacts.URL,
+      SourceMaps: artifacts.SourceMaps, simulator: null,
+    };
 
     const elementTable = this.makeElementTable(artifacts);
     if (!elementTable) {

package/core/audits/long-tasks.js

@@ -193,7 +193,8 @@ class LongTasks extends Audit {
 
       const simulatorOptions = {devtoolsLog, settings: context.settings};
       const pageGraph =
-        await PageDependencyGraph.request({settings, trace, devtoolsLog, URL, SourceMaps}, context);
+        // eslint-disable-next-line max-len
+        await PageDependencyGraph.request({settings, trace, devtoolsLog, URL, SourceMaps, fromTrace: false}, context);
       const simulator = await LoadSimulator.request(simulatorOptions, context);
       const simulation = simulator.simulate(pageGraph, {label: 'long-tasks-diagnostic'});
       for (const [node, timing] of simulation.nodeTimings.entries()) {

package/core/audits/metrics/first-contentful-paint.js

@@ -65,7 +65,9 @@ class FirstContentfulPaint extends Audit {
     const devtoolsLog = artifacts.devtoolsLogs[Audit.DEFAULT_PASS];
     const gatherContext = artifacts.GatherContext;
     const metricComputationData = {trace, devtoolsLog, gatherContext,
-      settings: context.settings, URL: artifacts.URL, SourceMaps: artifacts.SourceMaps};
+      settings: context.settings, URL: artifacts.URL,
+      SourceMaps: artifacts.SourceMaps, simulator: null,
+    };
     const metricResult = await ComputedFcp.request(metricComputationData, context);
     const options = context.options[context.settings.formFactor];
 

package/core/audits/metrics/interactive.js

@@ -70,8 +70,11 @@ class InteractiveMetric extends Audit {
     const trace = artifacts.traces[Audit.DEFAULT_PASS];
     const devtoolsLog = artifacts.devtoolsLogs[Audit.DEFAULT_PASS];
     const gatherContext = artifacts.GatherContext;
-    const metricComputationData = {trace, devtoolsLog, gatherContext,
-      settings: context.settings, URL: artifacts.URL, SourceMaps: artifacts.SourceMaps};
+    const metricComputationData = {
+      trace, devtoolsLog, gatherContext,
+      settings: context.settings, URL: artifacts.URL,
+      SourceMaps: artifacts.SourceMaps, simulator: null,
+    };
     const metricResult = await Interactive.request(metricComputationData, context);
     const timeInMs = metricResult.timing;
     const options = context.options[context.settings.formFactor];

package/core/audits/metrics/largest-contentful-paint.js

@@ -73,8 +73,11 @@ class LargestContentfulPaint extends Audit {
     const trace = artifacts.traces[Audit.DEFAULT_PASS];
     const devtoolsLog = artifacts.devtoolsLogs[Audit.DEFAULT_PASS];
     const gatherContext = artifacts.GatherContext;
-    const metricComputationData = {trace, devtoolsLog, gatherContext,
-      settings: context.settings, URL: artifacts.URL, SourceMaps: artifacts.SourceMaps};
+    const metricComputationData = {
+      trace, devtoolsLog, gatherContext,
+      settings: context.settings, URL: artifacts.URL,
+      SourceMaps: artifacts.SourceMaps, simulator: null,
+    };
 
     const metricResult = await ComputedLcp.request(metricComputationData, context);
     const options = context.options[context.settings.formFactor];

package/core/audits/metrics/max-potential-fid.js

@@ -120,8 +120,11 @@ class MaxPotentialFID extends Audit {
     const trace = artifacts.traces[Audit.DEFAULT_PASS];
     const devtoolsLog = artifacts.devtoolsLogs[Audit.DEFAULT_PASS];
     const gatherContext = artifacts.GatherContext;
-    const metricComputationData = {trace, devtoolsLog, gatherContext,
-      settings: context.settings, URL: artifacts.URL, SourceMaps: artifacts.SourceMaps};
+    const metricComputationData = {
+      trace, devtoolsLog, gatherContext,
+      settings: context.settings, URL: artifacts.URL,
+      SourceMaps: artifacts.SourceMaps, simulator: null,
+    };
     const metricResult = await ComputedFid.request(metricComputationData, context);
 
     const processedTrace = await ProcessedTrace.request(trace, context);

package/core/audits/metrics/speed-index.js

@@ -66,8 +66,11 @@ class SpeedIndex extends Audit {
     const trace = artifacts.traces[Audit.DEFAULT_PASS];
     const devtoolsLog = artifacts.devtoolsLogs[Audit.DEFAULT_PASS];
     const gatherContext = artifacts.GatherContext;
-    const metricComputationData = {trace, devtoolsLog, gatherContext,
-      settings: context.settings, URL: artifacts.URL, SourceMaps: artifacts.SourceMaps};
+    const metricComputationData = {
+      trace, devtoolsLog, gatherContext,
+      settings: context.settings, URL: artifacts.URL,
+      SourceMaps: artifacts.SourceMaps, simulator: null,
+    };
     const metricResult = await ComputedSi.request(metricComputationData, context);
     const options = context.options[context.settings.formFactor];
 

package/core/audits/metrics/total-blocking-time.js

@@ -88,8 +88,11 @@ class TotalBlockingTime extends Audit {
     const trace = artifacts.traces[Audit.DEFAULT_PASS];
     const devtoolsLog = artifacts.devtoolsLogs[Audit.DEFAULT_PASS];
     const gatherContext = artifacts.GatherContext;
-    const metricComputationData = {trace, devtoolsLog, gatherContext,
-      settings: context.settings, URL: artifacts.URL, SourceMaps: artifacts.SourceMaps};
+    const metricComputationData = {
+      trace, devtoolsLog, gatherContext,
+      settings: context.settings, URL: artifacts.URL,
+      SourceMaps: artifacts.SourceMaps, simulator: null,
+    };
     if (
       gatherContext.gatherMode === 'timespan' &&
       context.settings.throttlingMethod === 'simulate'

package/core/audits/predictive-perf.js

@@ -49,7 +49,8 @@ class PredictivePerf extends Audit {
     const {URL, SourceMaps} = artifacts;
     /** @type {LH.Config.Settings} */
     const settings = JSON.parse(JSON.stringify(defaultSettings)); // Use default settings.
-    const computationData = {trace, devtoolsLog, gatherContext, settings, URL, SourceMaps};
+    const computationData =
+      {trace, devtoolsLog, gatherContext, settings, URL, SourceMaps, simulator: null};
     const fcp = await LanternFirstContentfulPaint.request(computationData, context);
     const tti = await LanternInteractive.request(computationData, context);
     const si = await LanternSpeedIndex.request(computationData, context);

package/core/audits/prioritize-lcp-image.js

@@ -240,7 +240,8 @@ class PrioritizeLcpImage extends Audit {
     const devtoolsLog = artifacts.devtoolsLogs[PrioritizeLcpImage.DEFAULT_PASS];
     const {URL, SourceMaps} = artifacts;
     const settings = context.settings;
-    const metricData = {trace, devtoolsLog, gatherContext, settings, URL, SourceMaps};
+    const metricData =
+      {trace, devtoolsLog, gatherContext, settings, URL, SourceMaps, simulator: null};
     const lcpElement = artifacts.TraceElements
       .find(element => element.traceEventType === 'largest-contentful-paint');
 

package/core/audits/redirects.js

@@ -93,7 +93,8 @@ class Redirects extends Audit {
     const networkRecords = await NetworkRecords.request(devtoolsLog, context);
     const documentRequests = Redirects.getDocumentRequestChain(networkRecords, processedTrace);
 
-    const metricComputationData = {trace, devtoolsLog, gatherContext, settings, URL, SourceMaps};
+    const metricComputationData =
+      {trace, devtoolsLog, gatherContext, settings, URL, SourceMaps, simulator: null};
     const metricResult = await LanternInteractive.request(metricComputationData, context);
 
     /** @type {Map<string, LH.Gatherer.Simulation.NodeTiming>} */

package/core/audits/script-treemap-data.js

@@ -178,7 +178,7 @@ class ScriptTreemapDataAudit extends Audit {
       if (script.scriptLanguage !== 'JavaScript') continue;
 
       const name = script.url;
-      const bundle = bundles.find(bundle => script.scriptId === bundle.script.scriptId);
+      const bundle = bundles.find(bundle => script.scriptId === bundle.script.scriptId) ?? null;
       const scriptCoverage = /** @type {LH.Artifacts['JsUsage'][string] | undefined} */
         (artifacts.JsUsage[script.scriptId]);
       const unusedJavascriptSummary = scriptCoverage ?

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

@@ -124,8 +124,8 @@ class UsesRelPreconnectAudit extends Audit {
   static async audit(artifacts, context) {
     const trace = artifacts.traces[UsesRelPreconnectAudit.DEFAULT_PASS];
     const devtoolsLog = artifacts.devtoolsLogs[UsesRelPreconnectAudit.DEFAULT_PASS];
-    const settings = context.settings;
     const {URL, SourceMaps} = artifacts;
+    const settings = context.settings;
 
     let maxWastedLcp = 0;
     let maxWastedFcp = 0;
@@ -138,7 +138,8 @@ class UsesRelPreconnectAudit extends Audit {
         MainResource.request({devtoolsLog, URL}, context),
         LoadSimulator.request({devtoolsLog, settings}, context),
         ProcessedNavigation.request(trace, context),
-        PageDependencyGraph.request({settings, trace, devtoolsLog, URL, SourceMaps}, context),
+        PageDependencyGraph.request(
+          {settings, trace, devtoolsLog, URL, SourceMaps, fromTrace: false}, context),
       ]);
 
     const {rtt, additionalRttByOrigin} = loadSimulator.getOptions();

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

@@ -220,7 +220,8 @@ class UsesRelPreloadAudit extends Audit {
 
     const [mainResource, graph, simulator] = await Promise.all([
       MainResource.request({devtoolsLog, URL}, context),
-      PageDependencyGraph.request({settings, trace, devtoolsLog, URL, SourceMaps}, context),
+      PageDependencyGraph.request(
+        {settings, trace, devtoolsLog, URL, SourceMaps, fromTrace: false}, context),
       LoadSimulator.request(simulatorOptions, context),
     ]);
 

package/core/computed/computed-artifact.d.ts

@@ -4,7 +4,11 @@
  * @template {{name: string, compute_(dependencies: unknown, context: LH.Artifacts.ComputedContext): Promise<unknown>}} C
  * @template {Array<keyof LH.Util.FirstParamType<C['compute_']>>} K
  * @param {C} computableArtifact
- * @param {(K & ([keyof LH.Util.FirstParamType<C['compute_']>] extends [K[number]] ? unknown : never)) | null} keys List of properties of `dependencies` used by `compute_`; other properties are filtered out. Use `null` to allow all properties. Ensures that only required properties are used for caching result.
+ * @param {(K & ([keyof LH.Util.FirstParamType<C['compute_']>] extends [K[number]] ? unknown : never)) | null} keys
+ *        List of properties of `dependencies` used by `compute_`; other properties are filtered out.
+ *        Use `null` to allow all properties. Ensures that only required properties are used for caching result.
+ *        For optional properties of `dependencies`, undefined cannot be used and if found is treated as an error.
+ *        This is to guard against developer mistakes. For optional properties, make it nullable instead.
  */
 export function makeComputedArtifact<C extends {
     name: string;

package/core/computed/computed-artifact.js

@@ -7,6 +7,7 @@
 import log from 'lighthouse-logger';
 
 import {ArbitraryEqualityMap} from '../lib/arbitrary-equality-map.js';
+import {isUnderTest} from '../lib/lh-env.js';
 
 /**
  * Decorate computableArtifact with a caching `request()` method which will
@@ -14,7 +15,11 @@ import {ArbitraryEqualityMap} from '../lib/arbitrary-equality-map.js';
  * @template {{name: string, compute_(dependencies: unknown, context: LH.Artifacts.ComputedContext): Promise<unknown>}} C
  * @template {Array<keyof LH.Util.FirstParamType<C['compute_']>>} K
  * @param {C} computableArtifact
- * @param {(K & ([keyof LH.Util.FirstParamType<C['compute_']>] extends [K[number]] ? unknown : never)) | null} keys List of properties of `dependencies` used by `compute_`; other properties are filtered out. Use `null` to allow all properties. Ensures that only required properties are used for caching result.
+ * @param {(K & ([keyof LH.Util.FirstParamType<C['compute_']>] extends [K[number]] ? unknown : never)) | null} keys
+ *        List of properties of `dependencies` used by `compute_`; other properties are filtered out.
+ *        Use `null` to allow all properties. Ensures that only required properties are used for caching result.
+ *        For optional properties of `dependencies`, undefined cannot be used and if found is treated as an error.
+ *        This is to guard against developer mistakes. For optional properties, make it nullable instead.
  */
 function makeComputedArtifact(computableArtifact, keys) {
   // tsc (3.1) has more difficulty with template inter-references in jsdoc, so
@@ -27,13 +32,29 @@ function makeComputedArtifact(computableArtifact, keys) {
    * @return {ReturnType<C['compute_']>}
    */
   const request = (dependencies, context) => {
+    const computedName = computableArtifact.name;
+
+    // Guard against missing properties. Optional properties must be passed as null - if missing or
+    // undefined, throw an error.
+    for (const key of keys || []) {
+      if (dependencies && typeof dependencies === 'object' && dependencies[key] === undefined) {
+        // eslint-disable-next-line max-len
+        const err = new Error(`missing required key "${String(key)}" for computed artifact ${computableArtifact.name}`);
+        if (isUnderTest) {
+          throw err;
+        } else {
+          // For now, simply log in production.
+          log.error(`lh:computed:${computedName}`, err);
+        }
+      }
+    }
+
     const pickedDependencies = keys ?
       Object.fromEntries(keys.map(key => [key, dependencies[key]])) :
       dependencies;
 
     // NOTE: break immutability solely for this caching-controller function.
     const computedCache = /** @type {Map<string, ArbitraryEqualityMap>} */ (context.computedCache);
-    const computedName = computableArtifact.name;
 
     const cache = computedCache.get(computedName) || new ArbitraryEqualityMap();
     computedCache.set(computedName, cache);

package/core/computed/critical-request-chains.js

@@ -132,7 +132,7 @@ class CriticalRequestChains {
    */
   static async compute_(data, context) {
     const mainResource = await MainResource.request(data, context);
-    const graph = await PageDependencyGraph.request(data, context);
+    const graph = await PageDependencyGraph.request({...data, fromTrace: false}, context);
 
     return CriticalRequestChains.extractChainsFromGraph(mainResource, graph);
   }

package/core/computed/metrics/lantern-metric.js

@@ -20,7 +20,7 @@ async function getComputationDataParamsFromDevtoolsLog(data, context) {
     throw new Error(`Lantern metrics can only be computed on navigations`);
   }
 
-  const graph = await PageDependencyGraph.request(data, context);
+  const graph = await PageDependencyGraph.request({...data, fromTrace: false}, context);
   const processedNavigation = await ProcessedNavigation.request(data.trace, context);
   const simulator = data.simulator || (await LoadSimulator.request(data, context));
 

package/core/computed/metrics/metric.js

@@ -35,6 +35,7 @@ class Metric {
       settings: data.settings,
       URL: data.URL,
       SourceMaps: data.SourceMaps,
+      simulator: null,
     };
   }
 

package/core/computed/metrics/timing-summary.js

@@ -32,7 +32,9 @@ class TimingSummary {
      * @return {Promise<{metrics: LH.Artifacts.TimingSummary, debugInfo: Record<string,boolean>}>}
      */
   static async summarize(trace, devtoolsLog, gatherContext, settings, URL, SourceMaps, context) {
-    const metricComputationData = {trace, devtoolsLog, gatherContext, settings, URL, SourceMaps};
+    const metricComputationData =
+      {trace, devtoolsLog, gatherContext, settings, URL, SourceMaps, simulator: null};
+
     /**
      * @template TArtifacts
      * @template TReturn

package/core/computed/page-dependency-graph.d.ts

@@ -6,12 +6,12 @@ declare const PageDependencyGraphComputed: typeof PageDependencyGraph & {
         settings: LH.Audit.Context["settings"];
         URL: LH.Artifacts["URL"];
         SourceMaps: LH.Artifacts["SourceMaps"];
-        fromTrace?: boolean;
+        fromTrace: boolean;
     }, context: LH.Artifacts.ComputedContext) => ReturnType<typeof PageDependencyGraph.compute_>;
 };
 declare class PageDependencyGraph {
     /**
-     * @param {{trace: LH.Trace, devtoolsLog: LH.DevtoolsLog, settings: LH.Audit.Context['settings'], URL: LH.Artifacts['URL'], SourceMaps: LH.Artifacts['SourceMaps'], fromTrace?: boolean}} data
+     * @param {{trace: LH.Trace, devtoolsLog: LH.DevtoolsLog, settings: LH.Audit.Context['settings'], URL: LH.Artifacts['URL'], SourceMaps: LH.Artifacts['SourceMaps'], fromTrace: boolean}} data
      * @param {LH.Artifacts.ComputedContext} context
      * @return {Promise<LH.Gatherer.Simulation.GraphNode>}
      */
@@ -21,7 +21,7 @@ declare class PageDependencyGraph {
         settings: LH.Audit.Context["settings"];
         URL: LH.Artifacts["URL"];
         SourceMaps: LH.Artifacts["SourceMaps"];
-        fromTrace?: boolean;
+        fromTrace: boolean;
     }, context: LH.Artifacts.ComputedContext): Promise<LH.Gatherer.Simulation.GraphNode>;
 }
 //# sourceMappingURL=page-dependency-graph.d.ts.map

package/core/computed/page-dependency-graph.js

@@ -13,7 +13,7 @@ import {TraceEngineResult} from './trace-engine-result.js';
 
 class PageDependencyGraph {
   /**
-   * @param {{trace: LH.Trace, devtoolsLog: LH.DevtoolsLog, settings: LH.Audit.Context['settings'], URL: LH.Artifacts['URL'], SourceMaps: LH.Artifacts['SourceMaps'], fromTrace?: boolean}} data
+   * @param {{trace: LH.Trace, devtoolsLog: LH.DevtoolsLog, settings: LH.Audit.Context['settings'], URL: LH.Artifacts['URL'], SourceMaps: LH.Artifacts['SourceMaps'], fromTrace: boolean}} data
    * @param {LH.Artifacts.ComputedContext} context
    * @return {Promise<LH.Gatherer.Simulation.GraphNode>}
    */

package/core/computed/unused-javascript-summary.d.ts

@@ -7,7 +7,7 @@ export type WasteData = {
 export type ComputeInput = {
     scriptId: string;
     scriptCoverage: Omit<LH.Crdp.Profiler.ScriptCoverage, "url">;
-    bundle?: LH.Artifacts.Bundle | undefined;
+    bundle: LH.Artifacts.Bundle | null;
 };
 export type Summary = {
     scriptId: string;
@@ -32,7 +32,7 @@ declare const UnusedJavascriptSummaryComputed: typeof UnusedJavascriptSummary &
  * @typedef ComputeInput
  * @property {string} scriptId
  * @property {Omit<LH.Crdp.Profiler.ScriptCoverage, 'url'>} scriptCoverage
- * @property {LH.Artifacts.Bundle=} bundle
+ * @property {LH.Artifacts.Bundle|null} bundle
  */
 /**
  * @typedef Summary

package/core/computed/unused-javascript-summary.js

@@ -16,7 +16,7 @@ import {makeComputedArtifact} from './computed-artifact.js';
  * @typedef ComputeInput
  * @property {string} scriptId
  * @property {Omit<LH.Crdp.Profiler.ScriptCoverage, 'url'>} scriptCoverage
- * @property {LH.Artifacts.Bundle=} bundle
+ * @property {LH.Artifacts.Bundle|null} bundle
  */
 
 /**

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "lighthouse",
   "type": "module",
-  "version": "12.4.0-dev.20250319",
+  "version": "12.4.0-dev.20250321",
   "description": "Automated auditing, performance metrics, and best practices for the web.",
   "main": "./core/index.js",
   "bin": {

package/types/artifacts.d.ts

@@ -567,7 +567,7 @@ declare module Artifacts {
     trace: Trace;
     settings: Audit.Context['settings'];
     gatherContext: Artifacts['GatherContext'];
-    simulator?: Gatherer.Simulation.Simulator;
+    simulator: Gatherer.Simulation.Simulator | null;
     URL: Artifacts['URL'];
     SourceMaps: Artifacts['SourceMaps'];
   }