@atlaskit/renderer 118.6.7 → 118.6.9

package/CHANGELOG.md

@@ -1,5 +1,24 @@
 # @atlaskit/renderer
 
+## 118.6.9
+
+### Patch Changes
+
+- [#164686](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/pull-requests/164686)
+  [`4b2aaa6783398`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/4b2aaa6783398) -
+  Removed extra debug logging which was adding behind gate
+  platform_renderer_annotations_create_debug_logging
+- Updated dependencies
+
+## 118.6.8
+
+### Patch Changes
+
+- [#166323](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/pull-requests/166323)
+  [`51519a7335b4d`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/51519a7335b4d) -
+  [ux] Remove 1px margin top from inline macro wrapper that is causing hydration shift
+- Updated dependencies
+
 ## 118.6.7
 
 ### Patch Changes

package/dist/cjs/actions/index.js

@@ -22,6 +22,10 @@ var _matchesUtils = require("./matches-utils");
 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 RendererActions = exports.default = /*#__PURE__*/function () {
+  // Any kind of refence is allowed
+  // Ignored via go/ees005
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
   function RendererActions() {
     var initFromContext = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : false;
     (0, _classCallCheck2.default)(this, RendererActions);
@@ -29,55 +33,6 @@ var RendererActions = exports.default = /*#__PURE__*/function () {
     // This module can only be used when wrapped with
     // the <RendererContext> component for now.
     (0, _defineProperty2.default)(this, "initFromContext", false);
-    (0, _defineProperty2.default)(this, "debugAnalyticLoggingEnabled", false);
-    // TODO: EDITOR-595 - This method is temporary and should be removed once the following issue
-    // has been identified and fixed: https://atlassian.slack.com/archives/C08JK0WSCH5/p1745902609966999
-    // This is a copy of the original canApplyAnnotationOnRange. If that returns false this is run to figure out exactly why
-    // the annotation cannot be applied to the range.
-    (0, _defineProperty2.default)(this, "_privateWhyCannotApplyAnnotationOnRange", function (pos, doc, schema) {
-      var from = pos.from,
-        to = pos.to;
-      if (isNaN(from + to) || to - from <= 0 || to < 0 || from < 0) {
-        return 'position invalid';
-      }
-      var reasons = [];
-      doc.nodesBetween(from, to, function (node, _pos, parent) {
-        // Special exception for hardBreak nodes
-        if (schema.nodes.hardBreak === node.type) {
-          return false;
-        }
-
-        // For block elements or text nodes, we want to check
-        // if annotations are allowed inside this tree
-        // or if we're leaf and not text
-        if ((0, _platformFeatureFlags.fg)('editor_inline_comments_on_inline_nodes')) {
-          var isAllowedInlineNode = ['emoji', 'status', 'date', 'mention', 'inlineCard'].includes(node.type.name);
-          if (node.isInline && !node.isText && !isAllowedInlineNode) {
-            reasons.push("Err 1 - type: ".concat(node.type.name, ", isInline: ").concat(node.isInline, ", isText: ").concat(node.isText, ", isLeaf: ").concat(node.isLeaf));
-            return false;
-          } else if (node.isLeaf && !node.isText && !isAllowedInlineNode) {
-            reasons.push("Err 2 - type: ".concat(node.type.name, ", isInline: ").concat(node.isInline, ", isText: ").concat(node.isText, ", isLeaf: ").concat(node.isLeaf));
-            return false;
-          } else if (node.isText && !(parent !== null && parent !== void 0 && parent.type.allowsMarkType(schema.marks.annotation))) {
-            reasons.push("Err 3 - type: ".concat(node.type.name, ", isInline: ").concat(node.isInline, ", isText: ").concat(node.isText, ", isLeaf: ").concat(node.isLeaf, ", parent: ").concat(parent === null || parent === void 0 ? void 0 : parent.type.name));
-            return false;
-          }
-        } else {
-          if (node.isInline && !node.isText) {
-            reasons.push("Err 4 - type: ".concat(node.type.name, ", isInline: ").concat(node.isInline, ", isText: ").concat(node.isText, ", isLeaf: ").concat(node.isLeaf));
-            return false;
-          } else if (node.isLeaf && !node.isText) {
-            reasons.push("Err 5 - type: ".concat(node.type.name, ", isInline: ").concat(node.isInline, ", isText: ").concat(node.isText, ", isLeaf: ").concat(node.isLeaf));
-            return false;
-          } else if (node.isText && !(parent !== null && parent !== void 0 && parent.type.allowsMarkType(schema.marks.annotation))) {
-            reasons.push("Err 6 - type: ".concat(node.type.name, ", isInline: ").concat(node.isInline, ", isText: ").concat(node.isText, ", isLeaf: ").concat(node.isLeaf, ", parent: ").concat(parent === null || parent === void 0 ? void 0 : parent.type.name));
-            return false;
-          }
-        }
-        return true;
-      });
-      return reasons.join(', ');
-    });
     this.initFromContext = initFromContext;
     this.transformer = new _editorJsonTransformer.JSONTransformer();
   }
@@ -112,82 +67,23 @@ var RendererActions = exports.default = /*#__PURE__*/function () {
     key: "_privateValidatePositionsForAnnotation",
     value: function _privateValidatePositionsForAnnotation(from, to) {
       if (!this.doc || !this.schema) {
-        if (this.debugAnalyticLoggingEnabled && this.onAnalyticsEvent && (0, _platformFeatureFlags.fg)('platform_renderer_annotations_create_debug_logging')) {
-          this.onAnalyticsEvent({
-            // @ts-ignore
-            action: 'failed',
-            // @ts-ignore
-            actionSubject: 'applyAnnotation',
-            // @ts-ignore
-            actionSubjectId: 'inlineCommentFailureReason',
-            eventType: _analytics.EVENT_TYPE.OPERATIONAL,
-            attributes: {
-              // @ts-ignore
-              reason: "Annotation Position validation failed - Missing doc or schema",
-              commentsOnInlineNodesAllowed: (0, _platformFeatureFlags.fg)('editor_inline_comments_on_inline_nodes'),
-              isDocValid: !!this.doc,
-              isSchemaValid: !!this.schema
-            }
-          });
-        }
         return false;
       }
       var currentSelection = _state.TextSelection.create(this.doc, from, to);
       if ((0, _utils.isEmptyTextSelection)(currentSelection, this.schema)) {
-        if (this.debugAnalyticLoggingEnabled && this.onAnalyticsEvent && (0, _platformFeatureFlags.fg)('platform_renderer_annotations_create_debug_logging')) {
-          this.onAnalyticsEvent({
-            // @ts-ignore
-            action: 'failed',
-            // @ts-ignore
-            actionSubject: 'applyAnnotation',
-            // @ts-ignore
-            actionSubjectId: 'inlineCommentFailureReason',
-            eventType: _analytics.EVENT_TYPE.OPERATIONAL,
-            attributes: {
-              // @ts-ignore
-              reason: "Annotation Position validation - Empty Text Selection",
-              commentsOnInlineNodesAllowed: (0, _platformFeatureFlags.fg)('editor_inline_comments_on_inline_nodes'),
-              from: from,
-              to: to
-            }
-          });
-        }
         return false;
       }
       var result = (0, _utils.canApplyAnnotationOnRange)({
         from: from,
         to: to
       }, this.doc, this.schema);
-      if (this.debugAnalyticLoggingEnabled && !result && this.onAnalyticsEvent && (0, _platformFeatureFlags.fg)('platform_renderer_annotations_create_debug_logging')) {
-        this.onAnalyticsEvent({
-          // @ts-ignore
-          action: 'failed',
-          // @ts-ignore
-          actionSubject: 'applyAnnotation',
-          // @ts-ignore
-          actionSubjectId: 'inlineCommentFailureReason',
-          eventType: _analytics.EVENT_TYPE.OPERATIONAL,
-          attributes: {
-            // @ts-ignore
-            reason: "Annotation canApplyAnnotationOnRange failed",
-            details: this._privateWhyCannotApplyAnnotationOnRange({
-              from: from,
-              to: to
-            }, this.doc, this.schema),
-            commentsOnInlineNodesAllowed: (0, _platformFeatureFlags.fg)('editor_inline_comments_on_inline_nodes'),
-            from: from,
-            to: to
-          }
-        });
-      }
       return result;
     }
+
+    //#endregion
   }, {
     key: "deleteAnnotation",
-    value:
-    //#endregion
-
-    function deleteAnnotation(annotationId, annotationType) {
+    value: function deleteAnnotation(annotationId, annotationType) {
       if (!this.doc || !this.schema || !this.schema.marks.annotation) {
         return false;
       }
@@ -338,24 +234,6 @@ var RendererActions = exports.default = /*#__PURE__*/function () {
     key: "isValidAnnotationPosition",
     value: function isValidAnnotationPosition(pos) {
       if (!pos || !this.doc) {
-        if (this.debugAnalyticLoggingEnabled && this.onAnalyticsEvent && (0, _platformFeatureFlags.fg)('platform_renderer_annotations_create_debug_logging')) {
-          this.onAnalyticsEvent({
-            // @ts-ignore
-            action: 'failed',
-            // @ts-ignore
-            actionSubject: 'applyAnnotation',
-            // @ts-ignore
-            actionSubjectId: 'inlineCommentFailureReason',
-            eventType: _analytics.EVENT_TYPE.OPERATIONAL,
-            attributes: {
-              // @ts-ignore
-              reason: "Annotation Position validation failed - Missing doc or position",
-              commentsOnInlineNodesAllowed: (0, _platformFeatureFlags.fg)('editor_inline_comments_on_inline_nodes'),
-              isPosValid: !!pos,
-              isDocValid: !!this.doc
-            }
-          });
-        }
         return false;
       }
       return this._privateValidatePositionsForAnnotation(pos.from, pos.to);
@@ -417,25 +295,6 @@ var RendererActions = exports.default = /*#__PURE__*/function () {
     key: "applyAnnotation",
     value: function applyAnnotation(pos, annotation) {
       if (!this.doc || !pos || !this.schema) {
-        if (this.onAnalyticsEvent && (0, _platformFeatureFlags.fg)('platform_renderer_annotations_create_debug_logging')) {
-          this.onAnalyticsEvent({
-            // @ts-ignore
-            action: 'failed',
-            // @ts-ignore
-            actionSubject: 'applyAnnotation',
-            // @ts-ignore
-            actionSubjectId: 'inlineCommentFailureReason',
-            eventType: _analytics.EVENT_TYPE.OPERATIONAL,
-            attributes: {
-              // @ts-ignore
-              reason: "Annotation applyAnnotation failed - Missing doc, position or schema",
-              commentsOnInlineNodesAllowed: (0, _platformFeatureFlags.fg)('editor_inline_comments_on_inline_nodes'),
-              isPosValid: !!pos,
-              isDocValid: !!this.doc,
-              isSchemaValid: !!this.schema
-            }
-          });
-        }
         return false;
       }
       var from = pos.from,
@@ -469,22 +328,6 @@ var RendererActions = exports.default = /*#__PURE__*/function () {
         doc = _step$apply2.doc,
         failed = _step$apply2.failed;
       if (failed || !doc) {
-        if (this.onAnalyticsEvent && (0, _platformFeatureFlags.fg)('platform_renderer_annotations_create_debug_logging')) {
-          this.onAnalyticsEvent({
-            // @ts-ignore
-            action: 'failed',
-            // @ts-ignore
-            actionSubject: 'applyAnnotation',
-            // @ts-ignore
-            actionSubjectId: 'inlineCommentFailureReason',
-            eventType: _analytics.EVENT_TYPE.OPERATIONAL,
-            attributes: {
-              // @ts-ignore
-              reason: "Annotation applyAnnotation failed - ".concat(failed),
-              commentsOnInlineNodesAllowed: (0, _platformFeatureFlags.fg)('editor_inline_comments_on_inline_nodes')
-            }
-          });
-        }
         return false;
       }
       var originalSelection = doc.textBetween(from, to);
@@ -545,14 +388,5 @@ var RendererActions = exports.default = /*#__PURE__*/function () {
         schema: this.schema
       }, annotationId);
     }
-
-    /**
-     * This is a temporary method used for controlling when extra analytics shoulw be logged.
-     */
-  }, {
-    key: "_setDebugLogging",
-    value: function _setDebugLogging(enabled) {
-      this.debugAnalyticLoggingEnabled = enabled;
-    }
   }]);
 }();

package/dist/cjs/ui/ExtensionRenderer.js

@@ -19,21 +19,18 @@ var _utils = require("@atlaskit/editor-common/utils");
  */
 // eslint-disable-next-line @atlaskit/ui-styling-standard/use-compiled -- Ignored via go/DSP-18766
 
-// When running on server-side emotion will generate style tags before elements.
-// This caused packages/editor/editor-common/src/styles/shared/block-marks.ts to override the margin-top.
-// However as soon as the styles are extracted to <head> it adds back the margin.
-// The timing is tricky as it happens to be when UFO collects the dimension for the placeholder for TTVC calculation.
-// This resulted 1px mismatch on the image. Further cause everything on page to shift by 1px.
-var forceMarginTopStyle = (0, _react.css)({
-  // eslint-disable-next-line @atlaskit/ui-styling-standard/no-important-styles
-  marginTop: "1px !important"
-});
 var inlineExtensionStyle = (0, _react.css)({
   display: 'inline-block',
   maxWidth: '100%',
   verticalAlign: 'middle',
+  // 0px margin top is important here.
+  // When running on server-side emotion will generate style tags before elements.
+  // This caused packages/editor/editor-common/src/styles/shared/block-marks.ts to override the margin-top.
+  // However as soon as the styles are extracted to <head> it adds back the margin.
+  // The timing is tricky as it happens to be when UFO collects the dimension for the placeholder for TTVC calculation.
+  // This resulted 1px mismatch on the image. Further cause everything on page to shift by 1px.
   // es-lint-disable-next-line @atlaskit/design-system/ensure-design-token-usage
-  margin: "1px 1px ".concat("var(--ds-space-050, 4px)"),
+  margin: "0px 1px ".concat("var(--ds-space-050, 4px)"),
   // eslint-disable-next-line @atlaskit/ui-styling-standard/no-nested-selectors -- Ignored via go/DSP-18766
   '& .rich-media-item': {
     maxWidth: '100%'
@@ -137,12 +134,10 @@ function ExtensionRenderer(props) {
   });
 }
 var InlineNodeRendererWrapper = exports.InlineNodeRendererWrapper = function InlineNodeRendererWrapper(_ref) {
-  var _ref$forceMarginTop = _ref.forceMarginTop,
-    forceMarginTop = _ref$forceMarginTop === void 0 ? false : _ref$forceMarginTop,
-    children = _ref.children;
+  var children = _ref.children;
   return (0, _react.jsx)("div", {
     // eslint-disable-next-line @atlaskit/ui-styling-standard/no-classname-prop -- Ignored via go/DSP-18766
     className: "inline-extension-renderer",
-    css: [inlineExtensionStyle, forceMarginTop && forceMarginTopStyle]
+    css: inlineExtensionStyle
   }, children);
 };

package/dist/cjs/ui/Renderer/index.js

@@ -62,7 +62,7 @@ function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t =
 var NORMAL_SEVERITY_THRESHOLD = exports.NORMAL_SEVERITY_THRESHOLD = 2000;
 var DEGRADED_SEVERITY_THRESHOLD = exports.DEGRADED_SEVERITY_THRESHOLD = 3000;
 var packageName = "@atlaskit/renderer";
-var packageVersion = "118.6.7";
+var packageVersion = "118.6.9";
 var setAsQueryContainerStyles = (0, _react2.css)({
   containerName: 'ak-renderer-wrapper',
   containerType: 'inline-size'
@@ -422,7 +422,8 @@ var RendererFunctionalComponent = exports.RendererFunctionalComponent = function
       ssr: (_props$media = props.media) === null || _props$media === void 0 ? void 0 : _props$media.ssr,
       isInsideOfInlineExtension: props.isInsideOfInlineExtension,
       isTopLevelRenderer: rendererContext.isTopLevelRenderer,
-      shouldRemoveEmptySpaceAroundContent: props.shouldRemoveEmptySpaceAroundContent
+      shouldRemoveEmptySpaceAroundContent: props.shouldRemoveEmptySpaceAroundContent,
+      allowRendererContainerStyles: props.allowRendererContainerStyles
     }, props.enableSsrInlineScripts || props.noOpSSRInlineScript ? (0, _react2.jsx)(_breakoutSsr.BreakoutSSRInlineScript, {
       noOpSSRInlineScript: Boolean(props.noOpSSRInlineScript)
     }) : null, (0, _react2.jsx)(RendererActionsInternalUpdater, {
@@ -453,7 +454,8 @@ var RendererFunctionalComponent = exports.RendererFunctionalComponent = function
       onClick: function onClick(event) {
         return handleWrapperOnClick(event, props, mouseDownSelection);
       },
-      isTopLevelRenderer: rendererContext.isTopLevelRenderer
+      isTopLevelRenderer: rendererContext.isTopLevelRenderer,
+      allowRendererContainerStyles: props.allowRendererContainerStyles
     }, (0, _react2.jsx)(_ui.UnsupportedBlock, null));
   }
 };
@@ -545,7 +547,8 @@ var RendererWrapper = /*#__PURE__*/_react.default.memo(function (props) {
     ssr = props.ssr,
     isInsideOfInlineExtension = props.isInsideOfInlineExtension,
     allowTableResizing = props.allowTableResizing,
-    isTopLevelRenderer = props.isTopLevelRenderer;
+    isTopLevelRenderer = props.isTopLevelRenderer,
+    allowRendererContainerStyles = props.allowRendererContainerStyles;
   var createTelepointer = function createTelepointer() {
     var telepointer = document.createElement('span');
     telepointer.textContent = "\u200B";
@@ -620,20 +623,32 @@ var RendererWrapper = /*#__PURE__*/_react.default.memo(function (props) {
     allowColumnSorting: !!allowColumnSorting,
     useBlockRenderForCodeBlock: useBlockRenderForCodeBlock,
     allowAnnotations: props.allowAnnotations,
-    allowTableResizing: allowTableResizing
+    allowTableResizing: allowTableResizing,
+    allowRendererContainerStyles: allowRendererContainerStyles
   }, children))));
 
   // We can only make the wrapper div query container when we have a known width.
   // This is also required for SSR to work correctly. As WidthProvider/WithConsumer will not have the correct width during SSR.
   //
+
+  // allowRendererContainerStyles is not needed for comment container styling as container should always be set for comments
+  // eslint-disable-next-line @atlaskit/platform/ensure-feature-flag-prefix
+  if (appearance === 'comment' && isTopLevelRenderer && (0, _platformFeatureFlags.fg)('platform-ssr-table-resize')) {
+    return (0, _react2.jsx)("div", {
+      css: setAsQueryContainerStyles
+    }, renderer);
+  }
+
   // We are setting this wrapper div as query container conditionally.
   // Only apply container-type = inline-size when having a known width in full-page/full-width/comment mode.
   // Otherwise when appearance is unspecified the renderer size is decided by the content.
   // In this case we can't set the container-type = inline-size as it will collapse width to 0.
-  return (appearance === 'full-page' || appearance === 'full-width' || appearance === 'comment') &&
+  return (appearance === 'full-page' || appearance === 'full-width') &&
   // In case of having excerpt-include on page there are multiple renderers nested.
   // Make sure only the root renderer is set to be query container.
-  isTopLevelRenderer && (0, _platformFeatureFlags.fg)('platform-ssr-table-resize') ? (0, _react2.jsx)("div", {
+  isTopLevelRenderer && allowRendererContainerStyles &&
+  // eslint-disable-next-line @atlaskit/platform/ensure-feature-flag-prefix
+  (0, _platformFeatureFlags.fg)('platform-ssr-table-resize') ? (0, _react2.jsx)("div", {
     css: setAsQueryContainerStyles
   }, renderer) : renderer;
 });

package/dist/cjs/ui/annotations/selection/mounter.js

@@ -70,28 +70,8 @@ var SelectionInlineCommentMounter = exports.SelectionInlineCommentMounter = /*#_
     // !!! at this point, the documentPosition can be the wrong position if the user select something else
     var positionToAnnotate = selectionDraftDocumentPosition || documentPosition;
     if (!positionToAnnotate || !applyAnnotation) {
-      // TODO: EDITOR-595 - This analytic event is temporary and should be removed once the following issue
-      // has been identified and fixed: https://atlassian.slack.com/archives/C08JK0WSCH5/p1745902609966999
-      if (createAnalyticsEvent && (0, _platformFeatureFlags.fg)('platform_renderer_annotations_create_debug_logging')) {
-        createAnalyticsEvent({
-          action: 'failed',
-          actionSubject: 'applyAnnotation',
-          actionSubjectId: 'inlineCommentFailureReason',
-          attributes: {
-            reason: 'Annotation Position invalid',
-            draftDocumentPosition: selectionDraftDocumentPosition,
-            documentPosition: documentPosition,
-            applyAnnotation: !!applyAnnotation,
-            isDraftPositionFixEnabled: true
-          },
-          eventType: _analytics.EVENT_TYPE.OPERATIONAL
-        }).fire(_types.FabricChannel.editor);
-      }
       return false;
     }
-    if ((0, _platformFeatureFlags.fg)('platform_renderer_annotations_create_debug_logging')) {
-      actions._setDebugLogging(true);
-    }
 
     // Evaluate position validity when the user commits the position to be annotated
     var isCreateAllowedOnPosition = actions.isValidAnnotationPosition(positionToAnnotate);
@@ -113,13 +93,7 @@ var SelectionInlineCommentMounter = exports.SelectionInlineCommentMounter = /*#_
         eventType: _analytics.EVENT_TYPE.TRACK
       }).fire(_types.FabricChannel.editor);
     }
-    if ((0, _platformFeatureFlags.fg)('platform_renderer_annotations_create_debug_logging')) {
-      var result = applyAnnotation(positionToAnnotate, annotation);
-      actions._setDebugLogging(false);
-      return result;
-    } else {
-      return applyAnnotation(positionToAnnotate, annotation);
-    }
+    return applyAnnotation(positionToAnnotate, annotation);
   }, [actions, documentPosition, applyAnnotation, createAnalyticsEvent, inlineNodeTypes, selectionDraftDocumentPosition]);
   var createIndexCallback = (0, _react.useCallback)(function () {
     if (!documentPosition || !generateIndexMatch) {

package/dist/es2019/actions/index.js

@@ -10,62 +10,15 @@ import { createAnnotationStep, getPosFromRange } from '../steps';
 import { getRendererRangeInlineNodeNames } from './get-renderer-range-inline-node-names';
 import { getIndexMatch } from './matches-utils';
 export default class RendererActions {
+  // Any kind of refence is allowed
+  // Ignored via go/ees005
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+
   constructor(initFromContext = false) {
     // This is our psuedo feature flag for now
     // This module can only be used when wrapped with
     // the <RendererContext> component for now.
     _defineProperty(this, "initFromContext", false);
-    _defineProperty(this, "debugAnalyticLoggingEnabled", false);
-    // TODO: EDITOR-595 - This method is temporary and should be removed once the following issue
-    // has been identified and fixed: https://atlassian.slack.com/archives/C08JK0WSCH5/p1745902609966999
-    // This is a copy of the original canApplyAnnotationOnRange. If that returns false this is run to figure out exactly why
-    // the annotation cannot be applied to the range.
-    _defineProperty(this, "_privateWhyCannotApplyAnnotationOnRange", (pos, doc, schema) => {
-      const {
-        from,
-        to
-      } = pos;
-      if (isNaN(from + to) || to - from <= 0 || to < 0 || from < 0) {
-        return 'position invalid';
-      }
-      const reasons = [];
-      doc.nodesBetween(from, to, (node, _pos, parent) => {
-        // Special exception for hardBreak nodes
-        if (schema.nodes.hardBreak === node.type) {
-          return false;
-        }
-
-        // For block elements or text nodes, we want to check
-        // if annotations are allowed inside this tree
-        // or if we're leaf and not text
-        if (fg('editor_inline_comments_on_inline_nodes')) {
-          const isAllowedInlineNode = ['emoji', 'status', 'date', 'mention', 'inlineCard'].includes(node.type.name);
-          if (node.isInline && !node.isText && !isAllowedInlineNode) {
-            reasons.push(`Err 1 - type: ${node.type.name}, isInline: ${node.isInline}, isText: ${node.isText}, isLeaf: ${node.isLeaf}`);
-            return false;
-          } else if (node.isLeaf && !node.isText && !isAllowedInlineNode) {
-            reasons.push(`Err 2 - type: ${node.type.name}, isInline: ${node.isInline}, isText: ${node.isText}, isLeaf: ${node.isLeaf}`);
-            return false;
-          } else if (node.isText && !(parent !== null && parent !== void 0 && parent.type.allowsMarkType(schema.marks.annotation))) {
-            reasons.push(`Err 3 - type: ${node.type.name}, isInline: ${node.isInline}, isText: ${node.isText}, isLeaf: ${node.isLeaf}, parent: ${parent === null || parent === void 0 ? void 0 : parent.type.name}`);
-            return false;
-          }
-        } else {
-          if (node.isInline && !node.isText) {
-            reasons.push(`Err 4 - type: ${node.type.name}, isInline: ${node.isInline}, isText: ${node.isText}, isLeaf: ${node.isLeaf}`);
-            return false;
-          } else if (node.isLeaf && !node.isText) {
-            reasons.push(`Err 5 - type: ${node.type.name}, isInline: ${node.isInline}, isText: ${node.isText}, isLeaf: ${node.isLeaf}`);
-            return false;
-          } else if (node.isText && !(parent !== null && parent !== void 0 && parent.type.allowsMarkType(schema.marks.annotation))) {
-            reasons.push(`Err 6 - type: ${node.type.name}, isInline: ${node.isInline}, isText: ${node.isText}, isLeaf: ${node.isLeaf}, parent: ${parent === null || parent === void 0 ? void 0 : parent.type.name}`);
-            return false;
-          }
-        }
-        return true;
-      });
-      return reasons.join(', ');
-    });
     this.initFromContext = initFromContext;
     this.transformer = new JSONTransformer();
   }
@@ -94,76 +47,19 @@ export default class RendererActions {
    */
   _privateValidatePositionsForAnnotation(from, to) {
     if (!this.doc || !this.schema) {
-      if (this.debugAnalyticLoggingEnabled && this.onAnalyticsEvent && fg('platform_renderer_annotations_create_debug_logging')) {
-        this.onAnalyticsEvent({
-          // @ts-ignore
-          action: 'failed',
-          // @ts-ignore
-          actionSubject: 'applyAnnotation',
-          // @ts-ignore
-          actionSubjectId: 'inlineCommentFailureReason',
-          eventType: EVENT_TYPE.OPERATIONAL,
-          attributes: {
-            // @ts-ignore
-            reason: `Annotation Position validation failed - Missing doc or schema`,
-            commentsOnInlineNodesAllowed: fg('editor_inline_comments_on_inline_nodes'),
-            isDocValid: !!this.doc,
-            isSchemaValid: !!this.schema
-          }
-        });
-      }
       return false;
     }
     const currentSelection = TextSelection.create(this.doc, from, to);
     if (isEmptyTextSelection(currentSelection, this.schema)) {
-      if (this.debugAnalyticLoggingEnabled && this.onAnalyticsEvent && fg('platform_renderer_annotations_create_debug_logging')) {
-        this.onAnalyticsEvent({
-          // @ts-ignore
-          action: 'failed',
-          // @ts-ignore
-          actionSubject: 'applyAnnotation',
-          // @ts-ignore
-          actionSubjectId: 'inlineCommentFailureReason',
-          eventType: EVENT_TYPE.OPERATIONAL,
-          attributes: {
-            // @ts-ignore
-            reason: `Annotation Position validation - Empty Text Selection`,
-            commentsOnInlineNodesAllowed: fg('editor_inline_comments_on_inline_nodes'),
-            from,
-            to
-          }
-        });
-      }
       return false;
     }
     const result = canApplyAnnotationOnRange({
       from,
       to
     }, this.doc, this.schema);
-    if (this.debugAnalyticLoggingEnabled && !result && this.onAnalyticsEvent && fg('platform_renderer_annotations_create_debug_logging')) {
-      this.onAnalyticsEvent({
-        // @ts-ignore
-        action: 'failed',
-        // @ts-ignore
-        actionSubject: 'applyAnnotation',
-        // @ts-ignore
-        actionSubjectId: 'inlineCommentFailureReason',
-        eventType: EVENT_TYPE.OPERATIONAL,
-        attributes: {
-          // @ts-ignore
-          reason: `Annotation canApplyAnnotationOnRange failed`,
-          details: this._privateWhyCannotApplyAnnotationOnRange({
-            from,
-            to
-          }, this.doc, this.schema),
-          commentsOnInlineNodesAllowed: fg('editor_inline_comments_on_inline_nodes'),
-          from,
-          to
-        }
-      });
-    }
     return result;
   }
+
   //#endregion
 
   deleteAnnotation(annotationId, annotationType) {
@@ -314,24 +210,6 @@ export default class RendererActions {
   }
   isValidAnnotationPosition(pos) {
     if (!pos || !this.doc) {
-      if (this.debugAnalyticLoggingEnabled && this.onAnalyticsEvent && fg('platform_renderer_annotations_create_debug_logging')) {
-        this.onAnalyticsEvent({
-          // @ts-ignore
-          action: 'failed',
-          // @ts-ignore
-          actionSubject: 'applyAnnotation',
-          // @ts-ignore
-          actionSubjectId: 'inlineCommentFailureReason',
-          eventType: EVENT_TYPE.OPERATIONAL,
-          attributes: {
-            // @ts-ignore
-            reason: `Annotation Position validation failed - Missing doc or position`,
-            commentsOnInlineNodesAllowed: fg('editor_inline_comments_on_inline_nodes'),
-            isPosValid: !!pos,
-            isDocValid: !!this.doc
-          }
-        });
-      }
       return false;
     }
     return this._privateValidatePositionsForAnnotation(pos.from, pos.to);
@@ -389,25 +267,6 @@ export default class RendererActions {
   }
   applyAnnotation(pos, annotation) {
     if (!this.doc || !pos || !this.schema) {
-      if (this.onAnalyticsEvent && fg('platform_renderer_annotations_create_debug_logging')) {
-        this.onAnalyticsEvent({
-          // @ts-ignore
-          action: 'failed',
-          // @ts-ignore
-          actionSubject: 'applyAnnotation',
-          // @ts-ignore
-          actionSubjectId: 'inlineCommentFailureReason',
-          eventType: EVENT_TYPE.OPERATIONAL,
-          attributes: {
-            // @ts-ignore
-            reason: `Annotation applyAnnotation failed - Missing doc, position or schema`,
-            commentsOnInlineNodesAllowed: fg('editor_inline_comments_on_inline_nodes'),
-            isPosValid: !!pos,
-            isDocValid: !!this.doc,
-            isSchemaValid: !!this.schema
-          }
-        });
-      }
       return false;
     }
     const {
@@ -446,22 +305,6 @@ export default class RendererActions {
       failed
     } = step.apply(this.doc);
     if (failed || !doc) {
-      if (this.onAnalyticsEvent && fg('platform_renderer_annotations_create_debug_logging')) {
-        this.onAnalyticsEvent({
-          // @ts-ignore
-          action: 'failed',
-          // @ts-ignore
-          actionSubject: 'applyAnnotation',
-          // @ts-ignore
-          actionSubjectId: 'inlineCommentFailureReason',
-          eventType: EVENT_TYPE.OPERATIONAL,
-          attributes: {
-            // @ts-ignore
-            reason: `Annotation applyAnnotation failed - ${failed}`,
-            commentsOnInlineNodesAllowed: fg('editor_inline_comments_on_inline_nodes')
-          }
-        });
-      }
       return false;
     }
     const originalSelection = doc.textBetween(from, to);
@@ -523,11 +366,4 @@ export default class RendererActions {
       schema: this.schema
     }, annotationId);
   }
-
-  /**
-   * This is a temporary method used for controlling when extra analytics shoulw be logged.
-   */
-  _setDebugLogging(enabled) {
-    this.debugAnalyticLoggingEnabled = enabled;
-  }
 }