@newrelic/browser-agent 1.270.3 → 1.272.0

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.272.0](https://github.com/newrelic/newrelic-browser-agent/compare/v1.271.0...v1.272.0) (2024-11-07)
+
+
+### Features
+
+* Marks and measures ([#1189](https://github.com/newrelic/newrelic-browser-agent/issues/1189)) ([1a58409](https://github.com/newrelic/newrelic-browser-agent/commit/1a58409cda8b7b264e58fe284041540941dccd1c))
+
+## [1.271.0](https://github.com/newrelic/newrelic-browser-agent/compare/v1.270.3...v1.271.0) (2024-11-01)
+
+
+### Features
+
+* Ignore unhandled promise rejections that lack a valid reason ([#1233](https://github.com/newrelic/newrelic-browser-agent/issues/1233)) ([25a1fff](https://github.com/newrelic/newrelic-browser-agent/commit/25a1fffb91fa5936766a7bc89d735b3018aa62a2))
+
 ## [1.270.3](https://github.com/newrelic/newrelic-browser-agent/compare/v1.270.2...v1.270.3) (2024-10-31)
 
 

package/dist/cjs/common/config/init.js

@@ -80,9 +80,6 @@ const model = () => {
     page_action: {
       enabled: true
     },
-    user_actions: {
-      enabled: true
-    },
     page_view_event: {
       enabled: true,
       autoStart: true
@@ -92,6 +89,10 @@ const model = () => {
       harvestTimeSeconds: 30,
       autoStart: true
     },
+    performance: {
+      capture_marks: false,
+      capture_measures: false // false by default through experimental phase, but flipped to true once GA'd
+    },
     privacy: {
       cookies_enabled: true
     },
@@ -175,7 +176,10 @@ const model = () => {
       harvestTimeSeconds: 10,
       autoStart: true
     },
-    ssl: undefined
+    ssl: undefined,
+    user_actions: {
+      enabled: true
+    }
   };
 };
 const _cache = {};

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

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

@@ -107,6 +107,45 @@ class Aggregate extends _aggregateBase.AggregateBase {
           if (options.isFinalHarvest) this.addUserAction(this.userActionAggregator.aggregationEvent);
         });
       }
+
+      /**
+       * is it worth complicating the agent and skipping the POs for single repeating queries? maybe,
+       * but right now it was less desirable simply because it is a nice benefit of populating the event buffer
+       * immediately as events happen for payload evaluation purposes and that becomes a little more chaotic
+       * with an arbitrary query method. note: eventTypes: [...types] does not support the 'buffered' flag so we have
+       * to create up to two PO's here.
+       */
+      const performanceTypesToCapture = [...(agentRef.init.performance.capture_marks ? ['mark'] : []), ...(agentRef.init.performance.capture_measures ? ['measure'] : [])];
+      if (performanceTypesToCapture.length) {
+        try {
+          performanceTypesToCapture.forEach(type => {
+            if (PerformanceObserver.supportedEntryTypes.includes(type)) {
+              const observer = new PerformanceObserver(list => {
+                list.getEntries().forEach(entry => {
+                  try {
+                    this.addEvent({
+                      eventType: 'BrowserPerformance',
+                      timestamp: Math.floor(agentRef.runtime.timeKeeper.correctRelativeTimestamp(entry.startTime)),
+                      entryName: entry.name,
+                      entryDuration: entry.duration,
+                      entryType: type,
+                      ...(entry.detail && {
+                        entryDetail: entry.detail
+                      })
+                    });
+                  } catch (err) {}
+                });
+              });
+              observer.observe({
+                buffered: true,
+                type
+              });
+            }
+          });
+        } catch (err) {
+          // Something failed in our set up, likely the browser does not support PO's... do nothing
+        }
+      }
       this.harvestScheduler = new _harvestScheduler.HarvestScheduler('ins', {
         onFinished: (...args) => this.onHarvestFinished(...args)
       }, this);

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

@@ -17,7 +17,7 @@ class Instrument extends _instrumentBase.InstrumentBase {
   static featureName = _constants.FEATURE_NAME;
   constructor(agentRef, auto = true) {
     super(agentRef, _constants.FEATURE_NAME, auto);
-    const genericEventSourceConfigs = [agentRef.init.page_action.enabled, agentRef.init.user_actions.enabled
+    const genericEventSourceConfigs = [agentRef.init.page_action.enabled, agentRef.init.performance.capture_marks, agentRef.init.performance.capture_measures, agentRef.init.user_actions.enabled
     // other future generic event source configs to go here, like M&Ms, PageResouce, etc.
     ];
     if (_runtime.isBrowserScope && agentRef.init.user_actions.enabled) {

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

@@ -32,8 +32,15 @@ function castError(error) {
    * @returns {Error} An Error object with the message as the casted reason
    */
 function castPromiseRejectionEvent(promiseRejectionEvent) {
-  let prefix = 'Unhandled Promise Rejection';
-  if (canTrustError(promiseRejectionEvent?.reason)) {
+  const prefix = 'Unhandled Promise Rejection';
+
+  /**
+   * If the casted return value is falsy like this, it will get dropped and not produce an error event for harvest.
+   * We drop promise rejections that could not form a valid error stack or message deriving from the .reason attribute
+   * -- such as a manually invoked rejection without an argument -- since they lack reproduction value and create confusion.
+   * */
+  if (!promiseRejectionEvent?.reason) return;
+  if (canTrustError(promiseRejectionEvent.reason)) {
     try {
       promiseRejectionEvent.reason.message = prefix + ': ' + promiseRejectionEvent.reason.message;
       return castError(promiseRejectionEvent.reason);
@@ -41,7 +48,6 @@ function castPromiseRejectionEvent(promiseRejectionEvent) {
       return castError(promiseRejectionEvent.reason);
     }
   }
-  if (typeof promiseRejectionEvent.reason === 'undefined') return castError(prefix);
   const error = castError(promiseRejectionEvent.reason);
   error.message = prefix + ': ' + error?.message;
   return error;

package/dist/esm/common/config/init.js

@@ -72,9 +72,6 @@ const model = () => {
     page_action: {
       enabled: true
     },
-    user_actions: {
-      enabled: true
-    },
     page_view_event: {
       enabled: true,
       autoStart: true
@@ -84,6 +81,10 @@ const model = () => {
       harvestTimeSeconds: 30,
       autoStart: true
     },
+    performance: {
+      capture_marks: false,
+      capture_measures: false // false by default through experimental phase, but flipped to true once GA'd
+    },
     privacy: {
       cookies_enabled: true
     },
@@ -167,7 +168,10 @@ const model = () => {
       harvestTimeSeconds: 10,
       autoStart: true
     },
-    ssl: undefined
+    ssl: undefined,
+    user_actions: {
+      enabled: true
+    }
   };
 };
 const _cache = {};

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

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

package/dist/esm/features/generic_events/aggregate/index.js

@@ -100,6 +100,45 @@ export class Aggregate extends AggregateBase {
           if (options.isFinalHarvest) this.addUserAction(this.userActionAggregator.aggregationEvent);
         });
       }
+
+      /**
+       * is it worth complicating the agent and skipping the POs for single repeating queries? maybe,
+       * but right now it was less desirable simply because it is a nice benefit of populating the event buffer
+       * immediately as events happen for payload evaluation purposes and that becomes a little more chaotic
+       * with an arbitrary query method. note: eventTypes: [...types] does not support the 'buffered' flag so we have
+       * to create up to two PO's here.
+       */
+      const performanceTypesToCapture = [...(agentRef.init.performance.capture_marks ? ['mark'] : []), ...(agentRef.init.performance.capture_measures ? ['measure'] : [])];
+      if (performanceTypesToCapture.length) {
+        try {
+          performanceTypesToCapture.forEach(type => {
+            if (PerformanceObserver.supportedEntryTypes.includes(type)) {
+              const observer = new PerformanceObserver(list => {
+                list.getEntries().forEach(entry => {
+                  try {
+                    this.addEvent({
+                      eventType: 'BrowserPerformance',
+                      timestamp: Math.floor(agentRef.runtime.timeKeeper.correctRelativeTimestamp(entry.startTime)),
+                      entryName: entry.name,
+                      entryDuration: entry.duration,
+                      entryType: type,
+                      ...(entry.detail && {
+                        entryDetail: entry.detail
+                      })
+                    });
+                  } catch (err) {}
+                });
+              });
+              observer.observe({
+                buffered: true,
+                type
+              });
+            }
+          });
+        } catch (err) {
+          // Something failed in our set up, likely the browser does not support PO's... do nothing
+        }
+      }
       this.harvestScheduler = new HarvestScheduler('ins', {
         onFinished: (...args) => this.onHarvestFinished(...args)
       }, this);

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

@@ -11,7 +11,7 @@ export class Instrument extends InstrumentBase {
   static featureName = FEATURE_NAME;
   constructor(agentRef, auto = true) {
     super(agentRef, FEATURE_NAME, auto);
-    const genericEventSourceConfigs = [agentRef.init.page_action.enabled, agentRef.init.user_actions.enabled
+    const genericEventSourceConfigs = [agentRef.init.page_action.enabled, agentRef.init.performance.capture_marks, agentRef.init.performance.capture_measures, agentRef.init.user_actions.enabled
     // other future generic event source configs to go here, like M&Ms, PageResouce, etc.
     ];
     if (isBrowserScope && agentRef.init.user_actions.enabled) {

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

@@ -25,8 +25,15 @@ export function castError(error) {
    * @returns {Error} An Error object with the message as the casted reason
    */
 export function castPromiseRejectionEvent(promiseRejectionEvent) {
-  let prefix = 'Unhandled Promise Rejection';
-  if (canTrustError(promiseRejectionEvent?.reason)) {
+  const prefix = 'Unhandled Promise Rejection';
+
+  /**
+   * If the casted return value is falsy like this, it will get dropped and not produce an error event for harvest.
+   * We drop promise rejections that could not form a valid error stack or message deriving from the .reason attribute
+   * -- such as a manually invoked rejection without an argument -- since they lack reproduction value and create confusion.
+   * */
+  if (!promiseRejectionEvent?.reason) return;
+  if (canTrustError(promiseRejectionEvent.reason)) {
     try {
       promiseRejectionEvent.reason.message = prefix + ': ' + promiseRejectionEvent.reason.message;
       return castError(promiseRejectionEvent.reason);
@@ -34,7 +41,6 @@ export function castPromiseRejectionEvent(promiseRejectionEvent) {
       return castError(promiseRejectionEvent.reason);
     }
   }
-  if (typeof promiseRejectionEvent.reason === 'undefined') return castError(prefix);
   const error = castError(promiseRejectionEvent.reason);
   error.message = prefix + ': ' + error?.message;
   return error;

package/dist/types/common/config/init.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"init.d.ts","sourceRoot":"","sources":["../../../../src/common/config/init.js"],"names":[],"mappings":"AAkHA,+CAIC;AAED,0DAKC;AAED,+DAYC"}
+{"version":3,"file":"init.d.ts","sourceRoot":"","sources":["../../../../src/common/config/init.js"],"names":[],"mappings":"AAsHA,+CAIC;AAED,0DAKC;AAED,+DAYC"}

package/dist/types/features/generic_events/aggregate/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../src/features/generic_events/aggregate/index.js"],"names":[],"mappings":"AAoBA;IACE,2BAAiC;IACjC,2BAsFC;IAnFC,yBAA4B;IAC5B,wBAAyE;IAEzE,gCAAkG;IAClG,oBAA+B;IA6B3B,4CAAuD;IAEvD,mDAyBC;IAcH,mCAAuH;IAY3H;;;;;;;;;;;OAWG;IACH,eAHW,MAAM,YAAC,QAmCjB;IAED;;;;;;kBAkBC;IAED,qCAGC;IAED,yBAMC;CACF;8BApL6B,4BAA4B;4BAK9B,0BAA0B;sCAGhB,wCAAwC;iCAZ7C,2CAA2C"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../src/features/generic_events/aggregate/index.js"],"names":[],"mappings":"AAoBA;IACE,2BAAiC;IACjC,2BAyHC;IAtHC,yBAA4B;IAC5B,wBAAyE;IAEzE,gCAAkG;IAClG,oBAA+B;IA6B3B,4CAAuD;IAEvD,mDAyBC;IAiDH,mCAAuH;IAY3H;;;;;;;;;;;OAWG;IACH,eAHW,MAAM,YAAC,QAmCjB;IAED;;;;;;kBAkBC;IAED,qCAGC;IAED,yBAMC;CACF;8BAvN6B,4BAA4B;4BAK9B,0BAA0B;sCAGhB,wCAAwC;iCAZ7C,2CAA2C"}

package/dist/types/features/generic_events/instrument/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../src/features/generic_events/instrument/index.js"],"names":[],"mappings":"AAUA;IACE,2BAAiC;IACjC,2CAqBC;CACF;AAED,8CAAuC;+BA7BR,6BAA6B"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../src/features/generic_events/instrument/index.js"],"names":[],"mappings":"AAUA;IACE,2BAAiC;IACjC,2CAuBC;CACF;AAED,8CAAuC;+BA/BR,6BAA6B"}

package/dist/types/features/jserrors/shared/cast-error.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"cast-error.d.ts","sourceRoot":"","sources":["../../../../../src/features/jserrors/shared/cast-error.js"],"names":[],"mappings":"AAEA;;;;;KAKK;AACL,iCAHa,GAAG,GACD,KAAK,GAAC,aAAa,CAmBjC;AAED;;;;KAIK;AACL,uEAFe,KAAK,CAmBnB;AAED;;;;KAIK;AACL,2CAHa,UAAU,GACR,KAAK,GAAC,aAAa,CAUjC;8BAhE6B,kBAAkB"}
+{"version":3,"file":"cast-error.d.ts","sourceRoot":"","sources":["../../../../../src/features/jserrors/shared/cast-error.js"],"names":[],"mappings":"AAEA;;;;;KAKK;AACL,iCAHa,GAAG,GACD,KAAK,GAAC,aAAa,CAmBjC;AAED;;;;KAIK;AACL,uEAFe,KAAK,CAwBnB;AAED;;;;KAIK;AACL,2CAHa,UAAU,GACR,KAAK,GAAC,aAAa,CAUjC;8BArE6B,kBAAkB"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@newrelic/browser-agent",
-  "version": "1.270.3",
+  "version": "1.272.0",
   "private": false,
   "author": "New Relic Browser Agent Team <browser-agent@newrelic.com>",
   "description": "New Relic Browser Agent",
@@ -201,7 +201,8 @@
   "devDependencies": {
     "@babel/cli": "^7.23.4",
     "@babel/core": "^7.23.7",
-    "@babel/eslint-parser": "^7.23.3",
+    "@babel/eslint-parser": "^7.25.9",
+    "@babel/plugin-syntax-import-attributes": "^7.26.0",
     "@babel/plugin-transform-template-literals": "^7.24.7",
     "@babel/preset-env": "^7.23.8",
     "@babel/register": "^7.23.7",

package/src/common/config/init.js

@@ -48,9 +48,12 @@ const model = () => {
     metrics: { enabled: true, autoStart: true },
     obfuscate: undefined,
     page_action: { enabled: true },
-    user_actions: { enabled: true },
     page_view_event: { enabled: true, autoStart: true },
     page_view_timing: { enabled: true, harvestTimeSeconds: 30, autoStart: true },
+    performance: {
+      capture_marks: false,
+      capture_measures: false // false by default through experimental phase, but flipped to true once GA'd
+    },
     privacy: { cookies_enabled: true }, // *cli - per discussion, default should be true
     proxy: {
       assets: undefined, // if this value is set, it will be used to overwrite the webpack asset path used to fetch assets
@@ -105,7 +108,8 @@ const model = () => {
     session_trace: { enabled: true, harvestTimeSeconds: 10, autoStart: true },
     soft_navigations: { enabled: true, harvestTimeSeconds: 10, autoStart: true },
     spa: { enabled: true, harvestTimeSeconds: 10, autoStart: true },
-    ssl: undefined
+    ssl: undefined,
+    user_actions: { enabled: true }
   }
 }
 

package/src/features/generic_events/aggregate/index.js

@@ -97,6 +97,41 @@ export class Aggregate extends AggregateBase {
         })
       }
 
+      /**
+       * is it worth complicating the agent and skipping the POs for single repeating queries? maybe,
+       * but right now it was less desirable simply because it is a nice benefit of populating the event buffer
+       * immediately as events happen for payload evaluation purposes and that becomes a little more chaotic
+       * with an arbitrary query method. note: eventTypes: [...types] does not support the 'buffered' flag so we have
+       * to create up to two PO's here.
+       */
+      const performanceTypesToCapture = [...(agentRef.init.performance.capture_marks ? ['mark'] : []), ...(agentRef.init.performance.capture_measures ? ['measure'] : [])]
+      if (performanceTypesToCapture.length) {
+        try {
+          performanceTypesToCapture.forEach(type => {
+            if (PerformanceObserver.supportedEntryTypes.includes(type)) {
+              const observer = new PerformanceObserver((list) => {
+                list.getEntries().forEach(entry => {
+                  try {
+                    this.addEvent({
+                      eventType: 'BrowserPerformance',
+                      timestamp: Math.floor(agentRef.runtime.timeKeeper.correctRelativeTimestamp(entry.startTime)),
+                      entryName: entry.name,
+                      entryDuration: entry.duration,
+                      entryType: type,
+                      ...(entry.detail && { entryDetail: entry.detail })
+                    })
+                  } catch (err) {
+                  }
+                })
+              })
+              observer.observe({ buffered: true, type })
+            }
+          })
+        } catch (err) {
+        // Something failed in our set up, likely the browser does not support PO's... do nothing
+        }
+      }
+
       this.harvestScheduler = new HarvestScheduler('ins', { onFinished: (...args) => this.onHarvestFinished(...args) }, this)
       this.harvestScheduler.harvest.on('ins', (...args) => {
         preHarvestMethods.forEach(fn => fn(...args))

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

@@ -14,6 +14,8 @@ export class Instrument extends InstrumentBase {
     super(agentRef, FEATURE_NAME, auto)
     const genericEventSourceConfigs = [
       agentRef.init.page_action.enabled,
+      agentRef.init.performance.capture_marks,
+      agentRef.init.performance.capture_measures,
       agentRef.init.user_actions.enabled
       // other future generic event source configs to go here, like M&Ms, PageResouce, etc.
     ]

package/src/features/jserrors/shared/cast-error.js

@@ -31,9 +31,16 @@ export function castError (error) {
    * @returns {Error} An Error object with the message as the casted reason
    */
 export function castPromiseRejectionEvent (promiseRejectionEvent) {
-  let prefix = 'Unhandled Promise Rejection'
+  const prefix = 'Unhandled Promise Rejection'
 
-  if (canTrustError(promiseRejectionEvent?.reason)) {
+  /**
+   * If the casted return value is falsy like this, it will get dropped and not produce an error event for harvest.
+   * We drop promise rejections that could not form a valid error stack or message deriving from the .reason attribute
+   * -- such as a manually invoked rejection without an argument -- since they lack reproduction value and create confusion.
+   * */
+  if (!promiseRejectionEvent?.reason) return
+
+  if (canTrustError(promiseRejectionEvent.reason)) {
     try {
       promiseRejectionEvent.reason.message = prefix + ': ' + promiseRejectionEvent.reason.message
       return castError(promiseRejectionEvent.reason)
@@ -42,8 +49,6 @@ export function castPromiseRejectionEvent (promiseRejectionEvent) {
     }
   }
 
-  if (typeof promiseRejectionEvent.reason === 'undefined') return castError(prefix)
-
   const error = castError(promiseRejectionEvent.reason)
   error.message = prefix + ': ' + error?.message
   return error