@atlaskit/editor-plugin-annotation 23.0.0 → 23.0.2

package/CHANGELOG.md

@@ -1,5 +1,26 @@
 # @atlaskit/editor-plugin-annotation
 
+## 23.0.2
+
+### Patch Changes
+
+- [`d61565965ce82`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/d61565965ce82) -
+  Clean up experiment `platform_editor_annotations_sync_on_docchange`
+- Updated dependencies
+
+## 23.0.1
+
+### Patch Changes
+
+- [`45cf8c5ab1b31`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/45cf8c5ab1b31) -
+  Enable gated comments on MAUI charts in the Confluence editor, including the editor draft flow for
+  native-embed extension nodes. With `cc_maui_annotations_on_extensions` enabled, eligible extension
+  block selections can start a draft even when whitespace-only text comments are disallowed. The
+  extension-only DOM anchor fallback also requires this gate; existing text/media and gate-off
+  anchor behavior is preserved. The native-embed toolbar uses the same gate for its block-comment
+  opening hint.
+- Updated dependencies
+
 ## 23.0.0
 
 ### Patch Changes

package/dist/cjs/annotationPlugin.js

@@ -97,11 +97,11 @@ var annotationPlugin = exports.annotationPlugin = function annotationPlugin(_ref
       stripNonExistingAnnotations: _utils.stripNonExistingAnnotations,
       applyInlineCommentDraft: function applyInlineCommentDraft(annotationId) {
         var _api$analytics;
-        return (0, _editorCommands.createAnnotation)(api === null || api === void 0 || (_api$analytics = api.analytics) === null || _api$analytics === void 0 ? void 0 : _api$analytics.actions, api)(annotationId, _annotation.AnnotationTypes.INLINE_COMMENT, annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.supportedBlockNodes);
+        return (0, _editorCommands.createAnnotation)(api === null || api === void 0 || (_api$analytics = api.analytics) === null || _api$analytics === void 0 ? void 0 : _api$analytics.actions, api)(annotationId, _annotation.AnnotationTypes.INLINE_COMMENT, annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.supportedBlockNodes, annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.isBlockNodeSupported);
       },
       removeInlineCommentAnnotation: function removeInlineCommentAnnotation(annotationId) {
         var _api$analytics2, _annotationProviders$;
-        return (0, _editorCommands.removeInlineCommentFromDoc)(api === null || api === void 0 || (_api$analytics2 = api.analytics) === null || _api$analytics2 === void 0 ? void 0 : _api$analytics2.actions)(annotationId, (_annotationProviders$ = annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.supportedBlockNodes) !== null && _annotationProviders$ !== void 0 ? _annotationProviders$ : []);
+        return (0, _editorCommands.removeInlineCommentFromDoc)(api === null || api === void 0 || (_api$analytics2 = api.analytics) === null || _api$analytics2 === void 0 ? void 0 : _api$analytics2.actions)(annotationId, (_annotationProviders$ = annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.supportedBlockNodes) !== null && _annotationProviders$ !== void 0 ? _annotationProviders$ : [], annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.isBlockNodeSupported);
       },
       setInlineCommentDraftState: (0, _editorCommands.setInlineCommentDraftState)(api === null || api === void 0 || (_api$analytics3 = api.analytics) === null || _api$analytics3 === void 0 ? void 0 : _api$analytics3.actions, annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.supportedBlockNodes, undefined, annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.isBlockNodeSupported),
       showCommentForBlockNode: (0, _editorCommands.showInlineCommentForBlockNode)(annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.supportedBlockNodes, annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.isBlockNodeSupported),

package/dist/cjs/pm-plugins/plugin-factory.js

@@ -9,20 +9,10 @@ var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/de
 var _utils = require("@atlaskit/editor-common/utils");
 var _state = require("@atlaskit/editor-prosemirror/state");
 var _view = require("@atlaskit/editor-prosemirror/view");
-var _expValEquals = require("@atlaskit/tmp-editor-statsig/exp-val-equals");
 var _reducer = _interopRequireDefault(require("./reducer"));
 var _utils2 = require("./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 handleDocChanged = function handleDocChanged(tr, prevPluginState) {
-  if (!tr.getMeta('replaceDocument')) {
-    return getSelectionChangedHandler(false)(tr, prevPluginState);
-  }
-  return _objectSpread(_objectSpread({}, prevPluginState), {}, {
-    dirtyAnnotations: true
-  });
-};
-
 /**
  * Creates a handleDocChanged function with its own deleted annotations cache.
  * This ensures each editor instance has its own cache, avoiding cross-contamination.
@@ -208,16 +198,9 @@ var getSelectionChangedHandler = function getSelectionChangedHandler(reopenComme
 
 // Create the handler with cache once at module level
 var handleDocChangedWithSync = createHandleDocChanged();
-var getDocChangedHandler = function getDocChangedHandler(tr, prevPluginState) {
-  // Check feature flag at runtime to support test variants
-  if ((0, _expValEquals.expValEquals)('platform_editor_annotations_sync_on_docchange', 'isEnabled', true)) {
-    return handleDocChangedWithSync(tr, prevPluginState);
-  }
-  return handleDocChanged(tr, prevPluginState);
-};
 var dest = (0, _utils.pluginFactory)(_utils2.inlineCommentPluginKey, _reducer.default, {
   onSelectionChanged: getSelectionChangedHandler(true),
-  onDocChanged: getDocChangedHandler,
+  onDocChanged: handleDocChangedWithSync,
   mapping: function mapping(tr, pluginState, editorState) {
     var draftDecorationSet = pluginState.draftDecorationSet,
       bookmark = pluginState.bookmark;

package/dist/cjs/pm-plugins/utils.js

@@ -331,7 +331,11 @@ var isSelectionValid = exports.isSelectionValid = function isSelectionValid(stat
   if (containsInvalidNodes) {
     return _types.AnnotationSelectionType.DISABLED;
   }
-  if (disallowOnWhitespace && hasInvalidWhitespaceNode(selection, state.schema)) {
+
+  // Extension content lives outside the editor document, so an eligible whole-block
+  // selection must not be rejected for having no commentable text descendants.
+  var isSupportedExtensionSelection = (0, _fg.fg)('cc_maui_annotations_on_extensions') && isSupportedBlockNodeSelection && selection instanceof _state.NodeSelection && selection.node.type.name === 'extension';
+  if (disallowOnWhitespace && !isSupportedExtensionSelection && hasInvalidWhitespaceNode(selection, state.schema)) {
     return _types.AnnotationSelectionType.INVALID;
   }
   if (!(selection instanceof _state.NodeSelection) && (0, _utils.isEmptyTextSelection)(selection, state.schema)) {

package/dist/cjs/ui/InlineCommentView.js

@@ -13,7 +13,9 @@ var _hooks = require("@atlaskit/editor-common/hooks");
 var _useSharedPluginStateSelector = require("@atlaskit/editor-common/use-shared-plugin-state-selector");
 var _utils = require("@atlaskit/editor-common/utils");
 var _editorPluginConnectivity = require("@atlaskit/editor-plugin-connectivity");
+var _state = require("@atlaskit/editor-prosemirror/state");
 var _utils2 = require("@atlaskit/editor-prosemirror/utils");
+var _fg = require("@atlaskit/platform-feature-flags/fg");
 var _editorCommands = require("../editor-commands");
 var _utils3 = require("../pm-plugins/utils");
 var _types = require("../types");
@@ -69,14 +71,36 @@ function InlineCommentView(_ref) {
   var annotationsList = (0, _utils3.getAllAnnotations)(editorView.state.doc);
   var selection = (0, _utils3.getSelectionPositions)(state, bookmark);
   var position = findPosForDOM(selection);
+  var isExtensionSelection = selection instanceof _state.NodeSelection && selection.node.type.name === 'extension';
   var dom;
+  var domLookupError;
   try {
-    // Ignored via go/ees005
-    // eslint-disable-next-line @atlaskit/editor/no-as-casting
-    dom = (0, _utils2.findDomRefAtPos)(position, editorView.domAtPos.bind(editorView));
+    var domRef = (0, _utils2.findDomRefAtPos)(position, editorView.domAtPos.bind(editorView));
+    if ((0, _fg.fg)('cc_maui_annotations_on_extensions') && isExtensionSelection) {
+      if (domRef instanceof HTMLElement) {
+        dom = domRef;
+      }
+    } else {
+      // Preserve the existing text/media and gate-off anchor lookup.
+      // eslint-disable-next-line @atlaskit/editor/no-as-casting
+      dom = domRef;
+    }
   } catch (error) {
+    domLookupError = error;
+  }
+
+  // Native embed extension nodes are rendered by a node view. Depending on
+  // the wrapper, domAtPos can return no element (or throw) even though
+  // EditorView.nodeDOM can. Use that direct lookup as a block-comment anchor.
+  if ((0, _fg.fg)('cc_maui_annotations_on_extensions') && !dom && isExtensionSelection) {
+    var nodeDom = editorView.nodeDOM(selection.from);
+    if (nodeDom instanceof HTMLElement) {
+      dom = nodeDom;
+    }
+  }
+  if (!dom && domLookupError) {
     // eslint-disable-next-line no-console
-    console.warn(error);
+    console.warn(domLookupError);
     if (dispatchAnalyticsEvent) {
       var payload = {
         action: _analytics.ACTION.ERRORED,
@@ -87,7 +111,7 @@ function InlineCommentView(_ref) {
           selection: selection.toJSON(),
           position: position,
           docSize: editorView.state.doc.nodeSize,
-          error: error.toString()
+          error: domLookupError.toString()
         }
       };
       dispatchAnalyticsEvent(payload);

package/dist/es2019/annotationPlugin.js

@@ -65,11 +65,11 @@ export const annotationPlugin = ({
       stripNonExistingAnnotations,
       applyInlineCommentDraft: annotationId => {
         var _api$analytics;
-        return createAnnotation(api === null || api === void 0 ? void 0 : (_api$analytics = api.analytics) === null || _api$analytics === void 0 ? void 0 : _api$analytics.actions, api)(annotationId, AnnotationTypes.INLINE_COMMENT, annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.supportedBlockNodes);
+        return createAnnotation(api === null || api === void 0 ? void 0 : (_api$analytics = api.analytics) === null || _api$analytics === void 0 ? void 0 : _api$analytics.actions, api)(annotationId, AnnotationTypes.INLINE_COMMENT, annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.supportedBlockNodes, annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.isBlockNodeSupported);
       },
       removeInlineCommentAnnotation: annotationId => {
         var _api$analytics2, _annotationProviders$;
-        return removeInlineCommentFromDoc(api === null || api === void 0 ? void 0 : (_api$analytics2 = api.analytics) === null || _api$analytics2 === void 0 ? void 0 : _api$analytics2.actions)(annotationId, (_annotationProviders$ = annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.supportedBlockNodes) !== null && _annotationProviders$ !== void 0 ? _annotationProviders$ : []);
+        return removeInlineCommentFromDoc(api === null || api === void 0 ? void 0 : (_api$analytics2 = api.analytics) === null || _api$analytics2 === void 0 ? void 0 : _api$analytics2.actions)(annotationId, (_annotationProviders$ = annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.supportedBlockNodes) !== null && _annotationProviders$ !== void 0 ? _annotationProviders$ : [], annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.isBlockNodeSupported);
       },
       setInlineCommentDraftState: setInlineCommentDraftState(api === null || api === void 0 ? void 0 : (_api$analytics3 = api.analytics) === null || _api$analytics3 === void 0 ? void 0 : _api$analytics3.actions, annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.supportedBlockNodes, undefined, annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.isBlockNodeSupported),
       showCommentForBlockNode: showInlineCommentForBlockNode(annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.supportedBlockNodes, annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.isBlockNodeSupported),

package/dist/es2019/pm-plugins/plugin-factory.js

@@ -1,18 +1,8 @@
 import { pluginFactory } from '@atlaskit/editor-common/utils';
 import { NodeSelection } from '@atlaskit/editor-prosemirror/state';
 import { DecorationSet } from '@atlaskit/editor-prosemirror/view';
-import { expValEquals } from '@atlaskit/tmp-editor-statsig/exp-val-equals';
 import reducer from './reducer';
 import { decorationKey, findAnnotationsInSelection, inlineCommentPluginKey, isBlockNodeAnnotationsSelected, isSelectedAnnotationsChanged } from './utils';
-const handleDocChanged = (tr, prevPluginState) => {
-  if (!tr.getMeta('replaceDocument')) {
-    return getSelectionChangedHandler(false)(tr, prevPluginState);
-  }
-  return {
-    ...prevPluginState,
-    dirtyAnnotations: true
-  };
-};
 
 /**
  * Creates a handleDocChanged function with its own deleted annotations cache.
@@ -207,16 +197,9 @@ getSelectionChangeHandlerOld(reopenCommentView)(tr, pluginState);
 
 // Create the handler with cache once at module level
 const handleDocChangedWithSync = createHandleDocChanged();
-const getDocChangedHandler = (tr, prevPluginState) => {
-  // Check feature flag at runtime to support test variants
-  if (expValEquals('platform_editor_annotations_sync_on_docchange', 'isEnabled', true)) {
-    return handleDocChangedWithSync(tr, prevPluginState);
-  }
-  return handleDocChanged(tr, prevPluginState);
-};
 const dest = pluginFactory(inlineCommentPluginKey, reducer, {
   onSelectionChanged: getSelectionChangedHandler(true),
-  onDocChanged: getDocChangedHandler,
+  onDocChanged: handleDocChangedWithSync,
   mapping: (tr, pluginState, editorState) => {
     const {
       draftDecorationSet,

package/dist/es2019/pm-plugins/utils.js

@@ -295,7 +295,11 @@ export const isSelectionValid = (state, supportedBlockNodes = [], isBlockNodeSup
   if (containsInvalidNodes) {
     return AnnotationSelectionType.DISABLED;
   }
-  if (disallowOnWhitespace && hasInvalidWhitespaceNode(selection, state.schema)) {
+
+  // Extension content lives outside the editor document, so an eligible whole-block
+  // selection must not be rejected for having no commentable text descendants.
+  const isSupportedExtensionSelection = fg('cc_maui_annotations_on_extensions') && isSupportedBlockNodeSelection && selection instanceof NodeSelection && selection.node.type.name === 'extension';
+  if (disallowOnWhitespace && !isSupportedExtensionSelection && hasInvalidWhitespaceNode(selection, state.schema)) {
     return AnnotationSelectionType.INVALID;
   }
   if (!(selection instanceof NodeSelection) && isEmptyTextSelection(selection, state.schema)) {

package/dist/es2019/ui/InlineCommentView.js

@@ -5,7 +5,9 @@ import { useSharedPluginStateWithSelector } from '@atlaskit/editor-common/hooks'
 import { useSharedPluginStateSelector } from '@atlaskit/editor-common/use-shared-plugin-state-selector';
 import { getAnnotationInlineNodeTypes, getRangeInlineNodeNames } from '@atlaskit/editor-common/utils';
 import { isOfflineMode } from '@atlaskit/editor-plugin-connectivity';
+import { NodeSelection } from '@atlaskit/editor-prosemirror/state';
 import { findDomRefAtPos } from '@atlaskit/editor-prosemirror/utils';
+import { fg } from '@atlaskit/platform-feature-flags/fg';
 import { closeComponent, createAnnotation, removeInlineCommentNearSelection, setInlineCommentDraftState, updateInlineCommentResolvedState } from '../editor-commands';
 import { getAllAnnotations, getAnnotationViewKey, getSelectionPositions } from '../pm-plugins/utils';
 import { AnnotationTestIds } from '../types';
@@ -69,14 +71,36 @@ export function InlineCommentView({
   const annotationsList = getAllAnnotations(editorView.state.doc);
   const selection = getSelectionPositions(state, bookmark);
   const position = findPosForDOM(selection);
+  const isExtensionSelection = selection instanceof NodeSelection && selection.node.type.name === 'extension';
   let dom;
+  let domLookupError;
   try {
-    // Ignored via go/ees005
-    // eslint-disable-next-line @atlaskit/editor/no-as-casting
-    dom = findDomRefAtPos(position, editorView.domAtPos.bind(editorView));
+    const domRef = findDomRefAtPos(position, editorView.domAtPos.bind(editorView));
+    if (fg('cc_maui_annotations_on_extensions') && isExtensionSelection) {
+      if (domRef instanceof HTMLElement) {
+        dom = domRef;
+      }
+    } else {
+      // Preserve the existing text/media and gate-off anchor lookup.
+      // eslint-disable-next-line @atlaskit/editor/no-as-casting
+      dom = domRef;
+    }
   } catch (error) {
+    domLookupError = error;
+  }
+
+  // Native embed extension nodes are rendered by a node view. Depending on
+  // the wrapper, domAtPos can return no element (or throw) even though
+  // EditorView.nodeDOM can. Use that direct lookup as a block-comment anchor.
+  if (fg('cc_maui_annotations_on_extensions') && !dom && isExtensionSelection) {
+    const nodeDom = editorView.nodeDOM(selection.from);
+    if (nodeDom instanceof HTMLElement) {
+      dom = nodeDom;
+    }
+  }
+  if (!dom && domLookupError) {
     // eslint-disable-next-line no-console
-    console.warn(error);
+    console.warn(domLookupError);
     if (dispatchAnalyticsEvent) {
       const payload = {
         action: ACTION.ERRORED,
@@ -87,7 +111,7 @@ export function InlineCommentView({
           selection: selection.toJSON(),
           position,
           docSize: editorView.state.doc.nodeSize,
-          error: error.toString()
+          error: domLookupError.toString()
         }
       };
       dispatchAnalyticsEvent(payload);

package/dist/esm/annotationPlugin.js

@@ -90,11 +90,11 @@ export var annotationPlugin = function annotationPlugin(_ref) {
       stripNonExistingAnnotations: stripNonExistingAnnotations,
       applyInlineCommentDraft: function applyInlineCommentDraft(annotationId) {
         var _api$analytics;
-        return createAnnotation(api === null || api === void 0 || (_api$analytics = api.analytics) === null || _api$analytics === void 0 ? void 0 : _api$analytics.actions, api)(annotationId, AnnotationTypes.INLINE_COMMENT, annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.supportedBlockNodes);
+        return createAnnotation(api === null || api === void 0 || (_api$analytics = api.analytics) === null || _api$analytics === void 0 ? void 0 : _api$analytics.actions, api)(annotationId, AnnotationTypes.INLINE_COMMENT, annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.supportedBlockNodes, annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.isBlockNodeSupported);
       },
       removeInlineCommentAnnotation: function removeInlineCommentAnnotation(annotationId) {
         var _api$analytics2, _annotationProviders$;
-        return removeInlineCommentFromDoc(api === null || api === void 0 || (_api$analytics2 = api.analytics) === null || _api$analytics2 === void 0 ? void 0 : _api$analytics2.actions)(annotationId, (_annotationProviders$ = annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.supportedBlockNodes) !== null && _annotationProviders$ !== void 0 ? _annotationProviders$ : []);
+        return removeInlineCommentFromDoc(api === null || api === void 0 || (_api$analytics2 = api.analytics) === null || _api$analytics2 === void 0 ? void 0 : _api$analytics2.actions)(annotationId, (_annotationProviders$ = annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.supportedBlockNodes) !== null && _annotationProviders$ !== void 0 ? _annotationProviders$ : [], annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.isBlockNodeSupported);
       },
       setInlineCommentDraftState: setInlineCommentDraftState(api === null || api === void 0 || (_api$analytics3 = api.analytics) === null || _api$analytics3 === void 0 ? void 0 : _api$analytics3.actions, annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.supportedBlockNodes, undefined, annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.isBlockNodeSupported),
       showCommentForBlockNode: showInlineCommentForBlockNode(annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.supportedBlockNodes, annotationProviders === null || annotationProviders === void 0 ? void 0 : annotationProviders.inlineComment.isBlockNodeSupported),

package/dist/esm/pm-plugins/plugin-factory.js

@@ -4,17 +4,8 @@ function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t =
 import { pluginFactory } from '@atlaskit/editor-common/utils';
 import { NodeSelection } from '@atlaskit/editor-prosemirror/state';
 import { DecorationSet } from '@atlaskit/editor-prosemirror/view';
-import { expValEquals } from '@atlaskit/tmp-editor-statsig/exp-val-equals';
 import reducer from './reducer';
 import { decorationKey, findAnnotationsInSelection, inlineCommentPluginKey, isBlockNodeAnnotationsSelected, isSelectedAnnotationsChanged } from './utils';
-var handleDocChanged = function handleDocChanged(tr, prevPluginState) {
-  if (!tr.getMeta('replaceDocument')) {
-    return getSelectionChangedHandler(false)(tr, prevPluginState);
-  }
-  return _objectSpread(_objectSpread({}, prevPluginState), {}, {
-    dirtyAnnotations: true
-  });
-};
 
 /**
  * Creates a handleDocChanged function with its own deleted annotations cache.
@@ -201,16 +192,9 @@ var getSelectionChangedHandler = function getSelectionChangedHandler(reopenComme
 
 // Create the handler with cache once at module level
 var handleDocChangedWithSync = createHandleDocChanged();
-var getDocChangedHandler = function getDocChangedHandler(tr, prevPluginState) {
-  // Check feature flag at runtime to support test variants
-  if (expValEquals('platform_editor_annotations_sync_on_docchange', 'isEnabled', true)) {
-    return handleDocChangedWithSync(tr, prevPluginState);
-  }
-  return handleDocChanged(tr, prevPluginState);
-};
 var dest = pluginFactory(inlineCommentPluginKey, reducer, {
   onSelectionChanged: getSelectionChangedHandler(true),
-  onDocChanged: getDocChangedHandler,
+  onDocChanged: handleDocChangedWithSync,
   mapping: function mapping(tr, pluginState, editorState) {
     var draftDecorationSet = pluginState.draftDecorationSet,
       bookmark = pluginState.bookmark;

package/dist/esm/pm-plugins/utils.js

@@ -314,7 +314,11 @@ export var isSelectionValid = function isSelectionValid(state) {
   if (containsInvalidNodes) {
     return AnnotationSelectionType.DISABLED;
   }
-  if (disallowOnWhitespace && hasInvalidWhitespaceNode(selection, state.schema)) {
+
+  // Extension content lives outside the editor document, so an eligible whole-block
+  // selection must not be rejected for having no commentable text descendants.
+  var isSupportedExtensionSelection = fg('cc_maui_annotations_on_extensions') && isSupportedBlockNodeSelection && selection instanceof NodeSelection && selection.node.type.name === 'extension';
+  if (disallowOnWhitespace && !isSupportedExtensionSelection && hasInvalidWhitespaceNode(selection, state.schema)) {
     return AnnotationSelectionType.INVALID;
   }
   if (!(selection instanceof NodeSelection) && isEmptyTextSelection(selection, state.schema)) {

package/dist/esm/ui/InlineCommentView.js

@@ -6,7 +6,9 @@ import { useSharedPluginStateWithSelector } from '@atlaskit/editor-common/hooks'
 import { useSharedPluginStateSelector } from '@atlaskit/editor-common/use-shared-plugin-state-selector';
 import { getAnnotationInlineNodeTypes, getRangeInlineNodeNames } from '@atlaskit/editor-common/utils';
 import { isOfflineMode } from '@atlaskit/editor-plugin-connectivity';
+import { NodeSelection } from '@atlaskit/editor-prosemirror/state';
 import { findDomRefAtPos } from '@atlaskit/editor-prosemirror/utils';
+import { fg } from '@atlaskit/platform-feature-flags/fg';
 import { closeComponent, createAnnotation, removeInlineCommentNearSelection, setInlineCommentDraftState, updateInlineCommentResolvedState } from '../editor-commands';
 import { getAllAnnotations, getAnnotationViewKey, getSelectionPositions } from '../pm-plugins/utils';
 import { AnnotationTestIds } from '../types';
@@ -62,14 +64,36 @@ export function InlineCommentView(_ref) {
   var annotationsList = getAllAnnotations(editorView.state.doc);
   var selection = getSelectionPositions(state, bookmark);
   var position = findPosForDOM(selection);
+  var isExtensionSelection = selection instanceof NodeSelection && selection.node.type.name === 'extension';
   var dom;
+  var domLookupError;
   try {
-    // Ignored via go/ees005
-    // eslint-disable-next-line @atlaskit/editor/no-as-casting
-    dom = findDomRefAtPos(position, editorView.domAtPos.bind(editorView));
+    var domRef = findDomRefAtPos(position, editorView.domAtPos.bind(editorView));
+    if (fg('cc_maui_annotations_on_extensions') && isExtensionSelection) {
+      if (domRef instanceof HTMLElement) {
+        dom = domRef;
+      }
+    } else {
+      // Preserve the existing text/media and gate-off anchor lookup.
+      // eslint-disable-next-line @atlaskit/editor/no-as-casting
+      dom = domRef;
+    }
   } catch (error) {
+    domLookupError = error;
+  }
+
+  // Native embed extension nodes are rendered by a node view. Depending on
+  // the wrapper, domAtPos can return no element (or throw) even though
+  // EditorView.nodeDOM can. Use that direct lookup as a block-comment anchor.
+  if (fg('cc_maui_annotations_on_extensions') && !dom && isExtensionSelection) {
+    var nodeDom = editorView.nodeDOM(selection.from);
+    if (nodeDom instanceof HTMLElement) {
+      dom = nodeDom;
+    }
+  }
+  if (!dom && domLookupError) {
     // eslint-disable-next-line no-console
-    console.warn(error);
+    console.warn(domLookupError);
     if (dispatchAnalyticsEvent) {
       var payload = {
         action: ACTION.ERRORED,
@@ -80,7 +104,7 @@ export function InlineCommentView(_ref) {
           selection: selection.toJSON(),
           position: position,
           docSize: editorView.state.doc.nodeSize,
-          error: error.toString()
+          error: domLookupError.toString()
         }
       };
       dispatchAnalyticsEvent(payload);

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/editor-plugin-annotation",
-	"version": "23.0.0",
+	"version": "23.0.2",
 	"description": "Annotation plugin for @atlaskit/editor-core",
 	"author": "Atlassian Pty Ltd",
 	"license": "Apache-2.0",
@@ -31,11 +31,11 @@
 		"@atlaskit/editor-toolbar-model": "^1.2.0",
 		"@atlaskit/icon": "^38.0.0",
 		"@atlaskit/platform-feature-flags": "^2.2.0",
-		"@atlaskit/tmp-editor-statsig": "^219.0.0",
+		"@atlaskit/tmp-editor-statsig": "^220.0.0",
 		"@babel/runtime": "^7.0.0"
 	},
 	"peerDependencies": {
-		"@atlaskit/editor-common": "^127.0.0",
+		"@atlaskit/editor-common": "^127.2.0",
 		"react": "^18.2.0 || ^19.2.0",
 		"react-dom": "^18.2.0 || ^19.2.0",
 		"react-intl": "^5.25.1 || ^6.0.0 || ^7.0.0"