@atlaskit/react-ufo 2.4.5 → 2.4.6

package/CHANGELOG.md

@@ -1,5 +1,16 @@
 # @atlaskit/ufo-interaction-ignore
 
+## 2.4.6
+
+### Patch Changes
+
+- [#174829](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/174829)
+  [`381735c03773b`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/381735c03773b) -
+  add ufo: prefix to error count and stylesheet count metrics
+- [#174760](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/pull-requests/174760)
+  [`0c5bbf0079bee`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/0c5bbf0079bee) -
+  Remove display style attribute mutation check
+
 ## 2.4.5
 
 ### Patch Changes

package/dist/cjs/create-payload/index.js

@@ -638,16 +638,22 @@ function getStylesheetMetrics() {
       }
     }, 0);
     return {
-      stylesheets: stylesheetCount,
-      styleElements: styleElements,
-      styleProps: styleProps.length,
-      styleDeclarations: styleDeclarations,
-      cssrules: cssrules
+      'ufo:stylesheets': stylesheetCount,
+      'ufo:styleElements': styleElements,
+      'ufo:styleProps': styleProps.length,
+      'ufo:styleDeclarations': styleDeclarations,
+      'ufo:cssrules': cssrules
     };
   } catch (e) {
     return {};
   }
 }
+function getErrorCounts(interaction) {
+  return {
+    'ufo:errors:globalCount': (0, _globalErrorHandler.getGlobalErrorCount)(),
+    'ufo:errors:count': interaction.errors.length
+  };
+}
 function createInteractionMetricsPayload(interaction, interactionId) {
   var _window$location, _config$additionalPay;
   var interactionPayloadStart = performance.now();
@@ -742,7 +748,7 @@ function createInteractionMetricsPayload(interaction, interactionId) {
     source: 'measured',
     tags: ['observability'],
     attributes: {
-      properties: _objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread({
+      properties: _objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread({
         // basic
         'event:hostname': ((_window$location = window.location) === null || _window$location === void 0 ? void 0 : _window$location.hostname) || 'unknown',
         'event:product': config.product,
@@ -756,8 +762,7 @@ function createInteractionMetricsPayload(interaction, interactionId) {
         'event:region': config.region || 'unknown',
         'experience:key': 'custom.interaction-metrics',
         'experience:name': newUFOName
-      }, getBrowserMetadata()), getSSRProperties(type)), getPPSMetrics(interaction)), getPaintMetrics(type)), getNavigationMetrics(type)), getVCMetrics(interaction)), (_config$additionalPay = config.additionalPayloadData) === null || _config$additionalPay === void 0 ? void 0 : _config$additionalPay.call(config, interaction)), getTracingContextData(interaction)), getStylesheetMetrics()), {}, {
-        errorCount: (0, _globalErrorHandler.getGlobalErrorCount)(),
+      }, getBrowserMetadata()), getSSRProperties(type)), getPPSMetrics(interaction)), getPaintMetrics(type)), getNavigationMetrics(type)), getVCMetrics(interaction)), (_config$additionalPay = config.additionalPayloadData) === null || _config$additionalPay === void 0 ? void 0 : _config$additionalPay.call(config, interaction)), getTracingContextData(interaction)), getStylesheetMetrics()), getErrorCounts(interaction)), {}, {
         interactionMetrics: _objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread({
           namePrefix: config.namePrefix || '',
           segmentPrefix: config.segmentPrefix || '',
@@ -788,8 +793,7 @@ function createInteractionMetricsPayload(interaction, interactionId) {
           }),
           marks: optimizeMarks(interaction.marks),
           customData: optimizeCustomData(interaction),
-          reactProfilerTimings: optimizeReactProfilerTimings(interaction.reactProfilerTimings, start),
-          errorCount: interaction.errors.length
+          reactProfilerTimings: optimizeReactProfilerTimings(interaction.reactProfilerTimings, start)
         }, labelStack), getPageLoadInteractionMetrics()), getDetailedInteractionMetrics()), getPageLoadDetailedInteractionMetrics()), getBm3TrackerTimings(interaction)),
         'ufo:payloadTime': (0, _roundNumber.roundEpsilon)(performance.now() - interactionPayloadStart)
       })

package/dist/cjs/vc/vc-observer/observers/index.js

@@ -5,7 +5,6 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.Observers = void 0;
-var _slicedToArray2 = _interopRequireDefault(require("@babel/runtime/helpers/slicedToArray"));
 var _classCallCheck2 = _interopRequireDefault(require("@babel/runtime/helpers/classCallCheck"));
 var _createClass2 = _interopRequireDefault(require("@babel/runtime/helpers/createClass"));
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
@@ -13,9 +12,6 @@ var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 var _vcUtils = require("../media-wrapper/vc-utils");
 var _editorLnv = require("./editor-lnv");
 var _ssrPlaceholders = require("./ssr-placeholders");
-function _createForOfIteratorHelper(r, e) { var t = "undefined" != typeof Symbol && r[Symbol.iterator] || r["@@iterator"]; if (!t) { if (Array.isArray(r) || (t = _unsupportedIterableToArray(r)) || e && r && "number" == typeof r.length) { t && (r = t); var _n = 0, F = function F() {}; return { s: F, n: function n() { return _n >= r.length ? { done: !0 } : { done: !1, value: r[_n++] }; }, e: function e(r) { throw r; }, f: F }; } throw new TypeError("Invalid attempt to iterate non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); } var o, a = !0, u = !1; return { s: function s() { t = t.call(r); }, n: function n() { var r = t.next(); return a = r.done, r; }, e: function e(r) { u = !0, o = r; }, f: function f() { try { a || null == t.return || t.return(); } finally { if (u) throw o; } } }; }
-function _unsupportedIterableToArray(r, a) { if (r) { if ("string" == typeof r) return _arrayLikeToArray(r, a); var t = {}.toString.call(r).slice(8, -1); return "Object" === t && r.constructor && (t = r.constructor.name), "Map" === t || "Set" === t ? Array.from(r) : "Arguments" === t || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(t) ? _arrayLikeToArray(r, a) : void 0; } }
-function _arrayLikeToArray(r, a) { (null == a || a > r.length) && (a = r.length); for (var e = 0, n = Array(a); e < a; e++) n[e] = r[e]; return n; }
 function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
 function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { (0, _defineProperty2.default)(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
 var state = {
@@ -202,8 +198,7 @@ var Observers = exports.Observers = /*#__PURE__*/function () {
             });
           } else if (mutation.type === 'attributes') {
             if (mutation.target instanceof HTMLElement) {
-              var isDisplayStyleAttributeMutation = mutation.attributeName === 'style' && _this2.getStyleDisplay(mutation.target.getAttribute('style')) !== _this2.getStyleDisplay(mutation.oldValue);
-              if ((0, _platformFeatureFlags.fg)('platform-ufo-vc-observe-attributes') || isDisplayStyleAttributeMutation) {
+              if ((0, _platformFeatureFlags.fg)('platform-ufo-vc-observe-attributes')) {
                 _this2.observeElement(mutation.target, mutation, 'attr', ignoreReason);
               }
             }
@@ -212,40 +207,6 @@ var Observers = exports.Observers = /*#__PURE__*/function () {
         _this2.measureStop();
       }) : null;
     }
-  }, {
-    key: "getStyleDisplay",
-    value: function getStyleDisplay(styleAttributeValue) {
-      if (!styleAttributeValue) {
-        return undefined;
-      }
-
-      // Split the style string into individual declarations
-      var declarations = styleAttributeValue.split(';');
-      // Iterate over each declaration
-      var _iterator = _createForOfIteratorHelper(declarations),
-        _step;
-      try {
-        for (_iterator.s(); !(_step = _iterator.n()).done;) {
-          var declaration = _step.value;
-          // Trim whitespace and split into property and value
-          var _declaration$split$ma = declaration.split(':').map(function (part) {
-              return part.trim();
-            }),
-            _declaration$split$ma2 = (0, _slicedToArray2.default)(_declaration$split$ma, 2),
-            property = _declaration$split$ma2[0],
-            value = _declaration$split$ma2[1];
-          // Check if the property is 'display'
-          if (property && property.toLowerCase() === 'display') {
-            return value;
-          }
-        }
-      } catch (err) {
-        _iterator.e(err);
-      } finally {
-        _iterator.f();
-      }
-      return undefined;
-    }
   }, {
     key: "getElementName",
     value: function getElementName(element) {

package/dist/es2019/create-payload/index.js

@@ -630,16 +630,22 @@ function getStylesheetMetrics() {
       }
     }, 0);
     return {
-      stylesheets: stylesheetCount,
-      styleElements,
-      styleProps: styleProps.length,
-      styleDeclarations,
-      cssrules: cssrules
+      'ufo:stylesheets': stylesheetCount,
+      'ufo:styleElements': styleElements,
+      'ufo:styleProps': styleProps.length,
+      'ufo:styleDeclarations': styleDeclarations,
+      'ufo:cssrules': cssrules
     };
   } catch (e) {
     return {};
   }
 }
+function getErrorCounts(interaction) {
+  return {
+    'ufo:errors:globalCount': getGlobalErrorCount(),
+    'ufo:errors:count': interaction.errors.length
+  };
+}
 function createInteractionMetricsPayload(interaction, interactionId) {
   var _window$location, _config$additionalPay;
   const interactionPayloadStart = performance.now();
@@ -759,7 +765,7 @@ function createInteractionMetricsPayload(interaction, interactionId) {
         ...((_config$additionalPay = config.additionalPayloadData) === null || _config$additionalPay === void 0 ? void 0 : _config$additionalPay.call(config, interaction)),
         ...getTracingContextData(interaction),
         ...getStylesheetMetrics(),
-        errorCount: getGlobalErrorCount(),
+        ...getErrorCounts(interaction),
         interactionMetrics: {
           namePrefix: config.namePrefix || '',
           segmentPrefix: config.segmentPrefix || '',
@@ -791,7 +797,6 @@ function createInteractionMetricsPayload(interaction, interactionId) {
           marks: optimizeMarks(interaction.marks),
           customData: optimizeCustomData(interaction),
           reactProfilerTimings: optimizeReactProfilerTimings(interaction.reactProfilerTimings, start),
-          errorCount: interaction.errors.length,
           ...labelStack,
           ...getPageLoadInteractionMetrics(),
           ...getDetailedInteractionMetrics(),

package/dist/es2019/vc/vc-observer/observers/index.js

@@ -170,8 +170,7 @@ export class Observers {
           });
         } else if (mutation.type === 'attributes') {
           if (mutation.target instanceof HTMLElement) {
-            const isDisplayStyleAttributeMutation = mutation.attributeName === 'style' && this.getStyleDisplay(mutation.target.getAttribute('style')) !== this.getStyleDisplay(mutation.oldValue);
-            if (fg('platform-ufo-vc-observe-attributes') || isDisplayStyleAttributeMutation) {
+            if (fg('platform-ufo-vc-observe-attributes')) {
               this.observeElement(mutation.target, mutation, 'attr', ignoreReason);
             }
           }
@@ -180,24 +179,6 @@ export class Observers {
       this.measureStop();
     }) : null;
   }
-  getStyleDisplay(styleAttributeValue) {
-    if (!styleAttributeValue) {
-      return undefined;
-    }
-
-    // Split the style string into individual declarations
-    const declarations = styleAttributeValue.split(';');
-    // Iterate over each declaration
-    for (const declaration of declarations) {
-      // Trim whitespace and split into property and value
-      const [property, value] = declaration.split(':').map(part => part.trim());
-      // Check if the property is 'display'
-      if (property && property.toLowerCase() === 'display') {
-        return value;
-      }
-    }
-    return undefined;
-  }
   getElementName(element) {
     try {
       const tagName = element.localName;

package/dist/esm/create-payload/index.js

@@ -628,16 +628,22 @@ function getStylesheetMetrics() {
       }
     }, 0);
     return {
-      stylesheets: stylesheetCount,
-      styleElements: styleElements,
-      styleProps: styleProps.length,
-      styleDeclarations: styleDeclarations,
-      cssrules: cssrules
+      'ufo:stylesheets': stylesheetCount,
+      'ufo:styleElements': styleElements,
+      'ufo:styleProps': styleProps.length,
+      'ufo:styleDeclarations': styleDeclarations,
+      'ufo:cssrules': cssrules
     };
   } catch (e) {
     return {};
   }
 }
+function getErrorCounts(interaction) {
+  return {
+    'ufo:errors:globalCount': getGlobalErrorCount(),
+    'ufo:errors:count': interaction.errors.length
+  };
+}
 function createInteractionMetricsPayload(interaction, interactionId) {
   var _window$location, _config$additionalPay;
   var interactionPayloadStart = performance.now();
@@ -732,7 +738,7 @@ function createInteractionMetricsPayload(interaction, interactionId) {
     source: 'measured',
     tags: ['observability'],
     attributes: {
-      properties: _objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread({
+      properties: _objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread({
         // basic
         'event:hostname': ((_window$location = window.location) === null || _window$location === void 0 ? void 0 : _window$location.hostname) || 'unknown',
         'event:product': config.product,
@@ -746,8 +752,7 @@ function createInteractionMetricsPayload(interaction, interactionId) {
         'event:region': config.region || 'unknown',
         'experience:key': 'custom.interaction-metrics',
         'experience:name': newUFOName
-      }, getBrowserMetadata()), getSSRProperties(type)), getPPSMetrics(interaction)), getPaintMetrics(type)), getNavigationMetrics(type)), getVCMetrics(interaction)), (_config$additionalPay = config.additionalPayloadData) === null || _config$additionalPay === void 0 ? void 0 : _config$additionalPay.call(config, interaction)), getTracingContextData(interaction)), getStylesheetMetrics()), {}, {
-        errorCount: getGlobalErrorCount(),
+      }, getBrowserMetadata()), getSSRProperties(type)), getPPSMetrics(interaction)), getPaintMetrics(type)), getNavigationMetrics(type)), getVCMetrics(interaction)), (_config$additionalPay = config.additionalPayloadData) === null || _config$additionalPay === void 0 ? void 0 : _config$additionalPay.call(config, interaction)), getTracingContextData(interaction)), getStylesheetMetrics()), getErrorCounts(interaction)), {}, {
         interactionMetrics: _objectSpread(_objectSpread(_objectSpread(_objectSpread(_objectSpread({
           namePrefix: config.namePrefix || '',
           segmentPrefix: config.segmentPrefix || '',
@@ -778,8 +783,7 @@ function createInteractionMetricsPayload(interaction, interactionId) {
           }),
           marks: optimizeMarks(interaction.marks),
           customData: optimizeCustomData(interaction),
-          reactProfilerTimings: optimizeReactProfilerTimings(interaction.reactProfilerTimings, start),
-          errorCount: interaction.errors.length
+          reactProfilerTimings: optimizeReactProfilerTimings(interaction.reactProfilerTimings, start)
         }, labelStack), getPageLoadInteractionMetrics()), getDetailedInteractionMetrics()), getPageLoadDetailedInteractionMetrics()), getBm3TrackerTimings(interaction)),
         'ufo:payloadTime': roundEpsilon(performance.now() - interactionPayloadStart)
       })

package/dist/esm/vc/vc-observer/observers/index.js

@@ -1,10 +1,6 @@
-import _slicedToArray from "@babel/runtime/helpers/slicedToArray";
 import _classCallCheck from "@babel/runtime/helpers/classCallCheck";
 import _createClass from "@babel/runtime/helpers/createClass";
 import _defineProperty from "@babel/runtime/helpers/defineProperty";
-function _createForOfIteratorHelper(r, e) { var t = "undefined" != typeof Symbol && r[Symbol.iterator] || r["@@iterator"]; if (!t) { if (Array.isArray(r) || (t = _unsupportedIterableToArray(r)) || e && r && "number" == typeof r.length) { t && (r = t); var _n = 0, F = function F() {}; return { s: F, n: function n() { return _n >= r.length ? { done: !0 } : { done: !1, value: r[_n++] }; }, e: function e(r) { throw r; }, f: F }; } throw new TypeError("Invalid attempt to iterate non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); } var o, a = !0, u = !1; return { s: function s() { t = t.call(r); }, n: function n() { var r = t.next(); return a = r.done, r; }, e: function e(r) { u = !0, o = r; }, f: function f() { try { a || null == t.return || t.return(); } finally { if (u) throw o; } } }; }
-function _unsupportedIterableToArray(r, a) { if (r) { if ("string" == typeof r) return _arrayLikeToArray(r, a); var t = {}.toString.call(r).slice(8, -1); return "Object" === t && r.constructor && (t = r.constructor.name), "Map" === t || "Set" === t ? Array.from(r) : "Arguments" === t || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(t) ? _arrayLikeToArray(r, a) : void 0; } }
-function _arrayLikeToArray(r, a) { (null == a || a > r.length) && (a = r.length); for (var e = 0, n = Array(a); e < a; e++) n[e] = r[e]; return n; }
 function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
 function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { _defineProperty(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
 import { fg } from '@atlaskit/platform-feature-flags';
@@ -195,8 +191,7 @@ export var Observers = /*#__PURE__*/function () {
             });
           } else if (mutation.type === 'attributes') {
             if (mutation.target instanceof HTMLElement) {
-              var isDisplayStyleAttributeMutation = mutation.attributeName === 'style' && _this2.getStyleDisplay(mutation.target.getAttribute('style')) !== _this2.getStyleDisplay(mutation.oldValue);
-              if (fg('platform-ufo-vc-observe-attributes') || isDisplayStyleAttributeMutation) {
+              if (fg('platform-ufo-vc-observe-attributes')) {
                 _this2.observeElement(mutation.target, mutation, 'attr', ignoreReason);
               }
             }
@@ -205,40 +200,6 @@ export var Observers = /*#__PURE__*/function () {
         _this2.measureStop();
       }) : null;
     }
-  }, {
-    key: "getStyleDisplay",
-    value: function getStyleDisplay(styleAttributeValue) {
-      if (!styleAttributeValue) {
-        return undefined;
-      }
-
-      // Split the style string into individual declarations
-      var declarations = styleAttributeValue.split(';');
-      // Iterate over each declaration
-      var _iterator = _createForOfIteratorHelper(declarations),
-        _step;
-      try {
-        for (_iterator.s(); !(_step = _iterator.n()).done;) {
-          var declaration = _step.value;
-          // Trim whitespace and split into property and value
-          var _declaration$split$ma = declaration.split(':').map(function (part) {
-              return part.trim();
-            }),
-            _declaration$split$ma2 = _slicedToArray(_declaration$split$ma, 2),
-            property = _declaration$split$ma2[0],
-            value = _declaration$split$ma2[1];
-          // Check if the property is 'display'
-          if (property && property.toLowerCase() === 'display') {
-            return value;
-          }
-        }
-      } catch (err) {
-        _iterator.e(err);
-      } finally {
-        _iterator.f();
-      }
-      return undefined;
-    }
   }, {
     key: "getElementName",
     value: function getElementName(element) {