@newrelic/browser-agent 0.1.230 → 0.1.231

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

@@ -89,12 +89,21 @@ export class Aggregate extends AggregateBase {
     chunksForQueryString.push(param('pr', info.product))
     chunksForQueryString.push(param('af', getActivatedFeaturesFlags(this.agentIdentifier).join(',')))
 
-    if (globalScope.performance && typeof (globalScope.performance.timing) !== 'undefined') {
-      var navTimingApiData = ({
-        timing: addPT(agentRuntime.offset, globalScope.performance.timing, {}),
-        navigation: addPN(globalScope.performance.navigation, {})
-      })
-      chunksForQueryString.push(param('perf', stringify(navTimingApiData)))
+    if (globalScope.performance) {
+      if (typeof PerformanceNavigationTiming !== 'undefined') { // Navigation Timing level 2 API that replaced PerformanceTiming & PerformanceNavigation
+        const navTimingEntry = globalScope?.performance?.getEntriesByType('navigation')?.[0]
+        const perf = ({
+          timing: addPT(agentRuntime.offset, navTimingEntry, {}),
+          navigation: addPN(navTimingEntry, {})
+        })
+        chunksForQueryString.push(param('perf', stringify(perf)))
+      } else if (typeof PerformanceTiming !== 'undefined') { // Safari pre-15 did not support level 2 timing
+        const perf = ({
+          timing: addPT(agentRuntime.offset, globalScope.performance.timing, {}, true),
+          navigation: addPN(globalScope.performance.navigation, {})
+        })
+        chunksForQueryString.push(param('perf', 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

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

@@ -27,11 +27,7 @@ export class Aggregate extends AggregateBase {
     this.timings = []
     this.timingsSent = []
     this.curSessEndRecorded = false
-
-    try { // we (only) need to track cls state because it's attached to other timing events rather than reported on change...
-      this.clsSupported = PerformanceObserver.supportedEntryTypes.includes('layout-shift')
-      this.cls = 0
-    } catch (e) {}
+    this.cls = null // this should be null unless set to a numeric value by web-vitals so that we differentiate if CLS is supported
 
     /*! This is the section that used to be in the loader portion: !*/
     /* ------------------------------------------------------------ */
@@ -66,7 +62,7 @@ export class Aggregate extends AggregateBase {
 
     /* First Input Delay (+"First Interaction") - As of WV v3, it still imperfectly tries to detect document vis state asap and isn't supposed to report if page starts hidden. */
     onFID(({ name, value, entries }) => {
-      if (pageStartedHidden || this.alreadySent.has(name)) return
+      if (pageStartedHidden || this.alreadySent.has(name) || entries.length === 0) return
       this.alreadySent.add(name)
 
       // CWV will only report one (THE) first-input entry to us; fid isn't reported if there are no user interactions occurs before the *first* page hiding.
@@ -84,24 +80,28 @@ export class Aggregate extends AggregateBase {
       if (pageStartedHidden || this.alreadySent.has(name)) return
       this.alreadySent.add(name)
 
-      // CWV will only ever report one (THE) lcp entry to us; lcp is also only reported *once* on earlier(user interaction, page hidden).
-      const lcpEntry = entries[entries.length - 1] // this looks weird if we only expect one, but this is how cwv-attribution gets it so to be sure...
-      const attrs = {
-        size: lcpEntry.size,
-        eid: lcpEntry.id
-      }
-      this.addConnectionAttributes(attrs)
-      if (lcpEntry.url) {
-        attrs['elUrl'] = cleanURL(lcpEntry.url)
-      }
-      if (lcpEntry.element && lcpEntry.element.tagName) {
-        attrs['elTag'] = lcpEntry.element.tagName
+      const attributes = {}
+      if (entries.length > 0) {
+        // CWV will only ever report one (THE) lcp entry to us; lcp is also only reported *once* on earlier(user interaction, page hidden).
+        const lcpEntry = entries[entries.length - 1] // this looks weird if we only expect one, but this is how cwv-attribution gets it so to be sure...
+        attributes.size = lcpEntry.size
+        attributes.eid = lcpEntry.id
+
+        if (lcpEntry.url) {
+          attributes['elUrl'] = cleanURL(lcpEntry.url)
+        }
+        if (lcpEntry.element?.tagName) {
+          attributes['elTag'] = lcpEntry.element.tagName
+        }
       }
-      this.addTiming(name.toLowerCase(), value, attrs)
+
+      this.addConnectionAttributes(attributes)
+      this.addTiming(name.toLowerCase(), value, attributes)
     })
 
-    /* Cumulative Layout Shift - We don't have to limit this callback since cls is stored as a state and only sent as attribute on other timings. */
-    onCLS(({ value }) => this.cls = value)
+    /* Cumulative Layout Shift - We don't have to limit this callback since cls is stored as a state and only sent as attribute on other timings.
+      reportAllChanges ensures our tracked cls has the most recent rolling value to attach to 'unload' and 'pagehide'. */
+    onCLS(({ value }) => this.cls = value, { reportAllChanges: true })
 
     /* Interaction-to-Next-Paint */
     onINP(({ name, value, id }) => this.addTiming(name.toLowerCase(), value, { metricId: id }))
@@ -160,16 +160,27 @@ export class Aggregate extends AggregateBase {
    */
   recordPageUnload (timestamp) {
     this.addTiming('unload', timestamp, null)
-    // Because window's pageHide commonly fires before vis change and the final harvest occurs on the earlier of the two, we also have to add that now or it won't make it into the last payload out.
+    /*
+    Issue: Because window's pageHide commonly fires BEFORE vis change and "final" harvest would happen at the former in this case, we also have to add our vis-change event now or it may not be sent.
+    Affected: Safari < v14.1/.5 ; versions that don't support 'visiilitychange' event
+    Impact: For affected w/o this, NR 'pageHide' attribute may not be sent. For other browsers w/o this, NR 'pageHide' gets fragmented into its own harvest call on page unloading because of dual EoL logic.
+    Mitigation: NR 'unload' and 'pageHide' are both recorded when window pageHide fires, rather than only recording 'unload'.
+    Future: When EoL can become the singular subscribeToVisibilityChange, it's likely endCurrentSession isn't needed here as 'unload'-'pageHide' can be untangled.
+    */
     this.endCurrentSession(timestamp)
   }
 
   addTiming (name, value, attrs) {
     attrs = attrs || {}
 
-    // If CLS is supported, a cls value should exist and be reported, even at 0.
-    // *cli Mar'23 - At this time, it remains attached to all timings. See NEWRELIC-6143.
-    if (this.clsSupported) {
+    // If cls was set to another value by `onCLS`, then it's supported and is attached onto any timing but is omitted until such time.
+    /*
+    *cli Apr'23 - Convert attach-to-all -> attach-if-not-null. See NEWRELIC-6143.
+    Issue: Because NR 'pageHide' was only sent once with what is considered the "final" CLS value, in the case that 'pageHide' fires before 'load' happens, we incorrectly a final CLS of 0 for that page.
+    Mitigation: We've set initial CLS to null so that it's omitted from timings like 'pageHide' in that edge case. It should only be included if onCLS callback was executed at least once.
+    Future: onCLS value changes should be reported directly & CLS separated into its own timing node so it's not beholden to 'pageHide' firing. It'd also be possible to report the real final CLS.
+    */
+    if (this.cls !== null) {
       attrs['cls'] = this.cls
     }
 

package/src/features/page_view_timing/long-tasks.js

@@ -15,21 +15,21 @@ export const onLongTask = (onReport) => {
       const metric = {
         name: 'LT',
         value: entry.duration,
-        info: {	// this property deviates from CWV std interface but will hold the custom context to send to NRDB
-          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'
+        info: { // this property deviates from CWV std interface but will hold the custom context to send to NRDB
+          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'
         }
       }
-      if (metric.info.ltCtr !== 'window') {	// the following properties are only of relevance & appended for html elements
+      if (metric.info.ltCtr !== 'window') { // the following properties are only of relevance & appended for html elements
         Object.assign(metric.info, {
-          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
+          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
         })
       }
 
-      onReport(metric)	// report every long task observed unconditionally
+      onReport(metric) // report every long task observed unconditionally
     })
   }
 
@@ -53,7 +53,7 @@ export const onLongTask = (onReport) => {
   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
+    }, 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/src/features/session_trace/aggregate/index.js

@@ -75,7 +75,11 @@ export class Aggregate extends AggregateBase {
     this.laststart = 0
 
     registerHandler('feat-stn', () => {
-      this.storeTiming(window.performance.timing)
+      if (typeof PerformanceNavigationTiming !== 'undefined') {
+        this.storeTiming(window.performance?.getEntriesByType('navigation')?.[0] || {})
+      } else {
+        this.storeTiming(window.performance?.timing)
+      }
 
       var scheduler = new HarvestScheduler('resources', {
         onFinished: onHarvestFinished.bind(this),
@@ -135,25 +139,24 @@ export class Aggregate extends AggregateBase {
   processPVT (name, value, attrs) {
     var t = {}
     t[name] = value
-    this.storeTiming(t, true)
+    this.storeTiming(t)
     if (this.hasFID(name, attrs)) this.storeEvent({ type: 'fid', target: 'document' }, 'document', value, value + attrs.fid)
   }
 
-  storeTiming (_t, ignoreOffset) {
-    var key
-    var val
-    var timeOffset
-    var dateNow = Date.now()
-
+  storeTiming (_t) {
     // loop iterates through prototype also (for FF)
-    for (key in _t) {
-      val = _t[key]
+    for (let key in _t) {
+      const val = _t[key]
+
+      // ignore size and status type nodes that do not map to timestamp metrics
+      const lck = key.toLowerCase()
+      if (lck.indexOf('size') >= 0 || lck.indexOf('status') >= 0) continue
 
       // ignore inherited methods, meaningless 0 values, and bogus timestamps
       // that are in the future (Microsoft Edge seems to sometimes produce these)
-      if (!(typeof (val) === 'number' && val > 0 && val < dateNow)) continue
+      if (!(typeof (val) === 'number' && val >= 0)) continue
 
-      timeOffset = !ignoreOffset ? _t[key] - getRuntime(this.agentIdentifier).offset : _t[key]
+      const timeOffset = Math.round(_t[key])
 
       this.storeSTN({
         n: key,

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

@@ -78,8 +78,9 @@ export class Instrument extends InstrumentBase {
       observeResourceTimings()
     } else {
       // collect resource timings once when buffer is full
-      if (window.performance[CRT] && window.performance[ADD_EVENT_LISTENER])
-      { window.performance.addEventListener(RESOURCE_TIMING_BUFFER_FULL, this.onResourceTimingBufferFull, eventListenerOpts(false)) }
+      if (window.performance[CRT] && window.performance[ADD_EVENT_LISTENER]) {
+        window.performance.addEventListener(RESOURCE_TIMING_BUFFER_FULL, this.onResourceTimingBufferFull, eventListenerOpts(false))
+      }
     }
 
     function observeResourceTimings () {

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

@@ -24,8 +24,10 @@ export class Instrument extends InstrumentBase {
 
     if (!getRuntime(agentIdentifier).xhrWrappable) return
 
-    try { this.removeOnAbort = new AbortController() }
-    catch (e) {}
+    try {
+      this.removeOnAbort = new AbortController()
+    } catch (e) {}
+
     let depth = 0
     let startHash
 

package/src/loaders/api/api.js

@@ -35,7 +35,7 @@ export function setTopLevelCallers () {
         returnVals.push(val.api[fnName](...args))
       }
     })
-    return returnVals.length > 1 ? returnsVals : returnVals[0]
+    return returnVals.length > 1 ? returnVals : returnVals[0]
   }
 }
 

package/dist/cjs/common/constants/environment-variables.js

@@ -1,20 +0,0 @@
-"use strict";
-
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.VERSION = exports.DIST_METHOD = exports.BUILD_ENV = void 0;
-// Webpack and NPM builds define `process.env.BUILD_VERSION` at build time based on variables supplied at build time.
-// VERSION as defined here ends up decorating: agent runtime configuration, harvest, and supportability metrics.
-const VERSION = typeof "0.1.230" !== 'undefined' && "0.1.230" || '';
-
-// process.env.BUILD_ENV is replaced during webpack -or- npm build with environment vars supplied at build time
-// LOCAL, PROD, DEV, etc
-exports.VERSION = VERSION;
-const BUILD_ENV = typeof "NPM" !== 'undefined' && "NPM" || '';
-
-// process.env.DIST_METHOD is replaced during webpack -or- npm build with environment vars supplied at build time
-// CDN, NPM etc
-exports.BUILD_ENV = BUILD_ENV;
-const DIST_METHOD = typeof "NPM" !== 'undefined' && "NPM" || '';
-exports.DIST_METHOD = DIST_METHOD;

package/dist/cjs/common/wrap/wrap-console.js

@@ -1,54 +0,0 @@
-"use strict";
-
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.scopedEE = scopedEE;
-exports.wrapConsole = wrapConsole;
-var _contextualEe = require("../event-emitter/contextual-ee");
-var _wrapFunction = require("./wrap-function");
-var _globalScope = require("../util/global-scope");
-/*
- * Copyright 2020 New Relic Corporation. All rights reserved.
- * SPDX-License-Identifier: Apache-2.0
- */
-/**
- * @file Wraps `debug`, `error`, `info`, `log`, `trace, and `warn` methods of the global `console` object for instrumentation.
- * This module is used by: metrics.
- */
-
-const wrapped = {};
-const CONSOLE_METHODS = ['debug', 'error', 'info', 'log', 'trace', 'warn'];
-
-/**
- * Wraps the `debug`, `error`, `info`, `log`, `trace, and `warn` methods of global `console` object and returns a
- * corresponding event emitter scoped to the console object.
- * @param {Object} sharedEE - The shared event emitter on which a new scoped event emitter will be based.
- * @returns {Object} Scoped event emitter with a debug ID of `console`.
- */
-function wrapConsole(sharedEE) {
-  const ee = scopedEE(sharedEE);
-
-  // We want to wrap console once and only once for each agent instance (`ee.debugId`).
-  if (wrapped[ee.debugId]) {
-    return ee;
-  }
-  wrapped[ee.debugId] = true;
-  var functionWrapper = (0, _wrapFunction.createWrapperWithEmitter)(ee);
-  // Because the console object exists once on the global scope, we don't need to wrap the prototype's methods.
-  // We use the global scope instead of window to accomodate service workers.
-  // The leading hyphen on '-console-' tells `inPlace` to prefix emitted event names with the method name too.
-  functionWrapper.inPlace(_globalScope.globalScope.console, CONSOLE_METHODS, '-console-');
-  return ee;
-}
-
-/**
- * Returns an event emitter scoped specifically for the `console` context. This scoping is a remnant from when all the
- * features shared the same group in the event, to isolate events between features. It will likely be revisited.
- * @param {Object} sharedEE - Optional event emitter on which to base the scoped emitter.
- *    Uses `ee` on the global scope if undefined).
- * @returns {Object} Scoped event emitter with a debug ID of 'console'.
- */
-function scopedEE(sharedEE) {
-  return (sharedEE || _contextualEe.ee).get('console');
-}

package/dist/esm/common/constants/environment-variables.js

@@ -1,11 +0,0 @@
-// Webpack and NPM builds define `process.env.BUILD_VERSION` at build time based on variables supplied at build time.
-// VERSION as defined here ends up decorating: agent runtime configuration, harvest, and supportability metrics.
-export const VERSION = typeof "0.1.230" !== 'undefined' && "0.1.230" || '';
-
-// process.env.BUILD_ENV is replaced during webpack -or- npm build with environment vars supplied at build time
-// LOCAL, PROD, DEV, etc
-export const BUILD_ENV = typeof "NPM" !== 'undefined' && "NPM" || '';
-
-// process.env.DIST_METHOD is replaced during webpack -or- npm build with environment vars supplied at build time
-// CDN, NPM etc
-export const DIST_METHOD = typeof "NPM" !== 'undefined' && "NPM" || '';

package/dist/esm/common/wrap/wrap-console.js

@@ -1,46 +0,0 @@
-/*
- * Copyright 2020 New Relic Corporation. All rights reserved.
- * SPDX-License-Identifier: Apache-2.0
- */
-/**
- * @file Wraps `debug`, `error`, `info`, `log`, `trace, and `warn` methods of the global `console` object for instrumentation.
- * This module is used by: metrics.
- */
-import { ee as globalEE } from '../event-emitter/contextual-ee';
-import { createWrapperWithEmitter } from './wrap-function';
-import { globalScope } from '../util/global-scope';
-const wrapped = {};
-const CONSOLE_METHODS = ['debug', 'error', 'info', 'log', 'trace', 'warn'];
-
-/**
- * Wraps the `debug`, `error`, `info`, `log`, `trace, and `warn` methods of global `console` object and returns a
- * corresponding event emitter scoped to the console object.
- * @param {Object} sharedEE - The shared event emitter on which a new scoped event emitter will be based.
- * @returns {Object} Scoped event emitter with a debug ID of `console`.
- */
-export function wrapConsole(sharedEE) {
-  const ee = scopedEE(sharedEE);
-
-  // We want to wrap console once and only once for each agent instance (`ee.debugId`).
-  if (wrapped[ee.debugId]) {
-    return ee;
-  }
-  wrapped[ee.debugId] = true;
-  var functionWrapper = createWrapperWithEmitter(ee);
-  // Because the console object exists once on the global scope, we don't need to wrap the prototype's methods.
-  // We use the global scope instead of window to accomodate service workers.
-  // The leading hyphen on '-console-' tells `inPlace` to prefix emitted event names with the method name too.
-  functionWrapper.inPlace(globalScope.console, CONSOLE_METHODS, '-console-');
-  return ee;
-}
-
-/**
- * Returns an event emitter scoped specifically for the `console` context. This scoping is a remnant from when all the
- * features shared the same group in the event, to isolate events between features. It will likely be revisited.
- * @param {Object} sharedEE - Optional event emitter on which to base the scoped emitter.
- *    Uses `ee` on the global scope if undefined).
- * @returns {Object} Scoped event emitter with a debug ID of 'console'.
- */
-export function scopedEE(sharedEE) {
-  return (sharedEE || globalEE).get('console');
-}

package/dist/types/common/constants/environment-variables.d.ts

@@ -1,4 +0,0 @@
-export const VERSION: string;
-export const BUILD_ENV: string;
-export const DIST_METHOD: string;
-//# sourceMappingURL=environment-variables.d.ts.map

package/dist/types/common/constants/environment-variables.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"environment-variables.d.ts","sourceRoot":"","sources":["../../../../src/common/constants/environment-variables.js"],"names":[],"mappings":"AAEA,6BAA0G;AAI1G,+BAAoG;AAIpG,iCAA0G"}

package/dist/types/common/wrap/wrap-console.d.ts

@@ -1,16 +0,0 @@
-/**
- * Wraps the `debug`, `error`, `info`, `log`, `trace, and `warn` methods of global `console` object and returns a
- * corresponding event emitter scoped to the console object.
- * @param {Object} sharedEE - The shared event emitter on which a new scoped event emitter will be based.
- * @returns {Object} Scoped event emitter with a debug ID of `console`.
- */
-export function wrapConsole(sharedEE: Object): Object;
-/**
- * Returns an event emitter scoped specifically for the `console` context. This scoping is a remnant from when all the
- * features shared the same group in the event, to isolate events between features. It will likely be revisited.
- * @param {Object} sharedEE - Optional event emitter on which to base the scoped emitter.
- *    Uses `ee` on the global scope if undefined).
- * @returns {Object} Scoped event emitter with a debug ID of 'console'.
- */
-export function scopedEE(sharedEE: Object): Object;
-//# sourceMappingURL=wrap-console.d.ts.map

package/dist/types/common/wrap/wrap-console.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"wrap-console.d.ts","sourceRoot":"","sources":["../../../../src/common/wrap/wrap-console.js"],"names":[],"mappings":"AAeA;;;;;GAKG;AACH,sCAHW,MAAM,GACJ,MAAM,CAgBlB;AAED;;;;;;GAMG;AACH,mCAJW,MAAM,GAEJ,MAAM,CAIlB"}

package/src/common/constants/environment-variables.js

@@ -1,11 +0,0 @@
-// Webpack and NPM builds define `process.env.BUILD_VERSION` at build time based on variables supplied at build time.
-// VERSION as defined here ends up decorating: agent runtime configuration, harvest, and supportability metrics.
-export const VERSION = typeof process.env.BUILD_VERSION !== 'undefined' && process.env.BUILD_VERSION || ''
-
-// process.env.BUILD_ENV is replaced during webpack -or- npm build with environment vars supplied at build time
-// LOCAL, PROD, DEV, etc
-export const BUILD_ENV = typeof process.env.BUILD_ENV !== 'undefined' && process.env.BUILD_ENV || ''
-
-// process.env.DIST_METHOD is replaced during webpack -or- npm build with environment vars supplied at build time
-// CDN, NPM etc
-export const DIST_METHOD = typeof process.env.DIST_METHOD !== 'undefined' && process.env.DIST_METHOD || ''

package/src/common/wrap/wrap-console.js

@@ -1,47 +0,0 @@
-/*
- * Copyright 2020 New Relic Corporation. All rights reserved.
- * SPDX-License-Identifier: Apache-2.0
- */
-/**
- * @file Wraps `debug`, `error`, `info`, `log`, `trace, and `warn` methods of the global `console` object for instrumentation.
- * This module is used by: metrics.
- */
-import { ee as globalEE } from '../event-emitter/contextual-ee'
-import { createWrapperWithEmitter } from './wrap-function'
-import { globalScope } from '../util/global-scope'
-
-const wrapped = {}
-const CONSOLE_METHODS = ['debug', 'error', 'info', 'log', 'trace', 'warn']
-
-/**
- * Wraps the `debug`, `error`, `info`, `log`, `trace, and `warn` methods of global `console` object and returns a
- * corresponding event emitter scoped to the console object.
- * @param {Object} sharedEE - The shared event emitter on which a new scoped event emitter will be based.
- * @returns {Object} Scoped event emitter with a debug ID of `console`.
- */
-export function wrapConsole (sharedEE) {
-  const ee = scopedEE(sharedEE)
-
-  // We want to wrap console once and only once for each agent instance (`ee.debugId`).
-  if (wrapped[ee.debugId]) { return ee }
-  wrapped[ee.debugId] = true
-
-  var functionWrapper = createWrapperWithEmitter(ee)
-  // Because the console object exists once on the global scope, we don't need to wrap the prototype's methods.
-  // We use the global scope instead of window to accomodate service workers.
-  // The leading hyphen on '-console-' tells `inPlace` to prefix emitted event names with the method name too.
-  functionWrapper.inPlace(globalScope.console, CONSOLE_METHODS, '-console-')
-
-  return ee
-}
-
-/**
- * Returns an event emitter scoped specifically for the `console` context. This scoping is a remnant from when all the
- * features shared the same group in the event, to isolate events between features. It will likely be revisited.
- * @param {Object} sharedEE - Optional event emitter on which to base the scoped emitter.
- *    Uses `ee` on the global scope if undefined).
- * @returns {Object} Scoped event emitter with a debug ID of 'console'.
- */
-export function scopedEE (sharedEE) {
-  return (sharedEE || globalEE).get('console')
-}