@newrelic/browser-agent 1.239.1 → 1.241.0

package/dist/cjs/features/metrics/aggregate/index.js

@@ -33,13 +33,16 @@ class Aggregate extends _aggregateBase.AggregateBase {
     this.singleChecks(); // checks that are run only one time, at script load
     this.eachSessionChecks(); // the start of every time user engages with page
 
-    // *cli, Mar 23 - Per NR-94597, this feature should only harvest ONCE at the (potential) EoL time of the page.
-    scheduler = new _harvestScheduler.HarvestScheduler('jserrors', {
-      onUnload: () => this.unload()
-    }, this);
-    scheduler.harvest.on('jserrors', () => ({
-      body: this.aggregator.take(['cm', 'sm'])
-    }));
+    this.ee.on("drain-".concat(this.featureName), () => {
+      // *cli, Mar 23 - Per NR-94597, this feature should only harvest ONCE at the (potential) EoL time of the page.
+      scheduler = new _harvestScheduler.HarvestScheduler('jserrors', {
+        onUnload: () => this.unload()
+      }, this);
+      scheduler.harvest.on('jserrors', () => ({
+        body: this.aggregator.take(['cm', 'sm'])
+      }));
+    }); // this is needed to ensure EoL is "on" and sent
+
     this.drain();
   }
   storeSupportabilityMetrics(name, value) {
@@ -93,6 +96,13 @@ class Aggregate extends _aggregateBase.AggregateBase {
     const rules = (0, _obfuscate.getRules)(this.agentIdentifier);
     if (rules.length > 0) this.storeSupportabilityMetrics('Generic/Obfuscate/Detected');
     if (rules.length > 0 && !(0, _obfuscate.validateRules)(rules)) this.storeSupportabilityMetrics('Generic/Obfuscate/Invalid');
+
+    // Check if proxy for either chunks or beacon is being used
+    const {
+      proxy
+    } = (0, _config.getConfiguration)(this.agentIdentifier);
+    if (proxy.assets) this.storeSupportabilityMetrics('Config/AssetsUrl/Changed');
+    if (proxy.beacon) this.storeSupportabilityMetrics('Config/BeaconUrl/Changed');
   }
   eachSessionChecks() {
     if (!_runtime.isBrowserScope) return;

package/dist/cjs/features/page_view_event/aggregate/index.js

@@ -4,13 +4,9 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.Aggregate = void 0;
-var _handle = require("../../../common/event-emitter/handle");
-var _features = require("../../../loaders/features/features");
 var _runtime = require("../../../common/constants/runtime");
-var _webVitals = require("web-vitals");
 var _navTiming = require("../../../common/timing/nav-timing");
 var _stringify = require("../../../common/util/stringify");
-var _paintMetrics = require("../../../common/metrics/paint-metrics");
 var _config = require("../../../common/config/config");
 var _harvest = require("../../../common/harvest/harvest");
 var CONSTANTS = _interopRequireWildcard(require("../constants"));
@@ -18,44 +14,37 @@ var _initializedFeatures = require("./initialized-features");
 var _featureFlags = require("../../../common/util/feature-flags");
 var _console = require("../../../common/util/console");
 var _aggregateBase = require("../../utils/aggregate-base");
+var _firstContentfulPaint = require("../../../common/vitals/first-contentful-paint");
+var _firstPaint = require("../../../common/vitals/first-paint");
+var _timeToFirstByte = require("../../../common/vitals/time-to-first-byte");
 function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "function") return null; var cacheBabelInterop = new WeakMap(); var cacheNodeInterop = new WeakMap(); return (_getRequireWildcardCache = function (nodeInterop) { return nodeInterop ? cacheNodeInterop : cacheBabelInterop; })(nodeInterop); }
 function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || typeof obj !== "object" && typeof obj !== "function") { return { default: obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj.default = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
 class Aggregate extends _aggregateBase.AggregateBase {
   static featureName = CONSTANTS.FEATURE_NAME;
   constructor(agentIdentifier, aggregator) {
     super(agentIdentifier, aggregator, CONSTANTS.FEATURE_NAME);
-    if (typeof PerformanceNavigationTiming !== 'undefined' && !_runtime.isiOS) {
-      this.alreadySent = false; // we don't support timings on BFCache restores
-      const agentRuntime = (0, _config.getRuntime)(agentIdentifier); // we'll store timing values on the runtime obj to be read by the aggregate module
+    this.timeToFirstByte = 0;
+    this.firstByteToWindowLoad = 0; // our "frontend" duration
+    this.firstByteToDomContent = 0; // our "dom processing" duration
 
-      /* Time To First Byte
-        This listener must record these values *before* PVE's aggregate sends RUM. */
-      (0, _webVitals.onTTFB)(_ref => {
+    if (_runtime.isBrowserScope) {
+      _timeToFirstByte.timeToFirstByte.subscribe(_ref => {
         let {
           value,
           entries
         } = _ref;
-        if (this.alreadySent) return;
-        this.alreadySent = true;
-        agentRuntime[CONSTANTS.TTFB] = Math.round(value); // this is our "backend" duration; web-vitals will ensure it's lower bounded at 0
-
-        // Similar to what vitals does for ttfb, we have to factor in activation-start when calculating relative timings:
         const navEntry = entries[0];
-        const respOrActivStart = Math.max(navEntry.responseStart, navEntry.activationStart || 0);
-        agentRuntime[CONSTANTS.FBTWL] = Math.max(Math.round(navEntry.loadEventEnd - respOrActivStart), 0); // our "frontend" duration
-        (0, _handle.handle)('timing', ['load', Math.round(navEntry.loadEventEnd)], undefined, _features.FEATURE_NAMES.pageViewTiming, this.ee);
-        agentRuntime[CONSTANTS.FBTDC] = Math.max(Math.round(navEntry.domContentLoadedEventEnd - respOrActivStart), 0); // our "dom processing" duration
+        this.timeToFirstByte = Math.max(value, this.timeToFirstByte);
+        this.firstByteToWindowLoad = Math.max(Math.round(navEntry.loadEventEnd - this.timeToFirstByte), this.firstByteToWindowLoad); // our "frontend" duration
+        this.firstByteToDomContent = Math.max(Math.round(navEntry.domContentLoadedEventEnd - this.timeToFirstByte), this.firstByteToDomContent); // our "dom processing" duration
 
         this.sendRum();
       });
     } else {
-      this.sendRum(); // timings either already in runtime from instrument or is meant to get 0'd.
+      // worker agent build does not get TTFB values, use default 0 values
+      this.sendRum();
     }
   }
-
-  getScheme() {
-    return (0, _config.getConfigurationValue)(this.agentIdentifier, 'ssl') === false ? 'http' : 'https';
-  }
   sendRum() {
     const info = (0, _config.getInfo)(this.agentIdentifier);
     const agentRuntime = (0, _config.getRuntime)(this.agentIdentifier);
@@ -72,13 +61,13 @@ class Aggregate extends _aggregateBase.AggregateBase {
     // Following PR #428, which demands that all agents send RUM call, these need to be sent even outside of the main window context where PerformanceTiming
     // or PerformanceNavigationTiming do not exists. Hence, they'll be filled in by 0s instead in, for example, worker threads that still init the PVE module.
     this.aggregator.store('measures', 'be', {
-      value: _runtime.isBrowserScope ? agentRuntime[CONSTANTS.TTFB] : 0
+      value: this.timeToFirstByte
     });
     this.aggregator.store('measures', 'fe', {
-      value: _runtime.isBrowserScope ? agentRuntime[CONSTANTS.FBTWL] : 0
+      value: this.firstByteToWindowLoad
     });
     this.aggregator.store('measures', 'dc', {
-      value: _runtime.isBrowserScope ? agentRuntime[CONSTANTS.FBTDC] : 0
+      value: this.firstByteToDomContent
     });
     const queryParameters = {
       tt: info.ttGuid,
@@ -119,19 +108,8 @@ class Aggregate extends _aggregateBase.AggregateBase {
         queryParameters.perf = (0, _stringify.stringify)(perf);
       }
     }
-    try {
-      // PVTiming sends these too, albeit using web-vitals and slightly different; it's unknown why they're duplicated, but PVT should be the truth
-      var entries = _runtime.globalScope.performance.getEntriesByType('paint');
-      entries.forEach(function (entry) {
-        if (!entry.startTime || entry.startTime <= 0) return;
-        if (entry.name === 'first-paint') {
-          queryParameters.fp = String(Math.floor(entry.startTime));
-        } else if (entry.name === 'first-contentful-paint') {
-          queryParameters.fcp = String(Math.floor(entry.startTime));
-        }
-        _paintMetrics.paintMetrics[entry.name] = Math.floor(entry.startTime); // this is consumed by Spa module
-      });
-    } catch (e) {}
+    queryParameters.fp = _firstPaint.firstPaint.current.value;
+    queryParameters.fcp = _firstContentfulPaint.firstContentfulPaint.current.value;
     harvester.send({
       endpoint: 'rum',
       payload: {

package/dist/cjs/features/page_view_event/constants.js

@@ -3,13 +3,7 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.TTFB = exports.FEATURE_NAME = exports.FBTWL = exports.FBTDC = void 0;
+exports.FEATURE_NAME = void 0;
 var _features = require("../../loaders/features/features");
 const FEATURE_NAME = _features.FEATURE_NAMES.pageViewEvent;
-exports.FEATURE_NAME = FEATURE_NAME;
-const TTFB = 'firstbyte';
-exports.TTFB = TTFB;
-const FBTDC = 'domcontent';
-exports.FBTDC = FBTDC;
-const FBTWL = 'windowload';
-exports.FBTWL = FBTWL;
+exports.FEATURE_NAME = FEATURE_NAME;

package/dist/cjs/features/page_view_event/instrument/index.js

@@ -4,14 +4,8 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.Instrument = void 0;
-var _handle = require("../../../common/event-emitter/handle");
-var _runtime = require("../../../common/constants/runtime");
 var _instrumentBase = require("../../utils/instrument-base");
 var CONSTANTS = _interopRequireWildcard(require("../constants"));
-var _features = require("../../../loaders/features/features");
-var _config = require("../../../common/config/config");
-var _load = require("../../../common/window/load");
-var _now = require("../../../common/timing/now");
 function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "function") return null; var cacheBabelInterop = new WeakMap(); var cacheNodeInterop = new WeakMap(); return (_getRequireWildcardCache = function (nodeInterop) { return nodeInterop ? cacheNodeInterop : cacheBabelInterop; })(nodeInterop); }
 function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || typeof obj !== "object" && typeof obj !== "function") { return { default: obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj.default = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
 class Instrument extends _instrumentBase.InstrumentBase {
@@ -19,22 +13,6 @@ class Instrument extends _instrumentBase.InstrumentBase {
   constructor(agentIdentifier, aggregator) {
     let auto = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : true;
     super(agentIdentifier, aggregator, CONSTANTS.FEATURE_NAME, auto);
-    if ((typeof PerformanceNavigationTiming === 'undefined' || _runtime.isiOS) && typeof PerformanceTiming !== 'undefined') {
-      // For majority browser versions in which PNT exists, we can get load timings later from the nav entry (in the aggregate portion). At minimum, PT should exist for main window.
-      // *cli Mar'23 - iOS 15.2 & 15.4 testing in Sauce still fails with onTTFB. Hence, all iOS will fallback to this for now. Unknown if this is similar in nature to iOSBelow16 bug.
-      const agentRuntime = (0, _config.getRuntime)(agentIdentifier);
-      agentRuntime[CONSTANTS.TTFB] = Math.max(Date.now() - agentRuntime.offset, 0);
-      (0, _load.onDOMContentLoaded)(() => {
-        agentRuntime[CONSTANTS.FBTDC] = Math.max((0, _now.now)() - agentRuntime[CONSTANTS.TTFB], 0);
-      });
-      (0, _load.onWindowLoad)(() => {
-        const timeNow = (0, _now.now)();
-        agentRuntime[CONSTANTS.FBTWL] = Math.max(timeNow - agentRuntime[CONSTANTS.TTFB], 0);
-        (0, _handle.handle)('timing', ['load', timeNow], undefined, _features.FEATURE_NAMES.pageViewTiming, this.ee);
-      });
-    }
-    // Else, inference: inside worker or some other env where these events are irrelevant. They'll get filled in with 0s in RUM call.
-
     this.importAggregator();
   }
 }

package/dist/cjs/features/page_view_timing/aggregate/index.js

@@ -4,20 +4,23 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.Aggregate = void 0;
-var _webVitals = require("web-vitals");
-var _firstPaint = require("../first-paint");
-var _longTasks = require("../long-tasks");
-var _runtime = require("../../../common/constants/runtime");
 var _belSerializer = require("../../../common/serialize/bel-serializer");
 var _mapOwn = require("../../../common/util/map-own");
 var _harvestScheduler = require("../../../common/harvest/harvest-scheduler");
 var _registerHandler = require("../../../common/event-emitter/register-handler");
-var _cleanUrl = require("../../../common/url/clean-url");
 var _handle = require("../../../common/event-emitter/handle");
 var _config = require("../../../common/config/config");
 var _constants = require("../constants");
 var _features = require("../../../loaders/features/features");
 var _aggregateBase = require("../../utils/aggregate-base");
+var _cumulativeLayoutShift = require("../../../common/vitals/cumulative-layout-shift");
+var _firstContentfulPaint = require("../../../common/vitals/first-contentful-paint");
+var _firstInputDelay = require("../../../common/vitals/first-input-delay");
+var _firstPaint = require("../../../common/vitals/first-paint");
+var _interactionToNextPaint = require("../../../common/vitals/interaction-to-next-paint");
+var _largestContentfulPaint = require("../../../common/vitals/largest-contentful-paint");
+var _timeToFirstByte = require("../../../common/vitals/time-to-first-byte");
+var _longTask = require("../../../common/vitals/long-task");
 /*
  * Copyright 2020 New Relic Corporation. All rights reserved.
  * SPDX-License-Identifier: Apache-2.0
@@ -25,6 +28,14 @@ var _aggregateBase = require("../../utils/aggregate-base");
 
 class Aggregate extends _aggregateBase.AggregateBase {
   static featureName = _constants.FEATURE_NAME;
+  #handleVitalMetric = _ref => {
+    let {
+      name,
+      value,
+      attrs
+    } = _ref;
+    this.addTiming(name, value, attrs);
+  };
   constructor(agentIdentifier, aggregator) {
     var _this;
     super(agentIdentifier, aggregator, _constants.FEATURE_NAME);
@@ -32,163 +43,41 @@ class Aggregate extends _aggregateBase.AggregateBase {
     this.timings = [];
     this.timingsSent = [];
     this.curSessEndRecorded = false;
-    this.cls = null; // this should be null unless set to a numeric value by web-vitals so that we differentiate if CLS is supported
-
-    /* ! This is the section that used to be in the loader portion: ! */
-    /* ------------------------------------------------------------ */
-    const pageStartedHidden = (0, _config.getRuntime)(agentIdentifier).initHidden; // our attempt at recapturing initial vis state since this code runs post-load time
-    this.alreadySent = new Set(); // since we don't support timings on BFCache restores, this tracks and helps cap metrics that web-vitals report more than once
-
-    /* PerformancePaintTiming API - BFC is not yet supported. */
-    (0, _firstPaint.onFirstPaint)(_ref => {
-      let {
-        name,
-        value
-      } = _ref;
-      if (pageStartedHidden) return;
-      this.addTiming(name.toLowerCase(), Math.floor(value));
-    });
-
-    /* First Contentful Paint - As of WV v3, it still imperfectly tries to detect document vis state asap and isn't supposed to report if page starts hidden. */
-    if (_runtime.iOSBelow16) {
-      try {
-        if (!pageStartedHidden) {
-          // see ios-version.js for detail on this following bug case; tldr: buffered flag doesn't work but getEntriesByType does
-          const paintEntries = performance.getEntriesByType('paint');
-          paintEntries.forEach(entry => {
-            if (entry.name === 'first-contentful-paint') {
-              this.addTiming('fcp', Math.floor(entry.startTime));
-            }
-          });
-        }
-      } catch (e) {}
-    } else {
-      (0, _webVitals.onFCP)(_ref2 => {
-        let {
-          name,
-          value
-        } = _ref2;
-        if (pageStartedHidden || this.alreadySent.has(name)) return;
-        this.alreadySent.add(name);
-        this.addTiming(name.toLowerCase(), value);
-      });
-    }
-
-    /* First Input Delay (+"First Interaction") - As of WV v3, it still imperfectly tries to detect document vis state asap and isn't supposed to report if page starts hidden. */
-    (0, _webVitals.onFID)(_ref3 => {
-      let {
-        name,
-        value,
-        entries
-      } = _ref3;
-      if (pageStartedHidden || this.alreadySent.has(name) || entries.length === 0) return;
-      this.alreadySent.add(name);
-
-      // CWV will only report one (THE) first-input entry to us; fid isn't reported if there are no user interactions occurs before the *first* page hiding.
-      const fiEntry = entries[0];
-      const attributes = {
-        type: fiEntry.name,
-        fid: Math.round(value)
-      };
-      this.addConnectionAttributes(attributes);
-      this.addTiming('fi', Math.round(fiEntry.startTime), attributes);
-    });
-
-    /* Largest Contentful Paint - As of WV v3, it still imperfectly tries to detect document vis state asap and isn't supposed to report if page starts hidden. */
-    (0, _webVitals.onLCP)(_ref4 => {
+    _firstPaint.firstPaint.subscribe(this.#handleVitalMetric);
+    _firstContentfulPaint.firstContentfulPaint.subscribe(this.#handleVitalMetric);
+    _firstInputDelay.firstInputDelay.subscribe(this.#handleVitalMetric);
+    _largestContentfulPaint.largestContentfulPaint.subscribe(this.#handleVitalMetric);
+    _interactionToNextPaint.interactionToNextPaint.subscribe(this.#handleVitalMetric);
+    _timeToFirstByte.timeToFirstByte.subscribe(_ref2 => {
       let {
-        name,
-        value,
         entries
-      } = _ref4;
-      if (pageStartedHidden || this.alreadySent.has(name)) return;
-      this.alreadySent.add(name);
-      const attributes = {};
-      if (entries.length > 0) {
-        // CWV will only ever report one (THE) lcp entry to us; lcp is also only reported *once* on earlier(user interaction, page hidden).
-        const lcpEntry = entries[entries.length - 1]; // this looks weird if we only expect one, but this is how cwv-attribution gets it so to be sure...
-        attributes.size = lcpEntry.size;
-        attributes.eid = lcpEntry.id;
-        if (lcpEntry.url) {
-          attributes.elUrl = (0, _cleanUrl.cleanURL)(lcpEntry.url);
-        }
-        if (lcpEntry.element?.tagName) {
-          attributes.elTag = lcpEntry.element.tagName;
-        }
-      }
-      this.addConnectionAttributes(attributes);
-      this.addTiming(name.toLowerCase(), value, attributes);
-    });
-
-    /* Cumulative Layout Shift - We don't have to limit this callback since cls is stored as a state and only sent as attribute on other timings.
-      reportAllChanges ensures our tracked cls has the most recent rolling value to attach to 'unload' and 'pagehide'. */
-    (0, _webVitals.onCLS)(_ref5 => {
-      let {
-        value
-      } = _ref5;
-      this.cls = value;
-    }, {
-      reportAllChanges: true
-    });
-
-    /* Interaction-to-Next-Paint */
-    (0, _webVitals.onINP)(_ref6 => {
-      let {
-        name,
-        value,
-        id
-      } = _ref6;
-      return this.addTiming(name.toLowerCase(), value, {
-        metricId: id
-      });
+      } = _ref2;
+      this.addTiming('load', Math.round(entries[0].loadEventEnd));
     });
-
-    /* PerformanceLongTaskTiming API */
-    if ((0, _config.getConfigurationValue)(this.agentIdentifier, 'page_view_timing.long_task') === true) {
-      (0, _longTasks.onLongTask)(_ref7 => {
-        let {
-          name,
-          value,
-          info
-        } = _ref7;
-        return this.addTiming(name.toLowerCase(), value, info);
-      }); // lt context is passed as 'info'=attrs in the timing node
-    }
-    /* ------------------------------------End of ex-loader section */
+    if ((0, _config.getConfigurationValue)(this.agentIdentifier, 'page_view_timing.long_task') === true) _longTask.longTask.subscribe(this.#handleVitalMetric);
 
     /* It's important that CWV api, like "onLCP", is called before this scheduler is initialized. The reason is because they listen to the same
       on vis change or pagehide events, and we'd want ex. onLCP to record the timing (win the race) before we try to send "final harvest". */
-    this.scheduler = new _harvestScheduler.HarvestScheduler('events', {
-      onFinished: function () {
-        return _this.onHarvestFinished(...arguments);
-      },
-      getPayload: function () {
-        return _this.prepareHarvest(...arguments);
-      }
-    }, this);
-    (0, _registerHandler.registerHandler)('timing', (name, value, attrs) => this.addTiming(name, value, attrs), this.featureName, this.ee); // notice CLS is added to all timings via 4th param
+
     (0, _registerHandler.registerHandler)('docHidden', msTimestamp => this.endCurrentSession(msTimestamp), this.featureName, this.ee);
     (0, _registerHandler.registerHandler)('winPagehide', msTimestamp => this.recordPageUnload(msTimestamp), this.featureName, this.ee);
     const initialHarvestSeconds = (0, _config.getConfigurationValue)(this.agentIdentifier, 'page_view_timing.initialHarvestSeconds') || 10;
     const harvestTimeSeconds = (0, _config.getConfigurationValue)(this.agentIdentifier, 'page_view_timing.harvestTimeSeconds') || 30;
     // send initial data sooner, then start regular
     this.ee.on("drain-".concat(this.featureName), () => {
+      this.scheduler = new _harvestScheduler.HarvestScheduler('events', {
+        onFinished: function () {
+          return _this.onHarvestFinished(...arguments);
+        },
+        getPayload: function () {
+          return _this.prepareHarvest(...arguments);
+        }
+      }, this);
       this.scheduler.startTimer(harvestTimeSeconds, initialHarvestSeconds);
     });
     this.drain();
   }
 
-  // takes an attributes object and appends connection attributes if available
-  addConnectionAttributes(attributes) {
-    var connection = navigator.connection || navigator.mozConnection || navigator.webkitConnection; // to date, both window & worker shares the same support for connection
-    if (!connection) return;
-    if (connection.type) attributes['net-type'] = connection.type;
-    if (connection.effectiveType) attributes['net-etype'] = connection.effectiveType;
-    if (connection.rtt) attributes['net-rtt'] = connection.rtt;
-    if (connection.downlink) attributes['net-dlink'] = connection.downlink;
-    return attributes;
-  }
-
   /**
    * Add the time of _document visibilitychange to hidden_ to the next PVT harvest == NRDB pageHide attr.
    * @param {number} timestamp
@@ -225,8 +114,8 @@ class Aggregate extends _aggregateBase.AggregateBase {
     Mitigation: We've set initial CLS to null so that it's omitted from timings like 'pageHide' in that edge case. It should only be included if onCLS callback was executed at least once.
     Future: onCLS value changes should be reported directly & CLS separated into its own timing node so it's not beholden to 'pageHide' firing. It'd also be possible to report the real final CLS.
     */
-    if (this.cls !== null) {
-      attrs.cls = this.cls;
+    if (_cumulativeLayoutShift.cumulativeLayoutShift.current.value >= 0) {
+      attrs.cls = _cumulativeLayoutShift.cumulativeLayoutShift.current.value;
     }
     this.timings.push({
       name,

package/dist/cjs/features/page_view_timing/instrument/index.js

@@ -5,7 +5,6 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.Instrument = void 0;
 var _handle = require("../../../common/event-emitter/handle");
-var _config = require("../../../common/config/config");
 var _pageVisibility = require("../../../common/window/page-visibility");
 var _eventListenerOpts = require("../../../common/event-listener/event-listener-opts");
 var _now = require("../../../common/timing/now");
@@ -24,9 +23,7 @@ class Instrument extends _instrumentBase.InstrumentBase {
     super(agentIdentifier, aggregator, _constants.FEATURE_NAME, auto);
     if (!_runtime.isBrowserScope) return; // CWV is irrelevant outside web context
 
-    // Document visibility state becomes hidden; this should run as soon as possible in page life.
     // While we try to replicate web-vital's visibilitywatcher logic in an effort to defer that library to post-pageload, this isn't perfect and doesn't consider prerendering.
-    (0, _config.getRuntime)(agentIdentifier).initHidden = Boolean(document.visibilityState === 'hidden');
     (0, _pageVisibility.subscribeToVisibilityChange)(() => (0, _handle.handle)('docHidden', [(0, _now.now)()], undefined, _constants.FEATURE_NAME, this.ee), true);
 
     // Window fires its pagehide event (typically on navigation--this occurrence is a *subset* of vis change); don't defer this unless it's guarantee it cannot happen before load(?)