@newrelic/browser-agent 1.265.1 → 1.266.0

package/CHANGELOG.md

@@ -3,6 +3,18 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [1.266.0](https://github.com/newrelic/newrelic-browser-agent/compare/v1.265.1...v1.266.0) (2024-09-16)
+
+
+### Features
+
+* Removing long task ([#1153](https://github.com/newrelic/newrelic-browser-agent/issues/1153)) ([304e395](https://github.com/newrelic/newrelic-browser-agent/commit/304e3954df9639d6d61e545fcaa0c761346a5016))
+
+
+### Bug Fixes
+
+* Remove nullish coalescing inside getter ([#1180](https://github.com/newrelic/newrelic-browser-agent/issues/1180)) ([e537359](https://github.com/newrelic/newrelic-browser-agent/commit/e537359f354270fcbd607445b6853c236f0fdbfc))
+
 ## [1.265.1](https://github.com/newrelic/newrelic-browser-agent/compare/v1.265.0...v1.265.1) (2024-09-06)
 
 

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

@@ -87,7 +87,6 @@ const model = () => {
     page_view_timing: {
       enabled: true,
       harvestTimeSeconds: 30,
-      long_task: false,
       autoStart: true
     },
     privacy: {

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

package/dist/cjs/common/vitals/constants.js

@@ -11,6 +11,5 @@ const VITAL_NAMES = exports.VITAL_NAMES = {
   LARGEST_CONTENTFUL_PAINT: 'lcp',
   CUMULATIVE_LAYOUT_SHIFT: 'cls',
   INTERACTION_TO_NEXT_PAINT: 'inp',
-  LONG_TASK: 'lt',
   TIME_TO_FIRST_BYTE: 'ttfb'
 };

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

@@ -70,8 +70,7 @@ class Aggregate extends _aggregateBase.AggregateBase {
     } = (0, _runtime.getRuntime)(this.agentIdentifier);
     const {
       proxy,
-      privacy,
-      page_view_timing
+      privacy
     } = (0, _init.getConfiguration)(this.agentIdentifier);
     if (loaderType) this.storeSupportabilityMetrics("Generic/LoaderType/".concat(loaderType, "/Detected"));
     if (distMethod) this.storeSupportabilityMetrics("Generic/DistMethod/".concat(distMethod, "/Detected"));
@@ -89,7 +88,6 @@ class Aggregate extends _aggregateBase.AggregateBase {
         });
       });
       if (!privacy.cookies_enabled) this.storeSupportabilityMetrics('Config/SessionTracking/Disabled');
-      if (page_view_timing.long_task) this.storeSupportabilityMetrics('Config/LongTask/Enabled');
     } else if (_runtime2.isWorkerScope) {
       this.storeSupportabilityMetrics('Generic/Runtime/Worker/Detected');
     } else {

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

@@ -20,7 +20,6 @@ var _firstPaint = require("../../../common/vitals/first-paint");
 var _interactionToNextPaint = require("../../../common/vitals/interaction-to-next-paint");
 var _largestContentfulPaint = require("../../../common/vitals/largest-contentful-paint");
 var _timeToFirstByte = require("../../../common/vitals/time-to-first-byte");
-var _longTask = require("../../../common/vitals/long-task");
 var _pageVisibility = require("../../../common/window/page-visibility");
 var _constants2 = require("../../../common/vitals/constants");
 var _eventBuffer = require("../../utils/event-buffer");
@@ -42,7 +41,6 @@ class Aggregate extends _aggregateBase.AggregateBase {
     super(agentIdentifier, aggregator, _constants.FEATURE_NAME);
     this.timings = new _eventBuffer.EventBuffer();
     this.curSessEndRecorded = false;
-    if ((0, _init.getConfigurationValue)(this.agentIdentifier, 'page_view_timing.long_task') === true) _longTask.longTask.subscribe(this.#handleVitalMetric);
     (0, _registerHandler.registerHandler)('docHidden', msTimestamp => this.endCurrentSession(msTimestamp), this.featureName, this.ee);
     (0, _registerHandler.registerHandler)('winPagehide', msTimestamp => this.recordPageUnload(msTimestamp), this.featureName, this.ee);
     const harvestTimeSeconds = (0, _init.getConfigurationValue)(this.agentIdentifier, 'page_view_timing.harvestTimeSeconds') || 30;

package/dist/cjs/features/utils/event-buffer.js

@@ -50,7 +50,7 @@ class EventBuffer {
    * held is another event buffer
    */
   get held() {
-    this.#held ??= new EventBuffer(this.maxPayloadSize);
+    if (!this.#held) this.#held = new EventBuffer(this.maxPayloadSize);
     return this.#held;
   }
 

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

@@ -79,7 +79,6 @@ const model = () => {
     page_view_timing: {
       enabled: true,
       harvestTimeSeconds: 30,
-      long_task: false,
       autoStart: true
     },
     privacy: {

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

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

package/dist/esm/common/vitals/constants.js

@@ -5,6 +5,5 @@ export const VITAL_NAMES = {
   LARGEST_CONTENTFUL_PAINT: 'lcp',
   CUMULATIVE_LAYOUT_SHIFT: 'cls',
   INTERACTION_TO_NEXT_PAINT: 'inp',
-  LONG_TASK: 'lt',
   TIME_TO_FIRST_BYTE: 'ttfb'
 };

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

@@ -64,8 +64,7 @@ export class Aggregate extends AggregateBase {
     } = getRuntime(this.agentIdentifier);
     const {
       proxy,
-      privacy,
-      page_view_timing
+      privacy
     } = getConfiguration(this.agentIdentifier);
     if (loaderType) this.storeSupportabilityMetrics("Generic/LoaderType/".concat(loaderType, "/Detected"));
     if (distMethod) this.storeSupportabilityMetrics("Generic/DistMethod/".concat(distMethod, "/Detected"));
@@ -83,7 +82,6 @@ export class Aggregate extends AggregateBase {
         });
       });
       if (!privacy.cookies_enabled) this.storeSupportabilityMetrics('Config/SessionTracking/Disabled');
-      if (page_view_timing.long_task) this.storeSupportabilityMetrics('Config/LongTask/Enabled');
     } else if (isWorkerScope) {
       this.storeSupportabilityMetrics('Generic/Runtime/Worker/Detected');
     } else {

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

@@ -19,7 +19,6 @@ import { firstPaint } from '../../../common/vitals/first-paint';
 import { interactionToNextPaint } from '../../../common/vitals/interaction-to-next-paint';
 import { largestContentfulPaint } from '../../../common/vitals/largest-contentful-paint';
 import { timeToFirstByte } from '../../../common/vitals/time-to-first-byte';
-import { longTask } from '../../../common/vitals/long-task';
 import { subscribeToVisibilityChange } from '../../../common/window/page-visibility';
 import { VITAL_NAMES } from '../../../common/vitals/constants';
 import { EventBuffer } from '../../utils/event-buffer';
@@ -36,7 +35,6 @@ export class Aggregate extends AggregateBase {
     super(agentIdentifier, aggregator, FEATURE_NAME);
     this.timings = new EventBuffer();
     this.curSessEndRecorded = false;
-    if (getConfigurationValue(this.agentIdentifier, 'page_view_timing.long_task') === true) longTask.subscribe(this.#handleVitalMetric);
     registerHandler('docHidden', msTimestamp => this.endCurrentSession(msTimestamp), this.featureName, this.ee);
     registerHandler('winPagehide', msTimestamp => this.recordPageUnload(msTimestamp), this.featureName, this.ee);
     const harvestTimeSeconds = getConfigurationValue(this.agentIdentifier, 'page_view_timing.harvestTimeSeconds') || 30;

package/dist/esm/features/utils/event-buffer.js

@@ -45,7 +45,7 @@ export class EventBuffer {
    * held is another event buffer
    */
   get held() {
-    this.#held ??= new EventBuffer(this.maxPayloadSize);
+    if (!this.#held) this.#held = new EventBuffer(this.maxPayloadSize);
     return this.#held;
   }
 

package/dist/types/common/vitals/constants.d.ts

@@ -5,7 +5,6 @@ export namespace VITAL_NAMES {
     let LARGEST_CONTENTFUL_PAINT: string;
     let CUMULATIVE_LAYOUT_SHIFT: string;
     let INTERACTION_TO_NEXT_PAINT: string;
-    let LONG_TASK: string;
     let TIME_TO_FIRST_BYTE: string;
 }
 //# sourceMappingURL=constants.d.ts.map

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

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../src/features/metrics/aggregate/index.js"],"names":[],"mappings":"AAeA;IACE,2BAAiC;IACjC,mDAsBC;IAED,wDAKC;IAED,iDAKC;IAED,qBAsEC;IAED,0BAOC;IAED,eAkCC;IA/BG,mCAAyB;CAgC9B;8BAjK6B,4BAA4B"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../src/features/metrics/aggregate/index.js"],"names":[],"mappings":"AAeA;IACE,2BAAiC;IACjC,mDAsBC;IAED,wDAKC;IAED,iDAKC;IAED,qBAqEC;IAED,0BAOC;IAED,eAkCC;IA/BG,mCAAyB;CAgC9B;8BAhK6B,4BAA4B"}

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

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../src/features/page_view_timing/aggregate/index.js"],"names":[],"mappings":"AA0BA;IACE,2BAAiC;IAMjC,mDAyCC;IAtCC,qBAAgC;IAChC,4BAA+B;IAuCjC;;;OAGG;IACH,6BAFW,MAAM,QAOhB;IAED;;OAEG;IACH,uCAUC;IAED,mDAuBC;IAED,qCAGC;IAED,gDAWC;IAGD;;;;kBAUC;IAGD,8BAuBC;;CACF;8BA1K6B,4BAA4B;4BAW9B,0BAA0B"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../src/features/page_view_timing/aggregate/index.js"],"names":[],"mappings":"AAyBA;IACE,2BAAiC;IAMjC,mDAuCC;IApCC,qBAAgC;IAChC,4BAA+B;IAqCjC;;;OAGG;IACH,6BAFW,MAAM,QAOhB;IAED;;OAEG;IACH,uCAUC;IAED,mDAuBC;IAED,qCAGC;IAED,gDAWC;IAGD;;;;kBAUC;IAGD,8BAuBC;;CACF;8BAvK6B,4BAA4B;4BAU9B,0BAA0B"}

package/package.json

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

package/src/common/config/init.js

@@ -49,7 +49,7 @@ const model = () => {
     obfuscate: undefined,
     page_action: { enabled: true },
     page_view_event: { enabled: true, autoStart: true },
-    page_view_timing: { enabled: true, harvestTimeSeconds: 30, long_task: false, autoStart: true },
+    page_view_timing: { enabled: true, harvestTimeSeconds: 30, autoStart: true },
     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

package/src/common/vitals/constants.js

@@ -5,6 +5,5 @@ export const VITAL_NAMES = {
   LARGEST_CONTENTFUL_PAINT: 'lcp',
   CUMULATIVE_LAYOUT_SHIFT: 'cls',
   INTERACTION_TO_NEXT_PAINT: 'inp',
-  LONG_TASK: 'lt',
   TIME_TO_FIRST_BYTE: 'ttfb'
 }

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

@@ -56,7 +56,7 @@ export class Aggregate extends AggregateBase {
   singleChecks () {
     // report loaderType
     const { distMethod, loaderType } = getRuntime(this.agentIdentifier)
-    const { proxy, privacy, page_view_timing } = getConfiguration(this.agentIdentifier)
+    const { proxy, privacy } = getConfiguration(this.agentIdentifier)
 
     if (loaderType) this.storeSupportabilityMetrics(`Generic/LoaderType/${loaderType}/Detected`)
     if (distMethod) this.storeSupportabilityMetrics(`Generic/DistMethod/${distMethod}/Detected`)
@@ -77,7 +77,6 @@ export class Aggregate extends AggregateBase {
       })
 
       if (!privacy.cookies_enabled) this.storeSupportabilityMetrics('Config/SessionTracking/Disabled')
-      if (page_view_timing.long_task) this.storeSupportabilityMetrics('Config/LongTask/Enabled')
     } else if (isWorkerScope) {
       this.storeSupportabilityMetrics('Generic/Runtime/Worker/Detected')
     } else {

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

@@ -19,7 +19,6 @@ import { firstPaint } from '../../../common/vitals/first-paint'
 import { interactionToNextPaint } from '../../../common/vitals/interaction-to-next-paint'
 import { largestContentfulPaint } from '../../../common/vitals/largest-contentful-paint'
 import { timeToFirstByte } from '../../../common/vitals/time-to-first-byte'
-import { longTask } from '../../../common/vitals/long-task'
 import { subscribeToVisibilityChange } from '../../../common/window/page-visibility'
 import { VITAL_NAMES } from '../../../common/vitals/constants'
 import { EventBuffer } from '../../utils/event-buffer'
@@ -37,8 +36,6 @@ export class Aggregate extends AggregateBase {
     this.timings = new EventBuffer()
     this.curSessEndRecorded = false
 
-    if (getConfigurationValue(this.agentIdentifier, 'page_view_timing.long_task') === true) longTask.subscribe(this.#handleVitalMetric)
-
     registerHandler('docHidden', msTimestamp => this.endCurrentSession(msTimestamp), this.featureName, this.ee)
     registerHandler('winPagehide', msTimestamp => this.recordPageUnload(msTimestamp), this.featureName, this.ee)
 

package/src/features/utils/event-buffer.js

@@ -45,7 +45,7 @@ export class EventBuffer {
    * held is another event buffer
    */
   get held () {
-    this.#held ??= new EventBuffer(this.maxPayloadSize)
+    if (!this.#held) this.#held = new EventBuffer(this.maxPayloadSize)
     return this.#held
   }
 

package/dist/cjs/common/vitals/long-task.js

@@ -1,61 +0,0 @@
-"use strict";
-
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.longTask = void 0;
-var _runtime = require("../constants/runtime");
-var _eol = require("../unload/eol");
-var _constants = require("./constants");
-var _vitalMetric = require("./vital-metric");
-const longTask = exports.longTask = new _vitalMetric.VitalMetric(_constants.VITAL_NAMES.LONG_TASK);
-if (_runtime.isBrowserScope) {
-  const handleEntries = entries => {
-    entries.forEach(entry => {
-      longTask.update({
-        value: entry.duration,
-        attrs: {
-          ltFrame: entry.name,
-          // MDN: the browsing context or frame that can be attributed to the long task
-          ltStart: entry.startTime,
-          // MDN: a double representing the time (millisec) when the task started
-          ltCtr: entry.attribution[0].containerType,
-          // MDN: type of frame container: 'iframe', 'embed', or 'object' ... but this can also be 'window',
-          ...(entry.attribution[0].containerType !== 'window' && {
-            ltCtrSrc: entry.attribution[0].containerSrc,
-            // MDN: container's 'src' attribute
-            ltCtrId: entry.attribution[0].containerId,
-            // MDN: container's 'id' attribute
-            ltCtrName: entry.attribution[0].containerName // MDN: container's 'name' attribute
-          })
-        }
-      });
-    });
-  };
-  let observer;
-  try {
-    if (PerformanceObserver.supportedEntryTypes.includes('longtask')) {
-      observer = new PerformanceObserver(list => {
-        // Delay by a microtask to workaround a bug in Safari where the
-        // callback is invoked immediately, rather than in a separate task.
-        // See: https://github.com/GoogleChrome/web-vitals/issues/277
-        Promise.resolve().then(() => {
-          handleEntries(list.getEntries());
-        });
-      });
-      observer.observe({
-        type: 'longtask',
-        buffered: true
-      });
-    }
-  } catch (e) {
-    // Do nothing.
-  }
-  if (observer) {
-    (0, _eol.subscribeToEOL)(() => {
-      handleEntries(observer.takeRecords());
-    }, true); // this bool is a temp arg under staged BFCache work that runs the func under the new page session logic -- tb removed w/ the feature flag later
-
-    /* No work needed on BFCache restore for long task. */
-  }
-}

package/dist/esm/common/vitals/long-task.js

@@ -1,55 +0,0 @@
-import { isBrowserScope } from '../constants/runtime';
-import { subscribeToEOL } from '../unload/eol';
-import { VITAL_NAMES } from './constants';
-import { VitalMetric } from './vital-metric';
-export const longTask = new VitalMetric(VITAL_NAMES.LONG_TASK);
-if (isBrowserScope) {
-  const handleEntries = entries => {
-    entries.forEach(entry => {
-      longTask.update({
-        value: entry.duration,
-        attrs: {
-          ltFrame: entry.name,
-          // MDN: the browsing context or frame that can be attributed to the long task
-          ltStart: entry.startTime,
-          // MDN: a double representing the time (millisec) when the task started
-          ltCtr: entry.attribution[0].containerType,
-          // MDN: type of frame container: 'iframe', 'embed', or 'object' ... but this can also be 'window',
-          ...(entry.attribution[0].containerType !== 'window' && {
-            ltCtrSrc: entry.attribution[0].containerSrc,
-            // MDN: container's 'src' attribute
-            ltCtrId: entry.attribution[0].containerId,
-            // MDN: container's 'id' attribute
-            ltCtrName: entry.attribution[0].containerName // MDN: container's 'name' attribute
-          })
-        }
-      });
-    });
-  };
-  let observer;
-  try {
-    if (PerformanceObserver.supportedEntryTypes.includes('longtask')) {
-      observer = new PerformanceObserver(list => {
-        // Delay by a microtask to workaround a bug in Safari where the
-        // callback is invoked immediately, rather than in a separate task.
-        // See: https://github.com/GoogleChrome/web-vitals/issues/277
-        Promise.resolve().then(() => {
-          handleEntries(list.getEntries());
-        });
-      });
-      observer.observe({
-        type: 'longtask',
-        buffered: true
-      });
-    }
-  } catch (e) {
-    // Do nothing.
-  }
-  if (observer) {
-    subscribeToEOL(() => {
-      handleEntries(observer.takeRecords());
-    }, true); // this bool is a temp arg under staged BFCache work that runs the func under the new page session logic -- tb removed w/ the feature flag later
-
-    /* No work needed on BFCache restore for long task. */
-  }
-}

package/dist/types/common/vitals/long-task.d.ts

@@ -1,3 +0,0 @@
-export const longTask: VitalMetric;
-import { VitalMetric } from './vital-metric';
-//# sourceMappingURL=long-task.d.ts.map

package/dist/types/common/vitals/long-task.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"long-task.d.ts","sourceRoot":"","sources":["../../../../src/common/vitals/long-task.js"],"names":[],"mappings":"AAKA,mCAA8D;4BAFlC,gBAAgB"}

package/src/common/vitals/long-task.js

@@ -1,51 +0,0 @@
-import { isBrowserScope } from '../constants/runtime'
-import { subscribeToEOL } from '../unload/eol'
-import { VITAL_NAMES } from './constants'
-import { VitalMetric } from './vital-metric'
-
-export const longTask = new VitalMetric(VITAL_NAMES.LONG_TASK)
-
-if (isBrowserScope) {
-  const handleEntries = (entries) => {
-    entries.forEach(entry => {
-      longTask.update({
-        value: entry.duration,
-        attrs: {
-          ltFrame: entry.name, // MDN: the browsing context or frame that can be attributed to the long task
-          ltStart: entry.startTime, // MDN: a double representing the time (millisec) when the task started
-          ltCtr: entry.attribution[0].containerType, // MDN: type of frame container: 'iframe', 'embed', or 'object' ... but this can also be 'window',
-          ...(entry.attribution[0].containerType !== 'window' && {
-            ltCtrSrc: entry.attribution[0].containerSrc, // MDN: container's 'src' attribute
-            ltCtrId: entry.attribution[0].containerId, // MDN: container's 'id' attribute
-            ltCtrName: entry.attribution[0].containerName // MDN: container's 'name' attribute
-          })
-        }
-      })
-    })
-  }
-
-  let observer
-  try {
-    if (PerformanceObserver.supportedEntryTypes.includes('longtask')) {
-      observer = new PerformanceObserver((list) => {
-      // Delay by a microtask to workaround a bug in Safari where the
-      // callback is invoked immediately, rather than in a separate task.
-      // See: https://github.com/GoogleChrome/web-vitals/issues/277
-        Promise.resolve().then(() => {
-          handleEntries(list.getEntries())
-        })
-      })
-      observer.observe({ type: 'longtask', buffered: true })
-    }
-  } catch (e) {
-  // Do nothing.
-  }
-
-  if (observer) {
-    subscribeToEOL(() => {
-      handleEntries(observer.takeRecords())
-    }, true) // this bool is a temp arg under staged BFCache work that runs the func under the new page session logic -- tb removed w/ the feature flag later
-
-  /* No work needed on BFCache restore for long task. */
-  }
-}