@mui/internal-benchmark 0.0.3-canary.12 → 0.0.3-canary.14

package/index.mjs

@@ -206,17 +206,15 @@ export function benchmark(name, renderFn, interactionOrOptions, maybeOptions) {
 
     // Paint timings are recorded as one harness-owned `bench:paint` metric: the default sentinel
     // is the base series (`bench:paint`) and named `elementtiming` markers are sub-series
-    // (`bench:paint#grid-header`, …), all sharing a single definition. A default alarm keeps a
-    // >20% paint regression flagged, matching the previous behavior.
+    // (`bench:paint#grid-header`, …), all sharing a single definition. Paint is informational (no
+    // alarm): it dominates each test's total duration, so a per-test paint alarm just duplicates the
+    // Duration regression signal and floods the report on any broadly-regressed run.
     const paint = new ScalarMetric({
       name: 'bench:paint',
       format: {
         style: 'unit',
         unit: 'millisecond',
         maximumFractionDigits: 2
-      },
-      alarm: {
-        error: 0.2
       }
     });
     if (typeof window.gc !== 'function') {
@@ -286,13 +284,13 @@ export function benchmark(name, renderFn, interactionOrOptions, maybeOptions) {
       if (!isWarmup) {
         for (const entry of timing.elementEntries) {
           // Skip paints that happened while recording was paused. Attribute by the paint's
-          // `renderTime`, not by when the observer callback fired (which can lag the paint).
-          if (!recording.activeAt(entry.renderTime)) {
+          // `paintTime`, not by when the observer callback fired (which can lag the paint).
+          if (!recording.activeAt(entry.paintTime)) {
             continue;
           }
           // The default sentinel is the base series; named markers become sub-series.
           const id = entry.identifier === 'default' ? undefined : entry.identifier;
-          paint.record(entry.renderTime - iterationStart, id !== undefined ? {
+          paint.record(entry.paintTime - iterationStart, id !== undefined ? {
             id
           } : undefined);
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mui/internal-benchmark",
-  "version": "0.0.3-canary.12",
+  "version": "0.0.3-canary.14",
   "author": "MUI Team",
   "description": "Benchmark utilities for MUI projects. Internal package.",
   "repository": {
@@ -68,5 +68,5 @@
       }
     }
   },
-  "gitSha": "e8df28d5d6f73f954ef809e40e0ec985ed7e01b0"
+  "gitSha": "b3537f3ae9a46001df4b894cde740a0d44595786"
 }

package/reactRecording.d.mts

@@ -5,7 +5,7 @@ export interface ReactRecordingControls {
   readonly hadEmptyActiveWindow: boolean;
   /**
    * Whether recording was active at `time` (a `performance.now()` timestamp). Paint entries are
-   * observed asynchronously, so they are attributed by their `renderTime` rather than by the
+   * observed asynchronously, so they are attributed by their `paintTime` rather than by the
    * recording state at the moment the observer callback happens to fire.
    */
   activeAt(time: number): boolean;