@atlaskit/editor-plugin-synced-block 12.0.5 → 12.0.7

package/dist/es2019/ui/floating-toolbar.js

@@ -11,7 +11,6 @@ import DeleteIcon from '@atlaskit/icon/core/delete';
 import EditIcon from '@atlaskit/icon/core/edit';
 import LinkBrokenIcon from '@atlaskit/icon/core/link-broken';
 import MegaphoneIcon from '@atlaskit/icon/core/megaphone';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { expValEquals } from '@atlaskit/tmp-editor-statsig/exp-val-equals';
 import { copySyncedBlockReferenceToClipboard, editSyncedBlockSource, removeSyncedBlock, unsync } from '../editor-commands';
 import { findSyncBlockOrBodiedSyncBlock, isBodiedSyncBlockNode } from '../pm-plugins/utils/utils';
@@ -158,7 +157,7 @@ export const getToolbarConfig = (state, intl, api, syncBlockStore, isLivePage, o
       };
       items.push(editSourceButton);
     }
-    if (!isSyncBlockActivationEnabled && fg('platform_synced_block_patch_13')) {
+    if (!isSyncBlockActivationEnabled) {
       const separator = {
         type: 'separator'
       };

package/dist/esm/editor-commands/index.js

@@ -8,7 +8,6 @@ import { DOMSerializer, Fragment } from '@atlaskit/editor-prosemirror/model';
 import { NodeSelection, Selection, TextSelection } from '@atlaskit/editor-prosemirror/state';
 import { findSelectedNodeOfType, removeParentNodeOfType, removeSelectedNode, safeInsert } from '@atlaskit/editor-prosemirror/utils';
 import { getSourceProductFromResourceIdSafe } from '@atlaskit/editor-synced-block-provider/utils';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { expValEqualsNoExposure } from '@atlaskit/tmp-editor-statsig/exp-val-equals-no-exposure';
 import { creationMetaKey, deleteMechanismMetaKey, syncedBlockPluginKey } from '../pm-plugins/main';
 import { canBeConvertedToSyncBlock, deferDispatch, findSyncBlock, findSyncBlockOrBodiedSyncBlock, isBodiedSyncBlockNode } from '../pm-plugins/utils/utils';
@@ -108,9 +107,7 @@ export var createSyncedBlock = function createSyncedBlock(_ref) {
       // safeInsert does not move the selection into the new block, so place the
       // caret inside the block's empty paragraph so typing continues inside the
       // synced block (EDITOR-7949).
-      if (fg('platform_editor_blocks_patch_4')) {
-        tr = placeCaretInsideBodiedSyncBlock(tr, newBodiedSyncBlockNode.attrs.localId);
-      }
+      tr = placeCaretInsideBodiedSyncBlock(tr, newBodiedSyncBlockNode.attrs.localId);
     }
   } else {
     var conversionInfo = canBeConvertedToSyncBlock(tr.selection);
@@ -141,14 +138,10 @@ export var createSyncedBlock = function createSyncedBlock(_ref) {
       return false;
     }
     tr.replaceWith(conversionInfo.from, conversionInfo.to, _newBodiedSyncBlockNode).scrollIntoView();
-    if (fg('platform_editor_blocks_patch_4')) {
-      // Place the caret on the first editable position inside the converted
-      // content so typing continues inside the synced block (EDITOR-7949).
-      tr = placeCaretInsideBodiedSyncBlock(tr, _newBodiedSyncBlockNode.attrs.localId);
-    } else {
-      // set selection to the start of the previous selection for the position taken up by the start of the new synced block
-      tr.setSelection(TextSelection.create(tr.doc, conversionInfo.from));
-    }
+
+    // Place the caret on the first editable position inside the converted
+    // content so typing continues inside the synced block (EDITOR-7949).
+    tr = placeCaretInsideBodiedSyncBlock(tr, _newBodiedSyncBlockNode.attrs.localId);
   }
 
   // Stash creation-type signals on the final transaction (after any typeahead
@@ -252,7 +245,7 @@ var copySyncedBlockReferenceToClipboardInternal = function copySyncedBlockRefere
         attributes: _objectSpread({
           resourceId: referenceSyncBlockNode.attrs.resourceId,
           inputMethod: inputMethod
-        }, sourceJoinKey && fg('platform_editor_blocks_patch_4') && {
+        }, sourceJoinKey && {
           blockInstanceId: sourceJoinKey
         })
       });
@@ -333,11 +326,8 @@ export var removeSyncedBlock = function removeSyncedBlock(api) {
     }
 
     // Tag the transaction so analytics can report this as `deleteButton` rather
-    // than a keyboard delete (both produce a plain ReplaceStep). Gated so the
-    // gate-off transaction is unchanged.
-    if (fg('platform_editor_blocks_patch_4')) {
-      removeTr.setMeta(deleteMechanismMetaKey, 'deleteButton');
-    }
+    // than a keyboard delete (both produce a plain ReplaceStep).
+    removeTr.setMeta(deleteMechanismMetaKey, 'deleteButton');
     dispatch(removeTr);
     api === null || api === void 0 || api.core.actions.focus();
     return true;

package/dist/esm/nodeviews/bodiedSyncedBlock.js

@@ -7,7 +7,6 @@ import { ErrorBoundary } from '@atlaskit/editor-common/error-boundary';
 import { BodiedSyncBlockSharedCssClassName } from '@atlaskit/editor-common/sync-block';
 import { isOfflineMode } from '@atlaskit/editor-plugin-connectivity';
 import { DOMSerializer } from '@atlaskit/editor-prosemirror/model';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { SyncBlockLabel } from '../ui/SyncBlockLabel';
 import { isEmptySourceSyncBlock } from './isEmptySourceSyncBlock';
 var toDOM = function toDOM(node) {
@@ -44,15 +43,12 @@ export var BodiedSyncBlock = /*#__PURE__*/function () {
       // eslint-disable-next-line @atlaskit/platform/no-direct-document-usage -- NodeView DOM must be created against active runtime document
       labelContainer = document.createElement('div');
       this.dom.appendChild(labelContainer);
-    } else if (fg('platform_synced_block_patch_14')) {
-      // Render the label into a separate container for the patch path so
-      // portal implementations that write directly into the target do not clobber
-      // the ProseMirror contentDOM.
+    } else {
+      // Render the label into a separate container so portal implementations
+      // that write directly into the target do not clobber contentDOM.
       // eslint-disable-next-line @atlaskit/platform/no-direct-document-usage -- NodeView DOM must be created against active runtime document
       labelContainer = document.createElement('div');
       this.dom.appendChild(labelContainer);
-    } else {
-      labelContainer = this.dom;
     }
     this.labelKey = crypto.randomUUID();
     this.nodeViewPortalProviderAPI.render(function () {

package/dist/esm/nodeviews/syncedBlock.js

@@ -14,7 +14,6 @@ import { ErrorBoundary } from '@atlaskit/editor-common/error-boundary';
 import ReactNodeView from '@atlaskit/editor-common/react-node-view';
 import { SyncBlockSharedCssClassName, SyncBlockLabelSharedCssClassName, SyncBlockActionsProvider } from '@atlaskit/editor-common/sync-block';
 import { NodeSelection } from '@atlaskit/editor-prosemirror/state';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { removeSyncedBlockAtPos } from '../editor-commands';
 import { SyncBlockRendererWrapper } from '../ui/SyncBlockRendererWrapper';
 import { SyncBlockSSRReactContextsProvider } from '../ui/SyncBlockSSRReactContextsProvider';
@@ -62,17 +61,15 @@ export var SyncBlock = /*#__PURE__*/function (_ReactNodeView) {
       // eslint-disable-next-line @atlaskit/platform/no-direct-document-usage -- NodeView DOM must be created against active runtime document
       var domRef = document.createElement('div');
       domRef.classList.add(SyncBlockSharedCssClassName.prefix);
-      if (fg('platform_synced_block_patch_14')) {
-        // Prevent native browser drag on the contentEditable="false" wrapper.
-        // Without this, clicking in empty space (outside the contentEditable
-        // renderer but inside the domRef) initiates a native element drag.
-        domRef.draggable = false;
-        this.dragStartHandler = function (e) {
-          e.preventDefault();
-        };
-        // eslint-disable-next-line @atlaskit/design-system/no-direct-use-of-web-platform-drag-and-drop, @repo/internal/dom-events/no-unsafe-event-listeners
-        domRef.addEventListener('dragstart', this.dragStartHandler);
-      }
+      // Prevent native browser drag on the contentEditable="false" wrapper.
+      // Without this, clicking in empty space (outside the contentEditable
+      // renderer but inside the domRef) initiates a native element drag.
+      domRef.draggable = false;
+      this.dragStartHandler = function (e) {
+        e.preventDefault();
+      };
+      // eslint-disable-next-line @atlaskit/design-system/no-direct-use-of-web-platform-drag-and-drop, @repo/internal/dom-events/no-unsafe-event-listeners
+      domRef.addEventListener('dragstart', this.dragStartHandler);
       return domRef;
     }
 
@@ -105,9 +102,6 @@ export var SyncBlock = /*#__PURE__*/function (_ReactNodeView) {
   }, {
     key: "stopEvent",
     value: function stopEvent(event) {
-      if (!fg('platform_synced_block_patch_14')) {
-        return false;
-      }
       var target = event.target;
       if (!(target instanceof Element)) {
         return false;

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

@@ -18,9 +18,8 @@ import { isOfflineMode } from '@atlaskit/editor-plugin-connectivity';
 import { NodeSelection, PluginKey } from '@atlaskit/editor-prosemirror/state';
 import { ReplaceAroundStep, ReplaceStep } from '@atlaskit/editor-prosemirror/transform';
 import { DecorationSet, Decoration } from '@atlaskit/editor-prosemirror/view';
-import { convertPMNodesToSyncBlockNodes, rebaseTransaction } from '@atlaskit/editor-synced-block-provider';
+import { convertPMNodesToSyncBlockNodes } from '@atlaskit/editor-synced-block-provider';
 import { getSourceProductFromResourceIdSafe } from '@atlaskit/editor-synced-block-provider/utils';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { expValEqualsNoExposure } from '@atlaskit/tmp-editor-statsig/exp-val-equals-no-exposure';
 import { bodiedSyncBlockNodeView } from '../nodeviews/bodiedSyncedBlock';
 import { SyncBlock as SyncBlockView } from '../nodeviews/syncedBlock';
@@ -41,8 +40,7 @@ export var syncedBlockPluginKey = new PluginKey('syncedBlockPlugin');
  * `deleteButton` (otherwise indistinguishable from a keyboard delete — both are
  * plain ReplaceSteps). A separate key (rather than `syncedBlockPluginKey`, which
  * carries plugin-state transitions read in `apply()`, or a bare string) keeps
- * this transient signal uniquely namespaced and isolated. Only read behind
- * `platform_editor_blocks_patch_4`.
+ * this transient signal uniquely namespaced and isolated.
  */
 export var deleteMechanismMetaKey = new PluginKey('syncedBlockDeleteMechanism');
 var syncedBlockPromptedFeedbackMetaKey = new PluginKey('syncedBlockPromptedFeedback');
@@ -51,8 +49,7 @@ var syncedBlockPromptedFeedbackMetaKey = new PluginKey('syncedBlockPromptedFeedb
  * Creation-type signals set by {@link createSyncedBlock} on the creating
  * transaction. The async creation handler forwards them to the store manager,
  * which attaches them to the `syncedBlockCreate` event. A dedicated key keeps
- * this transient signal isolated from plugin state. Consumed behind
- * `platform_editor_blocks_patch_4`.
+ * this transient signal isolated from plugin state.
  */
 
 export var creationMetaKey = new PluginKey('syncedBlockCreationMeta');
@@ -308,17 +305,9 @@ var filterTransactionOnline = function filterTransactionOnline(_ref3) {
     });
   });
   if (bodiedSyncBlockRemoved.length > 0) {
-    if (!fg('platform_editor_blocks_patch_4')) {
-      // Legacy stash-and-rebase path: store the original delete transaction so it
-      // can be rebased against intervening edits and replayed on confirm. When
-      // platform_editor_blocks_patch_4 is on we instead recompute the delete fresh
-      // from the live document on confirm (see handleBodiedSyncBlockRemoval), so
-      // there is no need to stash the transaction here. See EDITOR-7889.
-      // eslint-disable-next-line no-param-reassign
-      confirmationTransactionRef.current = tr;
-    }
-    // Only derive the mechanism behind the gate — discarded gate-off.
-    var mechanism = fg('platform_editor_blocks_patch_4') ? getDeleteMechanism(tr, state) : undefined;
+    // Recompute the delete fresh from the live document on confirm
+    // (see handleBodiedSyncBlockRemoval). See EDITOR-7889.
+    var mechanism = getDeleteMechanism(tr, state);
     var feedbackContext = entryPoint && getDeleteReason(tr) !== 'source-block-unsynced' ? {
       blockType: 'source',
       entryPoint: entryPoint
@@ -356,7 +345,7 @@ var filterTransactionOnline = function filterTransactionOnline(_ref3) {
     // be wrong. The bodiedSyncBlock content will still be inserted but will
     // not be registered as a source block in Block Service, which is safer
     // than creating a zombie block under the wrong ARI.
-    if (isPasteOrDrop && fg('platform_synced_block_patch_13')) {
+    if (isPasteOrDrop) {
       return true;
     }
 
@@ -690,8 +679,8 @@ export var createPlugin = function createPlugin(options, pmPluginFactoryParams,
           var syncBlockNodes = instance.doc.children.filter(syncBlockStore.referenceManager.isReferenceBlock);
           syncBlockStore.referenceManager.fetchSyncBlocksData(convertPMNodesToSyncBlockNodes(syncBlockNodes));
 
-          // Populate source sync block cache from initial document
-          // When fg is ON, this replaces the constructor call in the nodeview
+          // Populate source sync block cache from initial document.
+          // This replaces the constructor call in the nodeview.
           instance.doc.forEach(function (node) {
             if (syncBlockStore.sourceManager.isSourceBlock(node)) {
               syncBlockStore.sourceManager.updateSyncBlockData(node, false);
@@ -923,18 +912,12 @@ export var createPlugin = function createPlugin(options, pmPluginFactoryParams,
           return false;
         },
         cut: function cut() {
-          if (fg('platform_synced_block_patch_13') || Boolean(options === null || options === void 0 ? void 0 : options.onGiveFeedback) && expValEqualsNoExposure('platform_editor_sync_block_activation', 'isEnabled', true)) {
-            ctx.consumeCutEvent();
-            ctx.markCutEvent();
-          }
+          ctx.consumeCutEvent();
+          ctx.markCutEvent();
           return false;
         }
       },
       transformPasted: function transformPasted(slice, _view) {
-        if (!fg('platform_synced_block_patch_13')) {
-          return slice;
-        }
-
         // Defense against bodiedSyncBlock nodes arriving via paste
         // (e.g. drag-and-drop, cut-paste, or browser-level clipboard
         // operations that bypass the transformCopied handler).
@@ -960,9 +943,8 @@ export var createPlugin = function createPlugin(options, pmPluginFactoryParams,
         var syncBlockStore = pluginState === null || pluginState === void 0 ? void 0 : pluginState.syncBlockStore;
         var schema = state.schema;
         var isCopy = ctx.consumeCopyEvent();
-        var isSyncedBlockPatch13Enabled = fg('platform_synced_block_patch_13');
         var wasCut = ctx.consumeCutEvent();
-        var isCut = isSyncedBlockPatch13Enabled && wasCut;
+        var isCut = wasCut;
         var isPromptedFeedbackCut = Boolean(syncBlockStore && (options === null || options === void 0 ? void 0 : options.onGiveFeedback) && wasCut) && expValEqualsNoExposure('platform_editor_sync_block_activation', 'isEnabled', true);
         if (!syncBlockStore || !isCopy && !isCut && !isPromptedFeedbackCut) {
           return slice;
@@ -1000,7 +982,7 @@ export var createPlugin = function createPlugin(options, pmPluginFactoryParams,
             if (!sliceFullyContainsNode(slice, node)) {
               return node.content;
             }
-            if (isSyncedBlockPatch13Enabled && isCut) {
+            if (isCut) {
               return node;
             }
             var newResourceId = syncBlockStore.referenceManager.generateResourceIdForReference(node.attrs.resourceId);
@@ -1157,18 +1139,8 @@ export var createPlugin = function createPlugin(options, pmPluginFactoryParams,
         });
       }
 
-      // Legacy stash-and-rebase path. When platform_editor_blocks_patch_4 is on the
-      // delete is recomputed fresh from the live document on confirm, so there is
-      // no stashed transaction to rebase here. See EDITOR-7889.
-      if (!fg('platform_editor_blocks_patch_4')) {
-        trs.filter(function (tr) {
-          return tr.docChanged;
-        }).forEach(function (tr) {
-          if (confirmationTransactionRef.current) {
-            confirmationTransactionRef.current = rebaseTransaction(confirmationTransactionRef.current, tr, newState);
-          }
-        });
-      }
+      // Delete is recomputed fresh from the live document on confirm.
+      // See EDITOR-7889.
       var _iterator2 = _createForOfIteratorHelper(trs),
         _step2;
       try {

package/dist/esm/pm-plugins/utils/handle-bodied-sync-block-removal.js

@@ -38,67 +38,49 @@ export var handleBodiedSyncBlockRemoval = function handleBodiedSyncBlockRemoval(
   syncBlockStore.sourceManager.deleteSyncBlocksWithConfirmation(bodiedSyncBlockRemoved.map(function (node) {
     return node.attrs;
   }), deletionReason, function () {
-    var _api$core3;
-    if (fg('platform_editor_blocks_patch_4')) {
-      var _api$core2;
-      // Recompute the delete fresh from the live document instead of
-      // replaying a stashed-and-rebased transaction. The stash/rebase
-      // approach produced stale, schema-invalid transactions when the
-      // document changed shape (local edits or remote collab) while the
-      // confirmation modal was open — the dominant signature being
-      // "Invalid content for node bodiedSyncBlock: <>". See EDITOR-7889.
-      api === null || api === void 0 || (_api$core2 = api.core) === null || _api$core2 === void 0 || _api$core2.actions.execute(function (_ref2) {
-        var tr = _ref2.tr;
-        // EDITOR-8230: unsync must remove only the sync wrapper and keep the block's
-        // content inline, whereas delete removes the whole block. Before this branch,
-        // the confirm path always recomputed a full delete, so unsyncing a source block
-        // silently deleted its content. When the deletion was actually triggered by
-        // unsync, recompute an unwrap (replaceWith content) instead of a delete.
-        var isUnsync = fg('platform_editor_blocks_patch_6') && deletionReason === 'source-block-unsynced';
-        var recomputedTr = isUnsync ? recomputeUnsyncTransaction(tr, syncBlockStore.sourceManager.isSourceBlock, bodiedSyncBlockRemoved.map(function (node) {
-          return node.attrs;
-        })) : recomputeDeleteTransaction(tr, syncBlockStore.sourceManager.isSourceBlock, bodiedSyncBlockRemoved.map(function (node) {
-          return node.attrs;
-        }));
+    var _api$core2;
+    // Recompute the delete fresh from the live document instead of
+    // replaying a stashed-and-rebased transaction. The stash/rebase
+    // approach produced stale, schema-invalid transactions when the
+    // document changed shape (local edits or remote collab) while the
+    // confirmation modal was open — the dominant signature being
+    // "Invalid content for node bodiedSyncBlock: <>". See EDITOR-7889.
+    api === null || api === void 0 || (_api$core2 = api.core) === null || _api$core2 === void 0 || _api$core2.actions.execute(function (_ref2) {
+      var tr = _ref2.tr;
+      // EDITOR-8230: unsync must remove only the sync wrapper and keep the block's
+      // content inline, whereas delete removes the whole block. Before this branch,
+      // the confirm path always recomputed a full delete, so unsyncing a source block
+      // silently deleted its content. When the deletion was actually triggered by
+      // unsync, recompute an unwrap (replaceWith content) instead of a delete.
+      var isUnsync = fg('platform_editor_blocks_patch_6') && deletionReason === 'source-block-unsynced';
+      var recomputedTr = isUnsync ? recomputeUnsyncTransaction(tr, syncBlockStore.sourceManager.isSourceBlock, bodiedSyncBlockRemoved.map(function (node) {
+        return node.attrs;
+      })) : recomputeDeleteTransaction(tr, syncBlockStore.sourceManager.isSourceBlock, bodiedSyncBlockRemoved.map(function (node) {
+        return node.attrs;
+      }));
 
-        // The target node(s) no longer exist in the live document (e.g.
-        // a remote collaborator already removed them). There is nothing
-        // to delete locally, so return `null` to skip the dispatch
-        // entirely — dispatching the untouched transaction would be a
-        // no-op that still runs the plugin's filter/append hooks. The
-        // backend deletion has already been issued by the store manager.
-        if (!recomputedTr) {
-          return null;
-        }
-        recomputedTr.setMeta('isConfirmedSyncBlockDeletion', true);
-        feedbackCallbacks === null || feedbackCallbacks === void 0 || feedbackCallbacks.onDeleteTransaction(recomputedTr);
-        if (!recomputedTr.getMeta(pmHistoryPluginKey)) {
-          // bodiedSyncBlock deletion is expected to be permanent (cannot undo)
-          // For a normal deletion (not triggered by undo), remove it from history so that it cannot be undone
-          recomputedTr.setMeta('addToHistory', false);
-        }
-        return recomputedTr;
-      });
-      return;
-    }
-    var confirmationTransaction = confirmationTransactionRef.current;
-    if (!confirmationTransaction) {
-      return;
-    }
-    api === null || api === void 0 || (_api$core3 = api.core) === null || _api$core3 === void 0 || _api$core3.actions.execute(function () {
-      var trToDispatch = confirmationTransaction.setMeta('isConfirmedSyncBlockDeletion', true);
-      feedbackCallbacks === null || feedbackCallbacks === void 0 || feedbackCallbacks.onDeleteTransaction(trToDispatch);
-      if (!trToDispatch.getMeta(pmHistoryPluginKey)) {
+      // The target node(s) no longer exist in the live document (e.g.
+      // a remote collaborator already removed them). There is nothing
+      // to delete locally, so return `null` to skip the dispatch
+      // entirely — dispatching the untouched transaction would be a
+      // no-op that still runs the plugin's filter/append hooks. The
+      // backend deletion has already been issued by the store manager.
+      if (!recomputedTr) {
+        return null;
+      }
+      recomputedTr.setMeta('isConfirmedSyncBlockDeletion', true);
+      feedbackCallbacks === null || feedbackCallbacks === void 0 || feedbackCallbacks.onDeleteTransaction(recomputedTr);
+      if (!recomputedTr.getMeta(pmHistoryPluginKey)) {
         // bodiedSyncBlock deletion is expected to be permanent (cannot undo)
         // For a normal deletion (not triggered by undo), remove it from history so that it cannot be undone
-        trToDispatch.setMeta('addToHistory', false);
+        recomputedTr.setMeta('addToHistory', false);
       }
-      return trToDispatch;
+      return recomputedTr;
     });
   }, function (success) {
-    var _api$core4;
+    var _api$core3;
     feedbackCallbacks === null || feedbackCallbacks === void 0 || feedbackCallbacks.onDeleteCompleted(success);
-    api === null || api === void 0 || (_api$core4 = api.core) === null || _api$core4 === void 0 || _api$core4.actions.execute(function (_ref3) {
+    api === null || api === void 0 || (_api$core3 = api.core) === null || _api$core3 === void 0 || _api$core3.actions.execute(function (_ref3) {
       var tr = _ref3.tr;
       var newState;
       if (!success) {

package/dist/esm/syncedBlockPlugin.js

@@ -2,7 +2,6 @@ import React from 'react';
 import { syncBlock, bodiedSyncBlock } from '@atlaskit/adf-schema';
 import { useSharedPluginStateWithSelector } from '@atlaskit/editor-common/hooks';
 import { SyncBlockStoreManager } from '@atlaskit/editor-synced-block-provider';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { expValEquals } from '@atlaskit/tmp-editor-statsig/exp-val-equals';
 import { flushBodiedSyncBlocks as _flushBodiedSyncBlocks, flushSyncBlocks, discardUnpublishedSyncBlocks as _discardUnpublishedSyncBlocks } from './editor-actions';
 import { copySyncedBlockReferenceToClipboardEditorCommand, createSyncedBlock } from './editor-commands';
@@ -189,9 +188,7 @@ export var syncedBlockPlugin = function syncedBlockPlugin(_ref2) {
     // Destroy the SyncBlockStoreManager on editor unmount to cancel
     // pending timers, subscriptions, and in-flight fetches.
     destroy: function destroy() {
-      if (fg('platform_synced_block_patch_14')) {
-        syncBlockStore.destroy();
-      }
+      syncBlockStore.destroy();
     }
   };
 };

package/dist/esm/ui/CreateSyncedBlockButton.js

@@ -6,7 +6,6 @@ import { syncBlockMessages } from '@atlaskit/editor-common/messages';
 import { isOfflineMode } from '@atlaskit/editor-plugin-connectivity';
 import { ToolbarButton, ToolbarTooltip } from '@atlaskit/editor-toolbar';
 import BlockSyncedIcon from '@atlaskit/icon-lab/core/block-synced';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { canBeConvertedToSyncBlock } from '../pm-plugins/utils/utils';
 import { SYNCED_BLOCK_BUTTON_TEST_ID } from '../types';
 export var CreateSyncedBlockButton = function CreateSyncedBlockButton(_ref) {
@@ -28,45 +27,34 @@ export var CreateSyncedBlockButton = function CreateSyncedBlockButton(_ref) {
   var canInsertEmptyBlock = Boolean(selection === null || selection === void 0 ? void 0 : selection.empty);
   var isDisabled = Boolean(isOfflineMode(mode) || !canBeConverted && !canInsertEmptyBlock);
   var onClick = useCallback(function () {
-    var _api$core3, _api$core4;
-    if (fg('platform_editor_blocks_patch_4')) {
-      var _api$core;
-      // Insert the synced block and stop preserving the selection in a single
-      // transaction so the caret that insertSyncedBlock places inside the new
-      // block survives (EDITOR-7949). The toolbar is not normally opened via the
-      // block menu, but stopping preservation is defensive and keeps behaviour
-      // consistent with the block-menu create item should preservation ever be
-      // active (e.g. a block is selected via block-controls). Then re-focus so
-      // the caret is active, but only when the transaction was actually
-      // dispatched — otherwise a failed insertion would still steal DOM focus
-      // into the editor (mirrors CreateSyncedBlockDropdownItem).
-      var dispatched = api === null || api === void 0 || (_api$core = api.core) === null || _api$core === void 0 ? void 0 : _api$core.actions.execute(function (_ref2) {
-        var _api$blockControls;
-        var tr = _ref2.tr;
-        var result = api === null || api === void 0 ? void 0 : api.syncedBlock.commands.insertSyncedBlock(INPUT_METHOD.SYNCED_BLOCK_TB)({
-          tr: tr
-        });
-        if (!result) {
-          return null;
-        }
-        api === null || api === void 0 || (_api$blockControls = api.blockControls) === null || _api$blockControls === void 0 || (_api$blockControls = _api$blockControls.commands) === null || _api$blockControls === void 0 || _api$blockControls.stopPreservingSelection()({
-          tr: tr
-        });
-        return tr;
+    var _api$core;
+    // Insert the synced block and stop preserving the selection in a single
+    // transaction so the caret that insertSyncedBlock places inside the new
+    // block survives (EDITOR-7949). The toolbar is not normally opened via the
+    // block menu, but stopping preservation is defensive and keeps behaviour
+    // consistent with the block-menu create item should preservation ever be
+    // active (e.g. a block is selected via block-controls). Then re-focus so
+    // the caret is active, but only when the transaction was actually
+    // dispatched — otherwise a failed insertion would still steal DOM focus
+    // into the editor (mirrors CreateSyncedBlockDropdownItem).
+    var dispatched = api === null || api === void 0 || (_api$core = api.core) === null || _api$core === void 0 ? void 0 : _api$core.actions.execute(function (_ref2) {
+      var _api$blockControls;
+      var tr = _ref2.tr;
+      var result = api === null || api === void 0 ? void 0 : api.syncedBlock.commands.insertSyncedBlock(INPUT_METHOD.SYNCED_BLOCK_TB)({
+        tr: tr
       });
-      if (dispatched) {
-        var _api$core2;
-        api === null || api === void 0 || (_api$core2 = api.core) === null || _api$core2 === void 0 || _api$core2.actions.focus();
+      if (!result) {
+        return null;
       }
-      return;
-    }
-    api === null || api === void 0 || (_api$core3 = api.core) === null || _api$core3 === void 0 || _api$core3.actions.execute(function (_ref3) {
-      var tr = _ref3.tr;
-      return api === null || api === void 0 ? void 0 : api.syncedBlock.commands.insertSyncedBlock(INPUT_METHOD.SYNCED_BLOCK_TB)({
+      api === null || api === void 0 || (_api$blockControls = api.blockControls) === null || _api$blockControls === void 0 || (_api$blockControls = _api$blockControls.commands) === null || _api$blockControls === void 0 || _api$blockControls.stopPreservingSelection()({
         tr: tr
       });
+      return tr;
     });
-    api === null || api === void 0 || (_api$core4 = api.core) === null || _api$core4 === void 0 || _api$core4.actions.focus();
+    if (dispatched) {
+      var _api$core2;
+      api === null || api === void 0 || (_api$core2 = api.core) === null || _api$core2 === void 0 || _api$core2.actions.focus();
+    }
   }, [api]);
   var message = intl.formatMessage(syncBlockMessages.syncBlockLabel);
   return /*#__PURE__*/React.createElement(ToolbarTooltip, {

package/dist/esm/ui/CreateSyncedBlockDropdownItem.js

@@ -39,53 +39,42 @@ var CreateSyncedBlockDropdownItem = function CreateSyncedBlockDropdownItem(_ref2
     return null;
   }
   var onClick = function onClick() {
-    var _api$core3, _api$core4, _api$blockControls3;
-    if (fg('platform_editor_blocks_patch_4')) {
-      var _api$core;
-      // Insert the synced block, close the block menu, and stop preserving the
-      // selection — all in a single transaction, mirroring the block-menu delete
-      // item. Stopping preservation is required so the caret that
-      // insertSyncedBlock places inside the new block survives; otherwise
-      // block-controls restores a whole-node NodeSelection over it, hiding the
-      // caret (EDITOR-7949). Then re-focus the editor so the caret is active,
-      // but only when the transaction was actually dispatched — otherwise a
-      // failed insertion would still steal DOM focus into the editor.
-      var dispatched = api === null || api === void 0 || (_api$core = api.core) === null || _api$core === void 0 ? void 0 : _api$core.actions.execute(function (_ref3) {
-        var _api$blockControls, _api$blockControls2;
-        var tr = _ref3.tr;
-        // If the insertion fails, bail out without closing the menu or
-        // stopping selection preservation — otherwise we would dispatch an
-        // effectively-empty transaction and close the menu despite nothing
-        // having been inserted.
-        var result = api === null || api === void 0 ? void 0 : api.syncedBlock.commands.insertSyncedBlock(INPUT_METHOD.BLOCK_MENU)({
-          tr: tr
-        });
-        if (!result) {
-          return null;
-        }
-        api === null || api === void 0 || (_api$blockControls = api.blockControls) === null || _api$blockControls === void 0 || (_api$blockControls = _api$blockControls.commands) === null || _api$blockControls === void 0 || _api$blockControls.toggleBlockMenu({
-          closeMenu: true
-        })({
-          tr: tr
-        });
-        api === null || api === void 0 || (_api$blockControls2 = api.blockControls) === null || _api$blockControls2 === void 0 || (_api$blockControls2 = _api$blockControls2.commands) === null || _api$blockControls2 === void 0 || _api$blockControls2.stopPreservingSelection()({
-          tr: tr
-        });
-        return tr;
+    var _api$core;
+    // Insert the synced block, close the block menu, and stop preserving the
+    // selection — all in a single transaction, mirroring the block-menu delete
+    // item. Stopping preservation is required so the caret that
+    // insertSyncedBlock places inside the new block survives; otherwise
+    // block-controls restores a whole-node NodeSelection over it, hiding the
+    // caret (EDITOR-7949). Then re-focus the editor so the caret is active,
+    // but only when the transaction was actually dispatched — otherwise a
+    // failed insertion would still steal DOM focus into the editor.
+    var dispatched = api === null || api === void 0 || (_api$core = api.core) === null || _api$core === void 0 ? void 0 : _api$core.actions.execute(function (_ref3) {
+      var _api$blockControls, _api$blockControls2;
+      var tr = _ref3.tr;
+      // If the insertion fails, bail out without closing the menu or
+      // stopping selection preservation — otherwise we would dispatch an
+      // effectively-empty transaction and close the menu despite nothing
+      // having been inserted.
+      var result = api === null || api === void 0 ? void 0 : api.syncedBlock.commands.insertSyncedBlock(INPUT_METHOD.BLOCK_MENU)({
+        tr: tr
       });
-      if (dispatched) {
-        var _api$core2;
-        api === null || api === void 0 || (_api$core2 = api.core) === null || _api$core2 === void 0 || _api$core2.actions.focus();
+      if (!result) {
+        return null;
       }
-      return;
+      api === null || api === void 0 || (_api$blockControls = api.blockControls) === null || _api$blockControls === void 0 || (_api$blockControls = _api$blockControls.commands) === null || _api$blockControls === void 0 || _api$blockControls.toggleBlockMenu({
+        closeMenu: true
+      })({
+        tr: tr
+      });
+      api === null || api === void 0 || (_api$blockControls2 = api.blockControls) === null || _api$blockControls2 === void 0 || (_api$blockControls2 = _api$blockControls2.commands) === null || _api$blockControls2 === void 0 || _api$blockControls2.stopPreservingSelection()({
+        tr: tr
+      });
+      return tr;
+    });
+    if (dispatched) {
+      var _api$core2;
+      api === null || api === void 0 || (_api$core2 = api.core) === null || _api$core2 === void 0 || _api$core2.actions.focus();
     }
-
-    // Legacy behaviour: insert then close the block menu as two separate executes
-    // (batching them caused selection collisions — EDITOR-2751).
-    api === null || api === void 0 || (_api$core3 = api.core) === null || _api$core3 === void 0 || _api$core3.actions.execute(api === null || api === void 0 ? void 0 : api.syncedBlock.commands.insertSyncedBlock(INPUT_METHOD.BLOCK_MENU));
-    api === null || api === void 0 || (_api$core4 = api.core) === null || _api$core4 === void 0 || _api$core4.actions.execute(api === null || api === void 0 || (_api$blockControls3 = api.blockControls) === null || _api$blockControls3 === void 0 || (_api$blockControls3 = _api$blockControls3.commands) === null || _api$blockControls3 === void 0 ? void 0 : _api$blockControls3.toggleBlockMenu({
-      closeMenu: true
-    }));
   };
   var isOffline = isOfflineMode(mode);
   var lozenge = /*#__PURE__*/React.createElement(SyncedBlockNewLozenge, {
@@ -114,9 +103,9 @@ var CopySyncedBlockDropdownItem = function CopySyncedBlockDropdownItem(_ref4) {
     }),
     mode = _useSharedPluginState2.mode;
   var onClick = function onClick() {
-    var _api$core5, _api$core6, _api$blockControls4;
-    api === null || api === void 0 || (_api$core5 = api.core) === null || _api$core5 === void 0 || _api$core5.actions.execute(api === null || api === void 0 ? void 0 : api.syncedBlock.commands.copySyncedBlockReferenceToClipboard(INPUT_METHOD.BLOCK_MENU));
-    api === null || api === void 0 || (_api$core6 = api.core) === null || _api$core6 === void 0 || _api$core6.actions.execute(api === null || api === void 0 || (_api$blockControls4 = api.blockControls) === null || _api$blockControls4 === void 0 || (_api$blockControls4 = _api$blockControls4.commands) === null || _api$blockControls4 === void 0 ? void 0 : _api$blockControls4.toggleBlockMenu({
+    var _api$core3, _api$core4, _api$blockControls3;
+    api === null || api === void 0 || (_api$core3 = api.core) === null || _api$core3 === void 0 || _api$core3.actions.execute(api === null || api === void 0 ? void 0 : api.syncedBlock.commands.copySyncedBlockReferenceToClipboard(INPUT_METHOD.BLOCK_MENU));
+    api === null || api === void 0 || (_api$core4 = api.core) === null || _api$core4 === void 0 || _api$core4.actions.execute(api === null || api === void 0 || (_api$blockControls3 = api.blockControls) === null || _api$blockControls3 === void 0 || (_api$blockControls3 = _api$blockControls3.commands) === null || _api$blockControls3 === void 0 ? void 0 : _api$blockControls3.toggleBlockMenu({
       closeMenu: true
     }));
   };