@atlaskit/editor-common 111.18.1 → 111.19.1

package/CHANGELOG.md

@@ -1,5 +1,26 @@
 # @atlaskit/editor-common
 
+## 111.19.1
+
+### Patch Changes
+
+- [`85444e8a5672a`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/85444e8a5672a) -
+  EDITOR-5526 clean up platform_synced_block_patch_2
+- Updated dependencies
+
+## 111.19.0
+
+### Minor Changes
+
+- [`46efd73fe9c9d`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/46efd73fe9c9d) -
+  [ux] Cleanup fg: platform_editor_fix_node_anchor_dom_cache
+
+### Patch Changes
+
+- [`2d2e663457f9a`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/2d2e663457f9a) -
+  Convert method signatures to property signature syntax to comply with
+  @typescript-eslint/method-signature-style rule. Removed 62 eslint-disable comments.
+
 ## 111.18.1
 
 ### Patch Changes

package/dist/cjs/annotation/manager.js

@@ -77,7 +77,7 @@ var SharedAnnotationManager = exports.SharedAnnotationManager = /*#__PURE__*/fun
   }, {
     key: "emit",
     value: function emit(event) {
-      this.emitter.emit(event.name, event.data);
+      this.emitter.emit(event.name, 'data' in event ? event.data : undefined);
       return this;
     }
   }, {
@@ -104,7 +104,7 @@ var SharedAnnotationManager = exports.SharedAnnotationManager = /*#__PURE__*/fun
       }
       try {
         return fn();
-      } catch (error) {
+      } catch (_unused) {
         return false;
       }
     }
@@ -120,7 +120,7 @@ var SharedAnnotationManager = exports.SharedAnnotationManager = /*#__PURE__*/fun
       }
       try {
         return fn();
-      } catch (error) {
+      } catch (_unused2) {
         return {
           success: false,
           reason: 'hook-execution-error'
@@ -139,7 +139,7 @@ var SharedAnnotationManager = exports.SharedAnnotationManager = /*#__PURE__*/fun
       }
       try {
         return fn();
-      } catch (error) {
+      } catch (_unused3) {
         return {
           success: false,
           reason: 'hook-execution-error'
@@ -158,7 +158,7 @@ var SharedAnnotationManager = exports.SharedAnnotationManager = /*#__PURE__*/fun
       }
       try {
         return fn(id);
-      } catch (error) {
+      } catch (_unused4) {
         return {
           success: false,
           reason: 'hook-execution-error'
@@ -177,7 +177,7 @@ var SharedAnnotationManager = exports.SharedAnnotationManager = /*#__PURE__*/fun
       }
       try {
         return fn();
-      } catch (error) {
+      } catch (_unused5) {
         return {
           success: false,
           reason: 'hook-execution-error'
@@ -199,7 +199,7 @@ var SharedAnnotationManager = exports.SharedAnnotationManager = /*#__PURE__*/fun
         // because the hook is responsible for the state of the selection. The manager is not responsible for the state of
         // the selection.
         return fn(id, isSelected);
-      } catch (error) {
+      } catch (_unused6) {
         return {
           success: false,
           reason: 'hook-execution-error'
@@ -218,7 +218,7 @@ var SharedAnnotationManager = exports.SharedAnnotationManager = /*#__PURE__*/fun
       }
       try {
         return fn(id, isHovered);
-      } catch (error) {
+      } catch (_unused7) {
         return {
           success: false,
           reason: 'hook-execution-error'
@@ -237,7 +237,7 @@ var SharedAnnotationManager = exports.SharedAnnotationManager = /*#__PURE__*/fun
       }
       try {
         return fn(id);
-      } catch (error) {
+      } catch (_unused8) {
         return {
           success: false,
           reason: 'hook-execution-error'

package/dist/cjs/messages/syncBlock.js

@@ -101,11 +101,6 @@ var syncBlockMessages = exports.syncBlockMessages = (0, _reactIntlNext.defineMes
     defaultMessage: 'This content is synced. Edit it at the source.',
     description: 'Tooltip that shows when you hover over a task in the destination synced block'
   },
-  deleteConfirmationModalTitleSingle: {
-    id: 'fabric.editor.deleteConfirmationModalTitleSingle',
-    defaultMessage: 'Delete synced block?',
-    description: 'Title of delete confirmation modal that appears when user tries to delete source synced block that has no reference'
-  },
   deletionConfirmationModalTitleSingle: {
     id: 'fabric.editor.deletionConfirmationModalTitleSingle',
     defaultMessage: 'Delete synced content?',
@@ -131,26 +126,11 @@ var syncBlockMessages = exports.syncBlockMessages = (0, _reactIntlNext.defineMes
     defaultMessage: 'Try again',
     description: 'Text on button which retries deleting the sync block when the previous deletion failed'
   },
-  deleteConfirmationModalDescriptionMultiple: {
-    id: 'fabric.editor.deleteConfirmationModalDescriptionMultiple',
-    defaultMessage: 'Deleting this content will also remove it from synced locations, where it will appear as an “Unsynced block”.',
-    description: 'Description of delete confirmation modal that appears when user tries to delete source synced block'
-  },
-  deleteConfirmationModalDescription: {
-    id: 'fabric.editor.deleteConfirmationModalDescriptionSingle',
-    defaultMessage: 'Deleting this content will also remove {syncBlockCount, plural, one {a synced block. References to this block} other {# synced blocks. References to these blocks}} in other locations will show an error. Continue with deletion?',
-    description: 'Description of delete confirmation modal that appears when user tries to delete source synced block'
-  },
   deletionConfirmationModalDescription: {
     id: 'fabric.editor.deletionConfirmationModalDescription',
     defaultMessage: 'If you delete this synced block, it will become an “Unsynced block” in other locations. This action is permanent and cannot be undone.',
     description: 'Description of delete confirmation modal that appears when user tries to delete source synced block'
   },
-  deleteConfirmationModalDescriptionNoRef: {
-    id: 'fabric.editor.deleteConfirmationModalDescriptionNoRef',
-    defaultMessage: "You'll no longer be able to reuse this synced block to keep content updated across locations. ",
-    description: 'Description of delete confirmation modal that appears when user tries to delete source synced block'
-  },
   deletionConfirmationModalDescriptionNoRef: {
     id: 'fabric.editor.deletionConfirmationModalDescriptionNoRef',
     defaultMessage: 'Your content will no longer appear, and the synced block will be deleted. This action is permanent and cannot be undone.',
@@ -346,21 +326,11 @@ var syncBlockMessages = exports.syncBlockMessages = (0, _reactIntlNext.defineMes
     defaultMessage: 'Unsync this content?',
     description: 'Title of unsync confirmation modal that appears when user tries to unsync source synced block'
   },
-  unsyncConfirmationModalDescriptionSingle: {
-    id: 'fabric.editor.unsyncConfirmationModalDescriptionSingle',
-    defaultMessage: 'Your content will stay here. It will no longer be a synced block.',
-    description: 'Description of unsync confirmation modal that appears when user tries to unsync source synced block with no reference'
-  },
   unsyncConfirmModalDescriptionSingle: {
     id: 'fabric.editor.unsyncConfirmModalDescriptionSingle',
     defaultMessage: 'Your content will stay here. It will no longer be a synced block. This action is permanent and cannot be undone.',
     description: 'Description of unsync confirmation modal that appears when user tries to unsync source synced block with no reference'
   },
-  unsyncConfirmationModalDescriptionMultiple: {
-    id: 'fabric.editor.unsyncConfirmationModalDescriptionMultiple',
-    defaultMessage: 'Your content will stay here. In {syncBlockCount, plural, one {1 other synced location} other {# other synced locations}} it will appear as an “Unsynced block”. ',
-    description: 'Description of unsync confirmation modal that appears when user tries to unsync source synced block with multiple references'
-  },
   unsyncConfirmModalDescriptionMultiple: {
     id: 'fabric.editor.unsyncConfirmModalDescriptionMultiple',
     defaultMessage: 'Your content will stay here. In {syncBlockCount, plural, one {1 other synced location} other {# other synced locations}} it will appear as an “Unsynced block”. This action is permanent and cannot be undone.',

package/dist/cjs/monitoring/error.js

@@ -19,7 +19,7 @@ function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t =
 function _interopRequireWildcard(e, t) { if ("function" == typeof WeakMap) var r = new WeakMap(), n = new WeakMap(); return (_interopRequireWildcard = function _interopRequireWildcard(e, t) { if (!t && e && e.__esModule) return e; var o, i, f = { __proto__: null, default: e }; if (null === e || "object" != _typeof(e) && "function" != typeof e) return f; if (o = t ? n : r) { if (o.has(e)) return o.get(e); o.set(e, f); } for (var _t in e) "default" !== _t && {}.hasOwnProperty.call(e, _t) && ((i = (o = Object.defineProperty) && Object.getOwnPropertyDescriptor(e, _t)) && (i.get || i.set) ? o(f, _t, i) : f[_t] = e[_t]); return f; })(e, t); }
 var SENTRY_DSN = 'https://0b10c8e02fb44d8796c047b102c9bee8@o55978.ingest.sentry.io/4505129224110080';
 var packageName = 'editor-common'; // Sentry doesn't accept '/' in its releases https://docs.sentry.io/platforms/javascript/configuration/releases/
-var packageVersion = "111.18.0";
+var packageVersion = "111.19.0";
 var sanitiseSentryEvents = function sanitiseSentryEvents(data, _hint) {
   // Remove URL as it has UGC
   // Ignored via go/ees007

package/dist/cjs/node-anchor/node-anchor-provider.js

@@ -92,12 +92,6 @@ var NodeAnchorProvider = exports.NodeAnchorProvider = /*#__PURE__*/function () {
   }, {
     key: "setIdForNode",
     value: function setIdForNode(node, id) {
-      // [FEATURE FLAG: platform_editor_fix_node_anchor_dom_cache]
-      // This method is part of the fix to prevent drag handle misalignment.
-      // To clean up: remove the feature flag check, keep the method body
-      if (!(0, _platformFeatureFlags.fg)('platform_editor_fix_node_anchor_dom_cache')) {
-        return;
-      }
       if (this.limitedMode) {
         return;
       }

package/dist/cjs/preset/core-plugin/index.js

@@ -5,7 +5,6 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.corePlugin = void 0;
 var _model = require("@atlaskit/editor-prosemirror/model");
-var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 var _nodeAnchorProvider = require("../../node-anchor/node-anchor-provider");
 var _processRawValue = require("../../utils/processRawValue");
 var _editorCommands = require("../editor-commands");
@@ -153,45 +152,26 @@ var corePlugin = exports.corePlugin = function corePlugin(_ref) {
           return undefined;
         }
 
-        // [FEATURE FLAG: platform_editor_fix_node_anchor_dom_cache]
-        // Fixes drag handle misalignment on first focus after clicking from title
-        // by checking DOM for existing anchor before generating a new ID with collision suffix.
-        // To clean up: remove if-else block, keep only flag-on behavior (lines 217-229), remove old behavior (lines 231-238)
-        if ((0, _platformFeatureFlags.fg)('platform_editor_fix_node_anchor_dom_cache')) {
-          // Check DOM first to avoid generating a new ID with a collision suffix
-          // when the DOM already has a valid anchor. This prevents misalignment
-          // of drag handles on first focus after clicking from the title.
-          var nodeDOM = view.nodeDOM(pos);
-          if (nodeDOM instanceof HTMLElement) {
-            var domAnchor = nodeDOM.getAttribute('data-node-anchor');
-            if (domAnchor) {
-              // Cache the DOM anchor for this node to maintain consistency
-              // This ensures subsequent calls return the same anchor
-              nodeIdProvider.setIdForNode(node, domAnchor);
-              return domAnchor;
-            }
+        // Check DOM first to avoid generating a new ID with a collision suffix
+        // when the DOM already has a valid anchor. This prevents misalignment
+        // of drag handles on first focus after clicking from the title.
+        var nodeDOM = view.nodeDOM(pos);
+        if (nodeDOM instanceof HTMLElement) {
+          var domAnchor = nodeDOM.getAttribute('data-node-anchor');
+          if (domAnchor) {
+            // Cache the DOM anchor for this node to maintain consistency
+            // This ensures subsequent calls return the same anchor
+            nodeIdProvider.setIdForNode(node, domAnchor);
+            return domAnchor;
           }
+        }
 
-          // Only generate a new ID if DOM doesn't have one
-          var generatedId = nodeIdProvider.getOrGenerateId(node, pos);
-          if (generatedId) {
-            return generatedId;
-          }
-          return undefined;
-        } else {
-          // OLD BEHAVIOR (to be removed when flag is cleaned up)
-          // This approach can generate incorrect IDs with collision suffixes when
-          // the DOM already has a valid anchor, causing drag handle misalignment
-          var _generatedId = nodeIdProvider.getOrGenerateId(node, pos);
-          if (_generatedId) {
-            return _generatedId;
-          }
-          var _nodeDOM = view.nodeDOM(pos);
-          if (_nodeDOM instanceof HTMLElement) {
-            return _nodeDOM.getAttribute('data-node-anchor') || undefined;
-          }
-          return undefined;
+        // Only generate a new ID if DOM doesn't have one
+        var generatedId = nodeIdProvider.getOrGenerateId(node, pos);
+        if (generatedId) {
+          return generatedId;
         }
+        return undefined;
       }
     }
   };

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

@@ -24,7 +24,7 @@ function _isNativeReflectConstruct() { try { var t = !Boolean.prototype.valueOf.
  * @jsx jsx
  */ // eslint-disable-next-line @atlaskit/ui-styling-standard/use-compiled -- Ignored via go/DSP-18766
 var packageName = "@atlaskit/editor-common";
-var packageVersion = "111.18.0";
+var packageVersion = "111.19.0";
 var halfFocusRing = 1;
 var dropOffset = '0, 8';
 var fadeIn = (0, _react2.keyframes)({

package/dist/cjs/utils/compareNodes.js

@@ -135,15 +135,15 @@ function getMetaFromNode(node, options, normalizeTextParser) {
       }
     case 'inlineCard':
       {
-        var _attrs = firstChild.attrs;
-        var maybeTitle = options.getInlineCardTextFromStore(_attrs);
+        var attrs = firstChild.attrs;
+        var maybeTitle = options.getInlineCardTextFromStore(attrs);
         if (maybeTitle) {
           return {
             type: ContentType.LINK,
             value: maybeTitle
           };
         }
-        var url = _attrs.url;
+        var url = attrs.url;
         return {
           type: ContentType.LINK,
           value: url ? url : ''

package/dist/es2019/annotation/manager.js

@@ -50,7 +50,7 @@ export class SharedAnnotationManager {
     return this;
   }
   emit(event) {
-    this.emitter.emit(event.name, event.data);
+    this.emitter.emit(event.name, 'data' in event ? event.data : undefined);
     return this;
   }
   hook(method, handler) {
@@ -71,7 +71,7 @@ export class SharedAnnotationManager {
     }
     try {
       return fn();
-    } catch (error) {
+    } catch {
       return false;
     }
   }
@@ -85,7 +85,7 @@ export class SharedAnnotationManager {
     }
     try {
       return fn();
-    } catch (error) {
+    } catch {
       return {
         success: false,
         reason: 'hook-execution-error'
@@ -102,7 +102,7 @@ export class SharedAnnotationManager {
     }
     try {
       return fn();
-    } catch (error) {
+    } catch {
       return {
         success: false,
         reason: 'hook-execution-error'
@@ -119,7 +119,7 @@ export class SharedAnnotationManager {
     }
     try {
       return fn(id);
-    } catch (error) {
+    } catch {
       return {
         success: false,
         reason: 'hook-execution-error'
@@ -136,7 +136,7 @@ export class SharedAnnotationManager {
     }
     try {
       return fn();
-    } catch (error) {
+    } catch {
       return {
         success: false,
         reason: 'hook-execution-error'
@@ -156,7 +156,7 @@ export class SharedAnnotationManager {
       // because the hook is responsible for the state of the selection. The manager is not responsible for the state of
       // the selection.
       return fn(id, isSelected);
-    } catch (error) {
+    } catch {
       return {
         success: false,
         reason: 'hook-execution-error'
@@ -173,7 +173,7 @@ export class SharedAnnotationManager {
     }
     try {
       return fn(id, isHovered);
-    } catch (error) {
+    } catch {
       return {
         success: false,
         reason: 'hook-execution-error'
@@ -190,7 +190,7 @@ export class SharedAnnotationManager {
     }
     try {
       return fn(id);
-    } catch (error) {
+    } catch {
       return {
         success: false,
         reason: 'hook-execution-error'

package/dist/es2019/messages/syncBlock.js

@@ -95,11 +95,6 @@ export const syncBlockMessages = defineMessages({
     defaultMessage: 'This content is synced. Edit it at the source.',
     description: 'Tooltip that shows when you hover over a task in the destination synced block'
   },
-  deleteConfirmationModalTitleSingle: {
-    id: 'fabric.editor.deleteConfirmationModalTitleSingle',
-    defaultMessage: 'Delete synced block?',
-    description: 'Title of delete confirmation modal that appears when user tries to delete source synced block that has no reference'
-  },
   deletionConfirmationModalTitleSingle: {
     id: 'fabric.editor.deletionConfirmationModalTitleSingle',
     defaultMessage: 'Delete synced content?',
@@ -125,26 +120,11 @@ export const syncBlockMessages = defineMessages({
     defaultMessage: 'Try again',
     description: 'Text on button which retries deleting the sync block when the previous deletion failed'
   },
-  deleteConfirmationModalDescriptionMultiple: {
-    id: 'fabric.editor.deleteConfirmationModalDescriptionMultiple',
-    defaultMessage: 'Deleting this content will also remove it from synced locations, where it will appear as an “Unsynced block”.',
-    description: 'Description of delete confirmation modal that appears when user tries to delete source synced block'
-  },
-  deleteConfirmationModalDescription: {
-    id: 'fabric.editor.deleteConfirmationModalDescriptionSingle',
-    defaultMessage: 'Deleting this content will also remove {syncBlockCount, plural, one {a synced block. References to this block} other {# synced blocks. References to these blocks}} in other locations will show an error. Continue with deletion?',
-    description: 'Description of delete confirmation modal that appears when user tries to delete source synced block'
-  },
   deletionConfirmationModalDescription: {
     id: 'fabric.editor.deletionConfirmationModalDescription',
     defaultMessage: 'If you delete this synced block, it will become an “Unsynced block” in other locations. This action is permanent and cannot be undone.',
     description: 'Description of delete confirmation modal that appears when user tries to delete source synced block'
   },
-  deleteConfirmationModalDescriptionNoRef: {
-    id: 'fabric.editor.deleteConfirmationModalDescriptionNoRef',
-    defaultMessage: "You'll no longer be able to reuse this synced block to keep content updated across locations. ",
-    description: 'Description of delete confirmation modal that appears when user tries to delete source synced block'
-  },
   deletionConfirmationModalDescriptionNoRef: {
     id: 'fabric.editor.deletionConfirmationModalDescriptionNoRef',
     defaultMessage: 'Your content will no longer appear, and the synced block will be deleted. This action is permanent and cannot be undone.',
@@ -340,21 +320,11 @@ export const syncBlockMessages = defineMessages({
     defaultMessage: 'Unsync this content?',
     description: 'Title of unsync confirmation modal that appears when user tries to unsync source synced block'
   },
-  unsyncConfirmationModalDescriptionSingle: {
-    id: 'fabric.editor.unsyncConfirmationModalDescriptionSingle',
-    defaultMessage: 'Your content will stay here. It will no longer be a synced block.',
-    description: 'Description of unsync confirmation modal that appears when user tries to unsync source synced block with no reference'
-  },
   unsyncConfirmModalDescriptionSingle: {
     id: 'fabric.editor.unsyncConfirmModalDescriptionSingle',
     defaultMessage: 'Your content will stay here. It will no longer be a synced block. This action is permanent and cannot be undone.',
     description: 'Description of unsync confirmation modal that appears when user tries to unsync source synced block with no reference'
   },
-  unsyncConfirmationModalDescriptionMultiple: {
-    id: 'fabric.editor.unsyncConfirmationModalDescriptionMultiple',
-    defaultMessage: 'Your content will stay here. In {syncBlockCount, plural, one {1 other synced location} other {# other synced locations}} it will appear as an “Unsynced block”. ',
-    description: 'Description of unsync confirmation modal that appears when user tries to unsync source synced block with multiple references'
-  },
   unsyncConfirmModalDescriptionMultiple: {
     id: 'fabric.editor.unsyncConfirmModalDescriptionMultiple',
     defaultMessage: 'Your content will stay here. In {syncBlockCount, plural, one {1 other synced location} other {# other synced locations}} it will appear as an “Unsynced block”. This action is permanent and cannot be undone.',

package/dist/es2019/monitoring/error.js

@@ -4,7 +4,7 @@ import { isFedRamp } from './environment';
 import { normaliseSentryBreadcrumbs, SERIALIZABLE_ATTRIBUTES } from './normalise-sentry-breadcrumbs';
 const SENTRY_DSN = 'https://0b10c8e02fb44d8796c047b102c9bee8@o55978.ingest.sentry.io/4505129224110080';
 const packageName = 'editor-common'; // Sentry doesn't accept '/' in its releases https://docs.sentry.io/platforms/javascript/configuration/releases/
-const packageVersion = "111.18.0";
+const packageVersion = "111.19.0";
 const sanitiseSentryEvents = (data, _hint) => {
   // Remove URL as it has UGC
   // Ignored via go/ees007

package/dist/es2019/node-anchor/node-anchor-provider.js

@@ -69,12 +69,6 @@ export class NodeAnchorProvider {
     return this.cache.get(node);
   }
   setIdForNode(node, id) {
-    // [FEATURE FLAG: platform_editor_fix_node_anchor_dom_cache]
-    // This method is part of the fix to prevent drag handle misalignment.
-    // To clean up: remove the feature flag check, keep the method body
-    if (!fg('platform_editor_fix_node_anchor_dom_cache')) {
-      return;
-    }
     if (this.limitedMode) {
       return;
     }

package/dist/es2019/preset/core-plugin/index.js

@@ -1,5 +1,4 @@
 import { Node } from '@atlaskit/editor-prosemirror/model';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { getNodeIdProvider } from '../../node-anchor/node-anchor-provider';
 import { processRawFragmentValue, processRawValue, processRawValueWithoutValidation } from '../../utils/processRawValue';
 import { editorCommandToPMCommand } from '../editor-commands';
@@ -155,45 +154,26 @@ export const corePlugin = ({
           return undefined;
         }
 
-        // [FEATURE FLAG: platform_editor_fix_node_anchor_dom_cache]
-        // Fixes drag handle misalignment on first focus after clicking from title
-        // by checking DOM for existing anchor before generating a new ID with collision suffix.
-        // To clean up: remove if-else block, keep only flag-on behavior (lines 217-229), remove old behavior (lines 231-238)
-        if (fg('platform_editor_fix_node_anchor_dom_cache')) {
-          // Check DOM first to avoid generating a new ID with a collision suffix
-          // when the DOM already has a valid anchor. This prevents misalignment
-          // of drag handles on first focus after clicking from the title.
-          const nodeDOM = view.nodeDOM(pos);
-          if (nodeDOM instanceof HTMLElement) {
-            const domAnchor = nodeDOM.getAttribute('data-node-anchor');
-            if (domAnchor) {
-              // Cache the DOM anchor for this node to maintain consistency
-              // This ensures subsequent calls return the same anchor
-              nodeIdProvider.setIdForNode(node, domAnchor);
-              return domAnchor;
-            }
+        // Check DOM first to avoid generating a new ID with a collision suffix
+        // when the DOM already has a valid anchor. This prevents misalignment
+        // of drag handles on first focus after clicking from the title.
+        const nodeDOM = view.nodeDOM(pos);
+        if (nodeDOM instanceof HTMLElement) {
+          const domAnchor = nodeDOM.getAttribute('data-node-anchor');
+          if (domAnchor) {
+            // Cache the DOM anchor for this node to maintain consistency
+            // This ensures subsequent calls return the same anchor
+            nodeIdProvider.setIdForNode(node, domAnchor);
+            return domAnchor;
           }
+        }
 
-          // Only generate a new ID if DOM doesn't have one
-          const generatedId = nodeIdProvider.getOrGenerateId(node, pos);
-          if (generatedId) {
-            return generatedId;
-          }
-          return undefined;
-        } else {
-          // OLD BEHAVIOR (to be removed when flag is cleaned up)
-          // This approach can generate incorrect IDs with collision suffixes when
-          // the DOM already has a valid anchor, causing drag handle misalignment
-          const generatedId = nodeIdProvider.getOrGenerateId(node, pos);
-          if (generatedId) {
-            return generatedId;
-          }
-          const nodeDOM = view.nodeDOM(pos);
-          if (nodeDOM instanceof HTMLElement) {
-            return nodeDOM.getAttribute('data-node-anchor') || undefined;
-          }
-          return undefined;
+        // Only generate a new ID if DOM doesn't have one
+        const generatedId = nodeIdProvider.getOrGenerateId(node, pos);
+        if (generatedId) {
+          return generatedId;
         }
+        return undefined;
       }
     }
   };

package/dist/es2019/ui/DropList/index.js

@@ -14,7 +14,7 @@ import withAnalyticsEvents from '@atlaskit/analytics-next/withAnalyticsEvents';
 import { fg } from '@atlaskit/platform-feature-flags';
 import Layer from '../Layer';
 const packageName = "@atlaskit/editor-common";
-const packageVersion = "111.18.0";
+const packageVersion = "111.19.0";
 const halfFocusRing = 1;
 const dropOffset = '0, 8';
 const fadeIn = keyframes({

package/dist/esm/annotation/manager.js

@@ -70,7 +70,7 @@ export var SharedAnnotationManager = /*#__PURE__*/function () {
   }, {
     key: "emit",
     value: function emit(event) {
-      this.emitter.emit(event.name, event.data);
+      this.emitter.emit(event.name, 'data' in event ? event.data : undefined);
       return this;
     }
   }, {
@@ -97,7 +97,7 @@ export var SharedAnnotationManager = /*#__PURE__*/function () {
       }
       try {
         return fn();
-      } catch (error) {
+      } catch (_unused) {
         return false;
       }
     }
@@ -113,7 +113,7 @@ export var SharedAnnotationManager = /*#__PURE__*/function () {
       }
       try {
         return fn();
-      } catch (error) {
+      } catch (_unused2) {
         return {
           success: false,
           reason: 'hook-execution-error'
@@ -132,7 +132,7 @@ export var SharedAnnotationManager = /*#__PURE__*/function () {
       }
       try {
         return fn();
-      } catch (error) {
+      } catch (_unused3) {
         return {
           success: false,
           reason: 'hook-execution-error'
@@ -151,7 +151,7 @@ export var SharedAnnotationManager = /*#__PURE__*/function () {
       }
       try {
         return fn(id);
-      } catch (error) {
+      } catch (_unused4) {
         return {
           success: false,
           reason: 'hook-execution-error'
@@ -170,7 +170,7 @@ export var SharedAnnotationManager = /*#__PURE__*/function () {
       }
       try {
         return fn();
-      } catch (error) {
+      } catch (_unused5) {
         return {
           success: false,
           reason: 'hook-execution-error'
@@ -192,7 +192,7 @@ export var SharedAnnotationManager = /*#__PURE__*/function () {
         // because the hook is responsible for the state of the selection. The manager is not responsible for the state of
         // the selection.
         return fn(id, isSelected);
-      } catch (error) {
+      } catch (_unused6) {
         return {
           success: false,
           reason: 'hook-execution-error'
@@ -211,7 +211,7 @@ export var SharedAnnotationManager = /*#__PURE__*/function () {
       }
       try {
         return fn(id, isHovered);
-      } catch (error) {
+      } catch (_unused7) {
         return {
           success: false,
           reason: 'hook-execution-error'
@@ -230,7 +230,7 @@ export var SharedAnnotationManager = /*#__PURE__*/function () {
       }
       try {
         return fn(id);
-      } catch (error) {
+      } catch (_unused8) {
         return {
           success: false,
           reason: 'hook-execution-error'