@newrelic/browser-agent 1.235.0 → 1.237.0

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

@@ -15,7 +15,7 @@ var _handle = require("../../../common/event-emitter/handle");
 var _mapOwn = require("../../../common/util/map-own");
 var _config = require("../../../common/config/config");
 var _now = require("../../../common/timing/now");
-var _globalScope = require("../../../common/util/global-scope");
+var _runtime = require("../../../common/constants/runtime");
 var _constants = require("../constants");
 var _drain = require("../../../common/drain/drain");
 var _features = require("../../../loaders/features/features");
@@ -100,7 +100,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
       (0, _mapOwn.mapOwn)(this.currentBody, (key, value) => {
         for (var i = 0; i < value.length; i++) {
           var bucket = value[i];
-          var name = this.getBucketName(bucket.params, bucket.custom);
+          var name = this.getBucketName(key, bucket.params, bucket.custom);
           this.aggregator.merge(key, name, bucket.metrics, bucket.params, bucket.custom);
         }
       });
@@ -110,7 +110,10 @@ class Aggregate extends _aggregateBase.AggregateBase {
   nameHash(params) {
     return (0, _stringHashCode.stringHashCode)("".concat(params.exceptionClass, "_").concat(params.message, "_").concat(params.stack_trace || params.browser_stack_hash));
   }
-  getBucketName(params, customParams) {
+  getBucketName(objType, params, customParams) {
+    if (objType === 'xhr') {
+      return (0, _stringHashCode.stringHashCode)((0, _stringify.stringify)(params)) + ':' + (0, _stringHashCode.stringHashCode)((0, _stringify.stringify)(customParams));
+    }
     return this.nameHash(params) + ':' + (0, _stringHashCode.stringHashCode)((0, _stringify.stringify)(customParams));
   }
 
@@ -153,7 +156,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
     const params = {
       stackHash: (0, _stringHashCode.stringHashCode)(canonicalStackString),
       exceptionClass: stackInfo.name,
-      request_uri: _globalScope.globalScope?.location.pathname
+      request_uri: _runtime.globalScope?.location.pathname
     };
     if (stackInfo.message) params.message = '' + stackInfo.message;
     // Notice if filterOutput isn't false|undefined OR our specified object, this func would've returned already (so it's unnecessary to req-check group).

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

@@ -3,9 +3,7 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.NR_ERR_PROP = exports.FEATURE_NAME = void 0;
+exports.FEATURE_NAME = void 0;
 var _features = require("../../loaders/features/features");
 const FEATURE_NAME = _features.FEATURE_NAMES.jserrors;
-exports.FEATURE_NAME = FEATURE_NAME;
-const NR_ERR_PROP = 'nr@seenError';
-exports.NR_ERR_PROP = NR_ERR_PROP;
+exports.FEATURE_NAME = FEATURE_NAME;

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

@@ -6,16 +6,13 @@ Object.defineProperty(exports, "__esModule", {
 exports.Instrument = void 0;
 var _handle = require("../../../common/event-emitter/handle");
 var _now = require("../../../common/timing/now");
-var _getOrSet = require("../../../common/util/get-or-set");
-var _wrap = require("../../../common/wrap");
-require("./debug");
 var _instrumentBase = require("../../utils/instrument-base");
 var _constants = require("../constants");
 var _features = require("../../../loaders/features/features");
-var _globalScope = require("../../../common/util/global-scope");
+var _runtime = require("../../../common/constants/runtime");
 var _eventListenerOpts = require("../../../common/event-listener/event-listener-opts");
-var _config = require("../../../common/config/config");
 var _stringify = require("../../../common/util/stringify");
+var _uncaughtError = require("./uncaught-error");
 /*
  * Copyright 2020 New Relic Corporation. All rights reserved.
  * SPDX-License-Identifier: Apache-2.0
@@ -23,51 +20,44 @@ var _stringify = require("../../../common/util/stringify");
 
 class Instrument extends _instrumentBase.InstrumentBase {
   static featureName = _constants.FEATURE_NAME;
+  #seenErrors = new Set();
   constructor(agentIdentifier, aggregator) {
     let auto = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : true;
     super(agentIdentifier, aggregator, _constants.FEATURE_NAME, auto);
-    // skipNext counter to keep track of uncaught
-    // errors that will be the same as caught errors.
-    this.skipNext = 0;
     try {
       // this try-catch can be removed when IE11 is completely unsupported & gone
       this.removeOnAbort = new AbortController();
     } catch (e) {}
-    const thisInstrument = this;
-    thisInstrument.ee.on('fn-start', function (args, obj, methodName) {
-      if (thisInstrument.abortHandler) thisInstrument.skipNext += 1;
-    });
-    thisInstrument.ee.on('fn-err', function (args, obj, err) {
-      if (thisInstrument.abortHandler && !err[_constants.NR_ERR_PROP]) {
-        (0, _getOrSet.getOrSet)(err, _constants.NR_ERR_PROP, function getVal() {
-          return true;
-        });
-        this.thrown = true;
-        (0, _handle.handle)('err', [err, (0, _now.now)()], undefined, _features.FEATURE_NAMES.jserrors, thisInstrument.ee);
-      }
-    });
-    thisInstrument.ee.on('fn-end', function () {
-      if (!thisInstrument.abortHandler) return;
-      if (!this.thrown && thisInstrument.skipNext > 0) thisInstrument.skipNext -= 1;
+
+    // Capture function errors early in case the spa feature is loaded
+    this.ee.on('fn-err', (args, obj, error) => {
+      if (!this.abortHandler || this.#seenErrors.has(error)) return;
+      this.#seenErrors.add(error);
+      (0, _handle.handle)('err', [this.#castError(error), (0, _now.now)()], undefined, _features.FEATURE_NAMES.jserrors, this.ee);
     });
-    thisInstrument.ee.on('internal-error', function (e) {
-      (0, _handle.handle)('ierr', [e, (0, _now.now)(), true], undefined, _features.FEATURE_NAMES.jserrors, thisInstrument.ee);
+    this.ee.on('internal-error', error => {
+      if (!this.abortHandler) return;
+      (0, _handle.handle)('ierr', [this.#castError(error), (0, _now.now)(), true], undefined, _features.FEATURE_NAMES.jserrors, this.ee);
     });
-
-    // Replace global error handler with our own.
-    this.origOnerror = _globalScope.globalScope.onerror;
-    _globalScope.globalScope.onerror = this.onerrorHandler.bind(this);
-    _globalScope.globalScope.addEventListener('unhandledrejection', e => {
-      /** rejections can contain data of any type -- this is an effort to keep the message human readable */
-      const err = castReasonToError(e.reason);
-      (0, _handle.handle)('err', [err, (0, _now.now)(), false, {
+    _runtime.globalScope.addEventListener('unhandledrejection', promiseRejectionEvent => {
+      if (!this.abortHandler) return;
+      (0, _handle.handle)('err', [this.#castPromiseRejectionEvent(promiseRejectionEvent), (0, _now.now)(), false, {
         unhandledPromiseRejection: 1
       }], undefined, _features.FEATURE_NAMES.jserrors, this.ee);
     }, (0, _eventListenerOpts.eventListenerOpts)(false, this.removeOnAbort?.signal));
-    (0, _wrap.wrapRaf)(this.ee);
-    (0, _wrap.wrapTimer)(this.ee);
-    (0, _wrap.wrapEvents)(this.ee);
-    if ((0, _config.getRuntime)(agentIdentifier).xhrWrappable) (0, _wrap.wrapXhr)(this.ee);
+    _runtime.globalScope.addEventListener('error', errorEvent => {
+      if (!this.abortHandler) return;
+
+      /**
+       * If the spa feature is loaded, errors may already have been captured in the `fn-err` listener above.
+       * This ensures those errors are not captured twice.
+       */
+      if (this.#seenErrors.has(errorEvent.error)) {
+        this.#seenErrors.delete(errorEvent.error);
+        return;
+      }
+      (0, _handle.handle)('err', [this.#castErrorEvent(errorEvent), (0, _now.now)()], undefined, _features.FEATURE_NAMES.jserrors, this.ee);
+    }, (0, _eventListenerOpts.eventListenerOpts)(false, this.removeOnAbort?.signal));
     this.abortHandler = this.#abort; // we also use this as a flag to denote that the feature is active or on and handling errors
     this.importAggregator();
   }
@@ -75,66 +65,67 @@ class Instrument extends _instrumentBase.InstrumentBase {
   /** Restoration and resource release tasks to be done if JS error loader is being aborted. Unwind changes to globals. */
   #abort() {
     this.removeOnAbort?.abort();
+    this.#seenErrors.clear();
     this.abortHandler = undefined; // weakly allow this abort op to run only once
   }
 
   /**
-   * FF and Android browsers do not provide error info to the 'error' event callback,
-   * so we must use window.onerror
-   * @param {string} message
-   * @param {string} filename
-   * @param {number} lineno
-   * @param {number} column
-   * @param {Error | *} errorObj
-   * @returns
+   * Any value can be used with the `throw` keyword. This function ensures that the value is
+   * either a proper Error instance or attempts to convert it to an UncaughtError instance.
+   * @param {any} error The value thrown
+   * @returns {Error|UncaughtError} The converted error instance
    */
-  onerrorHandler(message, filename, lineno, column, errorObj) {
-    if (typeof this.origOnerror === 'function') this.origOnerror(...arguments);
-    try {
-      if (this.skipNext) this.skipNext -= 1;else (0, _handle.handle)('err', [errorObj || new UncaughtException(message, filename, lineno), (0, _now.now)()], undefined, _features.FEATURE_NAMES.jserrors, this.ee);
-    } catch (e) {
+  #castError(error) {
+    if (error instanceof Error) {
+      return error;
+    }
+
+    /**
+     * The thrown value may contain a message property. If it does, try to treat the thrown
+     * value as an Error-like object.
+     */
+    if (typeof error?.message !== 'undefined') {
+      return new _uncaughtError.UncaughtError(error.message, error.filename || error.sourceURL, error.lineno || error.line, error.colno || error.col);
+    }
+    return new _uncaughtError.UncaughtError(typeof error === 'string' ? error : (0, _stringify.stringify)(error));
+  }
+
+  /**
+   * Attempts to convert a PromiseRejectionEvent object to an Error object
+   * @param {PromiseRejectionEvent} unhandledRejectionEvent The unhandled promise rejection event
+   * @returns {Error} An Error object with the message as the casted reason
+   */
+  #castPromiseRejectionEvent(promiseRejectionEvent) {
+    let prefix = 'Unhandled Promise Rejection: ';
+    if (promiseRejectionEvent?.reason instanceof Error) {
       try {
-        (0, _handle.handle)('ierr', [e, (0, _now.now)(), true], undefined, _features.FEATURE_NAMES.jserrors, this.ee);
-      } catch (err) {
-        // do nothing
+        promiseRejectionEvent.reason.message = prefix + promiseRejectionEvent.reason.message;
+        return promiseRejectionEvent.reason;
+      } catch (e) {
+        return promiseRejectionEvent.reason;
       }
     }
-    return false; // maintain default behavior of the error event of Window
+    if (typeof promiseRejectionEvent.reason === 'undefined') return new _uncaughtError.UncaughtError(prefix);
+    const error = this.#castError(promiseRejectionEvent.reason);
+    error.message = prefix + error.message;
+    return error;
   }
-}
 
-/**
- *
- * @param {string} message
- * @param {string} filename
- * @param {number} lineno
- */
-exports.Instrument = Instrument;
-function UncaughtException(message, filename, lineno) {
-  this.message = message || 'Uncaught error with no additional information';
-  this.sourceURL = filename;
-  this.line = lineno;
-}
-
-/**
- * Attempts to cast an unhandledPromiseRejection reason (reject(...)) to an Error object
- * @param {*} reason - The reason property from an unhandled promise rejection
- * @returns {Error} - An Error object with the message as the casted reason
- */
-function castReasonToError(reason) {
-  let prefix = 'Unhandled Promise Rejection: ';
-  if (reason instanceof Error) {
-    try {
-      reason.message = prefix + reason.message;
-      return reason;
-    } catch (e) {
-      return reason;
+  /**
+   * Attempts to convert an ErrorEvent object to an Error object
+   * @param {ErrorEvent} errorEvent The error event
+   * @returns {Error|UncaughtError} The error event converted to an Error object
+   */
+  #castErrorEvent(errorEvent) {
+    if (errorEvent.error instanceof Error) {
+      return errorEvent.error;
     }
+
+    /**
+     * Older browsers do not contain the `error` property on the ErrorEvent instance.
+     * https://caniuse.com/mdn-api_errorevent_error
+     */
+    return new _uncaughtError.UncaughtError(errorEvent.message, errorEvent.filename, errorEvent.lineno, errorEvent.colno);
   }
-  if (typeof reason === 'undefined') return new Error(prefix);
-  try {
-    return new Error(prefix + (0, _stringify.stringify)(reason));
-  } catch (err) {
-    return new Error(prefix);
-  }
-}
+}
+exports.Instrument = Instrument;

package/dist/cjs/features/jserrors/instrument/uncaught-error.js

@@ -0,0 +1,22 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.UncaughtError = void 0;
+/**
+ * Represents an uncaught non Error type error. This class does
+ * not extend the Error class to prevent an invalid stack trace
+ * from being created. Use this class to cast thrown errors that
+ * do not use the Error class (strings, etc) to an object.
+ */
+class UncaughtError {
+  constructor(message, filename, lineno, colno) {
+    this.name = 'UncaughtError';
+    this.message = message;
+    this.sourceURL = filename;
+    this.line = lineno;
+    this.column = colno;
+  }
+}
+exports.UncaughtError = UncaughtError;

package/dist/cjs/features/metrics/aggregate/framework-detection.js

@@ -4,7 +4,7 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.getFrameworks = getFrameworks;
-var _globalScope = require("../../../common/util/global-scope");
+var _runtime = require("../../../common/constants/runtime");
 const FRAMEWORKS = {
   REACT: 'React',
   ANGULAR: 'Angular',
@@ -18,7 +18,7 @@ const FRAMEWORKS = {
   MOOTOOLS: 'MooTools'
 };
 function getFrameworks() {
-  if (!_globalScope.isBrowserScope) return []; // don't bother detecting frameworks if not in the main window context
+  if (!_runtime.isBrowserScope) return []; // don't bother detecting frameworks if not in the main window context
 
   const frameworks = [];
   try {

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

@@ -16,7 +16,7 @@ var _obfuscate = require("../../../common/util/obfuscate");
 var _env = require("../../../common/constants/env.npm");
 var _load = require("../../../common/window/load");
 var _eventListenerOpts = require("../../../common/event-listener/event-listener-opts");
-var _globalScope = require("../../../common/util/global-scope");
+var _runtime = require("../../../common/constants/runtime");
 var _aggregateBase = require("../../utils/aggregate-base");
 var _stringify = require("../../../common/util/stringify");
 var _endpointMap = require("./endpoint-map");
@@ -77,7 +77,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
     if (distMethod) this.storeSupportabilityMetrics("Generic/DistMethod/".concat(distMethod, "/Detected"));
 
     // frameworks on page
-    if (_globalScope.isBrowserScope) {
+    if (_runtime.isBrowserScope) {
       (0, _load.onDOMContentLoaded)(() => {
         (0, _frameworkDetection.getFrameworks)().forEach(framework => {
           this.storeSupportabilityMetrics('Framework/' + framework + '/Detected');
@@ -99,7 +99,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
     if (rules.length > 0 && !(0, _obfuscate.validateRules)(rules)) this.storeSupportabilityMetrics('Generic/Obfuscate/Invalid');
   }
   eachSessionChecks() {
-    if (!_globalScope.isBrowserScope) return;
+    if (!_runtime.isBrowserScope) return;
 
     // [Temporary] Report restores from BFCache to NR1 while feature flag is in place in lieu of sending pageshow events.
     (0, _eventListenerOpts.windowAddEventListener)('pageshow', evt => {

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

@@ -12,7 +12,7 @@ var _cleanUrl = require("../../../common/url/clean-url");
 var _config = require("../../../common/config/config");
 var _constants = require("../constants");
 var _drain = require("../../../common/drain/drain");
-var _globalScope = require("../../../common/util/global-scope");
+var _runtime = require("../../../common/constants/runtime");
 var _aggregateBase = require("../../utils/aggregate-base");
 /*
  * Copyright 2020 New Relic Corporation. All rights reserved.
@@ -33,7 +33,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
     this.events = [];
     this.att = (0, _config.getInfo)(this.agentIdentifier).jsAttributes; // per-agent, aggregators-shared info context
 
-    if (_globalScope.isBrowserScope && document.referrer) this.referrerUrl = (0, _cleanUrl.cleanURL)(document.referrer);
+    if (_runtime.isBrowserScope && document.referrer) this.referrerUrl = (0, _cleanUrl.cleanURL)(document.referrer);
     (0, _registerHandler.registerHandler)('api-addPageAction', function () {
       return _this.addPageAction(...arguments);
     }, this.featureName, this.ee);
@@ -89,7 +89,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
     var width;
     var height;
     var eventAttributes = {};
-    if (_globalScope.isBrowserScope && window.document.documentElement) {
+    if (_runtime.isBrowserScope && window.document.documentElement) {
       // Doesn't include the nav bar when it disappears in mobile safari
       // https://github.com/jquery/jquery/blob/10399ddcf8a239acc27bdec9231b996b178224d3/src/dimensions.js#L23
       width = window.document.documentElement.clientWidth;

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

@@ -6,7 +6,7 @@ Object.defineProperty(exports, "__esModule", {
 exports.Aggregate = void 0;
 var _handle = require("../../../common/event-emitter/handle");
 var _features = require("../../../loaders/features/features");
-var _iosVersion = require("../../../common/browser-version/ios-version");
+var _runtime = require("../../../common/constants/runtime");
 var _webVitals = require("web-vitals");
 var _navTiming = require("../../../common/timing/nav-timing");
 var _stringify = require("../../../common/util/stringify");
@@ -15,7 +15,6 @@ var _config = require("../../../common/config/config");
 var _harvest = require("../../../common/harvest/harvest");
 var CONSTANTS = _interopRequireWildcard(require("../constants"));
 var _initializedFeatures = require("./initialized-features");
-var _globalScope = require("../../../common/util/global-scope");
 var _drain = require("../../../common/drain/drain");
 var _featureFlags = require("../../../common/util/feature-flags");
 var _console = require("../../../common/util/console");
@@ -26,7 +25,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
   static featureName = CONSTANTS.FEATURE_NAME;
   constructor(agentIdentifier, aggregator) {
     super(agentIdentifier, aggregator, CONSTANTS.FEATURE_NAME);
-    if (typeof PerformanceNavigationTiming !== 'undefined' && !_iosVersion.isiOS) {
+    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
 
@@ -74,13 +73,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: _globalScope.isBrowserScope ? agentRuntime[CONSTANTS.TTFB] : 0
+      value: _runtime.isBrowserScope ? agentRuntime[CONSTANTS.TTFB] : 0
     });
     this.aggregator.store('measures', 'fe', {
-      value: _globalScope.isBrowserScope ? agentRuntime[CONSTANTS.FBTWL] : 0
+      value: _runtime.isBrowserScope ? agentRuntime[CONSTANTS.FBTWL] : 0
     });
     this.aggregator.store('measures', 'dc', {
-      value: _globalScope.isBrowserScope ? agentRuntime[CONSTANTS.FBTDC] : 0
+      value: _runtime.isBrowserScope ? agentRuntime[CONSTANTS.FBTDC] : 0
     });
     const queryParameters = {
       tt: info.ttGuid,
@@ -103,10 +102,10 @@ class Aggregate extends _aggregateBase.AggregateBase {
         ja: info.jsAttributes
       };
     }
-    if (_globalScope.globalScope.performance) {
+    if (_runtime.globalScope.performance) {
       if (typeof PerformanceNavigationTiming !== 'undefined') {
         // Navigation Timing level 2 API that replaced PerformanceTiming & PerformanceNavigation
-        const navTimingEntry = _globalScope.globalScope?.performance?.getEntriesByType('navigation')?.[0];
+        const navTimingEntry = _runtime.globalScope?.performance?.getEntriesByType('navigation')?.[0];
         const perf = {
           timing: (0, _navTiming.addPT)(agentRuntime.offset, navTimingEntry, {}),
           navigation: (0, _navTiming.addPN)(navTimingEntry, {})
@@ -115,15 +114,15 @@ class Aggregate extends _aggregateBase.AggregateBase {
       } else if (typeof PerformanceTiming !== 'undefined') {
         // Safari pre-15 did not support level 2 timing
         const perf = {
-          timing: (0, _navTiming.addPT)(agentRuntime.offset, _globalScope.globalScope.performance.timing, {}, true),
-          navigation: (0, _navTiming.addPN)(_globalScope.globalScope.performance.navigation, {})
+          timing: (0, _navTiming.addPT)(agentRuntime.offset, _runtime.globalScope.performance.timing, {}, true),
+          navigation: (0, _navTiming.addPN)(_runtime.globalScope.performance.navigation, {})
         };
         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 = _globalScope.globalScope.performance.getEntriesByType('paint');
+      var entries = _runtime.globalScope.performance.getEntriesByType('paint');
       entries.forEach(function (entry) {
         if (!entry.startTime || entry.startTime <= 0) return;
         if (entry.name === 'first-paint') {

package/dist/cjs/features/page_view_event/aggregate/initialized-features.js

@@ -5,6 +5,7 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.getActivatedFeaturesFlags = getActivatedFeaturesFlags;
 var _features = require("../../../loaders/features/features");
+var _nreum = require("../../../common/window/nreum");
 /**
  * Get an array of flags required by downstream (NR UI) based on the features initialized in this agent
  * (aka what is running on the page).
@@ -13,25 +14,28 @@ var _features = require("../../../loaders/features/features");
  */
 function getActivatedFeaturesFlags(agentId) {
   const flagArr = [];
-  Object.keys(newrelic.initializedAgents[agentId].features).forEach(featName => {
-    switch (featName) {
-      case _features.FEATURE_NAMES.ajax:
-        flagArr.push('xhr');
-        break;
-      case _features.FEATURE_NAMES.jserrors:
-        flagArr.push('err');
-        break;
-      case _features.FEATURE_NAMES.pageAction:
-        flagArr.push('ins');
-        break;
-      case _features.FEATURE_NAMES.sessionTrace:
-        flagArr.push('stn');
-        break;
-      case _features.FEATURE_NAMES.spa:
-        flagArr.push('spa');
-        break;
-    }
-  });
+  const newrelic = (0, _nreum.gosNREUM)();
+  try {
+    Object.keys(newrelic.initializedAgents[agentId].features).forEach(featName => {
+      switch (featName) {
+        case _features.FEATURE_NAMES.ajax:
+          flagArr.push('xhr');
+          break;
+        case _features.FEATURE_NAMES.jserrors:
+          flagArr.push('err');
+          break;
+        case _features.FEATURE_NAMES.pageAction:
+          flagArr.push('ins');
+          break;
+        case _features.FEATURE_NAMES.sessionTrace:
+          flagArr.push('stn');
+          break;
+        case _features.FEATURE_NAMES.spa:
+          flagArr.push('spa');
+          break;
+      }
+    });
+  } catch (e) {}
   return flagArr;
 }
 

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

@@ -5,7 +5,7 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.Instrument = void 0;
 var _handle = require("../../../common/event-emitter/handle");
-var _iosVersion = require("../../../common/browser-version/ios-version");
+var _runtime = require("../../../common/constants/runtime");
 var _instrumentBase = require("../../utils/instrument-base");
 var CONSTANTS = _interopRequireWildcard(require("../constants"));
 var _features = require("../../../loaders/features/features");
@@ -19,7 +19,7 @@ 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' || _iosVersion.isiOS) && typeof PerformanceTiming !== 'undefined') {
+    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 iOS_below16 bug.
       const agentRuntime = (0, _config.getRuntime)(agentIdentifier);

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

@@ -7,7 +7,7 @@ exports.Aggregate = void 0;
 var _webVitals = require("web-vitals");
 var _firstPaint = require("../first-paint");
 var _longTasks = require("../long-tasks");
-var _iosVersion = require("../../../common/browser-version/ios-version");
+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");
@@ -51,7 +51,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
     });
 
     /* 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 (_iosVersion.iOS_below16) {
+    if (_runtime.iOS_below16) {
       try {
         if (!pageStartedHidden) {
           // see ios-version.js for detail on this following bug case; tldr: buffered flag doesn't work but getEntriesByType does
@@ -238,9 +238,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
   }
   onHarvestFinished(result) {
     if (result.retry && this.timingsSent.length > 0) {
-      for (var i = 0; i < this.timingsSent.length; i++) {
-        this.timings.push(this.timingsSent[i]);
-      }
+      this.timings.unshift(...this.timingsSent);
       this.timingsSent = [];
     }
   }

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

@@ -11,7 +11,7 @@ var _eventListenerOpts = require("../../../common/event-listener/event-listener-
 var _now = require("../../../common/timing/now");
 var _instrumentBase = require("../../utils/instrument-base");
 var _constants = require("../constants");
-var _globalScope = require("../../../common/util/global-scope");
+var _runtime = require("../../../common/constants/runtime");
 /*
  * Copyright 2020 New Relic Corporation. All rights reserved.
  * SPDX-License-Identifier: Apache-2.0
@@ -22,7 +22,7 @@ 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 (!_globalScope.isBrowserScope) return; // CWV is irrelevant outside web context
+    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.