@newrelic/browser-agent 1.261.0 → 1.261.2

package/CHANGELOG.md

@@ -3,6 +3,20 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [1.261.2](https://github.com/newrelic/newrelic-browser-agent/compare/v1.261.1...v1.261.2) (2024-07-01)
+
+
+### Bug Fixes
+
+* recordReplay will restart replays on same session page loads ([#1093](https://github.com/newrelic/newrelic-browser-agent/issues/1093)) ([cecddbe](https://github.com/newrelic/newrelic-browser-agent/commit/cecddbebc445dabcf3ce48c1fc88311198be6d0a))
+
+## [1.261.1](https://github.com/newrelic/newrelic-browser-agent/compare/v1.261.0...v1.261.1) (2024-06-26)
+
+
+### Bug Fixes
+
+* Unhandled promise rejection for session import ([#1088](https://github.com/newrelic/newrelic-browser-agent/issues/1088)) ([03efcf3](https://github.com/newrelic/newrelic-browser-agent/commit/03efcf36656ac883c4431829fda1bad6445373a1))
+
 ## [1.261.0](https://github.com/newrelic/newrelic-browser-agent/compare/v1.260.1...v1.261.0) (2024-06-26)
 
 

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.261.0";
+const VERSION = exports.VERSION = "1.261.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.261.0";
+const VERSION = exports.VERSION = "1.261.2";
 
 /**
  * Exposes the build type of the agent

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

@@ -32,7 +32,7 @@ class Instrument extends _instrumentBase.InstrumentBase {
       session = JSON.parse(localStorage.getItem("".concat(_constants.PREFIX, "_").concat(_constants.DEFAULT_KEY)));
     } catch (err) {}
     if ((0, _utils.hasReplayPrerequisite)(agentIdentifier)) {
-      this.ee.on('recordReplay', () => this.#apiStartOrRestartReplay());
+      this.ee.on(_constants2.SR_EVENT_EMITTER_TYPES.RECORD, () => this.#apiStartOrRestartReplay());
     }
     if (this.#canPreloadRecorder(session)) {
       this.#mode = session?.sessionReplayMode;

package/dist/cjs/features/session_replay/shared/utils.js

@@ -4,7 +4,6 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.buildNRMetaNode = buildNRMetaNode;
-exports.canImportReplayAgg = canImportReplayAgg;
 exports.hasReplayPrerequisite = hasReplayPrerequisite;
 exports.isPreloadAllowed = isPreloadAllowed;
 var _config = require("../../../common/config/config");
@@ -20,10 +19,6 @@ function hasReplayPrerequisite(agentId) {
 function isPreloadAllowed(agentId) {
   return (0, _config.getConfigurationValue)(agentId, 'session_replay.preload') === true && hasReplayPrerequisite(agentId);
 }
-function canImportReplayAgg(agentId, sessionMgr) {
-  if (!hasReplayPrerequisite(agentId)) return false;
-  return !!sessionMgr?.isNew || !!sessionMgr?.state.sessionReplayMode; // Session Replay should only try to run if already running from a previous page, or at the beginning of a session
-}
 function buildNRMetaNode(timestamp, timeKeeper) {
   const correctedTimestamp = timeKeeper.correctAbsoluteTimestamp(timestamp);
   return {

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

@@ -23,7 +23,7 @@ class AggregateBase extends _featureBase.FeatureBase {
    */
   waitForFlags() {
     let flagNames = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : [];
-    return new Promise((resolve, reject) => {
+    const flagsPromise = new Promise((resolve, reject) => {
       if (_featureFlags.activatedFeatures[this.agentIdentifier]) {
         resolve(buildOutput(_featureFlags.activatedFeatures[this.agentIdentifier]));
       } else {
@@ -39,6 +39,9 @@ class AggregateBase extends _featureBase.FeatureBase {
         });
       }
     });
+    return flagsPromise.catch(err => {
+      this.ee.emit('internal-error', [err]);
+    });
   }
   drain() {
     (0, _drain.drain)(this.agentIdentifier, this.featureName);

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

@@ -93,6 +93,7 @@ class InstrumentBase extends _featureBase.FeatureBase {
         }
       } catch (e) {
         (0, _console.warn)('A problem occurred when starting up session manager. This page will not start or extend any session.', e);
+        this.ee.emit('internal-error', [e]);
         if (this.featureName === _features.FEATURE_NAMES.sessionReplay) this.abortHandler?.(); // SR should stop recording if session DNE
       }
 
@@ -136,8 +137,15 @@ class InstrumentBase extends _featureBase.FeatureBase {
   * @returns
   */
   #shouldImportAgg(featureName, session) {
-    if (featureName === _features.FEATURE_NAMES.sessionReplay) return (0, _utils.canImportReplayAgg)(this.agentIdentifier, session);
-    return true;
+    switch (featureName) {
+      case _features.FEATURE_NAMES.sessionReplay:
+        // the session manager must be initialized successfully for Replay & Trace features
+        return (0, _utils.hasReplayPrerequisite)(this.agentIdentifier) && !!session;
+      case _features.FEATURE_NAMES.sessionTrace:
+        return !!session;
+      default:
+        return true;
+    }
   }
 }
 exports.InstrumentBase = InstrumentBase;

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

@@ -6,7 +6,7 @@
 /**
  * Exposes the version of the agent
  */
-export const VERSION = "1.261.0";
+export const VERSION = "1.261.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.261.0";
+export const VERSION = "1.261.2";
 
 /**
  * Exposes the build type of the agent

package/dist/esm/features/session_replay/instrument/index.js

@@ -26,7 +26,7 @@ export class Instrument extends InstrumentBase {
       session = JSON.parse(localStorage.getItem("".concat(PREFIX, "_").concat(DEFAULT_KEY)));
     } catch (err) {}
     if (hasReplayPrerequisite(agentIdentifier)) {
-      this.ee.on('recordReplay', () => this.#apiStartOrRestartReplay());
+      this.ee.on(SR_EVENT_EMITTER_TYPES.RECORD, () => this.#apiStartOrRestartReplay());
     }
     if (this.#canPreloadRecorder(session)) {
       this.#mode = session?.sessionReplayMode;

package/dist/esm/features/session_replay/shared/utils.js

@@ -11,10 +11,6 @@ export function hasReplayPrerequisite(agentId) {
 export function isPreloadAllowed(agentId) {
   return getConfigurationValue(agentId, 'session_replay.preload') === true && hasReplayPrerequisite(agentId);
 }
-export function canImportReplayAgg(agentId, sessionMgr) {
-  if (!hasReplayPrerequisite(agentId)) return false;
-  return !!sessionMgr?.isNew || !!sessionMgr?.state.sessionReplayMode; // Session Replay should only try to run if already running from a previous page, or at the beginning of a session
-}
 export function buildNRMetaNode(timestamp, timeKeeper) {
   const correctedTimestamp = timeKeeper.correctAbsoluteTimestamp(timestamp);
   return {

package/dist/esm/features/utils/aggregate-base.js

@@ -17,7 +17,7 @@ export class AggregateBase extends FeatureBase {
    */
   waitForFlags() {
     let flagNames = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : [];
-    return new Promise((resolve, reject) => {
+    const flagsPromise = new Promise((resolve, reject) => {
       if (activatedFeatures[this.agentIdentifier]) {
         resolve(buildOutput(activatedFeatures[this.agentIdentifier]));
       } else {
@@ -33,6 +33,9 @@ export class AggregateBase extends FeatureBase {
         });
       }
     });
+    return flagsPromise.catch(err => {
+      this.ee.emit('internal-error', [err]);
+    });
   }
   drain() {
     drain(this.agentIdentifier, this.featureName);

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

@@ -11,7 +11,7 @@ import { isBrowserScope } from '../../common/constants/runtime';
 import { warn } from '../../common/util/console';
 import { FEATURE_NAMES } from '../../loaders/features/features';
 import { getConfigurationValue } from '../../common/config/config';
-import { canImportReplayAgg } from '../session_replay/shared/utils';
+import { hasReplayPrerequisite } from '../session_replay/shared/utils';
 import { canEnableSessionTracking } from './feature-gates';
 import { single } from '../../common/util/invoke';
 
@@ -88,6 +88,7 @@ export class InstrumentBase extends FeatureBase {
         }
       } catch (e) {
         warn('A problem occurred when starting up session manager. This page will not start or extend any session.', e);
+        this.ee.emit('internal-error', [e]);
         if (this.featureName === FEATURE_NAMES.sessionReplay) this.abortHandler?.(); // SR should stop recording if session DNE
       }
 
@@ -131,7 +132,14 @@ export class InstrumentBase extends FeatureBase {
   * @returns
   */
   #shouldImportAgg(featureName, session) {
-    if (featureName === FEATURE_NAMES.sessionReplay) return canImportReplayAgg(this.agentIdentifier, session);
-    return true;
+    switch (featureName) {
+      case FEATURE_NAMES.sessionReplay:
+        // the session manager must be initialized successfully for Replay & Trace features
+        return hasReplayPrerequisite(this.agentIdentifier) && !!session;
+      case FEATURE_NAMES.sessionTrace:
+        return !!session;
+      default:
+        return true;
+    }
   }
 }

package/dist/types/features/session_replay/shared/utils.d.ts

@@ -1,6 +1,5 @@
 export function hasReplayPrerequisite(agentId: any): boolean;
 export function isPreloadAllowed(agentId: any): boolean;
-export function canImportReplayAgg(agentId: any, sessionMgr: any): boolean;
 export function buildNRMetaNode(timestamp: any, timeKeeper: any): {
     originalTimestamp: any;
     correctedTimestamp: any;

package/dist/types/features/session_replay/shared/utils.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"utils.d.ts","sourceRoot":"","sources":["../../../../../src/features/session_replay/shared/utils.js"],"names":[],"mappings":"AAIA,6DAIC;AAED,wDAEC;AAED,2EAGC;AAED;;;;;;;EAUC"}
+{"version":3,"file":"utils.d.ts","sourceRoot":"","sources":["../../../../../src/features/session_replay/shared/utils.js"],"names":[],"mappings":"AAIA,6DAIC;AAED,wDAEC;AAED;;;;;;;EAUC"}

package/dist/types/features/utils/aggregate-base.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"aggregate-base.d.ts","sourceRoot":"","sources":["../../../../src/features/utils/aggregate-base.js"],"names":[],"mappings":"AAOA;IACE,4BAGC;IAED;;;;OAIG;IACH,yBAHW,MAAM,EAAE,gBAmBlB;IAED,cAGC;IADC,6BAAmB;IAGrB;;;OAGG;IACH,2BAqBC;CACF;4BAnE2B,gBAAgB"}
+{"version":3,"file":"aggregate-base.d.ts","sourceRoot":"","sources":["../../../../src/features/utils/aggregate-base.js"],"names":[],"mappings":"AAOA;IACE,4BAGC;IAED;;;;OAIG;IACH,yBAHW,MAAM,EAAE,gBAsBlB;IAED,cAGC;IADC,6BAAmB;IAGrB;;;OAGG;IACH,2BAqBC;CACF;4BAtE2B,gBAAgB"}

package/dist/types/features/utils/instrument-base.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"instrument-base.d.ts","sourceRoot":"","sources":["../../../../src/features/utils/instrument-base.js"],"names":[],"mappings":"AAiBA;;;GAGG;AACH;IACE;;;;;;;;OAQG;IACH,6BAPW,MAAM,cACN,OAAO,mCAAmC,EAAE,UAAU,eACtD,MAAM,8BAqChB;IA9BC,cAAgB;IAEhB,8IAA8I;IAC9I,cADW,WAAW,SAAS,CACF;IAE7B;;;MAGE;IACF,eAHU,OAAO,kBAAkB,EAAE,aAAa,CAGpB;IAE9B;;;MAGE;IACF,kCAAoC;IAiBtC;;;;;OAKG;IACH,mEAgDC;;CAYF;4BA5H2B,gBAAgB"}
+{"version":3,"file":"instrument-base.d.ts","sourceRoot":"","sources":["../../../../src/features/utils/instrument-base.js"],"names":[],"mappings":"AAiBA;;;GAGG;AACH;IACE;;;;;;;;OAQG;IACH,6BAPW,MAAM,cACN,OAAO,mCAAmC,EAAE,UAAU,eACtD,MAAM,8BAqChB;IA9BC,cAAgB;IAEhB,8IAA8I;IAC9I,cADW,WAAW,SAAS,CACF;IAE7B;;;MAGE;IACF,eAHU,OAAO,kBAAkB,EAAE,aAAa,CAGpB;IAE9B;;;MAGE;IACF,kCAAoC;IAiBtC;;;;;OAKG;IACH,mEAiDC;;CAkBF;4BAnI2B,gBAAgB"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@newrelic/browser-agent",
-  "version": "1.261.0",
+  "version": "1.261.2",
   "private": false,
   "author": "New Relic Browser Agent Team <browser-agent@newrelic.com>",
   "description": "New Relic Browser Agent",

package/src/features/session_replay/instrument/index.js

@@ -28,7 +28,7 @@ export class Instrument extends InstrumentBase {
     } catch (err) { }
 
     if (hasReplayPrerequisite(agentIdentifier)) {
-      this.ee.on('recordReplay', () => this.#apiStartOrRestartReplay())
+      this.ee.on(SR_EVENT_EMITTER_TYPES.RECORD, () => this.#apiStartOrRestartReplay())
     }
 
     if (this.#canPreloadRecorder(session)) {

package/src/features/session_replay/shared/utils.js

@@ -12,11 +12,6 @@ export function isPreloadAllowed (agentId) {
   return getConfigurationValue(agentId, 'session_replay.preload') === true && hasReplayPrerequisite(agentId)
 }
 
-export function canImportReplayAgg (agentId, sessionMgr) {
-  if (!hasReplayPrerequisite(agentId)) return false
-  return !!sessionMgr?.isNew || !!sessionMgr?.state.sessionReplayMode // Session Replay should only try to run if already running from a previous page, or at the beginning of a session
-}
-
 export function buildNRMetaNode (timestamp, timeKeeper) {
   const correctedTimestamp = timeKeeper.correctAbsoluteTimestamp(timestamp)
   return {

package/src/features/utils/__mocks__/agent-session.js

@@ -1 +1,3 @@
-export const setupAgentSession = jest.fn()
+export const setupAgentSession = jest.fn(() => {
+  return {}
+})

package/src/features/utils/__mocks__/feature-base.js

@@ -5,7 +5,8 @@ export const FeatureBase = jest.fn(function (agentIdentifier, aggregator, featur
 
   this.ee = {
     abort: jest.fn(),
-    on: jest.fn()
+    on: jest.fn(),
+    emit: jest.fn()
   }
   this.blocked = false
 })

package/src/features/utils/aggregate-base.js

@@ -17,7 +17,7 @@ export class AggregateBase extends FeatureBase {
    * @returns {Promise}
    */
   waitForFlags (flagNames = []) {
-    return new Promise((resolve, reject) => {
+    const flagsPromise = new Promise((resolve, reject) => {
       if (activatedFeatures[this.agentIdentifier]) {
         resolve(buildOutput(activatedFeatures[this.agentIdentifier]))
       } else {
@@ -32,6 +32,9 @@ export class AggregateBase extends FeatureBase {
         })
       }
     })
+    return flagsPromise.catch(err => {
+      this.ee.emit('internal-error', [err])
+    })
   }
 
   drain () {

package/src/features/utils/instrument-base.js

@@ -11,7 +11,7 @@ import { isBrowserScope } from '../../common/constants/runtime'
 import { warn } from '../../common/util/console'
 import { FEATURE_NAMES } from '../../loaders/features/features'
 import { getConfigurationValue } from '../../common/config/config'
-import { canImportReplayAgg } from '../session_replay/shared/utils'
+import { hasReplayPrerequisite } from '../session_replay/shared/utils'
 import { canEnableSessionTracking } from './feature-gates'
 import { single } from '../../common/util/invoke'
 
@@ -86,6 +86,7 @@ export class InstrumentBase extends FeatureBase {
         }
       } catch (e) {
         warn('A problem occurred when starting up session manager. This page will not start or extend any session.', e)
+        this.ee.emit('internal-error', [e])
         if (this.featureName === FEATURE_NAMES.sessionReplay) this.abortHandler?.() // SR should stop recording if session DNE
       }
 
@@ -126,7 +127,13 @@ export class InstrumentBase extends FeatureBase {
  * @returns
  */
   #shouldImportAgg (featureName, session) {
-    if (featureName === FEATURE_NAMES.sessionReplay) return canImportReplayAgg(this.agentIdentifier, session)
-    return true
+    switch (featureName) {
+      case FEATURE_NAMES.sessionReplay: // the session manager must be initialized successfully for Replay & Trace features
+        return hasReplayPrerequisite(this.agentIdentifier) && !!session
+      case FEATURE_NAMES.sessionTrace:
+        return !!session
+      default:
+        return true
+    }
   }
 }

package/src/features/session_replay/shared/__mocks__/utils.js

@@ -1 +0,0 @@
-export const canImportReplayAgg = jest.fn()