@newrelic/browser-agent 1.258.0 → 1.258.2

package/CHANGELOG.md

@@ -3,6 +3,24 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [1.258.2](https://github.com/newrelic/newrelic-browser-agent/compare/v1.258.1...v1.258.2) (2024-05-07)
+
+
+### Bug Fixes
+
+* Prevent noticeError() API from running if not given an argument ([#1021](https://github.com/newrelic/newrelic-browser-agent/issues/1021)) ([c023a53](https://github.com/newrelic/newrelic-browser-agent/commit/c023a53c20a9f0f2472e1ba5ff78eb7210a906fa))
+
+## [1.258.1](https://github.com/newrelic/newrelic-browser-agent/compare/v1.258.0...v1.258.1) (2024-05-07)
+
+
+### Bug Fixes
+
+* Exclude data url requests as captured AJAX events ([#1012](https://github.com/newrelic/newrelic-browser-agent/issues/1012)) ([2a3fa57](https://github.com/newrelic/newrelic-browser-agent/commit/2a3fa57da1f389e6eabae2c92686e25285fc6cd3))
+* Improve Error Casting ([#1014](https://github.com/newrelic/newrelic-browser-agent/issues/1014)) ([d1dd20c](https://github.com/newrelic/newrelic-browser-agent/commit/d1dd20ce526ddb697962f695fbb5915410474987))
+* Remove API start()'s features param ([#1009](https://github.com/newrelic/newrelic-browser-agent/issues/1009)) ([38a502b](https://github.com/newrelic/newrelic-browser-agent/commit/38a502b08b4735259e7f5b863b0e5e5361c075b6))
+* Run inside cross-origin iframes for firefox/safari ([#1015](https://github.com/newrelic/newrelic-browser-agent/issues/1015)) ([6a4a73d](https://github.com/newrelic/newrelic-browser-agent/commit/6a4a73d72b056177268f8619a3a3b7810a7a2c79))
+* Session trace nodes de-duplication ([#1008](https://github.com/newrelic/newrelic-browser-agent/issues/1008)) ([44f229e](https://github.com/newrelic/newrelic-browser-agent/commit/44f229e4d35cd468bfe29b1796be8031bb9c72ff))
+
 ## [1.258.0](https://github.com/newrelic/newrelic-browser-agent/compare/v1.257.0...v1.258.0) (2024-04-29)
 
 

package/README.md

@@ -82,6 +82,7 @@ The following features may be disabled by adding `init` entries as shown above.
 - `metrics`
 - `page_action`
 - `page_view_timing`
+- `session_replay`
 - `session_trace`
 - `spa`
 
@@ -186,9 +187,7 @@ Please see our [official documentation](https://docs.newrelic.com/docs/browser/n
 ```
 
 ## Session Replay
-The Session Replay feature is currently in Limited Preview and only functional for customers participating in the early access program.  To request access, please visit [this link](https://newrelic.com/platform/session-replay-early-access). 
-
-Due to the sensitive nature of the feature, Session Replay has many configuration options, which are configurable in each browser application's *Application Settings* page on the New Relic site. These settings will only be accessible if you are participating in the limited preview. Additionally, you can control the sampling rates, obfuscation conditions and triggering rules of Session Replay.
+The Session Replay feature is now available for limited free use by all customers. The data collected by this feature will become billable starting May 15th, 2024. Please see the [Session Replay documentation](https://docs.newrelic.com/docs/browser/browser-monitoring/browser-pro-features/session-replay/) to get started using this new feature.
 
 ## Supported browsers
 
@@ -239,7 +238,7 @@ A lot of new frameworks support the concept of server-side rendering the pages o
 
 ## Disclaimers
 
-The session replay library shipping with this version of the browser agent is in *limited preview* and is not turned on by default. To use the feature, users will need to be part of the limited preview customer group and configure their browser application settings in the UI.
+The session replay library shipping as part of the browser agent is not turned on by default. For information on the use of this feature, see [Session Replay](#session-replay)
 
 ## Support
 

package/dist/cjs/common/constants/env.cdn.js

@@ -12,7 +12,7 @@ exports.VERSION = exports.RRWEB_VERSION = exports.DIST_METHOD = exports.BUILD_EN
 /**
  * Exposes the version of the agent
  */
-const VERSION = exports.VERSION = "1.258.0";
+const VERSION = exports.VERSION = "1.258.2";
 
 /**
  * Exposes the build type of the agent

package/dist/cjs/common/constants/env.npm.js

@@ -12,7 +12,7 @@ exports.VERSION = exports.RRWEB_VERSION = exports.DIST_METHOD = exports.BUILD_EN
 /**
  * Exposes the version of the agent
  */
-const VERSION = exports.VERSION = "1.258.0";
+const VERSION = exports.VERSION = "1.258.2";
 
 /**
  * Exposes the build type of the agent

package/dist/cjs/common/deny-list/deny-list.js

@@ -3,6 +3,7 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
+exports.hasUndefinedHostname = hasUndefinedHostname;
 exports.setDenyList = setDenyList;
 exports.shouldCollectEvent = shouldCollectEvent;
 /** An array of filter objects {hostname, pathname} for identifying XHR events to be excluded from collection.
@@ -17,14 +18,8 @@ var denyList = [];
  * @returns {boolean} `true` if request does not match any entries of {@link denyList|deny list}; else `false`
  */
 function shouldCollectEvent(params) {
-  if (denyList.length === 0) {
-    return true;
-  }
-
-  // XHR requests with an undefined hostname (e.g., data URLs) should not be collected.
-  if (params.hostname === undefined) {
-    return false;
-  }
+  if (hasUndefinedHostname(params)) return false;
+  if (denyList.length === 0) return true;
   for (var i = 0; i < denyList.length; i++) {
     var parsed = denyList[i];
     if (parsed.hostname === '*') {
@@ -36,6 +31,9 @@ function shouldCollectEvent(params) {
   }
   return true;
 }
+function hasUndefinedHostname(params) {
+  return params.hostname === undefined; // requests with an undefined hostname (e.g., data URLs) should not be collected.
+}
 
 /**
  * Initializes the {@link denyList|XHR deny list} by extracting hostname and pathname from an array of filter strings.

package/dist/cjs/common/vitals/time-to-first-byte.js

@@ -9,7 +9,15 @@ var _constants = require("./constants");
 var _vitalMetric = require("./vital-metric");
 var _attribution = require("web-vitals/attribution");
 const timeToFirstByte = exports.timeToFirstByte = new _vitalMetric.VitalMetric(_constants.VITAL_NAMES.TIME_TO_FIRST_BYTE);
-if (_runtime.isBrowserScope && typeof PerformanceNavigationTiming !== 'undefined' && !_runtime.isiOS) {
+
+/**
+ * onTTFB is not supported in the following scenarios:
+ * - in a non-browser scope
+ * - in browsers that do not support PerformanceNavigationTiming API
+ * - in an iOS browser
+ * - cross-origin iframes specifically in firefox and safari
+ */
+if (_runtime.isBrowserScope && typeof PerformanceNavigationTiming !== 'undefined' && !_runtime.isiOS && window === window.parent) {
   (0, _attribution.onTTFB)(_ref => {
     let {
       value,

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

@@ -19,6 +19,7 @@ var _constants = require("../constants");
 var _features = require("../../../loaders/features/features");
 var _constants2 = require("../../metrics/constants");
 var _now = require("../../../common/timing/now");
+var _denyList = require("../../../common/deny-list/deny-list");
 /*
  * Copyright 2020 New Relic Corporation. All rights reserved.
  * SPDX-License-Identifier: Apache-2.0
@@ -315,17 +316,17 @@ function subscribeToEvents(agentIdentifier, ee, handler, dt) {
   // eslint-disable-next-line handle-callback-err
   function onFetchDone(_, res) {
     this.endTime = (0, _now.now)();
-    if (!this.params) {
-      this.params = {};
-    }
+    if (!this.params) this.params = {};
+    if ((0, _denyList.hasUndefinedHostname)(this.params)) return; // don't bother with fetch to url with no hostname
+
     this.params.status = res ? res.status : 0;
 
     // convert rxSize to a number
-    var responseSize;
+    let responseSize;
     if (typeof this.rxSize === 'string' && this.rxSize.length > 0) {
       responseSize = +this.rxSize;
     }
-    var metrics = {
+    const metrics = {
       txSize: this.txSize,
       rxSize: responseSize,
       duration: (0, _now.now)() - this.startTime
@@ -335,14 +336,16 @@ function subscribeToEvents(agentIdentifier, ee, handler, dt) {
 
   // Create report for XHR request that has finished
   function end(xhr) {
-    var params = this.params;
-    var metrics = this.metrics;
+    const params = this.params;
+    const metrics = this.metrics;
     if (this.ended) return;
     this.ended = true;
-    for (var i = 0; i < handlersLen; i++) {
+    for (let i = 0; i < handlersLen; i++) {
       xhr.removeEventListener(handlers[i], this.listener, false);
     }
     if (params.aborted) return;
+    if ((0, _denyList.hasUndefinedHostname)(params)) return; // don't bother with XHR of url with no hostname
+
     metrics.duration = (0, _now.now)() - this.startTime;
     if (!this.loadCaptureCalled && xhr.readyState === 4) {
       captureXhrData(this, xhr);

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

@@ -140,6 +140,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
     return canonicalStackString;
   }
   storeError(err, time, internal, customAttributes, hasReplay) {
+    if (!err) return;
     // are we in an interaction
     time = time || (0, _now.now)();
     const agentRuntime = (0, _config.getRuntime)(this.agentIdentifier);
@@ -199,13 +200,17 @@ class Aggregate extends _aggregateBase.AggregateBase {
 
     // Trace sends the error in its payload, and both trace & replay simply listens for any error to occur.
     const jsErrorEvent = [type, bucketHash, params, newMetrics, customAttributes];
-    (0, _handle.handle)('errorAgg', jsErrorEvent, undefined, _features.FEATURE_NAMES.sessionTrace, this.ee);
+    (0, _handle.handle)('trace-jserror', jsErrorEvent, undefined, _features.FEATURE_NAMES.sessionTrace, this.ee);
     // still send EE events for other features such as above, but stop this one from aggregating internal data
     if (this.blocked) return;
+    if (err?.__newrelic?.[this.agentIdentifier]) {
+      params._interactionId = err.__newrelic[this.agentIdentifier].interactionId;
+      params._interactionNodeId = err.__newrelic[this.agentIdentifier].interactionNodeId;
+    }
     const softNavInUse = Boolean((0, _nreum.getNREUMInitializedAgent)(this.agentIdentifier)?.features[_features.FEATURE_NAMES.softNav]);
     // Note: the following are subject to potential race cond wherein if the other feature aren't fully initialized, it'll be treated as there being no associated interaction.
     // They each will also tack on their respective properties to the params object as part of the decision flow.
-    if (softNavInUse) (0, _handle.handle)('jserror', [params, time], undefined, _features.FEATURE_NAMES.softNav, this.ee);else (0, _handle.handle)('errorAgg', jsErrorEvent, undefined, _features.FEATURE_NAMES.spa, this.ee);
+    if (softNavInUse) (0, _handle.handle)('jserror', [params, time], undefined, _features.FEATURE_NAMES.softNav, this.ee);else (0, _handle.handle)('spa-jserror', jsErrorEvent, undefined, _features.FEATURE_NAMES.spa, this.ee);
     if (params.browserInteractionId && !params._softNavFinished) {
       // hold onto the error until the in-progress interaction is done, eithered saved or discarded
       this.bufferedErrorsUnderSpa[params.browserInteractionId] ??= [];

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

@@ -10,10 +10,9 @@ var _constants = require("../constants");
 var _features = require("../../../loaders/features/features");
 var _runtime = require("../../../common/constants/runtime");
 var _eventListenerOpts = require("../../../common/event-listener/event-listener-opts");
-var _stringify = require("../../../common/util/stringify");
-var _uncaughtError = require("./uncaught-error");
 var _now = require("../../../common/timing/now");
 var _constants2 = require("../../session_replay/constants");
+var _castError = require("../shared/cast-error");
 /*
  * Copyright 2020 New Relic Corporation. All rights reserved.
  * SPDX-License-Identifier: Apache-2.0
@@ -21,7 +20,6 @@ var _constants2 = require("../../session_replay/constants");
 
 class Instrument extends _instrumentBase.InstrumentBase {
   static featureName = _constants.FEATURE_NAME;
-  #seenErrors = new Set();
   #replayRunning = false;
   constructor(agentIdentifier, aggregator) {
     let auto = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : true;
@@ -30,38 +28,22 @@ class Instrument extends _instrumentBase.InstrumentBase {
       // this try-catch can be removed when IE11 is completely unsupported & gone
       this.removeOnAbort = new AbortController();
     } catch (e) {}
-
-    // 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);
-    });
     this.ee.on('internal-error', error => {
       if (!this.abortHandler) return;
-      (0, _handle.handle)('ierr', [this.#castError(error), (0, _now.now)(), true, {}, this.#replayRunning], undefined, _features.FEATURE_NAMES.jserrors, this.ee);
+      (0, _handle.handle)('ierr', [(0, _castError.castError)(error), (0, _now.now)(), true, {}, this.#replayRunning], undefined, _features.FEATURE_NAMES.jserrors, this.ee);
     });
     this.ee.on(_constants2.SR_EVENT_EMITTER_TYPES.REPLAY_RUNNING, isRunning => {
       this.#replayRunning = isRunning;
     });
     _runtime.globalScope.addEventListener('unhandledrejection', promiseRejectionEvent => {
       if (!this.abortHandler) return;
-      (0, _handle.handle)('err', [this.#castPromiseRejectionEvent(promiseRejectionEvent), (0, _now.now)(), false, {
+      (0, _handle.handle)('err', [(0, _castError.castPromiseRejectionEvent)(promiseRejectionEvent), (0, _now.now)(), false, {
         unhandledPromiseRejection: 1
       }, this.#replayRunning], undefined, _features.FEATURE_NAMES.jserrors, this.ee);
     }, (0, _eventListenerOpts.eventListenerOpts)(false, this.removeOnAbort?.signal));
     _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)(), false, {}, this.#replayRunning], undefined, _features.FEATURE_NAMES.jserrors, this.ee);
+      (0, _handle.handle)('err', [(0, _castError.castErrorEvent)(errorEvent), (0, _now.now)(), false, {}, this.#replayRunning], 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();
@@ -70,72 +52,7 @@ 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
   }
-
-  /**
-   * 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
-   */
-  #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 {
-        promiseRejectionEvent.reason.message = prefix + promiseRejectionEvent.reason.message;
-        return promiseRejectionEvent.reason;
-      } catch (e) {
-        return promiseRejectionEvent.reason;
-      }
-    }
-    if (typeof promiseRejectionEvent.reason === 'undefined') return new _uncaughtError.UncaughtError(prefix);
-    const error = this.#castError(promiseRejectionEvent.reason);
-    error.message = prefix + error.message;
-    return error;
-  }
-
-  /**
-   * 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 SyntaxError && !/:\d+$/.test(errorEvent.error.stack?.trim())) {
-      const error = new _uncaughtError.UncaughtError(errorEvent.message, errorEvent.filename, errorEvent.lineno, errorEvent.colno);
-      error.name = SyntaxError.name;
-      return error;
-    }
-    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);
-  }
 }
 exports.Instrument = Instrument;

package/dist/cjs/features/jserrors/shared/cast-error.js

@@ -0,0 +1,66 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.castError = castError;
+exports.castErrorEvent = castErrorEvent;
+exports.castPromiseRejectionEvent = castPromiseRejectionEvent;
+var _uncaughtError = require("./uncaught-error");
+/**
+   * 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
+   */
+function castError(error) {
+  /** Sometimes a browser can emit an error object with no stack */
+  if (canTrustError(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.
+     */
+  return new _uncaughtError.UncaughtError(error?.message !== undefined ? error.message : error, error?.filename || error?.sourceURL, error?.lineno || error?.line, error?.colno || error?.col, error?.__newrelic);
+}
+
+/**
+   * 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
+   */
+function castPromiseRejectionEvent(promiseRejectionEvent) {
+  let prefix = 'Unhandled Promise Rejection';
+  if (canTrustError(promiseRejectionEvent?.reason)) {
+    try {
+      promiseRejectionEvent.reason.message = prefix + ': ' + promiseRejectionEvent.reason.message;
+      return castError(promiseRejectionEvent.reason);
+    } catch (e) {
+      return castError(promiseRejectionEvent.reason);
+    }
+  }
+  if (typeof promiseRejectionEvent.reason === 'undefined') return castError(prefix);
+  const error = castError(promiseRejectionEvent.reason);
+  error.message = prefix + ': ' + error?.message;
+  return error;
+}
+
+/**
+   * 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
+   */
+function castErrorEvent(errorEvent) {
+  if (errorEvent.error instanceof SyntaxError && !/:\d+$/.test(errorEvent.error.stack?.trim())) {
+    const error = new _uncaughtError.UncaughtError(errorEvent.message, errorEvent.filename, errorEvent.lineno, errorEvent.colno, errorEvent.error.__newrelic);
+    error.name = SyntaxError.name;
+    return error;
+  }
+  if (canTrustError(errorEvent.error)) return errorEvent.error;
+  return castError(errorEvent);
+}
+function canTrustError(error) {
+  return error instanceof Error && !!error.stack;
+}

package/dist/cjs/features/jserrors/{instrument → shared}/uncaught-error.js

@@ -4,6 +4,7 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.UncaughtError = void 0;
+var _stringify = require("../../../common/util/stringify");
 /**
  * Represents an uncaught non Error type error. This class does
  * not extend the Error class to prevent an invalid stack trace
@@ -11,12 +12,13 @@ exports.UncaughtError = void 0;
  * do not use the Error class (strings, etc) to an object.
  */
 class UncaughtError {
-  constructor(message, filename, lineno, colno) {
+  constructor(message, filename, lineno, colno, newrelic) {
     this.name = 'UncaughtError';
-    this.message = message;
+    this.message = typeof message === 'string' ? message : (0, _stringify.stringify)(message);
     this.sourceURL = filename;
     this.line = lineno;
     this.column = colno;
+    this.__newrelic = newrelic;
   }
 }
 exports.UncaughtError = UncaughtError;

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

@@ -59,6 +59,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
     this.trace = {};
     this.nodeCount = 0;
     this.sentTrace = null;
+    this.prevStoredEvents = new Set();
     this.harvestTimeSeconds = (0, _config.getConfigurationValue)(agentIdentifier, 'session_trace.harvestTimeSeconds') || 10;
     this.maxNodesPerHarvest = (0, _config.getConfigurationValue)(agentIdentifier, 'session_trace.maxNodesPerHarvest') || 1000;
     /**
@@ -126,7 +127,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
         return controlTraceOp(on);
       }, this.featureName, this.ee);
     } else {
-      (0, _registerHandler.registerHandler)('errorAgg', () => {
+      (0, _registerHandler.registerHandler)('trace-jserror', () => {
         seenAnError = true;
         switchToFull();
       }, this.featureName, this.ee);
@@ -221,7 +222,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
       }
       return operationalGate.settle(() => _this.storeSTN(...args));
     }, this.featureName, this.ee);
-    (0, _registerHandler.registerHandler)('errorAgg', function () {
+    (0, _registerHandler.registerHandler)('trace-jserror', function () {
       for (var _len6 = arguments.length, args = new Array(_len6), _key6 = 0; _key6 < _len6; _key6++) {
         args[_key6] = arguments[_key6];
       }
@@ -270,6 +271,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
     }
   }
   #prepareHarvest(options) {
+    this.prevStoredEvents.clear(); // release references to past events for GC
     if (this.isStandalone) {
       if (this.ptid && (0, _now.now)() >= MAX_TRACE_DURATION) {
         // Perform a final harvest once we hit or exceed the max session trace time
@@ -336,6 +338,8 @@ class Aggregate extends _aggregateBase.AggregateBase {
   // Tracks the events and their listener's duration on objects wrapped by wrap-events.
   storeEvent(currentEvent, target, start, end) {
     if (this.shouldIgnoreEvent(currentEvent, target)) return;
+    if (this.prevStoredEvents.has(currentEvent)) return; // prevent multiple listeners of an event from creating duplicate trace nodes per occurrence. Cleared every harvest. near-zero chance for re-duplication after clearing per harvest since the timestamps of the event are considered for uniqueness.
+    this.prevStoredEvents.add(currentEvent);
     const evt = {
       n: this.evtName(currentEvent.type),
       s: start,

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

@@ -651,7 +651,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
         state.interactionsSent = [];
       }
     }
-    baseEE.on('errorAgg', function (type, name, params, metrics) {
+    baseEE.on('spa-jserror', function (type, name, params, metrics) {
       if (!state.currentNode) return;
       params._interactionId = state.currentNode.interaction.id;
       // do not capture parentNodeId when in root node
@@ -659,6 +659,16 @@ class Aggregate extends _aggregateBase.AggregateBase {
         params._interactionNodeId = state.currentNode.id;
       }
     });
+    (0, _registerHandler.registerHandler)('function-err', function (args, obj, error) {
+      if (!state.currentNode) return;
+      error.__newrelic ??= {};
+      error.__newrelic[agentIdentifier] = {
+        interactionId: state.currentNode.interaction.id
+      };
+      if (state.currentNode.type && state.currentNode.type !== 'interaction') {
+        error.__newrelic[agentIdentifier].interactionNodeId = state.currentNode.id;
+      }
+    }, this.featureName, baseEE);
     baseEE.on('interaction', saveInteraction);
     function getActionText(node) {
       var nodeType = node.tagName.toLowerCase();

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

@@ -10,6 +10,7 @@ var _instrumentBase = require("../../utils/instrument-base");
 var CONSTANTS = _interopRequireWildcard(require("../constants"));
 var _runtime = require("../../../common/constants/runtime");
 var _now = require("../../../common/timing/now");
+var _handle = require("../../../common/event-emitter/handle");
 function _getRequireWildcardCache(e) { if ("function" != typeof WeakMap) return null; var r = new WeakMap(), t = new WeakMap(); return (_getRequireWildcardCache = function (e) { return e ? t : r; })(e); }
 function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e; if (null === e || "object" != typeof e && "function" != typeof e) return { default: e }; var t = _getRequireWildcardCache(r); if (t && t.has(e)) return t.get(e); var n = { __proto__: null }, a = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var u in e) if ("default" !== u && Object.prototype.hasOwnProperty.call(e, u)) { var i = a ? Object.getOwnPropertyDescriptor(e, u) : null; i && (i.get || i.set) ? Object.defineProperty(n, u, i) : n[u] = e[u]; } return n.default = e, t && t.set(e, n), n; }
 /*
@@ -32,8 +33,10 @@ const {
 class Instrument extends _instrumentBase.InstrumentBase {
   static featureName = FEATURE_NAME;
   constructor(agentIdentifier, aggregator) {
+    var _this;
     let auto = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : true;
     super(agentIdentifier, aggregator, FEATURE_NAME, auto);
+    _this = this;
     if (!_runtime.isBrowserScope) return; // SPA not supported outside web env
 
     try {
@@ -56,6 +59,12 @@ class Instrument extends _instrumentBase.InstrumentBase {
     this.ee.on(FN_END, endTimestamp);
     promiseEE.on(CB_END, endTimestamp);
     jsonpEE.on(CB_END, endTimestamp);
+    this.ee.on('fn-err', function () {
+      for (var _len = arguments.length, args = new Array(_len), _key = 0; _key < _len; _key++) {
+        args[_key] = arguments[_key];
+      }
+      if (!args[2]?.__newrelic?.[agentIdentifier]) (0, _handle.handle)('function-err', [...args], undefined, _this.featureName, _this.ee);
+    });
     this.ee.buffer([FN_START, FN_END, 'xhr-resolved'], this.featureName);
     eventsEE.buffer([FN_START], this.featureName);
     timerEE.buffer(['setTimeout' + END, 'clearTimeout' + START, FN_START], this.featureName);

package/dist/cjs/features/utils/instrument-base.js

@@ -57,7 +57,7 @@ class InstrumentBase extends _featureBase.FeatureBase {
     if ((0, _config.getConfigurationValue)(this.agentIdentifier, "".concat(this.featureName, ".autoStart")) === false) this.auto = false;
     /** if the feature requires opt-in (!auto-start), it will get registered once the api has been called */
     if (this.auto) (0, _drain.registerDrain)(agentIdentifier, featureName);else {
-      this.ee.on("".concat(this.featureName, "-opt-in"), (0, _invoke.single)(() => {
+      this.ee.on('manual-start-all', (0, _invoke.single)(() => {
         // register the feature to drain only once the API has been called, it will drain when importAggregator finishes for all the features
         // called by the api in that cycle
         (0, _drain.registerDrain)(this.agentIdentifier, this.featureName);

package/dist/cjs/loaders/api/api.js

@@ -133,19 +133,10 @@ function setAPI(agentIdentifier, forceDrain) {
     }
     return appendJsAttribute('application.version', value, 'setApplicationVersion', false);
   };
-  apiInterface.start = features => {
+  apiInterface.start = () => {
     try {
-      const smTag = !features ? 'undefined' : 'defined';
-      (0, _handle.handle)(_constants.SUPPORTABILITY_METRIC_CHANNEL, ["API/start/".concat(smTag, "/called")], undefined, _features.FEATURE_NAMES.metrics, instanceEE);
-      const featNames = Object.values(_features.FEATURE_NAMES);
-      if (features === undefined) features = featNames;else {
-        features = Array.isArray(features) && features.length ? features : [features];
-        if (features.some(f => !featNames.includes(f))) return (0, _console.warn)("Invalid feature name supplied. Acceptable feature names are: ".concat(featNames));
-        if (!features.includes(_features.FEATURE_NAMES.pageViewEvent)) features.push(_features.FEATURE_NAMES.pageViewEvent);
-      }
-      features.forEach(feature => {
-        instanceEE.emit("".concat(feature, "-opt-in"));
-      });
+      (0, _handle.handle)(_constants.SUPPORTABILITY_METRIC_CHANNEL, ['API/start/called'], undefined, _features.FEATURE_NAMES.metrics, instanceEE);
+      instanceEE.emit('manual-start-all');
     } catch (err) {
       (0, _console.warn)('An unexpected issue occurred', err);
     }
@@ -176,9 +167,10 @@ function setAPI(agentIdentifier, forceDrain) {
           try {
             return cb.apply(this, arguments);
           } catch (err) {
-            tracerEE.emit('fn-err', [arguments, this, err], contextStore);
+            const error = typeof err === 'string' ? new Error(err) : err;
+            tracerEE.emit('fn-err', [arguments, this, error], contextStore);
             // the error came from outside the agent, so don't swallow
-            throw err;
+            throw error;
           } finally {
             tracerEE.emit('fn-end', [(0, _now.now)()], contextStore);
           }

package/dist/esm/common/constants/env.cdn.js

@@ -6,7 +6,7 @@
 /**
  * Exposes the version of the agent
  */
-export const VERSION = "1.258.0";
+export const VERSION = "1.258.2";
 
 /**
  * Exposes the build type of the agent

package/dist/esm/common/constants/env.npm.js

@@ -6,7 +6,7 @@
 /**
  * Exposes the version of the agent
  */
-export const VERSION = "1.258.0";
+export const VERSION = "1.258.2";
 
 /**
  * Exposes the build type of the agent

package/dist/esm/common/deny-list/deny-list.js

@@ -10,14 +10,8 @@ var denyList = [];
  * @returns {boolean} `true` if request does not match any entries of {@link denyList|deny list}; else `false`
  */
 export function shouldCollectEvent(params) {
-  if (denyList.length === 0) {
-    return true;
-  }
-
-  // XHR requests with an undefined hostname (e.g., data URLs) should not be collected.
-  if (params.hostname === undefined) {
-    return false;
-  }
+  if (hasUndefinedHostname(params)) return false;
+  if (denyList.length === 0) return true;
   for (var i = 0; i < denyList.length; i++) {
     var parsed = denyList[i];
     if (parsed.hostname === '*') {
@@ -29,6 +23,9 @@ export function shouldCollectEvent(params) {
   }
   return true;
 }
+export function hasUndefinedHostname(params) {
+  return params.hostname === undefined; // requests with an undefined hostname (e.g., data URLs) should not be collected.
+}
 
 /**
  * Initializes the {@link denyList|XHR deny list} by extracting hostname and pathname from an array of filter strings.

package/dist/esm/common/vitals/time-to-first-byte.js

@@ -3,7 +3,15 @@ import { VITAL_NAMES } from './constants';
 import { VitalMetric } from './vital-metric';
 import { onTTFB } from 'web-vitals/attribution';
 export const timeToFirstByte = new VitalMetric(VITAL_NAMES.TIME_TO_FIRST_BYTE);
-if (isBrowserScope && typeof PerformanceNavigationTiming !== 'undefined' && !isiOS) {
+
+/**
+ * onTTFB is not supported in the following scenarios:
+ * - in a non-browser scope
+ * - in browsers that do not support PerformanceNavigationTiming API
+ * - in an iOS browser
+ * - cross-origin iframes specifically in firefox and safari
+ */
+if (isBrowserScope && typeof PerformanceNavigationTiming !== 'undefined' && !isiOS && window === window.parent) {
   onTTFB(_ref => {
     let {
       value,