@atlaskit/react-ufo 4.1.4 → 4.1.6

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # @atlaskit/ufo-interaction-ignore
 
+## 4.1.6
+
+### Patch Changes
+
+- [#192382](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/pull-requests/192382)
+  [`5f4f054f58647`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/5f4f054f58647) -
+  Add hard limit for findReactFiber
+
+## 4.1.5
+
+### Patch Changes
+
+- [#191818](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/pull-requests/191818)
+  [`f8f8080d07991`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/f8f8080d07991) -
+  clean up ff platform_ufo_rev_ratios
+
 ## 4.1.4
 
 ### Patch Changes

package/dist/cjs/vc/vc-observer/getVCRevisionsData.js

@@ -6,7 +6,6 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.getVCRevisionsData = getVCRevisionsData;
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
-var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 var _config = require("../../config");
 var _hiddenTiming = require("../../hidden-timing");
 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; } } }; }
@@ -64,7 +63,7 @@ function getVCRevisionsData(_ref) {
       'metric:vc90': shouldHaveVCmetric ? calculatedVC.VC['90'] : null,
       vcDetails: createVCDetails(calculatedVC, shouldHaveVCmetric)
     };
-    if (shouldHaveVCmetric && (0, _platformFeatureFlags.fg)('platform_ufo_rev_ratios')) {
+    if (shouldHaveVCmetric) {
       revision.ratios = ratios;
     }
     availableVCRevisionPayloads.push(revision);
@@ -76,7 +75,7 @@ function getVCRevisionsData(_ref) {
       'metric:vc90': shouldHaveVCmetric ? calculatedVCNext.VC['90'] : null,
       vcDetails: createVCDetails(calculatedVCNext, shouldHaveVCmetric)
     };
-    if (shouldHaveVCmetric && (0, _platformFeatureFlags.fg)('platform_ufo_rev_ratios')) {
+    if (shouldHaveVCmetric) {
       _revision.ratios = ratios;
     }
     availableVCRevisionPayloads.push(_revision);

package/dist/cjs/vc/vc-observer-new/metric-calculator/abstract-base-vc-calculator.js

@@ -12,7 +12,6 @@ var _asyncToGenerator2 = _interopRequireDefault(require("@babel/runtime/helpers/
 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 _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 var _percentileCalc = require("./percentile-calc");
 var _getViewportHeight = _interopRequireDefault(require("./utils/get-viewport-height"));
 var _getViewportWidth = _interopRequireDefault(require("./utils/get-viewport-width"));
@@ -351,9 +350,7 @@ var AbstractVCCalculatorBase = exports.default = /*#__PURE__*/function () {
                 'metric:vc90': (_vcDetails$90$t = vcDetails === null || vcDetails === void 0 || (_vcDetails$ = vcDetails['90']) === null || _vcDetails$ === void 0 ? void 0 : _vcDetails$.t) !== null && _vcDetails$90$t !== void 0 ? _vcDetails$90$t : null,
                 vcDetails: vcDetails !== null && vcDetails !== void 0 ? vcDetails : undefined
               };
-              if ((0, _platformFeatureFlags.fg)('platform_ufo_rev_ratios')) {
-                result.ratios = this.calculateRatios(filteredEntries);
-              }
+              result.ratios = this.calculateRatios(filteredEntries);
               return _context2.abrupt("return", result);
             case 13:
             case "end":

package/dist/cjs/vc/vc-observer-new/viewport-observer/utils/check-within-component-and-extract-child-props.js

@@ -11,6 +11,7 @@ var _findReactFiber = _interopRequireDefault(require("./find-react-fiber"));
 var _getComponentName = _interopRequireDefault(require("./get-component-name"));
 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 DEFAULT_MAX_LEVEL = 20;
 // Using the React Fiber tree to traverse up the DOM and check if a node is within a specific component
 // and extract child component props if needed.
 function checkWithinComponentAndExtractChildProps(node, targetComponentName, childComponentConfig) {
@@ -18,11 +19,13 @@ function checkWithinComponentAndExtractChildProps(node, targetComponentName, chi
   if ((0, _platformFeatureFlags.fg)('platform_ufo_handle_non_react_element_for_3p')) {
     // Walk up the DOM tree to find React fiber (handles non-React-rendered elements)
     var currentElement = node;
-    while (currentElement && !fiber) {
+    var levelsTraversed = 0;
+    while (currentElement && !fiber && levelsTraversed < DEFAULT_MAX_LEVEL) {
       fiber = (0, _findReactFiber.default)(currentElement);
       if (!fiber) {
         currentElement = currentElement.parentElement;
       }
+      levelsTraversed++;
     }
   } else {
     fiber = (0, _findReactFiber.default)(node);

package/dist/es2019/vc/vc-observer/getVCRevisionsData.js

@@ -1,4 +1,3 @@
-import { fg } from '@atlaskit/platform-feature-flags';
 import { isVCRevisionEnabled } from '../../config';
 import { getPageVisibilityState } from '../../hidden-timing';
 const VCParts = ['25', '50', '75', '80', '85', '90', '95', '98', '99'];
@@ -48,7 +47,7 @@ export function getVCRevisionsData({
       'metric:vc90': shouldHaveVCmetric ? calculatedVC.VC['90'] : null,
       vcDetails: createVCDetails(calculatedVC, shouldHaveVCmetric)
     };
-    if (shouldHaveVCmetric && fg('platform_ufo_rev_ratios')) {
+    if (shouldHaveVCmetric) {
       revision.ratios = ratios;
     }
     availableVCRevisionPayloads.push(revision);
@@ -60,7 +59,7 @@ export function getVCRevisionsData({
       'metric:vc90': shouldHaveVCmetric ? calculatedVCNext.VC['90'] : null,
       vcDetails: createVCDetails(calculatedVCNext, shouldHaveVCmetric)
     };
-    if (shouldHaveVCmetric && fg('platform_ufo_rev_ratios')) {
+    if (shouldHaveVCmetric) {
       revision.ratios = ratios;
     }
     availableVCRevisionPayloads.push(revision);

package/dist/es2019/vc/vc-observer-new/metric-calculator/abstract-base-vc-calculator.js

@@ -1,4 +1,3 @@
-import { fg } from '@atlaskit/platform-feature-flags';
 import { calculateTTVCPercentilesWithDebugInfo } from './percentile-calc';
 import getViewportHeight from './utils/get-viewport-height';
 import getViewportWidth from './utils/get-viewport-width';
@@ -241,9 +240,7 @@ export default class AbstractVCCalculatorBase {
       'metric:vc90': (_vcDetails$90$t = vcDetails === null || vcDetails === void 0 ? void 0 : (_vcDetails$ = vcDetails['90']) === null || _vcDetails$ === void 0 ? void 0 : _vcDetails$.t) !== null && _vcDetails$90$t !== void 0 ? _vcDetails$90$t : null,
       vcDetails: vcDetails !== null && vcDetails !== void 0 ? vcDetails : undefined
     };
-    if (fg('platform_ufo_rev_ratios')) {
-      result.ratios = this.calculateRatios(filteredEntries);
-    }
+    result.ratios = this.calculateRatios(filteredEntries);
     return result;
   }
 }

package/dist/es2019/vc/vc-observer-new/viewport-observer/utils/check-within-component-and-extract-child-props.js

@@ -1,7 +1,7 @@
 import { fg } from '@atlaskit/platform-feature-flags';
 import findReactFiber from './find-react-fiber';
 import getComponentName from './get-component-name';
-
+const DEFAULT_MAX_LEVEL = 20;
 // Using the React Fiber tree to traverse up the DOM and check if a node is within a specific component
 // and extract child component props if needed.
 export default function checkWithinComponentAndExtractChildProps(node, targetComponentName, childComponentConfig) {
@@ -9,11 +9,13 @@ export default function checkWithinComponentAndExtractChildProps(node, targetCom
   if (fg('platform_ufo_handle_non_react_element_for_3p')) {
     // Walk up the DOM tree to find React fiber (handles non-React-rendered elements)
     let currentElement = node;
-    while (currentElement && !fiber) {
+    let levelsTraversed = 0;
+    while (currentElement && !fiber && levelsTraversed < DEFAULT_MAX_LEVEL) {
       fiber = findReactFiber(currentElement);
       if (!fiber) {
         currentElement = currentElement.parentElement;
       }
+      levelsTraversed++;
     }
   } else {
     fiber = findReactFiber(node);

package/dist/esm/vc/vc-observer/getVCRevisionsData.js

@@ -2,7 +2,6 @@ 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; }
-import { fg } from '@atlaskit/platform-feature-flags';
 import { isVCRevisionEnabled } from '../../config';
 import { getPageVisibilityState } from '../../hidden-timing';
 var VCParts = ['25', '50', '75', '80', '85', '90', '95', '98', '99'];
@@ -57,7 +56,7 @@ export function getVCRevisionsData(_ref) {
       'metric:vc90': shouldHaveVCmetric ? calculatedVC.VC['90'] : null,
       vcDetails: createVCDetails(calculatedVC, shouldHaveVCmetric)
     };
-    if (shouldHaveVCmetric && fg('platform_ufo_rev_ratios')) {
+    if (shouldHaveVCmetric) {
       revision.ratios = ratios;
     }
     availableVCRevisionPayloads.push(revision);
@@ -69,7 +68,7 @@ export function getVCRevisionsData(_ref) {
       'metric:vc90': shouldHaveVCmetric ? calculatedVCNext.VC['90'] : null,
       vcDetails: createVCDetails(calculatedVCNext, shouldHaveVCmetric)
     };
-    if (shouldHaveVCmetric && fg('platform_ufo_rev_ratios')) {
+    if (shouldHaveVCmetric) {
       _revision.ratios = ratios;
     }
     availableVCRevisionPayloads.push(_revision);

package/dist/esm/vc/vc-observer-new/metric-calculator/abstract-base-vc-calculator.js

@@ -10,7 +10,6 @@ function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t =
 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; }
-import { fg } from '@atlaskit/platform-feature-flags';
 import { calculateTTVCPercentilesWithDebugInfo } from './percentile-calc';
 import getViewportHeight from './utils/get-viewport-height';
 import getViewportWidth from './utils/get-viewport-width';
@@ -345,9 +344,7 @@ var AbstractVCCalculatorBase = /*#__PURE__*/function () {
                 'metric:vc90': (_vcDetails$90$t = vcDetails === null || vcDetails === void 0 || (_vcDetails$ = vcDetails['90']) === null || _vcDetails$ === void 0 ? void 0 : _vcDetails$.t) !== null && _vcDetails$90$t !== void 0 ? _vcDetails$90$t : null,
                 vcDetails: vcDetails !== null && vcDetails !== void 0 ? vcDetails : undefined
               };
-              if (fg('platform_ufo_rev_ratios')) {
-                result.ratios = this.calculateRatios(filteredEntries);
-              }
+              result.ratios = this.calculateRatios(filteredEntries);
               return _context2.abrupt("return", result);
             case 13:
             case "end":

package/dist/esm/vc/vc-observer-new/viewport-observer/utils/check-within-component-and-extract-child-props.js

@@ -4,7 +4,7 @@ function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t =
 import { fg } from '@atlaskit/platform-feature-flags';
 import findReactFiber from './find-react-fiber';
 import getComponentName from './get-component-name';
-
+var DEFAULT_MAX_LEVEL = 20;
 // Using the React Fiber tree to traverse up the DOM and check if a node is within a specific component
 // and extract child component props if needed.
 export default function checkWithinComponentAndExtractChildProps(node, targetComponentName, childComponentConfig) {
@@ -12,11 +12,13 @@ export default function checkWithinComponentAndExtractChildProps(node, targetCom
   if (fg('platform_ufo_handle_non_react_element_for_3p')) {
     // Walk up the DOM tree to find React fiber (handles non-React-rendered elements)
     var currentElement = node;
-    while (currentElement && !fiber) {
+    var levelsTraversed = 0;
+    while (currentElement && !fiber && levelsTraversed < DEFAULT_MAX_LEVEL) {
       fiber = findReactFiber(currentElement);
       if (!fiber) {
         currentElement = currentElement.parentElement;
       }
+      levelsTraversed++;
     }
   } else {
     fiber = findReactFiber(node);

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/react-ufo",
-	"version": "4.1.4",
+	"version": "4.1.6",
 	"description": "Parts of React UFO that are publicly available",
 	"author": "Atlassian Pty Ltd",
 	"license": "Apache-2.0",
@@ -159,9 +159,6 @@
 		"platform_ufo_enable_events_observer": {
 			"type": "boolean"
 		},
-		"platform_ufo_rev_ratios": {
-			"type": "boolean"
-		},
 		"platform_ufo_ssr_size_field": {
 			"type": "boolean"
 		},